@lifeaitools/rdc-skills 0.24.38 → 0.24.39
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/.claude/settings.json +15 -15
- package/.claude-plugin/marketplace.json +21 -21
- package/.claude-plugin/plugin.json +1371 -1371
- package/.github/workflows/publish.yml +34 -34
- package/.github/workflows/self-test.yml +58 -58
- package/CHANGELOG.md +310 -310
- package/LICENSE +21 -21
- package/MANIFEST.md +221 -221
- package/README.md +377 -377
- package/README.sandbox.md +3 -3
- package/assets/watcher/viewer.html +164 -164
- package/bin/rdc-skills-mcp.mjs +316 -316
- package/commands/build.md +183 -183
- package/commands/collab.md +180 -180
- package/commands/deploy.md +152 -152
- package/commands/design.md +31 -31
- package/commands/edit.md +28 -28
- package/commands/fixit.md +124 -124
- package/commands/handoff.md +173 -173
- package/commands/help.md +95 -95
- package/commands/overnight.md +220 -220
- package/commands/plan.md +158 -158
- package/commands/preplan.md +131 -131
- package/commands/prototype.md +145 -145
- package/commands/release.md +49 -49
- package/commands/report.md +99 -99
- package/commands/review.md +120 -120
- package/commands/self-test.md +113 -113
- package/commands/status.md +86 -86
- package/commands/watch.md +98 -98
- package/commands/workitems.md +137 -137
- package/git-sha.json +1 -1
- package/guides/agent-bootstrap.md +295 -295
- package/guides/agents/backend.md +104 -104
- package/guides/agents/content.md +94 -94
- package/guides/agents/cs2.md +56 -56
- package/guides/agents/data.md +87 -87
- package/guides/agents/design.md +77 -77
- package/guides/agents/frontend.md +92 -92
- package/guides/agents/infrastructure.md +81 -81
- package/guides/agents/setup.md +281 -281
- package/guides/agents/verify.md +151 -151
- package/guides/agents/viz.md +106 -106
- package/guides/backend.md +146 -146
- package/guides/content.md +147 -147
- package/guides/cs2.md +190 -190
- package/guides/data.md +123 -123
- package/guides/design.md +116 -116
- package/guides/engineering-behavior.md +43 -43
- package/guides/escalation-protocol.md +125 -125
- package/guides/frontend.md +151 -151
- package/guides/history-md-spec.md +297 -297
- package/guides/infrastructure.md +179 -179
- package/guides/lessons-learned-spec.md +153 -153
- package/guides/output-contract.md +108 -108
- package/guides/publish-md-spec.md +289 -289
- package/guides/rdc-skills-startup.md +30 -30
- package/guides/verify.md +11 -11
- package/hooks/check-cwd.js +31 -31
- package/hooks/check-rdc-environment.js +164 -164
- package/hooks/check-services.js +6 -6
- package/hooks/check-stale-work-items.js +19 -19
- package/hooks/foreground-process-gate.js +128 -128
- package/hooks/gate-watchdog-selfcheck.js +257 -257
- package/hooks/hook-logger.js +25 -25
- package/hooks/lib/run-evidence-gate.mjs +241 -241
- package/hooks/no-stop-open-epics.js +127 -127
- package/hooks/post-tool-batch-gate.js +203 -203
- package/hooks/post-work-check.js +21 -21
- package/hooks/postcompact-log.js +13 -13
- package/hooks/precompact-log.js +13 -13
- package/hooks/rate-limit-retry.js +46 -46
- package/hooks/rdc-invocation-marker.js +157 -157
- package/hooks/rdc-output-contract-gate.js +94 -94
- package/hooks/require-work-item-on-commit.js +294 -294
- package/hooks/restart-brief.js +19 -19
- package/hooks/run-hidden-hook.ps1 +47 -47
- package/hooks/task-completed-gate.js +274 -274
- package/hooks/work-item-exit-gate.js +944 -944
- package/lib/catalog.mjs +236 -236
- package/lib/cloud-rewrite.mjs +155 -155
- package/package.json +56 -56
- package/rules/work-items-rpc.md +520 -520
- package/scaffold/templates/HISTORY.md.template +39 -39
- package/scaffold/templates/PUBLISH.md.template +21 -21
- package/scaffold/templates/brochure-studio-default.html +70 -70
- package/scripts/acceptance.mjs +502 -502
- package/scripts/fixtures/guides/bad-guide.md +15 -15
- package/scripts/fixtures/guides-clean/good-guide.md +16 -16
- package/scripts/install-rdc-skills.js +1289 -1289
- package/scripts/install.ps1 +202 -202
- package/scripts/install.sh +132 -132
- package/scripts/lib/assertions.mjs +287 -287
- package/scripts/lib/manifest-schema.mjs +754 -754
- package/scripts/lib/runner.mjs +465 -465
- package/scripts/lib/sandbox.mjs +435 -435
- package/scripts/prepack.mjs +32 -32
- package/scripts/rdc-brochure.mjs +464 -464
- package/scripts/rdc-design-cli.mjs +134 -134
- package/scripts/rebuild-mcp.mjs +107 -107
- package/scripts/self-test.mjs +1460 -1460
- package/scripts/stamp-git-sha.mjs +29 -29
- package/scripts/test-guide-validator.mjs +196 -196
- package/scripts/test-rdc-hooks.mjs +145 -145
- package/scripts/uninstall.ps1 +77 -77
- package/scripts/uninstall.sh +69 -69
- package/scripts/update.ps1 +43 -43
- package/scripts/update.sh +43 -43
- package/scripts/validate-place-histories.js +461 -461
- package/scripts/validate-publish-manifests.js +424 -424
- package/scripts/watch-init.mjs +100 -100
- package/skills/brochure/SKILL.md +107 -107
- package/skills/build/SKILL.md +563 -563
- package/skills/channel-formatter/SKILL.md +533 -533
- package/skills/co-develop/SKILL.md +196 -196
- package/skills/collab/SKILL.md +239 -239
- package/skills/convert/SKILL.md +140 -140
- package/skills/deploy/SKILL.md +541 -541
- package/skills/design/SKILL.md +211 -211
- package/skills/design/reference/ownership.md +16 -16
- package/skills/design/reference/rampa.md +92 -92
- package/skills/design/reference/studio-model.md +153 -153
- package/skills/edit/SKILL.md +98 -98
- package/skills/fixit/SKILL.md +165 -165
- package/skills/fs-mcp/SKILL.md +148 -148
- package/skills/handoff/SKILL.md +236 -200
- package/skills/help/SKILL.md +143 -143
- package/skills/housekeeping/SKILL.md +189 -189
- package/skills/lifeai-brochure-author/SKILL.md +340 -340
- package/skills/overnight/SKILL.md +251 -251
- package/skills/plan/SKILL.md +345 -345
- package/skills/preplan/SKILL.md +90 -90
- package/skills/prototype/SKILL.md +150 -150
- package/skills/rdc-brochurify/SKILL.md +245 -245
- package/skills/rdc-extract-verifier-rules/SKILL.md +191 -191
- package/skills/release/SKILL.md +140 -140
- package/skills/report/SKILL.md +100 -100
- package/skills/review/SKILL.md +152 -152
- package/skills/rpms-filemap/SKILL.cloud.md +111 -111
- package/skills/rpms-filemap/SKILL.md +111 -111
- package/skills/self-test/SKILL.md +132 -132
- package/skills/status/SKILL.md +99 -99
- package/skills/terminal-config/SKILL.md +62 -62
- package/skills/tests/MATRIX.md +54 -54
- package/skills/tests/README.md +47 -47
- package/skills/tests/rdc-brochure.test.json +34 -34
- package/skills/tests/rdc-build.test.json +36 -36
- package/skills/tests/rdc-channel-formatter.test.json +45 -45
- package/skills/tests/rdc-co-develop.test.json +29 -29
- package/skills/tests/rdc-collab.test.json +29 -29
- package/skills/tests/rdc-convert.test.json +35 -35
- package/skills/tests/rdc-deploy.test.json +30 -30
- package/skills/tests/rdc-design.test.json +27 -27
- package/skills/tests/rdc-edit.test.json +29 -29
- package/skills/tests/rdc-fixit.test.json +36 -36
- package/skills/tests/rdc-fs-mcp.test.json +36 -36
- package/skills/tests/rdc-handoff.test.json +28 -28
- package/skills/tests/rdc-help.test.json +29 -29
- package/skills/tests/rdc-housekeeping.test.json +31 -31
- package/skills/tests/rdc-lifeai-brochure-author.test.json +35 -35
- package/skills/tests/rdc-overnight.test.json +37 -37
- package/skills/tests/rdc-plan.test.json +27 -27
- package/skills/tests/rdc-preplan.test.json +31 -31
- package/skills/tests/rdc-prototype.test.json +28 -28
- package/skills/tests/rdc-rdc-brochurify.test.json +23 -23
- package/skills/tests/rdc-rdc-extract-verifier-rules.test.json +34 -34
- package/skills/tests/rdc-release.test.json +29 -29
- package/skills/tests/rdc-report.test.json +28 -28
- package/skills/tests/rdc-review.test.json +29 -29
- package/skills/tests/rdc-rpms-filemap.test.json +28 -28
- package/skills/tests/rdc-self-test.test.json +24 -24
- package/skills/tests/rdc-status.test.json +29 -29
- package/skills/tests/rdc-terminal-config.test.json +29 -29
- package/skills/tests/rdc-watch.test.json +24 -24
- package/skills/tests/rdc-workitems.test.json +27 -27
- package/skills/watch/SKILL.md +97 -97
- package/skills/workitems/SKILL.md +151 -151
- package/tests/acceptance.test.mjs +59 -59
- package/tests/channel-formatter.contract.test.mjs +251 -251
- package/tests/curl-surface.test.mjs +289 -289
- package/tests/harness-gates.test.mjs +325 -325
- package/tests/help-surface.test.mjs +61 -61
- package/tests/install-rdc-skills.test.mjs +49 -49
- package/tests/manifest-contract-fields.test.mjs +78 -78
- package/tests/mcp.test.mjs +271 -271
- package/tests/require-work-item-on-commit.test.mjs +162 -162
- package/tests/run-evidence-gate.test.mjs +82 -82
- package/tests/skill-test-matrix.test.mjs +66 -66
- package/tests/validate-skills.js +27 -27
- package/tests/work-item-exit-gate-l2.test.mjs +368 -368
- package/tests/work-item-exit-gate-l3.test.mjs +197 -197
package/scripts/lib/runner.mjs
CHANGED
|
@@ -1,465 +1,465 @@
|
|
|
1
|
-
// runner.mjs — Tier 2 per-skill test orchestration.
|
|
2
|
-
//
|
|
3
|
-
// Public API:
|
|
4
|
-
// runManifest(manifest, { runId, supabaseBranchRef?, timeout?, engine?, claudeBin?, codexBin? })
|
|
5
|
-
// → { skill, pass, duration_ms, observed?, failures?, error? }
|
|
6
|
-
//
|
|
7
|
-
// Flow per manifest:
|
|
8
|
-
// 1. Create a worktree at .rdc-sandbox/<runId>/<skill-slug>/
|
|
9
|
-
// 2. Write precondition_files into the worktree
|
|
10
|
-
// 3. Snapshot HEAD sha + work_items rowcount (via Supabase branch if given)
|
|
11
|
-
// 4. Spawn the selected engine in headless JSONL mode:
|
|
12
|
-
// - claude: `claude --print "<prompt>" --output-format stream-json`
|
|
13
|
-
// - codex: `codex exec --json "<prompt>"`
|
|
14
|
-
// - cwd = worktree
|
|
15
|
-
// - env = { ...process.env, ...fixture.env, RDC_TEST: "1" }
|
|
16
|
-
// - timeout kills the child on overrun
|
|
17
|
-
// 5. Compute observed delta: commits since snapshot, files changed, work_items delta
|
|
18
|
-
// 6. Call evaluateAssertions
|
|
19
|
-
// 7. Return structured result — never throws (caller-friendly).
|
|
20
|
-
//
|
|
21
|
-
// Cleanup is the caller's responsibility (runner does NOT remove the worktree,
|
|
22
|
-
// so the caller can inspect on failure before teardown).
|
|
23
|
-
|
|
24
|
-
import { spawn, execFileSync } from "node:child_process";
|
|
25
|
-
import { writeFileSync, mkdirSync, existsSync } from "node:fs";
|
|
26
|
-
import { join, dirname, resolve } from "node:path";
|
|
27
|
-
import { fileURLToPath } from "node:url";
|
|
28
|
-
import { createWorktree } from "./sandbox.mjs";
|
|
29
|
-
import { evaluateAssertions } from "./assertions.mjs";
|
|
30
|
-
|
|
31
|
-
const DEFAULT_TIMEOUT_MS = 240_000;
|
|
32
|
-
|
|
33
|
-
function nowMs() {
|
|
34
|
-
return Date.now();
|
|
35
|
-
}
|
|
36
|
-
|
|
37
|
-
/** Run a git command inside `cwd`, return trimmed stdout or "". Never throws. */
|
|
38
|
-
function gitIn(cwd, args) {
|
|
39
|
-
try {
|
|
40
|
-
return execFileSync("git", args, {
|
|
41
|
-
cwd,
|
|
42
|
-
encoding: "utf8",
|
|
43
|
-
stdio: ["ignore", "pipe", "pipe"],
|
|
44
|
-
}).trim();
|
|
45
|
-
} catch {
|
|
46
|
-
return "";
|
|
47
|
-
}
|
|
48
|
-
}
|
|
49
|
-
|
|
50
|
-
function snapshotHead(cwd) {
|
|
51
|
-
return gitIn(cwd, ["rev-parse", "HEAD"]);
|
|
52
|
-
}
|
|
53
|
-
|
|
54
|
-
function commitsSince(cwd, baseSha) {
|
|
55
|
-
if (!baseSha) return [];
|
|
56
|
-
const out = gitIn(cwd, ["log", `${baseSha}..HEAD`, "--pretty=format:%H%x1f%s%x1f%b%x1e"]);
|
|
57
|
-
if (!out) return [];
|
|
58
|
-
return out
|
|
59
|
-
.split("\x1e")
|
|
60
|
-
.map((s) => s.trim())
|
|
61
|
-
.filter(Boolean)
|
|
62
|
-
.map((entry) => {
|
|
63
|
-
const [sha, subject, body] = entry.split("\x1f");
|
|
64
|
-
return { sha: sha || "", subject: subject || "", body: body || "" };
|
|
65
|
-
});
|
|
66
|
-
}
|
|
67
|
-
|
|
68
|
-
function filesModifiedSince(cwd, baseSha) {
|
|
69
|
-
if (!baseSha) return [];
|
|
70
|
-
const tracked = gitIn(cwd, ["diff", "--name-only", `${baseSha}..HEAD`]);
|
|
71
|
-
const untracked = gitIn(cwd, ["ls-files", "--others", "--exclude-standard"]);
|
|
72
|
-
const set = new Set();
|
|
73
|
-
for (const line of (tracked + "\n" + untracked).split(/\r?\n/)) {
|
|
74
|
-
const p = line.trim();
|
|
75
|
-
if (p) set.add(p.replace(/\\/g, "/"));
|
|
76
|
-
}
|
|
77
|
-
return [...set];
|
|
78
|
-
}
|
|
79
|
-
|
|
80
|
-
function writePreconditionFiles(worktreePath, files) {
|
|
81
|
-
if (!Array.isArray(files)) return;
|
|
82
|
-
for (const pf of files) {
|
|
83
|
-
if (!pf || typeof pf.path !== "string") continue;
|
|
84
|
-
const abs = join(worktreePath, pf.path);
|
|
85
|
-
mkdirSync(dirname(abs), { recursive: true });
|
|
86
|
-
writeFileSync(abs, typeof pf.content === "string" ? pf.content : "");
|
|
87
|
-
}
|
|
88
|
-
}
|
|
89
|
-
|
|
90
|
-
function spawnHiddenShell(command, args, { cwd, env, timeoutMs }) {
|
|
91
|
-
return new Promise((resolve) => {
|
|
92
|
-
let settled = false;
|
|
93
|
-
let stdout = "";
|
|
94
|
-
let stderr = "";
|
|
95
|
-
let child;
|
|
96
|
-
try {
|
|
97
|
-
// shell: true is required on Windows so `.cmd`/`.bat`/`.ps1` shims (npm global
|
|
98
|
-
// wrappers like `claude`) resolve. On Unix shell: true routes through sh,
|
|
99
|
-
// which is harmless here. Quote each arg so the shell preserves prompts
|
|
100
|
-
// with spaces/metacharacters as single argv tokens.
|
|
101
|
-
const safeArgs = args.map((arg) => JSON.stringify(String(arg)));
|
|
102
|
-
child = spawn(command, safeArgs, { cwd, env, shell: true, windowsHide: true, stdio: ["ignore", "pipe", "pipe"] });
|
|
103
|
-
} catch (e) {
|
|
104
|
-
resolve({ exit: -1, stdout: "", stderr: `spawn failed: ${e.message}`, timedOut: false });
|
|
105
|
-
return;
|
|
106
|
-
}
|
|
107
|
-
const timer = setTimeout(() => {
|
|
108
|
-
if (settled) return;
|
|
109
|
-
try { child.kill("SIGKILL"); } catch {}
|
|
110
|
-
settled = true;
|
|
111
|
-
resolve({ exit: -1, stdout, stderr, timedOut: true });
|
|
112
|
-
}, timeoutMs);
|
|
113
|
-
child.stdout.on("data", (d) => { stdout += d.toString(); });
|
|
114
|
-
child.stderr.on("data", (d) => { stderr += d.toString(); });
|
|
115
|
-
child.on("error", (e) => {
|
|
116
|
-
if (settled) return;
|
|
117
|
-
clearTimeout(timer);
|
|
118
|
-
settled = true;
|
|
119
|
-
resolve({ exit: -1, stdout, stderr: stderr + `\nspawn error: ${e.message}`, timedOut: false });
|
|
120
|
-
});
|
|
121
|
-
child.on("close", (code) => {
|
|
122
|
-
if (settled) return;
|
|
123
|
-
clearTimeout(timer);
|
|
124
|
-
settled = true;
|
|
125
|
-
resolve({ exit: code == null ? -1 : code, stdout, stderr, timedOut: false });
|
|
126
|
-
});
|
|
127
|
-
});
|
|
128
|
-
}
|
|
129
|
-
|
|
130
|
-
/** Spawn `claude --print ...` with a hard timeout. Resolves to {exit, stdout, stderr, timedOut}. */
|
|
131
|
-
function spawnClaude({ claudeBin, prompt, cwd, env, timeoutMs }) {
|
|
132
|
-
return spawnHiddenShell(
|
|
133
|
-
claudeBin,
|
|
134
|
-
["--print", prompt ?? "", "--output-format", "stream-json", "--verbose", "--dangerously-skip-permissions"],
|
|
135
|
-
{ cwd, env, timeoutMs },
|
|
136
|
-
);
|
|
137
|
-
}
|
|
138
|
-
|
|
139
|
-
/** Spawn `codex exec --json ...` with a hard timeout. Resolves to {exit, stdout, stderr, timedOut}. */
|
|
140
|
-
function spawnCodex({ codexBin, prompt, cwd, env, timeoutMs }) {
|
|
141
|
-
return spawnHiddenShell(
|
|
142
|
-
codexBin,
|
|
143
|
-
[
|
|
144
|
-
"exec",
|
|
145
|
-
"--json",
|
|
146
|
-
"--cd",
|
|
147
|
-
cwd,
|
|
148
|
-
"--sandbox",
|
|
149
|
-
"danger-full-access",
|
|
150
|
-
"--dangerously-bypass-approvals-and-sandbox",
|
|
151
|
-
"--dangerously-bypass-hook-trust",
|
|
152
|
-
"--ephemeral",
|
|
153
|
-
prompt ?? "",
|
|
154
|
-
],
|
|
155
|
-
{ cwd, env, timeoutMs },
|
|
156
|
-
);
|
|
157
|
-
}
|
|
158
|
-
|
|
159
|
-
// ─── work_items fetch (live) ────────────────────────────────────────────────
|
|
160
|
-
//
|
|
161
|
-
// These helpers talk to the Supabase PostgREST endpoint on the branch created
|
|
162
|
-
// by sandbox.createSupabaseTestBranch. They never throw — on any error (missing
|
|
163
|
-
// creds, network, bad response) they log a one-line warning and return a safe
|
|
164
|
-
// empty value so the runner degrades gracefully in environments without a
|
|
165
|
-
// live branch.
|
|
166
|
-
//
|
|
167
|
-
// supabaseBranchRef shape: { branchId, connectionString, apiUrl, anonKey }
|
|
168
|
-
|
|
169
|
-
/** Parse a PostgREST Content-Range header like "0-0/42" → 42. "*\/0" → 0. */
|
|
170
|
-
export function parseContentRange(header) {
|
|
171
|
-
if (typeof header !== "string" || header.length === 0) return 0;
|
|
172
|
-
const m = header.match(/\/(\d+|\*)\s*$/);
|
|
173
|
-
if (!m) return 0;
|
|
174
|
-
if (m[1] === "*") return 0;
|
|
175
|
-
const n = parseInt(m[1], 10);
|
|
176
|
-
return Number.isFinite(n) ? n : 0;
|
|
177
|
-
}
|
|
178
|
-
|
|
179
|
-
function hasBranchCreds(ref) {
|
|
180
|
-
return !!(ref && typeof ref === "object" && ref.apiUrl && ref.anonKey);
|
|
181
|
-
}
|
|
182
|
-
|
|
183
|
-
export async function fetchWorkItemsCount(supabaseBranchRef) {
|
|
184
|
-
if (!hasBranchCreds(supabaseBranchRef)) {
|
|
185
|
-
if (supabaseBranchRef) {
|
|
186
|
-
console.error(
|
|
187
|
-
"fetchWorkItemsCount: supabaseBranchRef missing apiUrl/anonKey — returning 0",
|
|
188
|
-
);
|
|
189
|
-
}
|
|
190
|
-
return 0;
|
|
191
|
-
}
|
|
192
|
-
const { apiUrl, anonKey } = supabaseBranchRef;
|
|
193
|
-
try {
|
|
194
|
-
const res = await fetch(`${apiUrl}/rest/v1/work_items?select=id`, {
|
|
195
|
-
headers: {
|
|
196
|
-
apikey: anonKey,
|
|
197
|
-
Authorization: `Bearer ${anonKey}`,
|
|
198
|
-
Prefer: "count=exact",
|
|
199
|
-
Range: "0-0",
|
|
200
|
-
"Range-Unit": "items",
|
|
201
|
-
},
|
|
202
|
-
});
|
|
203
|
-
if (!res.ok && res.status !== 206) {
|
|
204
|
-
console.error(
|
|
205
|
-
`fetchWorkItemsCount: HTTP ${res.status}: ${(await res.text()).slice(0, 200)}`,
|
|
206
|
-
);
|
|
207
|
-
return 0;
|
|
208
|
-
}
|
|
209
|
-
const cr = res.headers.get("content-range") || res.headers.get("Content-Range");
|
|
210
|
-
return parseContentRange(cr || "");
|
|
211
|
-
} catch (e) {
|
|
212
|
-
console.error(`fetchWorkItemsCount: ${e.message}`);
|
|
213
|
-
return 0;
|
|
214
|
-
}
|
|
215
|
-
}
|
|
216
|
-
|
|
217
|
-
/**
|
|
218
|
-
* Fetch metadata for the N newest work_items created since the snapshot.
|
|
219
|
-
* Returns an array of { id, status, labels } (matching the shape consumed by
|
|
220
|
-
* assertions.checkWorkItemsCreated). Never throws.
|
|
221
|
-
*/
|
|
222
|
-
export async function fetchWorkItemsDelta(supabaseBranchRef, snapshotCount) {
|
|
223
|
-
if (!hasBranchCreds(supabaseBranchRef)) return [];
|
|
224
|
-
const currentCount = await fetchWorkItemsCount(supabaseBranchRef);
|
|
225
|
-
const delta = currentCount - (Number(snapshotCount) || 0);
|
|
226
|
-
if (delta <= 0) return [];
|
|
227
|
-
|
|
228
|
-
const { apiUrl, anonKey } = supabaseBranchRef;
|
|
229
|
-
try {
|
|
230
|
-
const url =
|
|
231
|
-
`${apiUrl}/rest/v1/work_items` +
|
|
232
|
-
`?select=id,status,labels&order=created_at.desc&limit=${delta}`;
|
|
233
|
-
const res = await fetch(url, {
|
|
234
|
-
headers: {
|
|
235
|
-
apikey: anonKey,
|
|
236
|
-
Authorization: `Bearer ${anonKey}`,
|
|
237
|
-
},
|
|
238
|
-
});
|
|
239
|
-
if (!res.ok) {
|
|
240
|
-
console.error(
|
|
241
|
-
`fetchWorkItemsDelta: HTTP ${res.status}: ${(await res.text()).slice(0, 200)}`,
|
|
242
|
-
);
|
|
243
|
-
return [];
|
|
244
|
-
}
|
|
245
|
-
const body = await res.json();
|
|
246
|
-
if (!Array.isArray(body)) return [];
|
|
247
|
-
return body.map((wi) => ({
|
|
248
|
-
id: wi?.id,
|
|
249
|
-
status: wi?.status,
|
|
250
|
-
labels: Array.isArray(wi?.labels) ? wi.labels : [],
|
|
251
|
-
}));
|
|
252
|
-
} catch (e) {
|
|
253
|
-
console.error(`fetchWorkItemsDelta: ${e.message}`);
|
|
254
|
-
return [];
|
|
255
|
-
}
|
|
256
|
-
}
|
|
257
|
-
|
|
258
|
-
export async function runManifest(manifest, opts = {}) {
|
|
259
|
-
const started = nowMs();
|
|
260
|
-
const skill = manifest?.skill || "<unknown>";
|
|
261
|
-
const {
|
|
262
|
-
runId,
|
|
263
|
-
supabaseBranchRef = null,
|
|
264
|
-
timeout = manifest?.timeout_ms || DEFAULT_TIMEOUT_MS,
|
|
265
|
-
engine = process.env.RDC_ACCEPTANCE_ENGINE || "claude",
|
|
266
|
-
claudeBin = process.env.CLAUDE_BIN || "claude",
|
|
267
|
-
codexBin = process.env.CODEX_BIN || "codex",
|
|
268
|
-
// projectCwd: the repo claude --print will commit to (regen-root).
|
|
269
|
-
// Worktrees are created FROM this repo so git state tracking is correct.
|
|
270
|
-
// claude runs inside the worktree (not projectCwd directly) so each skill
|
|
271
|
-
// gets an isolated branch and commits don't pollute the working tree.
|
|
272
|
-
projectCwd = process.env.REGEN_ROOT || process.cwd(),
|
|
273
|
-
} = opts;
|
|
274
|
-
|
|
275
|
-
if (!runId) {
|
|
276
|
-
return { skill, pass: false, duration_ms: 0, error: "runManifest: runId is required" };
|
|
277
|
-
}
|
|
278
|
-
|
|
279
|
-
// Create worktree FROM projectCwd (regen-root), not from rdc-skills.
|
|
280
|
-
// This ensures commits made by claude land in the worktree branch and
|
|
281
|
-
// are visible to commitsSince() / filesModifiedSince().
|
|
282
|
-
let worktree;
|
|
283
|
-
try {
|
|
284
|
-
worktree = createWorktree(runId, skill, projectCwd);
|
|
285
|
-
} catch (e) {
|
|
286
|
-
return { skill, pass: false, duration_ms: nowMs() - started, error: `worktree: ${e.message}` };
|
|
287
|
-
}
|
|
288
|
-
|
|
289
|
-
// Write precondition files into the worktree (isolated copy of projectCwd).
|
|
290
|
-
try {
|
|
291
|
-
writePreconditionFiles(worktree.path, manifest.fixture?.precondition_files);
|
|
292
|
-
} catch (e) {
|
|
293
|
-
return { skill, pass: false, duration_ms: nowMs() - started, error: `precondition_files: ${e.message}` };
|
|
294
|
-
}
|
|
295
|
-
|
|
296
|
-
// Commit precondition files so HEAD snapshot is stable before claude runs.
|
|
297
|
-
try {
|
|
298
|
-
gitIn(worktree.path, ["add", "-A"]);
|
|
299
|
-
const status = gitIn(worktree.path, ["status", "--porcelain"]);
|
|
300
|
-
if (status) {
|
|
301
|
-
execFileSync("git", ["-c", "user.email=test@rdc", "-c", "user.name=rdc-test", "commit", "-m", "test: precondition"], {
|
|
302
|
-
cwd: worktree.path,
|
|
303
|
-
stdio: "ignore",
|
|
304
|
-
});
|
|
305
|
-
}
|
|
306
|
-
} catch {
|
|
307
|
-
// non-fatal — snapshot will still work against whatever HEAD is
|
|
308
|
-
}
|
|
309
|
-
|
|
310
|
-
const headBefore = snapshotHead(worktree.path);
|
|
311
|
-
const wiCountBefore = await fetchWorkItemsCount(supabaseBranchRef);
|
|
312
|
-
|
|
313
|
-
const selectedEngine = String(engine || "claude").toLowerCase();
|
|
314
|
-
if (!["claude", "codex"].includes(selectedEngine)) {
|
|
315
|
-
return { skill, pass: false, duration_ms: nowMs() - started, error: `unsupported engine: ${selectedEngine}` };
|
|
316
|
-
}
|
|
317
|
-
|
|
318
|
-
// Start from process.env but scrub inherited engine session vars — the child
|
|
319
|
-
// process should not see parent session context. Then set RDC_TEST
|
|
320
|
-
// and overlay fixture env.
|
|
321
|
-
const env = { ...process.env };
|
|
322
|
-
for (const key of Object.keys(env)) {
|
|
323
|
-
if (key.startsWith("CLAUDE_")) delete env[key];
|
|
324
|
-
if (key.startsWith("CODEX_") && key !== "CODEX_HOME") delete env[key];
|
|
325
|
-
}
|
|
326
|
-
env.RDC_TEST = "1";
|
|
327
|
-
Object.assign(env, manifest.fixture?.env || {});
|
|
328
|
-
|
|
329
|
-
// Run the agent inside the worktree — commits land there, git tracking is correct,
|
|
330
|
-
// and parallel skills each get their own isolated branch with no file conflicts.
|
|
331
|
-
let spawnRes;
|
|
332
|
-
try {
|
|
333
|
-
const spawnArgs = {
|
|
334
|
-
prompt: manifest.fixture?.prompt || "",
|
|
335
|
-
cwd: worktree.path,
|
|
336
|
-
env,
|
|
337
|
-
timeoutMs: timeout,
|
|
338
|
-
};
|
|
339
|
-
spawnRes = selectedEngine === "codex"
|
|
340
|
-
? await spawnCodex({ ...spawnArgs, codexBin })
|
|
341
|
-
: await spawnClaude({ ...spawnArgs, claudeBin });
|
|
342
|
-
} catch (e) {
|
|
343
|
-
return {
|
|
344
|
-
skill,
|
|
345
|
-
pass: false,
|
|
346
|
-
duration_ms: nowMs() - started,
|
|
347
|
-
error: `spawn ${selectedEngine}: ${e.message}`,
|
|
348
|
-
};
|
|
349
|
-
}
|
|
350
|
-
|
|
351
|
-
const commits = commitsSince(worktree.path, headBefore);
|
|
352
|
-
const files_modified = filesModifiedSince(worktree.path, headBefore);
|
|
353
|
-
const work_items_delta = await fetchWorkItemsDelta(supabaseBranchRef, wiCountBefore);
|
|
354
|
-
|
|
355
|
-
const observed = {
|
|
356
|
-
exit_code: spawnRes.exit,
|
|
357
|
-
stdout: spawnRes.stdout,
|
|
358
|
-
stderr: spawnRes.stderr,
|
|
359
|
-
engine: selectedEngine,
|
|
360
|
-
files_modified,
|
|
361
|
-
commits,
|
|
362
|
-
work_items_delta,
|
|
363
|
-
timed_out: spawnRes.timedOut,
|
|
364
|
-
};
|
|
365
|
-
|
|
366
|
-
const evalRes = evaluateAssertions(manifest.assertions || {}, observed);
|
|
367
|
-
let pass = evalRes.pass;
|
|
368
|
-
const failures = [...evalRes.failures];
|
|
369
|
-
if (spawnRes.timedOut) {
|
|
370
|
-
pass = false;
|
|
371
|
-
failures.unshift({ predicate: "timeout", message: `child killed after ${timeout}ms` });
|
|
372
|
-
}
|
|
373
|
-
|
|
374
|
-
return {
|
|
375
|
-
skill,
|
|
376
|
-
pass,
|
|
377
|
-
duration_ms: nowMs() - started,
|
|
378
|
-
observed,
|
|
379
|
-
failures,
|
|
380
|
-
worktree: worktree.path,
|
|
381
|
-
};
|
|
382
|
-
}
|
|
383
|
-
|
|
384
|
-
// ─── self-test ──────────────────────────────────────────────────────────────
|
|
385
|
-
|
|
386
|
-
const __isMain = (() => {
|
|
387
|
-
try {
|
|
388
|
-
return fileURLToPath(import.meta.url) === resolve(process.argv[1] || "");
|
|
389
|
-
} catch {
|
|
390
|
-
return false;
|
|
391
|
-
}
|
|
392
|
-
})();
|
|
393
|
-
|
|
394
|
-
if (__isMain) {
|
|
395
|
-
let passed = 0;
|
|
396
|
-
let total = 0;
|
|
397
|
-
function t(desc, ok) {
|
|
398
|
-
total++;
|
|
399
|
-
if (ok) passed++;
|
|
400
|
-
console.log(`${ok ? "PASS" : "FAIL"} ${desc}`);
|
|
401
|
-
}
|
|
402
|
-
|
|
403
|
-
console.log("\nrunner.mjs self-test\n");
|
|
404
|
-
|
|
405
|
-
// parseContentRange
|
|
406
|
-
t("parseContentRange '0-0/42' → 42", parseContentRange("0-0/42") === 42);
|
|
407
|
-
t("parseContentRange '0-0/0' → 0", parseContentRange("0-0/0") === 0);
|
|
408
|
-
t("parseContentRange '*/0' → 0", parseContentRange("*/0") === 0);
|
|
409
|
-
t("parseContentRange '0-24/100' → 100", parseContentRange("0-24/100") === 100);
|
|
410
|
-
t("parseContentRange '' → 0", parseContentRange("") === 0);
|
|
411
|
-
t("parseContentRange null → 0", parseContentRange(null) === 0);
|
|
412
|
-
t("parseContentRange 'garbage' → 0", parseContentRange("garbage") === 0);
|
|
413
|
-
|
|
414
|
-
// fetch helpers graceful-degrade path
|
|
415
|
-
(async () => {
|
|
416
|
-
const c0 = await fetchWorkItemsCount(null);
|
|
417
|
-
t("fetchWorkItemsCount(null) → 0", c0 === 0);
|
|
418
|
-
const c1 = await fetchWorkItemsCount({ branchId: "x" }); // missing apiUrl/anonKey
|
|
419
|
-
t("fetchWorkItemsCount(no-creds) → 0", c1 === 0);
|
|
420
|
-
const d0 = await fetchWorkItemsDelta(null, 0);
|
|
421
|
-
t(
|
|
422
|
-
"fetchWorkItemsDelta(null, 0) → []",
|
|
423
|
-
Array.isArray(d0) && d0.length === 0,
|
|
424
|
-
);
|
|
425
|
-
|
|
426
|
-
// evaluateAssertions with observed work_items_delta matching manifest
|
|
427
|
-
const observed = {
|
|
428
|
-
exit_code: 0,
|
|
429
|
-
stdout: "",
|
|
430
|
-
stderr: "",
|
|
431
|
-
files_modified: [],
|
|
432
|
-
commits: [],
|
|
433
|
-
work_items_delta: [{ id: "w1", status: "done", labels: ["fixit"] }],
|
|
434
|
-
timed_out: false,
|
|
435
|
-
};
|
|
436
|
-
const r = evaluateAssertions(
|
|
437
|
-
{
|
|
438
|
-
exit_code: 0,
|
|
439
|
-
work_items_created: {
|
|
440
|
-
min: 1,
|
|
441
|
-
max: 1,
|
|
442
|
-
status: "done",
|
|
443
|
-
labels_include: ["fixit"],
|
|
444
|
-
},
|
|
445
|
-
},
|
|
446
|
-
observed,
|
|
447
|
-
);
|
|
448
|
-
t("evaluateAssertions: matching work_items → pass", r.pass === true);
|
|
449
|
-
|
|
450
|
-
const r2 = evaluateAssertions(
|
|
451
|
-
{
|
|
452
|
-
work_items_created: { min: 1, labels_include: ["missing"] },
|
|
453
|
-
},
|
|
454
|
-
observed,
|
|
455
|
-
);
|
|
456
|
-
t(
|
|
457
|
-
"evaluateAssertions: label mismatch → fail",
|
|
458
|
-
r2.pass === false &&
|
|
459
|
-
r2.failures.some((f) => f.predicate === "work_items_created"),
|
|
460
|
-
);
|
|
461
|
-
|
|
462
|
-
console.log(`\n${passed}/${total} passed\n`);
|
|
463
|
-
process.exit(passed === total ? 0 : 1);
|
|
464
|
-
})();
|
|
465
|
-
}
|
|
1
|
+
// runner.mjs — Tier 2 per-skill test orchestration.
|
|
2
|
+
//
|
|
3
|
+
// Public API:
|
|
4
|
+
// runManifest(manifest, { runId, supabaseBranchRef?, timeout?, engine?, claudeBin?, codexBin? })
|
|
5
|
+
// → { skill, pass, duration_ms, observed?, failures?, error? }
|
|
6
|
+
//
|
|
7
|
+
// Flow per manifest:
|
|
8
|
+
// 1. Create a worktree at .rdc-sandbox/<runId>/<skill-slug>/
|
|
9
|
+
// 2. Write precondition_files into the worktree
|
|
10
|
+
// 3. Snapshot HEAD sha + work_items rowcount (via Supabase branch if given)
|
|
11
|
+
// 4. Spawn the selected engine in headless JSONL mode:
|
|
12
|
+
// - claude: `claude --print "<prompt>" --output-format stream-json`
|
|
13
|
+
// - codex: `codex exec --json "<prompt>"`
|
|
14
|
+
// - cwd = worktree
|
|
15
|
+
// - env = { ...process.env, ...fixture.env, RDC_TEST: "1" }
|
|
16
|
+
// - timeout kills the child on overrun
|
|
17
|
+
// 5. Compute observed delta: commits since snapshot, files changed, work_items delta
|
|
18
|
+
// 6. Call evaluateAssertions
|
|
19
|
+
// 7. Return structured result — never throws (caller-friendly).
|
|
20
|
+
//
|
|
21
|
+
// Cleanup is the caller's responsibility (runner does NOT remove the worktree,
|
|
22
|
+
// so the caller can inspect on failure before teardown).
|
|
23
|
+
|
|
24
|
+
import { spawn, execFileSync } from "node:child_process";
|
|
25
|
+
import { writeFileSync, mkdirSync, existsSync } from "node:fs";
|
|
26
|
+
import { join, dirname, resolve } from "node:path";
|
|
27
|
+
import { fileURLToPath } from "node:url";
|
|
28
|
+
import { createWorktree } from "./sandbox.mjs";
|
|
29
|
+
import { evaluateAssertions } from "./assertions.mjs";
|
|
30
|
+
|
|
31
|
+
const DEFAULT_TIMEOUT_MS = 240_000;
|
|
32
|
+
|
|
33
|
+
function nowMs() {
|
|
34
|
+
return Date.now();
|
|
35
|
+
}
|
|
36
|
+
|
|
37
|
+
/** Run a git command inside `cwd`, return trimmed stdout or "". Never throws. */
|
|
38
|
+
function gitIn(cwd, args) {
|
|
39
|
+
try {
|
|
40
|
+
return execFileSync("git", args, {
|
|
41
|
+
cwd,
|
|
42
|
+
encoding: "utf8",
|
|
43
|
+
stdio: ["ignore", "pipe", "pipe"],
|
|
44
|
+
}).trim();
|
|
45
|
+
} catch {
|
|
46
|
+
return "";
|
|
47
|
+
}
|
|
48
|
+
}
|
|
49
|
+
|
|
50
|
+
function snapshotHead(cwd) {
|
|
51
|
+
return gitIn(cwd, ["rev-parse", "HEAD"]);
|
|
52
|
+
}
|
|
53
|
+
|
|
54
|
+
function commitsSince(cwd, baseSha) {
|
|
55
|
+
if (!baseSha) return [];
|
|
56
|
+
const out = gitIn(cwd, ["log", `${baseSha}..HEAD`, "--pretty=format:%H%x1f%s%x1f%b%x1e"]);
|
|
57
|
+
if (!out) return [];
|
|
58
|
+
return out
|
|
59
|
+
.split("\x1e")
|
|
60
|
+
.map((s) => s.trim())
|
|
61
|
+
.filter(Boolean)
|
|
62
|
+
.map((entry) => {
|
|
63
|
+
const [sha, subject, body] = entry.split("\x1f");
|
|
64
|
+
return { sha: sha || "", subject: subject || "", body: body || "" };
|
|
65
|
+
});
|
|
66
|
+
}
|
|
67
|
+
|
|
68
|
+
function filesModifiedSince(cwd, baseSha) {
|
|
69
|
+
if (!baseSha) return [];
|
|
70
|
+
const tracked = gitIn(cwd, ["diff", "--name-only", `${baseSha}..HEAD`]);
|
|
71
|
+
const untracked = gitIn(cwd, ["ls-files", "--others", "--exclude-standard"]);
|
|
72
|
+
const set = new Set();
|
|
73
|
+
for (const line of (tracked + "\n" + untracked).split(/\r?\n/)) {
|
|
74
|
+
const p = line.trim();
|
|
75
|
+
if (p) set.add(p.replace(/\\/g, "/"));
|
|
76
|
+
}
|
|
77
|
+
return [...set];
|
|
78
|
+
}
|
|
79
|
+
|
|
80
|
+
function writePreconditionFiles(worktreePath, files) {
|
|
81
|
+
if (!Array.isArray(files)) return;
|
|
82
|
+
for (const pf of files) {
|
|
83
|
+
if (!pf || typeof pf.path !== "string") continue;
|
|
84
|
+
const abs = join(worktreePath, pf.path);
|
|
85
|
+
mkdirSync(dirname(abs), { recursive: true });
|
|
86
|
+
writeFileSync(abs, typeof pf.content === "string" ? pf.content : "");
|
|
87
|
+
}
|
|
88
|
+
}
|
|
89
|
+
|
|
90
|
+
function spawnHiddenShell(command, args, { cwd, env, timeoutMs }) {
|
|
91
|
+
return new Promise((resolve) => {
|
|
92
|
+
let settled = false;
|
|
93
|
+
let stdout = "";
|
|
94
|
+
let stderr = "";
|
|
95
|
+
let child;
|
|
96
|
+
try {
|
|
97
|
+
// shell: true is required on Windows so `.cmd`/`.bat`/`.ps1` shims (npm global
|
|
98
|
+
// wrappers like `claude`) resolve. On Unix shell: true routes through sh,
|
|
99
|
+
// which is harmless here. Quote each arg so the shell preserves prompts
|
|
100
|
+
// with spaces/metacharacters as single argv tokens.
|
|
101
|
+
const safeArgs = args.map((arg) => JSON.stringify(String(arg)));
|
|
102
|
+
child = spawn(command, safeArgs, { cwd, env, shell: true, windowsHide: true, stdio: ["ignore", "pipe", "pipe"] });
|
|
103
|
+
} catch (e) {
|
|
104
|
+
resolve({ exit: -1, stdout: "", stderr: `spawn failed: ${e.message}`, timedOut: false });
|
|
105
|
+
return;
|
|
106
|
+
}
|
|
107
|
+
const timer = setTimeout(() => {
|
|
108
|
+
if (settled) return;
|
|
109
|
+
try { child.kill("SIGKILL"); } catch {}
|
|
110
|
+
settled = true;
|
|
111
|
+
resolve({ exit: -1, stdout, stderr, timedOut: true });
|
|
112
|
+
}, timeoutMs);
|
|
113
|
+
child.stdout.on("data", (d) => { stdout += d.toString(); });
|
|
114
|
+
child.stderr.on("data", (d) => { stderr += d.toString(); });
|
|
115
|
+
child.on("error", (e) => {
|
|
116
|
+
if (settled) return;
|
|
117
|
+
clearTimeout(timer);
|
|
118
|
+
settled = true;
|
|
119
|
+
resolve({ exit: -1, stdout, stderr: stderr + `\nspawn error: ${e.message}`, timedOut: false });
|
|
120
|
+
});
|
|
121
|
+
child.on("close", (code) => {
|
|
122
|
+
if (settled) return;
|
|
123
|
+
clearTimeout(timer);
|
|
124
|
+
settled = true;
|
|
125
|
+
resolve({ exit: code == null ? -1 : code, stdout, stderr, timedOut: false });
|
|
126
|
+
});
|
|
127
|
+
});
|
|
128
|
+
}
|
|
129
|
+
|
|
130
|
+
/** Spawn `claude --print ...` with a hard timeout. Resolves to {exit, stdout, stderr, timedOut}. */
|
|
131
|
+
function spawnClaude({ claudeBin, prompt, cwd, env, timeoutMs }) {
|
|
132
|
+
return spawnHiddenShell(
|
|
133
|
+
claudeBin,
|
|
134
|
+
["--print", prompt ?? "", "--output-format", "stream-json", "--verbose", "--dangerously-skip-permissions"],
|
|
135
|
+
{ cwd, env, timeoutMs },
|
|
136
|
+
);
|
|
137
|
+
}
|
|
138
|
+
|
|
139
|
+
/** Spawn `codex exec --json ...` with a hard timeout. Resolves to {exit, stdout, stderr, timedOut}. */
|
|
140
|
+
function spawnCodex({ codexBin, prompt, cwd, env, timeoutMs }) {
|
|
141
|
+
return spawnHiddenShell(
|
|
142
|
+
codexBin,
|
|
143
|
+
[
|
|
144
|
+
"exec",
|
|
145
|
+
"--json",
|
|
146
|
+
"--cd",
|
|
147
|
+
cwd,
|
|
148
|
+
"--sandbox",
|
|
149
|
+
"danger-full-access",
|
|
150
|
+
"--dangerously-bypass-approvals-and-sandbox",
|
|
151
|
+
"--dangerously-bypass-hook-trust",
|
|
152
|
+
"--ephemeral",
|
|
153
|
+
prompt ?? "",
|
|
154
|
+
],
|
|
155
|
+
{ cwd, env, timeoutMs },
|
|
156
|
+
);
|
|
157
|
+
}
|
|
158
|
+
|
|
159
|
+
// ─── work_items fetch (live) ────────────────────────────────────────────────
|
|
160
|
+
//
|
|
161
|
+
// These helpers talk to the Supabase PostgREST endpoint on the branch created
|
|
162
|
+
// by sandbox.createSupabaseTestBranch. They never throw — on any error (missing
|
|
163
|
+
// creds, network, bad response) they log a one-line warning and return a safe
|
|
164
|
+
// empty value so the runner degrades gracefully in environments without a
|
|
165
|
+
// live branch.
|
|
166
|
+
//
|
|
167
|
+
// supabaseBranchRef shape: { branchId, connectionString, apiUrl, anonKey }
|
|
168
|
+
|
|
169
|
+
/** Parse a PostgREST Content-Range header like "0-0/42" → 42. "*\/0" → 0. */
|
|
170
|
+
export function parseContentRange(header) {
|
|
171
|
+
if (typeof header !== "string" || header.length === 0) return 0;
|
|
172
|
+
const m = header.match(/\/(\d+|\*)\s*$/);
|
|
173
|
+
if (!m) return 0;
|
|
174
|
+
if (m[1] === "*") return 0;
|
|
175
|
+
const n = parseInt(m[1], 10);
|
|
176
|
+
return Number.isFinite(n) ? n : 0;
|
|
177
|
+
}
|
|
178
|
+
|
|
179
|
+
function hasBranchCreds(ref) {
|
|
180
|
+
return !!(ref && typeof ref === "object" && ref.apiUrl && ref.anonKey);
|
|
181
|
+
}
|
|
182
|
+
|
|
183
|
+
export async function fetchWorkItemsCount(supabaseBranchRef) {
|
|
184
|
+
if (!hasBranchCreds(supabaseBranchRef)) {
|
|
185
|
+
if (supabaseBranchRef) {
|
|
186
|
+
console.error(
|
|
187
|
+
"fetchWorkItemsCount: supabaseBranchRef missing apiUrl/anonKey — returning 0",
|
|
188
|
+
);
|
|
189
|
+
}
|
|
190
|
+
return 0;
|
|
191
|
+
}
|
|
192
|
+
const { apiUrl, anonKey } = supabaseBranchRef;
|
|
193
|
+
try {
|
|
194
|
+
const res = await fetch(`${apiUrl}/rest/v1/work_items?select=id`, {
|
|
195
|
+
headers: {
|
|
196
|
+
apikey: anonKey,
|
|
197
|
+
Authorization: `Bearer ${anonKey}`,
|
|
198
|
+
Prefer: "count=exact",
|
|
199
|
+
Range: "0-0",
|
|
200
|
+
"Range-Unit": "items",
|
|
201
|
+
},
|
|
202
|
+
});
|
|
203
|
+
if (!res.ok && res.status !== 206) {
|
|
204
|
+
console.error(
|
|
205
|
+
`fetchWorkItemsCount: HTTP ${res.status}: ${(await res.text()).slice(0, 200)}`,
|
|
206
|
+
);
|
|
207
|
+
return 0;
|
|
208
|
+
}
|
|
209
|
+
const cr = res.headers.get("content-range") || res.headers.get("Content-Range");
|
|
210
|
+
return parseContentRange(cr || "");
|
|
211
|
+
} catch (e) {
|
|
212
|
+
console.error(`fetchWorkItemsCount: ${e.message}`);
|
|
213
|
+
return 0;
|
|
214
|
+
}
|
|
215
|
+
}
|
|
216
|
+
|
|
217
|
+
/**
|
|
218
|
+
* Fetch metadata for the N newest work_items created since the snapshot.
|
|
219
|
+
* Returns an array of { id, status, labels } (matching the shape consumed by
|
|
220
|
+
* assertions.checkWorkItemsCreated). Never throws.
|
|
221
|
+
*/
|
|
222
|
+
export async function fetchWorkItemsDelta(supabaseBranchRef, snapshotCount) {
|
|
223
|
+
if (!hasBranchCreds(supabaseBranchRef)) return [];
|
|
224
|
+
const currentCount = await fetchWorkItemsCount(supabaseBranchRef);
|
|
225
|
+
const delta = currentCount - (Number(snapshotCount) || 0);
|
|
226
|
+
if (delta <= 0) return [];
|
|
227
|
+
|
|
228
|
+
const { apiUrl, anonKey } = supabaseBranchRef;
|
|
229
|
+
try {
|
|
230
|
+
const url =
|
|
231
|
+
`${apiUrl}/rest/v1/work_items` +
|
|
232
|
+
`?select=id,status,labels&order=created_at.desc&limit=${delta}`;
|
|
233
|
+
const res = await fetch(url, {
|
|
234
|
+
headers: {
|
|
235
|
+
apikey: anonKey,
|
|
236
|
+
Authorization: `Bearer ${anonKey}`,
|
|
237
|
+
},
|
|
238
|
+
});
|
|
239
|
+
if (!res.ok) {
|
|
240
|
+
console.error(
|
|
241
|
+
`fetchWorkItemsDelta: HTTP ${res.status}: ${(await res.text()).slice(0, 200)}`,
|
|
242
|
+
);
|
|
243
|
+
return [];
|
|
244
|
+
}
|
|
245
|
+
const body = await res.json();
|
|
246
|
+
if (!Array.isArray(body)) return [];
|
|
247
|
+
return body.map((wi) => ({
|
|
248
|
+
id: wi?.id,
|
|
249
|
+
status: wi?.status,
|
|
250
|
+
labels: Array.isArray(wi?.labels) ? wi.labels : [],
|
|
251
|
+
}));
|
|
252
|
+
} catch (e) {
|
|
253
|
+
console.error(`fetchWorkItemsDelta: ${e.message}`);
|
|
254
|
+
return [];
|
|
255
|
+
}
|
|
256
|
+
}
|
|
257
|
+
|
|
258
|
+
export async function runManifest(manifest, opts = {}) {
|
|
259
|
+
const started = nowMs();
|
|
260
|
+
const skill = manifest?.skill || "<unknown>";
|
|
261
|
+
const {
|
|
262
|
+
runId,
|
|
263
|
+
supabaseBranchRef = null,
|
|
264
|
+
timeout = manifest?.timeout_ms || DEFAULT_TIMEOUT_MS,
|
|
265
|
+
engine = process.env.RDC_ACCEPTANCE_ENGINE || "claude",
|
|
266
|
+
claudeBin = process.env.CLAUDE_BIN || "claude",
|
|
267
|
+
codexBin = process.env.CODEX_BIN || "codex",
|
|
268
|
+
// projectCwd: the repo claude --print will commit to (regen-root).
|
|
269
|
+
// Worktrees are created FROM this repo so git state tracking is correct.
|
|
270
|
+
// claude runs inside the worktree (not projectCwd directly) so each skill
|
|
271
|
+
// gets an isolated branch and commits don't pollute the working tree.
|
|
272
|
+
projectCwd = process.env.REGEN_ROOT || process.cwd(),
|
|
273
|
+
} = opts;
|
|
274
|
+
|
|
275
|
+
if (!runId) {
|
|
276
|
+
return { skill, pass: false, duration_ms: 0, error: "runManifest: runId is required" };
|
|
277
|
+
}
|
|
278
|
+
|
|
279
|
+
// Create worktree FROM projectCwd (regen-root), not from rdc-skills.
|
|
280
|
+
// This ensures commits made by claude land in the worktree branch and
|
|
281
|
+
// are visible to commitsSince() / filesModifiedSince().
|
|
282
|
+
let worktree;
|
|
283
|
+
try {
|
|
284
|
+
worktree = createWorktree(runId, skill, projectCwd);
|
|
285
|
+
} catch (e) {
|
|
286
|
+
return { skill, pass: false, duration_ms: nowMs() - started, error: `worktree: ${e.message}` };
|
|
287
|
+
}
|
|
288
|
+
|
|
289
|
+
// Write precondition files into the worktree (isolated copy of projectCwd).
|
|
290
|
+
try {
|
|
291
|
+
writePreconditionFiles(worktree.path, manifest.fixture?.precondition_files);
|
|
292
|
+
} catch (e) {
|
|
293
|
+
return { skill, pass: false, duration_ms: nowMs() - started, error: `precondition_files: ${e.message}` };
|
|
294
|
+
}
|
|
295
|
+
|
|
296
|
+
// Commit precondition files so HEAD snapshot is stable before claude runs.
|
|
297
|
+
try {
|
|
298
|
+
gitIn(worktree.path, ["add", "-A"]);
|
|
299
|
+
const status = gitIn(worktree.path, ["status", "--porcelain"]);
|
|
300
|
+
if (status) {
|
|
301
|
+
execFileSync("git", ["-c", "user.email=test@rdc", "-c", "user.name=rdc-test", "commit", "-m", "test: precondition"], {
|
|
302
|
+
cwd: worktree.path,
|
|
303
|
+
stdio: "ignore",
|
|
304
|
+
});
|
|
305
|
+
}
|
|
306
|
+
} catch {
|
|
307
|
+
// non-fatal — snapshot will still work against whatever HEAD is
|
|
308
|
+
}
|
|
309
|
+
|
|
310
|
+
const headBefore = snapshotHead(worktree.path);
|
|
311
|
+
const wiCountBefore = await fetchWorkItemsCount(supabaseBranchRef);
|
|
312
|
+
|
|
313
|
+
const selectedEngine = String(engine || "claude").toLowerCase();
|
|
314
|
+
if (!["claude", "codex"].includes(selectedEngine)) {
|
|
315
|
+
return { skill, pass: false, duration_ms: nowMs() - started, error: `unsupported engine: ${selectedEngine}` };
|
|
316
|
+
}
|
|
317
|
+
|
|
318
|
+
// Start from process.env but scrub inherited engine session vars — the child
|
|
319
|
+
// process should not see parent session context. Then set RDC_TEST
|
|
320
|
+
// and overlay fixture env.
|
|
321
|
+
const env = { ...process.env };
|
|
322
|
+
for (const key of Object.keys(env)) {
|
|
323
|
+
if (key.startsWith("CLAUDE_")) delete env[key];
|
|
324
|
+
if (key.startsWith("CODEX_") && key !== "CODEX_HOME") delete env[key];
|
|
325
|
+
}
|
|
326
|
+
env.RDC_TEST = "1";
|
|
327
|
+
Object.assign(env, manifest.fixture?.env || {});
|
|
328
|
+
|
|
329
|
+
// Run the agent inside the worktree — commits land there, git tracking is correct,
|
|
330
|
+
// and parallel skills each get their own isolated branch with no file conflicts.
|
|
331
|
+
let spawnRes;
|
|
332
|
+
try {
|
|
333
|
+
const spawnArgs = {
|
|
334
|
+
prompt: manifest.fixture?.prompt || "",
|
|
335
|
+
cwd: worktree.path,
|
|
336
|
+
env,
|
|
337
|
+
timeoutMs: timeout,
|
|
338
|
+
};
|
|
339
|
+
spawnRes = selectedEngine === "codex"
|
|
340
|
+
? await spawnCodex({ ...spawnArgs, codexBin })
|
|
341
|
+
: await spawnClaude({ ...spawnArgs, claudeBin });
|
|
342
|
+
} catch (e) {
|
|
343
|
+
return {
|
|
344
|
+
skill,
|
|
345
|
+
pass: false,
|
|
346
|
+
duration_ms: nowMs() - started,
|
|
347
|
+
error: `spawn ${selectedEngine}: ${e.message}`,
|
|
348
|
+
};
|
|
349
|
+
}
|
|
350
|
+
|
|
351
|
+
const commits = commitsSince(worktree.path, headBefore);
|
|
352
|
+
const files_modified = filesModifiedSince(worktree.path, headBefore);
|
|
353
|
+
const work_items_delta = await fetchWorkItemsDelta(supabaseBranchRef, wiCountBefore);
|
|
354
|
+
|
|
355
|
+
const observed = {
|
|
356
|
+
exit_code: spawnRes.exit,
|
|
357
|
+
stdout: spawnRes.stdout,
|
|
358
|
+
stderr: spawnRes.stderr,
|
|
359
|
+
engine: selectedEngine,
|
|
360
|
+
files_modified,
|
|
361
|
+
commits,
|
|
362
|
+
work_items_delta,
|
|
363
|
+
timed_out: spawnRes.timedOut,
|
|
364
|
+
};
|
|
365
|
+
|
|
366
|
+
const evalRes = evaluateAssertions(manifest.assertions || {}, observed);
|
|
367
|
+
let pass = evalRes.pass;
|
|
368
|
+
const failures = [...evalRes.failures];
|
|
369
|
+
if (spawnRes.timedOut) {
|
|
370
|
+
pass = false;
|
|
371
|
+
failures.unshift({ predicate: "timeout", message: `child killed after ${timeout}ms` });
|
|
372
|
+
}
|
|
373
|
+
|
|
374
|
+
return {
|
|
375
|
+
skill,
|
|
376
|
+
pass,
|
|
377
|
+
duration_ms: nowMs() - started,
|
|
378
|
+
observed,
|
|
379
|
+
failures,
|
|
380
|
+
worktree: worktree.path,
|
|
381
|
+
};
|
|
382
|
+
}
|
|
383
|
+
|
|
384
|
+
// ─── self-test ──────────────────────────────────────────────────────────────
|
|
385
|
+
|
|
386
|
+
const __isMain = (() => {
|
|
387
|
+
try {
|
|
388
|
+
return fileURLToPath(import.meta.url) === resolve(process.argv[1] || "");
|
|
389
|
+
} catch {
|
|
390
|
+
return false;
|
|
391
|
+
}
|
|
392
|
+
})();
|
|
393
|
+
|
|
394
|
+
if (__isMain) {
|
|
395
|
+
let passed = 0;
|
|
396
|
+
let total = 0;
|
|
397
|
+
function t(desc, ok) {
|
|
398
|
+
total++;
|
|
399
|
+
if (ok) passed++;
|
|
400
|
+
console.log(`${ok ? "PASS" : "FAIL"} ${desc}`);
|
|
401
|
+
}
|
|
402
|
+
|
|
403
|
+
console.log("\nrunner.mjs self-test\n");
|
|
404
|
+
|
|
405
|
+
// parseContentRange
|
|
406
|
+
t("parseContentRange '0-0/42' → 42", parseContentRange("0-0/42") === 42);
|
|
407
|
+
t("parseContentRange '0-0/0' → 0", parseContentRange("0-0/0") === 0);
|
|
408
|
+
t("parseContentRange '*/0' → 0", parseContentRange("*/0") === 0);
|
|
409
|
+
t("parseContentRange '0-24/100' → 100", parseContentRange("0-24/100") === 100);
|
|
410
|
+
t("parseContentRange '' → 0", parseContentRange("") === 0);
|
|
411
|
+
t("parseContentRange null → 0", parseContentRange(null) === 0);
|
|
412
|
+
t("parseContentRange 'garbage' → 0", parseContentRange("garbage") === 0);
|
|
413
|
+
|
|
414
|
+
// fetch helpers graceful-degrade path
|
|
415
|
+
(async () => {
|
|
416
|
+
const c0 = await fetchWorkItemsCount(null);
|
|
417
|
+
t("fetchWorkItemsCount(null) → 0", c0 === 0);
|
|
418
|
+
const c1 = await fetchWorkItemsCount({ branchId: "x" }); // missing apiUrl/anonKey
|
|
419
|
+
t("fetchWorkItemsCount(no-creds) → 0", c1 === 0);
|
|
420
|
+
const d0 = await fetchWorkItemsDelta(null, 0);
|
|
421
|
+
t(
|
|
422
|
+
"fetchWorkItemsDelta(null, 0) → []",
|
|
423
|
+
Array.isArray(d0) && d0.length === 0,
|
|
424
|
+
);
|
|
425
|
+
|
|
426
|
+
// evaluateAssertions with observed work_items_delta matching manifest
|
|
427
|
+
const observed = {
|
|
428
|
+
exit_code: 0,
|
|
429
|
+
stdout: "",
|
|
430
|
+
stderr: "",
|
|
431
|
+
files_modified: [],
|
|
432
|
+
commits: [],
|
|
433
|
+
work_items_delta: [{ id: "w1", status: "done", labels: ["fixit"] }],
|
|
434
|
+
timed_out: false,
|
|
435
|
+
};
|
|
436
|
+
const r = evaluateAssertions(
|
|
437
|
+
{
|
|
438
|
+
exit_code: 0,
|
|
439
|
+
work_items_created: {
|
|
440
|
+
min: 1,
|
|
441
|
+
max: 1,
|
|
442
|
+
status: "done",
|
|
443
|
+
labels_include: ["fixit"],
|
|
444
|
+
},
|
|
445
|
+
},
|
|
446
|
+
observed,
|
|
447
|
+
);
|
|
448
|
+
t("evaluateAssertions: matching work_items → pass", r.pass === true);
|
|
449
|
+
|
|
450
|
+
const r2 = evaluateAssertions(
|
|
451
|
+
{
|
|
452
|
+
work_items_created: { min: 1, labels_include: ["missing"] },
|
|
453
|
+
},
|
|
454
|
+
observed,
|
|
455
|
+
);
|
|
456
|
+
t(
|
|
457
|
+
"evaluateAssertions: label mismatch → fail",
|
|
458
|
+
r2.pass === false &&
|
|
459
|
+
r2.failures.some((f) => f.predicate === "work_items_created"),
|
|
460
|
+
);
|
|
461
|
+
|
|
462
|
+
console.log(`\n${passed}/${total} passed\n`);
|
|
463
|
+
process.exit(passed === total ? 0 : 1);
|
|
464
|
+
})();
|
|
465
|
+
}
|