@amistio/cli 0.1.26 → 0.1.27

package/dist/index.js

@@ -1691,6 +1691,7 @@ var appEvaluationFindingResultSchema = z.object({
   proposedPlanTitle: z.string().trim().min(1).max(200).optional(),
   proposedPlanRepoPath: z.string().trim().min(1).max(300).optional(),
   proposedPlanContent: z.string().trim().min(1).max(3e4).optional(),
+  dedupeKey: z.string().trim().min(1).max(240).optional(),
   status: appEvaluationFindingStatusSchema.default("open")
 });
 var appEvaluationScanResultSchema = z.object({
@@ -1736,6 +1737,7 @@ var appEvaluationFindingItemSchema = baseItemSchema.extend({
   verificationPlan: z.array(z.string().trim().min(1).max(300)).min(1),
   proposedLifecycleAction: appEvaluationPlanLifecycleActionSchema.default("none"),
   relatedArtifactIds: z.array(z.string().trim().min(1).max(160)).default([]),
+  dedupeKey: z.string().trim().min(1).max(240).optional(),
   proposedPlanDocumentId: z.string().min(1).optional(),
   implementationWorkItemId: z.string().min(1).optional(),
   reviewNotes: z.string().trim().min(1).optional(),
@@ -5616,6 +5618,7 @@ function createTestQualityScanPrompt(workItem, context) {
     "Accepted command kind values: verify, test, coverage, lint, typecheck, build, focused.",
     "Accepted command status values: passed, failed, skipped, missing, blocked.",
     "Accepted finding categories: missingTests, missingCoverage, lowCoverage, failingTests, failingQuality, missingCommand, flakyTests, unverifiedImplementation, testGap, other.",
+    "Omit optional fields when unavailable; do not emit null for optional command summary fields such as exitCode, durationMs, outputExcerpt, commandId, or safePaths.",
     "",
     testQualityStart,
     '{"summary":"Whole-app verification exists, but coverage reporting is missing for one package.","profile":{"testProfileId":"test_profile_detected","repositoryLinkId":"repository_link_placeholder","status":"detected","enabled":true,"detectedPackageManager":"pnpm","packageManagers":["pnpm"],"commands":[{"commandId":"verify","kind":"verify","label":"Whole-app verification","source":"detected","scriptName":"verify","workingDirectory":".","required":true}],"coverageThresholds":{"lines":80},"defaultWholeAppCommandId":"verify","focusedCommandIds":[],"lastDetectedAt":"2026-01-01T00:00:00.000Z"},"commandSummaries":[{"commandId":"verify","kind":"verify","label":"Whole-app verification","status":"passed","exitCode":0,"summary":"The repository verification script completed successfully.","safePaths":["package.json"]}],"coverage":{"status":"missing","thresholds":{"lines":80},"summary":"No coverage report was found."},"findings":[{"title":"Coverage report is missing","category":"missingCoverage","severity":"medium","confidence":"high","summary":"The repository test profile does not produce a coverage summary.","affectedSurfaces":["Test verification"],"evidence":["The scan found a test command but no coverage output."],"safePaths":["package.json"],"suggestedAction":"Add or document a coverage command for the affected package and include it in whole-app verification.","verificationPlan":["Run the new coverage command locally","Confirm coverage appears in the Test panel"],"dedupeKey":"missing-coverage"}],"blockedReasons":[],"redactionState":{"status":"clean","redactedFields":[]},"verificationPlan":["Review generated Test findings","Run the whole-app verification command before implementation handoff"],"warnings":[]}',
@@ -5656,6 +5659,7 @@ function createImplementationTestGatePrompt(workItem) {
     "",
     "Print exactly one JSON object between the markers below. The CLI will submit only this structured gate result back to Amistio.",
     "Accepted outcome values: passed, failed, blocked, overridden.",
+    "Omit optional fields when unavailable; do not emit null for optional command summary fields such as exitCode, durationMs, outputExcerpt, commandId, or safePaths.",
     "",
     implementationTestGateStart,
     '{"outcome":"passed","summary":"Focused checks and whole-app verification passed.","commandSummaries":[{"commandId":"verify","kind":"verify","label":"Whole-app verification","status":"passed","exitCode":0,"summary":"The repository verification script completed successfully.","safePaths":["package.json"]}],"coverage":{"status":"unknown","thresholds":{},"summary":"No coverage threshold was configured for this gate."},"findings":[],"blockedReasons":[],"redactionState":{"status":"clean","redactedFields":[]},"verificationPlan":["Record this gate result before marking implementation complete"],"warnings":[]}',
@@ -5937,9 +5941,10 @@ function createAppEvaluationScanPrompt(workItem, context) {
     "Accepted severity values: info, low, medium, high, critical.",
     "Accepted confidence values: low, medium, high.",
     "Accepted proposedLifecycleAction values: createPlan, updatePlan, markCompleted, markImplemented, markSuperseded, markBlocked, archive, keepActive, none.",
+    "Include a stable dedupeKey for each finding based on the underlying issue identity, not scan time, generated wording, or transient ordering.",
     "",
     appEvaluationStart,
-    '{"summary":"The app is generally healthy, but one release-readiness plan needs cleanup and verification evidence should be refreshed.","baselineVersion":"amistio-app-evaluation-v1","findings":[{"title":"Stale release plan should be marked superseded","category":"planCleanup","severity":"low","confidence":"high","summary":"A release plan appears to describe a workflow that has since been replaced by a newer accepted plan.","affectedSurfaces":["docs/plans"],"evidence":["A newer plan references the same scope and status, while the older plan remains proposed."],"suggestedAction":"Prepare a cleanup update that marks the stale plan superseded and links to the newer plan.","verificationPlan":["Review both plan files","Confirm the newer plan is accepted before changing lifecycle state"],"safePaths":["docs/plans/PLAN-example.md"],"proposedLifecycleAction":"markSuperseded","relatedArtifactIds":[]}],"verificationPlan":["Review App Evaluation findings for approval","Run the canonical verify gate before implementing approved actions"]}',
+    '{"summary":"The app is generally healthy, but one release-readiness plan needs cleanup and verification evidence should be refreshed.","baselineVersion":"amistio-app-evaluation-v1","findings":[{"title":"Stale release plan should be marked superseded","category":"planCleanup","severity":"low","confidence":"high","dedupeKey":"plan-cleanup-docs-plans-plan-example-superseded","summary":"A release plan appears to describe a workflow that has since been replaced by a newer accepted plan.","affectedSurfaces":["docs/plans"],"evidence":["A newer plan references the same scope and status, while the older plan remains proposed."],"suggestedAction":"Prepare a cleanup update that marks the stale plan superseded and links to the newer plan.","verificationPlan":["Review both plan files","Confirm the newer plan is accepted before changing lifecycle state"],"safePaths":["docs/plans/PLAN-example.md"],"proposedLifecycleAction":"markSuperseded","relatedArtifactIds":[]}],"verificationPlan":["Review App Evaluation findings for approval","Run the canonical verify gate before implementing approved actions"]}',
     appEvaluationEnd,
     "",
     "Do not put Markdown fences around the markers. Do not implement improvements or cleanup."
@@ -6299,9 +6304,9 @@ function parseTestQualityScanResult(output) {
   }
   const payload = output.slice(start + testQualityStart.length, end).trim();
   const parsed = JSON.parse(stripJsonFence(payload));
-  return testQualityScanResultSchema.parse(normalizeTestQualityScanResultCommandKinds(parsed));
+  return testQualityScanResultSchema.parse(normalizeTestQualityScanResult(parsed));
 }
-function normalizeTestQualityScanResultCommandKinds(value) {
+function normalizeTestQualityScanResult(value) {
   if (!isObjectRecord(value)) {
     return value;
   }
@@ -6313,7 +6318,29 @@ function normalizeTestQualityScanResultCommandKinds(value) {
     };
   }
   if (Array.isArray(normalized.commandSummaries)) {
-    normalized.commandSummaries = normalized.commandSummaries.map(normalizeTestCommandKindObject);
+    normalized.commandSummaries = normalized.commandSummaries.map(normalizeTestCommandSummaryObject);
+  }
+  return normalized;
+}
+function normalizeImplementationTestGateResult(value) {
+  if (!isObjectRecord(value)) {
+    return value;
+  }
+  const normalized = { ...value };
+  if (Array.isArray(normalized.commandSummaries)) {
+    normalized.commandSummaries = normalized.commandSummaries.map(normalizeTestCommandSummaryObject);
+  }
+  return normalized;
+}
+function normalizeTestCommandSummaryObject(value) {
+  if (!isObjectRecord(value)) {
+    return value;
+  }
+  const normalized = { ...value, kind: normalizeTestCommandKind(value.kind) };
+  for (const key of ["commandId", "durationMs", "exitCode", "outputExcerpt", "safePaths"]) {
+    if (normalized[key] === null) {
+      delete normalized[key];
+    }
   }
   return normalized;
 }
@@ -6351,7 +6378,7 @@ function parseImplementationTestGateResult(output) {
   }
   const payload = output.slice(start + implementationTestGateStart.length, end).trim();
   const parsed = JSON.parse(stripJsonFence(payload));
-  return implementationTestGateResultSchema.parse(parsed);
+  return implementationTestGateResultSchema.parse(normalizeImplementationTestGateResult(parsed));
 }
 function projectContextRefreshSubmissionFailureSummary(result) {
   if (result.refresh.status !== "failed" && result.workItem.status !== "failed") {