fix: install.sh consent text claims manual skill-injection default — actual default is auto#36
Open
pstayets wants to merge 1 commit into
Open
Conversation
…ult is auto The install-time consent block described skill injection as 'on by default, manual mode' with skills 'installed once now and refreshed only when you run pilotctl update'. The real fresh-install default is AUTO: the docs consent page states 'auto (default on fresh install) — a reconcile pass runs every 15 minutes', skillinject's GetMode() returns ModeAuto when no persisted flag exists, and this script's own post-install banner says 'The daemon scans every 15 minutes and injects the Pilot Protocol skill'. The consent screen — the one place users grant this — was the only surface claiming manual. Align it with the actual behavior. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D2boz7GoTW34JmzymSjCpf
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Exact issue
The consent block in
install.shsays:This contradicts the actual default, which is AUTO, on three of the project's own surfaces:
/docs/consent#skillinject): "auto (default on fresh install) — A reconcile pass runs every 15 minutes."skillinject/config.go:GetMode()returnsModeAutowhen no persisted flag exists ("existing behaviour — live ticker").Skill injection writes instruction content into agent config directories (
~/.claude/…, OpenClaw, OpenHands, …) on a recurring background tick. The consent screen is precisely where the default must be stated accurately — understating it as "install once, manual" means users consent to less than what runs.Change
Consent text now states AUTO is the default with the 15-minute reconcile pass, and presents MANUAL/disable as the opt-outs. No behavior change — text only.
bash -npasses.Note for maintainers
skillinject/config.goalso declaresconst defaultMode = ModeManual("the mode used when no persisted value exists") directly above aGetMode()that returnsModeAutoin exactly that case — worth reconciling in code too, whichever direction you intend.🤖 Generated with Claude Code
https://claude.ai/code/session_01D2boz7GoTW34JmzymSjCpf