Skip to content

tests: run the brigade skill's ticket add and fire lines against the tool - #187

Merged
creedants merged 8 commits into
mainfrom
landing/e176
Oct 12, 2026
Merged

creedants merged 8 commits into
mainfrom
landing/e176

Conversation

@creedants

Copy link
Copy Markdown
Owner

A new test runs the ticket add and fire command lines the brigade skill shows, so an example cannot drift from what brigade.py accepts without the suite failing.

  • tests/test_skill_commands.py reads t3/added/brigade/SKILL.md, fills each placeholder in the alternative it runs, and runs every line against a scratch store. A line the tool refuses fails the test and names the line number in the skill.
  • The test fails, not skips, when it finds fewer than seven command lines.
  • Three regression tests run on a fixed sample held in the module, so a valid edit to a live example does not fail them.
  • docs/guide.md now names edits.tsv among the tables open writes.

Known limits. The scanner reads physical lines and does not follow a continued shell command. A bare command mention with no flags is skipped. Only the five-kind choice runs every alternative; other choices run their first. Tilde fences are not recognized.

Reviewed at 52c106e5.

@creedants
creedants merged commit 641c85e into main Oct 12, 2026
2 checks passed
@creedants
creedants deleted the landing/e176 branch October 12, 2026 07:27
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