Repository navigation
Conversation
🤖 Automated Review PanelReviewer A — Codex (inherited model)APPROVE — no confirmed actionable findings. Independently verified the pinned Herdr emitted status payload versus the separate subscription filter, source-time context preservation, private-field stripping, cross-workspace move/close behavior, durable replay, and the authorized Reviewer B — Gemini 3 Flash Preview (OpenRouter)APPROVE — no blocking findings. Informational notes: the nested Both reviewers received the same COD-439 goal, authority, project context, pinned-protocol references, and complete final diff at |
|
Local acceptance on endver (2026-10-13, Alice at Shiv's direction) — one defect to fix before merge:\n\n |
🤖 Automated Review Panel — COD-439 follow-up deltaScope: commits Reviewer A — Codex (inherited model)APPROVE — no confirmed actionable findings. The service Reviewer B — Gemini 3 Flash Preview (OpenRouter)APPROVE — no blocking findings. The corrected executable path aligns with the install documentation and package binary. The lockfile update is consistent with the stated dependency bump; the existing restrictive Both reviewers received the same COD-439 goal, Iris project context/conventions, current PR-head status, and exact two-commit delta. Current head is open, non-draft, mergeable, with all seven GitHub checks successful. |
Summary
POST /ingestcontract; add fixtures/tests and operational documentation for upstream retention and spool-eviction limits.AppendMessagewire representation so anIngestBatchcan serialize a typed message without colliding withMessage::kind; preserve route authentication and Iris-owned canonical hashing.Validation
cargo fmt --all -- --checkcargo build --locked --all-targetscargo test --locked --all-targetscargo clippy --locked --all-targets -- -D warningscargo run --locked -p iris-codegen --bin iris-codegen -- checkcreed diff,git diff --checkcargo audit: 0 vulnerable packages; existing baseline warnings remain for unsoundlexical-core 0.7.6and yankedchacha20 0.10.1(both unchanged from the base lockfile). Gitleaks 8.21.2: no leaks found.Acceptance boundary
Tests use pinned source-derived/synthetic fixtures and local deterministic transport/ingest paths only. No endver connection, credential installation, live service restart, deployment, or real message delivery was performed. Local
systemctl --userinstallation/restart, Herdr restart survival, journal flush observation, and real Iris delivery remain outstanding and must be exercised before COD-439 is marked Done. No merge is requested or authorized.Event-context note
The pinned emitted status frame carries source-time workspace and optional agent/display/state-label fields. The bridge preserves those fields, strips
title/raw fields, and enriches only synthetic current-status observations from a fresh pane snapshot; it does not rewrite historical event context from a later snapshot.