Skip to content

docs(dogfood): F3/F4 live acceptance evidence on 041896f - #673

Merged
delete-cloud merged 1 commit into
mainfrom
docs/dogfood-f234-acceptance-evidence
Jul 7, 2026
Merged

delete-cloud merged 1 commit into
mainfrom
docs/dogfood-f234-acceptance-evidence

Conversation

@delete-cloud

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

Copy link
Copy Markdown
Owner

Records the live acceptance probe for the F2/F3/F4 fix wave (#672, deployed via sre-infra #184).

  • Session 8bef270e: interrupted approval run ebddb3df → cancelled cleanly; same-session next prompt run 015122cb → completed. No 409 admission leak (F3), no tape rebind brick (F4).
  • 2 evidence records (record-run + negative probe judgment=pass) appended to the JSONL store; report re-rendered by the harness.

Docs/evidence only — no code changes.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the dogfood evidence report with the latest run totals and session counts.
    • Added new evidence entries covering recent run outcomes, including cancelled and completed runs.
    • Recorded a successful negative-check result confirming expected acceptance behavior.

Session 8bef270e: interrupted approval run ebddb3df cancelled, same-session
prompt run 015122cb completed — no 409, no tape rebind brick. Closes the
F2/F3/F4 fix wave (PR #672, sre-infra #184).

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

coderabbitai Bot commented Jul 7, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Documentation report and JSONL evidence log for semantic memory dogfood testing are updated with new records reflecting a July 5, 2026 test phase, including run-summary, cancelled/completed run entries, and a negative judgment marked pass, alongside updated summary totals.

Changes

Semantic memory dogfood evidence update

Layer / File(s) Summary
Evidence records and summary totals
coding-agent/docs/dogfood/SEMANTIC_MEMORY_RUN_EVIDENCE.md, coding-agent/docs/dogfood/semantic_memory_dogfood_evidence.jsonl
Summary counts (Records, Sessions, Runs) increased and the Records table gained a new run-summary session with cancelled/completed runs and a passing negative probe; matching raw JSONL entries were appended documenting the same run-summary, run statuses, and negative judgment.

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

🚥 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 is concise and accurately summarizes the docs-only change adding live acceptance evidence for the F3/F4 fix validation.
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/dogfood-f234-acceptance-evidence

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@coding-agent/docs/dogfood/SEMANTIC_MEMORY_RUN_EVIDENCE.md`:
- Around line 8-10: The summary in SEMANTIC_MEMORY_RUN_EVIDENCE is incorrect:
the Runs count is one too high compared with the underlying run rows. Update the
generated summary text so the aggregate values exactly match the JSONL source,
and verify the rendered evidence block stays consistent with the deterministic
report output.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: f2bc889d-5873-464b-b182-3e9bb8661b87

📥 Commits

Reviewing files that changed from the base of the PR and between 041896f and 6d7d51e.

📒 Files selected for processing (2)
  • coding-agent/docs/dogfood/SEMANTIC_MEMORY_RUN_EVIDENCE.md
  • coding-agent/docs/dogfood/semantic_memory_dogfood_evidence.jsonl

@delete-cloud
delete-cloud merged commit 0037e4c into main Jul 7, 2026
2 checks passed
@delete-cloud
delete-cloud deleted the docs/dogfood-f234-acceptance-evidence branch July 7, 2026 01:45
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