From 8e3d9be2e555559533d9d935ce7e6a57c06bf29e Mon Sep 17 00:00:00 2001 From: Mason Williams Date: Sat, 17 Jan 2026 15:55:36 -0500 Subject: [PATCH] fix: require CLI commands for all SDK methods 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. --- .github/workflows/update-cli-coverage.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/workflows/update-cli-coverage.yml b/.github/workflows/update-cli-coverage.yml index 50b4c17..44cc922 100644 --- a/.github/workflows/update-cli-coverage.yml +++ b/.github/workflows/update-cli-coverage.yml @@ -287,9 +287,8 @@ jobs: # Constraints - ALWAYS update the SDK version in go.mod - this is the primary purpose - ALWAYS perform the full enumeration (Steps 2-4) - this is critical for finding gaps - - Only implement genuinely missing CLI functionality - - Internal/admin methods may not need CLI commands - - Streaming methods may have different CLI implementations + - ALL SDK methods in api.md MUST have corresponding CLI commands - no exceptions + - Streaming methods may have different CLI implementations (e.g., follow flags) - Even if no coverage gaps are found, still create a PR for the SDK version bump - Ensure code compiles before pushing " --model opus-4.5 --force --output-format=text