fix(pi): keep thinking defaults specific to each model

[StiensWout]

Problem

Model discovery applied Pi's active thinking level as the default for every model.

Change

Mark the default only for the active provider/model pair. Leave other models' native defaults unspecified and keep the selected thinking level behavior.

Scope and approval

Small discovery bug fix: the active model's thinking level was incorrectly marked as the default for unrelated models.

Verification

Validation: 10 driver/status/thinking-capability tests, Pi typecheck and scoped lint/format checks passed.

_{Prepared for Wout by gpt-6.1-sol in Codex.}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR changes production Pi model defaults by limiting the configured thinking level to the active model and leaving other models to their native defaults. The change is localized and tested, but product-default changes require human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4029783f-123b-404b-845e-4d9cfe0dc0fa

---

---

📥 Commits

Reviewing files that changed from the base of the PR and between 8c777fb and 57e8bf7.

---

---

📒 Files selected for processing (3)

• docs/user/providers-pi.md
• packages/provider-pi/src/server/driver.test.ts
• packages/provider-pi/src/server/status.ts

---

---

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

---

---

📝 Walkthrough

📝 Walkthrough

Walkthrough

Model discovery now applies the configured thinking level only to the active model that matches the state provider and model ID. Tests cover known and unknown active models, and the documentation describes the picker default.

Changes

Pi thinking-level defaults

Layer / File(s)	Summary
Apply the thinking-level default to the active model packages/provider-pi/src/server/status.ts, packages/provider-pi/src/server/driver.test.ts, docs/user/providers-pi.md	The discovery parser receives the full state and applies its thinking level only to the matching active model. Tests cover known and unknown active models. The documentation describes the active model’s configured level as the picker default and states that other models retain their Pi defaults when no level is selected.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

---

---

Merge Risk: ⚪ Minimal · up to 57e8b

The change scopes the thinking-picker default to the active model, and other models keep their Pi defaults. No merge-blocking risk was identified.

Pre-merge checks | 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Title check		The title clearly and concisely describes the main change: keeping Pi thinking defaults specific to each model.
Description check		The description includes all required sections. It explains the problem, the fix, scope justification, focused verification, and the agent attribution.

---

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

---

---

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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