renaming and light refactoring to better support adding other harnesses

[peterj]

No new features, mostly renames and moving code around a bit, so it's easier to implement new harnesses (e.g. hermes).

To implement a new backend:

1. Add a new AgentHarnessBackendType value (e.g. hermes) in go/api/v1alpha2
2. Implement sandboxbackend.AsyncBackend

• e.g. add a hermes.go (for example) that embeds *agentHarnessOpenShellBackend (same as ClawBackend in openclaw.go.

3. Call newAgentHarnessOpenShellBackend(...) with a name, function that creates the sandbox (look at the buildAgentHarnessOpenshellCreateRequest for an example), implement preCreateSandbox where you translate the model config to something that hermes understands and OnAgentHarnessReady to run any configuration needed, so agent can run
4. Register the backend
   In buildOpenshellSandboxBackends in go/core/pkg/app/app.go, add an entry for the new backend

[Copilot]

Pull request overview

This PR refactors the OpenShell sandboxbackend implementation to better support multiple AgentHarness “harness” backends by splitting shared gRPC lifecycle logic into reusable components, extracting OpenClaw policy logic into a subpackage, and cleaning up naming to reflect AgentHarness semantics.

Changes:

• Replaced the monolithic grpcBackend with AgentHarnessOpenShellClient + agentHarnessOpenShellBackend and added a PreCreateSandboxFunc hook for backend-specific pre-create work.
• Extracted OpenClaw/NemoClaw policy building into openshell/openclaw and moved allowed-domain host normalization into sandboxbackend.NormalizeAllowedDomainHost.
• Renamed/rewired constructors and backend registration (NewOpenShellBackend, shared OpenClaw backend instance registered for both openclaw and nemoclaw).

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
go/core/pkg/sandboxbackend/openshell/translate.go	Renames request builder and gateway-name helper; switches to new policy builder.
go/core/pkg/sandboxbackend/openshell/translate_test.go	Updates tests to new builder name and to use openclaw exported constants.
go/core/pkg/sandboxbackend/openshell/policy.go	Refactors policy composition into mergeable fragments; uses shared allowed-domain host normalizer and extracted OpenClaw helpers.
go/core/pkg/sandboxbackend/openshell/openshell.go	Introduces reusable backend wrapper + pre-create hook; renames OpenShellBackend constructor.
go/core/pkg/sandboxbackend/openshell/openshell_test.go	Updates tests for renamed constructors and new exported exec helpers.
go/core/pkg/sandboxbackend/openshell/openclaw/policy.go	New package containing OpenClaw/NemoClaw baseline + channel policy logic and exported constants.
go/core/pkg/sandboxbackend/openshell/openclaw.go	Refactors OpenClaw backend to use the shared backend wrapper and adds translateModelConfig as pre-create hook.
go/core/pkg/sandboxbackend/openshell/grpc_backend.go	Removes the old shared gRPC backend implementation.
go/core/pkg/sandboxbackend/openshell/client.go	Clarifies the responsibility boundary of OpenShellClients vs higher-level clients.
go/core/pkg/sandboxbackend/openshell/agentharness_openshell_client.go	New higher-level OpenShell client wrapper for AgentHarness lifecycle and exec helpers.
go/core/pkg/sandboxbackend/network_host.go	New shared helper for normalizing allowed domain hosts.
go/core/pkg/sandboxbackend/network_host_test.go	Moves normalization tests into the sandboxbackend package.
go/core/pkg/app/app.go	Updates backend wiring; registers the same OpenClaw backend instance for both openclaw and nemoclaw.
go/core/internal/controller/agentharness_controller.go	Records status.backendRef.backend from spec.backend (needed for the openclaw/nemoclaw shared backend instance).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[chromatic-com]

Warning

Testing paused

Monthly snapshot limit reached. Update your plan to get more snapshots and resume testing.