Skip to content

feat: make maintainer command output and help self-explanatory - #272

Merged
LadyBluenotes merged 2 commits into
fix/maintainer-first-runfrom
feat/maintainer-ergonomics
Sep 11, 2026
Merged

LadyBluenotes merged 2 commits into
fix/maintainer-first-runfrom
feat/maintainer-ergonomics

Conversation

@LadyBluenotes

Copy link
Copy Markdown
Member

🎯 Changes

Stacked on #271. Every maintainer action shared one flat list of sixteen flags in --help, and the text output left the maintainer guessing what each command wrote.

  • intent maintainer --help prints the actions in the order a maintainer runs them, each with a one-line summary and a Writes: line. intent maintainer <action> --help prints that action's usage and only its options. Bare intent maintainer prints the overview and exits 1 instead of a cac argument error.
  • Unsupported options are reported in their kebab-case spelling (--plugin-name, not --pluginName) with a pointer to the action's help.
  • maintainer add lists every file it wrote and names the domain map that still needs developer tasks. maintainer sync prints each synchronized path, says Nothing to synchronize. on a no-op, and labels the consumer install commands.
  • maintainer status and check show the item kind, the changed files, or no recorded review, matching what intent review already printed.
  • maintainer check validates each skills root once instead of once per skill directory.
  • maintainer setup --distribution repo reports every missing input (repository, plugin name, skills) in one error instead of one per attempt.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 51 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 1a2f5ee3-6bb1-43df-b06c-f2c620fc35a6

📥 Commits

Reviewing files that changed from the base of the PR and between dc4857b and cef37f9.

📒 Files selected for processing (8)
  • .changeset/maintainer-ergonomics.md
  • packages/intent/src/cli.ts
  • packages/intent/src/commands/maintainer.ts
  • packages/intent/src/maintainer/add.ts
  • packages/intent/src/maintainer/distribution.ts
  • packages/intent/tests/cli.test.ts
  • packages/intent/tests/integration/packed-release.test.ts
  • packages/intent/tests/maintainer.test.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Print an ordered overview for maintainer --help and per-action options for
maintainer <action> --help. Name every file add and sync write, label the
consumer install commands, show changed files and missing reviews in status
and check, validate each skills root once, report all missing repository
distribution inputs together, and print unsupported options in kebab-case.
@LadyBluenotes
LadyBluenotes force-pushed the feat/maintainer-ergonomics branch from cd1b435 to cef37f9 Compare September 11, 2026 22:30
@LadyBluenotes
LadyBluenotes merged commit 33b5e8b into main Sep 11, 2026
8 of 12 checks passed
@LadyBluenotes
LadyBluenotes deleted the feat/maintainer-ergonomics branch September 11, 2026 22:32
@codspeed

codspeed Bot commented Sep 11, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

🆕 9 new benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
🆕 scans a consumer workspace N/A 98.2 ms N/A
🆕 checks a shipped skills tree N/A 161.9 ms N/A
🆕 reports workspace drift N/A 98.7 ms N/A
🆕 loads a direct dependency skill N/A 32.6 ms N/A
🆕 loads a direct dependency from a large workspace N/A 41.8 ms N/A
🆕 loads direct dependency content as json N/A 44.9 ms N/A
🆕 empty node process (baseline) N/A 4.2 ms N/A
🆕 intent --help N/A 4.2 ms N/A
🆕 reports workspace drift with shared artifacts N/A 99.3 ms N/A

Comparing feat/maintainer-ergonomics (cef37f9) with main (2823716)

Open in CodSpeed

@github-actions github-actions Bot mentioned this pull request Sep 12, 2026
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