Skip to content

docs(basecamp): cross-reference Agent Invariants from CLI Introspection#3

Open
vmpn wants to merge 1 commit intobasecamp:mainfrom
vmpn:docs/global-flags-note
Open

docs(basecamp): cross-reference Agent Invariants from CLI Introspection#3
vmpn wants to merge 1 commit intobasecamp:mainfrom
vmpn:docs/global-flags-note

Conversation

@vmpn
Copy link
Copy Markdown

@vmpn vmpn commented Apr 26, 2026

Summary

Adds a one-sentence footnote at the end of the CLI Introspection section of skills/basecamp/SKILL.md, noting that persistent global flags called out in the Agent Invariants rules (--jq, --agent, --styled, --verbose, …) are documented only at the root (basecamp --agent --help) and won't appear in a subcommand's inherited_flags. They still apply on every subcommand.

Why

Agent Invariant rule #1 already prescribes "Never pipe to external jq — use --jq instead". An agent following that rule will use --jq regardless of what <subcommand> --help shows. But an agent that inspects per-subcommand help to confirm the flag is supported can incorrectly conclude it isn't, because inherited_flags doesn't list root-level persistent flags. This footnote closes that discoverability gap by cross-referencing back to the Invariants.

Pure docs change — no behavior change.

Test plan

  • Diff inspected; only skills/basecamp/SKILL.md is touched (+2 lines)
  • Markdown renders correctly in preview

Created with assistance of AI agent with human review.

Copilot AI review requested due to automatic review settings April 26, 2026 17:57
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Basecamp skill documentation to clarify that persistent/global CLI flags are only listed in the root basecamp --agent --help output and may not appear under each subcommand’s inherited_flags, even though they still apply.

Changes:

  • Add a note in the “CLI Introspection” section explaining why global flags might be missing from per-subcommand help output and where to find the canonical list.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Persistent flags called out in the Agent Invariants ("Never pipe to
external jq — use --jq instead", etc.) are documented only at the
root (`basecamp --agent --help`) and don't appear in per-subcommand
`inherited_flags`. Adds a one-sentence footnote at the end of the
CLI Introspection section noting this, so an agent inspecting a
subcommand's help doesn't conclude a global flag is unsupported.

Pure docs change — no behavior change.
@vmpn vmpn force-pushed the docs/global-flags-note branch from 3005ae6 to b80f1ec Compare April 26, 2026 18:00
@vmpn vmpn changed the title docs(basecamp): note that persistent global flags aren't repeated in subcommand inherited_flags docs(basecamp): cross-reference Agent Invariants from CLI Introspection Apr 26, 2026
@vmpn vmpn requested a review from Copilot April 26, 2026 18:02
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants