-
Notifications
You must be signed in to change notification settings - Fork 2
feat(sdk): add Go client SDK with full API-tree parity #434
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
jfwoods
wants to merge
63
commits into
main
Choose a base branch
from
go-sdk
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
834a093
feat(sdk): add Go client SDK with full API-tree parity
jfwoods 3089c71
docs(sdk): add Go SDK documentation pages
jfwoods 61b436f
ci(sdk): wire Go SDK into CI and update AGENTS.md
jfwoods 28eb1cf
chore: trim verbose comments across SDK, Makefile, and docs
jfwoods b66413a
fix(sdk): resolve 18 golangci-lint findings in Go SDK
jfwoods 38b2397
refactor(sdk): shrink Go SDK diff by 114 lines, zero functionality loss
jfwoods 5a480bf
fix(sdk): address PR #434 review feedback — CI blockers + 51 review f…
jfwoods fc6346e
fix(sdk): address pre-push review round 1 — stream close race, tests,…
jfwoods e783f5b
fix(sdk): address pre-push review round 2 — terminal SSE errors, code…
jfwoods 6e166da
fix(sdk): address pre-push review round 3 — cursor ceiling honesty, A…
jfwoods 4b199b6
fix(sdk): address pre-push review round 4 — 64-bit codegen mapping, L…
jfwoods 978451c
fix(sdk): address pre-push review round 5 — Array(UInt8) round-trip, …
jfwoods 3a26401
fix(sdk): address pre-push review round 6 — e2e container gate, modul…
jfwoods 10cb253
test(sdk): guard conformance request capture with a mutex
jfwoods cade9d7
docs(sdk): address pre-push review round 7 — channel error visibility…
jfwoods 791c198
fix(sdk): address pre-push review round 8 — dead param, error wrappin…
jfwoods 478b215
fix(sdk): address pre-push review round 9 — SSE error delivery, test …
jfwoods a841f1a
fix(sdk): address pre-push review round 10 — terminal StatusClosed, d…
jfwoods 6a4e189
fix(sdk): address verified Codex review findings — Events() buffering…
jfwoods c88d1b1
fix(sdk): apply verified cavecrew/ponytail wave findings
jfwoods 697e978
Merge remote-tracking branch 'origin/main' into go-sdk
jfwoods c2c1021
docs: address pre-push review round 11 — stale changelog entry, ARGS/…
jfwoods 0ee1734
docs: address pre-push review round 12 — E2E lifecycle, lint install,…
jfwoods 51d8b45
docs: address pre-push review round 13 — 429 sweep, gotestsum format,…
jfwoods 4ee4f1f
docs: address pre-push review round 14 — not_like is rejected, not ca…
jfwoods 2d043da
Merge remote-tracking branch 'origin/main' into go-sdk
jfwoods 25e36cc
docs: restore branch edits lost in the merge, extend path-prefix docs…
jfwoods 1158ff3
test(sdk): pin BaseURL path-prefix support; qualify the 401 rows
jfwoods 0a46251
fix(sdk): clamp Retry-After before overflow; unalias insert-result po…
jfwoods 9da52dd
test(sdk): pin the fetchNextTyped marshal-error branch
jfwoods 4cc229a
docs: finish the TS side of the pagination parity sweep
jfwoods 6d29538
docs: flag the TS codegen Decimal mismatch; fix the Go release framing
jfwoods 7b85e06
chore(docs): revising Go SDK documentation
jfwoods d878791
Merge remote-tracking branch 'origin/main' into go-sdk
jfwoods 59b27bb
fix(sdk): bring the Go SDK current with main's API and SSE changes
jfwoods 5d26c34
docs(changelog): note the Go SDK's Headers option in the unreleased e…
jfwoods 90d59ef
style(docs): unwrap hard-wrapped prose in the Go SDK pages
jfwoods c460c02
build(lint): put the shared conformance runner under Biome
jfwoods 941b097
style(docs): unwrap the Go SDK aside in sdk/index.mdx
jfwoods c538ee3
build(deps): track the nested clients/go module in Dependabot
jfwoods 167e16d
build(labels): map the Go SDK paths to area/sdk
jfwoods 73edcbe
test(cov): give the Go SDK its own coverage floor
jfwoods 3ed1698
fix(sdk): compare stream filter timestamps as instants, not as text
jfwoods 3831741
fix(sdk): address unresolved CodeRabbit review threads
jfwoods ed2eace
docs(cov): compress the nested-module coverage comments
jfwoods 20bb752
refactor(sdk/go): cut comment bloat and mechanical verbosity
jfwoods 527dfea
refactor(sdk): compact the conformance fixture and both wire runners
jfwoods 059ad16
docs(go-sdk): cut duplication and padding from the Go SDK docs
jfwoods 0b8e450
test(sdk): consolidate Go SDK test setup and collapse sibling cases
jfwoods 29c6b6b
fix(sdk): replay a missed terminal stream error to late subscribers
jfwoods 3947df3
Merge remote-tracking branch 'origin/main' into go-sdk
jfwoods 48a24e8
fix(sdk): address the latest CodeRabbit review round
jfwoods 6c94796
fix: address the out-of-diff findings from CodeRabbit review 5020447247
jfwoods e17152f
test(sdk): cover the Go codegen CLI's argv parsing and schema fetch
jfwoods 50bdc2f
refactor(sdk): topic-first SDK docs + one make target family
jfwoods 23c58e2
fix(sdk): address pre-push review findings on the SDK docs merge
jfwoods 4094272
style(docs): updating copywriting on some pages
jfwoods f778e41
style(docs): updating copywriting on some pages again
jfwoods ce066c9
fix(sdk): address CodeRabbit findings on the SDK docs merge
jfwoods ff35c8b
fix(sdk): harden wavehouse-codegen and correct three stale docs claims
jfwoods 607d000
fix(sdk): gate generated map keys on encoding/json, not Go comparability
jfwoods 654924a
refactor(sdk)!: drop the Go codegen CLI
jfwoods edbc0b3
fix(sdk)!: keep the MaxRetries default; refuse credentialed REST redi…
jfwoods File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.