Skip to content

feat: display package provenance in hover#230

Merged
calebdw merged 1 commit into
mainfrom
calebdw/push-yvupyyskrozo
Jul 10, 2026
Merged

feat: display package provenance in hover#230
calebdw merged 1 commit into
mainfrom
calebdw/push-yvupyyskrozo

Conversation

@calebdw

@calebdw calebdw commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Show a colored badge in hover indicating where a symbol comes from:

  • 🟢 package/name for direct Composer dependencies
  • 🟠 package/name (transitive) for transitive dependencies
  • 🟣 PHP for core/extension symbols
  • No badge for project-local symbols

The vendor_package_origin_roots data structure now stores the Composer package name alongside the origin tier and install path, read from vendor/composer/installed.json. New package_info_for_path/uri methods on Backend return both the origin and the package name.

Provenance is shown in hover for classes, methods, properties, constants, and standalone functions.

Closes #228.

Show a colored badge in hover indicating where a symbol comes from:
- 🟢 `package/name` for direct Composer dependencies
- 🟠 `package/name` *(transitive)* for transitive dependencies
- 🟣 `PHP` for core/extension symbols
- No badge for project-local symbols

The vendor_package_origin_roots data structure now stores the Composer
package name alongside the origin tier and install path, read from
vendor/composer/installed.json. New package_info_for_path/uri methods
on Backend return both the origin and the package name.

Provenance is shown in hover for classes, methods, properties,
constants, and standalone functions.

Closes #228.
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 75.00000% with 18 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/hover/formatting.rs 68.75% 5 Missing ⚠️
src/hover/mod.rs 80.76% 5 Missing ⚠️
src/lib.rs 77.77% 4 Missing ⚠️
src/completion/resolve.rs 0.00% 2 Missing ⚠️
src/blade/mod.rs 0.00% 1 Missing ⚠️
src/classmap_scanner.rs 88.88% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@calebdw calebdw merged commit 8f4a79d into main Jul 10, 2026
7 checks passed
@calebdw calebdw deleted the calebdw/push-yvupyyskrozo branch July 10, 2026 22:27
@AJenbo

AJenbo commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Very nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: Display package in textDocument/hover

3 participants