docs(evidence): add #11504 stage1 code-reply proof - #11566
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
53fe4d9 to
c79bf9c
Compare
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
Reviewed locally as evidence-only, but GitHub will not let this account approve its own PR. Rebased onto origin/develop and pushed c79bf9c. I validated the trajectory JSON parses and matches the README claims: status=finished, one messageHandler stage, model=gemma-4-31b, contexts=[simple], requiresTool=false, and the replyText is a Python function body rather than the fallback. I also checked the README issue/artifact links, ran git diff --check, and ran a narrow credential-pattern scan for sk-/sk-proj/Bearer/Authorization/x-api-key shaped values with no matches. Biome ignores this .github evidence README in the current config, so there was no meaningful Biome file to check here. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
Closes #11504
Summary
The runtime fix for #11504 landed in #11555 (
c532e3b565e):isUnusableStage1Replyno longer defers valid bare-code or structured replies just because they contain repeated-character runs. This PR adds the missing live evidence artifact and manual review notes required to close the issue.Evidence Added
.github/issue-evidence/11504-stage1-code-replies/README.md.github/issue-evidence/11504-stage1-code-replies/live-cerebras-current/93432706-b3b2-08ea-ab6a-ba55340a8848/tj-74b5c8fef7d6a9.jsonManual review of the trajectory confirms:
status: finishedmessageHandlerstagegemma-4-31bcontexts:["simple"]withrequiresTool:falsedirect_replyinstead ofI'm not sure how to answer that.Verification
bun run --cwd packages/core test src/__tests__/message-runtime-stage1.test.ts->1 passed,75 passedbunx @biomejs/biome check packages/core/src/services/message.ts packages/core/src/__tests__/message-runtime-stage1.test.ts .github/issue-evidence/11504-stage1-code-replies/README.md->Checked 2 files ... No fixes applied.bun run --cwd packages/core typecheck-> passgemma-4-31bCerebras runtime capture -> pass, artifact abovebun run install:light-> pass, artifact sync skipped withELIZA_SKIP_ARTIFACT_SYNC=1bun installattempted, but postinstall artifact sync was still at7.0 MiB / 971 MiBwith141m 59sremaining, so it was stopped before completionbun run verifycurrently fails before typecheck/lint inaudit:type-safety-ratcheton current repo baseline (as unknown as: 77 / 76,?? 0core/agent/app-core:376 / 375). This PR changes only evidence artifacts, with no production-source diff.UI / Media
N/A - core runtime evidence only; no user-facing UI changed.