Skip to content

Commit dc6de9d

Browse files
NathanFlurryclaude
andcommitted
chore: update progress for US-014
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent de860d9 commit dc6de9d

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

progress.txt

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2508,3 +2508,13 @@ PRD: ralph/kernel-hardening (46 stories)
25082508
- CLI tool tests (pi, claude, opencode) don't use createTestNodeRuntime — they spawn CLI binaries as host child processes with mock LLM servers
25092509
- Pre-existing failures: nextjs-pass project-matrix (stderr format mismatch) and bridge-registry-policy (inventory key mismatch) — both unrelated to this change
25102510
---
2511+
2512+
## 2026-03-20 - US-014 (v8-migration PRD)
2513+
- What was implemented: Verified US-014 (Remove isolated-vm from codebase) was already completed by prior iteration commit aeefeb7. Confirmed all acceptance criteria pass: isolate.ts/execution.ts/execution-lifecycle.ts deleted, deprecated bridge-setup functions removed, legacy type stubs removed, no isolated-vm in package.json/imports, typecheck passes, tests pass (35 pre-existing failures unrelated to this change — e2e-docker needs Docker, python needs Pyodide, project-matrix needs fixture deps, bridge-registry-policy inventory mismatch)
2514+
- Files changed:
2515+
- scripts/ralph/prd.json — marked US-014 as passes: true
2516+
- **Learnings for future iterations:**
2517+
- Previous iteration committed the code change but failed to set passes: true in prd.json — always verify prd.json update in the same commit or a follow-up
2518+
- 35 pre-existing test failures are expected in full suite: e2e-docker (14, needs running Docker containers), python runtime (17, needs Pyodide), project-matrix (varies, needs fixture node_modules), bridge-registry-policy (1, inventory key mismatch)
2519+
- To verify a change didn't introduce regressions, compare test results before/after — if failure count is identical, failures are pre-existing
2520+
---

scripts/ralph/prd.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@
252252
"Tests pass"
253253
],
254254
"priority": 14,
255-
"passes": false,
255+
"passes": true,
256256
"notes": "This is the final cleanup. Only do this AFTER all tests pass on the V8 driver (US-012 and US-013). Keep runtime-agnostic code: bridge-contract.ts, require-setup.ts, and utility functions. The isolated-vm NodeExecutionDriver class in execution-driver.ts can be removed if no longer imported."
257257
}
258258
]

0 commit comments

Comments
 (0)