Skip to content

chore: deprecate qawolf automate - #1625

Merged
Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
mainfrom
chore/wiz-12173-remove-qawolf-automate
Sep 18, 2026
Merged

Atchyut Preetham Pulavarthi (theonly1me) merged 1 commit into
mainfrom
chore/wiz-12173-remove-qawolf-automate

Conversation

@theonly1me

Copy link
Copy Markdown
Member

Overview of Changes

qawolf automate is deprecated. The command has no hand-written code in this repo: registerPublicApiCommands mints it from the automate contract in @qawolf/api-contracts, and because that contract name has no dot it registers as a top-level command. The only in-repo lever is the generator's skip list.

  • Skip the automate contract so the generator no longer registers the command.
  • Split the skip list into handWrittenContractNames and deprecatedContractNames, since its comment gives serving a hand-written command as the only reason to skip a contract.
  • Drop request automation of draft flows from the qawolf-cli skill description and regenerate skills/qawolf-cli/SKILL.md.
  • Add a test that automate is not registered, so removing the skip-list entry fails.
  • Add a changeset with a minor bump.

Breaking Changes

  • qawolf automate no longer exists. Running it reports error: unknown command 'automate' and exits 2. The public API endpoint behind it is unchanged.

Testing

bun run typecheck
bun run lint
bun run format:check
bun run knip
bun run test

All pass, 2565 tests and 0 failures.

  • bun run dev -- --help no longer lists automate.
  • bun run dev -- automate reports error: unknown command 'automate' and exits 2.
  • skills/qawolf-cli/SKILL.md regenerates to the committed content, with the only changes being the description line and the qawolf automate table row.

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: qawolf/cli/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 90c83d45-ba10-4d36-b16b-7b37dda0b8bf

📥 Commits

Reviewing files that changed from the base of the PR and between 952de68 and f947424.

⛔ Files ignored due to path filters (1)
  • src/commands/__snapshots__/help.test.ts.snap is excluded by !**/*.snap
📒 Files selected for processing (5)
  • .changeset/remove-automate-command.md
  • skills/qawolf-cli/SKILL.md
  • src/commands/program.test.ts
  • src/commands/qawolfCliSkill.template.md
  • src/domains/publicApi/skippedContracts.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


Walkthrough

The deprecated qawolf automate command is removed from CLI skill descriptions and is verified as absent from the top-level command list. Skipped public API contracts now separate hand-written contracts from deprecated contracts and include automate in the deprecated group. A changeset documents the minor release and notes that the underlying public API endpoint remains unchanged.

Changes

Area Change
CLI surface Removed automate references from skill descriptions.
Contract handling Added a separate deprecated contract list containing automate.
Validation Added a test for the missing top-level command.
Release notes Added a minor-release changeset.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to f9474

The deprecated command removal and related documentation and test updates are consistent, with no actionable merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title follows Conventional Commits, uses an allowed type, uses imperative wording, identifies the deprecated qawolf automate command, and is under 72 characters.
Description check ✅ Passed The description explains the change, breaking behavior, implementation approach, and test results. It omits the template's separate Checklist section and does not report the build command, but it is m…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) merged commit 5fd077f into main Sep 18, 2026
7 checks passed
@theonly1me
Atchyut Preetham Pulavarthi (theonly1me) deleted the chore/wiz-12173-remove-qawolf-automate branch September 18, 2026 20:40
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