Skip to content

docs(skill): document the new runner surface - #1515

Merged
Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
feat/nova-1574-ship-only-changed-filesfrom
docs/nova-1574-document-the-new-runner-surface
Aug 21, 2026
Merged

docs(skill): document the new runner surface#1515
Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
feat/nova-1574-ship-only-changed-filesfrom
docs/nova-1574-document-the-new-runner-surface

Conversation

@theonly1me

Copy link
Copy Markdown
Member

Overview of Changes

packages/tester-skills reads skills/qawolf-cli/SKILL.md off disk at pod start and hands it to Tester verbatim, so the runner guide is the contract Tester works from. Four commands and three flags landed in this stack without it, and it still told a caller that a runner's first call has to be a whole run.

  • Document inspect's three subcommands, the one failure that covers three causes, and why it beats reading a value back out of the console stream
  • Document import-package, its --package-version flag and why it needs a live run
  • Document stop-run against terminate, which end different things and are told apart only by their names
  • Say that a --lines selection is the one call that does not need a run first, since the runner starts a browser for it
  • Correct runner run's own description, which still claimed it ships the current directory's files
  • Add the new commands to the end-to-end walkthrough
  • Add a minor changeset covering the whole runner surface, naming both breaking changes and the failureReason move

Testing

bun run typecheck
bun run lint --max-warnings 0
bun run format:check
bun run knip
bun run test

bun test gives 1810 pass 0 fail. The other five give exit 0.

  • bunx changeset status reports @qawolf/cli bumping at minor and nothing at major.
  • bun run generate picks up the corrected runner run description in both SKILL.md and the help snapshot, so the table and --help agree.
  • Added lines, excluding bun.lock and snapshots: 89.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 587e82e5-fb0b-4420-a865-03ef03420d68

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) merged commit ad44380 into main Aug 21, 2026
7 checks passed
@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) deleted the docs/nova-1574-document-the-new-runner-surface branch August 21, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants