Skip to content

[codex] Fix p848 CI clean-checkout assumptions#7

Merged
SproutSeeds merged 1 commit into
mainfrom
codex/p848-ci-clean-checkout-artifacts
Apr 25, 2026
Merged

[codex] Fix p848 CI clean-checkout assumptions#7
SproutSeeds merged 1 commit into
mainfrom
codex/p848-ci-clean-checkout-artifacts

Conversation

@SproutSeeds
Copy link
Copy Markdown
Owner

Summary

Fix the p848 CI failures from the clean GitHub Actions checkout by removing two local-machine assumptions:

  • p848 ORP/live-frontier audit scripts now fall back to tracked packet summaries when ignored orp/ or output/ artifacts are absent.
  • source-index audit scripts now use in-process JS source scanning instead of requiring system rg.
  • the tail-282 alignment test now checks the actual source-path existence value instead of assuming ignored live-frontier artifacts are present.

Root Cause

The April 19 main CI run failed after npm ci succeeded because the test suite expected local/generated artifacts that are intentionally ignored by git:

  • orp/research/...
  • output/frontier-engine-local/.../SIX_PREFIX_*_FAMILY_MENU.json

The same p848 test file also exercised scripts that shell out to rg, which is available on the development machine but not guaranteed in the GitHub Actions runner.

Impact

This blocked repo CI and skipped the CI npm pack --dry-run step. It did not appear to block normal npm installs of erdos-problems@0.3.8, but clean-checkout contributors and CI could fail on the p848 audit surface.

Validation

  • node --test test/p848-282-alignment-obstruction-packet.test.js
  • env PATH=/tmp/erdos-node-only-bin node --test test/p848-282-alignment-obstruction-packet.test.js
  • npm test
  • env PATH=/tmp/erdos-node-only-bin:/usr/bin:/bin node --test

The final full-suite no-rg run passed 238/238 tests.

@SproutSeeds SproutSeeds marked this pull request as ready for review April 25, 2026 04:47
@SproutSeeds SproutSeeds merged commit b727b47 into main Apr 25, 2026
2 checks passed
@SproutSeeds SproutSeeds deleted the codex/p848-ci-clean-checkout-artifacts branch April 25, 2026 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant