Skip to content

fix: bump courier-go SDK to v4.8.0 and fix explorer test#53

Closed
mikemilla wants to merge 3 commits intomainfrom
fix/bump-courier-go-sdk
Closed

fix: bump courier-go SDK to v4.8.0 and fix explorer test#53
mikemilla wants to merge 3 commits intomainfrom
fix/bump-courier-go-sdk

Conversation

@mikemilla
Copy link
Copy Markdown
Contributor

Summary

  • Bumps courier-go SDK from v4.7.0 to v4.8.0 to resolve 10 compilation errors in pkg/cmd/notification.go. The release PR (3.3.0) introduced notification commands (create, retrieve, archive, publish, replace, list-versions) that call SDK methods (New, Get, Archive, ListVersions, Publish, Replace) and reference param types (NotificationNewParams, NotificationGetParams, etc.) only available in v4.8.0.
  • Fixes internal/jsonview/explorer_test.go: removes duplicate require import and extracts orphaned marshalItemsToJSONArray assertions into a properly scoped TestMarshalItemsToJSONArray_WithHasRawJSON test with a rawJSONItem test double.

Test plan

  • go build ./... passes
  • go test ./... passes (all packages green)
  • CI should pass on this PR

Made with Cursor

stainless-app bot and others added 3 commits April 2, 2026 20:36
The release PR (3.3.0) introduced notification commands (create, retrieve,
archive, publish, replace, list-versions) that reference SDK methods only
available in courier-go v4.8.0. Upgrade the dependency from v4.7.0 to v4.8.0
so the build compiles.

Also fix explorer_test.go: remove duplicate require import and extract orphaned
marshalItemsToJSONArray assertions into their own test with a proper hasRawJSON
test double.

Made-with: Cursor
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch 12 times, most recently from 87a1988 to 1e9591e Compare April 9, 2026 16:16
Base automatically changed from release-please--branches--main--changes--next to main April 9, 2026 16:28
@xehl
Copy link
Copy Markdown
Member

xehl commented Apr 9, 2026

Closing — superseded by api-spec PR #91 (merged). Stainless regenerated the CLI release branch (PR #55), which is now fully green. Both the SDK version mismatch and the explorer test issue are resolved in the regenerated code.

@xehl xehl closed this Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants