Skip to content

refactor(resource): extract provider protocol crate - #398

Merged
schickling-assistant merged 2 commits into
mainfrom
schickling-assistant/2026-08-31-resource-protocol-consolidation
Aug 31, 2026
Merged

refactor(resource): extract provider protocol crate#398
schickling-assistant merged 2 commits into
mainfrom
schickling-assistant/2026-08-31-resource-protocol-consolidation

Conversation

@schickling-assistant

@schickling-assistant schickling-assistant commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Problem

Resource wire messages and codecs were owned by the st2 supervisor crate, forcing every native provider runtime to depend on supervisor implementation details and encouraging duplicate protocol copies.

Goal

Make one upstream crate the wire authority while preserving the existing st2 API surface and current native session-driver graph authority.

Decisions

  • Add st2-resource-protocol as a wire-only crate owning resource message types, validation, and JSON-line codecs.
  • Keep existing st2 public imports working through re-exports; no parallel protocol or compatibility shim is introduced.
  • Preserve the native driver graph and Codex support from merged feat: expose authoritative agent graph primitives #390 and Admit codex-cli 0.151.0 #394.
  • Make invalid Resource DING subjects actionable without changing graph/session-driver ownership.

Verification

Complexity

No additional runtime process or state owner is introduced. The new crate is a pure wire/codec boundary; the supervisor continues to own graph and session-driver behavior.

Concerns

None known.

Friction & bottlenecks

None.

Follow-ups

The dependent dotfiles stack consumes this exact upstream revision and owns the shared provider runtime plus thin resource adapters.

References

Posted on behalf of @schickling
field value
agent_identity dev3.direct.omp.jmg9x29x
session dev3.jmg9x29x
agent_persona generalist
agent_supervisor unavailable
agent_tool OMP
agent_tool_version 18.0.11
agent_runtime OMP 18.0.11
tooling_profile dotfiles@2161b9c

@schickling-assistant
schickling-assistant force-pushed the schickling-assistant/2026-08-31-resource-protocol-consolidation branch from 7762fdb to dac0662 Compare August 30, 2026 22:46
@schickling-assistant
schickling-assistant changed the base branch from main to schickling-assistant/2026-08-31-st2-health-foundation August 30, 2026 22:47
@schickling-assistant
schickling-assistant force-pushed the schickling-assistant/2026-08-31-resource-protocol-consolidation branch 2 times, most recently from 9a3aae2 to 88fbe48 Compare August 30, 2026 23:07
@schickling-assistant
schickling-assistant marked this pull request as ready for review August 30, 2026 23:16
@schickling-assistant
schickling-assistant force-pushed the schickling-assistant/2026-08-31-resource-protocol-consolidation branch from 88fbe48 to 232670a Compare August 31, 2026 00:04
Base automatically changed from schickling-assistant/2026-08-31-st2-health-foundation to main August 31, 2026 01:02
agent-identity: dev3.direct.omp.2cshu64q
agent-persona: generalist
agent-supervisor: unavailable
agent-tool: OMP
agent-tool-version: 18.0.9
agent-runtime: OMP 18.0.9
tooling-profile: dotfiles@b607597
agent-identity: dev3.direct.omp.2cshu64q
agent-persona: generalist
agent-supervisor: unavailable
agent-tool: OMP
agent-tool-version: 18.0.9
agent-runtime: OMP 18.0.9
tooling-profile: dotfiles@b607597
@schickling-assistant
schickling-assistant force-pushed the schickling-assistant/2026-08-31-resource-protocol-consolidation branch from 232670a to 66fe50e Compare August 31, 2026 01:02
@schickling-assistant
schickling-assistant merged commit ca0df38 into main Aug 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant