Skip to content

Advance the canonical Echo producer pin - #34

Merged
flyingrobots merged 1 commit into
mainfrom
task/33-advance-producer-pins
Aug 15, 2026
Merged

Advance the canonical Echo producer pin#34
flyingrobots merged 1 commit into
mainfrom
task/33-advance-producer-pins

Conversation

@flyingrobots

Copy link
Copy Markdown
Owner

Outcome

Advances the canonical Echo coordinate from c354d531679861fb7bbd52ab7b7703807909ab86 to 490134c0753a3df6a74da366cc71c1248764dc5a. Edict remains pinned at df80f92ad6242c6da31a64224666fd37aa43b0d0.

The full witness gate reproduced every vendored Edict lawpack and Echo provider artifact byte-for-byte, so no generated or application artifact changed. The only repository diff is the exact Echo coordinate.

Validation

  • ./tests/producer-lock.sh
  • ./tests/runtime.sh: golden, deterministic rerun, typed refusals, boundary, fixed-seed, and stress cases passed
  • ./tests/effect-runtime.sh: ordered observation, durability, replay, refusal, boundary, fixed-seed, and stress cases passed
  • ./tests/patch-runtime.sh: ordered patch, retry, replay, reconciliation, refusal, boundary, fixed-seed, and stress cases passed
  • ./tests/build-cleans-output.sh
  • ./tests/writer-epoch-assertions.sh: all cases passed
  • cargo fmt for both generated hosts
  • cargo clippy -D warnings for both generated hosts
  • git diff --check

Both producer worktrees were exact and clean at the named commits.

Closes #33

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: af14226f-e187-465c-87ac-2766f759ead1

📥 Commits

Reviewing files that changed from the base of the PR and between 1ad3f6d and 8db8961.

📒 Files selected for processing (1)
  • producers.lock.json
📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: Witness gate
🔇 Additional comments (1)
producers.lock.json (1)

9-9: LGTM!


Summary by CodeRabbit

  • Chores
    • Updated the locked version of the flyingrobots/echo dependency.

Walkthrough

The PR updates the locked flyingrobots/echo dependency from commit c354d531679861fb7bbd52ab7b7703807909ab86 to 490134c0753a3df6a74da366cc71c1248764dc5a. No public entities change.

Changes

Producer pin update

Layer / File(s) Summary
Update Echo lock pin
producers.lock.json
The locked Echo commit changes to 490134c0753a3df6a74da366cc71c1248764dc5a.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 8db89

This change advances the canonical Echo producer pin in the lock file without changing generated or application artifacts, and all supplied validation checks pass. No actionable merge-blocking risk remains beyond normal checks and review.

Poem

One hash leaves the chain,
A newer Echo takes its place,
The lock stays precise.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the primary change: advancing the canonical Echo producer pin.
Description check ✅ Passed The description directly explains the Echo pin update, retained Edict pin, validation results, and linked issue.
Linked Issues check ✅ Passed The PR satisfies the coding objectives in [#33], including the exact Echo pin, witness evidence, clean checkouts, and no semantic changes.
Out of Scope Changes check ✅ Passed The one-line lock-file change is fully aligned with [#33], and no unrelated code or artifact changes are reported.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@flyingrobots
flyingrobots merged commit ee45716 into main Aug 15, 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.

Advance the canonical producer pins with current-head witness evidence

1 participant