Skip to content

Make BMAD dependency preflight read-only - #9

Merged
pdurlej merged 1 commit into
mainfrom
codex/read-only-bmad-gate
Jul 16, 2026
Merged

Make BMAD dependency preflight read-only#9
pdurlej merged 1 commit into
mainfrom
codex/read-only-bmad-gate

Conversation

@pdurlej

@pdurlej pdurlej commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary\n\n- replace the stateful BMAD dependency invocation with a local read-only structural preflight\n- keep release/reference drift and state-cache write failures nonblocking\n- stop suggesting automatic BMAD check/sync remediation during task startup\n- verify installer behavior through the read-only X3 gate\n\n## Why\n\nThe previous compact gate invoked BMAD's sync helper even for a health check. In sandboxed Codex sessions that helper could write under ~/.codex, fail with PermissionError, and make X3/X4 appear unhealthy. That created unnecessary sync prompts and blocked normal work.\n\nThe new gate checks the installed local capability directly. Missing or structurally unusable BMAD files still fail closed. Update freshness remains advisory.\n\n## Verification\n\n- 107 deterministic tests passed using the same commands as main CI\n- X1, X2, X3, and X4 smoke checks all allow execution\n- X3 and X4 return bmad_status=ok with no remediation\n- the regression suite proves an unwritable BMADX state cache does not block X3 or suggest synchronization\n- git diff --check and Python byte-compilation passed\n\n## Scope\n\nThis PR intentionally excludes the unfinished value-study benchmark and its performance claims.

@pdurlej
pdurlej merged commit 89bbb1e into main Jul 16, 2026
6 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