feat: migrate document link to language-service

[9romise]

No description provided.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4cb9bb3e-f153-42dc-82c4-21c67a682804

📥 Commits

Reviewing files that changed from the base of the PR and between e3a9a92 and f70ef79.

📒 Files selected for processing (5)

• extensions/vscode/src/index.ts
• extensions/vscode/src/providers/document-link/index.ts
• extensions/vscode/src/providers/document-link/npmx.ts
• packages/language-service/src/index.ts
• packages/language-service/src/plugins/document-link.ts

💤 Files with no reviewable changes (3)

• extensions/vscode/src/index.ts
• extensions/vscode/src/providers/document-link/index.ts
• extensions/vscode/src/providers/document-link/npmx.ts

---

📝 Walkthrough

Walkthrough

This change migrates the document link provider from a VS Code extension-side implementation to a language service plugin architecture. The useDocumentLink provider and NpmxDocumentLinkProvider class were removed from the extension, whilst a new npmx-document-link language service plugin was added to provide equivalent functionality. The plugin reads package link configuration, identifies dependencies in compatible files, and generates document links to npmx package URLs based on the configured mode (declared or resolved versions).

Possibly related PRs

• feat: implement simple language server #87: Introduces the language-service plugin scaffold and createNpmxLanguageServicePlugins function that this PR extends with the new document-link service registration.
• refactor: extract providers into composables #69: Directly modifies the document-link provider registration and removal flows within the VS Code extension.
• feat: supports clicking on dependency names to jump #60: Implements the original document-link feature for npmx package names with configuration semantics that are now replicated in the language service plugin.

🚥 Pre-merge checks | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The pull request has no description provided by the author, making it impossible to assess whether any explanation was intended.	Add a pull request description explaining the rationale, changes, and benefits of migrating the document link functionality to the language-service package.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch document-link

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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