fix(ollama): try every browser before giving up on cookie import

[Finesssee]

Maintainer replacement for #430 because the original head is in an external fork. Includes the reviewed fix plus preservation of meaningful browser cookie extraction errors.\r\n\r\nSupersedes #430.\r\n

Summary by CodeRabbit

• Bug Fixes
  • Improved Ollama session detection across supported browsers.
  • Authentication now continues checking available browser cookie sources when earlier cookies are unrelated.
  • Prevents irrelevant cookie data from blocking valid sessions.
  • Handles missing or unusable sessions more reliably.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: fc54053d-473e-48df-9cb6-a09b80616425

📥 Commits

Reviewing files that changed from the base of the PR and between 1e3d04c and 570302a.

📒 Files selected for processing (1)

• rust/src/providers/ollama/cookies.rs

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

---

📝 Walkthrough

Walkthrough

Ollama cookie resolution now scans all detected browsers, returns the first recognized session header, preserves the first meaningful extraction error, and returns None when no usable session exists.

Changes

Ollama cookie resolution

Layer / File(s)	Summary
Browser cookie scan and session validation rust/src/providers/ollama/cookies.rs	Cookie lookup scans every detected browser. The helper selects the first recognized Ollama session header. Tests cover irrelevant earlier cookies and missing sessions.

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

Merge Risk: ⚪ Minimal · up to 57030

Ollama session-cookie resolution now continues through detected browsers until it finds a recognized session, while retaining meaningful extraction errors when none is available. No current merge-blocking risk is established.

🚥 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 clearly and concisely describes the main change: Ollama cookie import now tries every detected browser before it gives up.
Docstring Coverage	✅ Passed	Docstring coverage is 85.71% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 1 files.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/ollama-cookie-browser-priority

---

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