@quolu/lattice 0.58.4 → 0.59.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quolu/lattice",
-  "version": "0.58.4",
+  "version": "0.59.0",
   "description": "Schedulability compiler for multi-agent development: observe real code boundaries, refactor the conflicting seam, recompile the plan for parallel execution",
   "author": {
     "name": "Quo / クオ at kitepon.dev",

package/src/cli-help.mjs

@@ -262,7 +262,7 @@ const SUBCOMMAND_USAGE = Object.freeze({
   'todo retract': 'todo retract --plan <key> --task <id> --reason <text>',
   'todo block': 'todo block --plan <key> --task <id> --reason <text>',
   'todo unblock': 'todo unblock --plan <key> --task <id>',
-  'todo done': 'todo done --plan <key> --task <id> --evidence <file>',
+  'todo done': 'todo done --plan <key> --task <id> --evidence <file> [--test-result <markdown-file>]',
   'todo reopen': 'todo reopen --plan <key> --task <id> --reason <text> [--override-reason <text>]',
   'todo evidence': 'todo evidence promote --plan <key> --task <id> --evidence <file>',
   'todo evidence promote': 'todo evidence promote --plan <key> --task <id> --evidence <file>',

package/src/todo-cli.mjs

@@ -10,6 +10,7 @@ import { parseTree } from 'jsonc-parser';
 import {
   TODO_COORDINATION_MODES,
   TODO_DESIGN_MEMO_PROMPT,
+  TODO_TEST_RESULT_CONTRACT_ID,
   canonicalizeTodoArtifact,
   digestTodoArtifact,
   exactRecord,
@@ -18,6 +19,7 @@ import {
   isTodoDesignMemo,
   isTodoIdentifier,
   isTodoRef,
+  isTodoTestResult,
   todoSelfDigest,
   validateEvidenceDescriptor,
 } from './todo-contracts.mjs';
@@ -333,6 +335,16 @@ async function readNoteTextInput(repoRoot, inputRef) {
   catch { throw new TodoStoreError('INPUT_UNREADABLE', 'note_input_invalid_utf8'); }
 }
 
+async function readTestResultInput(repoRoot, inputRef) {
+  const result = await readNoteTextInput(repoRoot, inputRef);
+  if (!isTodoTestResult(result)) {
+    throw new TodoStoreError('INVALID_TEST_RESULT', 'test_result_must_be_non_empty_markdown', undefined, {
+      contract_id: TODO_TEST_RESULT_CONTRACT_ID,
+    });
+  }
+  return result;
+}
+
 function taskRef(plan, taskId) {
   return { project_id: plan.project_id, plan_key: plan.plan_key, task_id: taskId };
 }
@@ -650,12 +662,15 @@ function terminalAuditDoneAdvisory(plan, phases) {
 
 async function mutate({
   repoRoot, env, planKey, taskId, kind, payload, evidenceRef, advisory = null,
-  noteContext = null, structureContext = null,
+  noteContext = null, structureContext = null, testResultRef = null,
 }) {
   const actor = mutationActor(env);
   const evidence = evidenceRef === null ? null : await readEvidenceInput(repoRoot, evidenceRef);
+  const testResult = testResultRef === null ? null : await readTestResultInput(repoRoot, testResultRef);
   let eventPayload = payload;
-  if (kind === 'done' && payload === 'authored') eventPayload = { evidence };
+  if (kind === 'done' && payload === 'authored') {
+    eventPayload = { evidence, ...(testResult === null ? {} : { test_result: testResult }) };
+  }
   if (kind === 'done' && payload === 'evidence_promotion') {
     eventPayload = { done_mode: 'evidence_promotion', imported: true, evidence };
   }
@@ -1672,7 +1687,7 @@ async function todoDetail({ repoRoot, planKey, taskId }) {
     repoRoot, store, planKey, taskId: task.task_id,
   });
   const result = {
-    schema: 'lattice.todo_detail_result.v2',
+    schema: 'lattice.todo_detail_result.v3',
     project_id: store.project_id,
     plan_key: planKey,
     plan_version: member.plan.plan_version,
@@ -3926,12 +3941,13 @@ export async function runTodoCli({ argv, cwd, stdout, stderr, env = process.env
     && argv[3] === '--task' && isTodoIdentifier(argv[4])) {
     action = (repoRoot) => mutate({ repoRoot, env, planKey: argv[2], taskId: argv[4],
       kind: 'unblock', payload: {}, evidenceRef: null });
-  } else if (argv.length === 7 && argv[0] === 'done'
+  } else if ((argv.length === 7 || argv.length === 9) && argv[0] === 'done'
     && argv[1] === '--plan' && isTodoIdentifier(argv[2])
     && argv[3] === '--task' && isTodoIdentifier(argv[4])
-    && argv[5] === '--evidence' && isTodoRef(argv[6])) {
+    && argv[5] === '--evidence' && isTodoRef(argv[6])
+    && (argv.length === 7 || (argv[7] === '--test-result' && isTodoRef(argv[8])))) {
     action = (repoRoot) => mutate({ repoRoot, env, planKey: argv[2], taskId: argv[4],
-      kind: 'done', payload: 'authored', evidenceRef: argv[6] });
+      kind: 'done', payload: 'authored', evidenceRef: argv[6], testResultRef: argv[8] ?? null });
   } else if (argv.length === 8 && argv[0] === 'evidence' && argv[1] === 'promote'
     && argv[2] === '--plan' && isTodoIdentifier(argv[3])
     && argv[4] === '--task' && isTodoIdentifier(argv[5])

package/src/todo-contracts.mjs

@@ -62,10 +62,14 @@ const CONTROL = /[\u0000-\u001f\u007f]/u;
 const NOTE_FORBIDDEN_CONTROL = /[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f]/u;
 
 export const TODO_DESIGN_MEMO_PROMPT = 'あなたがこのToDoに対して、何も考えていないならば、設計メモに `NO_PLAN` と書いてください';
+export const TODO_TEST_RESULT_CONTRACT_ID = 'lattice.todo_test_result.v1';
 
 export const isTodoDigest = (value) => typeof value === 'string' && DIGEST.test(value);
 export const isTodoIdentifier = (value) => typeof value === 'string' && IDENTIFIER.test(value);
 export const isNonNegativeSafeInteger = (value) => Number.isSafeInteger(value) && value >= 0;
+export const isTodoTestResult = (value) => typeof value === 'string' && value.trim().length > 0
+  && Buffer.byteLength(value, 'utf8') <= TODO_LIMITS.noteBodyBytes
+  && !NOTE_FORBIDDEN_CONTROL.test(value);
 
 export function isStrictTodoTimestamp(value) {
   return isCanonicalUtcTimestamp(value);
@@ -527,8 +531,10 @@ function validPayload(event) {
   if (event.kind === 'block') return exactRecord(payload, ['reason']) && nullableText(payload.reason) && payload.reason !== null;
   if (event.kind === 'unblock') return exactRecord(payload, []);
   if (event.kind === 'done' && payload?.done_mode === 'authored') {
-    return exactRecord(payload, ['done_mode', 'imported', 'evidence'])
-      && payload.imported === false && evidence(payload.evidence);
+    return (exactRecord(payload, ['done_mode', 'imported', 'evidence'])
+      || exactRecord(payload, ['done_mode', 'imported', 'evidence', 'test_result']))
+      && payload.imported === false && evidence(payload.evidence)
+      && (payload.test_result === undefined || isTodoTestResult(payload.test_result));
   }
   if (event.kind === 'done' && payload?.done_mode === 'historical_import') {
     return exactRecord(payload, ['done_mode', 'imported', 'status', 'completed_at', 'evidence'])
@@ -569,22 +575,29 @@ function validPayload(event) {
 }
 
 function validCarriedState(value) {
-  if (!exactRecord(value, [
+  const legacy = exactRecord(value, [
     'status', 'started_at', 'done_at', 'blocked_reason', 'evidence', 'imported',
-  ]) || !['pending', 'in-progress', 'blocked', 'done'].includes(value.status)
+  ]);
+  const resultAware = exactRecord(value, [
+    'status', 'started_at', 'done_at', 'blocked_reason', 'evidence', 'imported', 'test_result',
+  ]);
+  if ((!legacy && !resultAware) || !['pending', 'in-progress', 'blocked', 'done'].includes(value.status)
     || (value.started_at !== null && !isStrictTodoTimestamp(value.started_at))
     || (value.done_at !== null && !isStrictTodoTimestamp(value.done_at))
     || (value.blocked_reason !== null && !nullableText(value.blocked_reason))
     || typeof value.imported !== 'boolean') return false;
+  const testResult = resultAware ? value.test_result : null;
+  if (testResult !== null && !isTodoTestResult(testResult)) return false;
   if (value.status === 'pending') return value.started_at === null && value.done_at === null
-    && value.blocked_reason === null && value.evidence === null && value.imported === false;
+    && value.blocked_reason === null && value.evidence === null && value.imported === false
+    && testResult === null;
   const activeEvidenceValid = value.imported
     ? value.evidence === null || validateTodoImportSource(value.evidence)
     : value.evidence === null;
   if (value.status === 'in-progress') return value.done_at === null && value.blocked_reason === null
-    && activeEvidenceValid;
+    && activeEvidenceValid && testResult === null;
   if (value.status === 'blocked') return value.done_at === null && value.blocked_reason !== null
-    && activeEvidenceValid;
+    && activeEvidenceValid && testResult === null;
   return value.blocked_reason === null && value.evidence !== null
     && (value.imported ? validateTodoImportSource(value.evidence) : evidence(value.evidence));
 }
@@ -700,20 +713,35 @@ export function validateTodoSnapshot(value) {
       'schema', 'project_id', 'plan_key', 'plan_version', 'projection_version', 'through_sequence',
       'journal_head_digest', 'tasks', 'phases', 'snapshot_digest',
     ]);
-    return (v1 || v2) && isTodoIdentifier(value.project_id)
+    const v3 = value?.schema === 'lattice.todo_snapshot.v3' && exactRecord(value, [
+      'schema', 'project_id', 'plan_key', 'plan_version', 'projection_version', 'through_sequence',
+      'journal_head_digest', 'tasks', 'snapshot_digest',
+    ]);
+    const v4 = value?.schema === 'lattice.todo_snapshot.v4' && exactRecord(value, [
+      'schema', 'project_id', 'plan_key', 'plan_version', 'projection_version', 'through_sequence',
+      'journal_head_digest', 'tasks', 'phases', 'snapshot_digest',
+    ]);
+    const resultAware = v3 || v4;
+    const phaseAware = v2 || v4;
+    return (v1 || v2 || v3 || v4) && isTodoIdentifier(value.project_id)
       && isTodoIdentifier(value.plan_key) && isTodoIdentifier(value.plan_version)
-      && value.projection_version === (v1 ? 1 : 2) && isNonNegativeSafeInteger(value.through_sequence)
+      && value.projection_version === (v1 ? 1 : v2 ? 2 : v3 ? 3 : 4)
+      && isNonNegativeSafeInteger(value.through_sequence)
       && isTodoDigest(value.journal_head_digest) && Array.isArray(value.tasks)
       && value.tasks.length > 0 && value.tasks.length <= TODO_LIMITS.tasksPerPlan
       && value.tasks.every((entry) => exactRecord(entry, [
         'task_id', 'status', 'started_at', 'done_at', 'blocked_reason', 'evidence', 'evidence_unverified', 'imported',
+        ...(resultAware ? ['test_result'] : []),
       ]) && isTodoIdentifier(entry.task_id) && ['pending', 'in-progress', 'blocked', 'done'].includes(entry.status)
         && (entry.started_at === null || isStrictTodoTimestamp(entry.started_at))
         && (entry.done_at === null || isStrictTodoTimestamp(entry.done_at)) && nullableText(entry.blocked_reason)
         && (entry.evidence === null || evidence(entry.evidence) || validateTodoImportSource(entry.evidence))
-        && typeof entry.evidence_unverified === 'boolean' && typeof entry.imported === 'boolean')
+        && typeof entry.evidence_unverified === 'boolean' && typeof entry.imported === 'boolean'
+        && (!resultAware || (entry.status === 'done'
+          ? entry.test_result === null || isTodoTestResult(entry.test_result)
+          : entry.test_result === null)))
       && value.tasks.every((entry, index) => index === 0 || value.tasks[index - 1].task_id < entry.task_id)
-      && (!v2 || (Array.isArray(value.phases) && value.phases.length > 0
+      && (!phaseAware || (Array.isArray(value.phases) && value.phases.length > 0
         && value.phases.every((entry) => exactRecord(entry, [
           'phase_id', 'status', 'review_event_digest', 'decision_event_digest', 'decision_evidence',
         ]) && isTodoIdentifier(entry.phase_id)

package/src/todo-store.mjs

@@ -302,7 +302,7 @@ async function readPlanScopedJournal(repoRoot, journalRef) {
 
 function taskState(taskId) {
   return { task_id: taskId, status: 'pending', started_at: null, done_at: null, blocked_reason: null,
-    evidence: null, evidence_unverified: false, imported: false };
+    evidence: null, evidence_unverified: false, imported: false, test_result: null };
 }
 
 function emptyPhaseState(phaseId) {
@@ -685,6 +685,7 @@ function replay(plan, events, { now = new Date(), verifyEvidence, verifyImportSo
           plan_key: plan.plan_key, task_id: event.task_id,
         });
         state.status = 'done'; state.done_at = event.recorded_at; state.evidence = event.payload.evidence;
+        state.test_result = event.payload.test_result ?? null;
         state.imported = false;
         completion.set(event.task_id, { mode: 'authored', completed_at: event.recorded_at });
       } else if (event.payload.done_mode === 'historical_import') {
@@ -731,7 +732,7 @@ function replay(plan, events, { now = new Date(), verifyEvidence, verifyImportSo
       }
       const startedSuccessor = localSuccessors(plan, event.task_id).some((id) => states.get(id).status !== 'pending');
       if (startedSuccessor && event.payload.override_reason === null) fail('STORE_INCONSISTENT', 'reopen_has_started_successor');
-      state.status = 'in-progress'; state.done_at = null; state.evidence = null;
+      state.status = 'in-progress'; state.done_at = null; state.evidence = null; state.test_result = null;
       completion.delete(event.task_id);
     }
   }
@@ -774,11 +775,21 @@ function snapshotFor(plan, events, tasks) {
   // ——ADR 0147以降の暗黙terminal-audit Phaseの状態は、この関数の外(readTodoStore/
   // appendTodoEventが返す`phases`という導出ビュー)で供給する。
   const phasePlan = isPhaseTodoPlanSchema(plan.schema);
+  const resultAware = events.some((event) => event.kind === 'done'
+    && typeof event.payload?.test_result === 'string')
+    || events.some((event) => event.kind === 'plan_genesis'
+      && event.state_migration?.some(({ state }) => typeof state?.test_result === 'string'));
+  const snapshotTasks = resultAware ? tasks.map((task) => ({ ...task, test_result: task.test_result ?? null }))
+    : tasks.map(({ test_result: _testResult, ...task }) => task);
   const snapshot = {
-    schema: phasePlan ? 'lattice.todo_snapshot.v2' : 'lattice.todo_snapshot.v1',
+    schema: resultAware
+      ? (phasePlan ? 'lattice.todo_snapshot.v4' : 'lattice.todo_snapshot.v3')
+      : (phasePlan ? 'lattice.todo_snapshot.v2' : 'lattice.todo_snapshot.v1'),
     project_id: plan.project_id, plan_key: plan.plan_key,
-    plan_version: plan.plan_version, projection_version: phasePlan ? 2 : 1, through_sequence: head.sequence,
-    journal_head_digest: head.event_digest, tasks, snapshot_digest: '',
+    plan_version: plan.plan_version,
+    projection_version: resultAware ? (phasePlan ? 4 : 3) : (phasePlan ? 2 : 1),
+    through_sequence: head.sequence,
+    journal_head_digest: head.event_digest, tasks: snapshotTasks, snapshot_digest: '',
     ...(phasePlan ? { phases: projectPhaseStates(plan, events,
       new Map(tasks.map((task) => [task.task_id, task]))) } : {}),
   };
@@ -1515,9 +1526,14 @@ export async function rebuildTodoSnapshot(options = {}) {
 
 function nextEvent(input, storeMember) {
   const previous = storeMember.journal.events.at(-1);
-  const payload = input.kind === 'done' && exactRecord(input.payload, ['evidence'])
-    ? { done_mode: 'authored', imported: false, evidence: input.payload.evidence }
-    : input.payload;
+  let payload = input.payload;
+  if (input.kind === 'done' && (exactRecord(input.payload, ['evidence'])
+    || exactRecord(input.payload, ['evidence', 'test_result']))) {
+    payload = {
+      done_mode: 'authored', imported: false, evidence: input.payload.evidence,
+      ...(input.payload.test_result === undefined ? {} : { test_result: input.payload.test_result }),
+    };
+  }
   const phaseCapablePlan = isPhaseTodoPlanSchema(storeMember.plan.schema);
   const phaseKind = ['phase_review', 'phase_accept', 'phase_reject', 'phase_reopen', 'phase_close_unaudited']
     .includes(input.kind);
@@ -2613,6 +2629,7 @@ function stateMigrationFor(previous, revision) {
     return { ...migration, state: {
       status: state.status, started_at: state.started_at, done_at: state.done_at,
       blocked_reason: state.blocked_reason, evidence: state.evidence, imported: state.imported,
+      ...(typeof state.test_result === 'string' ? { test_result: state.test_result } : {}),
     } };
   });
 }