Skip to content

fix: require CLI commands for all SDK methods#60

Merged
masnwilliams merged 1 commit intomainfrom
mason/require-full-cli-coverage
Jan 17, 2026
Merged

fix: require CLI commands for all SDK methods#60
masnwilliams merged 1 commit intomainfrom
mason/require-full-cli-coverage

Conversation

@masnwilliams
Copy link
Contributor

@masnwilliams masnwilliams commented Jan 17, 2026

Summary

  • Remove the "internal/admin" exception from the CLI coverage workflow
  • All SDK methods in api.md now require corresponding CLI commands with no exceptions

This fixes the issue where Agents.Auth, Agents.Auth.Invocations, and Credentials were being skipped as "internal/admin" when they should have CLI commands.


Note

Tightens the CLI coverage workflow constraints.

  • Updates update-cli-coverage.yml guidance to require CLI commands for every SDK method listed in api.md (no exceptions)
  • Clarifies that streaming methods may use different CLI implementations (e.g., follow flags)
  • Removes prior exception notes (internal/admin, etc.)

Written by Cursor Bugbot for commit 8e3d9be. This will update automatically on new commits. Configure here.

Remove the "internal/admin" exception that was causing the CLI coverage
workflow to skip implementing commands for Agents.Auth, Credentials, etc.
All SDK methods in api.md now require corresponding CLI commands.
@masnwilliams masnwilliams merged commit 8dc9875 into main Jan 17, 2026
5 checks passed
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

Comments