Skip to content

feat(app): add server connect links - #41152

Closed
OpeOginni wants to merge 1 commit into
anomalyco:devfrom
OpeOginni:connect-links
Closed

feat(app): add server connect links#41152
OpeOginni wants to merge 1 commit into
anomalyco:devfrom
OpeOginni:connect-links

Conversation

@OpeOginni

@OpeOginni OpeOginni commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Issue for this PR

  • Closes #
  • Not applicable.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

  • Adds #connect?server=...&directory=... links for the hosted app and opencode://connect?... links for desktop.
  • Pre-fills the validated server connection form without putting credentials in the link.
  • Opens an existing server and valid project directly; falls back to the remote directory picker when the project cannot be validated.
  • Keeps credentials and transient connection work scoped to the dialog, including cancellation handling.

How did you verify your code works?

  • Ran focused app tests for server state, server protocol/health, directory selection, and deep-link parsing.
  • Ran bun typecheck in packages/app and packages/desktop.
  • Ran the full workspace typecheck through the pre-push hook.
  • Tested the hosted-app link flow locally against a remote server.

Screenshots / recordings

  • Not included.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

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

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

Related PR:

  • feat(desktop): connect servers via external-scheme deep links #35968: feat(desktop): connect servers via external-scheme deep links
    • This PR appears to implement similar deep-link functionality for desktop using the opencode:// protocol scheme, which is directly related to the current PR's goal of adding server connect links. However, this is an older PR that may have different scope or implementation details.

The other PR found (#40843) about Remote SSH support appears less directly related to this specific feature.

Note: The current PR (#41152) is the primary implementation for the hosted app with both #connect?server=...&directory=... links and opencode://connect?... support, while #35968 may be a prior or complementary effort for desktop connectivity.

@zhangyx1998 zhangyx1998 mentioned this pull request Aug 24, 2026
Closed
1 task done
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Automated PR Cleanup

Thank you for contributing to opencode.

Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions.

This PR was closed because it matched the following cleanup criteria:

  • The PR was created more than 1 month ago
  • The PR had fewer than 2 positive reactions
  • Positive reactions are counted as thumbs-up, heart, celebration, or rocket reactions on the PR

PRs created within the last month are not affected by this cleanup.

If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate.

Thanks again for taking the time to contribute.

@github-actions github-actions Bot closed this Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

automated-pr-cleanup contributor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@OpeOginni