Bump fizzy SDK to v0.2.2#167
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the CLI to the Fizzy SDK v0.2.2, adds a new fizzy identity timezone-update --timezone NAME command, and tightens tests to ensure SDK-backed mutation response bodies are exposed via the CLI’s JSON envelope.
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
Changes:
- Bump
github.com/basecamp/fizzy-sdk/godependency tov0.2.2. - Add
identity timezone-updatecommand and document it in SURFACE + skill docs. - Add shared test assertions and extend mutation tests to validate surfaced response bodies.
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| SURFACE.txt | Registers the new identity timezone-update command + flags in the surfaced CLI contract. |
| skills/fizzy/SKILL.md | Documents the new identity timezone update capability. |
| internal/commands/identity.go | Implements fizzy identity timezone-update --timezone ... using the SDK. |
| internal/commands/identity_test.go | Adds coverage for timezone update command behavior and argument validation. |
| internal/commands/response_assertions_test.go | Adds a shared helper for asserting JSON response envelopes in tests. |
| internal/commands/user_test.go | Extends user update tests to assert surfaced mutation response bodies. |
| internal/commands/comment_test.go | Extends comment update tests to assert surfaced mutation response bodies. |
| internal/commands/column_test.go | Extends column update tests to assert surfaced mutation response bodies. |
| internal/commands/card_test.go | Extends card move tests to assert surfaced mutation response bodies. |
| internal/commands/board_test.go | Extends board update tests to assert surfaced mutation response bodies. |
| go.mod | Bumps fizzy SDK module version to v0.2.2. |
| go.sum | Updates checksums for the bumped fizzy SDK version. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
4 issues found across 12 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
|
Fixed — addressed Copilot review feedback by surfacing timezone response bodies and asserting response-only IDs in mutation tests. |
|
Fixed — addressed cubic review feedback by surfacing timezone response bodies and asserting response-only IDs in mutation tests. |
Summary
fizzy identity timezone-update --timezone NAMEfor the new timezone SDK operationTests
Summary by cubic
Adds
fizzy identity timezone-updateand ensures SDK-backed update/move commands surface mutation response bodies, with a fallback to requested values when the API returns no body.New Features
identity timezone-updateto set your timezone for the current account (requires--timezone, e.g., America/New_York)Dependencies
github.com/basecamp/fizzy-sdk/goto v0.2.2Written for commit b2155eb. Summary will update on new commits.