Skip to content

Run CI-equivalent checks before push - #47

Merged
happykhan merged 1 commit into
masterfrom
codex/local-pre-ci-gate
Sep 5, 2026
Merged

happykhan merged 1 commit into
masterfrom
codex/local-pre-ci-gate

Conversation

@happykhan

Copy link
Copy Markdown
Owner

Summary

  • make local and GitHub CI use the same verify:ci package scripts
  • install a tracked pre-push hook during npm ci/npm install
  • fail early when the local Node major does not match .nvmrc
  • run architecture checks with only clean-runner system commands to catch undeclared tools such as rg
  • keep dependency changes eligible for post-merge CI and update contributor instructions

Why

The failed master run was locally reproducible via the existing security audit, while the first renderer catch-up run failed because a quality script used rg, which happened to exist locally but not on the Ubuntu runner. This makes those checks automatic before push and makes CI invoke those exact same commands.

The nanoid lockfile update overlaps #45 so this branch can pass its own pre-push security gate against the current master; that diff will disappear automatically if #45 lands first.

Verification

  • clean npm ci with Node 24.18.1
  • confirmed the hook rejects Node 20.11.1 before running the suite
  • confirmed the portability guard passes with PATH restricted to /usr/bin:/bin
  • npm run verify
    • security audit: 0 unaccepted advisories
    • unit tests: 176 passed
    • production build: passed
    • production browser tests: 25 passed
    • development-runtime browser test: passed
  • real git push completed through the new pre-push hook

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying brigx with  Cloudflare Pages  Cloudflare Pages

Latest commit: 203aa9f
Status: ✅  Deploy successful!
Preview URL: https://f55e6d3a.brigx.pages.dev
Branch Preview URL: https://codex-local-pre-ci-gate.brigx.pages.dev

View logs

@happykhan
happykhan merged commit 0be436d into master Sep 5, 2026
3 checks passed
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