Skip to content

Docs: config/flag parity fixes - #119

Merged
tmashara merged 1 commit into
mainfrom
claude/docs-parity
Nov 2, 2026
Merged

tmashara merged 1 commit into
mainfrom
claude/docs-parity

Conversation

@tmashara

@tmashara tmashara commented Nov 2, 2026

Copy link
Copy Markdown
Owner

Doc fixes from a flag/config parity audit:

  • beforeShot: the example config annotates a top-level beforeShot (commented out, like lottieFrame), and the README's config prose lists it next to before. The README says every field is annotated in the example, which wasn't true for this one.
  • "What flags can't" list: the README no longer lists extra selectors to hide, since --hide exists.
  • Precedence: the README's precedence sentence and line 3 of the example config now say --mask and --hide add to the config's lists instead of replacing them.
  • Action routes input: the description warns that the input splits on spaces, so '/route @sel' doesn't work there and selectors belong in the config. docs/github-action.md already says this.

Docs only. format:check and lint pass.

🤖 Generated with Claude Code

The example config and README now cover beforeShot, the "flags can't" list drops
hide selectors (--hide exists), and the precedence notes say --mask and --hide add to
the config's lists. The action's routes input warns that it splits on spaces, so
'/route @sel' belongs in the config.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tmashara
tmashara merged commit e831d7a into main Nov 2, 2026
3 checks passed
@tmashara tmashara mentioned this pull request Nov 2, 2026
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.

1 participant