@haposoft/cafekit 0.16.0-rc.2 → 0.16.0-rc.4
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/bin/__tests__/codex-hooks.test.js +794 -21
- package/bin/__tests__/codex-native.test.js +228 -7
- package/bin/__tests__/develop-contract.test.js +1838 -112
- package/bin/__tests__/installer-safety.test.js +82 -29
- package/bin/__tests__/opencode-spec-gate.test.js +799 -0
- package/bin/__tests__/package-inventory.test.js +841 -10
- package/bin/__tests__/specs-v2-execution-closeout.test.js +421 -0
- package/bin/__tests__/specs-v2-policy-and-scaffold.test.js +601 -0
- package/bin/__tests__/specs-v2-validator-grounder-contract.test.js +952 -0
- package/bin/__tests__/test-runtime-dependency-closure.cjs +182 -0
- package/bin/__tests__/validator-grounding.test.js +2469 -71
- package/bin/lib/managed-writer.js +12 -3
- package/bin/phases/codex-runtime.js +17 -1
- package/package.json +1 -1
- package/src/claude/CLAUDE.md +6 -0
- package/src/claude/agents/code-auditor.md +20 -0
- package/src/claude/agents/spec-maker.md +153 -202
- package/src/claude/hooks/__tests__/completion-authority.test.js +953 -0
- package/src/claude/hooks/__tests__/privacy-block.test.js +172 -0
- package/src/claude/hooks/__tests__/semantic-review-authority.test.js +257 -0
- package/src/claude/hooks/__tests__/spec-gate.test.js +604 -56
- package/src/claude/hooks/completion-authority-check.cjs +56 -0
- package/src/claude/hooks/completion-authority-state.cjs +527 -0
- package/src/claude/hooks/completion-authority.cjs +154 -0
- package/src/claude/hooks/docs-sync.cjs +5 -2
- package/src/claude/hooks/lib/privacy-command-analysis.cjs +393 -0
- package/src/claude/hooks/lib/runtime-path-safety.cjs +119 -0
- package/src/claude/hooks/privacy-block.cjs +77 -30
- package/src/claude/hooks/semantic-review-authority.cjs +152 -0
- package/src/claude/hooks/session.cjs +33 -19
- package/src/claude/hooks/spec-gate.cjs +181 -139
- package/src/claude/hooks/spec-state.cjs +112 -47
- package/src/claude/migration-manifest.json +15 -2
- package/src/claude/rules/state-sync.md +52 -21
- package/src/claude/rules/workflow.md +17 -9
- package/src/claude/scripts/provenance.cjs +315 -0
- package/src/claude/scripts/spec-final-state.cjs +244 -0
- package/src/claude/scripts/spec-ground.cjs +535 -173
- package/src/claude/scripts/spec-readiness.cjs +118 -0
- package/src/claude/scripts/spec-receipt.cjs +162 -0
- package/src/claude/scripts/spec-resolver.cjs +501 -0
- package/src/claude/scripts/spec-scaffold.cjs +1945 -55
- package/src/claude/scripts/spec-semantic-model.cjs +396 -0
- package/src/claude/scripts/validate-spec-output.cjs +3204 -262
- package/src/claude/scripts/workflow-policy.cjs +1888 -203
- package/src/claude/settings/settings.json +12 -0
- package/src/claude/skills/code-review/SKILL.md +94 -66
- package/src/claude/skills/code-review/references/spec-compliance-review.md +10 -2
- package/src/claude/skills/code-review/references/verification-gate.md +13 -0
- package/src/claude/skills/develop/SKILL.md +138 -283
- package/src/claude/skills/develop/references/parallel-waves.md +9 -6
- package/src/claude/skills/develop/references/quality-gate.md +123 -148
- package/src/claude/skills/develop/references/subagent-patterns.md +42 -37
- package/src/claude/skills/specs/SKILL.md +192 -397
- package/src/claude/skills/specs/references/ask-user-question-gates.md +27 -18
- package/src/claude/skills/specs/references/cross-spec-dependency.md +8 -4
- package/src/claude/skills/specs/references/research-strategy.md +4 -1
- package/src/claude/skills/specs/references/review.md +200 -260
- package/src/claude/skills/specs/references/scope-inquiry.md +18 -5
- package/src/claude/skills/specs/rules/design-discovery-full.md +33 -81
- package/src/claude/skills/specs/rules/design-discovery-light.md +21 -42
- package/src/claude/skills/specs/rules/design-principles.md +136 -180
- package/src/claude/skills/specs/rules/design-review.md +42 -97
- package/src/claude/skills/specs/rules/ears-format.md +42 -50
- package/src/claude/skills/specs/rules/phase-decision-matrix.md +42 -41
- package/src/claude/skills/specs/rules/task-scoring-rubric.md +46 -46
- package/src/claude/skills/specs/rules/tasks-generation.md +189 -271
- package/src/claude/skills/specs/templates/design.md +49 -263
- package/src/claude/skills/specs/templates/requirements.md +26 -36
- package/src/claude/skills/specs/templates/research.md +9 -98
- package/src/claude/skills/specs/templates/spec-state.json +26 -67
- package/src/claude/skills/specs/templates/task.md +30 -82
- package/src/claude/skills/sync/SKILL.md +45 -15
- package/src/claude/skills/sync/references/sync-protocols.md +34 -16
- package/src/claude/skills/test/SKILL.md +151 -167
- package/src/codex/AGENTS.md +6 -0
- package/src/codex/hooks/completion-authority-check.cjs +63 -0
- package/src/codex/hooks/completion-authority-state.cjs +527 -0
- package/src/codex/hooks/completion-authority.cjs +106 -0
- package/src/codex/hooks/lib/hook-context.cjs +58 -4
- package/src/codex/hooks/lib/privacy-command-analysis.cjs +3 -0
- package/src/codex/hooks/lib/privacy-state.cjs +38 -16
- package/src/codex/hooks/lib/runtime-path-safety.cjs +3 -0
- package/src/codex/hooks/lib/spec-receipt.cjs +86 -115
- package/src/codex/hooks/lib/spec-utils.cjs +104 -27
- package/src/codex/hooks/privacy-approval.cjs +1 -1
- package/src/codex/hooks/privacy-block.cjs +67 -11
- package/src/codex/hooks/semantic-review-authority.cjs +130 -0
- package/src/codex/hooks/session.cjs +2 -0
- package/src/codex/hooks/spec-gate.cjs +151 -24
- package/src/codex/hooks/spec-state.cjs +65 -12
- package/src/codex/hooks/state.cjs +2 -0
- package/src/codex/hooks.json +16 -1
- package/src/codex/rules/state-sync.md +47 -27
- package/src/common/AGENTS.md +1 -1
- package/src/opencode/AGENTS.md +4 -2
- package/src/opencode/plugins/spec-gate.ts +296 -122
- package/src/opencode/plugins/spec-state.ts +183 -116
|
@@ -10,51 +10,1717 @@ const { test } = require('node:test');
|
|
|
10
10
|
const ROOT = path.resolve(__dirname, '../..');
|
|
11
11
|
const VALIDATOR = path.join(ROOT, 'src/claude/scripts/validate-spec-output.cjs');
|
|
12
12
|
const GROUNDING = path.join(ROOT, 'src/claude/scripts/spec-ground.cjs');
|
|
13
|
+
const SCAFFOLD = path.join(ROOT, 'src/claude/scripts/spec-scaffold.cjs');
|
|
14
|
+
const RESOLVER = path.join(ROOT, 'src/claude/scripts/spec-resolver.cjs');
|
|
15
|
+
const POLICY = require(path.join(ROOT, 'src/claude/scripts/workflow-policy.cjs'));
|
|
16
|
+
const SPEC_RESOLVER = require(RESOLVER);
|
|
17
|
+
const { normalizeCodexBody } = require(path.join(ROOT, 'bin/lib/codex-install.js'));
|
|
18
|
+
const { copyClaudeTestRuntime } = require('./test-runtime-dependency-closure.cjs');
|
|
13
19
|
|
|
14
20
|
function write(filePath, content) {
|
|
15
21
|
fs.mkdirSync(path.dirname(filePath), { recursive: true });
|
|
16
22
|
fs.writeFileSync(filePath, content);
|
|
17
23
|
}
|
|
18
24
|
|
|
19
|
-
function
|
|
20
|
-
|
|
25
|
+
function writeRaw(filePath, content) {
|
|
26
|
+
fs.mkdirSync(path.dirname(filePath), { recursive: true });
|
|
27
|
+
fs.writeFileSync(filePath, content);
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
function copyCodexRuntime(adapterRoot, entrypoints) {
|
|
31
|
+
const files = copyClaudeTestRuntime(ROOT, adapterRoot, entrypoints);
|
|
32
|
+
for (const relative of files) {
|
|
33
|
+
if (!/\.(?:cjs|js)$/.test(relative)) continue;
|
|
34
|
+
const target = path.join(adapterRoot, relative);
|
|
35
|
+
writeRaw(
|
|
36
|
+
target,
|
|
37
|
+
normalizeCodexBody(fs.readFileSync(target, 'utf8'), `src/claude/${relative}`),
|
|
38
|
+
);
|
|
39
|
+
}
|
|
40
|
+
return files;
|
|
41
|
+
}
|
|
42
|
+
|
|
43
|
+
function run(script, args, cwd, options = {}) {
|
|
44
|
+
const fixtureRoot = script === VALIDATOR && typeof args[0] === 'string'
|
|
45
|
+
? path.dirname(path.dirname(path.resolve(args[0])))
|
|
46
|
+
: null;
|
|
47
|
+
return spawnSync(process.execPath, [script, ...args], {
|
|
48
|
+
cwd,
|
|
49
|
+
encoding: 'utf8',
|
|
50
|
+
...options,
|
|
51
|
+
env: {
|
|
52
|
+
...process.env,
|
|
53
|
+
...(fixtureRoot ? { HOME: path.join(fixtureRoot, '.home'), USERPROFILE: path.join(fixtureRoot, '.home') } : {}),
|
|
54
|
+
...(options.env || {}),
|
|
55
|
+
},
|
|
56
|
+
});
|
|
57
|
+
}
|
|
58
|
+
|
|
59
|
+
function snapshotTree(root) {
|
|
60
|
+
const entries = [];
|
|
61
|
+
function visit(directory) {
|
|
62
|
+
for (const entry of fs.readdirSync(directory, { withFileTypes: true })) {
|
|
63
|
+
const fullPath = path.join(directory, entry.name);
|
|
64
|
+
const relative = path.relative(root, fullPath);
|
|
65
|
+
if (entry.isSymbolicLink()) {
|
|
66
|
+
entries.push(`${relative}\0symlink\0${fs.readlinkSync(fullPath)}`);
|
|
67
|
+
} else if (entry.isDirectory()) {
|
|
68
|
+
entries.push(`${relative}\0directory`);
|
|
69
|
+
visit(fullPath);
|
|
70
|
+
} else {
|
|
71
|
+
entries.push(`${relative}\0file\0${fs.readFileSync(fullPath).toString('base64')}`);
|
|
72
|
+
}
|
|
73
|
+
}
|
|
74
|
+
}
|
|
75
|
+
visit(root);
|
|
76
|
+
return entries.sort().join('\n');
|
|
77
|
+
}
|
|
78
|
+
|
|
79
|
+
function task({
|
|
80
|
+
id = 'R1-01',
|
|
81
|
+
mapping = '1.1',
|
|
82
|
+
related = '`src/one.js` | Modify',
|
|
83
|
+
relatedRows = null,
|
|
84
|
+
dependencies = 'none',
|
|
85
|
+
status = 'pending',
|
|
86
|
+
parallel = false,
|
|
87
|
+
extra = '',
|
|
88
|
+
} = {}) {
|
|
89
|
+
const relatedPath = related.match(/`([^`]+)`/)?.[1] || 'src/one.js';
|
|
90
|
+
const action = related.match(/\|\s*([A-Za-z]+)\b/i)?.[1] || 'Modify';
|
|
91
|
+
const acceptanceId = mapping.match(/\d+\.\d+/)?.[0];
|
|
92
|
+
const lifecycleRows = relatedRows === null ? (action.toLowerCase() === 'modify'
|
|
93
|
+
? `| \`${relatedPath}\` | Read | Inspect existing implementation |\n| ${related} | Update implementation |`
|
|
94
|
+
: `| ${related} | Apply declared lifecycle action |`) : relatedRows;
|
|
95
|
+
return `# Task ${id}: One\n\n**Status:** ${status}\n\n## Outcome\n\nCalling the real fixture entrypoint returns status value 1.\n\n## Scope and Typed Anchors\n\n- **In scope:** Implement the mapped observable behavior.\n- **Out of scope:** Unrelated runtime behavior.\n- **Contracts/Invariants:** none\n- **Canonical design anchors consumed:** A-D-01\n\n| ID | Type | Target | Role |\n|---|---|---|---|\n| A-${id}-01 | command | \`node --test test/one.test.js\` | planned verification command |\n\n## Changes\n\n- [ ] ${parallel ? '(P) ' : ''}Update the source boundary and preserve its negative path.${mapping ? ` _Requirements: ${mapping}_` : ''}\n\n## Acceptance\n\n${acceptanceId ? `- **R${acceptanceId}:**` : '-'} Calling \`${relatedPath}\` returns an observable result with status value 1.\n\n## Dependencies\n\n- ${dependencies}\n\n## Verification Plan\n\n- **Command:** \`node --test test/one.test.js\`\n- **Expected:** exit code 0 and one assertion for status value 1\n- **Negative path:** invalid input returns a deterministic error result\n- **Reachability:** \`${relatedPath}\` is invoked by the focused test\n\n## Related Files\n\n| Path | Action | Description |\n|---|---|---|---|\n${lifecycleRows}\n${extra}`;
|
|
96
|
+
}
|
|
97
|
+
|
|
98
|
+
function legacyWorkflowPolicy(lane = 'Standard') {
|
|
99
|
+
const proofObligations = lane === 'Critical'
|
|
100
|
+
? ['needsInspection', 'needsExecutionProof', 'needsIndependentAudit', 'needsResearchGrounding']
|
|
101
|
+
: ['needsInspection', 'needsExecutionProof'];
|
|
102
|
+
return {
|
|
103
|
+
version: '1',
|
|
104
|
+
lane,
|
|
105
|
+
automatic_lane: lane,
|
|
106
|
+
risks: lane === 'Critical' ? ['auth'] : [],
|
|
107
|
+
artifact_profile: lane === 'Critical' ? 'strict' : 'bounded',
|
|
108
|
+
proof_obligations: proofObligations,
|
|
109
|
+
actor_needs: proofObligations.map((obligation) => ({
|
|
110
|
+
capability: {
|
|
111
|
+
needsInspection: 'inspection',
|
|
112
|
+
needsExecutionProof: 'execution-proof',
|
|
113
|
+
needsIndependentAudit: 'audit',
|
|
114
|
+
needsResearchGrounding: 'research-grounding',
|
|
115
|
+
}[obligation],
|
|
116
|
+
independence: obligation === 'needsIndependentAudit' ? 'independent' : 'same-session',
|
|
117
|
+
})),
|
|
118
|
+
override_receipt: null,
|
|
119
|
+
};
|
|
120
|
+
}
|
|
121
|
+
|
|
122
|
+
function legacyTask({ relatedRows = '| `src/one.js` | Read | Inspect existing behavior |', extra = '' } = {}) {
|
|
123
|
+
return `# Task R1-01: One\n\n**Status:** pending\n**Dependencies:** none\n\n## Context\n\nThe existing fixture service returns one deterministic result.\n\n## Constraints\n\nPreserve the stable result contract and its invalid-input behavior.\n\n## Steps\n\n- [ ] Inspect the declared boundary and preserve status value 1. _Requirements: 1.1_\n\n## Requirements\n\n- 1.1 — Return status value 1 from the fixture service.\n\n## Related Files\n\n| Path | Action | Description |\n|---|---|---|\n${relatedRows}\n\n## Completion Criteria\n\n- Calling \`src/one.js\` returns result status value 1.\n\n## Evidence\n\n- **Command:** \`node --test test/one.test.js\`\n- **Expected:** exit code 0 and status value 1\n- **Inspect:** \`src/one.js\` runtime result\n- Runtime reachability verification\n - **Entrypoint/caller:** \`src/one.js\`\n\n## Risk Assessment\n\n| Risk | Severity | Mitigation |\n|---|---|---|\n| regression | low | focused test |\n${extra}`;
|
|
124
|
+
}
|
|
125
|
+
|
|
126
|
+
function lifecycleTimestamps({ tasks = true, research = false, review = false } = {}) {
|
|
127
|
+
return {
|
|
128
|
+
init: '2026-08-11T00:00:00+07:00',
|
|
129
|
+
requirements_done: '2026-08-11T00:01:00+07:00',
|
|
130
|
+
research_done: research ? '2026-08-11T00:02:00+07:00' : null,
|
|
131
|
+
design_done: '2026-08-11T00:03:00+07:00',
|
|
132
|
+
tasks_done: tasks ? '2026-08-11T00:04:00+07:00' : null,
|
|
133
|
+
code_done: null,
|
|
134
|
+
test_done: null,
|
|
135
|
+
review_done: review ? '2026-08-11T00:05:00+07:00' : null,
|
|
136
|
+
validation_done: '2026-08-11T00:06:00+07:00',
|
|
137
|
+
};
|
|
138
|
+
}
|
|
139
|
+
|
|
140
|
+
function semanticReviewNotRun() {
|
|
141
|
+
return {
|
|
142
|
+
status: 'not-run', reviewed_artifact_digest: null, reviewed_criteria: [], counterexamples: [],
|
|
143
|
+
};
|
|
144
|
+
}
|
|
145
|
+
|
|
146
|
+
function semanticDigest(specDir) {
|
|
147
|
+
const result = run(VALIDATOR, [specDir, '--semantic-digest'], ROOT);
|
|
148
|
+
assert.equal(result.status, 0, output(result));
|
|
149
|
+
return result.stdout.trim();
|
|
150
|
+
}
|
|
151
|
+
|
|
152
|
+
function completeSemanticReview(specDir, independence = 'same-session') {
|
|
153
|
+
const statePath = path.join(specDir, 'spec.json');
|
|
154
|
+
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
155
|
+
state.validation = isPlainObjectForTest(state.validation) ? state.validation : {};
|
|
156
|
+
state.validation.semantic_review = {
|
|
157
|
+
status: 'completed',
|
|
158
|
+
reviewed_artifact_digest: semanticDigest(specDir),
|
|
159
|
+
reviewed_criteria: ['R1.1'],
|
|
160
|
+
counterexamples: [{
|
|
161
|
+
criterion: 'R1.1',
|
|
162
|
+
scenario: 'The fixture receives invalid input at its runtime boundary.',
|
|
163
|
+
expected: 'The fixture returns the deterministic negative-path result.',
|
|
164
|
+
design_reference: 'D1',
|
|
165
|
+
}],
|
|
166
|
+
};
|
|
167
|
+
write(statePath, JSON.stringify(state, null, 2));
|
|
168
|
+
if (independence === 'independent') {
|
|
169
|
+
const root = path.dirname(path.dirname(specDir));
|
|
170
|
+
const featureName = state.feature_name;
|
|
171
|
+
const claim = `CAFEKIT_SEMANTIC_REVIEW_ATTESTATION ${JSON.stringify({
|
|
172
|
+
feature_name: featureName,
|
|
173
|
+
spec_file: path.relative(root, statePath),
|
|
174
|
+
semantic_digest: state.validation.semantic_review.reviewed_artifact_digest,
|
|
175
|
+
verdict: 'PASS',
|
|
176
|
+
})}`;
|
|
177
|
+
const result = spawnSync(process.execPath, [path.join(ROOT, 'src/claude/hooks/semantic-review-authority.cjs')], {
|
|
178
|
+
cwd: root,
|
|
179
|
+
env: { ...process.env, HOME: path.join(root, '.home'), USERPROFILE: path.join(root, '.home'), PROJECT_ROOT: root },
|
|
180
|
+
input: JSON.stringify({ hook_event_name: 'SubagentStop', session_id: 'host-session', agent_id: 'review-agent', agent_type: 'code_auditor', last_assistant_message: claim, cwd: root }),
|
|
181
|
+
encoding: 'utf8',
|
|
182
|
+
});
|
|
183
|
+
assert.equal(result.status, 0, output(result));
|
|
184
|
+
assert.equal(result.stderr, '');
|
|
185
|
+
}
|
|
186
|
+
return state;
|
|
187
|
+
}
|
|
188
|
+
|
|
189
|
+
function isPlainObjectForTest(value) {
|
|
190
|
+
return Boolean(value && typeof value === 'object' && !Array.isArray(value));
|
|
191
|
+
}
|
|
192
|
+
|
|
193
|
+
function obligationClosureBlock(obligation) {
|
|
194
|
+
const consumers = obligation.consumer_tasks.length > 0
|
|
195
|
+
? obligation.consumer_tasks.join(', ')
|
|
196
|
+
: 'none';
|
|
197
|
+
return `<!-- implementation-obligation:${obligation.id} -->\n- Requirements: ${obligation.requirements.join(', ')}\n- Canonical state: ${obligation.canonical_state}\n- Authority: ${obligation.authority}\n- Recovery: ${obligation.recovery}\n- Owner task: ${obligation.owner_task}\n- Consumers: ${consumers}\n- Verification: ${obligation.verification}\n- Evidence: ${obligation.evidence}\n`;
|
|
198
|
+
}
|
|
199
|
+
|
|
200
|
+
function executionClosure(obligation) {
|
|
201
|
+
return `\n## Execution Closure\n\n${obligationClosureBlock(obligation)}`;
|
|
202
|
+
}
|
|
203
|
+
|
|
204
|
+
function executionClosureTable(obligation) {
|
|
205
|
+
const consumers = obligation.consumer_tasks.length > 0
|
|
206
|
+
? obligation.consumer_tasks.join(', ')
|
|
207
|
+
: 'none';
|
|
208
|
+
return `\n## Execution Closure\n\n<!-- implementation-obligation:${obligation.id} -->\n` +
|
|
209
|
+
'| Obligation ID / Requirements | Canonical State | Authority | Crash / Retry Recovery | Owner Task | Consumer Tasks / Dependency Order | Verification / Evidence |\n' +
|
|
210
|
+
'|---|---|---|---|---|---|---|\n' +
|
|
211
|
+
`| ${obligation.id} / ${obligation.requirements.join(', ')} | ${obligation.canonical_state} | ${obligation.authority} | ${obligation.recovery} | \`${obligation.owner_task}\` | ${consumers} | \`${obligation.verification}\` → ${obligation.evidence} |\n`;
|
|
212
|
+
}
|
|
213
|
+
|
|
214
|
+
function criticalNoObligationDesign() {
|
|
215
|
+
return designFixture() +
|
|
216
|
+
'## Execution Closure\n\n' +
|
|
217
|
+
'N/A — this fixture has no cross-task state, retry, migration, or authority obligation.\n';
|
|
218
|
+
}
|
|
219
|
+
|
|
220
|
+
function designFixture() {
|
|
221
|
+
return '# Design\n\n## Boundary\n\nThe `src/one.js` module owns the fixture behavior and returns one deterministic result.\n\n' +
|
|
222
|
+
'## Typed Anchors\n\n| ID | Type | Target | Role |\n|---|---|---|---|\n' +
|
|
223
|
+
'| A-D-01 | file | `src/one.js` | existing owner and runtime entrypoint |\n\n' +
|
|
224
|
+
'## Decisions and Invariants\n\n### D1 — Stable result\n\n- **Decision:** Return status value 1.\n- **Negative path:** Invalid input returns a deterministic error.\n- **Anchors:** A-D-01\n\n' +
|
|
225
|
+
'## Verification\n\n| Requirement | Proof target | Expected result | Negative path / reachability |\n|---|---|---|---|\n' +
|
|
226
|
+
'| R1.1 | `node --test test/one.test.js` | exit code 0 and status value 1 | invalid input through `src/one.js` |\n\n';
|
|
227
|
+
}
|
|
228
|
+
|
|
229
|
+
function designFixture21() {
|
|
230
|
+
return '# Design\n\n## Boundary\n\nThe `src/one.js` module owns the fixture behavior and returns one deterministic result.\n\n' +
|
|
231
|
+
'## Typed Anchors\n\n| ID | Type | Target | Role | Access | Action |\n|---|---|---|---|---|---|\n' +
|
|
232
|
+
'| A-D-01 | file | `src/one.js` | runtime entrypoint | read | read |\n' +
|
|
233
|
+
'| A-D-02 | command | `node --test test/one.test.js` | proof command | read | read |\n\n' +
|
|
234
|
+
'## Decisions and Invariants\n\n### D1 — Stable result\n\nReturn status value 1 and reject invalid input.\n\n' +
|
|
235
|
+
'### I1 — Invalid-state invariant\n\nInvalid input never reports the success status.\n\n' +
|
|
236
|
+
'### C1 — Result contract\n\nThe result contains one numeric status.\n\n' +
|
|
237
|
+
'## Verification Definitions\n\n' +
|
|
238
|
+
'- **V1**: Subject criteria R1.1; Subject owner A-D-01; Proof criteria R1.2; Proof owner A-D-02; Evidence anchor A-D-02; Decision refs D1, I1, C1; Method command `node --test test/one.test.js`; Expected exit code 0 and status value 1; Negative/failure invalid input returns a deterministic error; Reachability/grounding entrypoint `src/one.js` via A-D-01, A-D-02.\n';
|
|
239
|
+
}
|
|
240
|
+
|
|
241
|
+
function createSpec(root, options = {}) {
|
|
242
|
+
const featureName = options.name || 'spec';
|
|
243
|
+
const specDir = path.join(root, 'specs', featureName);
|
|
244
|
+
const taskPath = 'tasks/task-R1-01-one.md';
|
|
245
|
+
write(
|
|
246
|
+
path.join(specDir, 'requirements.md'),
|
|
247
|
+
options.requirements || '# Requirements\n\n### Requirement 1: One\n\n- **R1.1** When input arrives, the service shall return one deterministic result.\n',
|
|
248
|
+
);
|
|
249
|
+
write(
|
|
250
|
+
path.join(specDir, 'design.md'),
|
|
251
|
+
options.design || designFixture(),
|
|
252
|
+
);
|
|
253
|
+
write(path.join(specDir, taskPath), options.task || task());
|
|
254
|
+
write(path.join(root, 'test/one.test.js'), "'use strict';\n// Separate proof artifact for fixture validation.\n");
|
|
255
|
+
write(path.join(specDir, 'spec.json'), JSON.stringify({
|
|
256
|
+
schema_version: '2.0',
|
|
257
|
+
feature_name: featureName,
|
|
258
|
+
created_at: '2026-08-11T00:00:00+07:00',
|
|
259
|
+
updated_at: '2026-08-11T00:05:00+07:00',
|
|
260
|
+
language: 'en',
|
|
261
|
+
status: 'in_progress',
|
|
262
|
+
current_phase: 'tasks',
|
|
263
|
+
scope_lock: { source: 'fixture', in_scope: ['1'], out_of_scope: [], expansion_policy: 'requires-user-approval' },
|
|
264
|
+
coordination: {
|
|
265
|
+
tasks_required: true,
|
|
266
|
+
phases_required: false,
|
|
267
|
+
reason: 'task_topology',
|
|
268
|
+
task_triggers: ['separate_proof'],
|
|
269
|
+
},
|
|
270
|
+
task_files: [taskPath],
|
|
271
|
+
task_registry: {
|
|
272
|
+
[taskPath]: {
|
|
273
|
+
id: 'R1-01', title: 'One', status: 'pending', dependencies: [], blocker: null,
|
|
274
|
+
started_at: null, completed_at: null, last_updated_at: null,
|
|
275
|
+
artifacts: ['test/one.test.js'],
|
|
276
|
+
},
|
|
277
|
+
},
|
|
278
|
+
approvals: { requirements: { generated: true, agent_validated: true, user_approved: true }, design: { generated: true, agent_validated: true, user_approved: true }, tasks: { generated: true, agent_validated: true, user_approved: true } },
|
|
279
|
+
validation: { status: 'not-run', semantic_review: semanticReviewNotRun() },
|
|
280
|
+
timestamps: { ...lifecycleTimestamps(), validation_done: null },
|
|
281
|
+
ready_for_implementation: false,
|
|
282
|
+
workflow_policy: POLICY.workflowPolicySnapshot({ riskSignals: {} }),
|
|
283
|
+
...(options.spec || {}),
|
|
284
|
+
}, null, 2));
|
|
285
|
+
return specDir;
|
|
286
|
+
}
|
|
287
|
+
|
|
288
|
+
function createBoundedSpec(root, name = 'bounded') {
|
|
289
|
+
const specDir = path.join(root, name);
|
|
290
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
291
|
+
write(path.join(specDir, 'requirements.md'), '# Requirements\n\n### Requirement 1: One\n\n- **R1.1** When input arrives, the service shall return status value 1.\n');
|
|
292
|
+
write(path.join(specDir, 'design.md'), designFixture());
|
|
293
|
+
write(path.join(specDir, 'spec.json'), JSON.stringify({
|
|
294
|
+
schema_version: '2.0',
|
|
295
|
+
feature_name: name,
|
|
296
|
+
created_at: '2026-08-11T00:00:00+07:00',
|
|
297
|
+
updated_at: '2026-08-11T00:03:00+07:00',
|
|
298
|
+
status: 'in_progress',
|
|
299
|
+
current_phase: 'design',
|
|
300
|
+
scope_lock: { source: 'fixture', in_scope: [], out_of_scope: [], expansion_policy: 'requires-user-approval' },
|
|
301
|
+
coordination: { tasks_required: false, phases_required: false, reason: 'taskless_requested' },
|
|
302
|
+
approvals: {
|
|
303
|
+
requirements: { generated: true, agent_validated: true, user_approved: false },
|
|
304
|
+
design: { generated: true, agent_validated: true, user_approved: false },
|
|
305
|
+
},
|
|
306
|
+
validation: { status: 'not-run', semantic_review: semanticReviewNotRun() },
|
|
307
|
+
timestamps: { ...lifecycleTimestamps({ tasks: false }), validation_done: null },
|
|
308
|
+
ready_for_implementation: false,
|
|
309
|
+
workflow_policy: policy,
|
|
310
|
+
}, null, 2));
|
|
311
|
+
return specDir;
|
|
312
|
+
}
|
|
313
|
+
|
|
314
|
+
function legacyTasklessSpec20State(name, policy) {
|
|
315
|
+
return {
|
|
316
|
+
schema_version: '2.0',
|
|
317
|
+
feature_name: name,
|
|
318
|
+
created_at: '2026-08-11T00:00:00+07:00',
|
|
319
|
+
updated_at: '2026-08-11T00:03:00+07:00',
|
|
320
|
+
language: 'en',
|
|
321
|
+
status: 'in_progress',
|
|
322
|
+
current_phase: 'design',
|
|
323
|
+
scope_lock: {
|
|
324
|
+
source: 'legacy fixture',
|
|
325
|
+
in_scope: [],
|
|
326
|
+
out_of_scope: [],
|
|
327
|
+
expansion_policy: 'requires-user-approval',
|
|
328
|
+
},
|
|
329
|
+
coordination: {
|
|
330
|
+
tasks_required: false,
|
|
331
|
+
phases_required: false,
|
|
332
|
+
reason: 'taskless_requested',
|
|
333
|
+
},
|
|
334
|
+
validation: { status: 'not-run', semantic_review: semanticReviewNotRun() },
|
|
335
|
+
timestamps: { ...lifecycleTimestamps({ tasks: false }), validation_done: null },
|
|
336
|
+
ready_for_implementation: false,
|
|
337
|
+
workflow_policy: policy,
|
|
338
|
+
};
|
|
21
339
|
}
|
|
22
340
|
|
|
23
|
-
function
|
|
24
|
-
|
|
25
|
-
|
|
341
|
+
function createLegacyTasklessSpec20(root, name, policy) {
|
|
342
|
+
const specDir = path.join(root, 'specs', name);
|
|
343
|
+
write(
|
|
344
|
+
path.join(specDir, 'requirements.md'),
|
|
345
|
+
'# Requirements\n\n### Requirement 1: Legacy baseline\n\n- **R1.1** The fixture shall preserve one deterministic result.\n',
|
|
346
|
+
);
|
|
347
|
+
write(path.join(specDir, 'design.md'), designFixture());
|
|
348
|
+
write(
|
|
349
|
+
path.join(specDir, 'spec.json'),
|
|
350
|
+
JSON.stringify(legacyTasklessSpec20State(name, policy), null, 2),
|
|
351
|
+
);
|
|
352
|
+
return specDir;
|
|
353
|
+
}
|
|
354
|
+
|
|
355
|
+
function output(result) {
|
|
356
|
+
return `${result.stdout}\n${result.stderr}`;
|
|
357
|
+
}
|
|
358
|
+
|
|
359
|
+
function completeScaffoldedSpec(specDir) {
|
|
360
|
+
const state = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
361
|
+
for (const [index, taskFile] of (state.task_files || []).entries()) {
|
|
362
|
+
const entry = state.task_registry[taskFile];
|
|
363
|
+
write(path.join(specDir, taskFile), task({
|
|
364
|
+
id: entry.id,
|
|
365
|
+
related: `\`src/${index === 0 ? 'one' : `task-${index + 1}`}.js\` | Read`,
|
|
366
|
+
dependencies: entry.dependencies.length > 0 ? entry.dependencies.join(',') : 'none',
|
|
367
|
+
}));
|
|
368
|
+
}
|
|
369
|
+
write(path.join(specDir, 'requirements.md'), '# Requirements\n\n### Requirement 1: Artifact\n\n- **R1.1** The task is verifiable.\n');
|
|
370
|
+
write(
|
|
371
|
+
path.join(specDir, 'design.md'),
|
|
372
|
+
state.workflow_policy?.assurance_level === 'Strict' ? criticalNoObligationDesign() : designFixture(),
|
|
373
|
+
);
|
|
374
|
+
if (state.research === 'research.md') {
|
|
375
|
+
write(path.join(specDir, 'research.md'), '# Research\n\n## Evidence Summary\n- Producer fixture.\n');
|
|
376
|
+
}
|
|
377
|
+
return state;
|
|
378
|
+
}
|
|
379
|
+
|
|
380
|
+
test('canonical legacy v2.0 taskless migration fixture is valid without execution evidence', () => {
|
|
381
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-legacy-taskless-baseline-'));
|
|
382
|
+
try {
|
|
383
|
+
const specDir = createLegacyTasklessSpec20(
|
|
384
|
+
root,
|
|
385
|
+
'legacy-taskless-baseline',
|
|
386
|
+
POLICY.workflowPolicySnapshot({ riskSignals: {} }),
|
|
387
|
+
);
|
|
388
|
+
const state = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
389
|
+
assert.equal(state.schema_version, '2.0');
|
|
390
|
+
assert.equal(state.ready_for_implementation, false);
|
|
391
|
+
assert.equal(Object.hasOwn(state, 'task_files'), false);
|
|
392
|
+
assert.equal(Object.hasOwn(state, 'task_registry'), false);
|
|
393
|
+
assert.equal(fs.existsSync(path.join(specDir, 'research.md')), false);
|
|
394
|
+
assert.equal(fs.existsSync(path.join(specDir, 'receipts')), false);
|
|
395
|
+
state.override_receipt = { legacy: true };
|
|
396
|
+
write(path.join(specDir, 'spec.json'), JSON.stringify(state, null, 2));
|
|
397
|
+
const validation = run(VALIDATOR, [specDir], ROOT);
|
|
398
|
+
assert.equal(validation.status, 0, output(validation));
|
|
399
|
+
} finally {
|
|
400
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
401
|
+
}
|
|
402
|
+
});
|
|
403
|
+
|
|
404
|
+
function completeScaffoldedSpec21(specDir) {
|
|
405
|
+
const statePath = path.join(specDir, 'spec.json');
|
|
406
|
+
let state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
407
|
+
const fixtureRoot = path.dirname(path.dirname(specDir));
|
|
408
|
+
const criteria = (state.task_files || []).map((_, index) => `R1.${index + 1}`);
|
|
409
|
+
write(path.join(fixtureRoot, 'test/one.test.js'), "require('node:test')('one', () => {});\n");
|
|
410
|
+
fs.mkdirSync(path.join(fixtureRoot, 'artifacts'), { recursive: true });
|
|
411
|
+
for (const [index, taskFile] of (state.task_files || []).entries()) {
|
|
412
|
+
const entry = state.task_registry[taskFile];
|
|
413
|
+
const ownedPath = `src/task-${index + 1}.js`;
|
|
414
|
+
const criterion = criteria[index];
|
|
415
|
+
const taskRole = index === 0 ? 'subject' : 'verifier';
|
|
416
|
+
write(path.join(fixtureRoot, ownedPath), 'module.exports = { status: 1 };\n');
|
|
417
|
+
write(path.join(specDir, taskFile), `# Task ${entry.id}: ${entry.title}
|
|
418
|
+
**Status:** pending
|
|
419
|
+
|
|
420
|
+
## Outcome
|
|
421
|
+
Return observable status value 1 through the real fixture entrypoint.
|
|
422
|
+
|
|
423
|
+
## Scope
|
|
424
|
+
- **In scope:** Exact behavior for ${ownedPath}.
|
|
425
|
+
- **Out of scope:** Unrelated runtime behavior.
|
|
426
|
+
|
|
427
|
+
## Anchors and Ownership
|
|
428
|
+
| ID | Type | Target | Role | Access | Action |
|
|
429
|
+
|---|---|---|---|---|---|
|
|
430
|
+
| A-${entry.id}-01 | file | \`${ownedPath}\` | owner | write | modify |
|
|
431
|
+
${taskRole === 'verifier' ? `| A-${entry.id}-02 | artifact | \`artifacts/${entry.id}.json\` | verifier | write | create |` : ''}
|
|
432
|
+
|
|
433
|
+
## Changes
|
|
434
|
+
- [ ] ${taskRole === 'verifier' ? 'Implement the separate proof behavior.' : 'Implement the exact owned behavior.'} _Requirements: ${criterion.slice(1)}_
|
|
435
|
+
|
|
436
|
+
## Acceptance
|
|
437
|
+
- **${criterion}:** ${taskRole === 'verifier' ? 'The focused proof reports status value 1 and rejects an invalid result.' : 'The fixture returns status value 1 and rejects invalid input.'}
|
|
438
|
+
|
|
439
|
+
## Dependencies
|
|
440
|
+
- ${entry.dependencies.length > 0 ? entry.dependencies.join(', ') : 'none'}
|
|
441
|
+
|
|
442
|
+
## Verification Plan
|
|
443
|
+
- **Verification ref:** V1
|
|
444
|
+
- **Task role:** ${taskRole}
|
|
445
|
+
- **Command:** \`node --test test/one.test.js\`
|
|
446
|
+
- **Expected:** Exit code 0 and exact status value 1.
|
|
447
|
+
- **Negative path:** Invalid input returns rejected state.
|
|
448
|
+
- **Reachability:** \`${ownedPath}\`
|
|
449
|
+
`);
|
|
450
|
+
}
|
|
451
|
+
write(path.join(specDir, 'requirements.md'), `# Requirements
|
|
452
|
+
|
|
453
|
+
### Requirement 1: Artifact
|
|
454
|
+
|
|
455
|
+
${criteria.map((criterion, index) => `- **${criterion}**: Task ${index + 1} shall expose its distinct observable status or proof outcome and reject invalid input.`).join('\n')}
|
|
456
|
+
`);
|
|
457
|
+
write(path.join(specDir, 'design.md'), `# Design
|
|
458
|
+
|
|
459
|
+
## Architecture
|
|
460
|
+
The fixture entrypoint delegates to the owned task boundary.
|
|
461
|
+
|
|
462
|
+
## Typed Anchors
|
|
463
|
+
| ID | Type | Target | Role | Access | Action |
|
|
464
|
+
|---|---|---|---|---|---|
|
|
465
|
+
| A-D-01 | command | \`node --test test/one.test.js\` | verification command | read | read |
|
|
466
|
+
|
|
467
|
+
## Canonical Contracts & Invariants
|
|
468
|
+
### D1 — Delegation decision
|
|
469
|
+
The entrypoint delegates exactly once.
|
|
470
|
+
### I1 — Invalid-state invariant
|
|
471
|
+
Invalid input never returns status value 1.
|
|
472
|
+
### C1 — Result contract
|
|
473
|
+
The result contains a numeric status.
|
|
474
|
+
### T1 — Artifact transition
|
|
475
|
+
The transition records success, failure, and recovery states.
|
|
476
|
+
|
|
477
|
+
## Verification Definitions
|
|
478
|
+
- **V1**: Subject criteria ${criteria[0]}; Subject owner ${state.task_registry[state.task_files[0]].id}; Proof criteria ${criteria.slice(1).join(', ')}; Proof owner ${state.task_registry[state.task_files[1]].id}; Evidence anchor A-${state.task_registry[state.task_files[1]].id}-02; Decision refs D1, I1, C1; Method command \`node --test test/one.test.js\`; Expected exit code 0 and exact status value 1; Negative/failure invalid input returns rejected state; Reachability/grounding entrypoint \`src/task-1.js\` via A-D-01, A-${state.task_registry[state.task_files[1]].id}-02.
|
|
479
|
+
`);
|
|
480
|
+
const promoted = run(
|
|
481
|
+
SCAFFOLD,
|
|
482
|
+
[path.basename(specDir), '--sync-semantic-model'],
|
|
483
|
+
path.dirname(path.dirname(specDir)),
|
|
484
|
+
);
|
|
485
|
+
assert.equal(promoted.status, 0, output(promoted));
|
|
486
|
+
state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
487
|
+
state.authoring.requirements = 'validated';
|
|
488
|
+
state.authoring.design = 'validated';
|
|
489
|
+
state.authoring.tasks = state.task_files?.length ? 'validated' : 'absent';
|
|
490
|
+
state.validation.status = 'completed';
|
|
491
|
+
state.validation.semantic_review = {
|
|
492
|
+
status: 'completed', semantic_digest: null, reviewed_criteria: criteria,
|
|
493
|
+
counterexamples: criteria.map((criterion, index) => ({
|
|
494
|
+
criterion, case_kind: index % 2 === 0 ? 'failure' : 'adversarial',
|
|
495
|
+
scenario: `Task ${index + 1} receives a distinct malformed status payload at its declared boundary.`,
|
|
496
|
+
expected: `Task ${index + 1} rejects that payload without publishing its distinct success outcome.`,
|
|
497
|
+
decision_refs: ['D1', 'I1', 'C1'], verification_ref: 'V1',
|
|
498
|
+
})),
|
|
499
|
+
};
|
|
500
|
+
write(statePath, JSON.stringify(state, null, 2));
|
|
501
|
+
state.validation.semantic_review.semantic_digest = semanticDigest(specDir);
|
|
502
|
+
state.ready_for_implementation = true;
|
|
503
|
+
write(statePath, JSON.stringify(state, null, 2));
|
|
504
|
+
return state;
|
|
505
|
+
}
|
|
506
|
+
|
|
507
|
+
test('Codex installer transform preserves all owned spec assets with idempotent runtime parity', () => {
|
|
508
|
+
for (const relative of [
|
|
509
|
+
'src/claude/skills/specs/SKILL.md',
|
|
510
|
+
'src/claude/skills/specs/rules/tasks-generation.md',
|
|
511
|
+
'src/claude/skills/specs/rules/design-principles.md',
|
|
512
|
+
'src/claude/skills/specs/references/review.md',
|
|
513
|
+
'src/claude/skills/specs/templates/design.md',
|
|
514
|
+
'src/claude/skills/specs/templates/task.md',
|
|
515
|
+
'src/claude/scripts/spec-scaffold.cjs',
|
|
516
|
+
'src/claude/scripts/spec-receipt.cjs',
|
|
517
|
+
'src/claude/scripts/validate-spec-output.cjs',
|
|
518
|
+
]) {
|
|
519
|
+
const source = fs.readFileSync(path.join(ROOT, relative), 'utf8');
|
|
520
|
+
const transformed = normalizeCodexBody(source, relative);
|
|
521
|
+
assert.equal(normalizeCodexBody(transformed, relative), transformed, `${relative} must normalize idempotently`);
|
|
522
|
+
assert.doesNotMatch(transformed, /node \.claude\/scripts\//, `${relative} retained a Claude executable path`);
|
|
523
|
+
if (/node \.claude\/scripts\//.test(source)) {
|
|
524
|
+
assert.match(transformed, /node \.codex\/scripts\//, `${relative} lost its Codex executable path`);
|
|
525
|
+
}
|
|
526
|
+
if (/implementation-obligation:/.test(source)) {
|
|
527
|
+
assert.match(transformed, /implementation-obligation:/, `${relative} lost closure semantics`);
|
|
528
|
+
}
|
|
529
|
+
}
|
|
530
|
+
|
|
531
|
+
const skill = fs.readFileSync(path.join(ROOT, 'src/claude/skills/specs/SKILL.md'), 'utf8');
|
|
532
|
+
const review = fs.readFileSync(path.join(ROOT, 'src/claude/skills/specs/references/review.md'), 'utf8');
|
|
533
|
+
const designRules = fs.readFileSync(path.join(ROOT, 'src/claude/skills/specs/rules/design-principles.md'), 'utf8');
|
|
534
|
+
const taskRules = fs.readFileSync(path.join(ROOT, 'src/claude/skills/specs/rules/tasks-generation.md'), 'utf8');
|
|
535
|
+
assert.match(skill, /\| `Compact` \| `spec\.json`, `requirements\.md`, `design\.md` \| only on a research trigger \| only on a typed topology trigger \|/);
|
|
536
|
+
assert.match(skill, /\| `Full` \| `spec\.json`, `requirements\.md`, `design\.md` \| only on a research trigger \| only on a typed topology trigger \|/);
|
|
537
|
+
assert.doesNotMatch(review, /Standard[^\n]{0,100}(?:must not|forbidden|never)[^\n]{0,60}tasks?/i);
|
|
538
|
+
assert.match(designRules, /## Conditional detail[\s\S]*Migration:[\s\S]*only when existing state or public contracts change/);
|
|
539
|
+
assert.match(taskRules, /distinct ownership, a durable[\s\S]{0,180}parallel coordination/);
|
|
540
|
+
assert.match(taskRules, /receipts\/<task-basename>\.md|execution closeout/);
|
|
541
|
+
|
|
542
|
+
const parityRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-codex-validator-parity-'));
|
|
543
|
+
try {
|
|
544
|
+
const installedScripts = path.join(parityRoot, '.codex/scripts');
|
|
545
|
+
const installedValidator = path.join(installedScripts, 'validate-spec-output.cjs');
|
|
546
|
+
copyCodexRuntime(path.dirname(installedScripts), ['scripts/validate-spec-output.cjs']);
|
|
547
|
+
|
|
548
|
+
const scaffolded = run(SCAFFOLD, ['runtime-parity'], parityRoot);
|
|
549
|
+
assert.equal(scaffolded.status, 0, output(scaffolded));
|
|
550
|
+
const specDir = path.join(parityRoot, 'specs/runtime-parity');
|
|
551
|
+
write(path.join(specDir, 'requirements.md'), '# Requirements\n\n### Requirement 1: One\n\n- **R1.1** The runtime shall return status value 1.\n- **R1.2** The proof shall reject an invalid status result.\n');
|
|
552
|
+
write(path.join(specDir, 'design.md'), designFixture21());
|
|
553
|
+
const promoted = run(SCAFFOLD, ['runtime-parity', '--sync-semantic-model'], parityRoot);
|
|
554
|
+
assert.equal(promoted.status, 0, output(promoted));
|
|
555
|
+
const sourcePending = run(VALIDATOR, [specDir], ROOT);
|
|
556
|
+
const codexPending = run(installedValidator, [specDir], ROOT);
|
|
557
|
+
assert.equal(sourcePending.status, 0, output(sourcePending));
|
|
558
|
+
assert.equal(codexPending.status, sourcePending.status, output(codexPending));
|
|
559
|
+
|
|
560
|
+
const statePath = path.join(specDir, 'spec.json');
|
|
561
|
+
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
562
|
+
state.status = 'paused';
|
|
563
|
+
writeRaw(statePath, JSON.stringify(state, null, 2));
|
|
564
|
+
const sourceDonePending = run(VALIDATOR, [specDir], ROOT);
|
|
565
|
+
const codexDonePending = run(installedValidator, [specDir], ROOT);
|
|
566
|
+
assert.equal(sourceDonePending.status, 0, output(sourceDonePending));
|
|
567
|
+
assert.equal(codexDonePending.status, sourceDonePending.status, output(codexDonePending));
|
|
568
|
+
} finally {
|
|
569
|
+
fs.rmSync(parityRoot, { recursive: true, force: true });
|
|
570
|
+
}
|
|
571
|
+
});
|
|
572
|
+
|
|
573
|
+
test('resolver binds feature identity and rejects cross-feature spec aliases', () => {
|
|
574
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-resolver-identity-'));
|
|
575
|
+
try {
|
|
576
|
+
const specsDir = path.join(root, 'specs');
|
|
577
|
+
write(path.join(specsDir, 'bar', 'spec.json'), JSON.stringify({ feature_name: 'bar', status: 'in_progress' }));
|
|
578
|
+
fs.mkdirSync(path.join(specsDir, 'foo'), { recursive: true });
|
|
579
|
+
fs.symlinkSync('../bar/spec.json', path.join(specsDir, 'foo', 'spec.json'));
|
|
580
|
+
|
|
581
|
+
const explicit = SPEC_RESOLVER.resolveActiveSpec({ projectRoot: root, explicitFeature: 'foo' });
|
|
582
|
+
assert.equal(explicit.error, 'explicit_malformed');
|
|
583
|
+
assert.match(explicit.reason, /does not belong to feature directory|canonical feature/);
|
|
584
|
+
|
|
585
|
+
const implicit = SPEC_RESOLVER.resolveActiveSpec({ projectRoot: root });
|
|
586
|
+
assert.equal(implicit.error, 'invalid_specs');
|
|
587
|
+
assert.deepEqual(implicit.candidates, ['foo']);
|
|
588
|
+
assert.doesNotMatch(implicit.reason, /Multiple active specs/);
|
|
589
|
+
assert.throws(
|
|
590
|
+
() => SPEC_RESOLVER.findAllActiveSpecs(root),
|
|
591
|
+
(error) => error.code === 'INVALID_SPECS' && /foo/.test(error.message),
|
|
592
|
+
);
|
|
593
|
+
|
|
594
|
+
const aliasRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-resolver-directory-alias-'));
|
|
595
|
+
try {
|
|
596
|
+
const aliasSpecs = path.join(aliasRoot, 'specs');
|
|
597
|
+
write(path.join(aliasSpecs, 'bar', 'spec.json'), JSON.stringify({ feature_name: 'bar', status: 'in_progress' }));
|
|
598
|
+
fs.symlinkSync('bar', path.join(aliasSpecs, 'foo'), 'dir');
|
|
599
|
+
const resolved = SPEC_RESOLVER.resolveActiveSpec({ projectRoot: aliasRoot });
|
|
600
|
+
assert.equal(resolved.featureName, 'bar');
|
|
601
|
+
assert.equal(SPEC_RESOLVER.findAllActiveSpecs(aliasRoot).length, 1);
|
|
602
|
+
} finally {
|
|
603
|
+
fs.rmSync(aliasRoot, { recursive: true, force: true });
|
|
604
|
+
}
|
|
605
|
+
} finally {
|
|
606
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
607
|
+
}
|
|
608
|
+
});
|
|
609
|
+
|
|
610
|
+
test('validator rejects hidden task symlinks instead of filtering them', () => {
|
|
611
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-hidden-task-link-'));
|
|
612
|
+
try {
|
|
613
|
+
const specDir = createSpec(root, { name: 'hidden-task-link' });
|
|
614
|
+
fs.symlinkSync('task-R1-01-one.md', path.join(specDir, 'tasks', '.hidden-task.md'));
|
|
615
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
616
|
+
assert.notEqual(result.status, 0);
|
|
617
|
+
assert.match(output(result), /tasks\/\.hidden-task\.md: symlink is not allowed/);
|
|
618
|
+
} finally {
|
|
619
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
620
|
+
}
|
|
621
|
+
});
|
|
622
|
+
|
|
623
|
+
test('validator rejects non-Markdown task artifacts instead of ignoring them', () => {
|
|
624
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-task-artifact-'));
|
|
625
|
+
try {
|
|
626
|
+
const specDir = createSpec(root, { name: 'non-markdown-task-artifact' });
|
|
627
|
+
write(path.join(specDir, 'tasks', 'notes.txt'), 'worker note\n');
|
|
628
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
629
|
+
assert.notEqual(result.status, 0);
|
|
630
|
+
assert.match(output(result), /tasks\/notes\.txt: unexpected task artifact/);
|
|
631
|
+
} finally {
|
|
632
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
633
|
+
}
|
|
634
|
+
});
|
|
635
|
+
|
|
636
|
+
test('validator keeps authoring receipt-free and validates taskless execution closeout proof', () => {
|
|
637
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-feature-receipt-'));
|
|
638
|
+
try {
|
|
639
|
+
const specDir = createBoundedSpec(root, 'weak-completed-feature-receipt');
|
|
640
|
+
write(path.join(specDir, 'feature-receipt.md'), [
|
|
641
|
+
'# Feature Verification Receipt', '', 'Verification: PASS',
|
|
642
|
+
'Command: npm test', 'Exit: 0', 'Base: a', 'Head: b',
|
|
643
|
+
].join('\n'));
|
|
644
|
+
const premature = run(VALIDATOR, [specDir], ROOT);
|
|
645
|
+
assert.notEqual(premature.status, 0);
|
|
646
|
+
assert.match(output(premature), /completed execution receipt is valid only on terminal status done, never for readiness/);
|
|
647
|
+
|
|
648
|
+
const weakSpecPath = path.join(specDir, 'spec.json');
|
|
649
|
+
const weakSpec = JSON.parse(fs.readFileSync(weakSpecPath, 'utf8'));
|
|
650
|
+
weakSpec.status = 'done';
|
|
651
|
+
weakSpec.current_phase = 'done';
|
|
652
|
+
write(weakSpecPath, JSON.stringify(weakSpec, null, 2));
|
|
653
|
+
const weak = run(VALIDATOR, [specDir], ROOT);
|
|
654
|
+
assert.notEqual(weak.status, 0);
|
|
655
|
+
assert.match(output(weak), /completed proof and spec\.json must both use status done/);
|
|
656
|
+
assert.match(output(weak), /observable expected result/);
|
|
657
|
+
assert.match(output(weak), /distinct 40- or 64-hex Base and Head bindings/);
|
|
658
|
+
|
|
659
|
+
const completedDir = createBoundedSpec(root, 'completed-feature-receipt');
|
|
660
|
+
const completedSpecPath = path.join(completedDir, 'spec.json');
|
|
661
|
+
const completedSpec = JSON.parse(fs.readFileSync(completedSpecPath, 'utf8'));
|
|
662
|
+
completedSpec.status = 'done';
|
|
663
|
+
completedSpec.current_phase = 'done';
|
|
664
|
+
write(completedSpecPath, JSON.stringify(completedSpec, null, 2));
|
|
665
|
+
write(path.join(completedDir, 'feature-receipt.md'), [
|
|
666
|
+
'# Feature Verification Receipt', '', 'Verification: PASS', 'Status: done',
|
|
667
|
+
'Command: `node --test test/one.test.js`',
|
|
668
|
+
'Expected result: exit code 0 and one test passes',
|
|
669
|
+
'Observed result: exit code 0 and one test passed',
|
|
670
|
+
'Artifact/runtime proof: `src/one.js` persisted state value 1',
|
|
671
|
+
`Base: ${'a'.repeat(40)}`,
|
|
672
|
+
`Head: ${'b'.repeat(40)}`,
|
|
673
|
+
].join('\n'));
|
|
674
|
+
const completed = run(VALIDATOR, [completedDir], ROOT);
|
|
675
|
+
assert.equal(completed.status, 0, output(completed));
|
|
676
|
+
|
|
677
|
+
} finally {
|
|
678
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
679
|
+
}
|
|
680
|
+
});
|
|
681
|
+
|
|
682
|
+
test('validator rejects required and optional spec artifacts that symlink outside the spec', () => {
|
|
683
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-artifact-links-'));
|
|
684
|
+
const external = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-external-artifacts-'));
|
|
685
|
+
try {
|
|
686
|
+
const required = createSpec(root, { name: 'required-artifact-link' });
|
|
687
|
+
fs.unlinkSync(path.join(required, 'requirements.md'));
|
|
688
|
+
fs.symlinkSync(path.join(external, 'requirements.md'), path.join(required, 'requirements.md'));
|
|
689
|
+
write(path.join(external, 'requirements.md'), '# External requirements\n');
|
|
690
|
+
const requiredResult = run(VALIDATOR, [required], ROOT);
|
|
691
|
+
assert.notEqual(requiredResult.status, 0);
|
|
692
|
+
assert.match(output(requiredResult), /requirements\.md: symlink is not allowed/);
|
|
693
|
+
|
|
694
|
+
const optional = createSpec(root, { name: 'optional-artifact-link' });
|
|
695
|
+
const optionalSpecPath = path.join(optional, 'spec.json');
|
|
696
|
+
const optionalSpec = JSON.parse(fs.readFileSync(optionalSpecPath, 'utf8'));
|
|
697
|
+
optionalSpec.research = 'research.md';
|
|
698
|
+
write(optionalSpecPath, JSON.stringify(optionalSpec, null, 2));
|
|
699
|
+
write(path.join(external, 'research.md'), '# External research\n');
|
|
700
|
+
fs.symlinkSync(path.join(external, 'research.md'), path.join(optional, 'research.md'));
|
|
701
|
+
const optionalResult = run(VALIDATOR, [optional], ROOT);
|
|
702
|
+
assert.notEqual(optionalResult.status, 0);
|
|
703
|
+
assert.match(output(optionalResult), /research\.md: symlink is not allowed/);
|
|
704
|
+
|
|
705
|
+
const bounded = createBoundedSpec(root, 'receipt-artifact-link');
|
|
706
|
+
write(path.join(external, 'feature-receipt.md'), '# External receipt\n');
|
|
707
|
+
fs.symlinkSync(path.join(external, 'feature-receipt.md'), path.join(bounded, 'feature-receipt.md'));
|
|
708
|
+
const boundedResult = run(VALIDATOR, [bounded], ROOT);
|
|
709
|
+
assert.notEqual(boundedResult.status, 0);
|
|
710
|
+
assert.match(output(boundedResult), /feature-receipt\.md: symlink is not allowed/);
|
|
711
|
+
} finally {
|
|
712
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
713
|
+
fs.rmSync(external, { recursive: true, force: true });
|
|
714
|
+
}
|
|
715
|
+
});
|
|
716
|
+
|
|
717
|
+
test('Standard bounded readiness requires canonical testable requirements and a concrete design boundary', () => {
|
|
718
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-standard-placeholders-'));
|
|
719
|
+
try {
|
|
720
|
+
const specDir = createBoundedSpec(root, 'standard-placeholders');
|
|
721
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
722
|
+
const spec = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
723
|
+
spec.ready_for_implementation = true;
|
|
724
|
+
spec.approvals.requirements.user_approved = true;
|
|
725
|
+
spec.approvals.design.user_approved = true;
|
|
726
|
+
spec.validation = { status: 'completed' };
|
|
727
|
+
spec.updated_at = '2026-08-11T00:06:00+07:00';
|
|
728
|
+
spec.timestamps = lifecycleTimestamps({ tasks: false });
|
|
729
|
+
spec.scope_lock.source = '{{PROJECT_DESCRIPTION}}';
|
|
730
|
+
fs.writeFileSync(specPath, JSON.stringify(spec, null, 2));
|
|
731
|
+
write(path.join(specDir, 'requirements.md'), '# Requirements\n\n{{REQUIREMENTS}}\n');
|
|
732
|
+
write(path.join(specDir, 'design.md'), '# Design\n\nTBD\n');
|
|
733
|
+
|
|
734
|
+
const rejected = run(VALIDATOR, [specDir], ROOT);
|
|
735
|
+
assert.notEqual(rejected.status, 0);
|
|
736
|
+
assert.match(output(rejected), /scope_lock\.source.*concrete scope source/);
|
|
737
|
+
assert.match(output(rejected), /requirements\.md: Standard readiness rejects placeholders/);
|
|
738
|
+
assert.match(output(rejected), /design\.md: Standard readiness rejects placeholders/);
|
|
739
|
+
assert.doesNotMatch(output(rejected), /feature-receipt\.md: canonical feature receipt required/);
|
|
740
|
+
|
|
741
|
+
spec.scope_lock.source = 'The bounded feature scope is the one described here';
|
|
742
|
+
fs.writeFileSync(specPath, JSON.stringify(spec, null, 2));
|
|
743
|
+
write(path.join(specDir, 'requirements.md'), '# Requirements\n\nThe bounded behavior is explicitly defined.\n');
|
|
744
|
+
write(path.join(specDir, 'design.md'), '# Design\n\nThe bounded implementation boundary is explicitly defined.\n');
|
|
745
|
+
const generic = run(VALIDATOR, [specDir], ROOT);
|
|
746
|
+
assert.notEqual(generic.status, 0);
|
|
747
|
+
assert.match(output(generic), /canonical numeric requirement id/);
|
|
748
|
+
assert.match(output(generic), /testable acceptance criterion using RN\.M/);
|
|
749
|
+
assert.match(output(generic), /grounded path\/code identifier/);
|
|
750
|
+
|
|
751
|
+
write(
|
|
752
|
+
path.join(specDir, 'requirements.md'),
|
|
753
|
+
'# Requirements\n\n## Requirements\n\n### Requirement 1: Locale persistence\n\n' +
|
|
754
|
+
'- **R1.1** When a locale is saved, the CLI shall write the selected value to `.codex/runtime.json` and return exit code 0.\n',
|
|
755
|
+
);
|
|
756
|
+
write(
|
|
757
|
+
path.join(specDir, 'design.md'),
|
|
758
|
+
'# Design\n\n## Boundary\n\nThe `.codex/runtime.json` file is owned by `saveLocale`; the CLI calls it before returning success.\n\n' +
|
|
759
|
+
'## Typed Anchors\n\n| ID | Type | Target | Role |\n|---|---|---|---|\n' +
|
|
760
|
+
'| A-D-01 | file | `.codex/runtime.json` | locale persistence target |\n' +
|
|
761
|
+
'| A-D-02 | symbol | `src/runtime.js#saveLocale` | locale writer |\n\n' +
|
|
762
|
+
'## Decisions and Invariants\n\n### D1 — Persist selected locale\n\n- **Decision:** `saveLocale` writes the selected value.\n- **Negative path:** Invalid locale leaves state unchanged.\n- **Anchors:** A-D-01, A-D-02\n\n' +
|
|
763
|
+
'## Verification\n\n| Requirement | Proof target | Expected result | Negative path / reachability |\n|---|---|---|---|\n' +
|
|
764
|
+
'| R1.1 | `node --test test/runtime.test.js` | exit code 0 and saved locale | invalid locale through `saveLocale` |\n',
|
|
765
|
+
);
|
|
766
|
+
completeSemanticReview(specDir);
|
|
767
|
+
const accepted = run(VALIDATOR, [specDir], ROOT);
|
|
768
|
+
assert.equal(accepted.status, 0, output(accepted));
|
|
769
|
+
} finally {
|
|
770
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
771
|
+
}
|
|
772
|
+
});
|
|
773
|
+
|
|
774
|
+
test('tasks-only validates the persisted policy before any filesystem mutation', () => {
|
|
775
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-tasks-only-preflight-'));
|
|
776
|
+
try {
|
|
777
|
+
const standardPolicy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
778
|
+
const boundaries = JSON.stringify([{
|
|
779
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01'],
|
|
780
|
+
write_sets: { 'R1-01': ['src/one.js'] },
|
|
781
|
+
}]);
|
|
782
|
+
const cases = [
|
|
783
|
+
['malformed', () => '{', /existing spec\.json is invalid JSON/],
|
|
784
|
+
['null-state', () => 'null', /existing spec\.json must contain a JSON object/],
|
|
785
|
+
['missing-snapshot', (state) => { delete state.workflow_policy; return JSON.stringify(state, null, 2); }, /existing spec must contain a persisted workflow_policy/],
|
|
786
|
+
['null-snapshot', (state) => { state.workflow_policy = null; return JSON.stringify(state, null, 2); }, /existing workflow_policy is invalid/],
|
|
787
|
+
['extra-field', (state) => {
|
|
788
|
+
state.workflow_policy = { ...standardPolicy, extra: true };
|
|
789
|
+
return JSON.stringify(state, null, 2);
|
|
790
|
+
}, /schema 2\.0 workflow_policy contains unsupported legacy authority field\(s\): extra/],
|
|
791
|
+
];
|
|
792
|
+
for (const [name, mutate, expected] of cases) {
|
|
793
|
+
const specDir = createLegacyTasklessSpec20(root, name, standardPolicy);
|
|
794
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
795
|
+
const state = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
796
|
+
write(specPath, mutate(state));
|
|
797
|
+
const before = fs.readFileSync(specPath, 'utf8');
|
|
798
|
+
const result = run(SCAFFOLD, [
|
|
799
|
+
name, '--tasks', 'R1-01-one', '--tasks-only', '--boundaries', boundaries,
|
|
800
|
+
], root);
|
|
801
|
+
assert.equal(result.status, 2, output(result));
|
|
802
|
+
assert.match(output(result), expected);
|
|
803
|
+
assert.equal(fs.readFileSync(specPath, 'utf8'), before);
|
|
804
|
+
assert.equal(fs.existsSync(path.join(specDir, 'tasks')), false);
|
|
805
|
+
assert.equal(fs.existsSync(path.join(specDir, 'research.md')), false);
|
|
806
|
+
}
|
|
807
|
+
} finally {
|
|
808
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
809
|
+
}
|
|
810
|
+
});
|
|
811
|
+
|
|
812
|
+
test('scaffold accepts only one safe alphanumeric-starting feature segment', () => {
|
|
813
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-feature-name-'));
|
|
814
|
+
try {
|
|
815
|
+
for (const feature of ['.', 'foo/bar', 'foo\\bar', '../escape', '..\\escape']) {
|
|
816
|
+
const before = snapshotTree(root);
|
|
817
|
+
const result = run(SCAFFOLD, [feature, '--lane', 'Standard'], root);
|
|
818
|
+
assert.equal(result.status, 2, `${feature}: ${output(result)}`);
|
|
819
|
+
assert.match(output(result), /feature|safe relative|path segment/i);
|
|
820
|
+
assert.equal(snapshotTree(root), before, `${feature} must not mutate the workspace`);
|
|
821
|
+
}
|
|
822
|
+
|
|
823
|
+
const beforeFreshSequence = snapshotTree(root);
|
|
824
|
+
const badFreshSequence = run(SCAFFOLD, ['bad-fresh-sequence', '--tasks', 'R1-1-one'], root);
|
|
825
|
+
assert.equal(badFreshSequence.status, 2, output(badFreshSequence));
|
|
826
|
+
assert.match(output(badFreshSequence), /SEQ 2 digits/);
|
|
827
|
+
assert.equal(snapshotTree(root), beforeFreshSequence, 'one-digit fresh task sequence mutated the workspace');
|
|
828
|
+
|
|
829
|
+
createLegacyTasklessSpec20(root, 'bad-tasks-only-sequence', POLICY.workflowPolicySnapshot({ riskSignals: {} }));
|
|
830
|
+
const beforeTasksOnlySequence = snapshotTree(root);
|
|
831
|
+
const badTasksOnlySequence = run(SCAFFOLD, [
|
|
832
|
+
'bad-tasks-only-sequence', '--tasks', 'R1-1-one', '--tasks-only',
|
|
833
|
+
], root);
|
|
834
|
+
assert.equal(badTasksOnlySequence.status, 2, output(badTasksOnlySequence));
|
|
835
|
+
assert.match(output(badTasksOnlySequence), /SEQ 2 digits/);
|
|
836
|
+
assert.equal(snapshotTree(root), beforeTasksOnlySequence, 'one-digit tasks-only sequence mutated the workspace');
|
|
837
|
+
} finally {
|
|
838
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
839
|
+
}
|
|
840
|
+
});
|
|
841
|
+
|
|
842
|
+
test('validator requires a strict persisted workflow-policy snapshot and never falls back to prose or legacy tier', () => {
|
|
843
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-policy-boundary-'));
|
|
844
|
+
try {
|
|
845
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
846
|
+
const cases = [
|
|
847
|
+
['missing', (state) => { delete state.workflow_policy; }, /missing persisted snapshot/],
|
|
848
|
+
['explicit-null', (state) => { state.workflow_policy = null; }, /workflow_policy must be an object/],
|
|
849
|
+
['malformed', (state) => { state.workflow_policy = { ...policy, proof_obligations: { needsExecutionProof: true } }; }, /proof_obligations must be an array/],
|
|
850
|
+
['extra-field', (state) => { state.workflow_policy = { ...policy, extra: true }; }, /workflow_policy fields must be exactly/],
|
|
851
|
+
['lane-mismatch', (state) => { state.workflow_policy = { ...policy, lane: 'Critical', automatic_lane: 'Critical' }; }, /lane is not the derived compatibility lane|automatic_lane is not the derived compatibility lane/],
|
|
852
|
+
['risk-mismatch', (state) => { state.workflow_policy = { ...policy, risks: ['auth'] }; }, /assurance_level must be at least Elevated/],
|
|
853
|
+
['obligation-mismatch', (state) => { state.workflow_policy = { ...policy, proof_obligations: ['needsInspection', 'needsExecutionProof'], actor_needs: [{ capability: 'inspection', independence: 'same-session' }, { capability: 'execution-proof', independence: 'same-session' }] }; }, /proof_obligations do not match assurance_level and risks/],
|
|
854
|
+
];
|
|
855
|
+
for (const [name, mutate, expected] of cases) {
|
|
856
|
+
const specDir = createSpec(root, { name, spec: { workflow_policy: policy } });
|
|
857
|
+
const statePath = path.join(specDir, 'spec.json');
|
|
858
|
+
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
859
|
+
mutate(state);
|
|
860
|
+
write(statePath, JSON.stringify(state, null, 2));
|
|
861
|
+
const beforeRun = snapshotTree(specDir);
|
|
862
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
863
|
+
assert.notEqual(result.status, 0, `${name} unexpectedly passed`);
|
|
864
|
+
assert.match(output(result), expected);
|
|
865
|
+
assert.equal(snapshotTree(specDir), beforeRun, `${name} validator mutated the snapshot`);
|
|
866
|
+
}
|
|
867
|
+
|
|
868
|
+
const legacyDir = createSpec(root, {
|
|
869
|
+
name: 'legacy-ready',
|
|
870
|
+
spec: {
|
|
871
|
+
workflow_policy: undefined,
|
|
872
|
+
design_context: { execution_tier: 'Deep' },
|
|
873
|
+
ready_for_implementation: true,
|
|
874
|
+
approvals: {
|
|
875
|
+
requirements: { generated: true, agent_validated: true, user_approved: true },
|
|
876
|
+
design: { generated: true, agent_validated: true, user_approved: true },
|
|
877
|
+
tasks: { generated: true, agent_validated: true, user_approved: true },
|
|
878
|
+
},
|
|
879
|
+
validation: { status: 'completed' },
|
|
880
|
+
updated_at: '2026-08-11T00:06:00+07:00',
|
|
881
|
+
timestamps: lifecycleTimestamps(),
|
|
882
|
+
},
|
|
883
|
+
});
|
|
884
|
+
const legacyBefore = snapshotTree(legacyDir);
|
|
885
|
+
const legacyResult = run(VALIDATOR, [legacyDir], ROOT);
|
|
886
|
+
assert.notEqual(legacyResult.status, 0);
|
|
887
|
+
assert.match(output(legacyResult), /execution_tier is a read-only legacy adapter/);
|
|
888
|
+
assert.doesNotMatch(output(legacyResult), /^PASS/m);
|
|
889
|
+
assert.equal(snapshotTree(legacyDir), legacyBefore);
|
|
890
|
+
} finally {
|
|
891
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
892
|
+
}
|
|
893
|
+
});
|
|
894
|
+
|
|
895
|
+
test('tasks-only preflights malformed topology, duplicates, and symlinks without mutation', () => {
|
|
896
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-topology-preflight-'));
|
|
897
|
+
try {
|
|
898
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
899
|
+
const taskPath = 'tasks/task-R1-01-one.md';
|
|
900
|
+
const registryEntry = {
|
|
901
|
+
id: 'R1-01', title: 'One', status: 'pending', dependencies: [], blocker: null,
|
|
902
|
+
started_at: null, completed_at: null, last_updated_at: null,
|
|
903
|
+
};
|
|
904
|
+
const cases = [
|
|
905
|
+
['malformed-task-files', { workflow_policy: policy, task_files: 'not-an-array', task_registry: {} }, /task_files must be an array/],
|
|
906
|
+
['malformed-registry', { workflow_policy: policy, task_files: [], task_registry: [] }, /task_registry must be an object/],
|
|
907
|
+
['duplicate-task-files', { workflow_policy: policy, task_files: [taskPath, taskPath], task_registry: { [taskPath]: registryEntry } }, /duplicate path/],
|
|
908
|
+
['unknown-dependency', { workflow_policy: policy, task_files: [taskPath], task_registry: { [taskPath]: { ...registryEntry, dependencies: ['tasks/task-R1-02-two.md'] } } }, /unknown task/],
|
|
909
|
+
['unsafe-research-path', { workflow_policy: policy, research: '../outside' }, /spec\.research must be a safe relative path/],
|
|
910
|
+
];
|
|
911
|
+
for (const [name, state, expected] of cases) {
|
|
912
|
+
const specDir = createLegacyTasklessSpec20(root, name, policy);
|
|
913
|
+
write(path.join(specDir, 'spec.json'), JSON.stringify({
|
|
914
|
+
...legacyTasklessSpec20State(name, policy),
|
|
915
|
+
...state,
|
|
916
|
+
}, null, 2));
|
|
917
|
+
for (const declaredTask of new Set(Array.isArray(state.task_files) ? state.task_files : [])) {
|
|
918
|
+
write(path.join(specDir, declaredTask), task());
|
|
919
|
+
}
|
|
920
|
+
const boundaryTasks = Array.isArray(state.task_files) && state.task_files.includes(taskPath)
|
|
921
|
+
? ['R1-01', 'R1-02', 'R1-03'] : ['R1-02', 'R1-03'];
|
|
922
|
+
const boundaries = JSON.stringify([{
|
|
923
|
+
id: 'B-OWN', type: 'ownership', tasks: boundaryTasks,
|
|
924
|
+
write_sets: Object.fromEntries(boundaryTasks.map((id) => [id, [`src/${id.toLowerCase()}.js`]])),
|
|
925
|
+
}]);
|
|
926
|
+
const before = snapshotTree(root);
|
|
927
|
+
const result = run(SCAFFOLD, [
|
|
928
|
+
name, '--tasks', 'R1-02-two,R1-03-three', '--tasks-only',
|
|
929
|
+
'--boundaries', boundaries,
|
|
930
|
+
], root);
|
|
931
|
+
assert.equal(result.status, 2, output(result));
|
|
932
|
+
assert.match(output(result), expected);
|
|
933
|
+
assert.equal(snapshotTree(root), before, `${name} mutated the existing topology`);
|
|
934
|
+
}
|
|
935
|
+
|
|
936
|
+
const conflictDir = createLegacyTasklessSpec20(root, 'conflicting-duplicate', policy);
|
|
937
|
+
write(path.join(conflictDir, 'tasks/task-R1-01-one.md'), 'already filled\n');
|
|
938
|
+
write(path.join(conflictDir, 'spec.json'), JSON.stringify({
|
|
939
|
+
...legacyTasklessSpec20State('conflicting-duplicate', policy),
|
|
940
|
+
task_files: [taskPath],
|
|
941
|
+
task_registry: { [taskPath]: { ...registryEntry, title: 'Conflicting title' } },
|
|
942
|
+
}, null, 2));
|
|
943
|
+
const conflictBefore = snapshotTree(root);
|
|
944
|
+
const singleBoundary = JSON.stringify([{
|
|
945
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01'], write_sets: { 'R1-01': ['src/one.js'] },
|
|
946
|
+
}]);
|
|
947
|
+
const conflict = run(SCAFFOLD, [
|
|
948
|
+
'conflicting-duplicate', '--tasks', 'R1-01-one', '--tasks-only',
|
|
949
|
+
'--boundaries', singleBoundary,
|
|
950
|
+
], root);
|
|
951
|
+
assert.equal(conflict.status, 2, output(conflict));
|
|
952
|
+
assert.match(output(conflict), /conflicting duplicate/);
|
|
953
|
+
assert.equal(snapshotTree(root), conflictBefore);
|
|
954
|
+
|
|
955
|
+
const duplicateDir = createLegacyTasklessSpec20(root, 'duplicate-cli', policy);
|
|
956
|
+
const duplicateBefore = snapshotTree(root);
|
|
957
|
+
const duplicate = run(SCAFFOLD, [
|
|
958
|
+
'duplicate-cli', '--tasks', 'R1-01-one,R1-01-two', '--tasks-only',
|
|
959
|
+
'--boundaries', singleBoundary,
|
|
960
|
+
], root);
|
|
961
|
+
assert.equal(duplicate.status, 2, output(duplicate));
|
|
962
|
+
assert.match(output(duplicate), /duplicate task id.*R1-01/);
|
|
963
|
+
assert.equal(snapshotTree(root), duplicateBefore);
|
|
964
|
+
|
|
965
|
+
const symlinkDir = createLegacyTasklessSpec20(root, 'symlink-tasks', policy);
|
|
966
|
+
const outside = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-symlink-target-'));
|
|
967
|
+
fs.symlinkSync(outside, path.join(symlinkDir, 'tasks'), 'dir');
|
|
968
|
+
const symlinkBefore = snapshotTree(root);
|
|
969
|
+
const symlinkResult = run(SCAFFOLD, [
|
|
970
|
+
'symlink-tasks', '--tasks', 'R1-01-one', '--tasks-only', '--boundaries', singleBoundary,
|
|
971
|
+
], root);
|
|
972
|
+
assert.equal(symlinkResult.status, 2, output(symlinkResult));
|
|
973
|
+
assert.match(output(symlinkResult), /symlink/);
|
|
974
|
+
assert.equal(snapshotTree(root), symlinkBefore);
|
|
975
|
+
fs.rmSync(outside, { recursive: true, force: true });
|
|
976
|
+
} finally {
|
|
977
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
978
|
+
}
|
|
979
|
+
});
|
|
980
|
+
|
|
981
|
+
test('tasks-only commits transactionally and preserves exact bytes after an injected partial failure', () => {
|
|
982
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-transaction-'));
|
|
983
|
+
try {
|
|
984
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
985
|
+
createLegacyTasklessSpec20(root, 'partial-write', policy);
|
|
986
|
+
const before = snapshotTree(root);
|
|
987
|
+
const result = run(
|
|
988
|
+
SCAFFOLD,
|
|
989
|
+
['partial-write', '--tasks', 'R1-01-one,R1-02-two', '--boundaries', JSON.stringify([{
|
|
990
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02'],
|
|
991
|
+
write_sets: { 'R1-01': ['src/one.js'], 'R1-02': ['src/two.js'] },
|
|
992
|
+
}]), '--tasks-only'],
|
|
993
|
+
root,
|
|
994
|
+
{ env: { CAFEKIT_SCAFFOLD_FAIL_AFTER_WRITES: '1' } },
|
|
995
|
+
);
|
|
996
|
+
assert.equal(result.status, 2, output(result));
|
|
997
|
+
assert.match(output(result), /injected failure.*transaction rolled back/);
|
|
998
|
+
assert.equal(snapshotTree(root), before);
|
|
999
|
+
} finally {
|
|
1000
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1001
|
+
}
|
|
1002
|
+
});
|
|
1003
|
+
|
|
1004
|
+
test('tasks-only migration derives only absent feature identity and rejects present conflicts atomically', () => {
|
|
1005
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-legacy-identity-'));
|
|
1006
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
1007
|
+
const boundaries = JSON.stringify([{
|
|
1008
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02'],
|
|
1009
|
+
write_sets: { 'R1-01': ['src/one.js'], 'R1-02': ['src/two.js'] },
|
|
1010
|
+
}]);
|
|
1011
|
+
const migrate = (feature) => run(SCAFFOLD, [
|
|
1012
|
+
feature, '--tasks', 'R1-01-one,R1-02-two', '--tasks-only', '--boundaries', boundaries,
|
|
1013
|
+
], root);
|
|
1014
|
+
try {
|
|
1015
|
+
const absentFeature = 'legacy-identity-absent';
|
|
1016
|
+
const absentDir = createLegacyTasklessSpec20(root, absentFeature, policy);
|
|
1017
|
+
const absentPath = path.join(absentDir, 'spec.json');
|
|
1018
|
+
const absentState = JSON.parse(fs.readFileSync(absentPath, 'utf8'));
|
|
1019
|
+
delete absentState.feature_name;
|
|
1020
|
+
write(absentPath, JSON.stringify(absentState, null, 2));
|
|
1021
|
+
|
|
1022
|
+
const migrated = migrate(absentFeature);
|
|
1023
|
+
assert.equal(migrated.status, 0, output(migrated));
|
|
1024
|
+
const canonical = JSON.parse(fs.readFileSync(absentPath, 'utf8'));
|
|
1025
|
+
assert.equal(canonical.schema_version, '2.1');
|
|
1026
|
+
assert.equal(canonical.feature_name, absentFeature);
|
|
1027
|
+
assert.equal(canonical.created_at, '2026-08-11T00:00:00+07:00');
|
|
1028
|
+
assert.equal(canonical.language, 'en');
|
|
1029
|
+
assert.equal(canonical.status, 'in_progress');
|
|
1030
|
+
assert.deepEqual(canonical.scope_lock, absentState.scope_lock);
|
|
1031
|
+
assert.deepEqual(canonical.authoring, {
|
|
1032
|
+
requirements: 'draft', design: 'draft', research: 'absent', tasks: 'draft',
|
|
1033
|
+
});
|
|
1034
|
+
assert.equal(canonical.ready_for_implementation, false);
|
|
1035
|
+
assert.equal(canonical.validation.status, 'not-run');
|
|
1036
|
+
assert.equal(canonical.validation.semantic_review.status, 'not-run');
|
|
1037
|
+
assert.equal(Object.hasOwn(canonical, 'research'), false);
|
|
1038
|
+
assert.equal(fs.existsSync(path.join(absentDir, 'research.md')), false);
|
|
1039
|
+
assert.equal(fs.existsSync(path.join(absentDir, 'receipts')), false);
|
|
1040
|
+
assert.equal(fs.existsSync(path.join(absentDir, 'reports')), false);
|
|
1041
|
+
for (const entry of Object.values(canonical.task_registry)) {
|
|
1042
|
+
assert.deepEqual(
|
|
1043
|
+
[entry.status, entry.blocker, entry.started_at, entry.completed_at, entry.last_updated_at],
|
|
1044
|
+
['pending', null, null, null, null],
|
|
1045
|
+
);
|
|
1046
|
+
}
|
|
1047
|
+
|
|
1048
|
+
for (const [label, featureName] of [
|
|
1049
|
+
['empty', ''], ['non-string', 42], ['different', 'another-feature'],
|
|
1050
|
+
]) {
|
|
1051
|
+
const feature = `legacy-identity-${label}`;
|
|
1052
|
+
const specDir = createLegacyTasklessSpec20(root, feature, policy);
|
|
1053
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
1054
|
+
const state = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
1055
|
+
state.feature_name = featureName;
|
|
1056
|
+
write(specPath, JSON.stringify(state, null, 2));
|
|
1057
|
+
const before = snapshotTree(specDir);
|
|
1058
|
+
const rejected = migrate(feature);
|
|
1059
|
+
assert.equal(rejected.status, 2, `${label}: ${output(rejected)}`);
|
|
1060
|
+
assert.match(output(rejected), /feature_name conflicts with the scaffold feature identity/);
|
|
1061
|
+
assert.equal(snapshotTree(specDir), before, `${label} identity rejection mutated the fixture`);
|
|
1062
|
+
}
|
|
1063
|
+
} finally {
|
|
1064
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1065
|
+
}
|
|
1066
|
+
});
|
|
1067
|
+
|
|
1068
|
+
test('fresh scaffold rolls back authority rejection and injected partial writes without deleting pre-existing content', () => {
|
|
1069
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-fresh-transaction-'));
|
|
1070
|
+
try {
|
|
1071
|
+
const codexRoot = path.join(root, '.codex');
|
|
1072
|
+
const installedScripts = path.join(codexRoot, 'scripts');
|
|
1073
|
+
const installedHooks = path.join(codexRoot, 'hooks');
|
|
1074
|
+
const installedTemplates = path.join(codexRoot, 'skills/specs/templates');
|
|
1075
|
+
fs.mkdirSync(installedScripts, { recursive: true });
|
|
1076
|
+
fs.mkdirSync(installedHooks, { recursive: true });
|
|
1077
|
+
fs.mkdirSync(path.dirname(installedTemplates), { recursive: true });
|
|
1078
|
+
fs.cpSync(path.join(ROOT, 'src/claude/skills/specs/templates'), installedTemplates, { recursive: true });
|
|
1079
|
+
copyCodexRuntime(codexRoot, ['scripts/spec-scaffold.cjs']);
|
|
1080
|
+
write(path.join(installedHooks, 'completion-authority-state.cjs'), 'module.exports = {};\n');
|
|
1081
|
+
write(
|
|
1082
|
+
path.join(installedHooks, 'completion-authority-check.cjs'),
|
|
1083
|
+
'module.exports = { observePolicyBaseline() { return { ok: false, reason: "fixture authority reject" }; } };\n',
|
|
1084
|
+
);
|
|
1085
|
+
write(path.join(root, 'specs/keep.txt'), 'pre-existing\n');
|
|
1086
|
+
|
|
1087
|
+
const beforeAuthority = snapshotTree(root);
|
|
1088
|
+
const authority = run(
|
|
1089
|
+
path.join(installedScripts, 'spec-scaffold.cjs'),
|
|
1090
|
+
['authority-reject', '--lane', 'Standard'],
|
|
1091
|
+
root,
|
|
1092
|
+
);
|
|
1093
|
+
assert.equal(authority.status, 2, output(authority));
|
|
1094
|
+
assert.match(output(authority), /fixture authority reject.*transaction rolled back/);
|
|
1095
|
+
assert.equal(snapshotTree(root), beforeAuthority);
|
|
1096
|
+
assert.equal(fs.existsSync(path.join(root, 'specs/authority-reject')), false);
|
|
1097
|
+
|
|
1098
|
+
write(
|
|
1099
|
+
path.join(installedHooks, 'completion-authority-check.cjs'),
|
|
1100
|
+
'module.exports = { observePolicyBaseline() { return { ok: true }; } };\n',
|
|
1101
|
+
);
|
|
1102
|
+
const seeded = run(
|
|
1103
|
+
path.join(installedScripts, 'spec-scaffold.cjs'),
|
|
1104
|
+
['tasks-authority', '--lane', 'Standard'],
|
|
1105
|
+
root,
|
|
1106
|
+
);
|
|
1107
|
+
assert.equal(seeded.status, 0, output(seeded));
|
|
1108
|
+
write(
|
|
1109
|
+
path.join(installedHooks, 'completion-authority-check.cjs'),
|
|
1110
|
+
'module.exports = { observePolicyBaseline() { return { ok: false, reason: "fixture tasks-only authority reject" }; } };\n',
|
|
1111
|
+
);
|
|
1112
|
+
const beforeTasksAuthority = snapshotTree(root);
|
|
1113
|
+
const tasksAuthority = run(
|
|
1114
|
+
path.join(installedScripts, 'spec-scaffold.cjs'),
|
|
1115
|
+
['tasks-authority', '--tasks', 'R1-01-one,R1-02-two', '--tasks-only',
|
|
1116
|
+
'--boundaries', JSON.stringify([{
|
|
1117
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02'],
|
|
1118
|
+
write_sets: { 'R1-01': ['src/one.js'], 'R1-02': ['src/two.js'] },
|
|
1119
|
+
}])],
|
|
1120
|
+
root,
|
|
1121
|
+
);
|
|
1122
|
+
assert.equal(tasksAuthority.status, 2, output(tasksAuthority));
|
|
1123
|
+
assert.match(output(tasksAuthority), /fixture tasks-only authority reject.*transaction rolled back/);
|
|
1124
|
+
assert.equal(snapshotTree(root), beforeTasksAuthority);
|
|
1125
|
+
assert.equal(fs.existsSync(path.join(root, 'specs/tasks-authority/tasks/task-R1-01-one.md')), false);
|
|
1126
|
+
|
|
1127
|
+
const beforeInjected = snapshotTree(root);
|
|
1128
|
+
const injected = run(
|
|
1129
|
+
SCAFFOLD,
|
|
1130
|
+
['fresh-injected', '--lane', 'Standard'],
|
|
1131
|
+
root,
|
|
1132
|
+
{ env: { CAFEKIT_SCAFFOLD_FAIL_AFTER_WRITES: '2' } },
|
|
1133
|
+
);
|
|
1134
|
+
assert.equal(injected.status, 2, output(injected));
|
|
1135
|
+
assert.match(output(injected), /injected failure.*transaction rolled back/);
|
|
1136
|
+
assert.equal(snapshotTree(root), beforeInjected);
|
|
1137
|
+
assert.equal(fs.readFileSync(path.join(root, 'specs/keep.txt'), 'utf8'), 'pre-existing\n');
|
|
1138
|
+
} finally {
|
|
1139
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1140
|
+
}
|
|
1141
|
+
});
|
|
1142
|
+
|
|
1143
|
+
test('installed Codex scaffold rejects a symlinked template parent before creating a spec', () => {
|
|
1144
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-template-symlink-'));
|
|
1145
|
+
const outside = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-template-outside-'));
|
|
1146
|
+
try {
|
|
1147
|
+
const codexRoot = path.join(root, '.codex');
|
|
1148
|
+
copyCodexRuntime(codexRoot, ['scripts/spec-scaffold.cjs']);
|
|
1149
|
+
write(
|
|
1150
|
+
path.join(codexRoot, 'hooks/completion-authority-check.cjs'),
|
|
1151
|
+
'module.exports = { observePolicyBaseline() { return { ok: true }; } };\n',
|
|
1152
|
+
);
|
|
1153
|
+
write(path.join(codexRoot, 'hooks/completion-authority-state.cjs'), 'module.exports = {};\n');
|
|
1154
|
+
fs.cpSync(path.join(ROOT, 'src/claude/skills/specs/templates'), outside, { recursive: true });
|
|
1155
|
+
const templatesParent = path.join(root, '.agents/skills/specs');
|
|
1156
|
+
fs.mkdirSync(templatesParent, { recursive: true });
|
|
1157
|
+
fs.symlinkSync(outside, path.join(templatesParent, 'templates'));
|
|
1158
|
+
|
|
1159
|
+
const result = run(
|
|
1160
|
+
path.join(codexRoot, 'scripts/spec-scaffold.cjs'),
|
|
1161
|
+
['template-parent-escape', '--lane', 'Standard'],
|
|
1162
|
+
root,
|
|
1163
|
+
);
|
|
1164
|
+
assert.equal(result.status, 2, output(result));
|
|
1165
|
+
assert.match(output(result), /template check failed|symlink component rejected/i);
|
|
1166
|
+
assert.equal(fs.existsSync(path.join(root, 'specs/template-parent-escape')), false);
|
|
1167
|
+
} finally {
|
|
1168
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1169
|
+
fs.rmSync(outside, { recursive: true, force: true });
|
|
1170
|
+
}
|
|
1171
|
+
});
|
|
1172
|
+
|
|
1173
|
+
test('tasks-only is idempotent for an identical task request and preserves the snapshot bytes', () => {
|
|
1174
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-idempotent-'));
|
|
1175
|
+
try {
|
|
1176
|
+
const policy = POLICY.workflowPolicySnapshot({ riskSignals: {} });
|
|
1177
|
+
createLegacyTasklessSpec20(root, 'idempotent', policy);
|
|
1178
|
+
const boundaryArgs = ['--boundaries', JSON.stringify([{
|
|
1179
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02'],
|
|
1180
|
+
write_sets: { 'R1-01': ['src/one.js'], 'R1-02': ['src/two.js'] },
|
|
1181
|
+
}])];
|
|
1182
|
+
const idempotentArgs = ['idempotent', '--tasks', 'R1-01-one,R1-02-two', '--tasks-only', ...boundaryArgs];
|
|
1183
|
+
const first = run(SCAFFOLD, idempotentArgs, root);
|
|
1184
|
+
assert.equal(first.status, 0, output(first));
|
|
1185
|
+
const beforeSecond = snapshotTree(root);
|
|
1186
|
+
const second = run(SCAFFOLD, idempotentArgs, root);
|
|
1187
|
+
assert.equal(second.status, 0, output(second));
|
|
1188
|
+
assert.match(output(second), /0 new task stub\(s\)/);
|
|
1189
|
+
assert.equal(snapshotTree(root), beforeSecond);
|
|
1190
|
+
|
|
1191
|
+
const artifactBase = run(SCAFFOLD, ['idempotent-artifact', '--lane', 'Standard'], root);
|
|
1192
|
+
assert.equal(artifactBase.status, 0, output(artifactBase));
|
|
1193
|
+
const proofArgs = ['--boundaries', JSON.stringify([{
|
|
1194
|
+
id: 'B-V', type: 'proof', subject: 'R1-01', verifier: 'R1-02',
|
|
1195
|
+
verification_ref: 'V1', artifact_anchor: 'A-R1-02-02',
|
|
1196
|
+
}])];
|
|
1197
|
+
const artifactArgs = [
|
|
1198
|
+
'idempotent-artifact', '--tasks', 'R1-01-one,R1-02-two', '--tasks-only', ...proofArgs,
|
|
1199
|
+
];
|
|
1200
|
+
const artifactFirst = run(SCAFFOLD, artifactArgs, root);
|
|
1201
|
+
assert.equal(artifactFirst.status, 0, output(artifactFirst));
|
|
1202
|
+
const artifactDir = path.join(root, 'specs/idempotent-artifact');
|
|
1203
|
+
completeScaffoldedSpec21(artifactDir);
|
|
1204
|
+
const artifactValidation = run(VALIDATOR, [artifactDir], ROOT);
|
|
1205
|
+
assert.equal(artifactValidation.status, 0, output(artifactValidation));
|
|
1206
|
+
const beforeArtifactSecond = snapshotTree(root);
|
|
1207
|
+
const artifactSecond = run(SCAFFOLD, artifactArgs, root);
|
|
1208
|
+
assert.equal(artifactSecond.status, 0, output(artifactSecond));
|
|
1209
|
+
assert.match(output(artifactSecond), /0 new task stub\(s\)/);
|
|
1210
|
+
assert.equal(snapshotTree(root), beforeArtifactSecond);
|
|
1211
|
+
} finally {
|
|
1212
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1213
|
+
}
|
|
1214
|
+
});
|
|
1215
|
+
|
|
1216
|
+
test('tasks-only consumes the policy snapshot and escalates auth to Elevated monotonically', () => {
|
|
1217
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-tasks-only-policy-'));
|
|
1218
|
+
try {
|
|
1219
|
+
const specDir = createLegacyTasklessSpec20(root, 'existing', POLICY.workflowPolicySnapshot({ riskSignals: {} }));
|
|
1220
|
+
const boundaries = JSON.stringify([{
|
|
1221
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02'],
|
|
1222
|
+
write_sets: { 'R1-01': ['src/auth.js'], 'R1-02': ['src/next.js'] },
|
|
1223
|
+
}]);
|
|
1224
|
+
const escalated = run(SCAFFOLD, [
|
|
1225
|
+
'existing', '--tasks', 'R1-01-auth,R1-02-next', '--risks', 'auth', '--tasks-only',
|
|
1226
|
+
'--boundaries', boundaries,
|
|
1227
|
+
], root);
|
|
1228
|
+
assert.equal(escalated.status, 0, output(escalated));
|
|
1229
|
+
assert.match(output(escalated), /2 new task stub\(s\)/);
|
|
1230
|
+
assert.doesNotMatch(output(escalated), /research\.md created/);
|
|
1231
|
+
const first = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
1232
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(first).lane, 'Standard');
|
|
1233
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(first).assurance_level, 'Elevated');
|
|
1234
|
+
assert.deepEqual(first.workflow_policy.risks, ['auth']);
|
|
1235
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(first).artifact_profile, 'bounded');
|
|
1236
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(first).proof_obligations.includes('needsResearchGrounding'), false);
|
|
1237
|
+
assert.equal(fs.existsSync(path.join(specDir, 'research.md')), false);
|
|
1238
|
+
assert.equal(fs.existsSync(path.join(specDir, 'reports')), false);
|
|
1239
|
+
|
|
1240
|
+
const policyBytes = JSON.stringify(first.workflow_policy);
|
|
1241
|
+
const noNewRisk = run(SCAFFOLD, [
|
|
1242
|
+
'existing', '--tasks', 'R1-01-auth,R1-02-next', '--tasks-only', '--boundaries', boundaries,
|
|
1243
|
+
], root);
|
|
1244
|
+
assert.equal(noNewRisk.status, 0, output(noNewRisk));
|
|
1245
|
+
assert.match(output(noNewRisk), /0 new task stub\(s\)/);
|
|
1246
|
+
const second = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
1247
|
+
assert.equal(JSON.stringify(second.workflow_policy), policyBytes);
|
|
1248
|
+
|
|
1249
|
+
const beforeRejectedLane = fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8');
|
|
1250
|
+
const rejectedBoundaries = JSON.stringify([{
|
|
1251
|
+
id: 'B-OWN', type: 'ownership', tasks: ['R1-01', 'R1-02', 'R1-03'],
|
|
1252
|
+
write_sets: {
|
|
1253
|
+
'R1-01': ['src/auth.js'], 'R1-02': ['src/next.js'], 'R1-03': ['src/invalid.js'],
|
|
1254
|
+
},
|
|
1255
|
+
}]);
|
|
1256
|
+
const rejectedLane = run(SCAFFOLD, [
|
|
1257
|
+
'existing', '--tasks', 'R1-03-invalid', '--lane', 'Critical', '--tasks-only',
|
|
1258
|
+
'--boundaries', rejectedBoundaries,
|
|
1259
|
+
], root);
|
|
1260
|
+
assert.equal(rejectedLane.status, 2, output(rejectedLane));
|
|
1261
|
+
assert.match(output(rejectedLane), /does not match persisted workflow_policy\.lane/);
|
|
1262
|
+
assert.equal(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'), beforeRejectedLane);
|
|
1263
|
+
assert.equal(fs.existsSync(path.join(specDir, 'tasks/task-R1-03-invalid.md')), false);
|
|
1264
|
+
} finally {
|
|
1265
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1266
|
+
}
|
|
1267
|
+
});
|
|
1268
|
+
|
|
1269
|
+
test('scaffold emits minimal 2.1 artifacts without default research or reports ceremony', () => {
|
|
1270
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-artifact-profile-'));
|
|
1271
|
+
try {
|
|
1272
|
+
const standardDefault = run(SCAFFOLD, ['standard-default', '--lane', 'Standard'], root);
|
|
1273
|
+
assert.equal(standardDefault.status, 0, output(standardDefault));
|
|
1274
|
+
const standardDir = path.join(root, 'specs', 'standard-default');
|
|
1275
|
+
assert.deepEqual(fs.readdirSync(standardDir).sort(), [
|
|
1276
|
+
'design.md', 'requirements.md', 'spec.json',
|
|
1277
|
+
]);
|
|
1278
|
+
|
|
1279
|
+
const strict = run(SCAFFOLD, ['strict-taskless', '--risks', 'auth', '--assurance-level', 'Strict'], root);
|
|
1280
|
+
assert.equal(strict.status, 0, output(strict));
|
|
1281
|
+
const strictDir = path.join(root, 'specs', 'strict-taskless');
|
|
1282
|
+
const strictSpec = JSON.parse(fs.readFileSync(path.join(strictDir, 'spec.json'), 'utf8'));
|
|
1283
|
+
assert.equal(strictSpec.schema_version, '2.1');
|
|
1284
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(strictSpec).lane, 'Critical');
|
|
1285
|
+
assert.equal(strictSpec.authoring.tasks, 'absent');
|
|
1286
|
+
assert.equal(fs.existsSync(path.join(strictDir, 'research.md')), false);
|
|
1287
|
+
assert.equal(fs.existsSync(path.join(strictDir, 'reports')), false);
|
|
1288
|
+
} finally {
|
|
1289
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1290
|
+
}
|
|
1291
|
+
});
|
|
1292
|
+
|
|
1293
|
+
test('strict validator requires numeric IDs, rejects phantom mappings, and grounds research content', () => {
|
|
1294
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-strict-grounding-'));
|
|
1295
|
+
try {
|
|
1296
|
+
const strictPolicy = POLICY.workflowPolicySnapshot({ riskSignals: { auth: true }, assurance_level: 'Strict' });
|
|
1297
|
+
const noIds = createSpec(root, {
|
|
1298
|
+
name: 'no-numeric-ids',
|
|
1299
|
+
requirements: '# Requirements\n\nThe behavior has no numeric identifier.\n',
|
|
1300
|
+
spec: { workflow_policy: strictPolicy },
|
|
1301
|
+
});
|
|
1302
|
+
const noIdsResult = run(VALIDATOR, [noIds], ROOT);
|
|
1303
|
+
assert.notEqual(noIdsResult.status, 0);
|
|
1304
|
+
assert.match(output(noIdsResult), /strict workflow requires numeric requirement IDs/);
|
|
1305
|
+
|
|
1306
|
+
const legacyDialect = createSpec(root, {
|
|
1307
|
+
name: 'legacy-req-dialect',
|
|
1308
|
+
requirements: '# Requirements\n\n## Requirements\n\n### REQ-01: One\n\n- **REQ-01.1** The service shall return one result.\n',
|
|
1309
|
+
spec: { workflow_policy: strictPolicy },
|
|
1310
|
+
});
|
|
1311
|
+
const legacyDialectResult = run(VALIDATOR, [legacyDialect], ROOT);
|
|
1312
|
+
assert.notEqual(legacyDialectResult.status, 0);
|
|
1313
|
+
assert.match(output(legacyDialectResult), /REQ-01 is not a canonical requirement id/);
|
|
1314
|
+
|
|
1315
|
+
const phantom = createSpec(root, {
|
|
1316
|
+
name: 'phantom-mapping',
|
|
1317
|
+
task: task({ mapping: '9.1' }),
|
|
1318
|
+
spec: { workflow_policy: strictPolicy },
|
|
1319
|
+
});
|
|
1320
|
+
const phantomResult = run(VALIDATOR, [phantom], ROOT);
|
|
1321
|
+
assert.notEqual(phantomResult.status, 0);
|
|
1322
|
+
assert.match(output(phantomResult), /references unknown requirement R9/);
|
|
1323
|
+
|
|
1324
|
+
const researchPlaceholder = createSpec(root, {
|
|
1325
|
+
name: 'research-placeholder',
|
|
1326
|
+
task: task({ related: '`src/strict-only.js` | Read' }),
|
|
1327
|
+
design: criticalNoObligationDesign(),
|
|
1328
|
+
spec: { workflow_policy: strictPolicy, research: 'research.md' },
|
|
1329
|
+
});
|
|
1330
|
+
write(path.join(researchPlaceholder, 'research.md'), '# Research\n\n## Evidence Summary\n- TBD\n');
|
|
1331
|
+
const placeholderResult = run(VALIDATOR, [researchPlaceholder], ROOT);
|
|
1332
|
+
assert.notEqual(placeholderResult.status, 0);
|
|
1333
|
+
assert.match(output(placeholderResult), /Evidence Summary must contain concrete evidence/);
|
|
1334
|
+
|
|
1335
|
+
write(path.join(researchPlaceholder, 'research.md'), '# Research\n\n## Evidence Summary\n- Scout inspected `src/one.js` and confirmed the caller path.\n');
|
|
1336
|
+
const concreteResult = run(VALIDATOR, [researchPlaceholder], ROOT);
|
|
1337
|
+
assert.equal(concreteResult.status, 0, output(concreteResult));
|
|
1338
|
+
} finally {
|
|
1339
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1340
|
+
}
|
|
1341
|
+
});
|
|
1342
|
+
|
|
1343
|
+
test('validator rejects unsupported or duplicate structured requirement IDs without mining incidental prose', () => {
|
|
1344
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-requirement-inventory-'));
|
|
1345
|
+
try {
|
|
1346
|
+
const unsupportedHeading = createSpec(root, {
|
|
1347
|
+
name: 'unsupported-heading',
|
|
1348
|
+
requirements: '# Requirements\n\n### R1A: One\n\n- **R1.1** The service shall return status value 1.\n',
|
|
1349
|
+
});
|
|
1350
|
+
let result = run(VALIDATOR, [unsupportedHeading], ROOT);
|
|
1351
|
+
assert.notEqual(result.status, 0);
|
|
1352
|
+
assert.match(output(result), /unsupported requirement id R1A/);
|
|
1353
|
+
|
|
1354
|
+
const unsupportedCriterion = createSpec(root, {
|
|
1355
|
+
name: 'unsupported-criterion',
|
|
1356
|
+
requirements: '# Requirements\n\n### Requirement 1: One\n\n- **R1-1** The service shall return status value 1.\n',
|
|
1357
|
+
});
|
|
1358
|
+
result = run(VALIDATOR, [unsupportedCriterion], ROOT);
|
|
1359
|
+
assert.notEqual(result.status, 0);
|
|
1360
|
+
assert.match(output(result), /unsupported acceptance criterion id R1-1/);
|
|
1361
|
+
|
|
1362
|
+
const duplicates = createSpec(root, {
|
|
1363
|
+
name: 'duplicate-inventory',
|
|
1364
|
+
requirements: '# Requirements\n\n### Requirement 1: First\n\n- **R1.1** The service shall return status value 1.\n\n' +
|
|
1365
|
+
'### R1: Duplicate\n\n- **R1.1** The service shall return status value 2.\n',
|
|
1366
|
+
});
|
|
1367
|
+
result = run(VALIDATOR, [duplicates], ROOT);
|
|
1368
|
+
assert.notEqual(result.status, 0);
|
|
1369
|
+
assert.match(output(result), /duplicate canonical requirement id R1/);
|
|
1370
|
+
assert.match(output(result), /duplicate canonical acceptance criterion id R1\.1/);
|
|
1371
|
+
|
|
1372
|
+
const incidental = createSpec(root, {
|
|
1373
|
+
name: 'incidental-rfc-adr-and-reference',
|
|
1374
|
+
task: task({ related: '`src/one.js` | Read' }),
|
|
1375
|
+
requirements: '# Requirements\n\n### Requirement 1: One\n\n' +
|
|
1376
|
+
'- **R1.1** The service shall return one deterministic result with status value 1.\n\n' +
|
|
1377
|
+
'See RFC 9110, ADR-001, and R7.2 for historical context only.\n',
|
|
1378
|
+
});
|
|
1379
|
+
result = run(VALIDATOR, [incidental], ROOT);
|
|
1380
|
+
assert.equal(result.status, 0, output(result));
|
|
1381
|
+
|
|
1382
|
+
const ignoredSyntax = createSpec(root, {
|
|
1383
|
+
name: 'ignored-fenced-and-commented-inventory',
|
|
1384
|
+
task: task({ related: '`src/one.js` | Read' }),
|
|
1385
|
+
requirements: '# Requirements\n\n### Requirement 1: One\n\n' +
|
|
1386
|
+
'- **R1.1** The service shall return one deterministic result with status value 1.\n\n' +
|
|
1387
|
+
'<!-- ### Requirement 1: commented duplicate\n- **R1.1** commented duplicate -->\n\n' +
|
|
1388
|
+
'```markdown\n### Requirement 1: example duplicate\n- **R1.1** example duplicate\n```\n',
|
|
1389
|
+
});
|
|
1390
|
+
result = run(VALIDATOR, [ignoredSyntax], ROOT);
|
|
1391
|
+
assert.equal(result.status, 0, output(result));
|
|
1392
|
+
|
|
1393
|
+
for (const [name, heading] of [
|
|
1394
|
+
['numeric-heading', '### 1: One'],
|
|
1395
|
+
['bracket-heading', '### [R1]: One'],
|
|
1396
|
+
]) {
|
|
1397
|
+
const invalidHeading = createSpec(root, {
|
|
1398
|
+
name,
|
|
1399
|
+
requirements: `# Requirements\n\n${heading}\n\n- **R1.1** The service shall return status value 1.\n`,
|
|
1400
|
+
});
|
|
1401
|
+
result = run(VALIDATOR, [invalidHeading], ROOT);
|
|
1402
|
+
assert.notEqual(result.status, 0);
|
|
1403
|
+
assert.match(output(result), /unsupported requirement id/);
|
|
1404
|
+
}
|
|
1405
|
+
} finally {
|
|
1406
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1407
|
+
}
|
|
1408
|
+
});
|
|
1409
|
+
|
|
1410
|
+
test('validator requires substantive Specs v2 task semantics and planned executable proof', () => {
|
|
1411
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-task-semantics-'));
|
|
1412
|
+
try {
|
|
1413
|
+
const invalid = createSpec(root, { name: 'weak-task-semantics' });
|
|
1414
|
+
const weakTask = task({ related: '`src/one.js` | Read' })
|
|
1415
|
+
.replace('Calling the real fixture entrypoint returns status value 1.', 'TBD')
|
|
1416
|
+
.replace('- **In scope:** Implement the mapped observable behavior.', '- **In scope:** TBD')
|
|
1417
|
+
.replace('- **Out of scope:** Unrelated runtime behavior.', '- **Out of scope:** N/A')
|
|
1418
|
+
.replace('- **R1.1:** Calling `src/one.js` returns an observable result with status value 1.', '- **R1.1:** Works')
|
|
1419
|
+
.replace('| A-R1-01-01 | command | `node --test test/one.test.js` |', '| A-R1-01-01 | command | `npm run fixture-check` |')
|
|
1420
|
+
.replace('- **Command:** `node --test test/one.test.js`', '- **Command:** run tests')
|
|
1421
|
+
.replace('exit code 0 and one assertion for status value 1', 'looks good')
|
|
1422
|
+
.replace('invalid input returns a deterministic error result', 'TBD')
|
|
1423
|
+
.replace('`src/one.js` is invoked by the focused test', 'TBD');
|
|
1424
|
+
writeRaw(path.join(invalid, 'tasks/task-R1-01-one.md'), weakTask);
|
|
1425
|
+
const rejected = run(VALIDATOR, [invalid], ROOT);
|
|
1426
|
+
assert.notEqual(rejected.status, 0);
|
|
1427
|
+
assert.match(output(rejected), /Outcome must be substantive/);
|
|
1428
|
+
assert.match(output(rejected), /Verification Plan requires a command-shaped invocation or justified N\/A/);
|
|
1429
|
+
assert.match(output(rejected), /Verification Plan requires an observable Expected result/);
|
|
1430
|
+
assert.match(output(rejected), /Verification Plan requires a negative-path disposition/);
|
|
1431
|
+
assert.match(output(rejected), /Verification Plan requires a concrete reachability target or justified N\/A/);
|
|
26
1432
|
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
1433
|
+
const emptyExecution = createSpec(root, {
|
|
1434
|
+
name: 'empty-steps-mapping-only-requirements',
|
|
1435
|
+
task: task({ related: '`src/empty.js` | Read' })
|
|
1436
|
+
.replace(/## Changes\n[\s\S]*?\n## Acceptance/, '## Changes\n\n## Acceptance'),
|
|
1437
|
+
});
|
|
1438
|
+
const emptyExecutionResult = run(VALIDATOR, [emptyExecution], ROOT);
|
|
1439
|
+
assert.notEqual(emptyExecutionResult.status, 0);
|
|
1440
|
+
assert.match(output(emptyExecutionResult), /Changes must be substantive|Changes must contain at least one substantive actionable checkbox/);
|
|
1441
|
+
|
|
1442
|
+
const justified = createSpec(root, {
|
|
1443
|
+
name: 'justified-command-na',
|
|
1444
|
+
task: task({ related: '`src/static-contract.js` | Read' }).replace(
|
|
1445
|
+
'- **Command:** `node --test test/one.test.js`',
|
|
1446
|
+
'- Command: N/A — no executable runtime exists because this task only inspects a static contract',
|
|
1447
|
+
),
|
|
1448
|
+
});
|
|
1449
|
+
const accepted = run(VALIDATOR, [justified], ROOT);
|
|
1450
|
+
assert.equal(accepted.status, 0, output(accepted));
|
|
1451
|
+
|
|
1452
|
+
const projectSpecific = createSpec(root, {
|
|
1453
|
+
name: 'project-specific-command-shape',
|
|
1454
|
+
task: task({ related: '`src/project-specific.js` | Read' })
|
|
1455
|
+
.replace('node --test test/one.test.js', 'npm run project-specific-check'),
|
|
1456
|
+
});
|
|
1457
|
+
const projectSpecificResult = run(VALIDATOR, [projectSpecific], ROOT);
|
|
1458
|
+
assert.equal(projectSpecificResult.status, 0, output(projectSpecificResult));
|
|
1459
|
+
} finally {
|
|
1460
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1461
|
+
}
|
|
1462
|
+
});
|
|
1463
|
+
|
|
1464
|
+
test('all ready non-Direct specs require semantic design and coherent authoring timestamps', () => {
|
|
1465
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-ready-lifecycle-'));
|
|
1466
|
+
|
|
1467
|
+
function readyFixture(name, mutate = () => {}) {
|
|
1468
|
+
const specDir = createSpec(root, {
|
|
1469
|
+
name,
|
|
1470
|
+
task: task({ related: '`src/one.js` | Read' }),
|
|
1471
|
+
spec: { workflow_policy: POLICY.workflowPolicySnapshot({ riskSignals: {} }) },
|
|
1472
|
+
});
|
|
1473
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
1474
|
+
const state = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
1475
|
+
state.ready_for_implementation = true;
|
|
1476
|
+
state.validation = { status: 'completed' };
|
|
1477
|
+
state.updated_at = '2026-08-11T00:06:00+07:00';
|
|
1478
|
+
state.timestamps = lifecycleTimestamps();
|
|
1479
|
+
mutate(state, specDir);
|
|
1480
|
+
write(specPath, JSON.stringify(state, null, 2));
|
|
1481
|
+
completeSemanticReview(
|
|
1482
|
+
specDir,
|
|
1483
|
+
state.workflow_policy.assurance_level === 'Strict' ? 'independent' : 'same-session',
|
|
1484
|
+
);
|
|
1485
|
+
return specDir;
|
|
1486
|
+
}
|
|
1487
|
+
|
|
1488
|
+
try {
|
|
1489
|
+
const critical = readyFixture('critical-generic-design', (state, specDir) => {
|
|
1490
|
+
state.workflow_policy = POLICY.workflowPolicySnapshot({ riskSignals: { auth: true }, assurance_level: 'Strict' });
|
|
1491
|
+
state.timestamps = lifecycleTimestamps({ research: true });
|
|
1492
|
+
write(path.join(specDir, 'design.md'), '# Design\n\n## Architecture\n\nThe implementation boundary is defined and appropriate.\n');
|
|
1493
|
+
});
|
|
1494
|
+
let result = run(VALIDATOR, [critical], ROOT);
|
|
1495
|
+
assert.notEqual(result.status, 0);
|
|
1496
|
+
assert.match(output(result), /design\.md: Critical readiness requires a grounded path\/code identifier/);
|
|
1497
|
+
|
|
1498
|
+
const strictAuthoring = readyFixture('critical-one-task-authoring', (state, specDir) => {
|
|
1499
|
+
state.workflow_policy = POLICY.workflowPolicySnapshot({ riskSignals: { auth: true }, assurance_level: 'Strict' });
|
|
1500
|
+
state.timestamps = lifecycleTimestamps({ research: true });
|
|
1501
|
+
write(path.join(specDir, 'design.md'), criticalNoObligationDesign());
|
|
1502
|
+
});
|
|
1503
|
+
result = run(VALIDATOR, [strictAuthoring], ROOT);
|
|
1504
|
+
assert.equal(result.status, 0, output(result));
|
|
1505
|
+
|
|
1506
|
+
const invalidCases = [
|
|
1507
|
+
['boolean-created-at', (state) => { state.created_at = true; }, /created_at: must be an ISO 8601 timestamp|created_at: ready handoff requires/],
|
|
1508
|
+
['boolean-task-time', (state) => { state.timestamps.tasks_done = true; }, /timestamps\.tasks_done: must be null or an ISO 8601 timestamp|ready handoff requires/],
|
|
1509
|
+
['missing-validation-time', (state) => { delete state.timestamps.validation_done; }, /timestamps\.validation_done: ready handoff requires/],
|
|
1510
|
+
['reversed-phase-time', (state) => { state.timestamps.design_done = '2026-08-11T00:00:30+07:00'; }, /final lifecycle timestamps must be ordered/],
|
|
1511
|
+
['mismatched-final-state', (state) => { state.updated_at = '2026-08-11T00:07:00+07:00'; }, /updated_at must describe the same final state/],
|
|
1512
|
+
];
|
|
1513
|
+
for (const [name, mutate, expected] of invalidCases) {
|
|
1514
|
+
const specDir = readyFixture(name, mutate);
|
|
1515
|
+
result = run(VALIDATOR, [specDir], ROOT);
|
|
1516
|
+
assert.notEqual(result.status, 0, `${name} unexpectedly passed`);
|
|
1517
|
+
assert.match(output(result), expected);
|
|
1518
|
+
}
|
|
1519
|
+
|
|
1520
|
+
const reviewDir = readyFixture('five-task-review-time', (state, specDir) => {
|
|
1521
|
+
state.design_context = { validation_recommended: true };
|
|
1522
|
+
for (let index = 2; index <= 5; index += 1) {
|
|
1523
|
+
const taskPath = `tasks/task-R1-0${index}-review.md`;
|
|
1524
|
+
state.task_files.push(taskPath);
|
|
1525
|
+
state.task_registry[taskPath] = {
|
|
1526
|
+
id: `R1-0${index}`, title: `Review ${index}`, status: 'pending', dependencies: [], blocker: null,
|
|
1527
|
+
started_at: null, completed_at: null, last_updated_at: null,
|
|
1528
|
+
};
|
|
1529
|
+
write(path.join(specDir, taskPath), task({ id: `R1-0${index}`, related: `\`src/review-${index}.js\` | Read` }));
|
|
1530
|
+
}
|
|
1531
|
+
});
|
|
1532
|
+
result = run(VALIDATOR, [reviewDir], ROOT);
|
|
1533
|
+
assert.notEqual(result.status, 0);
|
|
1534
|
+
assert.match(output(result), /timestamps\.review_done: required after validation for a large task graph/);
|
|
1535
|
+
|
|
1536
|
+
const reviewPath = path.join(reviewDir, 'spec.json');
|
|
1537
|
+
const reviewState = JSON.parse(fs.readFileSync(reviewPath, 'utf8'));
|
|
1538
|
+
reviewState.timestamps.review_done = '2026-08-11T00:05:00+07:00';
|
|
1539
|
+
write(reviewPath, JSON.stringify(reviewState, null, 2));
|
|
1540
|
+
result = run(VALIDATOR, [reviewDir], ROOT);
|
|
1541
|
+
assert.equal(result.status, 0, output(result));
|
|
1542
|
+
} finally {
|
|
1543
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1544
|
+
}
|
|
1545
|
+
});
|
|
1546
|
+
|
|
1547
|
+
test('spec-ready excludes execution receipts while preserving Strict review state', () => {
|
|
1548
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-spec-ready-'));
|
|
1549
|
+
try {
|
|
1550
|
+
const standardDir = createSpec(root, {
|
|
1551
|
+
name: 'pending-standard',
|
|
1552
|
+
task: task({ related: '`src/pending-only.js` | Read' }),
|
|
1553
|
+
spec: {
|
|
1554
|
+
workflow_policy: POLICY.workflowPolicySnapshot({ riskSignals: {} }),
|
|
1555
|
+
ready_for_implementation: true,
|
|
1556
|
+
approvals: {
|
|
1557
|
+
requirements: { generated: true, agent_validated: true, user_approved: true },
|
|
1558
|
+
design: { generated: true, agent_validated: true, user_approved: true },
|
|
1559
|
+
tasks: { generated: true, agent_validated: true, user_approved: true },
|
|
1560
|
+
},
|
|
1561
|
+
validation: { status: 'completed' },
|
|
1562
|
+
updated_at: '2026-08-11T00:06:00+07:00',
|
|
1563
|
+
timestamps: lifecycleTimestamps(),
|
|
45
1564
|
},
|
|
46
|
-
}
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
...(options.spec || {}),
|
|
51
|
-
}, null, 2));
|
|
52
|
-
return specDir;
|
|
53
|
-
}
|
|
1565
|
+
});
|
|
1566
|
+
completeSemanticReview(standardDir);
|
|
1567
|
+
const standardResult = run(VALIDATOR, [standardDir], ROOT);
|
|
1568
|
+
assert.equal(standardResult.status, 0, output(standardResult));
|
|
54
1569
|
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
}
|
|
1570
|
+
const criticalDir = createSpec(root, {
|
|
1571
|
+
name: 'pending-critical',
|
|
1572
|
+
task: task({ related: '`src/pending-only.js` | Read' }),
|
|
1573
|
+
design: criticalNoObligationDesign(),
|
|
1574
|
+
spec: {
|
|
1575
|
+
workflow_policy: POLICY.workflowPolicySnapshot({ riskSignals: { auth: true }, assurance_level: 'Strict' }),
|
|
1576
|
+
ready_for_implementation: true,
|
|
1577
|
+
approvals: {
|
|
1578
|
+
requirements: { generated: true, agent_validated: true, user_approved: true },
|
|
1579
|
+
design: { generated: true, agent_validated: true, user_approved: true },
|
|
1580
|
+
tasks: { generated: true, agent_validated: true, user_approved: true },
|
|
1581
|
+
},
|
|
1582
|
+
validation: { status: 'completed' },
|
|
1583
|
+
updated_at: '2026-08-11T00:06:00+07:00',
|
|
1584
|
+
timestamps: lifecycleTimestamps({ research: true, review: true }),
|
|
1585
|
+
},
|
|
1586
|
+
});
|
|
1587
|
+
completeSemanticReview(criticalDir, 'independent');
|
|
1588
|
+
const criticalResult = run(VALIDATOR, [criticalDir], ROOT);
|
|
1589
|
+
assert.equal(criticalResult.status, 0, output(criticalResult));
|
|
1590
|
+
} finally {
|
|
1591
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1592
|
+
}
|
|
1593
|
+
});
|
|
1594
|
+
|
|
1595
|
+
test('Specs v2 Compact validates without task registry or premature receipt and fails closed on policy shape errors', () => {
|
|
1596
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-bounded-'));
|
|
1597
|
+
try {
|
|
1598
|
+
const specDir = createBoundedSpec(root);
|
|
1599
|
+
const valid = run(VALIDATOR, [specDir], ROOT);
|
|
1600
|
+
assert.equal(valid.status, 0, output(valid));
|
|
1601
|
+
const persisted = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
1602
|
+
assert.equal(persisted.workflow_policy.artifact_profile, 'bounded');
|
|
1603
|
+
assert.equal(Object.hasOwn(persisted, 'task_files'), false);
|
|
1604
|
+
assert.equal(Object.hasOwn(persisted, 'task_registry'), false);
|
|
1605
|
+
|
|
1606
|
+
write(path.join(specDir, 'feature-receipt.md'), '# Feature Verification Receipt\n\nVerification: PENDING\n\nStatus: in_progress\n\nBlocker: awaiting execution proof\n');
|
|
1607
|
+
const prematureReceipt = run(VALIDATOR, [specDir], ROOT);
|
|
1608
|
+
assert.notEqual(prematureReceipt.status, 0);
|
|
1609
|
+
assert.match(output(prematureReceipt), /premature execution receipt/);
|
|
1610
|
+
fs.unlinkSync(path.join(specDir, 'feature-receipt.md'));
|
|
1611
|
+
const malformed = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
1612
|
+
malformed.workflow_policy.extra = true;
|
|
1613
|
+
fs.writeFileSync(path.join(specDir, 'spec.json'), JSON.stringify(malformed, null, 2));
|
|
1614
|
+
const malformedResult = run(VALIDATOR, [specDir], ROOT);
|
|
1615
|
+
assert.notEqual(malformedResult.status, 0);
|
|
1616
|
+
assert.match(output(malformedResult), /workflow_policy fields must be exactly/);
|
|
1617
|
+
} finally {
|
|
1618
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1619
|
+
}
|
|
1620
|
+
});
|
|
1621
|
+
|
|
1622
|
+
test('Specs v2 scaffold keeps planning artifacts independent from assurance and task topology', () => {
|
|
1623
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-lanes-'));
|
|
1624
|
+
try {
|
|
1625
|
+
const scaffoldSource = fs.readFileSync(SCAFFOLD, 'utf8');
|
|
1626
|
+
assert.match(scaffoldSource, /--lane Standard\|Critical/);
|
|
1627
|
+
assert.doesNotMatch(scaffoldSource, /\[--lane Direct\|/);
|
|
1628
|
+
const direct = run(SCAFFOLD, ['direct-scaffold', '--lane', 'Direct'], root);
|
|
1629
|
+
assert.equal(direct.status, 2, output(direct));
|
|
1630
|
+
assert.match(output(direct), /Direct lane does not create a spec/);
|
|
1631
|
+
assert.doesNotMatch(output(direct), /no trusted runtime issuer/i);
|
|
1632
|
+
|
|
1633
|
+
const standard = run(SCAFFOLD, ['bounded-standard', '--lane', 'Standard'], root);
|
|
1634
|
+
assert.equal(standard.status, 0, output(standard));
|
|
1635
|
+
const standardDir = path.join(root, 'specs', 'bounded-standard');
|
|
1636
|
+
const standardSpec = JSON.parse(fs.readFileSync(path.join(standardDir, 'spec.json'), 'utf8'));
|
|
1637
|
+
assert.equal(POLICY.readWorkflowPolicySnapshot(standardSpec).lane, 'Standard');
|
|
1638
|
+
assert.deepEqual(POLICY.readWorkflowPolicySnapshot(standardSpec).proof_obligations, ['needsExecutionProof']);
|
|
1639
|
+
assert.equal(fs.existsSync(path.join(standardDir, 'tasks')), false);
|
|
1640
|
+
assert.equal(fs.existsSync(path.join(standardDir, 'task_registry')), false);
|
|
1641
|
+
assert.equal(standardSpec.validation.semantic_review.status, 'not-run');
|
|
1642
|
+
assert.equal(fs.existsSync(path.join(standardDir, 'semantic-review.md')), false);
|
|
1643
|
+
write(path.join(standardDir, 'requirements.md'), '# Requirements\n\n### Requirement 1: One\n\n- **R1.1** The scaffolded command shall return status value 1.\n- **R1.2** The proof shall reject an invalid status result.\n');
|
|
1644
|
+
write(path.join(standardDir, 'design.md'), designFixture21());
|
|
1645
|
+
const standardPromoted = run(SCAFFOLD, ['bounded-standard', '--sync-semantic-model'], root);
|
|
1646
|
+
assert.equal(standardPromoted.status, 0, output(standardPromoted));
|
|
1647
|
+
const standardValidation = run(VALIDATOR, [standardDir], ROOT);
|
|
1648
|
+
assert.equal(standardValidation.status, 0, output(standardValidation));
|
|
1649
|
+
|
|
1650
|
+
const elevated = run(SCAFFOLD, ['elevated-risk', '--risks', 'auth'], root);
|
|
1651
|
+
assert.equal(elevated.status, 0, output(elevated));
|
|
1652
|
+
const elevatedTaskless = JSON.parse(fs.readFileSync(path.join(root, 'specs/elevated-risk/spec.json'), 'utf8'));
|
|
1653
|
+
assert.equal(elevatedTaskless.workflow_policy.planning_depth, 'Compact');
|
|
1654
|
+
assert.equal(elevatedTaskless.workflow_policy.assurance_level, 'Elevated');
|
|
1655
|
+
assert.equal(elevatedTaskless.workflow_policy.classified_minimum.assurance_level, 'Elevated');
|
|
1656
|
+
assert.deepEqual(elevatedTaskless.coordination, { boundaries: [] });
|
|
1657
|
+
assert.equal(fs.existsSync(path.join(root, 'specs/elevated-risk/research.md')), false);
|
|
1658
|
+
assert.equal(elevatedTaskless.validation.semantic_review.status, 'not-run');
|
|
1659
|
+
assert.equal(elevatedTaskless.authoring.tasks, 'absent');
|
|
1660
|
+
|
|
1661
|
+
const strict = run(SCAFFOLD, ['strict-opt-in', '--risks', 'auth', '--assurance-level', 'Strict'], root);
|
|
1662
|
+
assert.equal(strict.status, 0, output(strict));
|
|
1663
|
+
const strictTaskless = JSON.parse(fs.readFileSync(path.join(root, 'specs/strict-opt-in/spec.json'), 'utf8'));
|
|
1664
|
+
assert.equal(strictTaskless.workflow_policy.assurance_level, 'Strict');
|
|
1665
|
+
assert.equal(strictTaskless.workflow_policy.classified_minimum.assurance_level, 'Elevated');
|
|
1666
|
+
assert.equal(strictTaskless.authoring.tasks, 'absent');
|
|
1667
|
+
} finally {
|
|
1668
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1669
|
+
}
|
|
1670
|
+
});
|
|
1671
|
+
|
|
1672
|
+
test('Full phase scaffold fills canonical 2.1 artifacts and passes E2E validation', () => {
|
|
1673
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-full-phase-'));
|
|
1674
|
+
try {
|
|
1675
|
+
const phase = [{
|
|
1676
|
+
id: 'delivery',
|
|
1677
|
+
task_ids: ['R1-01', 'R1-02'],
|
|
1678
|
+
entry_condition: 'requirements and design are approved',
|
|
1679
|
+
exit_condition: 'task verification plan is complete',
|
|
1680
|
+
owner_boundary: 'B-T',
|
|
1681
|
+
}];
|
|
1682
|
+
const result = run(SCAFFOLD, [
|
|
1683
|
+
'full-phase', '--planning-depth', 'Full', '--tasks', 'R1-01-artifact,R1-02-consumer',
|
|
1684
|
+
'--boundaries', JSON.stringify([{
|
|
1685
|
+
id: 'B-T', type: 'transition', design_ref: 'T1', owner: 'R1-01', consumers: ['R1-02'],
|
|
1686
|
+
precondition: 'Input is ready for transition', postcondition: 'Output is durably complete',
|
|
1687
|
+
failure: 'Failure state is recorded', recovery: 'Recovery resumes from recorded state',
|
|
1688
|
+
}]), '--phases', JSON.stringify(phase),
|
|
1689
|
+
], root);
|
|
1690
|
+
assert.equal(result.status, 0, output(result));
|
|
1691
|
+
const specDir = path.join(root, 'specs/full-phase');
|
|
1692
|
+
const spec = JSON.parse(fs.readFileSync(path.join(specDir, 'spec.json'), 'utf8'));
|
|
1693
|
+
assert.deepEqual(spec.coordination.phases, phase);
|
|
1694
|
+
assert.equal(spec.coordination.boundaries[0].type, 'transition');
|
|
1695
|
+
completeScaffoldedSpec21(specDir);
|
|
1696
|
+
const validation = run(VALIDATOR, [specDir], ROOT);
|
|
1697
|
+
assert.equal(validation.status, 0, output(validation));
|
|
1698
|
+
} finally {
|
|
1699
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1700
|
+
}
|
|
1701
|
+
});
|
|
1702
|
+
|
|
1703
|
+
test('producer scaffold projects typed dependency deliverable into registry edges', () => {
|
|
1704
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-artifacts-'));
|
|
1705
|
+
try {
|
|
1706
|
+
const ordinary = run(SCAFFOLD, [
|
|
1707
|
+
'ordinary', '--tasks', 'R1-01-artifact,R1-02-consumer',
|
|
1708
|
+
'--boundaries', JSON.stringify([{
|
|
1709
|
+
id: 'B-D', type: 'dependency', producer: 'R1-01', consumer: 'R1-02',
|
|
1710
|
+
deliverable: 'artifacts/output.json',
|
|
1711
|
+
}]),
|
|
1712
|
+
], root);
|
|
1713
|
+
assert.equal(ordinary.status, 0, output(ordinary));
|
|
1714
|
+
const ordinarySpec = JSON.parse(fs.readFileSync(path.join(root, 'specs/ordinary/spec.json'), 'utf8'));
|
|
1715
|
+
assert.equal(Object.hasOwn(ordinarySpec.task_registry['tasks/task-R1-01-artifact.md'], 'artifacts'), false);
|
|
1716
|
+
assert.deepEqual(ordinarySpec.task_registry['tasks/task-R1-02-consumer.md'].dependencies, [
|
|
1717
|
+
'tasks/task-R1-01-artifact.md',
|
|
1718
|
+
]);
|
|
1719
|
+
assert.equal(ordinarySpec.coordination.boundaries[0].deliverable, 'artifacts/output.json');
|
|
1720
|
+
} finally {
|
|
1721
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1722
|
+
}
|
|
1723
|
+
});
|
|
58
1724
|
|
|
59
1725
|
test('validator rejects missing explicit requirement and sub-criterion mappings', () => {
|
|
60
1726
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-mapping-'));
|
|
@@ -79,46 +1745,84 @@ test('validator rejects missing explicit requirement and sub-criterion mappings'
|
|
|
79
1745
|
const partialResult = run(VALIDATOR, [partial], ROOT);
|
|
80
1746
|
assert.notEqual(partialResult.status, 0);
|
|
81
1747
|
assert.match(output(partialResult), /R1\.2: acceptance criterion not covered/);
|
|
1748
|
+
|
|
1749
|
+
const standardUnknown = createSpec(root, {
|
|
1750
|
+
name: 'standard-unknown-mapping',
|
|
1751
|
+
task: task({ mapping: '1.1, 9.1' }),
|
|
1752
|
+
spec: { workflow_policy: POLICY.workflowPolicySnapshot({ riskSignals: {} }) },
|
|
1753
|
+
});
|
|
1754
|
+
const standardUnknownResult = run(VALIDATOR, [standardUnknown], ROOT);
|
|
1755
|
+
assert.notEqual(standardUnknownResult.status, 0);
|
|
1756
|
+
assert.match(output(standardUnknownResult), /references unknown requirement R9/);
|
|
82
1757
|
} finally {
|
|
83
1758
|
fs.rmSync(root, { recursive: true, force: true });
|
|
84
1759
|
}
|
|
85
1760
|
});
|
|
86
1761
|
|
|
87
|
-
test('
|
|
1762
|
+
test('v1 adapter rejects unknown, missing, and divergent contract copies', () => {
|
|
88
1763
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-contract-'));
|
|
89
|
-
const design =
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
name
|
|
93
|
-
|
|
1764
|
+
const design = `${designFixture()}<!-- contract:PAYLOAD -->\n\`\`\`json\n{ "id": 1 }\n\`\`\`\n`;
|
|
1765
|
+
function contractFixture(name, { designText = design, extra = '' } = {}) {
|
|
1766
|
+
return createSpec(root, {
|
|
1767
|
+
name,
|
|
1768
|
+
design: designText,
|
|
1769
|
+
task: legacyTask({ extra }),
|
|
1770
|
+
spec: { workflow_policy: legacyWorkflowPolicy() },
|
|
94
1771
|
});
|
|
1772
|
+
}
|
|
1773
|
+
try {
|
|
1774
|
+
const unknown = contractFixture('unknown', { extra: 'Contracts: MISSING\n' });
|
|
95
1775
|
let result = run(VALIDATOR, [unknown], ROOT);
|
|
96
1776
|
assert.notEqual(result.status, 0);
|
|
97
1777
|
assert.match(output(result), /unknown contract "MISSING"/);
|
|
98
1778
|
|
|
99
|
-
const noCanonical =
|
|
100
|
-
|
|
101
|
-
|
|
1779
|
+
const noCanonical = contractFixture('no-canonical', {
|
|
1780
|
+
designText: designFixture(),
|
|
1781
|
+
extra: 'Contracts: PAYLOAD\n',
|
|
102
1782
|
});
|
|
103
1783
|
result = run(VALIDATOR, [noCanonical], ROOT);
|
|
104
1784
|
assert.notEqual(result.status, 0);
|
|
105
1785
|
assert.match(output(result), /defines no canonical contract blocks/);
|
|
106
1786
|
|
|
107
|
-
const missing =
|
|
108
|
-
name: 'missing', design,
|
|
109
|
-
task: task({ extra: 'Contracts: PAYLOAD\n' }),
|
|
110
|
-
});
|
|
1787
|
+
const missing = contractFixture('missing', { extra: 'Contracts: PAYLOAD\n' });
|
|
111
1788
|
result = run(VALIDATOR, [missing], ROOT);
|
|
112
1789
|
assert.notEqual(result.status, 0);
|
|
113
1790
|
assert.match(output(result), /contract "PAYLOAD" is missing/);
|
|
114
1791
|
|
|
115
|
-
const divergent =
|
|
116
|
-
|
|
117
|
-
task: task({ extra: 'Contracts: PAYLOAD\n\n```json\n{ "id": 2 }\n```\n' }),
|
|
1792
|
+
const divergent = contractFixture('divergent', {
|
|
1793
|
+
extra: 'Contracts: PAYLOAD\n\n```json\n{ "id": 2 }\n```\n',
|
|
118
1794
|
});
|
|
119
1795
|
result = run(VALIDATOR, [divergent], ROOT);
|
|
120
1796
|
assert.notEqual(result.status, 0);
|
|
121
1797
|
assert.match(output(result), /contract "PAYLOAD" body diverges/);
|
|
1798
|
+
|
|
1799
|
+
const duplicateDesign = contractFixture('duplicate-design-contract', {
|
|
1800
|
+
designText: `${design}\n<!-- contract:PAYLOAD -->\n\`\`\`json\n{ "id": 1 }\n\`\`\`\n`,
|
|
1801
|
+
});
|
|
1802
|
+
result = run(VALIDATOR, [duplicateDesign], ROOT);
|
|
1803
|
+
assert.notEqual(result.status, 0);
|
|
1804
|
+
assert.match(output(result), /duplicate canonical contract definition "PAYLOAD"/);
|
|
1805
|
+
|
|
1806
|
+
const repeatedDeclaration = contractFixture('repeated-task-contract-line', {
|
|
1807
|
+
extra: 'Contracts: PAYLOAD\nContracts: PAYLOAD\n\n```json\n{ "id": 1 }\n```\n',
|
|
1808
|
+
});
|
|
1809
|
+
result = run(VALIDATOR, [repeatedDeclaration], ROOT);
|
|
1810
|
+
assert.notEqual(result.status, 0);
|
|
1811
|
+
assert.match(output(result), /Contracts declaration must appear at most once/);
|
|
1812
|
+
|
|
1813
|
+
const duplicateName = contractFixture('duplicate-task-contract-name', {
|
|
1814
|
+
extra: 'Contracts: PAYLOAD, PAYLOAD\n\n```json\n{ "id": 1 }\n```\n',
|
|
1815
|
+
});
|
|
1816
|
+
result = run(VALIDATOR, [duplicateName], ROOT);
|
|
1817
|
+
assert.notEqual(result.status, 0);
|
|
1818
|
+
assert.match(output(result), /Contracts declaration contains duplicate name "PAYLOAD"/);
|
|
1819
|
+
|
|
1820
|
+
const orphanTagged = contractFixture('orphan-tagged-contract', {
|
|
1821
|
+
extra: '<!-- contract:PAYLOAD -->\n```json\n{ "id": 1 }\n```\n',
|
|
1822
|
+
});
|
|
1823
|
+
result = run(VALIDATOR, [orphanTagged], ROOT);
|
|
1824
|
+
assert.notEqual(result.status, 0);
|
|
1825
|
+
assert.match(output(result), /must declare it on one Contracts: line/);
|
|
122
1826
|
} finally {
|
|
123
1827
|
fs.rmSync(root, { recursive: true, force: true });
|
|
124
1828
|
}
|
|
@@ -136,8 +1840,8 @@ test('validator rejects invalid ready approval and validation transitions', () =
|
|
|
136
1840
|
});
|
|
137
1841
|
const result = run(VALIDATOR, [specDir], ROOT);
|
|
138
1842
|
assert.notEqual(result.status, 0);
|
|
139
|
-
assert.match(output(result), /requires generated and agent_validated
|
|
140
|
-
assert.match(output(result), /requires generated and agent_validated
|
|
1843
|
+
assert.match(output(result), /requires generated and agent_validated requirements evidence/);
|
|
1844
|
+
assert.match(output(result), /requires generated and agent_validated design evidence/);
|
|
141
1845
|
assert.match(output(result), /validation evidence is incomplete/);
|
|
142
1846
|
|
|
143
1847
|
// Legacy approved field must fail closed with migration guidance
|
|
@@ -170,6 +1874,37 @@ test('validator rejects invalid ready approval and validation transitions', () =
|
|
|
170
1874
|
}
|
|
171
1875
|
});
|
|
172
1876
|
|
|
1877
|
+
test('validator rejects precise square-bracket placeholders without flagging links or checkboxes', () => {
|
|
1878
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-square-placeholders-'));
|
|
1879
|
+
try {
|
|
1880
|
+
for (const [name, placeholder] of [
|
|
1881
|
+
['specific-value', '[specific value]'],
|
|
1882
|
+
['target-users', '[target users]'],
|
|
1883
|
+
['component-name', '[Component Name]'],
|
|
1884
|
+
]) {
|
|
1885
|
+
const specDir = createSpec(root, {
|
|
1886
|
+
name,
|
|
1887
|
+
task: task({ related: '`src/placeholder.js` | Read' }).replace(
|
|
1888
|
+
'Implement the mapped observable behavior.',
|
|
1889
|
+
`Implement the mapped ${placeholder} behavior.`,
|
|
1890
|
+
),
|
|
1891
|
+
});
|
|
1892
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
1893
|
+
assert.notEqual(result.status, 0);
|
|
1894
|
+
assert.match(output(result), /unresolved square-bracket placeholder/);
|
|
1895
|
+
}
|
|
1896
|
+
|
|
1897
|
+
const safe = createSpec(root, {
|
|
1898
|
+
name: 'safe-link-checkbox',
|
|
1899
|
+
task: `${task({ related: '`src/safe.js` | Read' })}\n[Component Name](https://example.invalid)\n- [ ] Keep this real checkbox.\n`,
|
|
1900
|
+
});
|
|
1901
|
+
const safeResult = run(VALIDATOR, [safe], ROOT);
|
|
1902
|
+
assert.equal(safeResult.status, 0, output(safeResult));
|
|
1903
|
+
} finally {
|
|
1904
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
1905
|
+
}
|
|
1906
|
+
});
|
|
1907
|
+
|
|
173
1908
|
test('validator rejects malformed Related Files and dependency topology', () => {
|
|
174
1909
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-shape-'));
|
|
175
1910
|
try {
|
|
@@ -181,7 +1916,8 @@ test('validator rejects malformed Related Files and dependency topology', () =>
|
|
|
181
1916
|
]) {
|
|
182
1917
|
const specDir = createSpec(root, {
|
|
183
1918
|
name,
|
|
184
|
-
task: related ?
|
|
1919
|
+
task: legacyTask({ relatedRows: related ? `| ${related} | Invalid fixture row |` : '' }),
|
|
1920
|
+
spec: { workflow_policy: legacyWorkflowPolicy() },
|
|
185
1921
|
});
|
|
186
1922
|
const result = run(VALIDATOR, [specDir], ROOT);
|
|
187
1923
|
assert.notEqual(result.status, 0);
|
|
@@ -191,10 +1927,19 @@ test('validator rejects malformed Related Files and dependency topology', () =>
|
|
|
191
1927
|
const cycle = createSpec(root, { name: 'cycle' });
|
|
192
1928
|
const statePath = path.join(cycle, 'spec.json');
|
|
193
1929
|
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
1930
|
+
state.coordination.task_triggers = ['distinct_ownership', 'real_dependency'];
|
|
194
1931
|
state.task_files.push('tasks/task-R1-02-two.md');
|
|
195
1932
|
state.task_registry['tasks/task-R1-01-one.md'].dependencies = ['tasks/task-R1-02-two.md'];
|
|
196
1933
|
state.task_registry['tasks/task-R1-02-two.md'] = { id: 'R1-02', title: 'Two', status: 'pending', dependencies: ['tasks/task-R1-01-one.md'], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
197
|
-
write(path.join(cycle, 'tasks/task-R1-
|
|
1934
|
+
write(path.join(cycle, 'tasks/task-R1-01-one.md'), task({
|
|
1935
|
+
related: '`src/one.js` | Read',
|
|
1936
|
+
dependencies: 'tasks/task-R1-02-two.md',
|
|
1937
|
+
}));
|
|
1938
|
+
write(path.join(cycle, 'tasks/task-R1-02-two.md'), task({
|
|
1939
|
+
id: 'R1-02',
|
|
1940
|
+
related: '`src/two.js` | Read',
|
|
1941
|
+
dependencies: 'tasks/task-R1-01-one.md',
|
|
1942
|
+
}));
|
|
198
1943
|
write(statePath, JSON.stringify(state, null, 2));
|
|
199
1944
|
const cycleResult = run(VALIDATOR, [cycle], ROOT);
|
|
200
1945
|
assert.notEqual(cycleResult.status, 0);
|
|
@@ -208,7 +1953,11 @@ test('validator rejects malformed Related Files and dependency topology', () =>
|
|
|
208
1953
|
const orderingState = JSON.parse(fs.readFileSync(orderingStatePath, 'utf8'));
|
|
209
1954
|
orderingState.task_files.push('tasks/task-R1-02-two.md');
|
|
210
1955
|
orderingState.task_registry['tasks/task-R1-02-two.md'] = { id: 'R1-02', title: 'Two', status: 'pending', dependencies: [], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
211
|
-
write(path.join(ordering, 'tasks/task-R1-02-two.md'), task({
|
|
1956
|
+
write(path.join(ordering, 'tasks/task-R1-02-two.md'), task({
|
|
1957
|
+
id: 'R1-02',
|
|
1958
|
+
related: '`src/generated.js` | Modify',
|
|
1959
|
+
relatedRows: '| `src/generated.js` | Modify | Consume generated implementation |',
|
|
1960
|
+
}));
|
|
212
1961
|
write(orderingStatePath, JSON.stringify(orderingState, null, 2));
|
|
213
1962
|
const orderingResult = run(VALIDATOR, [ordering], ROOT);
|
|
214
1963
|
assert.notEqual(orderingResult.status, 0);
|
|
@@ -218,6 +1967,608 @@ test('validator rejects malformed Related Files and dependency topology', () =>
|
|
|
218
1967
|
}
|
|
219
1968
|
});
|
|
220
1969
|
|
|
1970
|
+
test('scaffold validates one dependency map before writes and persists canonical registry edges', () => {
|
|
1971
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-scaffold-dependencies-'));
|
|
1972
|
+
const taskList = 'R1-01-owner,R1-02-consumer';
|
|
1973
|
+
const dependencyMap = JSON.stringify([{
|
|
1974
|
+
id: 'B-D', type: 'dependency', producer: 'R1-01', consumer: 'R1-02',
|
|
1975
|
+
deliverable: 'artifacts/owner-output.json',
|
|
1976
|
+
}]);
|
|
1977
|
+
try {
|
|
1978
|
+
const standard = run(SCAFFOLD, [
|
|
1979
|
+
'dependency-positive', '--lane', 'Standard', '--tasks', taskList,
|
|
1980
|
+
'--boundaries', dependencyMap,
|
|
1981
|
+
], root);
|
|
1982
|
+
assert.equal(standard.status, 0, output(standard));
|
|
1983
|
+
const standardDir = path.join(root, 'specs/dependency-positive');
|
|
1984
|
+
const standardSpec = JSON.parse(fs.readFileSync(path.join(standardDir, 'spec.json'), 'utf8'));
|
|
1985
|
+
assert.deepEqual(
|
|
1986
|
+
standardSpec.task_registry['tasks/task-R1-02-consumer.md'].dependencies,
|
|
1987
|
+
['tasks/task-R1-01-owner.md'],
|
|
1988
|
+
);
|
|
1989
|
+
assert.deepEqual(standardSpec.coordination.boundaries, JSON.parse(dependencyMap));
|
|
1990
|
+
const standardTask = fs.readFileSync(
|
|
1991
|
+
path.join(standardDir, 'tasks/task-R1-02-consumer.md'),
|
|
1992
|
+
'utf8',
|
|
1993
|
+
);
|
|
1994
|
+
assert.doesNotMatch(standardTask, /^## Execution Closure/m);
|
|
1995
|
+
assert.doesNotMatch(
|
|
1996
|
+
fs.readFileSync(path.join(standardDir, 'design.md'), 'utf8'),
|
|
1997
|
+
/^## Execution Closure/m,
|
|
1998
|
+
);
|
|
1999
|
+
|
|
2000
|
+
const invalid = run(SCAFFOLD, [
|
|
2001
|
+
'dependency-invalid', '--tasks', taskList, '--boundaries', JSON.stringify([{
|
|
2002
|
+
id: 'B-D', type: 'dependency', producer: 'R1-01', consumer: 'R1-02',
|
|
2003
|
+
}]),
|
|
2004
|
+
], root);
|
|
2005
|
+
assert.equal(invalid.status, 2, output(invalid));
|
|
2006
|
+
assert.match(output(invalid), /fields must be exactly.*deliverable|dependency fields must be exactly/);
|
|
2007
|
+
assert.equal(fs.existsSync(path.join(root, 'specs/dependency-invalid')), false);
|
|
2008
|
+
|
|
2009
|
+
} finally {
|
|
2010
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
2011
|
+
}
|
|
2012
|
+
});
|
|
2013
|
+
|
|
2014
|
+
test('task-level (P) requires dependency-free and non-overlapping task ownership', () => {
|
|
2015
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-parallel-'));
|
|
2016
|
+
const ownerPath = 'tasks/task-R1-01-one.md';
|
|
2017
|
+
const siblingPath = 'tasks/task-R1-02-two.md';
|
|
2018
|
+
|
|
2019
|
+
function parallelFixture(name, { dependency = false, overlap = false } = {}) {
|
|
2020
|
+
const ownerTask = task({
|
|
2021
|
+
related: overlap ? '`src/shared.js` | Read' : '`src/parallel-a.js` | Read',
|
|
2022
|
+
relatedRows: overlap
|
|
2023
|
+
? '| `src/shared.js` | Read | Inspect existing |\n| `src/shared.js` | Modify | Update owned behavior |'
|
|
2024
|
+
: null,
|
|
2025
|
+
parallel: true,
|
|
2026
|
+
});
|
|
2027
|
+
const specDir = createSpec(root, { name, task: ownerTask });
|
|
2028
|
+
const statePath = path.join(specDir, 'spec.json');
|
|
2029
|
+
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
2030
|
+
state.coordination.task_triggers = dependency
|
|
2031
|
+
? ['parallel_coordination', 'real_dependency', 'separate_proof']
|
|
2032
|
+
: ['parallel_coordination', 'separate_proof'];
|
|
2033
|
+
state.task_files.push(siblingPath);
|
|
2034
|
+
state.task_registry[siblingPath] = {
|
|
2035
|
+
id: 'R1-02', title: 'Two', status: 'pending',
|
|
2036
|
+
dependencies: dependency ? [ownerPath] : [],
|
|
2037
|
+
blocker: null, started_at: null, completed_at: null, last_updated_at: null,
|
|
2038
|
+
};
|
|
2039
|
+
const sibling = task({
|
|
2040
|
+
id: 'R1-02',
|
|
2041
|
+
related: overlap ? '`src/shared.js` | Read' : '`src/parallel-b.js` | Read',
|
|
2042
|
+
dependencies: dependency ? ownerPath : 'none',
|
|
2043
|
+
parallel: dependency,
|
|
2044
|
+
});
|
|
2045
|
+
write(path.join(specDir, siblingPath), sibling);
|
|
2046
|
+
write(statePath, JSON.stringify(state, null, 2));
|
|
2047
|
+
return specDir;
|
|
2048
|
+
}
|
|
2049
|
+
|
|
2050
|
+
try {
|
|
2051
|
+
const valid = parallelFixture('parallel-valid');
|
|
2052
|
+
assert.equal(run(VALIDATOR, [valid], ROOT).status, 0);
|
|
2053
|
+
|
|
2054
|
+
const dependency = parallelFixture('parallel-dependency', { dependency: true });
|
|
2055
|
+
const dependencyResult = run(VALIDATOR, [dependency], ROOT);
|
|
2056
|
+
assert.notEqual(dependencyResult.status, 0);
|
|
2057
|
+
assert.match(output(dependencyResult), /task-level \(P\) cannot declare sibling dependencies/);
|
|
2058
|
+
|
|
2059
|
+
const overlap = parallelFixture('parallel-overlap', { overlap: true });
|
|
2060
|
+
const overlapResult = run(VALIDATOR, [overlap], ROOT);
|
|
2061
|
+
assert.notEqual(overlapResult.status, 0);
|
|
2062
|
+
assert.match(output(overlapResult), /task-level \(P\) has Related Files contention on src\/shared\.js/);
|
|
2063
|
+
} finally {
|
|
2064
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
2065
|
+
}
|
|
2066
|
+
});
|
|
2067
|
+
|
|
2068
|
+
test('validator rejects dependency drift and non-canonical readiness statuses', () => {
|
|
2069
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-readiness-'));
|
|
2070
|
+
const ownerPath = 'tasks/task-R1-01-one.md';
|
|
2071
|
+
const consumerPath = 'tasks/task-R1-02-two.md';
|
|
2072
|
+
try {
|
|
2073
|
+
const driftDir = createSpec(root, {
|
|
2074
|
+
name: 'dependency-aligned',
|
|
2075
|
+
task: task({ related: '`src/owner.js` | Read' }),
|
|
2076
|
+
});
|
|
2077
|
+
const driftStatePath = path.join(driftDir, 'spec.json');
|
|
2078
|
+
const driftState = JSON.parse(fs.readFileSync(driftStatePath, 'utf8'));
|
|
2079
|
+
driftState.coordination.task_triggers = ['real_dependency', 'separate_proof'];
|
|
2080
|
+
driftState.task_files.push(consumerPath);
|
|
2081
|
+
driftState.task_registry[consumerPath] = {
|
|
2082
|
+
id: 'R1-02', title: 'Two', status: 'pending', dependencies: [ownerPath], blocker: null,
|
|
2083
|
+
started_at: null, completed_at: null, last_updated_at: null,
|
|
2084
|
+
};
|
|
2085
|
+
write(
|
|
2086
|
+
path.join(driftDir, consumerPath),
|
|
2087
|
+
task({
|
|
2088
|
+
id: 'R1-02',
|
|
2089
|
+
related: '`src/consumer.js` | Read',
|
|
2090
|
+
dependencies: ownerPath,
|
|
2091
|
+
}),
|
|
2092
|
+
);
|
|
2093
|
+
write(driftStatePath, JSON.stringify(driftState, null, 2));
|
|
2094
|
+
const aligned = run(VALIDATOR, [driftDir], ROOT);
|
|
2095
|
+
assert.equal(aligned.status, 0, output(aligned));
|
|
2096
|
+
|
|
2097
|
+
const consumerFile = path.join(driftDir, consumerPath);
|
|
2098
|
+
write(
|
|
2099
|
+
consumerFile,
|
|
2100
|
+
fs.readFileSync(consumerFile, 'utf8').replace(
|
|
2101
|
+
`- ${ownerPath}\n\n## Verification Plan`,
|
|
2102
|
+
'- none\n\n## Verification Plan',
|
|
2103
|
+
),
|
|
2104
|
+
);
|
|
2105
|
+
const drifted = run(VALIDATOR, [driftDir], ROOT);
|
|
2106
|
+
assert.notEqual(drifted.status, 0);
|
|
2107
|
+
assert.match(output(drifted), /dependency drift between spec\.json task_registry/);
|
|
2108
|
+
|
|
2109
|
+
const duplicateHeader = createSpec(root, {
|
|
2110
|
+
name: 'duplicate-dependency-section',
|
|
2111
|
+
task: `${task({ related: '`src/duplicate.js` | Read' })}\n## Dependencies\n\n- none\n`,
|
|
2112
|
+
});
|
|
2113
|
+
const duplicateHeaderResult = run(VALIDATOR, [duplicateHeader], ROOT);
|
|
2114
|
+
assert.notEqual(duplicateHeaderResult.status, 0);
|
|
2115
|
+
assert.match(output(duplicateHeaderResult), /machine-read section Dependencies.*must appear at most once/);
|
|
2116
|
+
|
|
2117
|
+
const duplicateSections = createSpec(root, {
|
|
2118
|
+
name: 'duplicate-machine-sections',
|
|
2119
|
+
task: `${task({ related: '`src/duplicate-sections.js` | Read' })}\n` +
|
|
2120
|
+
'## Verification Plan\n- **Command:** `npm test`\n\n' +
|
|
2121
|
+
'## Acceptance\n- **R1.1:** conflicting duplicate mapping\n',
|
|
2122
|
+
});
|
|
2123
|
+
const duplicateSectionsResult = run(VALIDATOR, [duplicateSections], ROOT);
|
|
2124
|
+
assert.notEqual(duplicateSectionsResult.status, 0);
|
|
2125
|
+
assert.match(output(duplicateSectionsResult), /machine-read section Verification Plan.*must appear at most once/);
|
|
2126
|
+
assert.match(output(duplicateSectionsResult), /machine-read section Acceptance.*must appear at most once/);
|
|
2127
|
+
|
|
2128
|
+
const ordinary = createSpec(root, {
|
|
2129
|
+
name: 'ordinary-in-progress',
|
|
2130
|
+
task: task({ related: '`src/ordinary.js` | Read', status: 'in_progress' }),
|
|
2131
|
+
spec: {
|
|
2132
|
+
task_registry: {
|
|
2133
|
+
[ownerPath]: {
|
|
2134
|
+
id: 'R1-01', title: 'One', status: 'in_progress', dependencies: [], blocker: null,
|
|
2135
|
+
started_at: '2026-08-12T00:00:00+07:00', completed_at: null,
|
|
2136
|
+
last_updated_at: '2026-08-12T00:01:00+07:00',
|
|
2137
|
+
artifacts: ['test/one.test.js'],
|
|
2138
|
+
},
|
|
2139
|
+
},
|
|
2140
|
+
},
|
|
2141
|
+
});
|
|
2142
|
+
const ordinaryResult = run(VALIDATOR, [ordinary], ROOT);
|
|
2143
|
+
assert.equal(ordinaryResult.status, 0, output(ordinaryResult));
|
|
2144
|
+
|
|
2145
|
+
const ordinaryStatePath = path.join(ordinary, 'spec.json');
|
|
2146
|
+
const ordinaryState = JSON.parse(fs.readFileSync(ordinaryStatePath, 'utf8'));
|
|
2147
|
+
const ordinaryTaskPath = path.join(ordinary, ownerPath);
|
|
2148
|
+
write(
|
|
2149
|
+
ordinaryTaskPath,
|
|
2150
|
+
fs.readFileSync(ordinaryTaskPath, 'utf8')
|
|
2151
|
+
.replace(/\n## Dependencies\n[\s\S]*?(?=\n## Verification Plan)/, ''),
|
|
2152
|
+
);
|
|
2153
|
+
ordinaryState.ready_for_implementation = true;
|
|
2154
|
+
ordinaryState.validation = { status: 'completed' };
|
|
2155
|
+
ordinaryState.timestamps = { validation_done: '2026-08-12T00:00:00+07:00' };
|
|
2156
|
+
write(ordinaryStatePath, JSON.stringify(ordinaryState, null, 2));
|
|
2157
|
+
const readyWithoutHeader = run(VALIDATOR, [ordinary], ROOT);
|
|
2158
|
+
assert.notEqual(readyWithoutHeader.status, 0);
|
|
2159
|
+
assert.match(output(readyWithoutHeader), /missing Dependencies/);
|
|
2160
|
+
|
|
2161
|
+
const invalidCases = [
|
|
2162
|
+
['bad-ready-type', (state) => { state.ready_for_implementation = 'true'; }, /ready_for_implementation: must be a boolean/],
|
|
2163
|
+
['bad-spec-status', (state) => { state.status = 'ready'; }, /spec\.json\.status: must be one of/],
|
|
2164
|
+
['bad-task-status', (state) => { state.task_registry[ownerPath].status = 'ready'; }, /task_registry.*status: must be one of/],
|
|
2165
|
+
['bad-validation-shape', (state) => { state.validation = 'not-run'; }, /validation: must be an object/],
|
|
2166
|
+
[
|
|
2167
|
+
'registry-id-path-drift',
|
|
2168
|
+
(state) => { state.task_registry[ownerPath].id = 'R9-99'; },
|
|
2169
|
+
/task_registry id must match path-derived id R1-01/,
|
|
2170
|
+
],
|
|
2171
|
+
[
|
|
2172
|
+
'markdown-heading-id-drift',
|
|
2173
|
+
(_state, specDir) => {
|
|
2174
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2175
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('# Task R1-01:', '# Task R9-99:'));
|
|
2176
|
+
},
|
|
2177
|
+
/Markdown heading id R9-99 must match path-derived id R1-01/,
|
|
2178
|
+
],
|
|
2179
|
+
[
|
|
2180
|
+
'registry-markdown-status-drift',
|
|
2181
|
+
(state) => {
|
|
2182
|
+
state.task_registry[ownerPath].status = 'done';
|
|
2183
|
+
state.task_registry[ownerPath].started_at = '2026-08-12T00:00:00+07:00';
|
|
2184
|
+
state.task_registry[ownerPath].completed_at = '2026-08-12T00:01:00+07:00';
|
|
2185
|
+
state.task_registry[ownerPath].last_updated_at = '2026-08-12T00:01:00+07:00';
|
|
2186
|
+
},
|
|
2187
|
+
/status drift between spec\.json task_registry done and Markdown Status header pending/,
|
|
2188
|
+
],
|
|
2189
|
+
[
|
|
2190
|
+
'done-missing-lifecycle-timestamps',
|
|
2191
|
+
(state, specDir) => {
|
|
2192
|
+
state.task_registry[ownerPath].status = 'done';
|
|
2193
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2194
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** done'));
|
|
2195
|
+
},
|
|
2196
|
+
/done status requires started_at, completed_at, and last_updated_at timestamps/,
|
|
2197
|
+
],
|
|
2198
|
+
[
|
|
2199
|
+
'blocked-missing-blocker',
|
|
2200
|
+
(state, specDir) => {
|
|
2201
|
+
state.task_registry[ownerPath].status = 'blocked';
|
|
2202
|
+
state.task_registry[ownerPath].last_updated_at = '2026-08-12T00:01:00+07:00';
|
|
2203
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2204
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** blocked'));
|
|
2205
|
+
},
|
|
2206
|
+
/blocked status requires a non-empty blocker/,
|
|
2207
|
+
],
|
|
2208
|
+
[
|
|
2209
|
+
'in-progress-with-blocker',
|
|
2210
|
+
(state, specDir) => {
|
|
2211
|
+
state.task_registry[ownerPath].status = 'in_progress';
|
|
2212
|
+
state.task_registry[ownerPath].blocker = 'waiting on owner';
|
|
2213
|
+
state.task_registry[ownerPath].started_at = '2026-08-12T00:00:00+07:00';
|
|
2214
|
+
state.task_registry[ownerPath].last_updated_at = '2026-08-12T00:01:00+07:00';
|
|
2215
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2216
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** in_progress'));
|
|
2217
|
+
},
|
|
2218
|
+
/in_progress status requires blocker null/,
|
|
2219
|
+
],
|
|
2220
|
+
[
|
|
2221
|
+
'in-progress-reversed-timestamps',
|
|
2222
|
+
(state, specDir) => {
|
|
2223
|
+
state.task_registry[ownerPath].status = 'in_progress';
|
|
2224
|
+
state.task_registry[ownerPath].started_at = '2026-08-12T00:02:00+07:00';
|
|
2225
|
+
state.task_registry[ownerPath].last_updated_at = '2026-08-12T00:01:00+07:00';
|
|
2226
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2227
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** in_progress'));
|
|
2228
|
+
},
|
|
2229
|
+
/in_progress timestamps must satisfy started_at <= last_updated_at/,
|
|
2230
|
+
],
|
|
2231
|
+
[
|
|
2232
|
+
'started-blocked-reversed-timestamps',
|
|
2233
|
+
(state, specDir) => {
|
|
2234
|
+
state.task_registry[ownerPath].status = 'blocked';
|
|
2235
|
+
state.task_registry[ownerPath].blocker = 'waiting for the declared external authority';
|
|
2236
|
+
state.task_registry[ownerPath].started_at = '2026-08-12T00:02:00+07:00';
|
|
2237
|
+
state.task_registry[ownerPath].last_updated_at = '2026-08-12T00:01:00+07:00';
|
|
2238
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2239
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** blocked'));
|
|
2240
|
+
},
|
|
2241
|
+
/started blocked timestamps must satisfy started_at <= last_updated_at/,
|
|
2242
|
+
],
|
|
2243
|
+
[
|
|
2244
|
+
'bad-validation-status',
|
|
2245
|
+
(state) => {
|
|
2246
|
+
state.ready_for_implementation = true;
|
|
2247
|
+
state.validation = { status: 'ready' };
|
|
2248
|
+
state.timestamps = { validation_done: '2026-08-12T00:00:00+07:00' };
|
|
2249
|
+
},
|
|
2250
|
+
/validation\.status: must be one of|requires validation\.status completed/,
|
|
2251
|
+
],
|
|
2252
|
+
[
|
|
2253
|
+
'blocked-ready',
|
|
2254
|
+
(state, specDir) => {
|
|
2255
|
+
state.ready_for_implementation = true;
|
|
2256
|
+
state.status = 'blocked';
|
|
2257
|
+
state.blocker = 'owner unavailable';
|
|
2258
|
+
state.validation = { status: 'completed' };
|
|
2259
|
+
state.timestamps = { validation_done: '2026-08-12T00:00:00+07:00' };
|
|
2260
|
+
state.task_registry[ownerPath].status = 'blocked';
|
|
2261
|
+
const taskFile = path.join(specDir, ownerPath);
|
|
2262
|
+
write(taskFile, fs.readFileSync(taskFile, 'utf8').replace('**Status:** pending', '**Status:** blocked'));
|
|
2263
|
+
},
|
|
2264
|
+
/cannot be blocked while ready_for_implementation is true/,
|
|
2265
|
+
],
|
|
2266
|
+
];
|
|
2267
|
+
for (const [name, mutate, expected] of invalidCases) {
|
|
2268
|
+
const specDir = createSpec(root, {
|
|
2269
|
+
name,
|
|
2270
|
+
task: task({ related: '`src/status.js` | Read' }),
|
|
2271
|
+
});
|
|
2272
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
2273
|
+
const state = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
2274
|
+
mutate(state, specDir);
|
|
2275
|
+
write(specPath, JSON.stringify(state, null, 2));
|
|
2276
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
2277
|
+
assert.notEqual(result.status, 0, `${name} unexpectedly passed`);
|
|
2278
|
+
assert.match(output(result), expected);
|
|
2279
|
+
}
|
|
2280
|
+
} finally {
|
|
2281
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
2282
|
+
}
|
|
2283
|
+
});
|
|
2284
|
+
|
|
2285
|
+
test('v1 Critical implementation obligations validate ownership, references, closure, and proof', () => {
|
|
2286
|
+
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-obligations-'));
|
|
2287
|
+
const ownerPath = 'tasks/task-R1-01-one.md';
|
|
2288
|
+
const consumerPath = 'tasks/task-R1-02-two.md';
|
|
2289
|
+
const criticalPolicy = legacyWorkflowPolicy('Critical');
|
|
2290
|
+
|
|
2291
|
+
function makeFixture(name, mutate = () => {}, policy = criticalPolicy) {
|
|
2292
|
+
const obligation = {
|
|
2293
|
+
id: 'canonical-transition',
|
|
2294
|
+
requirements: ['R1.1'],
|
|
2295
|
+
canonical_state: 'Persisted transition state and allowed values',
|
|
2296
|
+
authority: 'Owner module is the sole transition authority',
|
|
2297
|
+
recovery: 'Retry reconciles the persisted state before continuing',
|
|
2298
|
+
owner_task: ownerPath,
|
|
2299
|
+
consumer_tasks: [consumerPath],
|
|
2300
|
+
verification: 'node --test focused.test.js',
|
|
2301
|
+
evidence: 'Exit 0 and the declared state transition is observed',
|
|
2302
|
+
};
|
|
2303
|
+
const specDir = createSpec(root, {
|
|
2304
|
+
name,
|
|
2305
|
+
task: task({ related: '`src/owner.js` | Read' }),
|
|
2306
|
+
spec: { workflow_policy: policy },
|
|
2307
|
+
});
|
|
2308
|
+
const specPath = path.join(specDir, 'spec.json');
|
|
2309
|
+
write(path.join(specDir, 'research.md'), '# Research\n\n## Evidence Summary\n\n- The auth-risk fixture requires independent grounding before implementation.\n');
|
|
2310
|
+
const state = JSON.parse(fs.readFileSync(specPath, 'utf8'));
|
|
2311
|
+
state.task_files.push(consumerPath);
|
|
2312
|
+
state.task_registry[consumerPath] = {
|
|
2313
|
+
id: 'R1-02', title: 'Two', status: 'pending', dependencies: [ownerPath], blocker: null,
|
|
2314
|
+
started_at: null, completed_at: null, last_updated_at: null,
|
|
2315
|
+
};
|
|
2316
|
+
write(
|
|
2317
|
+
path.join(specDir, consumerPath),
|
|
2318
|
+
task({
|
|
2319
|
+
id: 'R1-02',
|
|
2320
|
+
related: '`src/consumer.js` | Read',
|
|
2321
|
+
dependencies: ownerPath,
|
|
2322
|
+
extra: executionClosure(obligation),
|
|
2323
|
+
})
|
|
2324
|
+
.replace('# Task R1-01: One', '# Task R1-02: Two'),
|
|
2325
|
+
);
|
|
2326
|
+
const ownerFile = path.join(specDir, ownerPath);
|
|
2327
|
+
write(
|
|
2328
|
+
ownerFile,
|
|
2329
|
+
`${fs.readFileSync(ownerFile, 'utf8')}${executionClosure(obligation)}`,
|
|
2330
|
+
);
|
|
2331
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2332
|
+
write(
|
|
2333
|
+
designFile,
|
|
2334
|
+
`${fs.readFileSync(designFile, 'utf8')}${executionClosure(obligation)}`,
|
|
2335
|
+
);
|
|
2336
|
+
state.implementation_obligations = [obligation];
|
|
2337
|
+
mutate(state, specDir);
|
|
2338
|
+
write(specPath, JSON.stringify(state, null, 2));
|
|
2339
|
+
return specDir;
|
|
2340
|
+
}
|
|
2341
|
+
|
|
2342
|
+
try {
|
|
2343
|
+
const omitted = createSpec(root, {
|
|
2344
|
+
name: 'critical-omitted-obligation-decision',
|
|
2345
|
+
spec: { workflow_policy: criticalPolicy },
|
|
2346
|
+
});
|
|
2347
|
+
write(path.join(omitted, 'research.md'), '# Research\n\n## Evidence Summary\n\n- The auth-risk fixture requires independent grounding before implementation.\n');
|
|
2348
|
+
const omittedResult = run(VALIDATOR, [omitted], ROOT);
|
|
2349
|
+
assert.notEqual(omittedResult.status, 0);
|
|
2350
|
+
assert.match(output(omittedResult), /Critical scope must declare non-empty.*or one justified N\/A/);
|
|
2351
|
+
|
|
2352
|
+
const valid = makeFixture('valid-obligation');
|
|
2353
|
+
const validResult = run(VALIDATOR, [valid], ROOT);
|
|
2354
|
+
assert.equal(validResult.status, 0, output(validResult));
|
|
2355
|
+
|
|
2356
|
+
const tableClosure = makeFixture('valid-table-closure', (state, specDir) => {
|
|
2357
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2358
|
+
const designPrefix = fs.readFileSync(designFile, 'utf8').split('\n## Execution Closure\n')[0];
|
|
2359
|
+
write(designFile, `${designPrefix}${executionClosureTable(state.implementation_obligations[0])}`);
|
|
2360
|
+
});
|
|
2361
|
+
const tableClosureResult = run(VALIDATOR, [tableClosure], ROOT);
|
|
2362
|
+
assert.equal(tableClosureResult.status, 0, output(tableClosureResult));
|
|
2363
|
+
|
|
2364
|
+
const swapped = makeFixture('marker-local-swapped-obligations', (state, specDir) => {
|
|
2365
|
+
const first = state.implementation_obligations[0];
|
|
2366
|
+
const second = {
|
|
2367
|
+
...first,
|
|
2368
|
+
id: 'secondary-transition',
|
|
2369
|
+
canonical_state: 'Secondary persisted state and terminal values',
|
|
2370
|
+
authority: 'Secondary module is the sole transition authority',
|
|
2371
|
+
recovery: 'Restart rolls the secondary state back before retry',
|
|
2372
|
+
verification: 'node --test secondary-focused.test.js',
|
|
2373
|
+
evidence: 'Exit 0 and the secondary persisted transition is observed',
|
|
2374
|
+
};
|
|
2375
|
+
state.implementation_obligations.push(second);
|
|
2376
|
+
for (const taskFile of [ownerPath, consumerPath]) {
|
|
2377
|
+
const filePath = path.join(specDir, taskFile);
|
|
2378
|
+
write(filePath, `${fs.readFileSync(filePath, 'utf8')}\n${obligationClosureBlock(second)}`);
|
|
2379
|
+
}
|
|
2380
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2381
|
+
const designPrefix = fs.readFileSync(designFile, 'utf8').split('\n## Execution Closure\n')[0];
|
|
2382
|
+
const firstWithSecondValues = { ...second, id: first.id };
|
|
2383
|
+
const secondWithFirstValues = { ...first, id: second.id };
|
|
2384
|
+
write(
|
|
2385
|
+
designFile,
|
|
2386
|
+
`${designPrefix}\n## Execution Closure\n\n` +
|
|
2387
|
+
`${obligationClosureBlock(firstWithSecondValues)}\n${obligationClosureBlock(secondWithFirstValues)}`,
|
|
2388
|
+
);
|
|
2389
|
+
});
|
|
2390
|
+
const swappedResult = run(VALIDATOR, [swapped], ROOT);
|
|
2391
|
+
assert.notEqual(swappedResult.status, 0);
|
|
2392
|
+
assert.match(output(swappedResult), /marker-local field/);
|
|
2393
|
+
|
|
2394
|
+
const cases = [
|
|
2395
|
+
[
|
|
2396
|
+
'standard-only-rejected',
|
|
2397
|
+
() => {},
|
|
2398
|
+
legacyWorkflowPolicy(),
|
|
2399
|
+
/allowed only for Critical workflow policy/,
|
|
2400
|
+
],
|
|
2401
|
+
[
|
|
2402
|
+
'owner-must-be-one-task',
|
|
2403
|
+
(state) => { state.implementation_obligations[0].owner_task = [ownerPath]; },
|
|
2404
|
+
criticalPolicy,
|
|
2405
|
+
/owner_task: must name exactly one registered task/,
|
|
2406
|
+
],
|
|
2407
|
+
[
|
|
2408
|
+
'unknown-requirement',
|
|
2409
|
+
(state) => { state.implementation_obligations[0].requirements = ['R9.1']; },
|
|
2410
|
+
criticalPolicy,
|
|
2411
|
+
/unknown requirement reference R9\.1/,
|
|
2412
|
+
],
|
|
2413
|
+
[
|
|
2414
|
+
'unknown-consumer',
|
|
2415
|
+
(state) => { state.implementation_obligations[0].consumer_tasks = ['tasks/task-R1-99-missing.md']; },
|
|
2416
|
+
criticalPolicy,
|
|
2417
|
+
/unknown task reference tasks\/task-R1-99-missing\.md/,
|
|
2418
|
+
],
|
|
2419
|
+
[
|
|
2420
|
+
'missing-dependency-closure',
|
|
2421
|
+
(state, specDir) => {
|
|
2422
|
+
state.task_registry[consumerPath].dependencies = [];
|
|
2423
|
+
const taskFile = path.join(specDir, consumerPath);
|
|
2424
|
+
write(
|
|
2425
|
+
taskFile,
|
|
2426
|
+
fs.readFileSync(taskFile, 'utf8').replace(
|
|
2427
|
+
`- ${ownerPath}\n\n## Verification Plan`,
|
|
2428
|
+
'- none\n\n## Verification Plan',
|
|
2429
|
+
),
|
|
2430
|
+
);
|
|
2431
|
+
},
|
|
2432
|
+
criticalPolicy,
|
|
2433
|
+
/must depend on owner tasks\/task-R1-01-one\.md/,
|
|
2434
|
+
],
|
|
2435
|
+
[
|
|
2436
|
+
'missing-proof',
|
|
2437
|
+
(state) => {
|
|
2438
|
+
state.implementation_obligations[0].verification = 'TBD';
|
|
2439
|
+
state.implementation_obligations[0].evidence = 'PENDING';
|
|
2440
|
+
},
|
|
2441
|
+
criticalPolicy,
|
|
2442
|
+
/verification: must be an executable command|evidence: must name an observable result/,
|
|
2443
|
+
],
|
|
2444
|
+
[
|
|
2445
|
+
'missing-state-authority-recovery',
|
|
2446
|
+
(state) => {
|
|
2447
|
+
delete state.implementation_obligations[0].canonical_state;
|
|
2448
|
+
state.implementation_obligations[0].authority = 'authority';
|
|
2449
|
+
state.implementation_obligations[0].recovery = 'recovery';
|
|
2450
|
+
},
|
|
2451
|
+
criticalPolicy,
|
|
2452
|
+
/canonical_state: must be concrete|authority: must be concrete|recovery: must be concrete/,
|
|
2453
|
+
],
|
|
2454
|
+
[
|
|
2455
|
+
'missing-design-correspondence',
|
|
2456
|
+
(_state, specDir) => {
|
|
2457
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2458
|
+
write(
|
|
2459
|
+
designFile,
|
|
2460
|
+
fs.readFileSync(designFile, 'utf8').replace(
|
|
2461
|
+
'\n<!-- implementation-obligation:canonical-transition -->\n',
|
|
2462
|
+
'\n',
|
|
2463
|
+
),
|
|
2464
|
+
);
|
|
2465
|
+
},
|
|
2466
|
+
criticalPolicy,
|
|
2467
|
+
/design\.md: missing implementation obligation marker canonical-transition/,
|
|
2468
|
+
],
|
|
2469
|
+
[
|
|
2470
|
+
'missing-owner-correspondence',
|
|
2471
|
+
(_state, specDir) => {
|
|
2472
|
+
const ownerFile = path.join(specDir, ownerPath);
|
|
2473
|
+
write(
|
|
2474
|
+
ownerFile,
|
|
2475
|
+
fs.readFileSync(ownerFile, 'utf8').replace(
|
|
2476
|
+
'\n<!-- implementation-obligation:canonical-transition -->\n',
|
|
2477
|
+
'\n',
|
|
2478
|
+
),
|
|
2479
|
+
);
|
|
2480
|
+
},
|
|
2481
|
+
criticalPolicy,
|
|
2482
|
+
/task-R1-01-one\.md: missing implementation obligation marker canonical-transition/,
|
|
2483
|
+
],
|
|
2484
|
+
[
|
|
2485
|
+
'marker-outside-execution-closure',
|
|
2486
|
+
(_state, specDir) => {
|
|
2487
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2488
|
+
const body = fs.readFileSync(designFile, 'utf8')
|
|
2489
|
+
.replace('\n<!-- implementation-obligation:canonical-transition -->\n', '\n');
|
|
2490
|
+
write(designFile, `${body}\n## Outside Closure\n\n<!-- implementation-obligation:canonical-transition -->\n`);
|
|
2491
|
+
},
|
|
2492
|
+
criticalPolicy,
|
|
2493
|
+
/implementation obligation marker canonical-transition must be inside Execution Closure/,
|
|
2494
|
+
],
|
|
2495
|
+
[
|
|
2496
|
+
'duplicate-closure-marker',
|
|
2497
|
+
(_state, specDir) => {
|
|
2498
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2499
|
+
write(
|
|
2500
|
+
designFile,
|
|
2501
|
+
fs.readFileSync(designFile, 'utf8').replace(
|
|
2502
|
+
'<!-- implementation-obligation:canonical-transition -->',
|
|
2503
|
+
'<!-- implementation-obligation:canonical-transition -->\n<!-- implementation-obligation:canonical-transition -->',
|
|
2504
|
+
),
|
|
2505
|
+
);
|
|
2506
|
+
},
|
|
2507
|
+
criticalPolicy,
|
|
2508
|
+
/duplicate implementation obligation marker canonical-transition/,
|
|
2509
|
+
],
|
|
2510
|
+
[
|
|
2511
|
+
'duplicate-execution-closure',
|
|
2512
|
+
(_state, specDir) => {
|
|
2513
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2514
|
+
write(designFile, `${fs.readFileSync(designFile, 'utf8')}\n## Execution Closure\n\nNo duplicate allowed.\n`);
|
|
2515
|
+
},
|
|
2516
|
+
criticalPolicy,
|
|
2517
|
+
/implementation obligation markers require exactly one Execution Closure section/,
|
|
2518
|
+
],
|
|
2519
|
+
[
|
|
2520
|
+
'closure-json-drift',
|
|
2521
|
+
(_state, specDir) => {
|
|
2522
|
+
const ownerFile = path.join(specDir, ownerPath);
|
|
2523
|
+
write(
|
|
2524
|
+
ownerFile,
|
|
2525
|
+
fs.readFileSync(ownerFile, 'utf8').replace(
|
|
2526
|
+
'Persisted transition state and allowed values',
|
|
2527
|
+
'A divergent local transition description',
|
|
2528
|
+
),
|
|
2529
|
+
);
|
|
2530
|
+
},
|
|
2531
|
+
criticalPolicy,
|
|
2532
|
+
/task-R1-01-one\.md: Execution Closure for canonical-transition must mirror spec\.json canonical_state/,
|
|
2533
|
+
],
|
|
2534
|
+
[
|
|
2535
|
+
'closure-requirements-drift',
|
|
2536
|
+
(_state, specDir) => {
|
|
2537
|
+
const designFile = path.join(specDir, 'design.md');
|
|
2538
|
+
write(
|
|
2539
|
+
designFile,
|
|
2540
|
+
fs.readFileSync(designFile, 'utf8').replace('- Requirements: R1.1', '- Requirements: R9.1'),
|
|
2541
|
+
);
|
|
2542
|
+
},
|
|
2543
|
+
criticalPolicy,
|
|
2544
|
+
/design\.md: Execution Closure for canonical-transition must mirror spec\.json requirements/,
|
|
2545
|
+
],
|
|
2546
|
+
[
|
|
2547
|
+
'duplicate-obligation-id',
|
|
2548
|
+
(state) => {
|
|
2549
|
+
state.implementation_obligations.push({ ...state.implementation_obligations[0] });
|
|
2550
|
+
},
|
|
2551
|
+
criticalPolicy,
|
|
2552
|
+
/duplicate obligation id canonical-transition/,
|
|
2553
|
+
],
|
|
2554
|
+
[
|
|
2555
|
+
'whitespace-obligation-id',
|
|
2556
|
+
(state) => { state.implementation_obligations[0].id = ' canonical-transition '; },
|
|
2557
|
+
criticalPolicy,
|
|
2558
|
+
/must be lowercase without surrounding whitespace/,
|
|
2559
|
+
],
|
|
2560
|
+
];
|
|
2561
|
+
for (const [name, mutate, policy, expected] of cases) {
|
|
2562
|
+
const specDir = makeFixture(name, mutate, policy);
|
|
2563
|
+
const result = run(VALIDATOR, [specDir], ROOT);
|
|
2564
|
+
assert.notEqual(result.status, 0, `${name} unexpectedly passed`);
|
|
2565
|
+
assert.match(output(result), expected);
|
|
2566
|
+
}
|
|
2567
|
+
} finally {
|
|
2568
|
+
fs.rmSync(root, { recursive: true, force: true });
|
|
2569
|
+
}
|
|
2570
|
+
});
|
|
2571
|
+
|
|
221
2572
|
test('grounding rejects empty sections, unsafe paths, unsupported actions, and zero-match globs', () => {
|
|
222
2573
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-grounding-'));
|
|
223
2574
|
try {
|
|
@@ -253,7 +2604,10 @@ test('validator enforces Read before Modify for existing implementation', () =>
|
|
|
253
2604
|
// Existing file Modify without preceding Read must fail
|
|
254
2605
|
const missingRead = createSpec(root, {
|
|
255
2606
|
name: 'missing-read',
|
|
256
|
-
task: task({
|
|
2607
|
+
task: task({
|
|
2608
|
+
related: '`src/existing.js` | Modify',
|
|
2609
|
+
relatedRows: '| `src/existing.js` | Modify | Update existing implementation |',
|
|
2610
|
+
}),
|
|
257
2611
|
});
|
|
258
2612
|
const missingResult = run(VALIDATOR, [missingRead], ROOT);
|
|
259
2613
|
assert.notEqual(missingResult.status, 0);
|
|
@@ -262,7 +2616,7 @@ test('validator enforces Read before Modify for existing implementation', () =>
|
|
|
262
2616
|
// Same path with Read in same task earlier row should pass
|
|
263
2617
|
const withReadSameTask = createSpec(root, {
|
|
264
2618
|
name: 'with-read-same',
|
|
265
|
-
task:
|
|
2619
|
+
task: task({ related: '`src/existing.js` | Modify' }),
|
|
266
2620
|
});
|
|
267
2621
|
const sameResult = run(VALIDATOR, [withReadSameTask], ROOT);
|
|
268
2622
|
assert.equal(sameResult.status, 0, output(sameResult));
|
|
@@ -274,8 +2628,13 @@ test('validator enforces Read before Modify for existing implementation', () =>
|
|
|
274
2628
|
depState.task_files.push('tasks/task-R1-02-two.md');
|
|
275
2629
|
depState.task_registry['tasks/task-R1-02-two.md'] = { id: 'R1-02', title: 'Two', status: 'pending', dependencies: [], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
276
2630
|
// Task 1 has Read, Task 2 modifies without depending
|
|
277
|
-
write(path.join(depRead, 'tasks/task-R1-01-one.md'),
|
|
278
|
-
write(path.join(depRead, 'tasks/task-R1-02-two.md'), task({
|
|
2631
|
+
write(path.join(depRead, 'tasks/task-R1-01-one.md'), task({ related: '`src/existing.js` | Read' }));
|
|
2632
|
+
write(path.join(depRead, 'tasks/task-R1-02-two.md'), task({
|
|
2633
|
+
id: 'R1-02',
|
|
2634
|
+
related: '`src/existing.js` | Modify',
|
|
2635
|
+
relatedRows: '| `src/existing.js` | Modify | Update existing implementation |',
|
|
2636
|
+
dependencies: 'tasks/task-R1-01-one.md',
|
|
2637
|
+
}));
|
|
279
2638
|
write(depStatePath, JSON.stringify(depState, null, 2));
|
|
280
2639
|
const depMissingResult = run(VALIDATOR, [depRead], ROOT);
|
|
281
2640
|
assert.notEqual(depMissingResult.status, 0);
|
|
@@ -283,6 +2642,7 @@ test('validator enforces Read before Modify for existing implementation', () =>
|
|
|
283
2642
|
|
|
284
2643
|
// If Modify task depends on Read task, should pass
|
|
285
2644
|
depState.task_registry['tasks/task-R1-02-two.md'].dependencies = ['tasks/task-R1-01-one.md'];
|
|
2645
|
+
depState.coordination.task_triggers = ['real_dependency', 'separate_proof'];
|
|
286
2646
|
write(depStatePath, JSON.stringify(depState, null, 2));
|
|
287
2647
|
const depPass = run(VALIDATOR, [depRead], ROOT);
|
|
288
2648
|
assert.equal(depPass.status, 0, output(depPass));
|
|
@@ -313,17 +2673,22 @@ test('validator rejects reachability without concrete anchor and accepts with pa
|
|
|
313
2673
|
}
|
|
314
2674
|
});
|
|
315
2675
|
|
|
316
|
-
test('validator
|
|
2676
|
+
test('validator keeps contract blocks opt-in regardless of task count', () => {
|
|
317
2677
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'cafekit-validator-contract-closed-'));
|
|
318
2678
|
try {
|
|
319
2679
|
const specDir = createSpec(root, { name: 'five-no-contract' });
|
|
320
2680
|
const statePath = path.join(specDir, 'spec.json');
|
|
321
2681
|
const state = JSON.parse(fs.readFileSync(statePath, 'utf8'));
|
|
2682
|
+
state.design_context = { validation_recommended: true };
|
|
322
2683
|
for (let i = 2; i <= 5; i += 1) {
|
|
323
2684
|
const taskFile = `tasks/task-R1-0${i}-extra.md`;
|
|
324
2685
|
state.task_files.push(taskFile);
|
|
325
2686
|
state.task_registry[taskFile] = { id: `R1-0${i}`, title: `Extra ${i}`, status: 'pending', dependencies: [], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
326
|
-
write(path.join(specDir, taskFile), task({
|
|
2687
|
+
write(path.join(specDir, taskFile), task({
|
|
2688
|
+
id: `R1-0${i}`,
|
|
2689
|
+
related: '`src/extra.js` | Modify',
|
|
2690
|
+
mapping: '1.1',
|
|
2691
|
+
}));
|
|
327
2692
|
}
|
|
328
2693
|
// Need to ensure each extra task has Read before Modify to avoid lifecycle error
|
|
329
2694
|
for (let i = 2; i <= 5; i += 1) {
|
|
@@ -337,8 +2702,7 @@ test('validator fails closed when 5+ tasks have no contract blocks', () => {
|
|
|
337
2702
|
write(path.join(specDir, 'tasks/task-R1-01-one.md'), first);
|
|
338
2703
|
write(statePath, JSON.stringify(state, null, 2));
|
|
339
2704
|
const result = run(VALIDATOR, [specDir], ROOT);
|
|
340
|
-
assert.
|
|
341
|
-
assert.match(output(result), /5\+ task spec requires contract blocks/);
|
|
2705
|
+
assert.equal(result.status, 0, output(result));
|
|
342
2706
|
} finally {
|
|
343
2707
|
fs.rmSync(root, { recursive: true, force: true });
|
|
344
2708
|
}
|
|
@@ -380,7 +2744,10 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
380
2744
|
// Same-task: Create after Read must fail (dead-code regression)
|
|
381
2745
|
const createAfterRead = createSpec(root, {
|
|
382
2746
|
name: 'create-after-read',
|
|
383
|
-
task:
|
|
2747
|
+
task: task({
|
|
2748
|
+
related: '`src/new.js` | Read',
|
|
2749
|
+
relatedRows: '| `src/new.js` | Read | Read before create |\n| `src/new.js` | Create | Create after read |',
|
|
2750
|
+
}),
|
|
384
2751
|
});
|
|
385
2752
|
const carResult = run(VALIDATOR, [createAfterRead], ROOT);
|
|
386
2753
|
assert.notEqual(carResult.status, 0);
|
|
@@ -389,7 +2756,10 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
389
2756
|
// Same-task: Create after Modify must fail
|
|
390
2757
|
const createAfterModify = createSpec(root, {
|
|
391
2758
|
name: 'create-after-modify',
|
|
392
|
-
task:
|
|
2759
|
+
task: task({
|
|
2760
|
+
related: '`src/new.js` | Modify',
|
|
2761
|
+
relatedRows: '| `src/new.js` | Modify | Modify before create |\n| `src/new.js` | Create | Create after modify |',
|
|
2762
|
+
}),
|
|
393
2763
|
});
|
|
394
2764
|
const camResult = run(VALIDATOR, [createAfterModify], ROOT);
|
|
395
2765
|
assert.notEqual(camResult.status, 0);
|
|
@@ -398,7 +2768,10 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
398
2768
|
// Same-task: Create before Read/Modify must pass; repeated Reads must not be rejected
|
|
399
2769
|
const createBefore = createSpec(root, {
|
|
400
2770
|
name: 'create-before',
|
|
401
|
-
task:
|
|
2771
|
+
task: task({
|
|
2772
|
+
related: '`src/new.js` | Create',
|
|
2773
|
+
relatedRows: '| `src/new.js` | Create | Create first |\n| `src/new.js` | Read | Read after create |\n| `src/new.js` | Modify | Modify after create |\n| `src/new.js` | Read | Second read |',
|
|
2774
|
+
}),
|
|
402
2775
|
});
|
|
403
2776
|
const cbResult = run(VALIDATOR, [createBefore], ROOT);
|
|
404
2777
|
assert.equal(cbResult.status, 0, output(cbResult));
|
|
@@ -406,7 +2779,10 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
406
2779
|
// Valid same-task repeated reads without create must pass (not rejected unnecessarily)
|
|
407
2780
|
const repeatedReads = createSpec(root, {
|
|
408
2781
|
name: 'repeated-reads',
|
|
409
|
-
task:
|
|
2782
|
+
task: task({
|
|
2783
|
+
related: '`src/existing.js` | Read',
|
|
2784
|
+
relatedRows: '| `src/existing.js` | Read | First read |\n| `src/existing.js` | Read | Second read |\n| `src/existing.js` | Modify | Modify after reads |',
|
|
2785
|
+
}),
|
|
410
2786
|
});
|
|
411
2787
|
const rrResult = run(VALIDATOR, [repeatedReads], ROOT);
|
|
412
2788
|
assert.equal(rrResult.status, 0, output(rrResult));
|
|
@@ -416,10 +2792,13 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
416
2792
|
const crossReadStatePath = path.join(crossRead, 'spec.json');
|
|
417
2793
|
const crossReadState = JSON.parse(fs.readFileSync(crossReadStatePath, 'utf8'));
|
|
418
2794
|
// Task1 creates src/gen.js
|
|
419
|
-
write(path.join(crossRead, 'tasks/task-R1-01-one.md'),
|
|
2795
|
+
write(path.join(crossRead, 'tasks/task-R1-01-one.md'), task({ related: '`src/gen.js` | Create' }));
|
|
420
2796
|
crossReadState.task_files.push('tasks/task-R1-02-two.md');
|
|
421
2797
|
crossReadState.task_registry['tasks/task-R1-02-two.md'] = { id: 'R1-02', title: 'Two', status: 'pending', dependencies: [], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
422
|
-
write(path.join(crossRead, 'tasks/task-R1-02-two.md'),
|
|
2798
|
+
write(path.join(crossRead, 'tasks/task-R1-02-two.md'), task({
|
|
2799
|
+
id: 'R1-02',
|
|
2800
|
+
related: '`src/gen.js` | Read',
|
|
2801
|
+
}));
|
|
423
2802
|
write(crossReadStatePath, JSON.stringify(crossReadState, null, 2));
|
|
424
2803
|
const crossReadFail = run(VALIDATOR, [crossRead], ROOT);
|
|
425
2804
|
assert.notEqual(crossReadFail.status, 0);
|
|
@@ -427,6 +2806,13 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
427
2806
|
|
|
428
2807
|
// Cross-task Read with dependency must pass
|
|
429
2808
|
crossReadState.task_registry['tasks/task-R1-02-two.md'].dependencies = ['tasks/task-R1-01-one.md'];
|
|
2809
|
+
crossReadState.coordination.task_triggers = ['real_dependency', 'separate_proof'];
|
|
2810
|
+
const crossReadTaskPath = path.join(crossRead, 'tasks/task-R1-02-two.md');
|
|
2811
|
+
write(crossReadTaskPath, task({
|
|
2812
|
+
id: 'R1-02',
|
|
2813
|
+
related: '`src/gen.js` | Read',
|
|
2814
|
+
dependencies: 'tasks/task-R1-01-one.md',
|
|
2815
|
+
}));
|
|
430
2816
|
write(crossReadStatePath, JSON.stringify(crossReadState, null, 2));
|
|
431
2817
|
const crossReadPass = run(VALIDATOR, [crossRead], ROOT);
|
|
432
2818
|
assert.equal(crossReadPass.status, 0, output(crossReadPass));
|
|
@@ -435,15 +2821,27 @@ test('validator enforces Create before Read/Modify/Delete within same task and c
|
|
|
435
2821
|
const crossMod = createSpec(root, { name: 'cross-mod-base' });
|
|
436
2822
|
const crossModStatePath = path.join(crossMod, 'spec.json');
|
|
437
2823
|
const crossModState = JSON.parse(fs.readFileSync(crossModStatePath, 'utf8'));
|
|
438
|
-
write(path.join(crossMod, 'tasks/task-R1-01-one.md'),
|
|
2824
|
+
write(path.join(crossMod, 'tasks/task-R1-01-one.md'), task({ related: '`src/gen2.js` | Create' }));
|
|
439
2825
|
crossModState.task_files.push('tasks/task-R1-02-two.md');
|
|
440
2826
|
crossModState.task_registry['tasks/task-R1-02-two.md'] = { id: 'R1-02', title: 'Two', status: 'pending', dependencies: [], blocker: null, started_at: null, completed_at: null, last_updated_at: null };
|
|
441
|
-
write(path.join(crossMod, 'tasks/task-R1-02-two.md'),
|
|
2827
|
+
write(path.join(crossMod, 'tasks/task-R1-02-two.md'), task({
|
|
2828
|
+
id: 'R1-02',
|
|
2829
|
+
related: '`src/gen2.js` | Modify',
|
|
2830
|
+
relatedRows: '| `src/gen2.js` | Modify | Modify without dependency |',
|
|
2831
|
+
}));
|
|
442
2832
|
write(crossModStatePath, JSON.stringify(crossModState, null, 2));
|
|
443
2833
|
const crossModFail = run(VALIDATOR, [crossMod], ROOT);
|
|
444
2834
|
assert.notEqual(crossModFail.status, 0);
|
|
445
2835
|
assert.match(output(crossModFail), /modify of src\/gen2\.js must depend on creator/);
|
|
446
2836
|
crossModState.task_registry['tasks/task-R1-02-two.md'].dependencies = ['tasks/task-R1-01-one.md'];
|
|
2837
|
+
crossModState.coordination.task_triggers = ['real_dependency', 'separate_proof'];
|
|
2838
|
+
const crossModTaskPath = path.join(crossMod, 'tasks/task-R1-02-two.md');
|
|
2839
|
+
write(crossModTaskPath, task({
|
|
2840
|
+
id: 'R1-02',
|
|
2841
|
+
related: '`src/gen2.js` | Modify',
|
|
2842
|
+
relatedRows: '| `src/gen2.js` | Modify | Modify generated implementation |',
|
|
2843
|
+
dependencies: 'tasks/task-R1-01-one.md',
|
|
2844
|
+
}));
|
|
447
2845
|
write(crossModStatePath, JSON.stringify(crossModState, null, 2));
|
|
448
2846
|
const crossModPass = run(VALIDATOR, [crossMod], ROOT);
|
|
449
2847
|
assert.equal(crossModPass.status, 0, output(crossModPass));
|