chore(ci): enable CodeRabbit approvals for stack reviews

[kalvenschraut]

The stacked feature PRs have completed CodeRabbit review, but their configuration prevents formal approval votes. Enable the request-changes workflow so CodeRabbit can approve after findings are addressed and the latest changes are reviewed.

This prerequisite supplies the setting to PRs #24–#30 through their branch ancestry. CodeRabbit reads each feature branch's configuration.

Validation: parsed the YAML and checked the diff. The change adds one boolean setting.

Model and harness: GPT-6 in Codex.

Summary by CodeRabbit

• Chores
  • Enabled an automated review workflow that can request changes during code review.

[kalvenschraut]

@coderabbitai review

[kalvenschraut]

@coderabbitai approve

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 40d9e825-adec-405e-a735-c8dc8d49c6c5

📥 Commits

Reviewing files that changed from the base of the PR and between 85dd528 and 2d4c31c.

📒 Files selected for processing (1)

• .coderabbit.yaml

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

The review configuration enables the request_changes_workflow setting.

Changes

Review workflow configuration

Layer / File(s)	Summary
Enable request changes workflow .coderabbit.yaml	Adds request_changes_workflow: true to the review configuration.

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

Merge Risk: ⚪ Minimal · up to 2d4c3

This enables the request-changes workflow for CodeRabbit reviews. No current merge-blocking risk remains.

Suggested reviewers: juliusmarminge

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Description check	✅ Passed	The description clearly explains what changed, why it changed, the affected PRs, and the validation performed. It omits the template headings and checklist, but the missing UI section is not applicabl…
Title check	✅ Passed	The title clearly identifies the configuration change and its purpose for CodeRabbit stack reviews.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch stacks/review-approval

---

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

[coderabbitai]

✅ Action performed

Comments resolved and changes approved.

[kalvenschraut]

Merge verdict: NO-GO (review gates incomplete).

Head: 2d4c31c19eab51ebcf6fe7c38d5e0b4f3ba3649f.

• Codex: NO-GO for merging under the requested two-reviewer gate. Merge readiness and review history were checked; the full dual code review is incomplete, and this comment is not a code approval.
• Fable 5.1 / Claude CLI: BLOCKED. The requested claude-fable-5-1 review was attempted, but the CLI reached its session limit before returning a completed verdict. Its reported reset is midnight America/Chicago. There is no fresh Fable GO from this pass.
• CodeRabbit: APPROVED on this exact head.
• GitHub merge state: MERGEABLE / CLEAN against gitea/workflows.
• Checks: the current rollup contains a successful CodeRabbit status only; it contains no build/test check result. No new tests or browser verification are claimed by this audit.

No merge performed. Resume the independent code reviews when Fable is available, obtain the missing/current CodeRabbit approval where applicable, and recheck the exact head and base before merging.

[kalvenschraut]

GO for this PR’s reviewed change.

Reviewed head: 2d4c31c19eab51ebcf6fe7c38d5e0b4f3ba3649f. Destination: gitea/workflows.

• Codex: GO. Enables CodeRabbit’s formal approval workflow. Configuration-only change; both reviewers found no blocker.
• Claude Fable 5.1 through Claude CLI: GO. Completed independent source review of this exact head.
• CodeRabbit: APPROVED on this exact head; the formal GitHub review was checked.
• Validation: Configuration diff inspected. The integrated server typecheck passed. These integrated checks do not claim that every earlier stack snapshot is free of inherited compiler errors; PR feat(pull-requests): approve native Gitea workflow runs #12’s baseline compiler errors and their correction in fix(pull-requests): preserve Gitea schema inference #22 are tracked separately.

GitHub reports mergeable/clean. The maintainer authorized merging when these gates pass. This merges into the PR’s declared base branch; it does not retarget the PR or approve other open slices. The head, checks, review threads and approval will be rechecked immediately before merging.

Native stack clarification: GitHub requires an asynchronous merge of the reviewed prefix #34 → #24 → #25, into stack trunk gitea/workflows. All three pass the requested gates. The request will stop at #25; it excludes blocked #26 and every later member. GitHub merge semantics.