@usepipr/runtime 0.4.3 → 0.5.0

package/dist/internal/pipr-result.mjs

@@ -0,0 +1,126 @@
+import { t as parseGeneratedMainCommentEnvelope } from "../main-comment-envelope-DFirHYhW.mjs";
+import { parsePiprResult } from "@usepipr/sdk";
+//#region src/internal/pipr-result.ts
+const genericFailureMessage = "Pipr failed; see the Action log for details.";
+const publicationFailureMessage = "Pipr could not complete publication; see the Action log for details.";
+function toPiprResult(input) {
+	return parsePiprResult(input.source === "local" ? localResult(input.result) : hostResult(input.result));
+}
+function toPiprErrorResult(_error) {
+	const result = parsePiprResult({
+		formatVersion: 2,
+		kind: "error",
+		message: genericFailureMessage
+	});
+	if (result.kind !== "error") throw new Error("Pipr error result schema returned an unexpected result kind");
+	return result;
+}
+function localResult(result) {
+	if (result.kind === "skipped") return {
+		formatVersion: 2,
+		kind: "skipped",
+		reason: result.skipReason
+	};
+	return reviewResult(result, { state: "disabled" });
+}
+function hostResult(result) {
+	switch (result.kind) {
+		case "ignored": return {
+			formatVersion: 2,
+			kind: "ignored",
+			reason: result.reason
+		};
+		case "dry-run": return {
+			formatVersion: 2,
+			kind: "dry-run"
+		};
+		case "command-help": return {
+			formatVersion: 2,
+			kind: "command-help",
+			reason: result.reason,
+			mainComment: result.body
+		};
+		case "command-response": return {
+			formatVersion: 2,
+			kind: "command-response",
+			run: publicRunSummary(result.run),
+			mainComment: result.response.body,
+			publication: {
+				state: "completed",
+				action: result.publication.action
+			}
+		};
+		case "verifier": return {
+			formatVersion: 2,
+			kind: "verifier",
+			run: publicRunSummary(result.run),
+			publication: {
+				state: "completed",
+				inlineResolutionErrorCount: result.errors.length
+			}
+		};
+		case "review": return reviewResult(result.review, {
+			state: "completed",
+			mainComment: { action: result.publication.mainComment.action },
+			inlineComments: result.publication.inlineComments,
+			inlinePublicationErrorCount: result.publication.metadata.inlinePublicationErrors.length,
+			inlineResolutionErrorCount: result.publication.metadata.inlineResolutionErrors.length
+		});
+		case "publication-error": return publicationErrorResult(result.publication);
+	}
+}
+function reviewResult(review, publication) {
+	return {
+		formatVersion: 2,
+		kind: "review",
+		run: publicRunSummary(review.run),
+		mainComment: stripPiprMainCommentMarkers(review.mainComment),
+		inlineFindings: review.inlineCommentDrafts.map((draft) => draft.finding),
+		droppedFindings: review.validated.droppedFindings,
+		taskChecks: review.taskChecks,
+		repairAttempted: review.repairAttempted,
+		publication
+	};
+}
+function publicRunSummary(run) {
+	const bounded = (value) => value.slice(0, 200);
+	return {
+		...run,
+		id: bounded(run.id),
+		baseSha: bounded(run.baseSha),
+		headSha: bounded(run.headSha),
+		tasks: run.tasks.slice(0, 200).map(bounded),
+		models: run.models.slice(0, 20).map(bounded)
+	};
+}
+function publicationErrorResult(publication) {
+	return {
+		formatVersion: 2,
+		kind: "publication-error",
+		message: publicationFailureMessage,
+		...publication ? { publication: {
+			inlineComments: publication.inlineComments,
+			inlinePublicationErrorCount: publication.metadata.inlinePublicationErrors.length,
+			inlineResolutionErrorCount: publication.metadata.inlineResolutionErrors.length
+		} } : {}
+	};
+}
+function stripPiprMainCommentMarkers(mainComment) {
+	const lines = mainComment.split("\n");
+	const envelope = parseGeneratedMainCommentEnvelope(lines);
+	const generatedMarkerIndexes = /* @__PURE__ */ new Set([
+		envelope.mainMarkerIndex,
+		envelope.headerMarkerIndex,
+		envelope.statsMarkerIndex,
+		envelope.statsRange?.start ?? -1,
+		envelope.statsRange?.end ?? -1,
+		lines[envelope.footerIndex] === "<!-- pipr:footer:hidden -->" ? envelope.footerIndex : -1
+	]);
+	const visibleLines = lines.filter((_line, index) => !generatedMarkerIndexes.has(index));
+	const firstContentLine = visibleLines.findIndex((line) => line !== "");
+	return firstContentLine === -1 ? "" : visibleLines.slice(firstContentLine).join("\n");
+}
+//#endregion
+export { stripPiprMainCommentMarkers, toPiprErrorResult, toPiprResult };
+
+//# sourceMappingURL=pipr-result.mjs.map

package/dist/internal/pipr-result.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"pipr-result.mjs","names":[],"sources":["../../src/internal/pipr-result.ts"],"sourcesContent":["import { type PiprResult, type PiprRunSummary, parsePiprResult } from \"@usepipr/sdk\";\nimport type { HostRunCommandResult, LocalReviewCommandResult } from \"../host-run/types.js\";\nimport { mainCommentFooterHiddenMarker } from \"../review/comment-branding.js\";\nimport { parseGeneratedMainCommentEnvelope } from \"../review/main-comment-envelope.js\";\nimport type { PublicationError } from \"../review/publication-result.js\";\n\nconst genericFailureMessage = \"Pipr failed; see the Action log for details.\";\nconst publicationFailureMessage =\n  \"Pipr could not complete publication; see the Action log for details.\";\n\ntype ResultInput =\n  | { source: \"host\"; result: HostRunCommandResult }\n  | { source: \"local\"; result: LocalReviewCommandResult }\n  | {\n      source: \"host\";\n      result: { kind: \"publication-error\"; publication: PublicationError[\"result\"] };\n    };\n\nexport function toPiprResult(input: ResultInput): PiprResult {\n  return parsePiprResult(\n    input.source === \"local\" ? localResult(input.result) : hostResult(input.result),\n  );\n}\n\nexport function toPiprErrorResult(_error: unknown): Extract<PiprResult, { kind: \"error\" }> {\n  const result = parsePiprResult({\n    formatVersion: 2,\n    kind: \"error\",\n    message: genericFailureMessage,\n  });\n  if (result.kind !== \"error\") {\n    throw new Error(\"Pipr error result schema returned an unexpected result kind\");\n  }\n  return result;\n}\n\nfunction localResult(result: LocalReviewCommandResult): PiprResult {\n  if (result.kind === \"skipped\") {\n    return { formatVersion: 2, kind: \"skipped\", reason: result.skipReason };\n  }\n  return reviewResult(result, { state: \"disabled\" });\n}\n\nfunction hostResult(\n  result:\n    | HostRunCommandResult\n    | { kind: \"publication-error\"; publication: PublicationError[\"result\"] },\n): PiprResult {\n  switch (result.kind) {\n    case \"ignored\":\n      return { formatVersion: 2, kind: \"ignored\", reason: result.reason };\n    case \"dry-run\":\n      return { formatVersion: 2, kind: \"dry-run\" };\n    case \"command-help\":\n      return {\n        formatVersion: 2,\n        kind: \"command-help\",\n        reason: result.reason,\n        mainComment: result.body,\n      };\n    case \"command-response\":\n      return {\n        formatVersion: 2,\n        kind: \"command-response\",\n        run: publicRunSummary(result.run),\n        mainComment: result.response.body,\n        publication: { state: \"completed\", action: result.publication.action },\n      };\n    case \"verifier\":\n      return {\n        formatVersion: 2,\n        kind: \"verifier\",\n        run: publicRunSummary(result.run),\n        publication: { state: \"completed\", inlineResolutionErrorCount: result.errors.length },\n      };\n    case \"review\":\n      return reviewResult(result.review, {\n        state: \"completed\",\n        mainComment: { action: result.publication.mainComment.action },\n        inlineComments: result.publication.inlineComments,\n        inlinePublicationErrorCount: result.publication.metadata.inlinePublicationErrors.length,\n        inlineResolutionErrorCount: result.publication.metadata.inlineResolutionErrors.length,\n      });\n    case \"publication-error\":\n      return publicationErrorResult(result.publication);\n  }\n}\n\nfunction reviewResult(\n  review: Extract<LocalReviewCommandResult, { kind: \"review\" }>,\n  publication: Extract<PiprResult, { kind: \"review\" }>[\"publication\"],\n): PiprResult {\n  return {\n    formatVersion: 2,\n    kind: \"review\",\n    run: publicRunSummary(review.run),\n    mainComment: stripPiprMainCommentMarkers(review.mainComment),\n    inlineFindings: review.inlineCommentDrafts.map((draft) => draft.finding),\n    droppedFindings: review.validated.droppedFindings,\n    taskChecks: review.taskChecks,\n    repairAttempted: review.repairAttempted,\n    publication,\n  };\n}\n\nfunction publicRunSummary(run: PiprRunSummary): PiprRunSummary {\n  const bounded = (value: string) => value.slice(0, 200);\n  return {\n    ...run,\n    id: bounded(run.id),\n    baseSha: bounded(run.baseSha),\n    headSha: bounded(run.headSha),\n    tasks: run.tasks.slice(0, 200).map(bounded),\n    models: run.models.slice(0, 20).map(bounded),\n  };\n}\n\nfunction publicationErrorResult(\n  publication: PublicationError[\"result\"],\n): Extract<PiprResult, { kind: \"publication-error\" }> {\n  return {\n    formatVersion: 2,\n    kind: \"publication-error\",\n    message: publicationFailureMessage,\n    ...(publication\n      ? {\n          publication: {\n            inlineComments: publication.inlineComments,\n            inlinePublicationErrorCount: publication.metadata.inlinePublicationErrors.length,\n            inlineResolutionErrorCount: publication.metadata.inlineResolutionErrors.length,\n          },\n        }\n      : {}),\n  };\n}\n\nexport function stripPiprMainCommentMarkers(mainComment: string): string {\n  const lines = mainComment.split(\"\\n\");\n  const envelope = parseGeneratedMainCommentEnvelope(lines);\n  const generatedMarkerIndexes = new Set([\n    envelope.mainMarkerIndex,\n    envelope.headerMarkerIndex,\n    envelope.statsMarkerIndex,\n    envelope.statsRange?.start ?? -1,\n    envelope.statsRange?.end ?? -1,\n    lines[envelope.footerIndex] === mainCommentFooterHiddenMarker ? envelope.footerIndex : -1,\n  ]);\n  const visibleLines = lines.filter((_line, index) => !generatedMarkerIndexes.has(index));\n  const firstContentLine = visibleLines.findIndex((line) => line !== \"\");\n  return firstContentLine === -1 ? \"\" : visibleLines.slice(firstContentLine).join(\"\\n\");\n}\n"],"mappings":";;;AAMA,MAAM,wBAAwB;AAC9B,MAAM,4BACJ;AAUF,SAAgB,aAAa,OAAgC;CAC3D,OAAO,gBACL,MAAM,WAAW,UAAU,YAAY,MAAM,MAAM,IAAI,WAAW,MAAM,MAAM,CAChF;AACF;AAEA,SAAgB,kBAAkB,QAAyD;CACzF,MAAM,SAAS,gBAAgB;EAC7B,eAAe;EACf,MAAM;EACN,SAAS;CACX,CAAC;CACD,IAAI,OAAO,SAAS,SAClB,MAAM,IAAI,MAAM,6DAA6D;CAE/E,OAAO;AACT;AAEA,SAAS,YAAY,QAA8C;CACjE,IAAI,OAAO,SAAS,WAClB,OAAO;EAAE,eAAe;EAAG,MAAM;EAAW,QAAQ,OAAO;CAAW;CAExE,OAAO,aAAa,QAAQ,EAAE,OAAO,WAAW,CAAC;AACnD;AAEA,SAAS,WACP,QAGY;CACZ,QAAQ,OAAO,MAAf;EACE,KAAK,WACH,OAAO;GAAE,eAAe;GAAG,MAAM;GAAW,QAAQ,OAAO;EAAO;EACpE,KAAK,WACH,OAAO;GAAE,eAAe;GAAG,MAAM;EAAU;EAC7C,KAAK,gBACH,OAAO;GACL,eAAe;GACf,MAAM;GACN,QAAQ,OAAO;GACf,aAAa,OAAO;EACtB;EACF,KAAK,oBACH,OAAO;GACL,eAAe;GACf,MAAM;GACN,KAAK,iBAAiB,OAAO,GAAG;GAChC,aAAa,OAAO,SAAS;GAC7B,aAAa;IAAE,OAAO;IAAa,QAAQ,OAAO,YAAY;GAAO;EACvE;EACF,KAAK,YACH,OAAO;GACL,eAAe;GACf,MAAM;GACN,KAAK,iBAAiB,OAAO,GAAG;GAChC,aAAa;IAAE,OAAO;IAAa,4BAA4B,OAAO,OAAO;GAAO;EACtF;EACF,KAAK,UACH,OAAO,aAAa,OAAO,QAAQ;GACjC,OAAO;GACP,aAAa,EAAE,QAAQ,OAAO,YAAY,YAAY,OAAO;GAC7D,gBAAgB,OAAO,YAAY;GACnC,6BAA6B,OAAO,YAAY,SAAS,wBAAwB;GACjF,4BAA4B,OAAO,YAAY,SAAS,uBAAuB;EACjF,CAAC;EACH,KAAK,qBACH,OAAO,uBAAuB,OAAO,WAAW;CACpD;AACF;AAEA,SAAS,aACP,QACA,aACY;CACZ,OAAO;EACL,eAAe;EACf,MAAM;EACN,KAAK,iBAAiB,OAAO,GAAG;EAChC,aAAa,4BAA4B,OAAO,WAAW;EAC3D,gBAAgB,OAAO,oBAAoB,KAAK,UAAU,MAAM,OAAO;EACvE,iBAAiB,OAAO,UAAU;EAClC,YAAY,OAAO;EACnB,iBAAiB,OAAO;EACxB;CACF;AACF;AAEA,SAAS,iBAAiB,KAAqC;CAC7D,MAAM,WAAW,UAAkB,MAAM,MAAM,GAAG,GAAG;CACrD,OAAO;EACL,GAAG;EACH,IAAI,QAAQ,IAAI,EAAE;EAClB,SAAS,QAAQ,IAAI,OAAO;EAC5B,SAAS,QAAQ,IAAI,OAAO;EAC5B,OAAO,IAAI,MAAM,MAAM,GAAG,GAAG,CAAC,CAAC,IAAI,OAAO;EAC1C,QAAQ,IAAI,OAAO,MAAM,GAAG,EAAE,CAAC,CAAC,IAAI,OAAO;CAC7C;AACF;AAEA,SAAS,uBACP,aACoD;CACpD,OAAO;EACL,eAAe;EACf,MAAM;EACN,SAAS;EACT,GAAI,cACA,EACE,aAAa;GACX,gBAAgB,YAAY;GAC5B,6BAA6B,YAAY,SAAS,wBAAwB;GAC1E,4BAA4B,YAAY,SAAS,uBAAuB;EAC1E,EACF,IACA,CAAC;CACP;AACF;AAEA,SAAgB,4BAA4B,aAA6B;CACvE,MAAM,QAAQ,YAAY,MAAM,IAAI;CACpC,MAAM,WAAW,kCAAkC,KAAK;CACxD,MAAM,yCAAyB,IAAI,IAAI;EACrC,SAAS;EACT,SAAS;EACT,SAAS;EACT,SAAS,YAAY,SAAS;EAC9B,SAAS,YAAY,OAAO;EAC5B,MAAM,SAAS,iBAAA,gCAAiD,SAAS,cAAc;CACzF,CAAC;CACD,MAAM,eAAe,MAAM,QAAQ,OAAO,UAAU,CAAC,uBAAuB,IAAI,KAAK,CAAC;CACtF,MAAM,mBAAmB,aAAa,WAAW,SAAS,SAAS,EAAE;CACrE,OAAO,qBAAqB,KAAK,KAAK,aAAa,MAAM,gBAAgB,CAAC,CAAC,KAAK,IAAI;AACtF"}

package/dist/internal/testing.d.mts

@@ -1,5 +1,5 @@
-import { r as runHostRunCommandWithDependencies } from "../commands-DA0Ij4Di.mjs";
-import { a as HostRunCommandResult, b as RepositoryPermission, f as piBuiltinToolNames, g as SecretRedactor, h as piThinkingLevels, k as RepositoryRef, m as piRequiredCliFlags, p as piReadOnlyToolNames, w as ChangeRequestRef, y as CodeHostAdapter } from "../types-BAis_Lwm.mjs";
+import { r as runHostRunCommandWithDependencies } from "../commands-CF59DZXx.mjs";
+import { a as HostRunCommandResult, b as RepositoryPermission, f as piBuiltinToolNames, g as SecretRedactor, h as piThinkingLevels, k as RepositoryRef, m as piRequiredCliFlags, p as piReadOnlyToolNames, w as ChangeRequestRef, y as CodeHostAdapter } from "../types-Dn1U3MDG.mjs";
 import { z } from "zod";
 //#region src/hosts/github/command.d.ts
 type GitHubPullRequestDetails = {
@@ -38,23 +38,23 @@ declare const githubIssueCommentSchema: z.ZodPipe<z.ZodObject<{
   } | undefined;
 }>>;
 declare const githubReviewCommentSchema: z.ZodPipe<z.ZodObject<{
+  id: z.ZodNumber;
+  body: z.ZodOptional<z.ZodNullable<z.ZodString>>;
   path: z.ZodOptional<z.ZodString>;
   commit_id: z.ZodOptional<z.ZodString>;
   line: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
   start_line: z.ZodOptional<z.ZodNullable<z.ZodNumber>>;
   side: z.ZodOptional<z.ZodEnum<{
-    RIGHT: "RIGHT";
     LEFT: "LEFT";
+    RIGHT: "RIGHT";
   }>>;
   start_side: z.ZodOptional<z.ZodNullable<z.ZodEnum<{
-    RIGHT: "RIGHT";
     LEFT: "LEFT";
+    RIGHT: "RIGHT";
   }>>>;
   user: z.ZodOptional<z.ZodObject<{
     login: z.ZodString;
   }, z.core.$loose>>;
-  id: z.ZodNumber;
-  body: z.ZodOptional<z.ZodNullable<z.ZodString>>;
 }, z.core.$loose>, z.ZodTransform<{
   id: number;
   body: string | null | undefined;
@@ -63,22 +63,22 @@ declare const githubReviewCommentSchema: z.ZodPipe<z.ZodObject<{
   commitId: string | undefined;
   line: number | undefined;
   startLine: number | undefined;
-  side: "RIGHT" | "LEFT" | undefined;
-  startSide: "RIGHT" | "LEFT" | undefined;
+  side: "LEFT" | "RIGHT" | undefined;
+  startSide: "LEFT" | "RIGHT" | undefined;
 }, {
   [x: string]: unknown;
   id: number;
+  body?: string | null | undefined;
   path?: string | undefined;
   commit_id?: string | undefined;
   line?: number | null | undefined;
   start_line?: number | null | undefined;
-  side?: "RIGHT" | "LEFT" | undefined;
-  start_side?: "RIGHT" | "LEFT" | null | undefined;
+  side?: "LEFT" | "RIGHT" | undefined;
+  start_side?: "LEFT" | "RIGHT" | null | undefined;
   user?: {
     [x: string]: unknown;
     login: string;
   } | undefined;
-  body?: string | null | undefined;
 }>>;
 declare const githubReviewThreadSchema: z.ZodObject<{
   id: z.ZodString;

package/dist/internal/testing.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"testing.d.mts","names":[],"sources":["../../src/hosts/github/command.ts","../../src/hosts/github/publication-client.ts","../../src/hosts/github/adapter.ts","../../src/shared/secret-redactor.ts"],"mappings":";;;;KAiDY;EACV,YAAY;EACZ,QAAQ;;KAGE;EACV,eAAe;IACb,YAAY;IACZ;MACE,QAAQ;EACZ,wBAAwB;IACtB,YAAY;IACZ;MACE,QAAQ;;;;cChDR,0BAAwB,EAAA,QAAA,EAAA;;;;;;;;;;;;;;;;;;;cAMxB,2BAAyB,EAAA,QAAA,EAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;cA2BzB,0BAAwB,EAAA;;;;;cAwExB,sBAAoB,EAAA;;;;KAKd,qBAAqB,EAAE,aAAa;KACpC,sBAAsB,EAAE,aAAa;KACrC,qBAAqB,EAAE,aAAa;KACpC,iBAAiB,EAAE,aAAa;KAEhC;EACV,6BAA6B;EAC7B,sBAAsB;IAAW;IAAc;MAA8B;EAC7E,kBAAkB;IAAW;IAAc;MAAwB,QAAQ;EAC3E,mBAAmB;IACjB;IACA;IACA;MACE;IAAU;;EACd,mBAAmB;IACjB;IACA;IACA;MACE;IAAU;;EACd,mBAAmB;IACjB;IACA;MACE,QAAQ;EACZ,kBAAkB;IAChB;IACA;MACE,QAAQ;EACZ,oBAAoB;IAClB;IACA;IACA;IACA;IACA;IACA;IACA;IACA;IACA;MACE;IAAU;;EACd,yBAAyB;IACvB;IACA;IACA;IACA;MACE;IAAU;;EACd,oBAAoB;IAAW;MAAqB;EACpD,eAAe;IACb;IACA;IACA;IACA;MACE,QAAQ;EACZ,eAAe;IACb;IACA;IACA;IACA;IACA;MACE;;;;KCjKM;EACV,MAAM,OAAO;EACb,gBAAgB;EAChB,oBAAoB;;iBAGN,wBAAwB,UAAS,2BAAgC;;;iBCvBjE,0BAA0B;EAAY,MAAM,OAAO;IAAA"}
+{"version":3,"file":"testing.d.mts","names":[],"sources":["../../src/hosts/github/command.ts","../../src/hosts/github/publication-client.ts","../../src/hosts/github/adapter.ts","../../src/shared/secret-redactor.ts"],"mappings":";;;;KAiDY;EACV,YAAY;EACZ,QAAQ;;KAGE;EACV,eAAe;IACb,YAAY;IACZ;MACE,QAAQ;EACZ,wBAAwB;IACtB,YAAY;IACZ;MACE,QAAQ;;;;cChDR,0BAAwB,EAAA,QAAA,EAAA;;;;;;;;;;;;;;;;;;;cAMxB,2BAAyB,EAAA,QAAA,EAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;cA2BzB,0BAAwB,EAAA;;;;GAI5B,EAAA,KAAA;cAoEI,sBAAoB,EAAA;;;GAGxB,EAAA,KAAA;KAEU,qBAAqB,EAAE,aAAa;KACpC,sBAAsB,EAAE,aAAa;KACrC,qBAAqB,EAAE,aAAa;KACpC,iBAAiB,EAAE,aAAa;KAEhC;EACV,6BAA6B;EAC7B,sBAAsB;IAAW;IAAc;MAA8B;EAC7E,kBAAkB;IAAW;IAAc;MAAwB,QAAQ;EAC3E,mBAAmB;IACjB;IACA;IACA;MACE;IAAU;;EACd,mBAAmB;IACjB;IACA;IACA;MACE;IAAU;;EACd,mBAAmB;IACjB;IACA;MACE,QAAQ;EACZ,kBAAkB;IAChB;IACA;MACE,QAAQ;EACZ,oBAAoB;IAClB;IACA;IACA;IACA;IACA;IACA;IACA;IACA;IACA;MACE;IAAU;;EACd,yBAAyB;IACvB;IACA;IACA;IACA;MACE;IAAU;;EACd,oBAAoB;IAAW;MAAqB;EACpD,eAAe;IACb;IACA;IACA;IACA;MACE,QAAQ;EACZ,eAAe;IACb;IACA;IACA;IACA;IACA;MACE;;;;KChKM;EACV,MAAM,OAAO;EACb,gBAAgB;EAChB,oBAAoB;;iBAGN,wBAAwB,UAAS,2BAAgC;;;iBCxBjE,0BAA0B;EAAY,MAAM,OAAO;IAAA"}

package/dist/internal/testing.mjs

@@ -1,2 +1,2 @@
-import { _ as piReadOnlyToolNames, g as piBuiltinToolNames, l as createGitHubHostAdapter, m as createKnownSecretRedactor, r as runHostRunCommandWithDependencies, v as piRequiredCliFlags, y as piThinkingLevels } from "../commands-B9qNW4pk.mjs";
+import { _ as piReadOnlyToolNames, g as piBuiltinToolNames, l as createGitHubHostAdapter, m as createKnownSecretRedactor, r as runHostRunCommandWithDependencies, v as piRequiredCliFlags, y as piThinkingLevels } from "../commands-DgZU7tf7.mjs";
 export { createGitHubHostAdapter, createKnownSecretRedactor, piBuiltinToolNames, piReadOnlyToolNames, piRequiredCliFlags, piThinkingLevels, runHostRunCommandWithDependencies };

package/dist/main-comment-envelope-DFirHYhW.mjs

@@ -0,0 +1,69 @@
+//#region src/review/comment-branding.ts
+const piprLogoUrl = "https://pipr.run/images/pipr/pipr-mark.svg";
+const piprRepositoryUrl = "https://github.com/somus/pipr";
+const mainCommentTitle = `# <img src="${piprLogoUrl}" width="22" height="22" alt=""> Pipr Review`;
+const mainCommentTitles = /* @__PURE__ */ new Set([
+	"# pipr Review",
+	"# Pipr Review",
+	mainCommentTitle
+]);
+const mainCommentHeaderHiddenMarker = "<!-- pipr:header:hidden -->";
+const mainCommentFooterHiddenMarker = "<!-- pipr:footer:hidden -->";
+const reviewStatsStartMarker = "<!-- pipr:stats:start -->";
+const reviewStatsEndMarker = "<!-- pipr:stats:end -->";
+const reviewStatsHiddenMarker = "<!-- pipr:stats:hidden -->";
+const escapedRepositoryUrl = piprRepositoryUrl.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+const mainCommentAttributionSource = `Review generated by \\[Pipr\\]\\(${escapedRepositoryUrl}\\) for commit \`[^\`]+\`\\.(?: Config SDK \\d+\\.\\d+\\.\\d+ is behind (?:Pipr \\d+\\.\\d+\\.\\d+|\\[Pipr \\d+\\.\\d+\\.\\d+\\]\\(${escapedRepositoryUrl}/releases/tag/v\\d+\\.\\d+\\.\\d+\\))\\.)?`;
+const mainCommentAttributionTextPattern = new RegExp(`^${mainCommentAttributionSource}$`);
+const mainCommentAttributionPattern = new RegExp(`^<sub>${mainCommentAttributionSource}</sub>$`);
+//#endregion
+//#region src/review/main-comment-envelope.ts
+const generatedReviewStatsShape = [
+	/^$/,
+	/^\| Metric \| Total \|$/,
+	/^\| --- \| ---: \|$/,
+	/^\| Models \| .+ \|$/,
+	/^\| Agent runs \| \d+ \|$/,
+	/^\| Elapsed \| .+ \|$/,
+	/^\| Input tokens \| (?:Unavailable|[\d,]+(?: \(reported\))?) \|$/,
+	/^\| Output tokens \| (?:Unavailable|[\d,]+(?: \(reported\))?) \|$/,
+	/^\| Cost \(USD\) \| (?:Unavailable|\$\d+(?:\.\d+)?(?:e[+-]?\d+)?(?: \(reported\))?) \|$/,
+	/^$/,
+	/^<\/details>$/,
+	/^<!-- pipr:stats:end -->$/
+];
+function parseGeneratedMainCommentEnvelope(lines) {
+	const mainMarkerIndex = lines.findIndex((line) => line.startsWith("<!-- pipr:main-comment "));
+	const headerCandidateOffset = lines.slice(mainMarkerIndex + 1).findIndex((line) => line !== "");
+	const headerCandidateIndex = mainMarkerIndex + 1 + headerCandidateOffset;
+	const headerMarkerIndex = mainMarkerIndex >= 0 && headerCandidateOffset >= 0 && lines[headerCandidateIndex] === "<!-- pipr:header:hidden -->" ? headerCandidateIndex : -1;
+	const lastLineIndex = lines.findLastIndex((line) => line !== "");
+	const lastLine = lines[lastLineIndex] ?? "";
+	const footerIndex = lastLine === "<!-- pipr:footer:hidden -->" || mainCommentAttributionPattern.test(lastLine) ? lastLineIndex : -1;
+	const lastContentIndex = lines.slice(0, footerIndex < 0 ? lines.length : footerIndex).findLastIndex((line) => line !== "");
+	return {
+		mainMarkerIndex,
+		headerMarkerIndex,
+		statsMarkerIndex: lines[lastContentIndex] === "<!-- pipr:stats:hidden -->" ? lastContentIndex : -1,
+		statsRange: generatedReviewStatsRange(lines, footerIndex),
+		footerIndex
+	};
+}
+function generatedReviewStatsRange(lines, generatedFooterIndex) {
+	const end = lines.slice(0, generatedFooterIndex < 0 ? lines.length : generatedFooterIndex).findLastIndex((line) => line !== "");
+	if (end < 0 || lines[end] !== "<!-- pipr:stats:end -->" || lines[end - 1] !== "</details>") return;
+	const start = lines.lastIndexOf(reviewStatsStartMarker, end - 2);
+	if (start < 0 || lines[start + 1] !== "<details>" || lines[start + 2] !== "<summary>Review stats</summary>" || !matchesGeneratedReviewStatsShape(lines, start, end)) return;
+	return {
+		start,
+		end
+	};
+}
+function matchesGeneratedReviewStatsShape(lines, start, end) {
+	const generatedShape = lines.slice(start + 3, end + 1);
+	return generatedShape.length === generatedReviewStatsShape.length && generatedReviewStatsShape.every((pattern, index) => pattern.test(generatedShape[index] ?? ""));
+}
+//#endregion
+export { mainCommentTitle as a, reviewStatsEndMarker as c, mainCommentHeaderHiddenMarker as i, reviewStatsHiddenMarker as l, mainCommentAttributionTextPattern as n, mainCommentTitles as o, mainCommentFooterHiddenMarker as r, piprRepositoryUrl as s, parseGeneratedMainCommentEnvelope as t, reviewStatsStartMarker as u };
+
+//# sourceMappingURL=main-comment-envelope-DFirHYhW.mjs.map

package/dist/main-comment-envelope-DFirHYhW.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"main-comment-envelope-DFirHYhW.mjs","names":[],"sources":["../src/review/comment-branding.ts","../src/review/main-comment-envelope.ts"],"sourcesContent":["const piprLogoUrl = \"https://pipr.run/images/pipr/pipr-mark.svg\";\nexport const piprRepositoryUrl = \"https://github.com/somus/pipr\";\n\nexport const mainCommentTitle = `# <img src=\"${piprLogoUrl}\" width=\"22\" height=\"22\" alt=\"\"> Pipr Review`;\n\nexport const mainCommentTitles = new Set([\"# pipr Review\", \"# Pipr Review\", mainCommentTitle]);\nexport const mainCommentHeaderHiddenMarker = \"<!-- pipr:header:hidden -->\";\nexport const mainCommentFooterHiddenMarker = \"<!-- pipr:footer:hidden -->\";\nexport const reviewStatsStartMarker = \"<!-- pipr:stats:start -->\";\nexport const reviewStatsEndMarker = \"<!-- pipr:stats:end -->\";\nexport const reviewStatsHiddenMarker = \"<!-- pipr:stats:hidden -->\";\n\nconst escapedRepositoryUrl = piprRepositoryUrl.replace(/[.*+?^${}()|[\\]\\\\]/g, \"\\\\$&\");\nconst mainCommentAttributionSource = `Review generated by \\\\[Pipr\\\\]\\\\(${escapedRepositoryUrl}\\\\) for commit \\`[^\\`]+\\`\\\\.(?: Config SDK \\\\d+\\\\.\\\\d+\\\\.\\\\d+ is behind (?:Pipr \\\\d+\\\\.\\\\d+\\\\.\\\\d+|\\\\[Pipr \\\\d+\\\\.\\\\d+\\\\.\\\\d+\\\\]\\\\(${escapedRepositoryUrl}/releases/tag/v\\\\d+\\\\.\\\\d+\\\\.\\\\d+\\\\))\\\\.)?`;\n\nexport const mainCommentAttributionTextPattern = new RegExp(`^${mainCommentAttributionSource}$`);\n\nexport const mainCommentAttributionPattern = new RegExp(\n  `^<sub>${mainCommentAttributionSource}</sub>$`,\n);\n","import {\n  mainCommentAttributionPattern,\n  mainCommentFooterHiddenMarker,\n  mainCommentHeaderHiddenMarker,\n  reviewStatsEndMarker,\n  reviewStatsHiddenMarker,\n  reviewStatsStartMarker,\n} from \"./comment-branding.js\";\n\nconst generatedReviewStatsShape = [\n  /^$/,\n  /^\\| Metric \\| Total \\|$/,\n  /^\\| --- \\| ---: \\|$/,\n  /^\\| Models \\| .+ \\|$/,\n  /^\\| Agent runs \\| \\d+ \\|$/,\n  /^\\| Elapsed \\| .+ \\|$/,\n  /^\\| Input tokens \\| (?:Unavailable|[\\d,]+(?: \\(reported\\))?) \\|$/,\n  /^\\| Output tokens \\| (?:Unavailable|[\\d,]+(?: \\(reported\\))?) \\|$/,\n  /^\\| Cost \\(USD\\) \\| (?:Unavailable|\\$\\d+(?:\\.\\d+)?(?:e[+-]?\\d+)?(?: \\(reported\\))?) \\|$/,\n  /^$/,\n  /^<\\/details>$/,\n  /^<!-- pipr:stats:end -->$/,\n];\n\nexport type GeneratedMainCommentEnvelope = {\n  mainMarkerIndex: number;\n  headerMarkerIndex: number;\n  statsMarkerIndex: number;\n  statsRange: { start: number; end: number } | undefined;\n  footerIndex: number;\n};\n\nexport function parseGeneratedMainCommentEnvelope(lines: string[]): GeneratedMainCommentEnvelope {\n  const mainMarkerIndex = lines.findIndex((line) => line.startsWith(\"<!-- pipr:main-comment \"));\n  const headerCandidateOffset = lines.slice(mainMarkerIndex + 1).findIndex((line) => line !== \"\");\n  const headerCandidateIndex = mainMarkerIndex + 1 + headerCandidateOffset;\n  const headerMarkerIndex =\n    mainMarkerIndex >= 0 &&\n    headerCandidateOffset >= 0 &&\n    lines[headerCandidateIndex] === mainCommentHeaderHiddenMarker\n      ? headerCandidateIndex\n      : -1;\n  const lastLineIndex = lines.findLastIndex((line) => line !== \"\");\n  const lastLine = lines[lastLineIndex] ?? \"\";\n  const footerIndex =\n    lastLine === mainCommentFooterHiddenMarker || mainCommentAttributionPattern.test(lastLine)\n      ? lastLineIndex\n      : -1;\n  const lastContentIndex = lines\n    .slice(0, footerIndex < 0 ? lines.length : footerIndex)\n    .findLastIndex((line) => line !== \"\");\n  const statsMarkerIndex =\n    lines[lastContentIndex] === reviewStatsHiddenMarker ? lastContentIndex : -1;\n\n  return {\n    mainMarkerIndex,\n    headerMarkerIndex,\n    statsMarkerIndex,\n    statsRange: generatedReviewStatsRange(lines, footerIndex),\n    footerIndex,\n  };\n}\n\nfunction generatedReviewStatsRange(\n  lines: string[],\n  generatedFooterIndex: number,\n): { start: number; end: number } | undefined {\n  const end = lines\n    .slice(0, generatedFooterIndex < 0 ? lines.length : generatedFooterIndex)\n    .findLastIndex((line) => line !== \"\");\n  if (end < 0 || lines[end] !== reviewStatsEndMarker || lines[end - 1] !== \"</details>\") {\n    return undefined;\n  }\n  const start = lines.lastIndexOf(reviewStatsStartMarker, end - 2);\n  if (\n    start < 0 ||\n    lines[start + 1] !== \"<details>\" ||\n    lines[start + 2] !== \"<summary>Review stats</summary>\" ||\n    !matchesGeneratedReviewStatsShape(lines, start, end)\n  ) {\n    return undefined;\n  }\n  return { start, end };\n}\n\nfunction matchesGeneratedReviewStatsShape(lines: string[], start: number, end: number): boolean {\n  const generatedShape = lines.slice(start + 3, end + 1);\n  return (\n    generatedShape.length === generatedReviewStatsShape.length &&\n    generatedReviewStatsShape.every((pattern, index) => pattern.test(generatedShape[index] ?? \"\"))\n  );\n}\n"],"mappings":";AAAA,MAAM,cAAc;AACpB,MAAa,oBAAoB;AAEjC,MAAa,mBAAmB,eAAe,YAAY;AAE3D,MAAa,oCAAoB,IAAI,IAAI;CAAC;CAAiB;CAAiB;AAAgB,CAAC;AAC7F,MAAa,gCAAgC;AAC7C,MAAa,gCAAgC;AAC7C,MAAa,yBAAyB;AACtC,MAAa,uBAAuB;AACpC,MAAa,0BAA0B;AAEvC,MAAM,uBAAuB,kBAAkB,QAAQ,uBAAuB,MAAM;AACpF,MAAM,+BAA+B,oCAAoC,qBAAqB,qIAAqI,qBAAqB;AAExP,MAAa,oCAAoC,IAAI,OAAO,IAAI,6BAA6B,EAAE;AAE/F,MAAa,gCAAgC,IAAI,OAC/C,SAAS,6BAA6B,QACxC;;;ACVA,MAAM,4BAA4B;CAChC;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;CACA;AACF;AAUA,SAAgB,kCAAkC,OAA+C;CAC/F,MAAM,kBAAkB,MAAM,WAAW,SAAS,KAAK,WAAW,yBAAyB,CAAC;CAC5F,MAAM,wBAAwB,MAAM,MAAM,kBAAkB,CAAC,CAAC,CAAC,WAAW,SAAS,SAAS,EAAE;CAC9F,MAAM,uBAAuB,kBAAkB,IAAI;CACnD,MAAM,oBACJ,mBAAmB,KACnB,yBAAyB,KACzB,MAAM,0BAAA,gCACF,uBACA;CACN,MAAM,gBAAgB,MAAM,eAAe,SAAS,SAAS,EAAE;CAC/D,MAAM,WAAW,MAAM,kBAAkB;CACzC,MAAM,cACJ,aAAA,iCAA8C,8BAA8B,KAAK,QAAQ,IACrF,gBACA;CACN,MAAM,mBAAmB,MACtB,MAAM,GAAG,cAAc,IAAI,MAAM,SAAS,WAAW,CAAC,CACtD,eAAe,SAAS,SAAS,EAAE;CAItC,OAAO;EACL;EACA;EACA,kBALA,MAAM,sBAAA,+BAAgD,mBAAmB;EAMzE,YAAY,0BAA0B,OAAO,WAAW;EACxD;CACF;AACF;AAEA,SAAS,0BACP,OACA,sBAC4C;CAC5C,MAAM,MAAM,MACT,MAAM,GAAG,uBAAuB,IAAI,MAAM,SAAS,oBAAoB,CAAC,CACxE,eAAe,SAAS,SAAS,EAAE;CACtC,IAAI,MAAM,KAAK,MAAM,SAAA,6BAAiC,MAAM,MAAM,OAAO,cACvE;CAEF,MAAM,QAAQ,MAAM,YAAY,wBAAwB,MAAM,CAAC;CAC/D,IACE,QAAQ,KACR,MAAM,QAAQ,OAAO,eACrB,MAAM,QAAQ,OAAO,qCACrB,CAAC,iCAAiC,OAAO,OAAO,GAAG,GAEnD;CAEF,OAAO;EAAE;EAAO;CAAI;AACtB;AAEA,SAAS,iCAAiC,OAAiB,OAAe,KAAsB;CAC9F,MAAM,iBAAiB,MAAM,MAAM,QAAQ,GAAG,MAAM,CAAC;CACrD,OACE,eAAe,WAAW,0BAA0B,UACpD,0BAA0B,OAAO,SAAS,UAAU,QAAQ,KAAK,eAAe,UAAU,EAAE,CAAC;AAEjG"}

package/dist/{official-github-workflow-Bjlu-FL3.mjs → official-github-workflow-DNzV5Z70.mjs}

@@ -281,7 +281,7 @@ export default definePipr((pipr) => {
           "| --- | ---: |",
           \`| Inline findings | \${result.inlineFindings.length} |\`,
           "",
-          context.platform.id === "local" ? localInlineFindingSummary : inlineFindingSummary,
+          context.run.trigger === "local" ? localInlineFindingSummary : inlineFindingSummary,
         ].join("\\n"),
         inlineFindings: result.inlineFindings,
       };
@@ -1222,10 +1222,10 @@ export default definePipr((pipr) => {
 const pluginToolReviewRecipe = {
 	id: "plugin-tool-review",
 	title: "Plugin Tool Review",
-	description: "Typed R2-backed memory plugin for reviewer agents that need durable project context.",
+	description: "Typed R2-backed memory plugin with search-only review and explicit maintainer curation.",
 	sourceTools: ["Cloudflare R2", "Reviewer memory"],
 	configTs: `import { definePipr } from "@usepipr/sdk";
-import { r2MemoryPlugin } from "./r2-memory";
+import { memoryLimits, r2MemoryPlugin } from "./r2-memory";
 
 export default definePipr((pipr) => {
   const model = pipr.model({
@@ -1277,8 +1277,44 @@ export default definePipr((pipr) => {
     },
   });
 
+  const rememberTask = pipr.task<{ lesson: string }>({
+    name: "remember-review-memory",
+    async run(ctx, input) {
+      if (!ctx.command) {
+        throw new Error("remember-review-memory is a command-only task");
+      }
+      const lesson = input.lesson.trim();
+      if (lesson.length === 0) {
+        await ctx.command.reply("Usage: @pipr remember <lesson...>");
+        return;
+      }
+      if (lesson.length > memoryLimits.bodyCharacters) {
+        await ctx.command.reply(
+          "Reviewer memory must be " + memoryLimits.bodyCharacters + " characters or fewer.",
+        );
+        return;
+      }
+      const stored = await memory.curate(
+        {
+          subject: lesson.slice(0, memoryLimits.subjectCharacters),
+          body: lesson,
+          tags: ["maintainer-curated"],
+        },
+        ctx,
+      );
+      await ctx.command.reply("Stored reviewer memory \`" + stored.id + "\`.");
+    },
+  });
+
   pipr.on.changeRequest({ actions: ["opened", "updated"], task });
   pipr.command({ pattern: "@pipr memory-review", permission: "write", task });
+  pipr.command({
+    pattern: "@pipr remember <lesson...>",
+    permission: "write",
+    description: "Store an explicit maintainer-curated reviewer lesson.",
+    parse: (args) => ({ lesson: args.lesson ?? "" }),
+    task: rememberTask,
+  });
 });
 `,
 	files: [{
@@ -1286,22 +1322,55 @@ export default definePipr((pipr) => {
 		contents: `import { S3Client } from "bun";
 import { definePlugin, type SecretRef, type TaskContext, z } from "@usepipr/sdk";
 
+export const memoryLimits = {
+  subjectCharacters: 120,
+  bodyCharacters: 4000,
+  tagCount: 12,
+  tagCharacters: 50,
+  queryCharacters: 500,
+  resultDefault: 5,
+  resultMinimum: 1,
+  resultMaximum: 20,
+  searchObjectMaximum: 2000,
+} as const;
+
+const memorySource = z.strictObject({
+  kind: z.enum(["maintainer-command", "agent-tool"]),
+  runId: z.string().min(1).max(200),
+  platform: z.string().min(1).max(50),
+  changeRequestNumber: z.number().int().nonnegative().optional(),
+  headSha: z.string().min(1).max(200),
+});
+
 const memoryItem = z.strictObject({
-  subject: z.string(),
-  body: z.string(),
-  tags: z.array(z.string()).optional(),
-  updatedAt: z.string().optional(),
+  id: z.string().uuid().optional(),
+  subject: z.string().trim().min(1).max(memoryLimits.subjectCharacters),
+  body: z.string().trim().min(1).max(memoryLimits.bodyCharacters),
+  tags: z
+    .array(z.string().trim().min(1).max(memoryLimits.tagCharacters))
+    .max(memoryLimits.tagCount)
+    .optional(),
+  source: memorySource.optional(),
+  updatedAt: z.string().max(50).optional(),
 });
 
 const memorySearchInput = z.strictObject({
-  query: z.string(),
-  limit: z.number().optional(),
+  query: z.string().trim().min(1).max(memoryLimits.queryCharacters),
+  limit: z
+    .number()
+    .int()
+    .min(memoryLimits.resultMinimum)
+    .max(memoryLimits.resultMaximum)
+    .optional(),
 });
 
 const memoryStoreInput = z.strictObject({
-  subject: z.string(),
-  body: z.string(),
-  tags: z.array(z.string()).optional(),
+  subject: z.string().trim().min(1).max(memoryLimits.subjectCharacters),
+  body: z.string().trim().min(1).max(memoryLimits.bodyCharacters),
+  tags: z
+    .array(z.string().trim().min(1).max(memoryLimits.tagCharacters))
+    .max(memoryLimits.tagCount)
+    .optional(),
 });
 
 type MemoryItem = ReturnType<typeof memoryItem.parse>;
@@ -1328,6 +1397,7 @@ export function r2MemoryPlugin(options: R2MemoryOptions) {
       id: "memory/search-output",
       schema: z.strictObject({
         memories: z.array(memoryItem),
+        skippedObjects: z.number().int().nonnegative(),
       }),
     });
     const storeInput = pipr.schema({
@@ -1339,6 +1409,7 @@ export function r2MemoryPlugin(options: R2MemoryOptions) {
       schema: z.strictObject({
         stored: z.boolean(),
         key: z.string(),
+        id: z.string().uuid(),
       }),
     });
 
@@ -1361,12 +1432,15 @@ export function r2MemoryPlugin(options: R2MemoryOptions) {
         input: storeInput,
         output: storeOutput,
         async run({ input, ctx, signal }) {
-          return await storeMemory(input, ctx, options, signal);
+          return await storeMemory(input, ctx, options, "agent-tool", signal);
         },
         toModelOutput(output) {
           return output;
         },
       }),
+      curate(input: MemoryStoreInput, ctx: TaskContext, signal?: AbortSignal) {
+        return storeMemory(input, ctx, options, "maintainer-command", signal);
+      },
     };
   });
 }
@@ -1376,29 +1450,52 @@ async function searchMemory(
   ctx: TaskContext,
   options: R2MemoryOptions,
   signal?: AbortSignal,
-): Promise<{ memories: MemoryItem[] }> {
+): Promise<{ memories: MemoryItem[]; skippedObjects: number }> {
   signal?.throwIfAborted();
   const bucket = r2Bucket(ctx, options);
-  const listed = await bucket.list({ prefix: memoryPrefix(ctx, options) + "/", maxKeys: 200 });
   const memories: MemoryItem[] = [];
+  let continuationToken: string | undefined;
+  let scannedObjects = 0;
+  let skippedObjects = 0;
 
-  for (const object of listed.contents ?? []) {
+  do {
     signal?.throwIfAborted();
-    try {
-      const value = memoryItem.parse(await bucket.file(object.key).json());
-      if (matchesMemory(value, input.query)) {
-        memories.push(value);
+    const listed = await bucket.list({
+      prefix: memoryPrefix(ctx, options) + "/",
+      maxKeys: 200,
+      continuationToken,
+    });
+
+    const objects = (listed.contents ?? []).slice(
+      0,
+      memoryLimits.searchObjectMaximum - scannedObjects,
+    );
+    scannedObjects += objects.length;
+    for (const object of objects) {
+      signal?.throwIfAborted();
+      try {
+        const value = memoryItem.parse(await bucket.file(object.key).json());
+        if (matchesMemory(value, input.query)) {
+          memories.push(value);
+        }
+      } catch {
+        // Exclude malformed or concurrently deleted objects and report the count.
+        skippedObjects += 1;
       }
-    } catch {
-      // Ignore malformed or concurrently deleted memory objects.
     }
-  }
 
-  const limit = Math.min(Math.max(Math.trunc(input.limit ?? 5), 1), 20);
+    continuationToken = listed.isTruncated ? listed.nextContinuationToken : undefined;
+  } while (continuationToken && scannedObjects < memoryLimits.searchObjectMaximum);
+
+  const limit = Math.min(
+    Math.max(Math.trunc(input.limit ?? memoryLimits.resultDefault), memoryLimits.resultMinimum),
+    memoryLimits.resultMaximum,
+  );
   return {
     memories: memories
       .sort((left, right) => (right.updatedAt ?? "").localeCompare(left.updatedAt ?? ""))
       .slice(0, limit),
+    skippedObjects,
   };
 }
 
@@ -1406,14 +1503,53 @@ async function storeMemory(
   input: MemoryStoreInput,
   ctx: TaskContext,
   options: R2MemoryOptions,
+  sourceKind: "maintainer-command" | "agent-tool",
   signal?: AbortSignal,
-): Promise<{ stored: boolean; key: string }> {
+): Promise<{ stored: boolean; key: string; id: string }> {
   signal?.throwIfAborted();
   const bucket = r2Bucket(ctx, options);
-  const entry: MemoryItem = { ...input, updatedAt: new Date().toISOString() };
-  const key = memoryKey(input.subject, ctx, options);
+  const parsedInput = memoryStoreInput.parse(input);
+  const curatedKey =
+    sourceKind === "maintainer-command"
+      ? memoryPrefix(ctx, options) +
+        "/maintainer-command/" +
+        encodeURIComponent(ctx.run.id) +
+        ".json"
+      : undefined;
+
+  const id = curatedKey ? await stableCommandMemoryId(ctx.run.id) : crypto.randomUUID();
+  const entry = memoryItem.parse({
+    ...parsedInput,
+    id,
+    source: {
+      kind: sourceKind,
+      runId: ctx.run.id,
+      platform: ctx.platform.id,
+      changeRequestNumber: ctx.change.number,
+      headSha: ctx.change.head.sha,
+    },
+    updatedAt: new Date().toISOString(),
+  });
+  const key = curatedKey ?? memoryKey(id, parsedInput.subject, ctx, options);
   await bucket.write(key, JSON.stringify(entry, null, 2), { type: "application/json" });
-  return { stored: true, key };
+  return { stored: true, key, id };
+}
+
+async function stableCommandMemoryId(runId: string): Promise<string> {
+  const digest = new Uint8Array(
+    await crypto.subtle.digest(
+      "SHA-256",
+      new TextEncoder().encode("pipr-memory/maintainer-command/" + runId),
+    ),
+  );
+  digest[6] = (digest[6]! & 0x0f) | 0x50;
+  digest[8] = (digest[8]! & 0x3f) | 0x80;
+  const hex = Array.from(digest.slice(0, 16), (byte) => byte.toString(16).padStart(2, "0")).join(
+    "",
+  );
+  return [hex.slice(0, 8), hex.slice(8, 12), hex.slice(12, 16), hex.slice(16, 20), hex.slice(20)].join(
+    "-",
+  );
 }
 
 function r2Bucket(ctx: TaskContext, options: R2MemoryOptions): S3Client {
@@ -1444,14 +1580,26 @@ function cleanPathSegment(value: string): string {
   );
 }
 
-function memoryKey(subject: string, ctx: TaskContext, options: R2MemoryOptions): string {
+function memoryKey(
+  id: string,
+  subject: string,
+  ctx: TaskContext,
+  options: R2MemoryOptions,
+): string {
   const slug = subject
     .toLowerCase()
     .replace(/[^a-z0-9]+/g, "-")
     .replace(/^-|-$/g, "")
     .slice(0, 60);
   return (
-    memoryPrefix(ctx, options) + "/" + new Date().toISOString() + "-" + (slug || "memory") + ".json"
+    memoryPrefix(ctx, options) +
+    "/" +
+    new Date().toISOString() +
+    "-" +
+    id +
+    "-" +
+    (slug || "memory") +
+    ".json"
   );
 }
 
@@ -1488,7 +1636,9 @@ function matchesMemory(item: MemoryItem, query: string): boolean {
 
 This recipe uses Bun's S3-compatible client against Cloudflare R2. R2 credentials are declared with \`pipr.secret(...)\`, then resolved inside tool execution with \`ctx.secret(...)\`. The generated GitHub workflow maps \`PIPR_R2_MEMORY_BUCKET\`, \`PIPR_R2_MEMORY_ENDPOINT\`, \`PIPR_R2_MEMORY_ACCESS_KEY_ID\`, and \`PIPR_R2_MEMORY_SECRET_ACCESS_KEY\` repository secrets into matching runtime environment variables.
 
-R2 is object storage, not a search index. The sample lists recent JSON memory objects under \`prefix/repository-owner/repository-name\` and filters them locally, which is enough for small reviewer-memory sets. Change \`prefix\` in \`.pipr/config.ts\` when multiple repositories share one bucket; Pipr still adds the repository scope below it. The generated reviewer treats memory as untrusted historical context and requires current repository evidence for findings. It only searches memory by default. The store tool is available for explicit customization, but full review summaries are not persisted automatically.
+R2 is object storage, not a search index. The sample paginates up to 2,000 JSON memory objects under \`prefix/repository-owner/repository-name\` and filters them locally, which keeps each search bounded and is enough for small reviewer-memory sets. Change \`prefix\` in \`.pipr/config.ts\` when multiple repositories share one bucket; Pipr still adds the repository scope below it. The generated defaults cap subjects at 120 characters, bodies at 4,000 characters, tags at 12 entries of 50 characters, queries at 500 characters, and results at 20. Existing entries without ids or provenance remain readable when they satisfy those bounds. Search results include \`skippedObjects\`, the number of malformed, unavailable, or over-limit objects excluded during that search, so upgrades do not hide incompatible entries without a diagnostic.
+
+The generated reviewer treats memory as untrusted historical context and requires current repository evidence for findings. It only searches memory by default. A repository user with write permission can store one bounded, provenance-bearing lesson with \`@pipr remember <lesson...>\`; re-delivery of the same command run deterministically reuses its stored object and UUID. Full review summaries and human feedback are not persisted automatically; feedback collection, eval generation, scheduling, and proposal pull requests remain user-owned extensions.
 
 `
 };
@@ -2703,7 +2853,7 @@ function isOfficialInitRecipeId(recipe) {
 }
 //#endregion
 //#region src/config/official-github-workflow.ts
-const defaultWorkflowActionRef = "somus/pipr@v0.4.3";
+const defaultWorkflowActionRef = "somus/pipr@v0.5.0";
 /** Internal shared renderer for `pipr init` and generated recipe docs. */
 function renderOfficialGithubWorkflow(options = {}) {
 	const relativeConfigDir = options.relativeConfigDir ?? ".pipr";
@@ -2748,4 +2898,4 @@ function githubExpression(expression) {
 //#endregion
 export { officialInitRecipeWorkflowEnvSecrets as a, officialInitRecipeFiles as i, listOfficialInitRecipes as n, supportedOfficialInitRecipes as o, officialInitRecipeConfigTs as r, renderOfficialGithubWorkflow as t };
 
-//# sourceMappingURL=official-github-workflow-Bjlu-FL3.mjs.map
+//# sourceMappingURL=official-github-workflow-DNzV5Z70.mjs.map