Skip to content

fix: sanitize Claude auth subprocess environment - #119

Open
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/sanitize-claude-auth-env
Open

fix: sanitize Claude auth subprocess environment#119
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/sanitize-claude-auth-env

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • sanitize the environment used by Claude authentication status and login subprocesses
  • remove inherited CLAUDECODE markers case-insensitively so auth works when Claudify runs under Claude Code
  • add regression coverage for both status and PTY-backed login paths

Verification

  • npm run build — passed: TypeScript compilation completed
  • npm test — passed: 110 tests
  • git diff --check — passed

Risk

  • Low: the change only copies the existing auth subprocess environment while omitting the nested-Claude marker, matching the response subprocess behavior.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

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