-
Notifications
You must be signed in to change notification settings - Fork 130
Small changes to make dev / test work in Claude Code Web #147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Adds a conditional setup section to CLAUDE.md that guides Claude Code Web sessions through DNS configuration, mise installation, and tool setup. Local sessions with mise already configured can skip this section. https://claude.ai/code/session_01SU7inAhzJ74XaAH3Lxc7g1
The environment variable CLAUDE_CODE_REMOTE=true is set in web/cloud sessions, providing a reliable way to detect when setup steps are needed. https://claude.ai/code/session_01SU7inAhzJ74XaAH3Lxc7g1
Entire-Checkpoint: adac80fffbb8
5461166 to
aa45335
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Improves the developer/test experience in Claude Code Web by adding remote-session environment setup and ensuring git commits made during tests don’t require GPG signing.
Changes:
- Add a SessionStart hook script for Claude Code Web that configures DNS and installs/trusts/installs tools via mise.
- Disable
commit.gpgsignin unit and integration tests to avoid failures when users have global GPG signing enabled. - Document the remote-session setup behavior in
CLAUDE.md.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
cmd/entire/cli/strategy/hooks_test.go |
Disables GPG signing in git-based unit tests before making commits. |
cmd/entire/cli/integration_test/testenv.go |
Sets repo config to disable GPG signing for integration test commits. |
CLAUDE.md |
Documents remote-session auto-setup via Claude hooks and script. |
.claude/settings.json |
Adds a SessionStart command hook to run remote environment setup. |
.claude/scripts/remote-setup.sh |
New script to configure DNS and install/trust/install mise tools in remote sessions. |
Comments suppressed due to low confidence (1)
.claude/settings.json:14
- The SessionStart hook still invokes
go run ...directly, so it will use the systemgounless mise’s shims are already on PATH. Since the remote setup runs as a separate command, its exported PATH won’t carry over here. Consider invoking the hook via mise (e.g.,mise exec ...) or using a single wrapper command that sources the setup before runninggo run, so the intended Go version is guaranteed.
{
"type": "command",
"command": "go run ${CLAUDE_PROJECT_DIR}/cmd/entire/main.go hooks claude-code session-start"
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 6 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Entire-Checkpoint: 69c601b407e4
Entire-Checkpoint: 5504c1f4266f
Even when trying to use Claude Code Web with a full access environment mise wasn't working and an older go lang version was used. This makes everything work by doing these things:
CLAUDE_CODE_REMOTEmise trustand thenmise installThis should give a fully working Claude Code Web session that can run tests and everything.
Note
Medium Risk
Touches dev-environment bootstrapping by modifying
/etc/resolv.confand executing a remote installer script, which could affect container behavior if misconfigured; code changes are otherwise test/dev-only.Overview
Improves Claude Code Web sessions by adding a
SessionStarthook that runs a new.claude/scripts/remote-setup.shto configure container DNS, install and runmise(trust/install), and persistmiseenvironment exports for subsequent commands.Stabilizes integration/strategy tests that create git commits by explicitly disabling
commit.gpgsign(both viagit configin CLI-driven tests and via go-git repo config inTestEnv.InitRepo) to prevent failures when global GPG signing is enabled.Written by Cursor Bugbot for commit dc58c0b. This will update automatically on new commits. Configure here.