Skip to content

Commit 8dc9875

Browse files
authored
Merge pull request #60 from kernel/mason/require-full-cli-coverage
fix: require CLI commands for all SDK methods
2 parents 1372840 + 8e3d9be commit 8dc9875

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)