Repository navigation
Tell the Maintainer to Correct, Not Unset, an Uncontainable Prefix - #1909
Conversation
The drift message for a prefix registered where the host cannot contain said to re-run the installer to unset it, while a missing or broken deployed prefix is repaired and re-registered by that re-run instead. It now says to re-run the installer to correct it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The change is a narrow wording correction that improves accuracy of operator guidance without affecting behavior.
Review effort: Lite
Findings: None
What changed in this PR
Updates the --report drift message for an uncontainable CLAUDE_CODE_SHELL_PREFIX value to direct maintainers to re-run the installer “to correct it” (rather than “to unset it”), aligning the guidance with both stale/missing-prefix and uncontainable-prefix scenarios.
Changes:
- Reword the remediation guidance for a containment prefix that is configured on a host that cannot apply containment.
| File | Description |
|---|---|
| host-setup/agent-safety/claude/install.py | Adjusts the drift message to recommend correcting the prefix value via installer re-run. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
This answers a previously-missed finding on promotion #1907. The
--reportdrift message for this kit's prefix, registered where the host cannot contain, said to re-run the installer "to unset it". The same verdict also covers a missing or broken deployed prefix, which the re-run repairs and re-registers rather than unsets. It now says "to correct it", which holds on both paths.🤖 Generated with Claude Code