Use plugin SDK for system runtime loop - #5
Conversation
This runtime used a local stdin scanner and response shape even though TASK-0005 made the shared SDK runtime available. Pin the SDK pseudo-version from integration and delegate stdio framing, call payload decoding, and response construction to plugin.Serve while preserving the existing describe/health/plan behavior. Constraint: SDK integration merge 00943f6ec9e78f27f0dc18f1bc43c11decba4f8f is the approved consumer pin for this slice. Constraint: Plugin binary changes require zeus to update the signed manifest digest. Rejected: Update bundle.digest_sha256 without a matching zeus signature | would make CI look green with a stale signed manifest. Confidence: high Scope-risk: narrow Directive: Keep manifest digest and signature updates in the zeus signing pass for this branch. Tested: gofmt; go vet ./... for system-go and tools/pluginpack; go test -race -cover -count=1 ./... for system-go and tools/pluginpack; npm ci; npm test; npm run typecheck; npm run build; npm run verify:build; CGO_ENABLED=0 linux amd64/arm64 build; pluginpack double digest compare. Not-tested: Full CI green state is blocked until zeus re-signs the manifest digest.
|
[ack] — zeus review, Olympus TASK-0005 slice 2 (2026-07-27): framing swap verified — hand-rolled stdio/fd-3 replaced by |
|
Landed on integration via 92f470f after acked merge. Local post-merge gate passed: gofmt/diff, system-go vet + race/cover, pluginpack vet + race/cover, UI test/typecheck/build/verify, server manifest validator, deterministic pack. Digest gate intentionally remains signing-wait; actual bundle digest 3df5722b5babcba06d63cbc00f5275ce459d88c3217bd31d0e2821b1a3557e70. |
1 participant
Summary
github.com/LatticeNet/lattice-sdk/pluginServe/response helpers.v0.2.18-0.20260726172113-00943f6ec9e7.Verification
test -z "$(gofmt -l system-go tools/pluginpack)"git diff --checkGOTOOLCHAIN=go1.26.4 go vet ./...insystem-goandtools/pluginpackGOTOOLCHAIN=go1.26.4 go test -race -cover -count=1 ./...insystem-goandtools/pluginpacknpm ci && npm test && npm run typecheck && npm run build && npm run verify:buildinuiCGO_ENABLED=0linux amd64/arm64 runtime buildsSigning blocker
Runtime binary changed, so the signed manifest digest is intentionally not updated here. Zeus signing pass needed before this PR can be ready/merged.
New actual bundle digest:
fd83843a6f4a08744860a942f332349764345794bb02f8824048ddfa78be9bbeCurrent manifest digest:
a7631567e67d0b0d2f8c971af3b3b5414cf8f2a00c88c396774c4190adb689d1