Skip to content

Make OMX summaries acceptable to Colony#12

Merged
NagyVikt merged 1 commit into
devfrom
agent/codex/fix-colony-runtime-summary-schema-2026-05-01-16-49
May 1, 2026
Merged

Make OMX summaries acceptable to Colony#12
NagyVikt merged 1 commit into
devfrom
agent/codex/fix-colony-runtime-summary-schema-2026-05-01-16-49

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

@NagyVikt NagyVikt commented May 1, 2026

Automated by Agent 2.\n\nVerification:\n- npm run build\n- node --test dist/runtime/tests/colony-runtime-summary.test.js\n- npm run lint\n- pnpm --filter @colony/core test -- test/omx-runtime-summary.test.ts\n- pnpm --filter @imdeadpool/colony-cli test -- test/health.test.ts -t 'shows OMX runtime bridge available when a fresh v1 summary exists'

Colony rejects the runtime summary file unless the payload declares the colony-runtime-summary-v1 schema, so the writer now emits the required schema fields and validates the payload before the atomic rename.

Constraint: Current Colony health parser requires schema=colony-runtime-summary-v1 and uses timestamp for freshness.

Rejected: Patch Colony to accept the older OMX version-only shape | malformed summaries should be fixed at the producer.

Confidence: high

Scope-risk: narrow

Directive: Keep last_seen_at and timestamp aligned until Colony drops the timestamp freshness input.

Tested: npm run build; node --test dist/runtime/__tests__/colony-runtime-summary.test.js; npm run lint; pnpm --filter @colony/core test -- test/omx-runtime-summary.test.ts; pnpm --filter @imdeadpool/colony-cli test -- test/health.test.ts -t 'shows OMX runtime bridge available when a fresh v1 summary exists'

Not-tested: Full oh-my-codex npm test suite

Co-authored-by: OmX <omx@oh-my-codex.dev>
@NagyVikt NagyVikt merged commit 267b228 into dev May 1, 2026
8 of 10 checks passed
@NagyVikt NagyVikt deleted the agent/codex/fix-colony-runtime-summary-schema-2026-05-01-16-49 branch May 1, 2026 15:05
@github-actions github-actions Bot added the size/M label May 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant