@kody-ade/kody-engine 0.4.257 → 0.4.259

package/dist/agent-actions/types.ts

@@ -469,23 +469,23 @@ export interface Context {
      * `@kody <next>` comment, which is silently ignored when Kody comments as
      * a GitHub App (bot author), stalling the pipeline at classify.
      */
-  nextDispatch?: {
-    action?: string
-    agentResponsibility?: string
-    agentAction?: string
-    cliArgs: Record<string, unknown>
-    saveReport?: boolean
-  }
+    nextDispatch?: {
+      action?: string
+      agentResponsibility?: string
+      agentAction?: string
+      cliArgs: Record<string, unknown>
+      saveReport?: boolean
+    }
     /** In-process hand-off to a full Job, preserving job identity in task state. */
     nextJob?: Job
     /** Where to return after nextJob succeeds. Used by task-jobs to keep draining pending work. */
-  afterNextJob?: {
-    action?: string
-    agentResponsibility?: string
-    agentAction?: string
-    cliArgs: Record<string, unknown>
-    saveReport?: boolean
-  }
+    afterNextJob?: {
+      action?: string
+      agentResponsibility?: string
+      agentAction?: string
+      cliArgs: Record<string, unknown>
+      saveReport?: boolean
+    }
   }
   /**
    * If a preflight script sets this to true, the executor skips the agent
@@ -552,6 +552,6 @@ export interface Job {
   flavor: JobFlavor
   /** Manual force-run (bypass cadence) for a scheduled job. */
   force?: boolean
-  /** Save this responsibility run output as reports/<responsibility>.md. */
+  /** Ask the owning goal/loop to refresh reports/<goal-or-loop>.md after it applies evidence. */
   saveReport?: boolean
 }

package/dist/bin/kody.js

@@ -15,7 +15,7 @@ var init_package = __esm({
   "package.json"() {
     package_default = {
       name: "@kody-ade/kody-engine",
-      version: "0.4.257",
+      version: "0.4.259",
       description: "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative agentAction profiles.",
       license: "MIT",
       type: "module",
@@ -8090,15 +8090,17 @@ function agentResponsibilityReportToEvidence(report) {
   };
 }
 function agentResponsibilityResultToEvidence(result, fallbackGoalId, explicitEvidence) {
-  const targetId = result.target?.type === "goal" ? result.target.id : fallbackGoalId;
+  if (result.target && result.target.type !== "goal") return null;
+  const targetId = result.target?.id ?? fallbackGoalId;
   if (!targetId) return null;
+  const hasEvidenceValues = Object.keys(result.evidence ?? {}).length > 0;
   return {
     version: 1,
     target: { type: "goal", id: targetId },
     status: result.status,
     summary: result.summary,
     evidence: result.evidence,
-    explicitEvidence,
+    explicitEvidence: hasEvidenceValues ? void 0 : explicitEvidence,
     facts: result.facts,
     artifacts: result.artifacts,
     missingEvidence: result.missingEvidence,
@@ -8138,7 +8140,8 @@ function applyAgentResponsibilityEvidenceToGoalState(state, evidence) {
     nextFacts[key] = value;
   }
   const pending = typeof nextFacts.pendingEvidence === "string" ? nextFacts.pendingEvidence : "";
-  const statusEvidence = evidence.explicitEvidence || pending;
+  const hasEvidenceValues = Object.keys(evidence.evidence ?? {}).length > 0;
+  const statusEvidence = evidence.explicitEvidence || (hasEvidenceValues ? "" : pending);
   const hasPendingEvidenceValue = pending ? Object.hasOwn(evidence.evidence ?? {}, pending) : false;
   const terminalStatus = evidence.status === "pass" || evidence.status === "fail" || evidence.status === "blocked";
   if (statusEvidence && !Object.hasOwn(evidence.evidence ?? {}, statusEvidence)) {
@@ -8249,6 +8252,135 @@ function flushLogs(ctx) {
     );
   }
 }
+function writeGoalDashboardReport(ctx, goalId, state, snapshot, evidenceItems) {
+  if (ctx.data.jobSaveReport !== true) return;
+  if (!REPORT_SLUG_RE.test(goalId)) {
+    fail(ctx, `goal report: invalid goal id "${goalId}"`);
+    return;
+  }
+  const filePath = `reports/${goalId}.md`;
+  const body = goalReportBody(goalId, state, snapshot, evidenceItems);
+  try {
+    const current = readStateText(ctx.config, ctx.cwd, filePath);
+    if (current?.content === body) {
+      recordGoalReport(ctx.data, { slug: goalId, path: current.path, changed: false });
+      return;
+    }
+    upsertStateText(ctx.config, ctx.cwd, filePath, body, `chore(reports): refresh ${goalId}`);
+    recordGoalReport(ctx.data, { slug: goalId, path: filePath, changed: true });
+  } catch (err) {
+    fail(ctx, `goal report: ${err instanceof Error ? err.message : String(err)}`);
+  }
+}
+function recordGoalReport(data, report) {
+  const prior = Array.isArray(data.goalReports) ? data.goalReports : [];
+  data.goalReports = [...prior, report];
+}
+function goalReportBody(goalId, state, snapshot, evidenceItems) {
+  const outputs = evidenceItems.map(responsibilityEvidenceOutput);
+  const latestOutput = outputs.at(-1);
+  const nextStep = state.state === "done" ? "done" : latestOutput ? nextStepFromEvidence(snapshot, latestOutput) : "wait";
+  const facts = recordField2(snapshot, "facts") ?? recordField2(state.extra, "facts") ?? {};
+  const blockers = uniqueStrings2([
+    ...stringArrayField(snapshot, "blockers"),
+    ...evidenceItems.flatMap((item) => item.blockers)
+  ]);
+  const missingEvidence = uniqueStrings2([
+    ...stringArrayField(snapshot, "missingEvidence"),
+    ...evidenceItems.flatMap((item) => item.missingEvidence)
+  ]);
+  const artifacts = uniqueArtifacts2(evidenceItems.flatMap((item) => item.artifacts));
+  return [
+    `# ${goalId}`,
+    "",
+    "## Status",
+    `- State: ${state.state}`,
+    `- Stage: ${stringField3(snapshot, "stage") ?? stringField3(state.extra, "stage") ?? "unknown"}`,
+    `- Next step: ${nextStep}`,
+    `- Updated: ${state.updatedAt ?? state.createdAt ?? state.startedAt ?? "unknown"}`,
+    "",
+    "## Decision",
+    `- Reason: ${decisionReason(state, latestOutput, missingEvidence, blockers)}`,
+    `- Required evidence: ${listOrNone(stringArrayField(snapshot, "requiredEvidence"))}`,
+    `- Satisfied evidence: ${listOrNone(stringArrayField(snapshot, "satisfiedEvidence"))}`,
+    `- Missing evidence: ${listOrNone(missingEvidence)}`,
+    `- Blockers: ${listOrNone(blockers)}`,
+    "",
+    "## Responsibility Evidence",
+    ...outputs.flatMap((output, index) => evidenceOutputMarkdown(index + 1, output)),
+    "",
+    "## Facts",
+    fencedJson(facts),
+    "",
+    "## Artifacts",
+    ...artifactMarkdown(artifacts),
+    ""
+  ].join("\n");
+}
+function decisionReason(state, latestOutput, missingEvidence, blockers) {
+  if (state.state === "done") return "destination evidence satisfied";
+  if (blockers.length > 0) return blockers[0] ?? "blocked";
+  const summary = stringField3(latestOutput, "summary");
+  if (summary) return summary;
+  if (missingEvidence.length > 0) return `waiting for ${missingEvidence[0]}`;
+  return "waiting for more evidence";
+}
+function evidenceOutputMarkdown(index, output) {
+  return [
+    `### Output ${index}`,
+    `- Status: ${stringField3(output, "status") ?? "unknown"}`,
+    `- Summary: ${stringField3(output, "summary") ?? "no summary"}`,
+    `- Sources: ${listOrNone(stringArrayField(output, "sources"))}`,
+    `- Evidence values: ${inlineJson(recordField2(output, "evidence") ?? {})}`,
+    `- Missing evidence: ${listOrNone(stringArrayField(output, "missingEvidence"))}`,
+    `- Blockers: ${listOrNone(stringArrayField(output, "blockers"))}`,
+    ""
+  ];
+}
+function artifactMarkdown(artifacts) {
+  if (artifacts.length === 0) return ["- none"];
+  return artifacts.map((artifact) => {
+    if (artifact.url) return `- [${artifact.label}](${artifact.url})`;
+    return `- ${artifact.label}: ${artifact.path}`;
+  });
+}
+function fencedJson(value) {
+  return ["```json", JSON.stringify(value, null, 2), "```"].join("\n");
+}
+function inlineJson(value) {
+  return JSON.stringify(value);
+}
+function listOrNone(values) {
+  return values.length > 0 ? values.join(", ") : "none";
+}
+function uniqueStrings2(values) {
+  return [...new Set(values)].sort();
+}
+function stringField3(record2, key) {
+  const value = record2?.[key];
+  return typeof value === "string" && value.trim() ? value : void 0;
+}
+function recordField2(record2, key) {
+  const value = record2?.[key];
+  return value && typeof value === "object" && !Array.isArray(value) ? { ...value } : void 0;
+}
+function uniqueArtifacts2(artifacts) {
+  const seen = /* @__PURE__ */ new Set();
+  const out = [];
+  for (const artifact of artifacts) {
+    const key = `${artifact.label}
+${artifact.url ?? ""}
+${artifact.path ?? ""}`;
+    if (seen.has(key)) continue;
+    seen.add(key);
+    out.push(artifact);
+  }
+  return out;
+}
+function fail(ctx, reason) {
+  ctx.output.reason = ctx.output.reason ? `${ctx.output.reason}; ${reason}` : reason;
+  if (ctx.output.exitCode === 0) ctx.output.exitCode = 99;
+}
 function collectReports(raw, agentResult) {
   const out = [];
   if (Array.isArray(raw)) {
@@ -8362,7 +8494,7 @@ function describeMessage(goalId, evidenceItems) {
   const pieces = evidenceItems.map((evidence) => `${evidence.sources.join("+")}:${evidence.status}`);
   return `Apply agentResponsibility output to ${goalId}${pieces.length > 0 ? ` (${pieces.join("; ")})` : ""}`;
 }
-var applyAgentResponsibilityReports;
+var REPORT_SLUG_RE, applyAgentResponsibilityReports;
 var init_applyAgentResponsibilityReports = __esm({
   "src/scripts/applyAgentResponsibilityReports.ts"() {
     "use strict";
@@ -8373,6 +8505,8 @@ var init_applyAgentResponsibilityReports = __esm({
     init_runLog();
     init_state2();
     init_stateStore();
+    init_stateRepo();
+    REPORT_SLUG_RE = /^[a-z0-9][a-z0-9_-]{0,63}$/;
     applyAgentResponsibilityReports = async (ctx, _profile, agentResult) => {
       const reports = collectReports(ctx.data.agentResponsibilityReports, agentResult);
       const results = collectResults(ctx.data.dutyResults, agentResult);
@@ -8452,11 +8586,20 @@ var init_applyAgentResponsibilityReports = __esm({
             change: goalRunLogChange(beforeCompletionSnapshot, afterCompletionSnapshot)
           });
         }
-        if (serializeGoalState(next) === serializeGoalState(prior)) {
+        const changed = serializeGoalState(next) !== serializeGoalState(prior);
+        const nextForOutput = changed ? { ...next, updatedAt: nowIso() } : next;
+        writeGoalDashboardReport(
+          ctx,
+          goalId,
+          nextForOutput,
+          afterCompletionSnapshot ?? beforeCompletionSnapshot,
+          goalEvidence
+        );
+        if (!changed) {
           flushLogs(ctx);
           continue;
         }
-        putGoalState(ctx.config, goalId, { ...next, updatedAt: nowIso() }, describeMessage(goalId, goalEvidence), ctx.cwd);
+        putGoalState(ctx.config, goalId, nextForOutput, describeMessage(goalId, goalEvidence), ctx.cwd);
         flushLogs(ctx);
       }
     };
@@ -12036,7 +12179,7 @@ var init_markFlowSuccess = __esm({
 });
 
 // src/goal/operations.ts
-function fail(err) {
+function fail2(err) {
   if (err instanceof Error) {
     const lines = err.message.split("\n").filter(Boolean);
     return { ok: false, error: lines[0] ?? err.message };
@@ -12048,7 +12191,7 @@ function commentOnIssue(issueNumber, body, cwd) {
     gh(["issue", "comment", String(issueNumber), "--body", body], { cwd });
     return { ok: true };
   } catch (err) {
-    return fail(err);
+    return fail2(err);
   }
 }
 function mergePrSquash(prNumber, cwd) {
@@ -12056,7 +12199,7 @@ function mergePrSquash(prNumber, cwd) {
     gh(["pr", "merge", String(prNumber), "--squash", "--delete-branch"], { cwd });
     return { ok: true };
   } catch (err) {
-    return fail(err);
+    return fail2(err);
   }
 }
 var init_operations = __esm({
@@ -12587,21 +12730,21 @@ var init_openQaIssue = __esm({
         ctx.data.action = failedAction3(reason);
         return;
       }
-      const reportBody2 = agentResult.finalText.trim();
-      if (!reportBody2) {
+      const reportBody = agentResult.finalText.trim();
+      if (!reportBody) {
         process.stderr.write("qa-engineer: agent produced no report body\n");
         ctx.output.exitCode = 1;
         ctx.output.reason = "empty report body";
         ctx.data.action = failedAction3("empty report body");
         return;
       }
-      const verdict = detectVerdict(reportBody2);
+      const verdict = detectVerdict(reportBody);
       ctx.data.qaVerdict = verdict;
-      ctx.data.qaReport = reportBody2;
+      ctx.data.qaReport = reportBody;
       const existingIssue = ctx.args.issue;
       if (typeof existingIssue === "number" && Number.isFinite(existingIssue) && existingIssue > 0) {
         try {
-          postIssueComment(existingIssue, reportBody2, ctx.cwd);
+          postIssueComment(existingIssue, reportBody, ctx.cwd);
         } catch (err) {
           const msg = err instanceof Error ? err.message : String(err);
           ctx.output.exitCode = 4;
@@ -12623,7 +12766,7 @@ QA_REPORT_POSTED=https://github.com/${ctx.config.github.owner}/${ctx.config.gith
       const hasLabel = ensureLabel2(ctx.cwd);
       let created;
       try {
-        created = createQaIssue(title, reportBody2, hasLabel, ctx.cwd);
+        created = createQaIssue(title, reportBody, hasLabel, ctx.cwd);
       } catch (err) {
         const msg = err instanceof Error ? err.message : String(err);
         ctx.output.exitCode = 4;
@@ -15955,86 +16098,6 @@ var init_writeJobStateFile = __esm({
   }
 });
 
-// src/scripts/writeResponsibilityReport.ts
-function writeReport(ctx, profile, agentResult) {
-  const slug2 = responsibilitySlug(ctx);
-  if (!slug2) {
-    fail2(ctx, "writeResponsibilityReport: missing responsibility slug");
-    return;
-  }
-  if (!REPORT_SLUG_RE.test(slug2)) {
-    fail2(ctx, `writeResponsibilityReport: invalid responsibility slug "${slug2}"`);
-    return;
-  }
-  const body = reportBody(ctx, agentResult, profile);
-  if (!body.trim()) {
-    fail2(ctx, `writeResponsibilityReport: ${slug2} produced no report output`);
-    return;
-  }
-  const filePath = `reports/${slug2}.md`;
-  const current = readStateText(ctx.config, ctx.cwd, filePath);
-  if (current?.content === body) {
-    ctx.data.responsibilityReport = { slug: slug2, path: current.path, changed: false };
-    return;
-  }
-  upsertStateText(
-    ctx.config,
-    ctx.cwd,
-    filePath,
-    body,
-    `chore(reports): refresh ${slug2}`
-  );
-  ctx.data.responsibilityReport = { slug: slug2, path: filePath, changed: true };
-}
-function responsibilitySlug(ctx) {
-  const candidates = [
-    ctx.data.jobAgentResponsibility,
-    ctx.data.agentResponsibilitySlug,
-    ctx.data.jobSlug
-  ];
-  for (const candidate of candidates) {
-    if (typeof candidate === "string" && candidate.trim()) return candidate.trim();
-  }
-  return null;
-}
-function reportBody(ctx, agentResult, profile) {
-  const prSummary = ctx.data.prSummary;
-  if (typeof prSummary === "string" && prSummary.trim()) return ensureTrailingNewline(prSummary.trim());
-  if (agentResult?.finalText.trim()) return ensureTrailingNewline(agentResult.finalText.trim());
-  const reason = ctx.output.reason || ctx.data.agentFailureReason || agentResult?.error;
-  if (typeof reason === "string" && reason.trim()) {
-    return `# ${profile.name}
-
-FAILED: ${reason.trim()}
-`;
-  }
-  return "";
-}
-function fail2(ctx, reason) {
-  ctx.output.reason = ctx.output.reason ? `${ctx.output.reason}; ${reason}` : reason;
-  if (ctx.output.exitCode === 0) ctx.output.exitCode = 99;
-}
-function ensureTrailingNewline(value) {
-  return value.endsWith("\n") ? value : `${value}
-`;
-}
-var REPORT_SLUG_RE, writeResponsibilityReport;
-var init_writeResponsibilityReport = __esm({
-  "src/scripts/writeResponsibilityReport.ts"() {
-    "use strict";
-    init_stateRepo();
-    REPORT_SLUG_RE = /^[a-z0-9][a-z0-9_-]{0,63}$/;
-    writeResponsibilityReport = async (ctx, profile, agentResult) => {
-      if (ctx.data.jobSaveReport !== true) return;
-      try {
-        writeReport(ctx, profile, agentResult);
-      } catch (err) {
-        fail2(ctx, `writeResponsibilityReport: ${err instanceof Error ? err.message : String(err)}`);
-      }
-    };
-  }
-});
-
 // src/scripts/index.ts
 var preflightScripts, postflightScripts, allScriptNames;
 var init_scripts = __esm({
@@ -16136,7 +16199,6 @@ var init_scripts = __esm({
     init_writeAgentRunSummary();
     init_writeIssueStateComment();
     init_writeJobStateFile();
-    init_writeResponsibilityReport();
     preflightScripts = {
       runFlow,
       fixFlow,
@@ -16215,7 +16277,6 @@ var init_scripts = __esm({
       postReviewResult,
       persistArtifacts,
       writeAgentRunSummary,
-      writeResponsibilityReport,
       saveTaskState,
       mirrorStateToPr,
       startFlow,
@@ -16775,7 +16836,6 @@ async function runAgentAction(profileName, input) {
         outcome: postOutcome
       });
     }
-    await writeResponsibilityReport(ctx, profile, agentResult);
     return finishAndEnd({
       exitCode: ctx.output.exitCode ?? 0,
       prUrl: ctx.output.prUrl,
@@ -17173,9 +17233,8 @@ var init_executor = __esm({
     init_registry();
     init_runtimePaths();
     init_scripts();
-    init_writeResponsibilityReport();
-    init_subagents();
     init_stateWorkspace();
+    init_subagents();
     init_task_artifacts();
     init_tools();
     MUTATING_POSTFLIGHTS = /* @__PURE__ */ new Set([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.4.257",
+  "version": "0.4.259",
   "description": "kody — autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative agentAction profiles.",
   "license": "MIT",
   "type": "module",