Skip to content

docs(openspec): archive Maker forwarder conformance - #100

Merged
csmithington merged 1 commit into
developfrom
ed/cute-taxes-cheat-jnrod
Aug 3, 2026
Merged

docs(openspec): archive Maker forwarder conformance#100
csmithington merged 1 commit into
developfrom
ed/cute-taxes-cheat-jnrod

Conversation

@csmithington

@csmithington csmithington commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • sync the archive-forwarder-durable-acceptance and strategy-runtime-archive-ingestion specifications
  • archive maker-archive-forwarder-conformance after implementation and cross-service verification
  • record CEX Broker v0.2.37, Maker PR #1067, CI runs, release digests, Linear ownership, and the replacement of production soak task 9.7 with the FIET-937 deployment gate

This PR contains no runtime changes.

Validation

  • openspec validate --all --strict
  • bun test: 596 pass, 0 fail
  • reviewed diff against develop to confirm only OpenSpec synchronization/archive evidence is included

Summary by CodeRabbit

  • Documentation
    • Added specifications for durable archive forwarding, including persistence-before-acknowledgment, bounded retention, restart recovery, retries, deduplication, and health reporting.
    • Documented strategy archive validation, supported schema versions, provenance checks, approved tables, and compatibility requirements.
    • Recorded release evidence, verification results, ownership boundaries, migration guidance, and rollback procedures.
    • Marked coordinated release and conformance tasks complete.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0aa2560f-f76e-42cc-8a4c-76fefef1c10d

📥 Commits

Reviewing files that changed from the base of the PR and between 5eec63d and dcaa4de.

📒 Files selected for processing (9)
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/.openspec.yaml
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/design.md
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/implementation-evidence.md
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/proposal.md
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/specs/archive-forwarder-durable-acceptance/spec.md
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/specs/strategy-runtime-archive-ingestion/spec.md
  • openspec/changes/archive/2026-08-03-maker-archive-forwarder-conformance/tasks.md
  • openspec/specs/archive-forwarder-durable-acceptance/spec.md
  • openspec/specs/strategy-runtime-archive-ingestion/spec.md

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

This PR adds OpenSpec metadata, design, proposals, and conformance requirements for Maker strategy archive ingestion. It defines validation, durable SQLite admission, retries, health behavior, telemetry, ClickHouse compatibility, CI coverage, release evidence, and ownership boundaries.

Changes

Maker archive conformance

Layer / File(s) Summary
Strategy archive contract
openspec/changes/.../proposal.md, openspec/changes/.../specs/strategy-runtime-archive-ingestion/spec.md, openspec/specs/strategy-runtime-archive-ingestion/spec.md, openspec/changes/.../design.md
Defines the hb_runtime envelope, approved tables, schema versions, identity and provenance validation, additive ClickHouse migrations, pinned fixtures, and preserved non-strategy direct ingestion.
Durable admission and delivery
openspec/changes/.../specs/archive-forwarder-durable-acceptance/spec.md, openspec/specs/archive-forwarder-durable-acceptance/spec.md, openspec/changes/.../design.md
Defines transactional SQLite admission before HTTP 202, fixed quota and retention, restart recovery, per-table retries, deduplication, health reporting, bounded telemetry, and required CI coverage.
Release evidence and completion
openspec/changes/.../.openspec.yaml, openspec/changes/.../implementation-evidence.md, openspec/changes/.../tasks.md, openspec/changes/.../design.md
Adds OpenSpec metadata, records CEX Broker and Maker release evidence, documents ownership boundaries, and marks coordinated tasks complete.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Poem

I’m a rabbit with a spool in the snow,
Saving each batch before it can go.
Retries hop softly, dedupe marks gleam,
Health tells the truth through the ClickHouse stream.
Maker and Broker release as one—
Durable archives, neatly done.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ed/cute-taxes-cheat-jnrod

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@csmithington
csmithington merged commit a9d3d1b into develop Aug 3, 2026
4 of 5 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