Skip to content

docs(adr): accept ADR-0074 with live calibration evidence - #670

Merged
delete-cloud merged 1 commit into
mainfrom
docs/adr-0074-accept
Jul 5, 2026
Merged

delete-cloud merged 1 commit into
mainfrom
docs/adr-0074-accept

Conversation

@delete-cloud

@delete-cloud delete-cloud commented Jul 5, 2026 •

Copy link
Copy Markdown
Owner

Reopens #669 (auto-closed by a failed merge + branch deletion; identical content, CI was green: test pass + CodeRabbit pass).

Flip ADR-0074 Proposed → Accepted: implementation merged (#668) and live o6n acceptance complete — two-round calibration (final floors recall_min_score=0.5 / recall_min_overlap=0.3 via sre-infra #182/#183), restic regression grounds from the sre KB corpus with zero cross-topic noise, relevant seeded-topic recall survives. Evidence bundle updated to 28 records. New follow-ups recorded: unlabeled rendered score scales (feeds D0) and D9 band separability.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the architecture decision record to reflect accepted status and completed acceptance criteria.
    • Added post-merge implementation notes and live validation results.
    • Expanded dogfood evidence with new records, including updated summary counts and additional run/topic observations.

Two-round o6n calibration (floors 0.5/0.3 live via sre-infra #182/#183);
restic regression grounds from the KB corpus with zero cross-topic noise
while relevant seeded-topic recall survives. Evidence updated (28 records).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 5, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c64e4320-11d8-4191-93ec-1000c5f80c51

📥 Commits

Reviewing files that changed from the base of the PR and between a4b2eee and 01f3289.

📒 Files selected for processing (3)
  • coding-agent/docs/adr/0074-semantic-recall-relevance-threshold.md
  • coding-agent/docs/dogfood/SEMANTIC_MEMORY_RUN_EVIDENCE.md
  • coding-agent/docs/dogfood/semantic_memory_dogfood_evidence.jsonl

📝 Walkthrough

Walkthrough

ADR-0074 is updated from Proposed to Accepted status with completed acceptance criteria and a post-merge implementation narrative. Corresponding dogfood evidence files are updated: new JSONL records (seed, topic probes, run-summary/run) are appended, and the Markdown evidence report's summary counts and records table are updated to reflect them.

Changes

ADR Acceptance and Dogfood Evidence

Layer / File(s) Summary
ADR-0074 status and acceptance criteria update
coding-agent/docs/adr/0074-semantic-recall-relevance-threshold.md
Status changes from Proposed to Accepted, all acceptance checklist items are checked, and a merged implementation narrative with test results, calibrated values, and dogfood evidence is added.
Dogfood evidence log and summary update
coding-agent/docs/dogfood/semantic_memory_dogfood_evidence.jsonl, coding-agent/docs/dogfood/SEMANTIC_MEMORY_RUN_EVIDENCE.md
Six new JSONL records (seed, failing then passing topic probes, run-summary/run) are appended, and the Markdown report's summary counts and records table (#21–#26) are updated to match.

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

Related Issues: Not specified in the provided context.

Related PRs: Not specified in the provided context.

Suggested labels: documentation, adr

Suggested reviewers: Not specified in the provided context.

🐰 A hop through docs, both tidy and neat,
ADR-0074 now marked complete,
Evidence rows line up in a row,
Seeds and probes both pass and show,
Accepted status, job well done, sweet.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: accepting ADR-0074 and backing it with live calibration evidence.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/adr-0074-accept

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.

@delete-cloud
delete-cloud merged commit 072da13 into main Jul 5, 2026
3 checks passed
@delete-cloud
delete-cloud deleted the docs/adr-0074-accept branch July 5, 2026 04:40
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