docs+fix: v0.9.1 final-check findings — CHANGELOG gaps, doc/log nits, review guardrail - #691
Conversation
… review guardrail - CHANGELOG [Unreleased]: correct #623's "byte-identical" claim (the same PRs also changed the tag-count refusal to over_limit, added deny_unknown_fields to schema/audit, capped keyset cursors at 1032 bytes, and moved a capacity race to stream_refusal), and record the seven public-contract changes the milestone shipped without entries: new /metrics series (#563/#616), compaction outcome fields (#586), explain's limit_to_reach_reason + contract repairs (#601), DELETE mid-rename 409 (#561), limit=0 floor on paged listings (#585/#676), restore exit-code contract + pre-manifest refusal (#616/#618/#619). - src/registry.rs: embedding_failed's doc named a metric that does not exist (taguru_embed_refresh_total) — the real series is taguru_embedding_requests_total{operation="refresh"}. - src/registry/engine.rs: three warn! sites used string interpolation for context/error while their siblings use structured fields; align. - .coderabbit.yaml: add a review lens for public-contract changes missing or misdescribed in CHANGELOG's [Unreleased]. Claude-Session: https://claude.ai/code/session_01D9qnXBxou4stoxmacnDbKb
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
Included review availability: 1 review is currently available. Based on recent review activity, included reviews refill at 2 per hour. 📝 WalkthroughWalkthrough公開契約の変更を Changes公開契約とログ記録
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR updates release documentation, corrects metric documentation and log formatting, and adds a review guardrail; no actionable merge-blocking risk remains beyond normal checks and review. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
v0.9.1 リリース前最終チェック(マイルストーン全44 issueの横断検証)で見つかった所見の一括修正。機能デグレはゼロで、所見はすべて CHANGELOG/doc/レビュー体制レベル。
[Unreleased](high 1件+medium 7件):over_limitへのコード変更・schema/auditのdeny_unknown_fields・keysetカーソルの1032バイト上限・容量競合のstream_refusal化も行っていたため、エントリを実態に合わせて拡充。/metrics新系列4本(registry: cue/BM25 キャッシュの静かな劣化 5 件 #563/ship/replica: クラウド障害・シャットダウン時のフェイルセーフ不備 #616)、compaction outcome のimage_persisted/skipped(永続化層の診断不能な失敗経路・チョークポイントの穴 #586)、explain のlimit_to_reach_reason+契約修理(検索/explain の公開契約の破れ #601)、DELETE mid-rename の 409 化(registry: rename/boot のクラッシュ安全性 9 件 #561)、ページングlimit=0の floor 1(registry メタサイドカーの静かな劣化・リビジョン後退 #585/registry: 連想retract/変更フィードの公開契約破れ #676)、taguru restoreの exit code 契約と pre-manifest 世代拒否(ship/replica: クラウド障害・シャットダウン時のフェイルセーフ不備 #616/ship/replica: 未検証分岐(shipper/restore/replicaの主要分岐にテストが無い) #618/ship/replica: 重複コード・doc陳腐化・簡素化候補 #619)。src/registry.rs(low):embedding_failedの doc が実在しないメトリクス名taguru_embed_refresh_totalを参照 → 実名taguru_embedding_requests_total{operation="refresh"}に訂正。src/registry/engine.rs(low): warn! 3箇所が'{name}': {error}の文字列補間で、同ファイルの他箇所の構造化フィールド(context = %name, %error)と不整合 → 構造化フィールドに統一(ログ文言をピン留めするテストなし、grep確認済み)。.coderabbit.yaml: 公開契約変更(レスポンス形状・エラーコード・メトリクス・exit code・オンディスク形式)が CHANGELOG[Unreleased]に正確に記載されているか(記載済みでも diff と矛盾しないか)を確認するレビュー観点をsrc/**/*.rsに追加 — 今回の記載漏れ8件の再発防止。GitHub側の対応(このPR外): follow-up issue #690 起票(PR #636 がフラグした
cross_search_passagesの span 非対称)、PR #645 本文の「issue paths unchanged」誤記載を訂正済み。Test plan
cargo fmt --check— cleancargo clippy --all-targets— cleancargo test— full suite green (3,856 passed / 0 failed / 6 ignored)https://claude.ai/code/session_01D9qnXBxou4stoxmacnDbKb
Summary by CodeRabbit
ドキュメント
改善