Skip to content

refactor(hub): validate the sidecar manifest and host reports - #18

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-11858-validate-json-the-hub-reads-from-disk-and-the-wire-with
Oct 31, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-11858-validate-json-the-hub-reads-from-disk-and-the-wire-with

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11858

What changed

  • The local sidecar manifest read asserts with an arktype schema; its type is derived from it. An invalid manifest takes the existing removed-and-logged path.
  • The live-app host report is one arktype type in live-app/host-report.ts, used by the host for its type and by the runner to assert each stdout line; a bad line takes the existing warn path.
  • Left as they are: the OpenAI-compatible request body (the hub's own JSON.stringify output, so a schema adds nothing) and the workflow refusal-body narrowing (it reads each field independently by design; a schema version came out longer and no clearer).

How it was verified

bun run lint, bun run typecheck, bun run format:check and bun run test (66 pass) pass.

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11858

@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

@TheGreatAxios
TheGreatAxios force-pushed the cl-11858-validate-json-the-hub-reads-from-disk-and-the-wire-with branch from b807f1e to 19cf294 Compare October 31, 2026 15:00
@TheGreatAxios
TheGreatAxios force-pushed the cl-11858-validate-json-the-hub-reads-from-disk-and-the-wire-with branch from 19cf294 to 2d22568 Compare October 31, 2026 15:01
@TheGreatAxios
TheGreatAxios merged commit 2d22568 into main Oct 31, 2026
5 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-11858-validate-json-the-hub-reads-from-disk-and-the-wire-with branch October 31, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant