chore(substrate): Bump substrate to v0.0.9 and remove collapsing of image env into actor env#2208
Merged
Conversation
Signed-off-by: JM Huibonhoa <jm.huibonhoa@solo.io>
Signed-off-by: JM Huibonhoa <jm.huibonhoa@solo.io>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the kagent Substrate integration to align with substrate v0.0.9, including protobuf/API shape changes (e.g., actor identity/version moving under Metadata) and removing the prior workaround that manually injected image runtime ENV into the ActorTemplate env (per issue #2206).
Changes:
- Bump
github.com/kagent-dev/substratereplace target tov0.0.9(and updatego.sumaccordingly). - Migrate Substrate client + handlers + tests to the new v0.0.9 API/proto shapes (
ObjectRef,ResourceMetadata, updated RPC request/response types). - Remove the Python runtime image ENV re-injection (
pythonRuntimeImageEnv) now that substrate applies image ENV directives.
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| go/go.sum | Updates dependency checksums for substrate v0.0.9. |
| go/go.mod | Bumps the substrate replace to github.com/kagent-dev/substrate v0.0.9. |
| go/core/pkg/sandboxbackend/substrate/openclaw.go | Uses metadata-based actor identity/atespace for endpoint display. |
| go/core/pkg/sandboxbackend/substrate/lifecycle_delete_test.go | Updates cleanup-path fakes to match new ControlClient RPC shapes. |
| go/core/pkg/sandboxbackend/substrate/config_hash_test.go | Updates actor test fixtures to use ResourceMetadata{Name: ...}. |
| go/core/pkg/sandboxbackend/substrate/client.go | Migrates substrate client calls to ObjectRef + metadata-based identity and updated RPC signatures. |
| go/core/pkg/sandboxbackend/substrate/client_test.go | Updates CreateAtespace fake to match new request/return types. |
| go/core/pkg/sandboxbackend/substrate/agentharness_actor.go | Switches actor identification from ActorId to metadata name. |
| go/core/pkg/sandboxbackend/substrate/agent_lifecycle.go | Removes Python image ENV injection workaround per substrate v0.0.9 behavior. |
| go/core/pkg/sandboxbackend/substrate/agent_lifecycle_test.go | Updates tests to reflect image ENV no longer being manually re-supplied. |
| go/core/pkg/sandboxbackend/substrate/agent_actor.go | Switches actor identification from ActorId to metadata name. |
| go/core/internal/httpserver/handlers/substrate.go | Updates HTTP API mapping to pull Actor/Worker identity/version from metadata/assignment. |
| go/core/internal/httpserver/handlers/substrate_test.go | Updates handler test fixtures to the new proto shapes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
262
to
263
| require.Equal(t, tc.wantConfigEnv, names["KAGENT_CONFIG_JSON"], "declarative agents materialize config from secret env; BYO does not") | ||
| require.True(t, names["KAGENT_CONFIG_JSON"], "every agent type gets the rendered config via secret env (BYO decides for itself whether to consume it)") |
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
EItanya
approved these changes
Jul 15, 2026
EItanya
approved these changes
Jul 15, 2026
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.
Bump substrate to
v0.0.9and fix #2206