Repository navigation
Add Conductor-Agents to Go SDK - #285
Merged
Merged
Conversation
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
3 times, most recently
from
September 16, 2026 21:32
0f3679c to
37f7de7
Compare
ling-senpeng13
marked this pull request as draft
September 16, 2026 21:34
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
2 times, most recently
from
September 17, 2026 04:08
fc70f37 to
00d7809
Compare
ling-senpeng13
marked this pull request as ready for review
September 17, 2026 15:58
NicholasDCole
approved these changes
Sep 17, 2026
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
from
September 17, 2026 16:32
00d7809 to
02db740
Compare
ling-senpeng13
changed the base branch from
feat/agent-sdk-3-workers
to
main
September 17, 2026 16:32
This was referenced Sep 17, 2026
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
4 times, most recently
from
September 17, 2026 20:19
87ea19b to
2d1943a
Compare
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
2 times, most recently
from
September 17, 2026 21:41
75c8099 to
ad23b2d
Compare
Contributor
Contributor
Author
|
In addition to porting the Python example tests, I also ported the applicable Python E2E tests with LLM replay to increase test coverage and these json files are the artifacts. Through this process, I found a few SDK bugs as well. General thoughts:
|
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
4 times, most recently
from
September 22, 2026 18:44
831d4de to
5b255ea
Compare
kowser-orkes
approved these changes
Sep 22, 2026
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
from
September 22, 2026 19:50
5b255ea to
6875944
Compare
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
5 times, most recently
from
September 22, 2026 22:46
3027a04 to
ca8af9b
Compare
v1r3n
approved these changes
Sep 24, 2026
Co-Authored-By: nicholascole <nicholas.colesd@gmail.com>
ling-senpeng13
force-pushed
the
feat/agent-sdk-4-runtime
branch
from
September 24, 2026 18:31
55b9d49 to
d1db4f4
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.


Adds the agents feature to the Go SDK as
sdk/ai, a port of the Python SDK's agent surface.What it adds
Agentmodel and its serializer,Runtime,AgentHandle, and the agent REST client.docs/ai/, and the Python agent examples ported underexamples/agents/.How it is verified
TestGoldenAgentConfigserializes a Go agent for every fixture undersdk/ai/testdata/agent_config/and requires the document the Python serializer produces, so the wire format cannot drift from Python's. The Python e2e suites are ported undertest/integration_tests/ai/and replayed against recorded LLM traffic, one directory per test file: 105 tests pass, and the 13 skips are listed with their reasons in that directory's README.CONDUCTOR_SERVER_URL=http://localhost:8080/api go test -tags integration ./test/integration_tests/ai/Fixes outside sdk/ai
Six gaps the ports exposed, each a feature that was declared but never wired up: tool-level guardrails now run in the worker; terminations, stop-when predicates and router functions each get the worker the server waits for;
Agent.Statefulroutes a run to a per-execution worker domain;Runtime.Plancompiles a skill from its own document; an event carries the execution it came from, so a nested agent's request for human input is answered on that execution; and a schedule's input survives being read back.The lint workflow now compares a pull request against its merge base. The patch API it used refuses a diff this size and then silently lints the whole repository.
Two tests stay skipped until conductor-oss/conductor#1633 lands, which fixes numeric and agent-envelope matching in the LLM recorder.