chore: add handler and router for coder scheme URIs#145
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
This is attached to the coder scheme.
|
I've run into the same issue described in #98, but with |
There was a problem hiding this comment.
Pull Request Overview
This PR adds a handler and router to process coder scheme URIs while updating the project dependency configuration and deep link handling. Key changes include:
- Adding the URLRouting package to the project configuration.
- Implementing a new router in VPNLib to parse and match coder URLs.
- Incorporating tests for various routing scenarios and updating deep link handling in the app delegate.
Reviewed Changes
Copilot reviewed 6 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| Coder-Desktop/project.yml | Added URLRouting package dependency and updated targets. |
| Coder-Desktop/VPNLibTests/CoderRouterTests.swift | Introduced comprehensive tests for the new router implementation. |
| Coder-Desktop/VPNLib/CoderRouter.swift | Implemented the router and its associated types for coder URIs. |
| Coder-Desktop/Coder-Desktop/URLHandler.swift | Added a URL handler to process matching coder URLs with error handling. |
| Coder-Desktop/Coder-Desktop/Coder_DesktopApp.swift | Updated deep link handling by integrating the new URL handler. |
Files not reviewed (3)
- Coder-Desktop/Coder-Desktop/Info.plist: Language not supported
- Makefile: Language not supported
- scripts/build.sh: Language not supported
32ea67d to
fcfad50
Compare
fcfad50 to
823c11b
Compare
Merge activity
|

Relates to #96.
Closes #95