@reviewflow/sdk 6.23.0 → 6.24.0

package/build/index-browser.es.js

@@ -2307,6 +2307,8 @@ const createGitHubTransport = (send) => ({
   }
 });
 
+const pluralOf = (count, word, plural = `${word}s`) => `${String(count)} ${count === 1 ? word : plural}`;
+
 const prStatuses = [
   "approved",
   "changesRequested",
@@ -3163,7 +3165,6 @@ const maxFailedCheckNames = 9;
 const bucketsSilentOnPassedChecks = /* @__PURE__ */ new Set(["changes-requested", "drafts"]);
 const reRequestedBucket = "re-requested-reviews";
 const requestingAttentionGroup = "requesting-attention";
-const counted = (count, one, many) => `${String(count)} ${count === 1 ? one : many}`;
 const people = (logins, viewer) => logins.map((login) => login === viewer ? "you" : `@${login}`).join(", ");
 const failedChecksText = ({
   checksConclusion,
@@ -3191,7 +3192,7 @@ const checksProgressText = ({
   checksRunning
 }, bucketKey) => {
   if (checksRunning > 0) {
-    return `${counted(checksRunning, "check", "checks")} running`;
+    return `${pluralOf(checksRunning, "check")} running`;
   }
   return checksConclusion === "passed" && !bucketsSilentOnPassedChecks.has(bucketKey) ? "checks passed" : null;
 };
@@ -3216,7 +3217,7 @@ const reviewflowPrProgress = (pr, context) => {
   const parts = [
     checksProgressText(pr, context.bucketKey),
     pr.approvedBy.length === 0 ? null : `approved by ${people(pr.approvedBy, context.viewer)}`,
-    pr.unresolvedThreads ? counted(pr.unresolvedThreads, "unresolved thread", "unresolved threads") : null,
+    pr.unresolvedThreads ? pluralOf(pr.unresolvedThreads, "unresolved thread") : null,
     awaitedText(pr, context)
   ].filter((part) => part !== null);
   return parts.length === 0 ? null : parts.join(" \xB7 ");
@@ -3243,7 +3244,7 @@ const formatPrChanges = ({
   changedFiles,
   additions,
   deletions
-}) => `${counted(changedFiles, "file", "files")} +${String(additions)} \u2212${String(deletions)}`;
+}) => `${pluralOf(changedFiles, "file")} +${String(additions)} \u2212${String(deletions)}`;
 
 const lingerMs = 5 * 6e4;
 const boardReadDelayMs = 3e3;
@@ -3505,5 +3506,5 @@ const insertCheckoutEntry = (source, ownerRepo) => {
   return { source: text, cursor: value };
 };
 
-export { accountOfOwnerRepo, applyPullRequestEdit, applyStreamEvent, applySuggestionsToText, assembleBoard, boardReadDelayMs, buildPrQuery, changedPullRequestParts, checkOutcomes, checksFromReviewflow, coAuthorTrailers, commitSuggestions, createGitHubTransport, createReviewflowClient, createReviewflowFollower, diffLines, diffPaths, diffThemeNames, diffThemeOptions, diffThemes, editPullRequest, fail, fetchCompareDiff, fetchPullRequestCandidates, fetchPullRequestHead, fetchPullRequestPart, fetchPullRequestRevisions, formatPrChanges, formatRelativeDate, insertCheckoutEntry, interpretGraphql, isAttachablePath, isDiffTheme, isDiffWhitespace, isPullRequestOnBoard, isPullRequestPart, isPullRequestReactionContent, isReviewflowFailure, lingerMs, maxExtraPagesPerBucket, maxFailedCheckNames, maxLookupTargets, maxRetryMs, parsePrQueryResponse, parsePullRequestCandidates, parsePullRequestChecks, parsePullRequestCommits, parsePullRequestDetail, parsePullRequestEdit, parsePullRequestFiles, parsePullRequestRevisions, parsePullRequestTimeline, parseSuggestions, partFingerprint, planSuggestions, prFields, prHasConflicts, prNumberOf, prStatus, prStatusLabels, prStatuses, prUrl, pullRequestCandidatesQuery, pullRequestChecksQuery, pullRequestCommitsQuery, pullRequestDetailQuery, pullRequestEditMovesBoard, pullRequestFilesQuery, pullRequestHeadQuery, pullRequestParts, pullRequestPartsStaleAfter, pullRequestReactionContents, pullRequestRevisionsQuery, pullRequestTimelineQuery, readAccountBoards, readPrDetail, readReviewflowStream, reconnectDelayMs, responseFailure, restErrorMessage, reviewflowErrorCode, reviewflowPrChangesRequested, reviewflowPrFailure, reviewflowPrFlowDate, reviewflowPrOwners, reviewflowPrProgress, reviewflowPrStatus, reviewflowReviewDecisions, splitOwnerRepo, streamClosedWithinDays, streamIdleTimeoutMs, subjectsOf, suggestionCommitRefusal, suggestionOriginalLines, suggestionRangesOverlap, toBoardPr, toFetchedPr, withAccountBoard, withFileViewed };
+export { accountOfOwnerRepo, applyPullRequestEdit, applyStreamEvent, applySuggestionsToText, assembleBoard, boardReadDelayMs, buildPrQuery, changedPullRequestParts, checkOutcomes, checksFromReviewflow, coAuthorTrailers, commitSuggestions, createGitHubTransport, createReviewflowClient, createReviewflowFollower, diffLines, diffPaths, diffThemeNames, diffThemeOptions, diffThemes, editPullRequest, fail, fetchCompareDiff, fetchPullRequestCandidates, fetchPullRequestHead, fetchPullRequestPart, fetchPullRequestRevisions, formatPrChanges, formatRelativeDate, insertCheckoutEntry, interpretGraphql, isAttachablePath, isDiffTheme, isDiffWhitespace, isPullRequestOnBoard, isPullRequestPart, isPullRequestReactionContent, isReviewflowFailure, lingerMs, maxExtraPagesPerBucket, maxFailedCheckNames, maxLookupTargets, maxRetryMs, parsePrQueryResponse, parsePullRequestCandidates, parsePullRequestChecks, parsePullRequestCommits, parsePullRequestDetail, parsePullRequestEdit, parsePullRequestFiles, parsePullRequestRevisions, parsePullRequestTimeline, parseSuggestions, partFingerprint, planSuggestions, pluralOf, prFields, prHasConflicts, prNumberOf, prStatus, prStatusLabels, prStatuses, prUrl, pullRequestCandidatesQuery, pullRequestChecksQuery, pullRequestCommitsQuery, pullRequestDetailQuery, pullRequestEditMovesBoard, pullRequestFilesQuery, pullRequestHeadQuery, pullRequestParts, pullRequestPartsStaleAfter, pullRequestReactionContents, pullRequestRevisionsQuery, pullRequestTimelineQuery, readAccountBoards, readPrDetail, readReviewflowStream, reconnectDelayMs, responseFailure, restErrorMessage, reviewflowErrorCode, reviewflowPrChangesRequested, reviewflowPrFailure, reviewflowPrFlowDate, reviewflowPrOwners, reviewflowPrProgress, reviewflowPrStatus, reviewflowReviewDecisions, splitOwnerRepo, streamClosedWithinDays, streamIdleTimeoutMs, subjectsOf, suggestionCommitRefusal, suggestionOriginalLines, suggestionRangesOverlap, toBoardPr, toFetchedPr, withAccountBoard, withFileViewed };
 //# sourceMappingURL=index-browser.es.js.map

package/build/index-node.mjs

@@ -2307,6 +2307,8 @@ const createGitHubTransport = (send) => ({
   }
 });
 
+const pluralOf = (count, word, plural = `${word}s`) => `${String(count)} ${count === 1 ? word : plural}`;
+
 const prStatuses = [
   "approved",
   "changesRequested",
@@ -3163,7 +3165,6 @@ const maxFailedCheckNames = 9;
 const bucketsSilentOnPassedChecks = /* @__PURE__ */ new Set(["changes-requested", "drafts"]);
 const reRequestedBucket = "re-requested-reviews";
 const requestingAttentionGroup = "requesting-attention";
-const counted = (count, one, many) => `${String(count)} ${count === 1 ? one : many}`;
 const people = (logins, viewer) => logins.map((login) => login === viewer ? "you" : `@${login}`).join(", ");
 const failedChecksText = ({
   checksConclusion,
@@ -3191,7 +3192,7 @@ const checksProgressText = ({
   checksRunning
 }, bucketKey) => {
   if (checksRunning > 0) {
-    return `${counted(checksRunning, "check", "checks")} running`;
+    return `${pluralOf(checksRunning, "check")} running`;
   }
   return checksConclusion === "passed" && !bucketsSilentOnPassedChecks.has(bucketKey) ? "checks passed" : null;
 };
@@ -3216,7 +3217,7 @@ const reviewflowPrProgress = (pr, context) => {
   const parts = [
     checksProgressText(pr, context.bucketKey),
     pr.approvedBy.length === 0 ? null : `approved by ${people(pr.approvedBy, context.viewer)}`,
-    pr.unresolvedThreads ? counted(pr.unresolvedThreads, "unresolved thread", "unresolved threads") : null,
+    pr.unresolvedThreads ? pluralOf(pr.unresolvedThreads, "unresolved thread") : null,
     awaitedText(pr, context)
   ].filter((part) => part !== null);
   return parts.length === 0 ? null : parts.join(" \xB7 ");
@@ -3243,7 +3244,7 @@ const formatPrChanges = ({
   changedFiles,
   additions,
   deletions
-}) => `${counted(changedFiles, "file", "files")} +${String(additions)} \u2212${String(deletions)}`;
+}) => `${pluralOf(changedFiles, "file")} +${String(additions)} \u2212${String(deletions)}`;
 
 const lingerMs = 5 * 6e4;
 const boardReadDelayMs = 3e3;
@@ -3505,5 +3506,5 @@ const insertCheckoutEntry = (source, ownerRepo) => {
   return { source: text, cursor: value };
 };
 
-export { accountOfOwnerRepo, applyPullRequestEdit, applyStreamEvent, applySuggestionsToText, assembleBoard, boardReadDelayMs, buildPrQuery, changedPullRequestParts, checkOutcomes, checksFromReviewflow, coAuthorTrailers, commitSuggestions, createGitHubTransport, createReviewflowClient, createReviewflowFollower, diffLines, diffPaths, diffThemeNames, diffThemeOptions, diffThemes, editPullRequest, fail, fetchCompareDiff, fetchPullRequestCandidates, fetchPullRequestHead, fetchPullRequestPart, fetchPullRequestRevisions, formatPrChanges, formatRelativeDate, insertCheckoutEntry, interpretGraphql, isAttachablePath, isDiffTheme, isDiffWhitespace, isPullRequestOnBoard, isPullRequestPart, isPullRequestReactionContent, isReviewflowFailure, lingerMs, maxExtraPagesPerBucket, maxFailedCheckNames, maxLookupTargets, maxRetryMs, parsePrQueryResponse, parsePullRequestCandidates, parsePullRequestChecks, parsePullRequestCommits, parsePullRequestDetail, parsePullRequestEdit, parsePullRequestFiles, parsePullRequestRevisions, parsePullRequestTimeline, parseSuggestions, partFingerprint, planSuggestions, prFields, prHasConflicts, prNumberOf, prStatus, prStatusLabels, prStatuses, prUrl, pullRequestCandidatesQuery, pullRequestChecksQuery, pullRequestCommitsQuery, pullRequestDetailQuery, pullRequestEditMovesBoard, pullRequestFilesQuery, pullRequestHeadQuery, pullRequestParts, pullRequestPartsStaleAfter, pullRequestReactionContents, pullRequestRevisionsQuery, pullRequestTimelineQuery, readAccountBoards, readPrDetail, readReviewflowStream, reconnectDelayMs, responseFailure, restErrorMessage, reviewflowErrorCode, reviewflowPrChangesRequested, reviewflowPrFailure, reviewflowPrFlowDate, reviewflowPrOwners, reviewflowPrProgress, reviewflowPrStatus, reviewflowReviewDecisions, splitOwnerRepo, streamClosedWithinDays, streamIdleTimeoutMs, subjectsOf, suggestionCommitRefusal, suggestionOriginalLines, suggestionRangesOverlap, toBoardPr, toFetchedPr, withAccountBoard, withFileViewed };
+export { accountOfOwnerRepo, applyPullRequestEdit, applyStreamEvent, applySuggestionsToText, assembleBoard, boardReadDelayMs, buildPrQuery, changedPullRequestParts, checkOutcomes, checksFromReviewflow, coAuthorTrailers, commitSuggestions, createGitHubTransport, createReviewflowClient, createReviewflowFollower, diffLines, diffPaths, diffThemeNames, diffThemeOptions, diffThemes, editPullRequest, fail, fetchCompareDiff, fetchPullRequestCandidates, fetchPullRequestHead, fetchPullRequestPart, fetchPullRequestRevisions, formatPrChanges, formatRelativeDate, insertCheckoutEntry, interpretGraphql, isAttachablePath, isDiffTheme, isDiffWhitespace, isPullRequestOnBoard, isPullRequestPart, isPullRequestReactionContent, isReviewflowFailure, lingerMs, maxExtraPagesPerBucket, maxFailedCheckNames, maxLookupTargets, maxRetryMs, parsePrQueryResponse, parsePullRequestCandidates, parsePullRequestChecks, parsePullRequestCommits, parsePullRequestDetail, parsePullRequestEdit, parsePullRequestFiles, parsePullRequestRevisions, parsePullRequestTimeline, parseSuggestions, partFingerprint, planSuggestions, pluralOf, prFields, prHasConflicts, prNumberOf, prStatus, prStatusLabels, prStatuses, prUrl, pullRequestCandidatesQuery, pullRequestChecksQuery, pullRequestCommitsQuery, pullRequestDetailQuery, pullRequestEditMovesBoard, pullRequestFilesQuery, pullRequestHeadQuery, pullRequestParts, pullRequestPartsStaleAfter, pullRequestReactionContents, pullRequestRevisionsQuery, pullRequestTimelineQuery, readAccountBoards, readPrDetail, readReviewflowStream, reconnectDelayMs, responseFailure, restErrorMessage, reviewflowErrorCode, reviewflowPrChangesRequested, reviewflowPrFailure, reviewflowPrFlowDate, reviewflowPrOwners, reviewflowPrProgress, reviewflowPrStatus, reviewflowReviewDecisions, splitOwnerRepo, streamClosedWithinDays, streamIdleTimeoutMs, subjectsOf, suggestionCommitRefusal, suggestionOriginalLines, suggestionRangesOverlap, toBoardPr, toFetchedPr, withAccountBoard, withFileViewed };
 //# sourceMappingURL=index-node.mjs.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reviewflow/sdk",
-  "version": "6.23.0",
+  "version": "6.24.0",
   "description": "pull request model shared by the reviewflow app server and its ui: diff and api types, pure utils, the GitHub layer behind an injected transport, and (node only) reading a diff from a local checkout (@reviewflow/sdk/git) and editing a text file in place (@reviewflow/sdk/files)",
   "keywords": [
     "code-review",

package/src/index.ts

@@ -20,6 +20,7 @@ export type * from "./api/pullRequest.ts";
 export type * from "./types/entities.ts";
 export type * from "./types/ids.ts";
 export type * from "./types/reviewflow.ts";
+export * from "./utils/pluralOf.ts";
 export * from "./utils/prStatus.ts";
 export * from "./utils/pullRequestEdit.ts";
 export * from "./utils/pullRequestPart.ts";

package/src/reviewflow/boardFacts.ts

@@ -1,4 +1,5 @@
 import type { ReviewflowPrChanges } from "../types/reviewflow.ts";
+import { pluralOf } from "../utils/pluralOf.ts";
 import type { FormatRelativeDateOptions } from "../utils/relativeDate.ts";
 import { formatRelativeDate } from "../utils/relativeDate.ts";
 import type { ReviewflowBoardPr } from "./board.ts";
@@ -26,9 +27,6 @@ const reRequestedBucket = "re-requested-reviews";
 /** someone else's pull request requests a review, the operator's awaits one */
 const requestingAttentionGroup = "requesting-attention";
 
-const counted = (count: number, one: string, many: string): string =>
-  `${String(count)} ${count === 1 ? one : many}`;
-
 const people = (logins: string[], viewer: string | null): string =>
   logins.map((login) => (login === viewer ? "you" : `@${login}`)).join(", ");
 
@@ -81,7 +79,7 @@ const checksProgressText = (
   bucketKey: string,
 ): string | null => {
   if (checksRunning > 0) {
-    return `${counted(checksRunning, "check", "checks")} running`;
+    return `${pluralOf(checksRunning, "check")} running`;
   }
   return checksConclusion === "passed" &&
     !bucketsSilentOnPassedChecks.has(bucketKey)
@@ -133,7 +131,7 @@ export const reviewflowPrProgress = (
       ? null
       : `approved by ${people(pr.approvedBy, context.viewer)}`,
     pr.unresolvedThreads
-      ? counted(pr.unresolvedThreads, "unresolved thread", "unresolved threads")
+      ? pluralOf(pr.unresolvedThreads, "unresolved thread")
       : null,
     awaitedText(pr, context),
   ].filter((part) => part !== null);
@@ -181,4 +179,4 @@ export const formatPrChanges = ({
   additions,
   deletions,
 }: ReviewflowPrChanges): string =>
-  `${counted(changedFiles, "file", "files")} +${String(additions)} −${String(deletions)}`;
+  `${pluralOf(changedFiles, "file")} +${String(additions)} −${String(deletions)}`;

package/src/utils/pluralOf.ts

@@ -0,0 +1,6 @@
+/** "1 file", "2 files"; `plural` defaults to `word` + "s". */
+export const pluralOf = (
+  count: number,
+  word: string,
+  plural = `${word}s`,
+): string => `${String(count)} ${count === 1 ? word : plural}`;