Skip to content

fix: Remove duplicate options in specify.md#1765

Open
ryo8000 wants to merge 1 commit intogithub:mainfrom
ryo8000:fix/remove-duplicate-options
Open

fix: Remove duplicate options in specify.md#1765
ryo8000 wants to merge 1 commit intogithub:mainfrom
ryo8000:fix/remove-duplicate-options

Conversation

@ryo8000
Copy link

@ryo8000 ryo8000 commented Mar 5, 2026

Description

  • The scripts frontmatter in specify.md included --json / -Json flags directly in the script path assignment (e.g., scripts/bash/create-new-feature.sh --json "{ARGS}").
  • This caused the flag to appear twice when an agent ran the script — once baked into {SCRIPT} and again in the usage examples within the command body (e.g., {SCRIPT} --json --number 5 ...), producing commands like create-new-feature.sh --json "$ARGUMENTS" --json --number 5 ....
  • The fix removes --json / -Json from the scripts frontmatter, so {SCRIPT} resolves to just the script path.

Testing

  • Tested locally with uv run specify --help
  • Ran existing tests with uv sync && uv run pytest
  • Tested with a sample project (if applicable)

AI Disclosure

  • I did not use AI assistance for this contribution
  • I did use AI assistance (describe below)

@ryo8000 ryo8000 requested a review from mnriem as a code owner March 5, 2026 14:57
Copilot AI review requested due to automatic review settings March 5, 2026 14:57
Copy link
Contributor

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

Removes duplicated --json / -Json options from the specify.md frontmatter script definitions so that usage examples don’t result in repeated flags when {SCRIPT} is expanded/executed.

Changes:

  • Update the scripts frontmatter entries to contain only the script path plus "{ARGS}", removing baked-in JSON flags.

💡 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