@popoverai/dotrequirements 0.26.0 → 0.26.2
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/dist/codebase-to-spec/area-name.d.ts +13 -0
- package/dist/codebase-to-spec/area-name.js +18 -0
- package/dist/codebase-to-spec/cache.d.ts +31 -0
- package/dist/codebase-to-spec/cache.js +29 -1
- package/dist/codebase-to-spec/compose.d.ts +7 -4
- package/dist/codebase-to-spec/compose.js +10 -21
- package/dist/codebase-to-spec/dispatch.js +1 -1
- package/dist/codebase-to-spec/present.js +11 -7
- package/dist/codebase-to-spec/prompts/planner-initial.d.ts +3 -2
- package/dist/codebase-to-spec/prompts/planner-initial.js +3 -2
- package/dist/codebase-to-spec/renumber.d.ts +52 -0
- package/dist/codebase-to-spec/renumber.js +105 -0
- package/dist/codebase-to-spec/schemas.d.ts +35 -240
- package/dist/codebase-to-spec/schemas.js +5 -173
- package/dist/codebase-to-spec/validate.js +3 -2
- package/dist/commands/acceptance-test.js +4 -2
- package/dist/commands/ai-setup.js +97 -59
- package/dist/commands/codebase-to-spec/dispatch-editor.js +1 -1
- package/dist/commands/codebase-to-spec/dispatch-spec.js +1 -1
- package/dist/commands/codebase-to-spec/index.js +7 -103
- package/dist/commands/codebase-to-spec/pack.js +8 -1
- package/dist/commands/codebase-to-spec/present-orchestrator.d.ts +3 -1
- package/dist/commands/codebase-to-spec/present-orchestrator.js +11 -4
- package/dist/commands/get.js +6 -2
- package/dist/commands/init.js +7 -5
- package/dist/commands/link-resolution.d.ts +3 -1
- package/dist/commands/link-resolution.js +4 -2
- package/dist/commands/pull.js +36 -3
- package/dist/commands/push.js +54 -16
- package/dist/commands/report.js +18 -3
- package/dist/commands/review-test.js +16 -8
- package/dist/commands/tests-for.js +13 -13
- package/dist/commands/validate.js +14 -14
- package/dist/harness/cache.d.ts +19 -3
- package/dist/harness/cache.js +38 -12
- package/dist/harness/finalize.js +33 -1
- package/dist/harness/index.js +16 -9
- package/dist/harness/requirementsLoader.js +12 -0
- package/dist/harness/tracking.d.ts +17 -2
- package/dist/harness/tracking.js +83 -9
- package/dist/mcp/handlers/authoring.js +13 -4
- package/dist/mcp/handlers/get.js +7 -3
- package/dist/mcp/handlers/push.js +59 -13
- package/dist/mcp/handlers/review.d.ts +1 -0
- package/dist/mcp/handlers/review.js +58 -15
- package/dist/mcp/handlers/test-mapping.js +47 -12
- package/dist/mcp/handlers/types.d.ts +14 -0
- package/dist/mcp/handlers/types.js +27 -0
- package/dist/mcp/index.js +4 -0
- package/dist/push/core.d.ts +50 -0
- package/dist/push/core.js +149 -11
- package/dist/push/index.d.ts +1 -1
- package/dist/push/index.js +1 -1
- package/dist/requirements/cloud-coverage.d.ts +12 -2
- package/dist/requirements/cloud-coverage.js +30 -3
- package/dist/requirements/grep.d.ts +7 -2
- package/dist/requirements/grep.js +75 -47
- package/dist/schema/builder.d.ts +1 -1
- package/dist/schema/builder.js +13 -0
- package/dist/schema/conversions.d.ts +7 -2
- package/dist/schema/conversions.js +13 -4
- package/dist/schema/parser-core.d.ts +28 -0
- package/dist/schema/parser-core.js +80 -9
- package/dist/schema/parser.d.ts +8 -26
- package/dist/schema/parser.js +23 -251
- package/dist/schema/resolver.js +18 -8
- package/dist/templates/skills/codebase-to-spec/SKILL.md +10 -4
- package/dist/utils/env.js +17 -1
- package/dist/utils/oauth-flow.js +8 -0
- package/dist/utils/project-settings.d.ts +4 -0
- package/dist/utils/project-settings.js +14 -1
- package/package.json +1 -1
- package/dist/codebase-to-spec/edit-loop.d.ts +0 -54
- package/dist/codebase-to-spec/edit-loop.js +0 -195
- package/dist/codebase-to-spec/editor.d.ts +0 -54
- package/dist/codebase-to-spec/editor.js +0 -74
- package/dist/codebase-to-spec/fan-out.d.ts +0 -63
- package/dist/codebase-to-spec/fan-out.js +0 -215
- package/dist/codebase-to-spec/outline-review-loop.d.ts +0 -51
- package/dist/codebase-to-spec/outline-review-loop.js +0 -187
- package/dist/codebase-to-spec/planner.d.ts +0 -41
- package/dist/codebase-to-spec/planner.js +0 -76
- package/dist/codebase-to-spec/prompts/outline-reviewer.d.ts +0 -12
- package/dist/codebase-to-spec/prompts/outline-reviewer.js +0 -89
- package/dist/codebase-to-spec/slice.d.ts +0 -49
- package/dist/codebase-to-spec/slice.js +0 -111
- package/dist/codebase-to-spec/specifier.d.ts +0 -60
- package/dist/codebase-to-spec/specifier.js +0 -85
- package/dist/codebase-to-spec/summary.d.ts +0 -51
- package/dist/codebase-to-spec/summary.js +0 -183
- package/dist/commands/codebase-to-spec/compose.d.ts +0 -14
- package/dist/commands/codebase-to-spec/compose.js +0 -57
- package/dist/commands/codebase-to-spec/edit-loop.d.ts +0 -16
- package/dist/commands/codebase-to-spec/edit-loop.js +0 -83
- package/dist/commands/codebase-to-spec/fan-out.d.ts +0 -19
- package/dist/commands/codebase-to-spec/fan-out.js +0 -77
- package/dist/commands/codebase-to-spec/plan-loop.d.ts +0 -26
- package/dist/commands/codebase-to-spec/plan-loop.js +0 -105
- package/dist/commands/codebase-to-spec/present.d.ts +0 -26
- package/dist/commands/codebase-to-spec/present.js +0 -97
- package/dist/commands/codebase-to-spec/run.d.ts +0 -20
- package/dist/commands/codebase-to-spec/run.js +0 -86
- package/dist/commands/codebase-to-spec/specify-area.d.ts +0 -18
- package/dist/commands/codebase-to-spec/specify-area.js +0 -82
|
@@ -19,7 +19,7 @@ Do **not** use for new-feature design (use the `dotreq-requirements` skill), bug
|
|
|
19
19
|
|
|
20
20
|
This skill runs its pipeline as a dynamic Workflow. If the `Workflow` tool is not available in this environment, **stop** and tell the user:
|
|
21
21
|
|
|
22
|
-
> "codebase-to-spec runs as a dynamic workflow, which this Claude Code version/configuration doesn't support.
|
|
22
|
+
> "codebase-to-spec runs as a dynamic workflow, which this Claude Code version/configuration doesn't support. Update to a build with dynamic-workflow support to run it."
|
|
23
23
|
|
|
24
24
|
Do not attempt a non-workflow fallback.
|
|
25
25
|
|
|
@@ -33,9 +33,15 @@ If the user gave a scope (a path), use it. Otherwise ask one concise question:
|
|
|
33
33
|
|
|
34
34
|
For large codebases, encourage scoping to a single area — the pack has a context budget.
|
|
35
35
|
|
|
36
|
+
**Isolation check.** Before packing, check whether the target project already has a dotrequirements spec (a `.requirements/` directory). If it does, the user is likely re-running CTS on an already-specced codebase — often a dogfood or a quality eval — and the present stage would otherwise write over that spec. Offer an isolated run:
|
|
37
|
+
|
|
38
|
+
> "This project already has a `.requirements/` spec. Want me to isolate this run — ignore the existing spec as input, and write my output to `.requirements/cts/` so I never touch what's there? Or proceed normally and treat the existing spec as part of the codebase?"
|
|
39
|
+
|
|
40
|
+
On accept, run pack with `--ignore-requirements` in step 2; the decision is recorded with the run, so present honors it automatically — you don't pass it again. On decline, or when there's no `.requirements/`, pack normally. (CTSO-CONV-9.)
|
|
41
|
+
|
|
36
42
|
### 2. Pack (deterministic)
|
|
37
43
|
|
|
38
|
-
Run `{{DOTREQ_CLI}} cts pack --scope <PATH>` via Bash. Deterministic, no LLM. It emits `[CTS] pack/done` then `pack/budget-ok`, or exits **10 (BudgetExceeded)** if the compressed pack is too large. On BudgetExceeded, surface the limit and offer a narrower scope (back to step 1).
|
|
44
|
+
Run `{{DOTREQ_CLI}} cts pack --scope <PATH>` via Bash — add `--ignore-requirements` if the user accepted isolation in step 1. Deterministic, no LLM. It emits `[CTS] pack/done` then `pack/budget-ok`, or exits **10 (BudgetExceeded)** if the compressed pack is too large. On BudgetExceeded, surface the limit and offer a narrower scope (back to step 1).
|
|
39
45
|
|
|
40
46
|
**Stale install gate.** If pack also emits a `pack/stale-version` line, this install of codebase-to-spec is out of date. Before launching the workflow, tell the user (naming both versions from the line) and offer to refresh:
|
|
41
47
|
|
|
@@ -66,7 +72,7 @@ The workflow returns one of:
|
|
|
66
72
|
### 5. Present (deterministic)
|
|
67
73
|
|
|
68
74
|
On `status: "done"`, the workflow has already composed the spec and run the cross-area pass. Write the final file(s):
|
|
69
|
-
- `{{DOTREQ_CLI}} cts present-orchestrator` — writes the composed spec to file(s) under `.requirements/`.
|
|
75
|
+
- `{{DOTREQ_CLI}} cts present-orchestrator` — writes the composed spec to file(s) under `.requirements/`. For an isolated run (step 1), it writes under `.requirements/cts/` instead — it reads the decision recorded at pack time, so you pass no flag here.
|
|
70
76
|
|
|
71
77
|
`present-orchestrator` errors on existing-file conflicts unless given `--overwrite` or `--skip-existing`. When a conflict is reported, ask the user which they want, then re-run with that flag.
|
|
72
78
|
|
|
@@ -104,4 +110,4 @@ If the user declines the team-review offer, note that it stands for later and do
|
|
|
104
110
|
|
|
105
111
|
## Host portability
|
|
106
112
|
|
|
107
|
-
This skill requires Claude Code with dynamic-workflow support (it launches the `specify-codebase` workflow).
|
|
113
|
+
This skill requires Claude Code with dynamic-workflow support (it launches the `specify-codebase` workflow). There is no non-workflow fallback; on hosts without dynamic-workflow support, codebase-to-spec is unavailable.
|
package/dist/utils/env.js
CHANGED
|
@@ -23,7 +23,7 @@ export function loadEnvFile(cwd = process.cwd(), force = false) {
|
|
|
23
23
|
const match = line.match(/^([^=]+)=(.*)$/);
|
|
24
24
|
if (match) {
|
|
25
25
|
const key = match[1].trim();
|
|
26
|
-
const value = match[2].trim();
|
|
26
|
+
const value = stripSurroundingQuotes(match[2].trim());
|
|
27
27
|
// Set if force=true, or if not already in process.env
|
|
28
28
|
if (force || !process.env[key]) {
|
|
29
29
|
process.env[key] = value;
|
|
@@ -36,4 +36,20 @@ export function loadEnvFile(cwd = process.cwd(), force = false) {
|
|
|
36
36
|
// The commands will error appropriately if env vars are missing
|
|
37
37
|
}
|
|
38
38
|
}
|
|
39
|
+
/**
|
|
40
|
+
* Strip a single matching pair of surrounding single or double quotes from a
|
|
41
|
+
* value, matching dotenv behavior. Credentials written in standard
|
|
42
|
+
* dotenv/Next.js style (KEY="value") should not carry the quote characters
|
|
43
|
+
* into process.env. Mismatched or interior quotes are left untouched.
|
|
44
|
+
*/
|
|
45
|
+
function stripSurroundingQuotes(value) {
|
|
46
|
+
if (value.length >= 2) {
|
|
47
|
+
const first = value[0];
|
|
48
|
+
const last = value[value.length - 1];
|
|
49
|
+
if ((first === '"' || first === "'") && first === last) {
|
|
50
|
+
return value.slice(1, -1);
|
|
51
|
+
}
|
|
52
|
+
}
|
|
53
|
+
return value;
|
|
54
|
+
}
|
|
39
55
|
//# sourceMappingURL=env.js.map
|
package/dist/utils/oauth-flow.js
CHANGED
|
@@ -32,6 +32,14 @@ export async function executeOAuthFlow() {
|
|
|
32
32
|
port: 3010,
|
|
33
33
|
timeoutMs: 120000,
|
|
34
34
|
});
|
|
35
|
+
// Attach a no-op rejection handler immediately. startCallbackServer can
|
|
36
|
+
// reject synchronously-adjacent to creation (e.g. listen EADDRINUSE when the
|
|
37
|
+
// callback port is busy), but the real handler isn't awaited until after the
|
|
38
|
+
// browser launch below. Without this, an early rejection has no handler and
|
|
39
|
+
// Node escalates it to a fatal unhandledRejection, bypassing wrapCommand's
|
|
40
|
+
// clean CLI error formatting. The awaited `callbackPromise` below still sees
|
|
41
|
+
// the rejection and surfaces it as a normal thrown error.
|
|
42
|
+
callbackPromise.catch(() => { });
|
|
35
43
|
// Build authorization URL
|
|
36
44
|
const authUrl = buildAuthorizationUrl({
|
|
37
45
|
clientId: WORKOS_CLIENT_ID,
|
|
@@ -54,6 +54,10 @@ export declare function readProjectSettings(projectRoot: string): ProjectSetting
|
|
|
54
54
|
/**
|
|
55
55
|
* Write project settings to .requirements/project-settings.json
|
|
56
56
|
* Creates .requirements/ directory if it doesn't exist
|
|
57
|
+
*
|
|
58
|
+
* SETTINGS-2.3/2.4: fields being written replace the stored ones, but any
|
|
59
|
+
* other existing settings (defaultURL, browserTest, ...) survive — re-linking
|
|
60
|
+
* with fresh credentials must not erase the rest of the file.
|
|
57
61
|
*/
|
|
58
62
|
export declare function writeProjectSettings(projectRoot: string, settings: ProjectSettings): void;
|
|
59
63
|
/**
|
|
@@ -106,6 +106,10 @@ export function readProjectSettings(projectRoot) {
|
|
|
106
106
|
/**
|
|
107
107
|
* Write project settings to .requirements/project-settings.json
|
|
108
108
|
* Creates .requirements/ directory if it doesn't exist
|
|
109
|
+
*
|
|
110
|
+
* SETTINGS-2.3/2.4: fields being written replace the stored ones, but any
|
|
111
|
+
* other existing settings (defaultURL, browserTest, ...) survive — re-linking
|
|
112
|
+
* with fresh credentials must not erase the rest of the file.
|
|
109
113
|
*/
|
|
110
114
|
export function writeProjectSettings(projectRoot, settings) {
|
|
111
115
|
const requirementsDir = path.join(projectRoot, REQUIREMENTS_DIR);
|
|
@@ -114,7 +118,16 @@ export function writeProjectSettings(projectRoot, settings) {
|
|
|
114
118
|
if (!fs.existsSync(requirementsDir)) {
|
|
115
119
|
fs.mkdirSync(requirementsDir, { recursive: true });
|
|
116
120
|
}
|
|
117
|
-
|
|
121
|
+
let existing;
|
|
122
|
+
try {
|
|
123
|
+
existing = readProjectSettings(projectRoot);
|
|
124
|
+
}
|
|
125
|
+
catch {
|
|
126
|
+
// Unreadable or invalid existing file: nothing preservable, write fresh
|
|
127
|
+
existing = undefined;
|
|
128
|
+
}
|
|
129
|
+
const merged = existing ? { ...existing, ...settings } : settings;
|
|
130
|
+
const content = `${JSON.stringify(merged, null, 2)}\n`;
|
|
118
131
|
fs.writeFileSync(settingsPath, content, "utf-8");
|
|
119
132
|
}
|
|
120
133
|
/**
|
package/package.json
CHANGED
|
@@ -1,54 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Spec review + editor loop.
|
|
3
|
-
*
|
|
4
|
-
* Same structure as the outline review loop, scaled up for document-level
|
|
5
|
-
* revision: stateful spec reviewer + stateless editor. The reviewer
|
|
6
|
-
* persists across turns so it can compare prior spec versions to the latest.
|
|
7
|
-
*
|
|
8
|
-
* Requirements covered:
|
|
9
|
-
* - CTS-EDIT-1: stateful spec reviewer
|
|
10
|
-
* - CTS-EDIT-2: approved → proceed
|
|
11
|
-
* - CTS-EDIT-3: approved-with-revisions → one mechanical edit pass
|
|
12
|
-
* - CTS-EDIT-4: requires-another-review → revise & resubmit
|
|
13
|
-
* - CTS-EDIT-5: editor operates on cohesive document
|
|
14
|
-
* - CTS-EDIT-6: max-turns + convergence nudge
|
|
15
|
-
*/
|
|
16
|
-
import { type ClaudeRunner } from "./claude.js";
|
|
17
|
-
import type { ProgressEmitter } from "./progress.js";
|
|
18
|
-
import { type SpecReview } from "./schemas.js";
|
|
19
|
-
export declare const DEFAULT_EDIT_LOOP_MAX_TURNS = 6;
|
|
20
|
-
export interface EditLoopOptions {
|
|
21
|
-
/** Path to the composed spec (input). */
|
|
22
|
-
composedSpecPath: string;
|
|
23
|
-
/**
|
|
24
|
-
* Resolver: given a turn number (1-indexed), return the path the editor's
|
|
25
|
-
* output for that turn should land at. Loop persists each version.
|
|
26
|
-
*/
|
|
27
|
-
specVersionFor: (turn: number) => string;
|
|
28
|
-
/** Path to the codebase pack. */
|
|
29
|
-
fullPackPath: string;
|
|
30
|
-
/** Directories for filesystem access. */
|
|
31
|
-
addDirs: string[];
|
|
32
|
-
maxTurns?: number;
|
|
33
|
-
runner?: ClaudeRunner;
|
|
34
|
-
model?: string;
|
|
35
|
-
progress?: ProgressEmitter;
|
|
36
|
-
/** Override session id (for tests / reproducibility). */
|
|
37
|
-
sessionId?: string;
|
|
38
|
-
}
|
|
39
|
-
export interface EditLoopResult {
|
|
40
|
-
/** Path to the final spec (post-edit if applicable). */
|
|
41
|
-
finalSpecPath: string;
|
|
42
|
-
/** Per-turn spec paths (turn 1 is the composed-spec copy, then each revision). */
|
|
43
|
-
specPathsByTurn: string[];
|
|
44
|
-
/** Per-turn reviewer results. */
|
|
45
|
-
reviewsByTurn: SpecReview[];
|
|
46
|
-
/** The final reviewer verdict. */
|
|
47
|
-
finalVerdict: SpecReview["verdict"];
|
|
48
|
-
/** Number of review turns used. */
|
|
49
|
-
turnsUsed: number;
|
|
50
|
-
/** True if the loop hit max-turns without converging. */
|
|
51
|
-
hitMaxTurns: boolean;
|
|
52
|
-
}
|
|
53
|
-
export declare function runEditLoop(options: EditLoopOptions): Promise<EditLoopResult>;
|
|
54
|
-
//# sourceMappingURL=edit-loop.d.ts.map
|
|
@@ -1,195 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Spec review + editor loop.
|
|
3
|
-
*
|
|
4
|
-
* Same structure as the outline review loop, scaled up for document-level
|
|
5
|
-
* revision: stateful spec reviewer + stateless editor. The reviewer
|
|
6
|
-
* persists across turns so it can compare prior spec versions to the latest.
|
|
7
|
-
*
|
|
8
|
-
* Requirements covered:
|
|
9
|
-
* - CTS-EDIT-1: stateful spec reviewer
|
|
10
|
-
* - CTS-EDIT-2: approved → proceed
|
|
11
|
-
* - CTS-EDIT-3: approved-with-revisions → one mechanical edit pass
|
|
12
|
-
* - CTS-EDIT-4: requires-another-review → revise & resubmit
|
|
13
|
-
* - CTS-EDIT-5: editor operates on cohesive document
|
|
14
|
-
* - CTS-EDIT-6: max-turns + convergence nudge
|
|
15
|
-
*/
|
|
16
|
-
import { randomUUID } from "node:crypto";
|
|
17
|
-
import { copyFileSync } from "node:fs";
|
|
18
|
-
import { runClaude } from "./claude.js";
|
|
19
|
-
import { runEditor } from "./editor.js";
|
|
20
|
-
import { SPEC_REVIEWER_PROMPT } from "./prompts/spec-reviewer.js";
|
|
21
|
-
import { parseSpecReview, SPEC_REVIEW_JSON_SCHEMA, } from "./schemas.js";
|
|
22
|
-
export const DEFAULT_EDIT_LOOP_MAX_TURNS = 6;
|
|
23
|
-
function reviewerUserMessageInitial(specPath, packPath) {
|
|
24
|
-
return [
|
|
25
|
-
`Initial spec review.`,
|
|
26
|
-
``,
|
|
27
|
-
`Composed spec: ${specPath}`,
|
|
28
|
-
`Codebase pack: ${packPath}`,
|
|
29
|
-
``,
|
|
30
|
-
`Read both via the Read tool, then produce your critique JSON.`,
|
|
31
|
-
].join("\n");
|
|
32
|
-
}
|
|
33
|
-
function reviewerUserMessageRevised(turn, specPath, packPath, nudge) {
|
|
34
|
-
const lines = [
|
|
35
|
-
`Review turn ${turn}. The editor has revised the spec based on your prior feedback.`,
|
|
36
|
-
``,
|
|
37
|
-
];
|
|
38
|
-
if (nudge) {
|
|
39
|
-
lines.push(nudge, ``);
|
|
40
|
-
}
|
|
41
|
-
lines.push(`Revised spec: ${specPath}`, `Codebase pack at ${packPath} is unchanged.`, ``, `Read the revised spec, compare to the prior version, and produce your critique JSON.`);
|
|
42
|
-
return lines.join("\n");
|
|
43
|
-
}
|
|
44
|
-
function convergenceNudge(turn, maxTurns) {
|
|
45
|
-
return `CONVERGENCE NUDGE: This is review turn ${turn} of ${maxTurns} maximum. Please prioritize convergence. Your job is still to only approve a genuinely-ready spec, but if it is close, lean toward \`approved-with-revisions\` over \`requires-another-review\`. Reserve \`requires-another-review\` for genuine issues that the editor has not addressed despite prior feedback.`;
|
|
46
|
-
}
|
|
47
|
-
async function callReviewer(args) {
|
|
48
|
-
const result = await args.runner({
|
|
49
|
-
systemPrompt: SPEC_REVIEWER_PROMPT,
|
|
50
|
-
userMessage: args.userMessage,
|
|
51
|
-
model: args.model,
|
|
52
|
-
tools: ["Read", "Grep", "Glob"],
|
|
53
|
-
addDirs: args.addDirs,
|
|
54
|
-
jsonSchema: SPEC_REVIEW_JSON_SCHEMA,
|
|
55
|
-
session: {
|
|
56
|
-
kind: args.isFirstTurn ? "fresh" : "resume",
|
|
57
|
-
sessionId: args.sessionId,
|
|
58
|
-
},
|
|
59
|
-
});
|
|
60
|
-
if (result.exitCode !== 0) {
|
|
61
|
-
throw new Error(`Spec reviewer failed (exit ${result.exitCode}): ${result.stderr || result.stdout}`);
|
|
62
|
-
}
|
|
63
|
-
// Retry-once on parse failure (CTS-EDIT-1 convention, same as outline loop).
|
|
64
|
-
try {
|
|
65
|
-
return parseSpecReview(result.stdout);
|
|
66
|
-
}
|
|
67
|
-
catch (err) {
|
|
68
|
-
const retry = await args.runner({
|
|
69
|
-
systemPrompt: SPEC_REVIEWER_PROMPT,
|
|
70
|
-
userMessage: `${args.userMessage}\n\n(Your prior output failed schema validation: ${err instanceof Error ? err.message : String(err)}. Please try again, emitting JSON only.)`,
|
|
71
|
-
model: args.model,
|
|
72
|
-
tools: ["Read", "Grep", "Glob"],
|
|
73
|
-
addDirs: args.addDirs,
|
|
74
|
-
jsonSchema: SPEC_REVIEW_JSON_SCHEMA,
|
|
75
|
-
session: { kind: "resume", sessionId: args.sessionId },
|
|
76
|
-
});
|
|
77
|
-
if (retry.exitCode !== 0) {
|
|
78
|
-
throw new Error(`Spec reviewer retry failed (exit ${retry.exitCode}): ${retry.stderr || retry.stdout}`);
|
|
79
|
-
}
|
|
80
|
-
return parseSpecReview(retry.stdout);
|
|
81
|
-
}
|
|
82
|
-
}
|
|
83
|
-
export async function runEditLoop(options) {
|
|
84
|
-
const { composedSpecPath, specVersionFor, fullPackPath, addDirs, maxTurns = DEFAULT_EDIT_LOOP_MAX_TURNS, runner = runClaude, model, progress, sessionId = randomUUID(), } = options;
|
|
85
|
-
// Turn 1: copy the composed spec to spec-v1 and review it.
|
|
86
|
-
const specV1 = specVersionFor(1);
|
|
87
|
-
copyFileSync(composedSpecPath, specV1);
|
|
88
|
-
progress?.emit({
|
|
89
|
-
stage: "spec-review",
|
|
90
|
-
step: "turn-1",
|
|
91
|
-
message: `Reviewing composed spec (${specV1})`,
|
|
92
|
-
});
|
|
93
|
-
let review = await callReviewer({
|
|
94
|
-
runner,
|
|
95
|
-
sessionId,
|
|
96
|
-
isFirstTurn: true,
|
|
97
|
-
userMessage: reviewerUserMessageInitial(specV1, fullPackPath),
|
|
98
|
-
addDirs,
|
|
99
|
-
model,
|
|
100
|
-
});
|
|
101
|
-
const specPathsByTurn = [specV1];
|
|
102
|
-
const reviewsByTurn = [review];
|
|
103
|
-
let currentSpecPath = specV1;
|
|
104
|
-
progress?.emit({
|
|
105
|
-
stage: "spec-review",
|
|
106
|
-
step: "turn-1",
|
|
107
|
-
message: `Verdict: ${review.verdict}`,
|
|
108
|
-
data: { verdict: review.verdict },
|
|
109
|
-
});
|
|
110
|
-
let turn = 1;
|
|
111
|
-
while (review.verdict === "requires-another-review" && turn < maxTurns) {
|
|
112
|
-
turn++;
|
|
113
|
-
const nextSpec = specVersionFor(turn);
|
|
114
|
-
progress?.emit({
|
|
115
|
-
stage: "spec-review",
|
|
116
|
-
step: `edit-turn-${turn}`,
|
|
117
|
-
message: `Editor revising spec (turn ${turn}/${maxTurns})`,
|
|
118
|
-
});
|
|
119
|
-
const editResult = await runEditor({
|
|
120
|
-
inputSpecPath: currentSpecPath,
|
|
121
|
-
outputSpecPath: nextSpec,
|
|
122
|
-
fullPackPath,
|
|
123
|
-
addDirs,
|
|
124
|
-
critique: review,
|
|
125
|
-
mode: "revise",
|
|
126
|
-
runner,
|
|
127
|
-
model,
|
|
128
|
-
});
|
|
129
|
-
if (!editResult.wroteSpec) {
|
|
130
|
-
throw new Error(`Editor produced no output spec at ${nextSpec} (exit ${editResult.exitCode}): ${editResult.stderr}`);
|
|
131
|
-
}
|
|
132
|
-
currentSpecPath = nextSpec;
|
|
133
|
-
specPathsByTurn.push(currentSpecPath);
|
|
134
|
-
const nudge = turn >= maxTurns - 1 && turn !== maxTurns
|
|
135
|
-
? convergenceNudge(turn, maxTurns)
|
|
136
|
-
: undefined;
|
|
137
|
-
progress?.emit({
|
|
138
|
-
stage: "spec-review",
|
|
139
|
-
step: `turn-${turn}`,
|
|
140
|
-
message: `Reviewing revised spec (turn ${turn}/${maxTurns})`,
|
|
141
|
-
data: { turn, maxTurns, nudge: !!nudge },
|
|
142
|
-
});
|
|
143
|
-
review = await callReviewer({
|
|
144
|
-
runner,
|
|
145
|
-
sessionId,
|
|
146
|
-
isFirstTurn: false,
|
|
147
|
-
userMessage: reviewerUserMessageRevised(turn, currentSpecPath, fullPackPath, nudge),
|
|
148
|
-
addDirs,
|
|
149
|
-
model,
|
|
150
|
-
});
|
|
151
|
-
reviewsByTurn.push(review);
|
|
152
|
-
progress?.emit({
|
|
153
|
-
stage: "spec-review",
|
|
154
|
-
step: `turn-${turn}`,
|
|
155
|
-
message: `Verdict: ${review.verdict}`,
|
|
156
|
-
data: { verdict: review.verdict },
|
|
157
|
-
});
|
|
158
|
-
}
|
|
159
|
-
const hitMaxTurns = review.verdict === "requires-another-review" && turn >= maxTurns;
|
|
160
|
-
// approved-with-revisions → apply mechanical revisions to a final version.
|
|
161
|
-
if (review.verdict === "approved-with-revisions" &&
|
|
162
|
-
review.revisions.length > 0) {
|
|
163
|
-
const finalTurn = turn + 1;
|
|
164
|
-
const finalPath = specVersionFor(finalTurn);
|
|
165
|
-
progress?.emit({
|
|
166
|
-
stage: "spec-review",
|
|
167
|
-
step: "apply-revisions",
|
|
168
|
-
message: `Applying ${review.revisions.length} mechanical revision(s)`,
|
|
169
|
-
});
|
|
170
|
-
const applyResult = await runEditor({
|
|
171
|
-
inputSpecPath: currentSpecPath,
|
|
172
|
-
outputSpecPath: finalPath,
|
|
173
|
-
fullPackPath,
|
|
174
|
-
addDirs,
|
|
175
|
-
critique: review,
|
|
176
|
-
mode: "apply",
|
|
177
|
-
runner,
|
|
178
|
-
model,
|
|
179
|
-
});
|
|
180
|
-
if (!applyResult.wroteSpec) {
|
|
181
|
-
throw new Error(`Apply-mode editor produced no output spec at ${finalPath} (exit ${applyResult.exitCode}): ${applyResult.stderr}`);
|
|
182
|
-
}
|
|
183
|
-
currentSpecPath = finalPath;
|
|
184
|
-
specPathsByTurn.push(currentSpecPath);
|
|
185
|
-
}
|
|
186
|
-
return {
|
|
187
|
-
finalSpecPath: currentSpecPath,
|
|
188
|
-
specPathsByTurn,
|
|
189
|
-
reviewsByTurn,
|
|
190
|
-
finalVerdict: review.verdict,
|
|
191
|
-
turnsUsed: turn,
|
|
192
|
-
hitMaxTurns,
|
|
193
|
-
};
|
|
194
|
-
}
|
|
195
|
-
//# sourceMappingURL=edit-loop.js.map
|
|
@@ -1,54 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Editor worker: revise the composed spec in place based on a reviewer's
|
|
3
|
-
* critique.
|
|
4
|
-
*
|
|
5
|
-
* Two modes:
|
|
6
|
-
* - apply: apply mechanical revisions verbatim (when verdict is `approved-with-revisions`)
|
|
7
|
-
* - revise: address the critique using the editor's own judgment (when verdict is `requires-another-review`)
|
|
8
|
-
*
|
|
9
|
-
* Requirements covered:
|
|
10
|
-
* - CTS-EDIT-3: approved-with-revisions triggers apply mode
|
|
11
|
-
* - CTS-EDIT-4: requires-another-review triggers revise mode
|
|
12
|
-
* - CTS-EDIT-5: editor operates on the cohesive document, reads codebase only on demand
|
|
13
|
-
*/
|
|
14
|
-
import { type ClaudeRunner } from "./claude.js";
|
|
15
|
-
import type { SpecReview } from "./schemas.js";
|
|
16
|
-
export type EditorMode = "apply" | "revise";
|
|
17
|
-
export interface EditorContext {
|
|
18
|
-
/**
|
|
19
|
-
* Input spec path. The editor reads from here.
|
|
20
|
-
* If `outputSpecPath` differs, the input is copied to the output path first
|
|
21
|
-
* (so the editor's in-place edits land on the output).
|
|
22
|
-
*/
|
|
23
|
-
inputSpecPath: string;
|
|
24
|
-
/** Output spec path (may equal inputSpecPath for true in-place editing). */
|
|
25
|
-
outputSpecPath: string;
|
|
26
|
-
/** Path to the codebase pack the editor may consult on demand. */
|
|
27
|
-
fullPackPath: string;
|
|
28
|
-
/** Directories to grant filesystem access to. */
|
|
29
|
-
addDirs: string[];
|
|
30
|
-
/** Reviewer's critique. */
|
|
31
|
-
critique: SpecReview;
|
|
32
|
-
/** Apply or revise. */
|
|
33
|
-
mode: EditorMode;
|
|
34
|
-
runner?: ClaudeRunner;
|
|
35
|
-
model?: string;
|
|
36
|
-
}
|
|
37
|
-
export interface EditorResult {
|
|
38
|
-
/** True if the output file exists and is non-empty after the editor ran. */
|
|
39
|
-
wroteSpec: boolean;
|
|
40
|
-
/** Size of the output in bytes (0 if missing). */
|
|
41
|
-
outputBytes: number;
|
|
42
|
-
/** Process exit code. */
|
|
43
|
-
exitCode: number;
|
|
44
|
-
/** Captured stdout (one-line confirmation). */
|
|
45
|
-
stdout: string;
|
|
46
|
-
/** Captured stderr. */
|
|
47
|
-
stderr: string;
|
|
48
|
-
}
|
|
49
|
-
/**
|
|
50
|
-
* Run the editor. If outputSpecPath differs from inputSpecPath, copy first
|
|
51
|
-
* so the in-place edits land on the output.
|
|
52
|
-
*/
|
|
53
|
-
export declare function runEditor(ctx: EditorContext): Promise<EditorResult>;
|
|
54
|
-
//# sourceMappingURL=editor.d.ts.map
|
|
@@ -1,74 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Editor worker: revise the composed spec in place based on a reviewer's
|
|
3
|
-
* critique.
|
|
4
|
-
*
|
|
5
|
-
* Two modes:
|
|
6
|
-
* - apply: apply mechanical revisions verbatim (when verdict is `approved-with-revisions`)
|
|
7
|
-
* - revise: address the critique using the editor's own judgment (when verdict is `requires-another-review`)
|
|
8
|
-
*
|
|
9
|
-
* Requirements covered:
|
|
10
|
-
* - CTS-EDIT-3: approved-with-revisions triggers apply mode
|
|
11
|
-
* - CTS-EDIT-4: requires-another-review triggers revise mode
|
|
12
|
-
* - CTS-EDIT-5: editor operates on the cohesive document, reads codebase only on demand
|
|
13
|
-
*/
|
|
14
|
-
import { copyFileSync, existsSync, statSync } from "node:fs";
|
|
15
|
-
import { runClaude } from "./claude.js";
|
|
16
|
-
import { EDITOR_PROMPT } from "./prompts/editor.js";
|
|
17
|
-
function userMessage(ctx) {
|
|
18
|
-
const mode = ctx.mode === "apply" ? "APPLY" : "REVISE";
|
|
19
|
-
const lines = [
|
|
20
|
-
`Mode: ${mode}`,
|
|
21
|
-
``,
|
|
22
|
-
`Spec to edit (in place via Edit tool): ${ctx.outputSpecPath}`,
|
|
23
|
-
`Codebase pack (consult only on demand): ${ctx.fullPackPath}`,
|
|
24
|
-
``,
|
|
25
|
-
`## Reviewer critique`,
|
|
26
|
-
`\`\`\`json`,
|
|
27
|
-
JSON.stringify(ctx.critique, null, 2),
|
|
28
|
-
`\`\`\``,
|
|
29
|
-
``,
|
|
30
|
-
];
|
|
31
|
-
if (ctx.mode === "apply") {
|
|
32
|
-
lines.push(`You are in APPLY mode. The reviewer's verdict is \`approved-with-revisions\`. Apply each revision in the \`revisions\` array verbatim. Do not restructure. When done, output a one-line confirmation.`);
|
|
33
|
-
}
|
|
34
|
-
else {
|
|
35
|
-
lines.push(`You are in REVISE mode. The reviewer's verdict is \`requires-another-review\`. Address each finding using your judgment. Use the Edit tool for targeted changes. Consult the codebase only to verify specific findings before adding requirements.`);
|
|
36
|
-
}
|
|
37
|
-
return lines.join("\n");
|
|
38
|
-
}
|
|
39
|
-
/**
|
|
40
|
-
* Run the editor. If outputSpecPath differs from inputSpecPath, copy first
|
|
41
|
-
* so the in-place edits land on the output.
|
|
42
|
-
*/
|
|
43
|
-
export async function runEditor(ctx) {
|
|
44
|
-
if (ctx.inputSpecPath !== ctx.outputSpecPath) {
|
|
45
|
-
copyFileSync(ctx.inputSpecPath, ctx.outputSpecPath);
|
|
46
|
-
}
|
|
47
|
-
const runner = ctx.runner ?? runClaude;
|
|
48
|
-
const result = await runner({
|
|
49
|
-
systemPrompt: EDITOR_PROMPT,
|
|
50
|
-
userMessage: userMessage(ctx),
|
|
51
|
-
model: ctx.model,
|
|
52
|
-
tools: ["Read", "Edit", "Grep", "Glob"],
|
|
53
|
-
addDirs: ctx.addDirs,
|
|
54
|
-
});
|
|
55
|
-
let wroteSpec = false;
|
|
56
|
-
let outputBytes = 0;
|
|
57
|
-
if (existsSync(ctx.outputSpecPath)) {
|
|
58
|
-
try {
|
|
59
|
-
outputBytes = statSync(ctx.outputSpecPath).size;
|
|
60
|
-
wroteSpec = outputBytes > 0;
|
|
61
|
-
}
|
|
62
|
-
catch {
|
|
63
|
-
// Treat as not written.
|
|
64
|
-
}
|
|
65
|
-
}
|
|
66
|
-
return {
|
|
67
|
-
wroteSpec,
|
|
68
|
-
outputBytes,
|
|
69
|
-
exitCode: result.exitCode,
|
|
70
|
-
stdout: result.stdout,
|
|
71
|
-
stderr: result.stderr,
|
|
72
|
-
};
|
|
73
|
-
}
|
|
74
|
-
//# sourceMappingURL=editor.js.map
|
|
@@ -1,63 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Specifier fan-out orchestrator.
|
|
3
|
-
*
|
|
4
|
-
* For each area in the outline, extract a slice, spawn a specifier worker,
|
|
5
|
-
* and write the resulting partial to the cache. Workers run in parallel up
|
|
6
|
-
* to a bounded concurrency limit. On per-area failure (no partial written,
|
|
7
|
-
* empty partial), the worker is retried once; on second failure, the area
|
|
8
|
-
* is recorded as failed for surfacing to the user.
|
|
9
|
-
*
|
|
10
|
-
* Requirements covered:
|
|
11
|
-
* - CTS-SPEC-1.2: bounded concurrent execution
|
|
12
|
-
* - CTS-SPEC-5: failed/empty specifier output is detected and retried
|
|
13
|
-
* - CTS-OBSERVE-2.0: failure details are logged with area name + cause
|
|
14
|
-
* - CTS-RESUME-1.2: existing non-empty partial means the specifier is skipped
|
|
15
|
-
*/
|
|
16
|
-
import type { ClaudeRunner } from "./claude.js";
|
|
17
|
-
import type { ProgressEmitter } from "./progress.js";
|
|
18
|
-
import type { Area, Outline } from "./schemas.js";
|
|
19
|
-
export declare const DEFAULT_FAN_OUT_CONCURRENCY = 8;
|
|
20
|
-
export interface FanOutOptions {
|
|
21
|
-
outline: Outline;
|
|
22
|
-
/** Path to the uncompressed pack (slices come from here). */
|
|
23
|
-
fullPackPath: string;
|
|
24
|
-
/** Path resolver: given a sanitized area name, return the partial path. */
|
|
25
|
-
partialPathFor: (sanitizedAreaName: string) => string;
|
|
26
|
-
/** Path resolver: given a sanitized area name, return the slice path. */
|
|
27
|
-
slicePathFor: (sanitizedAreaName: string) => string;
|
|
28
|
-
/** Command (path + args) the specifier should run to invoke schema validation. Run before style-check. */
|
|
29
|
-
validateCommand: string;
|
|
30
|
-
/** Command (path + args) the specifier should run to invoke style-check. */
|
|
31
|
-
styleCheckCommand: string;
|
|
32
|
-
/** Directories the specifier needs filesystem access to. */
|
|
33
|
-
addDirs: string[];
|
|
34
|
-
concurrency?: number;
|
|
35
|
-
runner?: ClaudeRunner;
|
|
36
|
-
model?: string;
|
|
37
|
-
progress?: ProgressEmitter;
|
|
38
|
-
}
|
|
39
|
-
export interface AreaOutcome {
|
|
40
|
-
area: Area;
|
|
41
|
-
status: "completed" | "skipped-resume" | "failed";
|
|
42
|
-
/** Number of attempts made (1 = first attempt succeeded; 2 = retry succeeded; 2 with failed = retried and still failed). */
|
|
43
|
-
attempts: number;
|
|
44
|
-
/** Reason the area failed, if applicable. */
|
|
45
|
-
failureCause?: string;
|
|
46
|
-
/** Path to the partial (whether or not it was written successfully). */
|
|
47
|
-
partialPath: string;
|
|
48
|
-
}
|
|
49
|
-
export interface FanOutResult {
|
|
50
|
-
outcomes: AreaOutcome[];
|
|
51
|
-
/** True if every area produced a non-empty partial (either now or from cache). */
|
|
52
|
-
allSucceeded: boolean;
|
|
53
|
-
}
|
|
54
|
-
/**
|
|
55
|
-
* Convert an area name to a stable filesystem-safe identifier.
|
|
56
|
-
* E.g., "Reading & Editing Files" → "reading-editing-files"
|
|
57
|
-
*/
|
|
58
|
-
export declare function sanitizeAreaName(name: string): string;
|
|
59
|
-
/**
|
|
60
|
-
* Run the fan-out: extract slices, invoke specifiers, retry on failure.
|
|
61
|
-
*/
|
|
62
|
-
export declare function runFanOut(options: FanOutOptions): Promise<FanOutResult>;
|
|
63
|
-
//# sourceMappingURL=fan-out.d.ts.map
|