Add configuration to control existence of "go to <type>" on hover

[kshitijgetsac]

Summary

Fixes #1581
Adds configuration option to control whether hover tooltips display "Go to definition" links.

Following pattern from #1421.

Configuration

{
  "python.analysis.showHoverGoToLinks": false
}

Default: true (backward compatible)

Changes

• workspace.rs: Add config field to LspAnalysisConfig
• server.rs: Read config and pass to hover implementation
• hover.rs: Conditionally show links based on config
• playground.rs: Pass default value (true)
• hover.rs (test): Update test

5 files changed, +18 insertions, -4 deletions

Testing

✅ Tested with config enabled/disabled
✅ Backward compatible (defaults to true)
✅ All existing tests pass

[kshitijgetsac]

@yangdanny97 @kinto0 this is the pr for issue 1581

[kinto0]

Looks great! thanks for working on this. a few suggestions:

• looks like CI is red for formatting, could you run the formatter?
• would you be able to update ide.mdx and lsp/readme.md with a description of the new setting?
• would you be able to add this config value to lsp/package.json so clients like vscode show it in the vscode settings?

[kshitijgetsac]

thanks @kinto0 for the feedback I have made the required changes could you please review it and let me know if anything else needs to be done

[meta-codesync]

@kinto0 has imported this pull request. If you are a Meta employee, you can view this in D87336223.

[yangdanny97]

Review automatically exported from Phabricator review in Meta.

[meta-codesync]

@kinto0 merged this pull request in ed570ab.