fix: disclose auto-star and add --no-star flag (fixes #4)#5
Open
alexey-max-fedorov wants to merge 1 commit into
Open
fix: disclose auto-star and add --no-star flag (fixes #4)#5alexey-max-fedorov wants to merge 1 commit into
alexey-max-fedorov wants to merge 1 commit into
Conversation
setup.sh was silently starring the repo using the user's gh credentials with no disclosure. This adds: - README: explicit disclosure that setup stars the repo + --no-star docs - setup.sh: --no-star flag to skip the gh star call - commands/setup.md: disclosure in the Step 1 question; Claude detects --no-star / "without starring" / "skip the star" phrases and passes the flag through to setup.sh - skills/fablize/SKILL.md: same disclosure + no-star option in the first-run onboarding AskUserQuestion Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AbKvt63WSYH5YfsY8A495C
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.
Fixes #4 —
setup.shwas silently starring the repo using the user'sghcredentials with all output suppressed and no prior disclosure.Changes
README.mdAdded a disclosure block under Install explaining that setup stars the repo via
ghcredentials, with--no-starusage shown.setup/setup.shReplaced the single positional arg with a proper arg parser.
--no-starcan now appear anywhere in the args; if present, thegh api -X PUTstar block is skipped entirely.commands/setup.md(the/fablize:setupcommand)Step 1 question now includes the star disclosure. Step 2 shows both variants of the
setup.shcall. Claude is instructed to detect--no-star, "without starring", "skip the star", "no star" etc. and pass--no-starthrough.skills/fablize/SKILL.md(auto-onboarding on first/fablize)Disclosure added to the onboarding question, a 4th "Local, no star" option added, and a note that
--no-starphrases anywhere in the invocation should propagate tosetup.sh.Test plan
setup.sh local— stars as beforesetup.sh local --no-star— skips thegh apicall entirelysetup.sh global --no-star— same, global scope/fablize:setup --no-star— Claude detects the flag and passes it through/fablize:setup(plain) — disclosure visible in Step 1 question before consent/fablizerun (SKILL.md §0) — disclosure in onboarding question, "Local, no star" option works