Skip to content

Add code-lens to show references and impls - #48151

Closed
mustakimali wants to merge 11 commits into
zed-industries:mainfrom
mustakimali:mustakim/codelens
Closed

Add code-lens to show references and impls#48151
mustakimali wants to merge 11 commits into
zed-industries:mainfrom
mustakimali:mustakim/codelens

Conversation

@mustakimali

@mustakimali mustakimali commented Feb 1, 2026

Copy link
Copy Markdown

Me and Claude trying to implement: #11565

WIP - buggy.

image

@cla-bot

cla-bot Bot commented Feb 1, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@zed-community-bot zed-community-bot Bot added the first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions label Feb 1, 2026
@cla-bot

cla-bot Bot commented Feb 1, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot

cla-bot Bot commented Feb 1, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot

cla-bot Bot commented Feb 1, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@maxdeviant maxdeviant changed the title feat: code-lens to show references and impls Add code-lens to show references and impls Feb 1, 2026
@zed-industries-bot

Copy link
Copy Markdown
Contributor
Warnings
⚠️

This PR is missing release notes.

Please add a "Release Notes" section that describes the change:

Release Notes:

- Added/Fixed/Improved ...

If your change is not user-facing, you can use "N/A" for the entry:

Release Notes:

- N/A
Messages
📖

This PR includes links to the following GitHub Issues: #11565
If this PR aims to close an issue, please include a Closes #ISSUE line at the top of the PR body.

Generated by 🚫 dangerJS against cd72f2e

- Add method to remove refresh tasks from CodeLensCache
- Invalidate code lens cache when relevant LSP events occur
- Re-enable code lens caching in LspStore with correct server/version
  checks
- Ensure code lens cache is cleared when servers change or versions
  update
@cla-bot

cla-bot Bot commented Feb 2, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot

cla-bot Bot commented Feb 3, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot

cla-bot Bot commented Feb 4, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @mustakimali on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@SomeoneToIgnore

Copy link
Copy Markdown
Contributor
image

Hm, seems that CLA is not signed and no activity for a week or so.
Also, sorry for introducing a lot of conflicts, me and Claude were a bit faster with the refactorings: there's a relatively active work happening around.

I suspect I can work on code lens soon after #48780 is over, so maybe that would be a bit less conflicting way to have the feature working?

Will close this one to keep the queue a bit cleaner, thank you.

@mustakimali

Copy link
Copy Markdown
Author

Hm, seems that CLA is not signed and no activity for a week or so. Also, sorry for introducing a lot of conflicts, me and Claude were a bit faster with the refactorings: there's a relatively active work happening around.

I suspect I can work on code lens soon after #48780 is over, so maybe that would be a bit less conflicting way to have the feature working?

Will close this one to keep the queue a bit cleaner, thank you.

That's fine @SomeoneToIgnore - I was making progress very slowly anyway.

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

Labels

first contribution the author's first pull request to Zed. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants