Skip to content

fix: env flag parsing and document ACP exec backend - #8

Merged
capy-ai[bot] merged 2 commits into
feat/skip-subagent-headlessfrom
capy/fix-skip-headless-review
Nov 13, 2026
Merged

capy-ai[bot] merged 2 commits into
feat/skip-subagent-headlessfrom
capy/fix-skip-headless-review

Conversation

@MaheshtheDev

Copy link
Copy Markdown
Member

Problem: Review of feat/skip-subagent-headless found that env flags only accepted exact values, and that the opt-in ACP exec backend being treated as headless was undocumented.

Changes

  • Env flags accept true/1/yes/on and false/0/no/off, and the env value wins both ways.
  • The README notes that MUSE_CODE_ACP_BACKEND=exec sessions count as headless unless captureHeadless: true is set.

Verification: npm test 11/11, and all five env combinations checked by hand.

Targets Dhravya's feat/skip-subagent-headless branch, which has no PR yet.

MaheshtheDev and others added 2 commits November 12, 2026 00:14
…ocument the ACP exec backend

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
@capy-ai
capy-ai Bot marked this pull request as ready for review November 13, 2026 00:54
@capy-ai
capy-ai Bot merged commit 03d1f7e into feat/skip-subagent-headless Nov 13, 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