@momentiq/dark-factory-cli 1.1.0 → 2.1.0
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/README.md +86 -2
- package/dist/adapters/critic-result-schema.d.ts +14 -14
- package/dist/adapters/index.d.ts +1 -0
- package/dist/adapters/index.d.ts.map +1 -1
- package/dist/adapters/index.js +7 -0
- package/dist/adapters/index.js.map +1 -1
- package/dist/adapters/static-schema-lint.d.ts +68 -0
- package/dist/adapters/static-schema-lint.d.ts.map +1 -0
- package/dist/adapters/static-schema-lint.js +813 -0
- package/dist/adapters/static-schema-lint.js.map +1 -0
- package/dist/cli.d.ts +17 -1
- package/dist/cli.d.ts.map +1 -1
- package/dist/cli.js +313 -41
- package/dist/cli.js.map +1 -1
- package/dist/commands/findings.d.ts +6 -0
- package/dist/commands/findings.d.ts.map +1 -0
- package/dist/commands/findings.js +196 -0
- package/dist/commands/findings.js.map +1 -0
- package/dist/commands/onboard.d.ts +6 -0
- package/dist/commands/onboard.d.ts.map +1 -0
- package/dist/commands/onboard.js +111 -0
- package/dist/commands/onboard.js.map +1 -0
- package/dist/commands/skills.d.ts +6 -0
- package/dist/commands/skills.d.ts.map +1 -0
- package/dist/commands/skills.js +248 -0
- package/dist/commands/skills.js.map +1 -0
- package/dist/cycle-doc-validator/validate_cycle_doc.py +131 -60
- package/dist/doctor.d.ts +17 -0
- package/dist/doctor.d.ts.map +1 -1
- package/dist/doctor.js +107 -0
- package/dist/doctor.js.map +1 -1
- package/dist/evidence/docker-build.d.ts +6 -0
- package/dist/evidence/docker-build.d.ts.map +1 -0
- package/dist/evidence/docker-build.js +199 -0
- package/dist/evidence/docker-build.js.map +1 -0
- package/dist/evidence/index.d.ts +1 -0
- package/dist/evidence/index.d.ts.map +1 -1
- package/dist/evidence/index.js +6 -0
- package/dist/evidence/index.js.map +1 -1
- package/dist/index.d.ts +1 -0
- package/dist/index.d.ts.map +1 -1
- package/dist/index.js +4 -0
- package/dist/index.js.map +1 -1
- package/dist/mcp/server.d.ts.map +1 -1
- package/dist/mcp/server.js +2 -0
- package/dist/mcp/server.js.map +1 -1
- package/dist/mcp/tools/doctor.d.ts.map +1 -1
- package/dist/mcp/tools/doctor.js +5 -0
- package/dist/mcp/tools/doctor.js.map +1 -1
- package/dist/mcp/tools/review-bypass.d.ts.map +1 -1
- package/dist/mcp/tools/review-bypass.js +21 -0
- package/dist/mcp/tools/review-bypass.js.map +1 -1
- package/dist/mcp/tools/skills-install.d.ts +6 -0
- package/dist/mcp/tools/skills-install.d.ts.map +1 -0
- package/dist/mcp/tools/skills-install.js +260 -0
- package/dist/mcp/tools/skills-install.js.map +1 -0
- package/dist/mcp/tools/stats-gate.d.ts.map +1 -1
- package/dist/mcp/tools/stats-gate.js +63 -15
- package/dist/mcp/tools/stats-gate.js.map +1 -1
- package/dist/onboard/analyze.d.ts +7 -0
- package/dist/onboard/analyze.d.ts.map +1 -0
- package/dist/onboard/analyze.js +86 -0
- package/dist/onboard/analyze.js.map +1 -0
- package/dist/onboard/analyzer.d.ts +15 -0
- package/dist/onboard/analyzer.d.ts.map +1 -0
- package/dist/onboard/analyzer.js +52 -0
- package/dist/onboard/analyzer.js.map +1 -0
- package/dist/onboard/analyzers/ci.d.ts +3 -0
- package/dist/onboard/analyzers/ci.d.ts.map +1 -0
- package/dist/onboard/analyzers/ci.js +112 -0
- package/dist/onboard/analyzers/ci.js.map +1 -0
- package/dist/onboard/analyzers/docs.d.ts +3 -0
- package/dist/onboard/analyzers/docs.d.ts.map +1 -0
- package/dist/onboard/analyzers/docs.js +151 -0
- package/dist/onboard/analyzers/docs.js.map +1 -0
- package/dist/onboard/analyzers/git.d.ts +3 -0
- package/dist/onboard/analyzers/git.d.ts.map +1 -0
- package/dist/onboard/analyzers/git.js +87 -0
- package/dist/onboard/analyzers/git.js.map +1 -0
- package/dist/onboard/analyzers/lockfile.d.ts +3 -0
- package/dist/onboard/analyzers/lockfile.d.ts.map +1 -0
- package/dist/onboard/analyzers/lockfile.js +285 -0
- package/dist/onboard/analyzers/lockfile.js.map +1 -0
- package/dist/onboard/analyzers/manifest.d.ts +3 -0
- package/dist/onboard/analyzers/manifest.d.ts.map +1 -0
- package/dist/onboard/analyzers/manifest.js +228 -0
- package/dist/onboard/analyzers/manifest.js.map +1 -0
- package/dist/onboard/analyzers/tree.d.ts +3 -0
- package/dist/onboard/analyzers/tree.d.ts.map +1 -0
- package/dist/onboard/analyzers/tree.js +196 -0
- package/dist/onboard/analyzers/tree.js.map +1 -0
- package/dist/onboard/fixtures/replay-git-history.d.ts +20 -0
- package/dist/onboard/fixtures/replay-git-history.d.ts.map +1 -0
- package/dist/onboard/fixtures/replay-git-history.js +91 -0
- package/dist/onboard/fixtures/replay-git-history.js.map +1 -0
- package/dist/onboard/schema.d.ts +545 -0
- package/dist/onboard/schema.d.ts.map +1 -0
- package/dist/onboard/schema.js +142 -0
- package/dist/onboard/schema.js.map +1 -0
- package/dist/policy/gate.d.ts +1 -1
- package/dist/policy/gate.d.ts.map +1 -1
- package/dist/policy/gate.js +96 -12
- package/dist/policy/gate.js.map +1 -1
- package/dist/prompt.d.ts +1 -0
- package/dist/prompt.d.ts.map +1 -1
- package/dist/prompt.js +105 -1
- package/dist/prompt.js.map +1 -1
- package/dist/report.d.ts +106 -3
- package/dist/report.d.ts.map +1 -1
- package/dist/report.js +282 -10
- package/dist/report.js.map +1 -1
- package/dist/runner.d.ts +2 -0
- package/dist/runner.d.ts.map +1 -1
- package/dist/runner.js +86 -2
- package/dist/runner.js.map +1 -1
- package/dist/self-consistency.d.ts +144 -0
- package/dist/self-consistency.d.ts.map +1 -0
- package/dist/self-consistency.js +368 -0
- package/dist/self-consistency.js.map +1 -0
- package/dist/skills/config.d.ts +176 -0
- package/dist/skills/config.d.ts.map +1 -0
- package/dist/skills/config.js +251 -0
- package/dist/skills/config.js.map +1 -0
- package/dist/skills/index.d.ts +4 -0
- package/dist/skills/index.d.ts.map +1 -0
- package/dist/skills/index.js +8 -0
- package/dist/skills/index.js.map +1 -0
- package/dist/skills/install.d.ts +62 -0
- package/dist/skills/install.d.ts.map +1 -0
- package/dist/skills/install.js +315 -0
- package/dist/skills/install.js.map +1 -0
- package/dist/skills/template.d.ts +42 -0
- package/dist/skills/template.d.ts.map +1 -0
- package/dist/skills/template.js +95 -0
- package/dist/skills/template.js.map +1 -0
- package/dist/trusted-surface/rebind.d.ts.map +1 -1
- package/dist/trusted-surface/rebind.js +14 -1
- package/dist/trusted-surface/rebind.js.map +1 -1
- package/package.json +36 -2
- package/skills/README.md +89 -0
- package/skills/chief-engineer-blitz/SKILL.md.tmpl +443 -0
- package/skills/chief-engineer-blitz/skill.json +53 -0
- package/skills/chief-engineer-review/SKILL.md.tmpl +184 -0
- package/skills/chief-engineer-review/references/review-examples.md.tmpl +130 -0
- package/skills/chief-engineer-review/skill.json +67 -0
- package/skills/chief-engineer-review/templates/code-review-prompt.md.tmpl +111 -0
- package/skills/chief-engineer-review/templates/escalation-prompt.md.tmpl +56 -0
- package/skills/chief-engineer-review/templates/plan-review-prompt.md.tmpl +74 -0
- package/skills/skill-schema.json +73 -0
|
@@ -0,0 +1,248 @@
|
|
|
1
|
+
// `df skills install/list` — CLI surface for the bundled-skill installer.
|
|
2
|
+
//
|
|
3
|
+
// DFP #192: the OSS CLI owns the source-of-truth for skills that ship as
|
|
4
|
+
// part of the Dark Factory adoption (chief-engineer-review,
|
|
5
|
+
// chief-engineer-blitz, and any future ones). This subcommand renders the
|
|
6
|
+
// bundled skill templates against the consumer's `darkfactory.yaml` and
|
|
7
|
+
// writes the result to `.claude/skills/<name>/` in the consumer repo.
|
|
8
|
+
//
|
|
9
|
+
// Sub-subcommand pattern mirrors `df flow` — top-level `df skills` fans
|
|
10
|
+
// out to `install` / `list` inside cmdSkills rather than registering each
|
|
11
|
+
// at the top of cli.ts. Keeps the namespace grouped.
|
|
12
|
+
//
|
|
13
|
+
// CLI mirror of: `df_skills_install` + `df_skills_list` MCP tools (the
|
|
14
|
+
// same `installSkill` / `listBundledSkills` core powers both).
|
|
15
|
+
import { enabledSkillNames, loadDarkFactoryConfig, } from "../skills/config.js";
|
|
16
|
+
import { installSkill, KNOWN_SKILLS, listBundledSkills, } from "../skills/install.js";
|
|
17
|
+
const TOP_HELP = [
|
|
18
|
+
"df skills — install consumer-shape templated skills bundled with @momentiq/dark-factory-cli.",
|
|
19
|
+
"",
|
|
20
|
+
"Usage:",
|
|
21
|
+
" df skills install <name> Render + install one bundled skill.",
|
|
22
|
+
" df skills install --all Install every skill declared `enabled: true`",
|
|
23
|
+
" in darkfactory.yaml.",
|
|
24
|
+
" df skills list List bundled skills + their summaries.",
|
|
25
|
+
" df skills --help, -h Show this message.",
|
|
26
|
+
"",
|
|
27
|
+
"Install flags (df skills install):",
|
|
28
|
+
" --force Overwrite a hand-edited rendered file",
|
|
29
|
+
" (skipped by default — the renderer detects",
|
|
30
|
+
" the absence of the GENERATED marker).",
|
|
31
|
+
" --target-dir <path> Override the install location",
|
|
32
|
+
" (default: <cwd>/.claude/skills/<name>/).",
|
|
33
|
+
" --json Print the install result as JSON.",
|
|
34
|
+
"",
|
|
35
|
+
"Bundled skills:",
|
|
36
|
+
...KNOWN_SKILLS.map((s) => ` - ${s}`),
|
|
37
|
+
"",
|
|
38
|
+
"Consumer config: place `darkfactory.yaml` at the repo root to override the",
|
|
39
|
+
"rendered values (REPO_NAME, MANIFESTO_PATH, ADR_DIR, etc.). See the project",
|
|
40
|
+
"README for the schema.",
|
|
41
|
+
"",
|
|
42
|
+
].join("\n");
|
|
43
|
+
function parseInstallArgs(rest) {
|
|
44
|
+
let skillName;
|
|
45
|
+
let all = false;
|
|
46
|
+
let force = false;
|
|
47
|
+
let json = false;
|
|
48
|
+
let targetDir;
|
|
49
|
+
for (let i = 0; i < rest.length; i++) {
|
|
50
|
+
const a = rest[i] ?? "";
|
|
51
|
+
if (a === "--all") {
|
|
52
|
+
all = true;
|
|
53
|
+
continue;
|
|
54
|
+
}
|
|
55
|
+
if (a === "--force") {
|
|
56
|
+
force = true;
|
|
57
|
+
continue;
|
|
58
|
+
}
|
|
59
|
+
if (a === "--json") {
|
|
60
|
+
json = true;
|
|
61
|
+
continue;
|
|
62
|
+
}
|
|
63
|
+
if (a === "--target-dir") {
|
|
64
|
+
const next = rest[i + 1];
|
|
65
|
+
if (next === undefined || next.startsWith("--")) {
|
|
66
|
+
return { error: "--target-dir requires a value." };
|
|
67
|
+
}
|
|
68
|
+
targetDir = next;
|
|
69
|
+
i++;
|
|
70
|
+
continue;
|
|
71
|
+
}
|
|
72
|
+
if (a.startsWith("--target-dir=")) {
|
|
73
|
+
targetDir = a.slice("--target-dir=".length);
|
|
74
|
+
continue;
|
|
75
|
+
}
|
|
76
|
+
if (a.startsWith("--")) {
|
|
77
|
+
return { error: `unknown flag: ${a}` };
|
|
78
|
+
}
|
|
79
|
+
if (skillName === undefined) {
|
|
80
|
+
skillName = a;
|
|
81
|
+
continue;
|
|
82
|
+
}
|
|
83
|
+
return { error: `unexpected positional arg: ${a}` };
|
|
84
|
+
}
|
|
85
|
+
if (!all && skillName === undefined) {
|
|
86
|
+
return { error: "missing skill name (or --all). Run `df skills --help`." };
|
|
87
|
+
}
|
|
88
|
+
if (all && skillName !== undefined) {
|
|
89
|
+
return { error: "cannot pass both <name> and --all." };
|
|
90
|
+
}
|
|
91
|
+
const out = {
|
|
92
|
+
all,
|
|
93
|
+
force,
|
|
94
|
+
json,
|
|
95
|
+
...(skillName !== undefined ? { skillName } : {}),
|
|
96
|
+
...(targetDir !== undefined ? { targetDir } : {}),
|
|
97
|
+
};
|
|
98
|
+
return out;
|
|
99
|
+
}
|
|
100
|
+
function renderInstallTextSummary(r) {
|
|
101
|
+
const lines = [];
|
|
102
|
+
lines.push(`df skills install: ${r.skillName} v${r.manifestVersion} → ${r.files.length} file(s) processed`);
|
|
103
|
+
lines.push(` config: ${r.configPath}${r.configIsDefault ? " (defaults — no darkfactory.yaml present)" : ""}`);
|
|
104
|
+
for (const f of r.files) {
|
|
105
|
+
if (f.action === "skipped") {
|
|
106
|
+
lines.push(` ! ${f.relTarget} — SKIPPED (${f.reason ?? "skipped"})`);
|
|
107
|
+
}
|
|
108
|
+
else {
|
|
109
|
+
lines.push(` - ${f.relTarget} — ${f.action}`);
|
|
110
|
+
}
|
|
111
|
+
}
|
|
112
|
+
if (r.resolvedVariables.length > 0) {
|
|
113
|
+
lines.push(` variables resolved:`);
|
|
114
|
+
for (const v of r.resolvedVariables) {
|
|
115
|
+
// Truncate multi-line list values to first line + ellipsis for the
|
|
116
|
+
// text summary — full values are visible in --json.
|
|
117
|
+
const oneLine = v.value.includes("\n")
|
|
118
|
+
? `${v.value.split("\n")[0]} … (${v.value.split("\n").length} lines)`
|
|
119
|
+
: v.value;
|
|
120
|
+
lines.push(` {{${v.name}}} = ${oneLine}`);
|
|
121
|
+
}
|
|
122
|
+
}
|
|
123
|
+
return lines.join("\n");
|
|
124
|
+
}
|
|
125
|
+
async function cmdSkillsInstall(rest, io) {
|
|
126
|
+
if (rest.includes("--help") || rest.includes("-h")) {
|
|
127
|
+
io.stdout(TOP_HELP);
|
|
128
|
+
return 0;
|
|
129
|
+
}
|
|
130
|
+
const parsed = parseInstallArgs(rest);
|
|
131
|
+
if ("error" in parsed) {
|
|
132
|
+
io.stderr(`df skills install: ${parsed.error}\n`);
|
|
133
|
+
return 2;
|
|
134
|
+
}
|
|
135
|
+
const cwd = process.cwd();
|
|
136
|
+
let targetSkills;
|
|
137
|
+
if (parsed.all) {
|
|
138
|
+
// Reject --all + --target-dir at the CLI surface: every bundled skill
|
|
139
|
+
// targets SKILL.md, so a shared target-dir would have the second install
|
|
140
|
+
// overwrite the first. Surface a clear error so the consumer either drops
|
|
141
|
+
// --target-dir (gets the default per-skill .claude/skills/<name>/) or
|
|
142
|
+
// splits the invocations one-skill-at-a-time.
|
|
143
|
+
if (parsed.targetDir !== undefined) {
|
|
144
|
+
io.stderr(`df skills install: --all is incompatible with --target-dir (bundled skills share target filenames like SKILL.md, so a single dir would overwrite). Install each skill separately, or omit --target-dir to use the default <cwd>/.claude/skills/<name>/.\n`);
|
|
145
|
+
return 2;
|
|
146
|
+
}
|
|
147
|
+
const loaded = loadDarkFactoryConfig(cwd);
|
|
148
|
+
const enabled = enabledSkillNames(loaded.config);
|
|
149
|
+
if (enabled.length === 0) {
|
|
150
|
+
io.stderr(`df skills install: --all found no skills marked enabled: true in ${loaded.configPath} (${loaded.isDefault ? "no darkfactory.yaml present" : "present"}).\n`);
|
|
151
|
+
return 1;
|
|
152
|
+
}
|
|
153
|
+
const unknown = enabled.filter((name) => !KNOWN_SKILLS.includes(name));
|
|
154
|
+
if (unknown.length > 0) {
|
|
155
|
+
io.stderr(`df skills install: --all rejected — unknown skill name(s) in ${loaded.configPath}: ${unknown.join(", ")}. Known skills: ${KNOWN_SKILLS.join(", ")}.\n`);
|
|
156
|
+
return 2;
|
|
157
|
+
}
|
|
158
|
+
targetSkills = enabled;
|
|
159
|
+
}
|
|
160
|
+
else if (parsed.skillName !== undefined) {
|
|
161
|
+
targetSkills = [parsed.skillName];
|
|
162
|
+
}
|
|
163
|
+
else {
|
|
164
|
+
// Shouldn't be reachable — parseInstallArgs guards.
|
|
165
|
+
io.stderr(`df skills install: no skill named.\n`);
|
|
166
|
+
return 2;
|
|
167
|
+
}
|
|
168
|
+
const results = [];
|
|
169
|
+
let exitCode = 0;
|
|
170
|
+
for (const name of targetSkills) {
|
|
171
|
+
try {
|
|
172
|
+
const opts = {
|
|
173
|
+
cwd,
|
|
174
|
+
skillName: name,
|
|
175
|
+
force: parsed.force,
|
|
176
|
+
...(parsed.targetDir !== undefined ? { targetDir: parsed.targetDir } : {}),
|
|
177
|
+
};
|
|
178
|
+
const r = installSkill(opts);
|
|
179
|
+
results.push(r);
|
|
180
|
+
// Non-fatal but worth surfacing: if any file was skipped (hand-edited
|
|
181
|
+
// without --force), exit code 3 so CI catches the partial install.
|
|
182
|
+
if (r.files.some((f) => f.action === "skipped")) {
|
|
183
|
+
exitCode = Math.max(exitCode, 3);
|
|
184
|
+
}
|
|
185
|
+
}
|
|
186
|
+
catch (err) {
|
|
187
|
+
io.stderr(`df skills install: ${err.message}\n`);
|
|
188
|
+
return 1;
|
|
189
|
+
}
|
|
190
|
+
}
|
|
191
|
+
if (parsed.json) {
|
|
192
|
+
io.stdout(`${JSON.stringify({ installed: results }, null, 2)}\n`);
|
|
193
|
+
}
|
|
194
|
+
else {
|
|
195
|
+
for (const r of results) {
|
|
196
|
+
io.stdout(`${renderInstallTextSummary(r)}\n`);
|
|
197
|
+
}
|
|
198
|
+
if (exitCode === 3) {
|
|
199
|
+
io.stdout(`\nNote: one or more files were skipped because they were hand-edited. Re-run with --force to overwrite.\n`);
|
|
200
|
+
}
|
|
201
|
+
}
|
|
202
|
+
return exitCode;
|
|
203
|
+
}
|
|
204
|
+
function cmdSkillsList(rest, io) {
|
|
205
|
+
if (rest.includes("--help") || rest.includes("-h")) {
|
|
206
|
+
io.stdout(TOP_HELP);
|
|
207
|
+
return 0;
|
|
208
|
+
}
|
|
209
|
+
const json = rest.includes("--json");
|
|
210
|
+
const items = listBundledSkills();
|
|
211
|
+
if (json) {
|
|
212
|
+
io.stdout(`${JSON.stringify({ skills: items }, null, 2)}\n`);
|
|
213
|
+
return 0;
|
|
214
|
+
}
|
|
215
|
+
if (items.length === 0) {
|
|
216
|
+
io.stdout(`df skills list: no bundled skills.\n`);
|
|
217
|
+
return 0;
|
|
218
|
+
}
|
|
219
|
+
io.stdout(`df skills list — ${items.length} bundled skill(s)\n`);
|
|
220
|
+
for (const item of items) {
|
|
221
|
+
io.stdout(`\n ${item.name} (v${item.version})`);
|
|
222
|
+
if (item.originatingRepo)
|
|
223
|
+
io.stdout(` origin: ${item.originatingRepo}`);
|
|
224
|
+
io.stdout(`\n ${item.summary}\n`);
|
|
225
|
+
}
|
|
226
|
+
return 0;
|
|
227
|
+
}
|
|
228
|
+
export async function cmdSkills(rest) {
|
|
229
|
+
const io = {
|
|
230
|
+
stdout: (s) => process.stdout.write(s),
|
|
231
|
+
stderr: (s) => process.stderr.write(s),
|
|
232
|
+
};
|
|
233
|
+
if (rest.length === 0 || rest[0] === "--help" || rest[0] === "-h") {
|
|
234
|
+
io.stdout(TOP_HELP);
|
|
235
|
+
return 0;
|
|
236
|
+
}
|
|
237
|
+
const sub = rest[0];
|
|
238
|
+
const subRest = rest.slice(1);
|
|
239
|
+
if (sub === "install") {
|
|
240
|
+
return await cmdSkillsInstall(subRest, io);
|
|
241
|
+
}
|
|
242
|
+
if (sub === "list") {
|
|
243
|
+
return cmdSkillsList(subRest, io);
|
|
244
|
+
}
|
|
245
|
+
io.stderr(`df skills: unknown subcommand "${sub}". Run \`df skills --help\`.\n`);
|
|
246
|
+
return 2;
|
|
247
|
+
}
|
|
248
|
+
//# sourceMappingURL=skills.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"skills.js","sourceRoot":"","sources":["../../src/commands/skills.ts"],"names":[],"mappings":"AAAA,0EAA0E;AAC1E,EAAE;AACF,yEAAyE;AACzE,4DAA4D;AAC5D,0EAA0E;AAC1E,wEAAwE;AACxE,sEAAsE;AACtE,EAAE;AACF,wEAAwE;AACxE,0EAA0E;AAC1E,qDAAqD;AACrD,EAAE;AACF,uEAAuE;AACvE,+DAA+D;AAE/D,OAAO,EACL,iBAAiB,EACjB,qBAAqB,GACtB,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EACL,YAAY,EACZ,YAAY,EACZ,iBAAiB,GAElB,MAAM,sBAAsB,CAAC;AAO9B,MAAM,QAAQ,GAAG;IACf,8FAA8F;IAC9F,EAAE;IACF,QAAQ;IACR,sEAAsE;IACtE,+EAA+E;IAC/E,uDAAuD;IACvD,yEAAyE;IACzE,qDAAqD;IACrD,EAAE;IACF,oCAAoC;IACpC,wEAAwE;IACxE,6EAA6E;IAC7E,wEAAwE;IACxE,gEAAgE;IAChE,2EAA2E;IAC3E,oEAAoE;IACpE,EAAE;IACF,iBAAiB;IACjB,GAAG,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACtC,EAAE;IACF,4EAA4E;IAC5E,6EAA6E;IAC7E,wBAAwB;IACxB,EAAE;CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAUb,SAAS,gBAAgB,CAAC,IAAc;IACtC,IAAI,SAA6B,CAAC;IAClC,IAAI,GAAG,GAAG,KAAK,CAAC;IAChB,IAAI,KAAK,GAAG,KAAK,CAAC;IAClB,IAAI,IAAI,GAAG,KAAK,CAAC;IACjB,IAAI,SAA6B,CAAC;IAClC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACrC,MAAM,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACxB,IAAI,CAAC,KAAK,OAAO,EAAE,CAAC;YAClB,GAAG,GAAG,IAAI,CAAC;YACX,SAAS;QACX,CAAC;QACD,IAAI,CAAC,KAAK,SAAS,EAAE,CAAC;YACpB,KAAK,GAAG,IAAI,CAAC;YACb,SAAS;QACX,CAAC;QACD,IAAI,CAAC,KAAK,QAAQ,EAAE,CAAC;YACnB,IAAI,GAAG,IAAI,CAAC;YACZ,SAAS;QACX,CAAC;QACD,IAAI,CAAC,KAAK,cAAc,EAAE,CAAC;YACzB,MAAM,IAAI,GAAG,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;YACzB,IAAI,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;gBAChD,OAAO,EAAE,KAAK,EAAE,gCAAgC,EAAE,CAAC;YACrD,CAAC;YACD,SAAS,GAAG,IAAI,CAAC;YACjB,CAAC,EAAE,CAAC;YACJ,SAAS;QACX,CAAC;QACD,IAAI,CAAC,CAAC,UAAU,CAAC,eAAe,CAAC,EAAE,CAAC;YAClC,SAAS,GAAG,CAAC,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;YAC5C,SAAS;QACX,CAAC;QACD,IAAI,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;YACvB,OAAO,EAAE,KAAK,EAAE,iBAAiB,CAAC,EAAE,EAAE,CAAC;QACzC,CAAC;QACD,IAAI,SAAS,KAAK,SAAS,EAAE,CAAC;YAC5B,SAAS,GAAG,CAAC,CAAC;YACd,SAAS;QACX,CAAC;QACD,OAAO,EAAE,KAAK,EAAE,8BAA8B,CAAC,EAAE,EAAE,CAAC;IACtD,CAAC;IACD,IAAI,CAAC,GAAG,IAAI,SAAS,KAAK,SAAS,EAAE,CAAC;QACpC,OAAO,EAAE,KAAK,EAAE,wDAAwD,EAAE,CAAC;IAC7E,CAAC;IACD,IAAI,GAAG,IAAI,SAAS,KAAK,SAAS,EAAE,CAAC;QACnC,OAAO,EAAE,KAAK,EAAE,oCAAoC,EAAE,CAAC;IACzD,CAAC;IACD,MAAM,GAAG,GAAsB;QAC7B,GAAG;QACH,KAAK;QACL,IAAI;QACJ,GAAG,CAAC,SAAS,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACjD,GAAG,CAAC,SAAS,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KAClD,CAAC;IACF,OAAO,GAAG,CAAC;AACb,CAAC;AAED,SAAS,wBAAwB,CAAC,CAAgB;IAChD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,CAAC,IAAI,CACR,sBAAsB,CAAC,CAAC,SAAS,KAAK,CAAC,CAAC,eAAe,MAAM,CAAC,CAAC,KAAK,CAAC,MAAM,oBAAoB,CAChG,CAAC;IACF,KAAK,CAAC,IAAI,CACR,aAAa,CAAC,CAAC,UAAU,GAAG,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,2CAA2C,CAAC,CAAC,CAAC,EAAE,EAAE,CACnG,CAAC;IACF,KAAK,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,CAAC;QACxB,IAAI,CAAC,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;YAC3B,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,SAAS,eAAe,CAAC,CAAC,MAAM,IAAI,SAAS,GAAG,CAAC,CAAC;QACxE,CAAC;aAAM,CAAC;YACN,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,SAAS,MAAM,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;QACjD,CAAC;IACH,CAAC;IACD,IAAI,CAAC,CAAC,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACnC,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;QACpC,KAAK,MAAM,CAAC,IAAI,CAAC,CAAC,iBAAiB,EAAE,CAAC;YACpC,mEAAmE;YACnE,oDAAoD;YACpD,MAAM,OAAO,GAAG,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC;gBACpC,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,SAAS;gBACrE,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC;YACZ,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,QAAQ,OAAO,EAAE,CAAC,CAAC;QAC/C,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED,KAAK,UAAU,gBAAgB,CAAC,IAAc,EAAE,EAAY;IAC1D,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACnD,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QACpB,OAAO,CAAC,CAAC;IACX,CAAC;IACD,MAAM,MAAM,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;IACtC,IAAI,OAAO,IAAI,MAAM,EAAE,CAAC;QACtB,EAAE,CAAC,MAAM,CAAC,sBAAsB,MAAM,CAAC,KAAK,IAAI,CAAC,CAAC;QAClD,OAAO,CAAC,CAAC;IACX,CAAC;IACD,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAC1B,IAAI,YAAsB,CAAC;IAC3B,IAAI,MAAM,CAAC,GAAG,EAAE,CAAC;QACf,sEAAsE;QACtE,yEAAyE;QACzE,0EAA0E;QAC1E,sEAAsE;QACtE,8CAA8C;QAC9C,IAAI,MAAM,CAAC,SAAS,KAAK,SAAS,EAAE,CAAC;YACnC,EAAE,CAAC,MAAM,CACP,2PAA2P,CAC5P,CAAC;YACF,OAAO,CAAC,CAAC;QACX,CAAC;QACD,MAAM,MAAM,GAAG,qBAAqB,CAAC,GAAG,CAAC,CAAC;QAC1C,MAAM,OAAO,GAAG,iBAAiB,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;QACjD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACzB,EAAE,CAAC,MAAM,CACP,oEAAoE,MAAM,CAAC,UAAU,KAAK,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,6BAA6B,CAAC,CAAC,CAAC,SAAS,MAAM,CAC7J,CAAC;YACF,OAAO,CAAC,CAAC;QACX,CAAC;QACD,MAAM,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC;QACvE,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACvB,EAAE,CAAC,MAAM,CACP,gEAAgE,MAAM,CAAC,UAAU,KAAK,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,mBAAmB,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CACxJ,CAAC;YACF,OAAO,CAAC,CAAC;QACX,CAAC;QACD,YAAY,GAAG,OAAO,CAAC;IACzB,CAAC;SAAM,IAAI,MAAM,CAAC,SAAS,KAAK,SAAS,EAAE,CAAC;QAC1C,YAAY,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;IACpC,CAAC;SAAM,CAAC;QACN,oDAAoD;QACpD,EAAE,CAAC,MAAM,CAAC,sCAAsC,CAAC,CAAC;QAClD,OAAO,CAAC,CAAC;IACX,CAAC;IAED,MAAM,OAAO,GAAoB,EAAE,CAAC;IACpC,IAAI,QAAQ,GAAG,CAAC,CAAC;IACjB,KAAK,MAAM,IAAI,IAAI,YAAY,EAAE,CAAC;QAChC,IAAI,CAAC;YACH,MAAM,IAAI,GAAuC;gBAC/C,GAAG;gBACH,SAAS,EAAE,IAAI;gBACf,KAAK,EAAE,MAAM,CAAC,KAAK;gBACnB,GAAG,CAAC,MAAM,CAAC,SAAS,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,MAAM,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;aAC3E,CAAC;YACF,MAAM,CAAC,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC;YAC7B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAChB,sEAAsE;YACtE,mEAAmE;YACnE,IAAI,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,SAAS,CAAC,EAAE,CAAC;gBAChD,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC;YACnC,CAAC;QACH,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,EAAE,CAAC,MAAM,CAAC,sBAAuB,GAAa,CAAC,OAAO,IAAI,CAAC,CAAC;YAC5D,OAAO,CAAC,CAAC;QACX,CAAC;IACH,CAAC;IAED,IAAI,MAAM,CAAC,IAAI,EAAE,CAAC;QAChB,EAAE,CAAC,MAAM,CAAC,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC;IACpE,CAAC;SAAM,CAAC;QACN,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;YACxB,EAAE,CAAC,MAAM,CAAC,GAAG,wBAAwB,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAChD,CAAC;QACD,IAAI,QAAQ,KAAK,CAAC,EAAE,CAAC;YACnB,EAAE,CAAC,MAAM,CACP,2GAA2G,CAC5G,CAAC;QACJ,CAAC;IACH,CAAC;IACD,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,aAAa,CAAC,IAAc,EAAE,EAAY;IACjD,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACnD,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QACpB,OAAO,CAAC,CAAC;IACX,CAAC;IACD,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IACrC,MAAM,KAAK,GAAG,iBAAiB,EAAE,CAAC;IAClC,IAAI,IAAI,EAAE,CAAC;QACT,EAAE,CAAC,MAAM,CAAC,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC;QAC7D,OAAO,CAAC,CAAC;IACX,CAAC;IACD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvB,EAAE,CAAC,MAAM,CAAC,sCAAsC,CAAC,CAAC;QAClD,OAAO,CAAC,CAAC;IACX,CAAC;IACD,EAAE,CAAC,MAAM,CAAC,oBAAoB,KAAK,CAAC,MAAM,qBAAqB,CAAC,CAAC;IACjE,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,EAAE,CAAC,MAAM,CAAC,OAAO,IAAI,CAAC,IAAI,MAAM,IAAI,CAAC,OAAO,GAAG,CAAC,CAAC;QACjD,IAAI,IAAI,CAAC,eAAe;YAAE,EAAE,CAAC,MAAM,CAAC,aAAa,IAAI,CAAC,eAAe,EAAE,CAAC,CAAC;QACzE,EAAE,CAAC,MAAM,CAAC,SAAS,IAAI,CAAC,OAAO,IAAI,CAAC,CAAC;IACvC,CAAC;IACD,OAAO,CAAC,CAAC;AACX,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,SAAS,CAAC,IAAc;IAC5C,MAAM,EAAE,GAAa;QACnB,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;QACtC,MAAM,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;KACvC,CAAC;IACF,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,QAAQ,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAClE,EAAE,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC;QACpB,OAAO,CAAC,CAAC;IACX,CAAC;IACD,MAAM,GAAG,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;IACpB,MAAM,OAAO,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;IAC9B,IAAI,GAAG,KAAK,SAAS,EAAE,CAAC;QACtB,OAAO,MAAM,gBAAgB,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IAC7C,CAAC;IACD,IAAI,GAAG,KAAK,MAAM,EAAE,CAAC;QACnB,OAAO,aAAa,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;IACpC,CAAC;IACD,EAAE,CAAC,MAAM,CAAC,kCAAkC,GAAG,gCAAgC,CAAC,CAAC;IACjF,OAAO,CAAC,CAAC;AACX,CAAC"}
|
|
@@ -134,7 +134,17 @@ def _resolve_repo_root() -> Path:
|
|
|
134
134
|
|
|
135
135
|
|
|
136
136
|
REPO_ROOT = _resolve_repo_root()
|
|
137
|
+
# Cycle-doc lookup paths in priority order. Legacy `docs/roadmap/cycles/` is
|
|
138
|
+
# tried first for backward compatibility with sage3c / dark-factory-platform;
|
|
139
|
+
# `docs/cycles/` is the newer dark-factory-dashboard convention (PR #105
|
|
140
|
+
# moved it there to align with the eventual upstream convention). Consumers
|
|
141
|
+
# that have moved to the new path get a working validator without
|
|
142
|
+
# configuration; consumers still on the legacy path are unchanged.
|
|
143
|
+
CYCLE_DOC_DIRS: tuple[str, ...] = ("docs/roadmap/cycles", "docs/cycles")
|
|
137
144
|
CYCLES_DIR = REPO_ROOT / "docs" / "roadmap" / "cycles"
|
|
145
|
+
# Glob and the singular `CYCLES_DIR` retained for backward compatibility with
|
|
146
|
+
# downstream tooling that imports them. New code should prefer `CYCLE_DOC_DIRS`
|
|
147
|
+
# + `_is_cycle_doc_path()` which check both layouts.
|
|
138
148
|
CYCLE_DOC_GLOB = "docs/roadmap/cycles/cycle*.md"
|
|
139
149
|
|
|
140
150
|
# Trailers we recognize. ``git-interpret-trailers`` defines a trailer as
|
|
@@ -238,14 +248,27 @@ def normalize_cycle_id(raw: str) -> str | None:
|
|
|
238
248
|
|
|
239
249
|
|
|
240
250
|
def find_cycle_doc(cycle_id: str) -> CycleDoc | None:
|
|
241
|
-
"""Locate
|
|
242
|
-
|
|
243
|
-
|
|
244
|
-
)
|
|
251
|
+
"""Locate ``<cycle_dir>/cycle<id>*.md`` and read its status.
|
|
252
|
+
|
|
253
|
+
Searches every candidate dir in ``CYCLE_DOC_DIRS`` (legacy
|
|
254
|
+
``docs/roadmap/cycles/`` first, then ``docs/cycles/``) so consumers
|
|
255
|
+
that have moved get a working lookup without configuration. If the
|
|
256
|
+
same cycle exists in both dirs (transitional state), prefers the
|
|
257
|
+
longer-named match across all hits — same disambiguation rule as
|
|
258
|
+
before, just over a wider candidate set.
|
|
259
|
+
"""
|
|
260
|
+
matches: list[Path] = []
|
|
261
|
+
for cycle_dir in CYCLE_DOC_DIRS:
|
|
262
|
+
dir_path = REPO_ROOT / cycle_dir
|
|
263
|
+
if not dir_path.exists():
|
|
264
|
+
continue
|
|
265
|
+
matches.extend(sorted(dir_path.glob(f"cycle{cycle_id}-*.md")))
|
|
266
|
+
matches.extend(sorted(dir_path.glob(f"cycle{cycle_id}.md")))
|
|
245
267
|
if not matches:
|
|
246
268
|
return None
|
|
247
|
-
# If multiple matches exist (legacy ``cycle318.md`` + new ``cycle318-foo.md
|
|
248
|
-
# prefer the one with the longer name (more
|
|
269
|
+
# If multiple matches exist (legacy ``cycle318.md`` + new ``cycle318-foo.md``,
|
|
270
|
+
# or one in each layout), prefer the one with the longer name (more
|
|
271
|
+
# descriptive slug).
|
|
249
272
|
path = max(matches, key=lambda p: len(p.name))
|
|
250
273
|
status, superseded_by = read_cycle_frontmatter(path)
|
|
251
274
|
return CycleDoc(path=path, status=status, superseded_by=superseded_by)
|
|
@@ -332,50 +355,89 @@ def base_cycle_doc(
|
|
|
332
355
|
if gh_token:
|
|
333
356
|
env["GH_TOKEN"] = gh_token
|
|
334
357
|
|
|
335
|
-
|
|
336
|
-
|
|
337
|
-
|
|
338
|
-
|
|
339
|
-
|
|
340
|
-
|
|
341
|
-
|
|
342
|
-
|
|
343
|
-
|
|
344
|
-
|
|
345
|
-
|
|
346
|
-
|
|
358
|
+
# Collect candidates across EVERY candidate cycle-doc dir on the base
|
|
359
|
+
# ref — DON'T break on the first successful listing. A consumer can
|
|
360
|
+
# have BOTH layouts present (transitional state — e.g. dark-factory-
|
|
361
|
+
# dashboard pre-#105 had cycle1-5 at legacy + cycle6 at the new path
|
|
362
|
+
# for one PR's worth of overlap), and a legacy dir that exists but
|
|
363
|
+
# lacks the cited cycle (sage3c with cycle331.x at legacy + a new
|
|
364
|
+
# cycle 6 at the new path) must fall through to the next dir. Per-dir
|
|
365
|
+
# 404 is the normal "consumer has not moved (or has moved fully) to
|
|
366
|
+
# this layout" signal; only fail if EVERY dir 404s or any dir hits a
|
|
367
|
+
# non-404 error.
|
|
368
|
+
pattern = re.compile(rf"^cycle{re.escape(cycle_id)}(?:-.+)?\.md$")
|
|
369
|
+
candidates: list[dict] = []
|
|
370
|
+
dirs_listed: list[str] = []
|
|
371
|
+
last_404: BaseCycleDocFetchError | None = None
|
|
372
|
+
for cycle_dir in CYCLE_DOC_DIRS:
|
|
373
|
+
try:
|
|
374
|
+
listing_result = subprocess.run(
|
|
375
|
+
[
|
|
376
|
+
"gh",
|
|
377
|
+
"api",
|
|
378
|
+
f"repos/{repo}/contents/{cycle_dir}?ref={base_ref}",
|
|
379
|
+
],
|
|
380
|
+
check=True,
|
|
381
|
+
capture_output=True,
|
|
382
|
+
text=True,
|
|
383
|
+
env=env,
|
|
384
|
+
timeout=60,
|
|
385
|
+
)
|
|
386
|
+
except FileNotFoundError as exc:
|
|
387
|
+
# gh CLI itself missing — environmental, fail loud regardless
|
|
388
|
+
# of which cycle_dir we were probing.
|
|
389
|
+
raise BaseCycleDocFetchError(
|
|
390
|
+
"gh CLI not on PATH; cannot read cycle docs from base ref."
|
|
391
|
+
) from exc
|
|
392
|
+
except subprocess.CalledProcessError as exc:
|
|
393
|
+
# Treat 404 (or "not found") as "this dir doesn't exist on
|
|
394
|
+
# base ref; keep trying the others". Any other non-zero exit
|
|
395
|
+
# (auth, rate-limit, server error) escapes immediately —
|
|
396
|
+
# don't silently swallow a real failure as a missing-dir.
|
|
397
|
+
if _is_gh_not_found_error(exc):
|
|
398
|
+
last_404 = BaseCycleDocFetchError(
|
|
399
|
+
f"gh api repos/{repo}/contents/{cycle_dir} returned 404 "
|
|
400
|
+
f"(exit {exc.returncode}): {(exc.stderr or '').strip()}"
|
|
401
|
+
)
|
|
402
|
+
continue
|
|
403
|
+
raise BaseCycleDocFetchError(
|
|
404
|
+
f"gh api repos/{repo}/contents/{cycle_dir} failed "
|
|
405
|
+
f"(exit {exc.returncode}): {(exc.stderr or '').strip()}"
|
|
406
|
+
) from exc
|
|
407
|
+
except subprocess.TimeoutExpired as exc:
|
|
408
|
+
raise BaseCycleDocFetchError(
|
|
409
|
+
f"gh api repos/{repo}/contents/{cycle_dir} timed out after {exc.timeout}s"
|
|
410
|
+
) from exc
|
|
411
|
+
|
|
412
|
+
dirs_listed.append(cycle_dir)
|
|
413
|
+
try:
|
|
414
|
+
entries = json.loads(listing_result.stdout)
|
|
415
|
+
except json.JSONDecodeError as exc:
|
|
416
|
+
raise BaseCycleDocFetchError(
|
|
417
|
+
f"GitHub contents API returned malformed JSON for {cycle_dir} listing."
|
|
418
|
+
) from exc
|
|
419
|
+
|
|
420
|
+
candidates.extend(
|
|
421
|
+
entry
|
|
422
|
+
for entry in entries
|
|
423
|
+
if isinstance(entry, dict)
|
|
424
|
+
and entry.get("type") == "file"
|
|
425
|
+
and isinstance(entry.get("name"), str)
|
|
426
|
+
and pattern.match(entry["name"])
|
|
347
427
|
)
|
|
348
|
-
except FileNotFoundError as exc:
|
|
349
|
-
raise BaseCycleDocFetchError(
|
|
350
|
-
"gh CLI not on PATH; cannot read cycle docs from base ref."
|
|
351
|
-
) from exc
|
|
352
|
-
except subprocess.CalledProcessError as exc:
|
|
353
|
-
raise BaseCycleDocFetchError(
|
|
354
|
-
f"gh api repos/{repo}/contents/docs/roadmap/cycles failed "
|
|
355
|
-
f"(exit {exc.returncode}): {exc.stderr.strip()}"
|
|
356
|
-
) from exc
|
|
357
|
-
except subprocess.TimeoutExpired as exc:
|
|
358
|
-
raise BaseCycleDocFetchError(
|
|
359
|
-
f"gh api repos/{repo}/contents/docs/roadmap/cycles timed out after {exc.timeout}s"
|
|
360
|
-
) from exc
|
|
361
428
|
|
|
362
|
-
|
|
363
|
-
|
|
364
|
-
|
|
365
|
-
|
|
366
|
-
|
|
367
|
-
|
|
429
|
+
if not dirs_listed:
|
|
430
|
+
# EVERY candidate dir 404'd — no cycle docs anywhere on base ref.
|
|
431
|
+
# Surface the last 404 so consumers see a concrete actionable error
|
|
432
|
+
# naming what was tried.
|
|
433
|
+
raise last_404 or BaseCycleDocFetchError(
|
|
434
|
+
"no candidate cycle-doc directory found on base ref "
|
|
435
|
+
f"(tried: {', '.join(CYCLE_DOC_DIRS)})"
|
|
436
|
+
)
|
|
368
437
|
|
|
369
|
-
pattern = re.compile(rf"^cycle{re.escape(cycle_id)}(?:-.+)?\.md$")
|
|
370
|
-
candidates = [
|
|
371
|
-
entry
|
|
372
|
-
for entry in entries
|
|
373
|
-
if isinstance(entry, dict)
|
|
374
|
-
and entry.get("type") == "file"
|
|
375
|
-
and isinstance(entry.get("name"), str)
|
|
376
|
-
and pattern.match(entry["name"])
|
|
377
|
-
]
|
|
378
438
|
if not candidates:
|
|
439
|
+
# At least one dir existed but doesn't hold the cited cycle —
|
|
440
|
+
# treat as "new cycle, OK to proceed" (the doc lands in this PR).
|
|
379
441
|
return None
|
|
380
442
|
|
|
381
443
|
chosen = max(candidates, key=lambda entry: len(entry["name"]))
|
|
@@ -657,18 +719,22 @@ def _parse_paginated_commit_messages(raw: str) -> str:
|
|
|
657
719
|
|
|
658
720
|
|
|
659
721
|
def _is_cycle_doc_path(path: str) -> bool:
|
|
660
|
-
"""Match
|
|
722
|
+
"""Match ``<cycle_dir>/cycle<id>(-<slug>)?.md`` under any layout.
|
|
661
723
|
|
|
662
|
-
|
|
663
|
-
``
|
|
664
|
-
``
|
|
665
|
-
|
|
724
|
+
Accepts both legacy ``docs/roadmap/cycles/`` and the newer
|
|
725
|
+
``docs/cycles/`` per ``CYCLE_DOC_DIRS``. The classifier counts only
|
|
726
|
+
files whose basename starts with ``cycle`` followed by an id
|
|
727
|
+
character. Sibling files like ``<cycle_dir>/README.md`` are
|
|
728
|
+
index/meta content and do NOT trigger the plan-PR contract.
|
|
666
729
|
"""
|
|
667
|
-
|
|
668
|
-
if not path.startswith(prefix) or not path.endswith(".md"):
|
|
730
|
+
if not path.endswith(".md"):
|
|
669
731
|
return False
|
|
670
|
-
|
|
671
|
-
|
|
732
|
+
for cycle_dir in CYCLE_DOC_DIRS:
|
|
733
|
+
prefix = f"{cycle_dir}/cycle"
|
|
734
|
+
if path.startswith(prefix):
|
|
735
|
+
next_char = path[len(prefix) : len(prefix) + 1]
|
|
736
|
+
return next_char.isdigit()
|
|
737
|
+
return False
|
|
672
738
|
|
|
673
739
|
|
|
674
740
|
def is_plan_pr(
|
|
@@ -731,7 +797,9 @@ def status_completion_in_diff(diff: str) -> list[str]:
|
|
|
731
797
|
in_frontmatter = False
|
|
732
798
|
seen_completed_in_frontmatter = False
|
|
733
799
|
continue
|
|
734
|
-
if not current_file or not
|
|
800
|
+
if not current_file or not any(
|
|
801
|
+
current_file.startswith(f"{d}/") for d in CYCLE_DOC_DIRS
|
|
802
|
+
):
|
|
735
803
|
continue
|
|
736
804
|
# Naive frontmatter tracking inside the diff hunk: ``---`` delimiters
|
|
737
805
|
# toggle the flag. False positives are acceptable here — the goal
|
|
@@ -789,7 +857,7 @@ def cycle_docs_transitioned_to_terminal(
|
|
|
789
857
|
cycle_paths = [
|
|
790
858
|
p
|
|
791
859
|
for p in changed_files
|
|
792
|
-
if p.startswith("
|
|
860
|
+
if any(p.startswith(f"{d}/cycle") for d in CYCLE_DOC_DIRS) and p.endswith(".md")
|
|
793
861
|
]
|
|
794
862
|
if not cycle_paths:
|
|
795
863
|
return []
|
|
@@ -1030,8 +1098,9 @@ def validate(
|
|
|
1030
1098
|
if doc is None:
|
|
1031
1099
|
errors.append(
|
|
1032
1100
|
f"[cycle-doc] FAIL ({pr_type}): cycle `{cycle_id}` not found in "
|
|
1033
|
-
f"
|
|
1034
|
-
"PR that creates the
|
|
1101
|
+
f"any of: {', '.join(f'`{d}/`' for d in CYCLE_DOC_DIRS)}. "
|
|
1102
|
+
"Either fix the trailer or open a plan PR that creates the "
|
|
1103
|
+
"cycle doc first."
|
|
1035
1104
|
)
|
|
1036
1105
|
return errors
|
|
1037
1106
|
|
|
@@ -1056,7 +1125,9 @@ def validate(
|
|
|
1056
1125
|
# PR that doesn't touch its cycle's doc is mis-labelled.
|
|
1057
1126
|
assert doc is not None # plan PR without cycle_id already returned
|
|
1058
1127
|
cycle_paths = [
|
|
1059
|
-
f
|
|
1128
|
+
f
|
|
1129
|
+
for f in changed_files
|
|
1130
|
+
if any(f.startswith(f"{d}/") for d in CYCLE_DOC_DIRS)
|
|
1060
1131
|
]
|
|
1061
1132
|
doc_relative = doc.path.relative_to(REPO_ROOT).as_posix()
|
|
1062
1133
|
if doc_relative not in cycle_paths:
|
package/dist/doctor.d.ts
CHANGED
|
@@ -9,6 +9,23 @@ export interface DoctorOptions {
|
|
|
9
9
|
profileName?: string;
|
|
10
10
|
}
|
|
11
11
|
export declare function runDoctor(options: DoctorOptions): Promise<DoctorCheck[]>;
|
|
12
|
+
export declare const CLOUD_ENV_MARKERS: readonly ["CODESPACES", "REMOTE_CONTAINERS", "CLAUDE_CODE_SANDBOX", "DEVCONTAINER"];
|
|
13
|
+
export type CloudEnvMarker = (typeof CLOUD_ENV_MARKERS)[number];
|
|
14
|
+
export interface CloudEnvDetection {
|
|
15
|
+
detected: boolean;
|
|
16
|
+
/**
|
|
17
|
+
* The subset of `CLOUD_ENV_MARKERS` whose values were truthy in
|
|
18
|
+
* `process.env` at detection time (each one of "true", "1", "yes",
|
|
19
|
+
* case-insensitive; presence-only markers like a non-empty
|
|
20
|
+
* `CODESPACE_NAME` are NOT considered — keep the contract explicit
|
|
21
|
+
* boolean-ish to avoid false positives from generic env-set state).
|
|
22
|
+
*/
|
|
23
|
+
markers: CloudEnvMarker[];
|
|
24
|
+
}
|
|
25
|
+
export interface DetectCloudEnvOptions {
|
|
26
|
+
env?: NodeJS.ProcessEnv;
|
|
27
|
+
}
|
|
28
|
+
export declare function detectCloudEnv(options?: DetectCloudEnvOptions): CloudEnvDetection;
|
|
12
29
|
export type DoctorTriageState = "config_missing" | "auth_pending" | "ok";
|
|
13
30
|
export interface ClassifyDoctorStateOptions {
|
|
14
31
|
config: AgentReviewConfig;
|
package/dist/doctor.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"doctor.d.ts","sourceRoot":"","sources":["../src/doctor.ts"],"names":[],"mappings":"AA0CA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AACvD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAM9D,OAAO,KAAK,EACV,iBAAiB,EACjB,WAAW,EACZ,MAAM,gCAAgC,CAAC;AAIxC,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,YAAY,CAAC;IACrB,QAAQ,EAAE,eAAe,CAAC;IAK1B,SAAS,CAAC,EAAE,eAAe,CAAC;IAO5B,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,wBAAsB,SAAS,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,
|
|
1
|
+
{"version":3,"file":"doctor.d.ts","sourceRoot":"","sources":["../src/doctor.ts"],"names":[],"mappings":"AA0CA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AACvD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAM9D,OAAO,KAAK,EACV,iBAAiB,EACjB,WAAW,EACZ,MAAM,gCAAgC,CAAC;AAIxC,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,YAAY,CAAC;IACrB,QAAQ,EAAE,eAAe,CAAC;IAK1B,SAAS,CAAC,EAAE,eAAe,CAAC;IAO5B,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,wBAAsB,SAAS,CAAC,OAAO,EAAE,aAAa,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CA4L9E;AAuBD,eAAO,MAAM,iBAAiB,qFAKpB,CAAC;AAEX,MAAM,MAAM,cAAc,GAAG,CAAC,OAAO,iBAAiB,CAAC,CAAC,MAAM,CAAC,CAAC;AAEhE,MAAM,WAAW,iBAAiB;IAChC,QAAQ,EAAE,OAAO,CAAC;IAClB;;;;;;OAMG;IACH,OAAO,EAAE,cAAc,EAAE,CAAC;CAC3B;AAED,MAAM,WAAW,qBAAqB;IACpC,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;CACzB;AAED,wBAAgB,cAAc,CAC5B,OAAO,GAAE,qBAA0B,GAClC,iBAAiB,CAYnB;AAiPD,MAAM,MAAM,iBAAiB,GACzB,gBAAgB,GAChB,cAAc,GACd,IAAI,CAAC;AAET,MAAM,WAAW,0BAA0B;IACzC,MAAM,EAAE,iBAAiB,CAAC;IAC1B;;;;;OAKG;IACH,WAAW,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC;;;;;;OAMG;IACH,eAAe,EAAE,SAAS,WAAW,EAAE,CAAC;CACzC;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,iBAAiB,CAAC;IACzB;;;;;OAKG;IACH,IAAI,EAAE,MAAM,CAAC;CACd;AAED,wBAAgB,mBAAmB,CACjC,OAAO,EAAE,0BAA0B,GAClC,kBAAkB,CAiDpB;AA0BD,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,WAAW,CAwDjE;AAoED,eAAO,MAAM,2BAA2B,QACQ,CAAC;AA8GjD,wBAAsB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC,CA6F3E"}
|