Skip to content

fix(pm): convert inline backtick preflights to multiline bash blocks#208

Merged
veinkr merged 2 commits intomainfrom
spec-fix
Feb 22, 2026
Merged

fix(pm): convert inline backtick preflights to multiline bash blocks#208
veinkr merged 2 commits intomainfrom
spec-fix

Conversation

@veinkr-bot
Copy link
Copy Markdown
Contributor

@veinkr-bot veinkr-bot commented Feb 22, 2026

Summary

Fixes a permission checker error that occurred when running pm skills with preflight commands containing quoted characters in flag values (e.g. --jq '...').

Changes

  • Converted all inline backtick preflight syntax (!...``) to multiline !```bash blocks across all affected pm skills:
    • spec-abandon: 3 preflights converted
    • spec-status: 3 preflights converted
    • spec-next: 3 preflights converted
    • spec-init: 3 preflights converted
    • spec-list: 1 preflight converted
    • spec-decompose: 1 preflight converted
    • spec-plan: 1 preflight converted

Related Issues

Follows the bash block rule established in .claude/rules/skill-bash-blocks.md and the fix from #207.

Testing

  • Tested locally
  • Docker build/container works (if applicable)
  • Manual testing completed

Checklist

  • Code follows project conventions (shell script style, Python formatting)
  • Documentation updated (README.md, CLAUDE.md, or relevant docs)
  • No breaking changes (or clearly documented with migration guide if unavoidable)
  • GitHub Actions workflows pass (if modified)
  • Commit messages are clear and follow conventional commit style

@veinkr veinkr marked this pull request as ready for review February 22, 2026 03:42
@veinkr-bot veinkr-bot changed the title spec fix fix(pm): convert inline backtick preflights to multiline bash blocks Feb 22, 2026
@veinkr-bot veinkr-bot added the pr-updated PR description has been updated label Feb 22, 2026
@veinkr veinkr merged commit ac44ee4 into main Feb 22, 2026
@veinkr veinkr deleted the spec-fix branch February 22, 2026 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-updated PR description has been updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants