Skip to content

Commit 8e3d9be

Browse files
committed
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.
1 parent 1372840 commit 8e3d9be

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

.github/workflows/update-cli-coverage.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -287,9 +287,8 @@ jobs:
287287
# Constraints
288288
- ALWAYS update the SDK version in go.mod - this is the primary purpose
289289
- ALWAYS perform the full enumeration (Steps 2-4) - this is critical for finding gaps
290-
- Only implement genuinely missing CLI functionality
291-
- Internal/admin methods may not need CLI commands
292-
- Streaming methods may have different CLI implementations
290+
- ALL SDK methods in api.md MUST have corresponding CLI commands - no exceptions
291+
- Streaming methods may have different CLI implementations (e.g., follow flags)
293292
- Even if no coverage gaps are found, still create a PR for the SDK version bump
294293
- Ensure code compiles before pushing
295294
" --model opus-4.5 --force --output-format=text

0 commit comments

Comments
 (0)