Skip to content

Fixed crash on ConstructSignature when computing interactive inlay hints #61973

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Andarist
Copy link
Contributor

fixes #61965

@Copilot Copilot AI review requested due to automatic review settings June 30, 2025 09:25
@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Jun 30, 2025
@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Jun 30, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a crash when computing interactive inlay hints by adding support for construct signatures.

  • Added a new test case in tests/cases/fourslash/inlayHintsVariableTypes3.ts.
  • Updated the corresponding baseline in tests/baselines/reference/inlayHintsVariableTypes3.baseline.
  • Extended inlay hint processing in src/services/inlayHints.ts to handle construct signatures.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
tests/cases/fourslash/inlayHintsVariableTypes3.ts New test case for construct signature inlay hints.
tests/baselines/reference/inlayHintsVariableTypes3.baseline Baseline update reflecting the new inlay hint output.
src/services/inlayHints.ts Added branch to process construct signatures in inlay hints.

@Andarist Andarist force-pushed the fix/construct-signature-crash-inlay-hints branch from d5a1c6e to 6f432d9 Compare June 30, 2025 09:46
@github-project-automation github-project-automation bot moved this from Not started to Needs merge in PR Backlog Jun 30, 2025
@jakebailey jakebailey merged commit d297037 into microsoft:main Jun 30, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Jun 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Inlay hints are not displayed for certain code
3 participants