Skip to content

Conversation

@Brendonovich
Copy link
Collaborator

Refactors a lot of the logic involved with determining which server to connect to on startup, how we test server health, and how we handle retries and fall back to local servers (there was also some bugs with how local servers were being initialized - --port={port} is invalid syntax). Not sure how much this will solve but it's a definitive improvement.

@github-actions
Copy link
Contributor

Hey! Your PR title desktop: improve server detection & connection logic doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@Brendonovich Brendonovich changed the title desktop: improve server detection & connection logic fix(desktop): improve server detection & connection logic Jan 12, 2026
@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

Based on my search, I found one potentially related PR:

PR #3726: Adding the auto-detection of ollama local with a variable for baseURL

However, PR #3726 appears to be older and focused specifically on ollama with baseURL variables, while PR #7962 is a broader refactor of server detection, health checking, and retry logic. They may be complementary rather than direct duplicates.

@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@Brendonovich Brendonovich merged commit ebbb4dd into dev Jan 12, 2026
6 checks passed
Eric-Guo added a commit to Eric-Guo/opencode that referenced this pull request Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants