docs: reconcile --gitignore-global wording with shipped --visibility=global-default - #85
Merged
Merged
Conversation
Closes #65. The canonical instructions (.agent/AGENTS.md, symlinked as AGENTS.md / CLAUDE.md / .agent/CLAUDE.md) still described machine-wide git excludes as a *planned, not-yet-exposed* `--gitignore-global` flag. That capability shipped as `agent-init init --visibility=global-default` (#52/#61, plus the hidden/local modes in #70), so the doc contradicted the binary and risked someone re-implementing it under the old flag name. - Add `--visibility` and `--private` rows to the "Flags on init" table. - Replace the "Planned `--gitignore-global`" section with "Visibility and global excludes", describing the shipped modes and noting global-default is the superseding form. Machine-wide footgun warning preserved. - Update the kebab-case flag example and the global-git-config guardrail to reference `--visibility=global-default`. - Reframe the "Global gitignore is a footgun" note as shipped, keeping the warning. README.md and docs/cli.md already document --visibility (incl. global-default) with no stale `--gitignore-global` mentions; verified, no change needed there. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Closes #65.
Problem
The canonical instructions in
.agent/AGENTS.md(symlinked asAGENTS.md/CLAUDE.md/.agent/CLAUDE.md) still described machine-wide git excludes as a planned, not-yet-exposed--gitignore-globalflag. That capability has shipped asagent-init init --visibility=global-default(#52 / #61, alongside thelocalandhiddenmodes in #70). The doc contradicted the binary and risked a contributor re-implementing it under the old flag name.Changes (all in
.agent/AGENTS.md)--visibilityand--privaterows.--gitignore-global" section → replaced with "Visibility and global excludes", describing the four shipped modes and noting--visibility=global-defaultis the superseding form. Machine-wide footgun warning kept.--gitignore-global→ real shipped flags (--no-git,--dry-run,--visibility).--visibility=global-default.Two
--gitignore-globalmentions remain by design, both framing it as the old name superseded by--visibility=global-default.Not changed (verified consistent)
README.mdanddocs/cli.mdalready document--visibilityincludingglobal-default, with no stale--gitignore-globalreferences — the issue's third acceptance criterion was already satisfied.Notes
Touches this repo's own
.agent/canonical instructions, normally off-limits without an explicit ask — issue #65 is that authorization. Docs-only; no code or templates changed, so no golden/smoke impact.go vetandgo test ./...pass.🤖 Generated with Claude Code