Skip to content

fix(setup): tolerate unavailable app-server during state probe - #112

Open
wstczyw wants to merge 1 commit into
sendbird:mainfrom
wstczyw:codex/fix-setup-spawn-eperm-20260919
Open

wstczyw wants to merge 1 commit into
sendbird:mainfrom
wstczyw:codex/fix-setup-spawn-eperm-20260919

Conversation

@wstczyw

@wstczyw wstczyw commented Sep 19, 2026

Copy link
Copy Markdown

Summary

  • keep cc:setup usable when the Codex app-server probe exits before responding (including Windows spawn EPERM)
  • continue verifying plugin state directly and report the probe failure as diagnostic context
  • preserve the existing repair guidance when direct plugin-state access is unavailable

Why

A transient or sandboxed app-server failure currently makes setup report the plugin as unavailable even when the plugin state directory is directly writable. That blocks setup and review-gate configuration on affected Windows hosts.

Tests

  • node --test --test-name-pattern="setup remains ready when the app-server reports spawn EPERM|setup reports plugin-state repair guidance" tests/integration/claude-companion.test.mjs
  • npm run lint -- --no-warn-ignored scripts/claude-companion.mjs tests/integration/claude-companion.test.mjs
  • npm run typecheck
  • npm run check:version-sync
  • npm run check:changelog

The full integration suite requires an authenticated Claude CLI in the test environment; the targeted setup regressions pass.

@wstczyw
wstczyw force-pushed the codex/fix-setup-spawn-eperm-20260919 branch from 19266a3 to 0f7bc8c Compare September 19, 2026 02:12
@wstczyw
wstczyw force-pushed the codex/fix-setup-spawn-eperm-20260919 branch from 0f7bc8c to c155e93 Compare September 19, 2026 02:13
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