Pull Request Overview
This PR improves handling of package: URIs in the Dart extension's terminal link provider and document links by addressing scenarios where multiple packages might match the same URI. The changes enhance the user experience by providing options when multiple matches exist and prioritizing project-specific package maps for document links.
- Enhanced terminal link handling to support multiple package URI matches with peek functionality
- Improved document link resolution to prioritize current project's package map
- Added TODO comments for future test coverage improvements
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/extension/terminal/package_uri_link_provider.ts | Enhanced package URI resolution to handle multiple matches and improved document link provider logic |
| src/test/dart/providers/terminal_link_providers.test.ts | Added TODO comments outlining needed test coverage for multiple package URI scenarios |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Comment on lines +85 to +86
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
…links
DanTup deleted the multiple-terminal-links branch
September 11, 2025 13:16