devlog: record the late #2362 review and what retirement cost

[lidge-jun]

Summary

Amends 090 with the #2362 review that arrived after its lane had been retired under DISPATCH-RETIRE-01.

The lane went silent for three wait cycles, so it was retired and the PR was reviewed directly by the main agent. The lane then returned, and it had found more: three defects in the resolver itself, beyond the config-surface gap the direct review caught. All three were reproduced at the PR head before being accepted:

B2 canonical-openai:      {"graceMs":500}
B4 invalid-falls-through: {"graceMs":750}
B3 My-Model: 500  my-model: 1500  MY-MODEL: 1500

Retiring the lane was correct — the phase would still be waiting otherwise. But retirement is not a verdict, and dropping the late result would have lost three measured defects in a config surface. The doc records the rule: when a retired lane returns, re-read it against what was already concluded.

Findings posted to #2362. Disposition unchanged (leave open); the blocker list is longer and now measured.

Verification

Docs only. privacy:scan and repo-hygiene green earlier in this unit; no production file touched.

Checklist

• Docs-only
• No security triage in devlog
• Numbered lexicographic filename

Summary by CodeRabbit

• Documentation
  • Updated the backlog disposition record to document the return of a retired review lane.
  • Preserved the existing configuration-validation finding and added three reproduced resolver defects.
  • Recorded additional lookup and testing weaknesses.
  • Clarified that retiring a review lane is not a verdict and that the tracked item remains open with an expanded list of blockers.

[github-actions]

✅ Deterministic PR hygiene checks passed.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 31bb9cb2-6048-407c-b672-ac72d78a813c

📥 Commits

Reviewing files that changed from the base of the PR and between 7dfe57a and 0b7a771.

📒 Files selected for processing (1)

• devlog/_plan/260822_backlog_disposition_program/090_wp9_new_pr_disposition.md

---

📝 Walkthrough

Walkthrough

The plan amendment documents late-review context, configuration-validation gaps, three reproduced resolver defects, additional lookup and test weaknesses, and the decision to keep issue 2362 open.

Changes

Resolver review disposition

Layer / File(s)	Summary
Late review findings devlog/_plan/260822_backlog_disposition_program/090_wp9_new_pr_disposition.md:123-166	Documents missing validation wiring for three operator-facing configuration keys. Records resolver defects involving OpenAI forward providers, per-model grace values, and case-folded duplicate model keys.
Issue disposition devlog/_plan/260822_backlog_disposition_program/090_wp9_new_pr_disposition.md:168-182	States that lane retirement does not determine the verdict. Defines handling for late results and confirms issue 2362 remains open with additional blockers.

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

Suggested reviewers: ingwannu

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/wp9-2362-amend

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}