@amistio/cli 0.1.25 → 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(),
@@ -5452,6 +5454,68 @@ var canonicalAppEvaluationCategories = /* @__PURE__ */ new Map([
   ["operations", "other"],
   ["ops", "other"]
 ]);
+var canonicalTestCommandKinds = /* @__PURE__ */ new Map([
+  ["verify", "verify"],
+  ["verification", "verify"],
+  ["wholeapp", "verify"],
+  ["wholeappverification", "verify"],
+  ["full", "verify"],
+  ["fullcheck", "verify"],
+  ["fullchecks", "verify"],
+  ["allcheck", "verify"],
+  ["allchecks", "verify"],
+  ["ci", "verify"],
+  ["continuousintegration", "verify"],
+  ["test", "test"],
+  ["tests", "test"],
+  ["testing", "test"],
+  ["unittest", "test"],
+  ["unittests", "test"],
+  ["integrationtest", "test"],
+  ["integrationtests", "test"],
+  ["e2etest", "test"],
+  ["e2etests", "test"],
+  ["endtoend", "test"],
+  ["endtoendtest", "test"],
+  ["spec", "test"],
+  ["specs", "test"],
+  ["suite", "test"],
+  ["testsuite", "test"],
+  ["jest", "test"],
+  ["vitest", "test"],
+  ["coverage", "coverage"],
+  ["coveragereport", "coverage"],
+  ["lint", "lint"],
+  ["linting", "lint"],
+  ["eslint", "lint"],
+  ["format", "lint"],
+  ["formatting", "lint"],
+  ["style", "lint"],
+  ["quality", "lint"],
+  ["qualitycheck", "lint"],
+  ["staticanalysis", "lint"],
+  ["typecheck", "typecheck"],
+  ["typechecks", "typecheck"],
+  ["typechecking", "typecheck"],
+  ["typecheck", "typecheck"],
+  ["tsc", "typecheck"],
+  ["typescript", "typecheck"],
+  ["build", "build"],
+  ["compile", "build"],
+  ["focused", "focused"],
+  ["focus", "focused"],
+  ["targeted", "focused"],
+  ["scoped", "focused"],
+  ["package", "focused"],
+  ["packaged", "focused"],
+  ["affected", "focused"],
+  ["affectedtests", "focused"],
+  ["touched", "focused"],
+  ["touchedtests", "focused"],
+  ["changed", "focused"],
+  ["changedtests", "focused"],
+  ["other", "focused"]
+]);
 var canonicalProjectContextCitationSources = /* @__PURE__ */ new Map([
   ["projectbrain", "projectBrain"],
   ["approvedbrain", "projectBrain"],
@@ -5551,9 +5615,10 @@ function createTestQualityScanPrompt(workItem, context) {
     "## Output Contract",
     "",
     "Print exactly one JSON object between the markers below. The CLI will submit only this structured test scan result back to Amistio.",
-    "Accepted command kind values: lint, typecheck, unitTest, integrationTest, e2eTest, coverage, build, verify, other.",
+    "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":[]}',
@@ -5594,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":[]}',
@@ -5875,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."
@@ -6237,7 +6304,71 @@ function parseTestQualityScanResult(output) {
   }
   const payload = output.slice(start + testQualityStart.length, end).trim();
   const parsed = JSON.parse(stripJsonFence(payload));
-  return testQualityScanResultSchema.parse(parsed);
+  return testQualityScanResultSchema.parse(normalizeTestQualityScanResult(parsed));
+}
+function normalizeTestQualityScanResult(value) {
+  if (!isObjectRecord(value)) {
+    return value;
+  }
+  const normalized = { ...value };
+  if (isObjectRecord(normalized.profile) && Array.isArray(normalized.profile.commands)) {
+    normalized.profile = {
+      ...normalized.profile,
+      commands: normalized.profile.commands.map(normalizeTestCommandKindObject)
+    };
+  }
+  if (Array.isArray(normalized.commandSummaries)) {
+    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;
+}
+function normalizeTestCommandKindObject(value) {
+  if (!isObjectRecord(value)) {
+    return value;
+  }
+  return { ...value, kind: normalizeTestCommandKind(value.kind) };
+}
+function normalizeTestCommandKind(value) {
+  if (typeof value !== "string") {
+    return value;
+  }
+  const trimmed = value.trim();
+  if (!trimmed) {
+    return value;
+  }
+  const direct = canonicalTestCommandKinds.get(normalizeEnumKey(trimmed));
+  if (direct) {
+    return direct;
+  }
+  for (const segment of trimmed.split(/[\/,|]+/)) {
+    const segmentKind = canonicalTestCommandKinds.get(normalizeEnumKey(segment));
+    if (segmentKind) {
+      return segmentKind;
+    }
+  }
+  return "focused";
 }
 function parseImplementationTestGateResult(output) {
   const start = output.indexOf(implementationTestGateStart);
@@ -6247,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") {