aiwcli 0.12.6 → 0.12.8
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/dev.cmd +3 -3
- package/bin/dev.js +16 -16
- package/bin/run.cmd +3 -3
- package/bin/run.js +21 -21
- package/dist/commands/branch.js +7 -2
- package/dist/lib/bmad-installer.js +37 -37
- package/dist/lib/terminal.d.ts +2 -0
- package/dist/lib/terminal.js +57 -7
- package/dist/templates/CLAUDE.md +232 -205
- package/dist/templates/_shared/.claude/settings.json +65 -65
- package/dist/templates/_shared/.claude/{commands/handoff.md → skills/handoff/SKILL.md} +13 -12
- package/dist/templates/_shared/.claude/{commands/handoff-resume.md → skills/handoff-resume/SKILL.md} +13 -12
- package/dist/templates/_shared/.codex/workflows/handoff.md +226 -226
- package/dist/templates/_shared/.windsurf/workflows/handoff.md +226 -226
- package/dist/templates/_shared/handoff-system/CLAUDE.md +15 -3
- package/dist/templates/_shared/handoff-system/lib/document-generator.ts +215 -215
- package/dist/templates/_shared/handoff-system/lib/handoff-reader.ts +158 -158
- package/dist/templates/_shared/handoff-system/scripts/resume_handoff.ts +373 -373
- package/dist/templates/_shared/handoff-system/scripts/save_handoff.ts +469 -469
- package/dist/templates/_shared/handoff-system/workflows/handoff-resume.md +66 -66
- package/dist/templates/_shared/handoff-system/workflows/handoff.md +254 -254
- package/dist/templates/_shared/hooks-ts/_utils/git-state.ts +2 -2
- package/dist/templates/_shared/hooks-ts/archive_plan.ts +159 -159
- package/dist/templates/_shared/hooks-ts/context_monitor.ts +147 -147
- package/dist/templates/_shared/hooks-ts/file-suggestion.ts +128 -128
- package/dist/templates/_shared/hooks-ts/pre_compact.ts +49 -49
- package/dist/templates/_shared/hooks-ts/session_end.ts +196 -196
- package/dist/templates/_shared/hooks-ts/session_start.ts +163 -163
- package/dist/templates/_shared/hooks-ts/task_create_capture.ts +48 -48
- package/dist/templates/_shared/hooks-ts/task_update_capture.ts +74 -74
- package/dist/templates/_shared/hooks-ts/user_prompt_submit.ts +93 -93
- package/dist/templates/_shared/lib-ts/CLAUDE.md +367 -367
- package/dist/templates/_shared/lib-ts/base/atomic-write.ts +138 -138
- package/dist/templates/_shared/lib-ts/base/constants.ts +24 -6
- package/dist/templates/_shared/lib-ts/base/git-state.ts +58 -58
- package/dist/templates/_shared/lib-ts/base/hook-utils.ts +582 -582
- package/dist/templates/_shared/lib-ts/base/inference.ts +301 -301
- package/dist/templates/_shared/lib-ts/base/logger.ts +247 -247
- package/dist/templates/_shared/lib-ts/base/state-io.ts +202 -202
- package/dist/templates/_shared/lib-ts/base/stop-words.ts +184 -184
- package/dist/templates/_shared/lib-ts/base/utils.ts +184 -184
- package/dist/templates/_shared/lib-ts/context/CLAUDE.md +134 -0
- package/dist/templates/_shared/lib-ts/context/context-formatter.ts +566 -566
- package/dist/templates/_shared/lib-ts/context/context-selector.ts +524 -524
- package/dist/templates/_shared/lib-ts/context/context-store.ts +712 -712
- package/dist/templates/_shared/lib-ts/context/plan-manager.ts +312 -312
- package/dist/templates/_shared/lib-ts/context/task-tracker.ts +185 -185
- package/dist/templates/_shared/lib-ts/package.json +20 -20
- package/dist/templates/_shared/lib-ts/templates/formatters.ts +102 -102
- package/dist/templates/_shared/lib-ts/templates/plan-context.ts +58 -58
- package/dist/templates/_shared/lib-ts/tsconfig.json +13 -13
- package/dist/templates/_shared/lib-ts/types.ts +186 -186
- package/dist/templates/_shared/scripts/resolve_context.ts +33 -33
- package/dist/templates/_shared/scripts/status_line.ts +687 -690
- package/dist/templates/cc-native/.claude/commands/cc-native/rlm/ask.md +136 -136
- package/dist/templates/cc-native/.claude/commands/cc-native/rlm/index.md +21 -21
- package/dist/templates/cc-native/.claude/commands/cc-native/rlm/overview.md +56 -56
- package/dist/templates/cc-native/.claude/commands/cc-native/specdev.md +10 -10
- package/dist/templates/cc-native/.claude/settings.json +3 -2
- package/dist/templates/cc-native/.windsurf/workflows/cc-native/fix.md +8 -8
- package/dist/templates/cc-native/.windsurf/workflows/cc-native/implement.md +8 -8
- package/dist/templates/cc-native/.windsurf/workflows/cc-native/research.md +8 -8
- package/dist/templates/cc-native/CC-NATIVE-README.md +189 -189
- package/dist/templates/cc-native/TEMPLATE-SCHEMA.md +304 -304
- package/dist/templates/cc-native/_cc-native/agents/CLAUDE.md +143 -143
- package/dist/templates/cc-native/_cc-native/agents/PLAN-ORCHESTRATOR.md +213 -213
- package/dist/templates/cc-native/_cc-native/agents/plan-questions/PLAN-QUESTIONER.md +70 -70
- package/dist/templates/cc-native/_cc-native/artifacts/CLAUDE.md +64 -0
- package/dist/templates/cc-native/_cc-native/{lib-ts/artifacts → artifacts/lib}/format.ts +1 -1
- package/dist/templates/cc-native/_cc-native/{lib-ts/artifacts → artifacts/lib}/write.ts +2 -2
- package/dist/templates/cc-native/_cc-native/cc-native.config.json +96 -96
- package/dist/templates/cc-native/_cc-native/hooks/CLAUDE.md +14 -24
- package/dist/templates/cc-native/_cc-native/hooks/cc-native-plan-review.ts +1 -1
- package/dist/templates/cc-native/_cc-native/hooks/enhance_plan_post_subagent.ts +54 -54
- package/dist/templates/cc-native/_cc-native/hooks/enhance_plan_post_write.ts +51 -51
- package/dist/templates/cc-native/_cc-native/hooks/mark_questions_asked.ts +53 -53
- package/dist/templates/cc-native/_cc-native/hooks/plan_questions_early.ts +61 -61
- package/dist/templates/cc-native/_cc-native/hooks/validate_task_prompt.ts +76 -0
- package/dist/templates/cc-native/_cc-native/lib-ts/aggregate-agents.ts +9 -2
- package/dist/templates/cc-native/_cc-native/lib-ts/cc-native-state.ts +319 -319
- package/dist/templates/cc-native/_cc-native/lib-ts/cli-output-parser.ts +144 -144
- package/dist/templates/cc-native/_cc-native/lib-ts/config.ts +57 -57
- package/dist/templates/cc-native/_cc-native/lib-ts/constants.ts +83 -83
- package/dist/templates/cc-native/_cc-native/lib-ts/debug.ts +79 -79
- package/dist/templates/cc-native/_cc-native/lib-ts/index.ts +4 -4
- package/dist/templates/cc-native/_cc-native/lib-ts/json-parser.ts +168 -168
- package/dist/templates/cc-native/_cc-native/lib-ts/plan-discovery.ts +80 -80
- package/dist/templates/cc-native/_cc-native/lib-ts/plan-enhancement.ts +41 -41
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/CLAUDE.md +480 -480
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/embedding-indexer.ts +287 -287
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/hyde.ts +148 -148
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/index.ts +54 -54
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/logger.ts +58 -58
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/ollama-client.ts +208 -208
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/retrieval-pipeline.ts +460 -460
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/transcript-indexer.ts +446 -446
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/transcript-loader.ts +280 -280
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/transcript-searcher.ts +274 -274
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/types.ts +201 -201
- package/dist/templates/cc-native/_cc-native/lib-ts/rlm/vector-store.ts +278 -278
- package/dist/templates/cc-native/_cc-native/lib-ts/settings.ts +184 -184
- package/dist/templates/cc-native/_cc-native/lib-ts/state.ts +275 -275
- package/dist/templates/cc-native/_cc-native/lib-ts/tsconfig.json +18 -18
- package/dist/templates/cc-native/_cc-native/lib-ts/types.ts +1 -1
- package/dist/templates/cc-native/_cc-native/plan-review/CLAUDE.md +149 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/CLAUDE.md +143 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/PLAN-ORCHESTRATOR.md +213 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-questions/PLAN-QUESTIONER.md +70 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/ARCH-EVOLUTION.md +62 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/ARCH-PATTERNS.md +61 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/ARCH-STRUCTURE.md +62 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/ASSUMPTION-TRACER.md +56 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/CLARITY-AUDITOR.md +53 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/COMPLETENESS-FEASIBILITY.md +66 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/COMPLETENESS-GAPS.md +70 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/COMPLETENESS-ORDERING.md +62 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/CONSTRAINT-VALIDATOR.md +72 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/DESIGN-ADR-VALIDATOR.md +61 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/DESIGN-SCALE-MATCHER.md +64 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/DEVILS-ADVOCATE.md +56 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/DOCUMENTATION-PHILOSOPHY.md +86 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/HANDOFF-READINESS.md +59 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/HIDDEN-COMPLEXITY.md +58 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/INCREMENTAL-DELIVERY.md +66 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/RISK-DEPENDENCY.md +62 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/RISK-FMEA.md +66 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/RISK-PREMORTEM.md +71 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/RISK-REVERSIBILITY.md +74 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/SCOPE-BOUNDARY.md +77 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/SIMPLICITY-GUARDIAN.md +62 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/SKEPTIC.md +68 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TESTDRIVEN-BEHAVIOR-AUDITOR.md +61 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TESTDRIVEN-CHARACTERIZATION.md +71 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TESTDRIVEN-FIRST-VALIDATOR.md +61 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TESTDRIVEN-PYRAMID-ANALYZER.md +61 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TRADEOFF-COSTS.md +67 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/TRADEOFF-STAKEHOLDERS.md +65 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/VERIFY-COVERAGE.md +74 -0
- package/dist/templates/cc-native/_cc-native/plan-review/agents/plan-review/VERIFY-STRENGTH.md +69 -0
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/agent-selection.ts +3 -3
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/corroboration.ts +1 -1
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/graduation.ts +1 -1
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/orchestrator.ts +2 -2
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/output-builder.ts +3 -3
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/plan-questions.ts +6 -6
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/review-pipeline.ts +15 -15
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/agent.ts +5 -5
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/base/base-agent.ts +4 -4
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/providers/claude-agent.ts +4 -4
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/providers/codex-agent.ts +6 -6
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/providers/gemini-agent.ts +1 -1
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/providers/orchestrator-claude-agent.ts +4 -4
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/types.ts +3 -3
- package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/verdict.ts +1 -1
- package/oclif.manifest.json +1 -1
- package/package.json +108 -108
- package/dist/templates/cc-native/_cc-native/lib-ts/artifacts.ts +0 -21
- package/dist/templates/cc-native/_cc-native/lib-ts/nul +0 -3
- /package/dist/templates/cc-native/_cc-native/{lib-ts/artifacts → artifacts/lib}/index.ts +0 -0
- /package/dist/templates/cc-native/_cc-native/{lib-ts/artifacts → artifacts/lib}/tracker.ts +0 -0
- /package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/index.ts +0 -0
- /package/dist/templates/cc-native/_cc-native/{lib-ts → plan-review/lib}/reviewers/schemas.ts +0 -0
- /package/dist/templates/cc-native/_cc-native/{workflows → plan-review/workflows}/specdev.md +0 -0
|
@@ -1,275 +1,275 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Iteration state management for plan review cycles.
|
|
3
|
-
* State files are stored adjacent to plan files (e.g., foo.md → foo.state.json).
|
|
4
|
-
* See cc-native-plan-review-spec.md §4.7
|
|
5
|
-
*/
|
|
6
|
-
|
|
7
|
-
import * as fs from "node:fs";
|
|
8
|
-
import * as path from "node:path";
|
|
9
|
-
import { atomicWrite } from "../../_shared/lib-ts/base/atomic-write.js";
|
|
10
|
-
import { logInfo, logWarn, logError } from "../../_shared/lib-ts/base/logger.js";
|
|
11
|
-
import { nowIso } from "../../_shared/lib-ts/base/utils.js";
|
|
12
|
-
import { validatePlanPath } from "./constants.js";
|
|
13
|
-
import type { IterationState, IterationEntry } from "./types.js";
|
|
14
|
-
|
|
15
|
-
// ---------------------------------------------------------------------------
|
|
16
|
-
// Constants
|
|
17
|
-
// ---------------------------------------------------------------------------
|
|
18
|
-
|
|
19
|
-
const STATE_SCHEMA_VERSION = "1.0.0";
|
|
20
|
-
|
|
21
|
-
export const DEFAULT_REVIEW_ITERATIONS: Record<string, number> = {
|
|
22
|
-
simple: 1,
|
|
23
|
-
medium: 3,
|
|
24
|
-
high: 5,
|
|
25
|
-
};
|
|
26
|
-
|
|
27
|
-
// ---------------------------------------------------------------------------
|
|
28
|
-
// State File Management
|
|
29
|
-
// ---------------------------------------------------------------------------
|
|
30
|
-
|
|
31
|
-
/**
|
|
32
|
-
* Derive state file path from plan file path with security validation.
|
|
33
|
-
* Example: ~/.claude/plans/foo.md → ~/.claude/plans/foo.state.json
|
|
34
|
-
*
|
|
35
|
-
* @throws Error if planPath is invalid or insecure
|
|
36
|
-
*/
|
|
37
|
-
export function getStateFilePath(planPath: string): string {
|
|
38
|
-
const validated = validatePlanPath(planPath);
|
|
39
|
-
const parsed = path.parse(validated);
|
|
40
|
-
return path.join(parsed.dir, `${parsed.name}.state.json`);
|
|
41
|
-
}
|
|
42
|
-
|
|
43
|
-
/**
|
|
44
|
-
* Load state file with schema validation and migration.
|
|
45
|
-
*/
|
|
46
|
-
export function loadState(planPath: string): Record<string, unknown> | null {
|
|
47
|
-
try {
|
|
48
|
-
const stateFile = getStateFilePath(planPath);
|
|
49
|
-
|
|
50
|
-
if (!fs.existsSync(stateFile)) {
|
|
51
|
-
return null;
|
|
52
|
-
}
|
|
53
|
-
|
|
54
|
-
const state = JSON.parse(
|
|
55
|
-
fs.readFileSync(stateFile, "utf-8"),
|
|
56
|
-
) as Record<string, unknown>;
|
|
57
|
-
|
|
58
|
-
// Handle schema version (backward compatible)
|
|
59
|
-
const schemaVersion = state.schema_version as string | undefined;
|
|
60
|
-
|
|
61
|
-
if (schemaVersion === undefined) {
|
|
62
|
-
state.schema_version = STATE_SCHEMA_VERSION;
|
|
63
|
-
logInfo(
|
|
64
|
-
"state",
|
|
65
|
-
`Migrated state file to schema v${STATE_SCHEMA_VERSION}`,
|
|
66
|
-
);
|
|
67
|
-
} else if (schemaVersion !== STATE_SCHEMA_VERSION) {
|
|
68
|
-
logWarn(
|
|
69
|
-
"state",
|
|
70
|
-
`Schema mismatch (expected ${STATE_SCHEMA_VERSION}, got ${schemaVersion})`,
|
|
71
|
-
);
|
|
72
|
-
}
|
|
73
|
-
|
|
74
|
-
return state;
|
|
75
|
-
} catch (e: unknown) {
|
|
76
|
-
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
77
|
-
logError("state", `SECURITY: Invalid plan path: ${e}`);
|
|
78
|
-
} else {
|
|
79
|
-
logError("state", `Failed to load state: ${e}`);
|
|
80
|
-
}
|
|
81
|
-
return null;
|
|
82
|
-
}
|
|
83
|
-
}
|
|
84
|
-
|
|
85
|
-
/**
|
|
86
|
-
* Save state file with schema version and validation.
|
|
87
|
-
* Returns true on success, false on failure.
|
|
88
|
-
*/
|
|
89
|
-
export function saveStateToPlan(
|
|
90
|
-
planPath: string,
|
|
91
|
-
state: Record<string, unknown>,
|
|
92
|
-
): boolean {
|
|
93
|
-
try {
|
|
94
|
-
const stateFile = getStateFilePath(planPath);
|
|
95
|
-
|
|
96
|
-
const stateWithVersion = {
|
|
97
|
-
schema_version: STATE_SCHEMA_VERSION,
|
|
98
|
-
...state,
|
|
99
|
-
};
|
|
100
|
-
|
|
101
|
-
const [success, error] = atomicWrite(
|
|
102
|
-
stateFile,
|
|
103
|
-
JSON.stringify(stateWithVersion, null, 2),
|
|
104
|
-
);
|
|
105
|
-
|
|
106
|
-
if (!success) {
|
|
107
|
-
logError("state", `Failed to save state: ${error}`);
|
|
108
|
-
return false;
|
|
109
|
-
}
|
|
110
|
-
|
|
111
|
-
return true;
|
|
112
|
-
} catch (e: unknown) {
|
|
113
|
-
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
114
|
-
logError("state", `SECURITY: Invalid plan path: ${e}`);
|
|
115
|
-
} else {
|
|
116
|
-
logError("state", String(e));
|
|
117
|
-
}
|
|
118
|
-
return false;
|
|
119
|
-
}
|
|
120
|
-
}
|
|
121
|
-
|
|
122
|
-
/**
|
|
123
|
-
* Delete state file after successful archive.
|
|
124
|
-
* Returns true if deleted or didn't exist, false on error.
|
|
125
|
-
*/
|
|
126
|
-
export function deleteState(planPath: string): boolean {
|
|
127
|
-
try {
|
|
128
|
-
const stateFile = getStateFilePath(planPath);
|
|
129
|
-
if (fs.existsSync(stateFile)) {
|
|
130
|
-
fs.unlinkSync(stateFile);
|
|
131
|
-
logInfo("state", `Deleted state file: ${stateFile}`);
|
|
132
|
-
}
|
|
133
|
-
return true;
|
|
134
|
-
} catch (e: unknown) {
|
|
135
|
-
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
136
|
-
logError("state", `SECURITY: Invalid plan path in delete: ${e}`);
|
|
137
|
-
return false;
|
|
138
|
-
}
|
|
139
|
-
logWarn("state", `Failed to delete state file: ${e}`);
|
|
140
|
-
return false;
|
|
141
|
-
}
|
|
142
|
-
}
|
|
143
|
-
|
|
144
|
-
// ---------------------------------------------------------------------------
|
|
145
|
-
// Iteration State Management
|
|
146
|
-
// ---------------------------------------------------------------------------
|
|
147
|
-
|
|
148
|
-
/**
|
|
149
|
-
* Get or initialize iteration state based on complexity.
|
|
150
|
-
*/
|
|
151
|
-
export function getIterationState(
|
|
152
|
-
state: Record<string, unknown>,
|
|
153
|
-
complexity: string,
|
|
154
|
-
config?: Record<string, unknown>,
|
|
155
|
-
): IterationState {
|
|
156
|
-
if (state.iteration) {
|
|
157
|
-
return state.iteration as IterationState;
|
|
158
|
-
}
|
|
159
|
-
|
|
160
|
-
// Initialize new iteration state
|
|
161
|
-
const reviewIterations = { ...DEFAULT_REVIEW_ITERATIONS };
|
|
162
|
-
if (config) {
|
|
163
|
-
const overrides = config.reviewIterations as
|
|
164
|
-
| Record<string, number>
|
|
165
|
-
| undefined;
|
|
166
|
-
if (overrides) {
|
|
167
|
-
Object.assign(reviewIterations, overrides);
|
|
168
|
-
}
|
|
169
|
-
}
|
|
170
|
-
|
|
171
|
-
return {
|
|
172
|
-
current: 1,
|
|
173
|
-
max: reviewIterations[complexity] ?? 1,
|
|
174
|
-
complexity,
|
|
175
|
-
history: [],
|
|
176
|
-
graduated: [],
|
|
177
|
-
passStreaks: {},
|
|
178
|
-
lastPlanHash: "",
|
|
179
|
-
lastPlanPath: "",
|
|
180
|
-
};
|
|
181
|
-
}
|
|
182
|
-
|
|
183
|
-
/**
|
|
184
|
-
* Record review result in iteration history and update state.
|
|
185
|
-
*/
|
|
186
|
-
export function updateIterationState(
|
|
187
|
-
state: Record<string, unknown>,
|
|
188
|
-
iteration: IterationState,
|
|
189
|
-
planHash: string,
|
|
190
|
-
verdict: string,
|
|
191
|
-
): Record<string, unknown> {
|
|
192
|
-
const entry: IterationEntry = {
|
|
193
|
-
hash: planHash,
|
|
194
|
-
verdict,
|
|
195
|
-
timestamp: nowIso(),
|
|
196
|
-
};
|
|
197
|
-
|
|
198
|
-
iteration.history.push(entry);
|
|
199
|
-
state.iteration = iteration;
|
|
200
|
-
return state;
|
|
201
|
-
}
|
|
202
|
-
|
|
203
|
-
/**
|
|
204
|
-
* Determine if more review iterations are needed.
|
|
205
|
-
*/
|
|
206
|
-
export function shouldContinueIterating(
|
|
207
|
-
iteration: IterationState,
|
|
208
|
-
verdict: string,
|
|
209
|
-
config?: Record<string, unknown>,
|
|
210
|
-
): boolean {
|
|
211
|
-
const current = iteration.current;
|
|
212
|
-
const maxIter = iteration.max;
|
|
213
|
-
|
|
214
|
-
// At or past max iterations
|
|
215
|
-
if (current >= maxIter) {
|
|
216
|
-
logInfo(
|
|
217
|
-
"state",
|
|
218
|
-
`At max iterations (${current}/${maxIter}), no more iterations`,
|
|
219
|
-
);
|
|
220
|
-
return false;
|
|
221
|
-
}
|
|
222
|
-
|
|
223
|
-
// Check early exit on all pass
|
|
224
|
-
let earlyExit = true;
|
|
225
|
-
if (config) {
|
|
226
|
-
earlyExit = (config.earlyExitOnAllPass as boolean) ?? true;
|
|
227
|
-
}
|
|
228
|
-
if (earlyExit && verdict === "pass") {
|
|
229
|
-
logInfo(
|
|
230
|
-
"state",
|
|
231
|
-
"All reviewers passed and earlyExitOnAllPass=true, exiting early",
|
|
232
|
-
);
|
|
233
|
-
return false;
|
|
234
|
-
}
|
|
235
|
-
|
|
236
|
-
logInfo(
|
|
237
|
-
"state",
|
|
238
|
-
`Continuing to next iteration (${current + 1}/${maxIter}), verdict=${verdict}`,
|
|
239
|
-
);
|
|
240
|
-
return true;
|
|
241
|
-
}
|
|
242
|
-
|
|
243
|
-
// ---------------------------------------------------------------------------
|
|
244
|
-
// Review-level Iteration State I/O (stored in reviews dir)
|
|
245
|
-
// ---------------------------------------------------------------------------
|
|
246
|
-
|
|
247
|
-
/**
|
|
248
|
-
* Load iteration state from the reviews directory.
|
|
249
|
-
*/
|
|
250
|
-
export function loadIterationState(reviewsDir: string): IterationState | null {
|
|
251
|
-
const iterationFile = path.join(reviewsDir, "iteration.json");
|
|
252
|
-
if (!fs.existsSync(iterationFile)) return null;
|
|
253
|
-
try {
|
|
254
|
-
return JSON.parse(fs.readFileSync(iterationFile, "utf-8")) as IterationState;
|
|
255
|
-
} catch (e) {
|
|
256
|
-
logError("state", `Failed to load iteration state: ${e}`);
|
|
257
|
-
return null;
|
|
258
|
-
}
|
|
259
|
-
}
|
|
260
|
-
|
|
261
|
-
/**
|
|
262
|
-
* Save iteration state to the reviews directory.
|
|
263
|
-
*/
|
|
264
|
-
export function saveIterationState(reviewsDir: string, state: IterationState): boolean {
|
|
265
|
-
const iterationFile = path.join(reviewsDir, "iteration.json");
|
|
266
|
-
try {
|
|
267
|
-
fs.mkdirSync(reviewsDir, { recursive: true });
|
|
268
|
-
const toWrite = { ...state, schema_version: "1.0.0" };
|
|
269
|
-
fs.writeFileSync(iterationFile, JSON.stringify(toWrite, null, 2), "utf-8");
|
|
270
|
-
return true;
|
|
271
|
-
} catch (e) {
|
|
272
|
-
logError("state", `Failed to save iteration state: ${e}`);
|
|
273
|
-
return false;
|
|
274
|
-
}
|
|
275
|
-
}
|
|
1
|
+
/**
|
|
2
|
+
* Iteration state management for plan review cycles.
|
|
3
|
+
* State files are stored adjacent to plan files (e.g., foo.md → foo.state.json).
|
|
4
|
+
* See cc-native-plan-review-spec.md §4.7
|
|
5
|
+
*/
|
|
6
|
+
|
|
7
|
+
import * as fs from "node:fs";
|
|
8
|
+
import * as path from "node:path";
|
|
9
|
+
import { atomicWrite } from "../../_shared/lib-ts/base/atomic-write.js";
|
|
10
|
+
import { logInfo, logWarn, logError } from "../../_shared/lib-ts/base/logger.js";
|
|
11
|
+
import { nowIso } from "../../_shared/lib-ts/base/utils.js";
|
|
12
|
+
import { validatePlanPath } from "./constants.js";
|
|
13
|
+
import type { IterationState, IterationEntry } from "./types.js";
|
|
14
|
+
|
|
15
|
+
// ---------------------------------------------------------------------------
|
|
16
|
+
// Constants
|
|
17
|
+
// ---------------------------------------------------------------------------
|
|
18
|
+
|
|
19
|
+
const STATE_SCHEMA_VERSION = "1.0.0";
|
|
20
|
+
|
|
21
|
+
export const DEFAULT_REVIEW_ITERATIONS: Record<string, number> = {
|
|
22
|
+
simple: 1,
|
|
23
|
+
medium: 3,
|
|
24
|
+
high: 5,
|
|
25
|
+
};
|
|
26
|
+
|
|
27
|
+
// ---------------------------------------------------------------------------
|
|
28
|
+
// State File Management
|
|
29
|
+
// ---------------------------------------------------------------------------
|
|
30
|
+
|
|
31
|
+
/**
|
|
32
|
+
* Derive state file path from plan file path with security validation.
|
|
33
|
+
* Example: ~/.claude/plans/foo.md → ~/.claude/plans/foo.state.json
|
|
34
|
+
*
|
|
35
|
+
* @throws Error if planPath is invalid or insecure
|
|
36
|
+
*/
|
|
37
|
+
export function getStateFilePath(planPath: string): string {
|
|
38
|
+
const validated = validatePlanPath(planPath);
|
|
39
|
+
const parsed = path.parse(validated);
|
|
40
|
+
return path.join(parsed.dir, `${parsed.name}.state.json`);
|
|
41
|
+
}
|
|
42
|
+
|
|
43
|
+
/**
|
|
44
|
+
* Load state file with schema validation and migration.
|
|
45
|
+
*/
|
|
46
|
+
export function loadState(planPath: string): Record<string, unknown> | null {
|
|
47
|
+
try {
|
|
48
|
+
const stateFile = getStateFilePath(planPath);
|
|
49
|
+
|
|
50
|
+
if (!fs.existsSync(stateFile)) {
|
|
51
|
+
return null;
|
|
52
|
+
}
|
|
53
|
+
|
|
54
|
+
const state = JSON.parse(
|
|
55
|
+
fs.readFileSync(stateFile, "utf-8"),
|
|
56
|
+
) as Record<string, unknown>;
|
|
57
|
+
|
|
58
|
+
// Handle schema version (backward compatible)
|
|
59
|
+
const schemaVersion = state.schema_version as string | undefined;
|
|
60
|
+
|
|
61
|
+
if (schemaVersion === undefined) {
|
|
62
|
+
state.schema_version = STATE_SCHEMA_VERSION;
|
|
63
|
+
logInfo(
|
|
64
|
+
"state",
|
|
65
|
+
`Migrated state file to schema v${STATE_SCHEMA_VERSION}`,
|
|
66
|
+
);
|
|
67
|
+
} else if (schemaVersion !== STATE_SCHEMA_VERSION) {
|
|
68
|
+
logWarn(
|
|
69
|
+
"state",
|
|
70
|
+
`Schema mismatch (expected ${STATE_SCHEMA_VERSION}, got ${schemaVersion})`,
|
|
71
|
+
);
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
return state;
|
|
75
|
+
} catch (e: unknown) {
|
|
76
|
+
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
77
|
+
logError("state", `SECURITY: Invalid plan path: ${e}`);
|
|
78
|
+
} else {
|
|
79
|
+
logError("state", `Failed to load state: ${e}`);
|
|
80
|
+
}
|
|
81
|
+
return null;
|
|
82
|
+
}
|
|
83
|
+
}
|
|
84
|
+
|
|
85
|
+
/**
|
|
86
|
+
* Save state file with schema version and validation.
|
|
87
|
+
* Returns true on success, false on failure.
|
|
88
|
+
*/
|
|
89
|
+
export function saveStateToPlan(
|
|
90
|
+
planPath: string,
|
|
91
|
+
state: Record<string, unknown>,
|
|
92
|
+
): boolean {
|
|
93
|
+
try {
|
|
94
|
+
const stateFile = getStateFilePath(planPath);
|
|
95
|
+
|
|
96
|
+
const stateWithVersion = {
|
|
97
|
+
schema_version: STATE_SCHEMA_VERSION,
|
|
98
|
+
...state,
|
|
99
|
+
};
|
|
100
|
+
|
|
101
|
+
const [success, error] = atomicWrite(
|
|
102
|
+
stateFile,
|
|
103
|
+
JSON.stringify(stateWithVersion, null, 2),
|
|
104
|
+
);
|
|
105
|
+
|
|
106
|
+
if (!success) {
|
|
107
|
+
logError("state", `Failed to save state: ${error}`);
|
|
108
|
+
return false;
|
|
109
|
+
}
|
|
110
|
+
|
|
111
|
+
return true;
|
|
112
|
+
} catch (e: unknown) {
|
|
113
|
+
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
114
|
+
logError("state", `SECURITY: Invalid plan path: ${e}`);
|
|
115
|
+
} else {
|
|
116
|
+
logError("state", String(e));
|
|
117
|
+
}
|
|
118
|
+
return false;
|
|
119
|
+
}
|
|
120
|
+
}
|
|
121
|
+
|
|
122
|
+
/**
|
|
123
|
+
* Delete state file after successful archive.
|
|
124
|
+
* Returns true if deleted or didn't exist, false on error.
|
|
125
|
+
*/
|
|
126
|
+
export function deleteState(planPath: string): boolean {
|
|
127
|
+
try {
|
|
128
|
+
const stateFile = getStateFilePath(planPath);
|
|
129
|
+
if (fs.existsSync(stateFile)) {
|
|
130
|
+
fs.unlinkSync(stateFile);
|
|
131
|
+
logInfo("state", `Deleted state file: ${stateFile}`);
|
|
132
|
+
}
|
|
133
|
+
return true;
|
|
134
|
+
} catch (e: unknown) {
|
|
135
|
+
if (e instanceof Error && e.message.includes("Invalid plan path")) {
|
|
136
|
+
logError("state", `SECURITY: Invalid plan path in delete: ${e}`);
|
|
137
|
+
return false;
|
|
138
|
+
}
|
|
139
|
+
logWarn("state", `Failed to delete state file: ${e}`);
|
|
140
|
+
return false;
|
|
141
|
+
}
|
|
142
|
+
}
|
|
143
|
+
|
|
144
|
+
// ---------------------------------------------------------------------------
|
|
145
|
+
// Iteration State Management
|
|
146
|
+
// ---------------------------------------------------------------------------
|
|
147
|
+
|
|
148
|
+
/**
|
|
149
|
+
* Get or initialize iteration state based on complexity.
|
|
150
|
+
*/
|
|
151
|
+
export function getIterationState(
|
|
152
|
+
state: Record<string, unknown>,
|
|
153
|
+
complexity: string,
|
|
154
|
+
config?: Record<string, unknown>,
|
|
155
|
+
): IterationState {
|
|
156
|
+
if (state.iteration) {
|
|
157
|
+
return state.iteration as IterationState;
|
|
158
|
+
}
|
|
159
|
+
|
|
160
|
+
// Initialize new iteration state
|
|
161
|
+
const reviewIterations = { ...DEFAULT_REVIEW_ITERATIONS };
|
|
162
|
+
if (config) {
|
|
163
|
+
const overrides = config.reviewIterations as
|
|
164
|
+
| Record<string, number>
|
|
165
|
+
| undefined;
|
|
166
|
+
if (overrides) {
|
|
167
|
+
Object.assign(reviewIterations, overrides);
|
|
168
|
+
}
|
|
169
|
+
}
|
|
170
|
+
|
|
171
|
+
return {
|
|
172
|
+
current: 1,
|
|
173
|
+
max: reviewIterations[complexity] ?? 1,
|
|
174
|
+
complexity,
|
|
175
|
+
history: [],
|
|
176
|
+
graduated: [],
|
|
177
|
+
passStreaks: {},
|
|
178
|
+
lastPlanHash: "",
|
|
179
|
+
lastPlanPath: "",
|
|
180
|
+
};
|
|
181
|
+
}
|
|
182
|
+
|
|
183
|
+
/**
|
|
184
|
+
* Record review result in iteration history and update state.
|
|
185
|
+
*/
|
|
186
|
+
export function updateIterationState(
|
|
187
|
+
state: Record<string, unknown>,
|
|
188
|
+
iteration: IterationState,
|
|
189
|
+
planHash: string,
|
|
190
|
+
verdict: string,
|
|
191
|
+
): Record<string, unknown> {
|
|
192
|
+
const entry: IterationEntry = {
|
|
193
|
+
hash: planHash,
|
|
194
|
+
verdict,
|
|
195
|
+
timestamp: nowIso(),
|
|
196
|
+
};
|
|
197
|
+
|
|
198
|
+
iteration.history.push(entry);
|
|
199
|
+
state.iteration = iteration;
|
|
200
|
+
return state;
|
|
201
|
+
}
|
|
202
|
+
|
|
203
|
+
/**
|
|
204
|
+
* Determine if more review iterations are needed.
|
|
205
|
+
*/
|
|
206
|
+
export function shouldContinueIterating(
|
|
207
|
+
iteration: IterationState,
|
|
208
|
+
verdict: string,
|
|
209
|
+
config?: Record<string, unknown>,
|
|
210
|
+
): boolean {
|
|
211
|
+
const current = iteration.current;
|
|
212
|
+
const maxIter = iteration.max;
|
|
213
|
+
|
|
214
|
+
// At or past max iterations
|
|
215
|
+
if (current >= maxIter) {
|
|
216
|
+
logInfo(
|
|
217
|
+
"state",
|
|
218
|
+
`At max iterations (${current}/${maxIter}), no more iterations`,
|
|
219
|
+
);
|
|
220
|
+
return false;
|
|
221
|
+
}
|
|
222
|
+
|
|
223
|
+
// Check early exit on all pass
|
|
224
|
+
let earlyExit = true;
|
|
225
|
+
if (config) {
|
|
226
|
+
earlyExit = (config.earlyExitOnAllPass as boolean) ?? true;
|
|
227
|
+
}
|
|
228
|
+
if (earlyExit && verdict === "pass") {
|
|
229
|
+
logInfo(
|
|
230
|
+
"state",
|
|
231
|
+
"All reviewers passed and earlyExitOnAllPass=true, exiting early",
|
|
232
|
+
);
|
|
233
|
+
return false;
|
|
234
|
+
}
|
|
235
|
+
|
|
236
|
+
logInfo(
|
|
237
|
+
"state",
|
|
238
|
+
`Continuing to next iteration (${current + 1}/${maxIter}), verdict=${verdict}`,
|
|
239
|
+
);
|
|
240
|
+
return true;
|
|
241
|
+
}
|
|
242
|
+
|
|
243
|
+
// ---------------------------------------------------------------------------
|
|
244
|
+
// Review-level Iteration State I/O (stored in reviews dir)
|
|
245
|
+
// ---------------------------------------------------------------------------
|
|
246
|
+
|
|
247
|
+
/**
|
|
248
|
+
* Load iteration state from the reviews directory.
|
|
249
|
+
*/
|
|
250
|
+
export function loadIterationState(reviewsDir: string): IterationState | null {
|
|
251
|
+
const iterationFile = path.join(reviewsDir, "iteration.json");
|
|
252
|
+
if (!fs.existsSync(iterationFile)) return null;
|
|
253
|
+
try {
|
|
254
|
+
return JSON.parse(fs.readFileSync(iterationFile, "utf-8")) as IterationState;
|
|
255
|
+
} catch (e) {
|
|
256
|
+
logError("state", `Failed to load iteration state: ${e}`);
|
|
257
|
+
return null;
|
|
258
|
+
}
|
|
259
|
+
}
|
|
260
|
+
|
|
261
|
+
/**
|
|
262
|
+
* Save iteration state to the reviews directory.
|
|
263
|
+
*/
|
|
264
|
+
export function saveIterationState(reviewsDir: string, state: IterationState): boolean {
|
|
265
|
+
const iterationFile = path.join(reviewsDir, "iteration.json");
|
|
266
|
+
try {
|
|
267
|
+
fs.mkdirSync(reviewsDir, { recursive: true });
|
|
268
|
+
const toWrite = { ...state, schema_version: "1.0.0" };
|
|
269
|
+
fs.writeFileSync(iterationFile, JSON.stringify(toWrite, null, 2), "utf-8");
|
|
270
|
+
return true;
|
|
271
|
+
} catch (e) {
|
|
272
|
+
logError("state", `Failed to save iteration state: ${e}`);
|
|
273
|
+
return false;
|
|
274
|
+
}
|
|
275
|
+
}
|
|
@@ -1,18 +1,18 @@
|
|
|
1
|
-
{
|
|
2
|
-
"compilerOptions": {
|
|
3
|
-
"target": "es2022",
|
|
4
|
-
"module": "ESNext",
|
|
5
|
-
"moduleResolution": "bundler",
|
|
6
|
-
"strict": true,
|
|
7
|
-
"noUncheckedIndexedAccess": true,
|
|
8
|
-
"noFallthroughCasesInSwitch": true,
|
|
9
|
-
"outDir": "dist",
|
|
10
|
-
"rootDir": "../..",
|
|
11
|
-
"declaration": true,
|
|
12
|
-
"noEmit": true,
|
|
13
|
-
"types": ["bun-types"],
|
|
14
|
-
"skipLibCheck": true
|
|
15
|
-
},
|
|
16
|
-
"include": ["./**/*.ts", "../../_shared/lib-ts/**/*.ts"],
|
|
17
|
-
"exclude": ["../../_shared/lib-ts/__tests__/**"]
|
|
18
|
-
}
|
|
1
|
+
{
|
|
2
|
+
"compilerOptions": {
|
|
3
|
+
"target": "es2022",
|
|
4
|
+
"module": "ESNext",
|
|
5
|
+
"moduleResolution": "bundler",
|
|
6
|
+
"strict": true,
|
|
7
|
+
"noUncheckedIndexedAccess": true,
|
|
8
|
+
"noFallthroughCasesInSwitch": true,
|
|
9
|
+
"outDir": "dist",
|
|
10
|
+
"rootDir": "../..",
|
|
11
|
+
"declaration": true,
|
|
12
|
+
"noEmit": true,
|
|
13
|
+
"types": ["bun-types"],
|
|
14
|
+
"skipLibCheck": true
|
|
15
|
+
},
|
|
16
|
+
"include": ["./**/*.ts", "../../_shared/lib-ts/**/*.ts"],
|
|
17
|
+
"exclude": ["../../_shared/lib-ts/__tests__/**"]
|
|
18
|
+
}
|
|
@@ -273,7 +273,7 @@ export interface Reviewer {
|
|
|
273
273
|
// JSON Schemas (moved to reviewers/schemas.ts)
|
|
274
274
|
// ---------------------------------------------------------------------------
|
|
275
275
|
// Re-export for backwards compatibility
|
|
276
|
-
export { REVIEW_SCHEMA, ORCHESTRATOR_SCHEMA, REVIEW_PROMPT_PREFIX, AGENT_REVIEW_PROMPT_PREFIX } from "
|
|
276
|
+
export { REVIEW_SCHEMA, ORCHESTRATOR_SCHEMA, REVIEW_PROMPT_PREFIX, AGENT_REVIEW_PROMPT_PREFIX } from "../plan-review/lib/reviewers/schemas.js";
|
|
277
277
|
|
|
278
278
|
// ---------------------------------------------------------------------------
|
|
279
279
|
// Display Defaults
|