Fix v0.7.1 release-validation hang with deterministic failure fixture - #23
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1e2b4de4e7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| run: npm run research:validate | ||
|
|
||
| - name: Validate v0.7.1 regression suite | ||
| run: npm run v071:validate |
There was a problem hiding this comment.
Fix TEST F before requiring the regression suite in CI
On the Ubuntu/Linux runner configured for this job, this newly required command never completes: v071:validate runs TEST F, where bootstrapProject('/proc/invalid_dir/test') hangs in the recursive fs.mkdirSync call. I verified that TEST K now exits successfully in isolation, but TEST F and the full command time out, so every push and pull request will consume the 15-minute job timeout rather than reaching the remaining validation step. Replace TEST F's /proc fixture with a reliably unwritable, non-hanging fixture before enabling this gate.
AGENTS.md reference: AGENTS.md:L27-L27
Useful? React with 👍 / 👎.
Problem
The controlled v0.8.1 release reached
npm run v071:validateafter all preceding release gates passed, then remained alive until the release workflow's 25-minute timeout cancelled the job. No tag or package was published.Targeted process-isolated diagnostics proved that TEST F was the only failing/hanging regression. The test used OS-specific special paths (
Z:\\...on Windows and/proc/...on Linux) to simulate an unwritable project root. The Linux/procpseudo-filesystem made that failure contract non-deterministic in GitHub Actions.Fix
ENOTDIR/equivalent failure on Windows, Linux, and macOS.npm run v071:validategate to normal pull-request CI so this suite is exercised before release time.Verification
npm run v071:validatecommand used by the release pipeline now passes in GitHub Actions.Release safety
This does not bypass or weaken the release gate. It makes the existing failure-path test deterministic and moves the same canonical regression suite earlier into PR validation. v0.8.1 will be retried only after this PR is merged.