Preserve original stack traces in cross-project panic handling

[Copilot]

handleCrossProject catches panics in worker goroutines and re-panics after RunAndWait, but the re-panic creates a new stack starting at crossproject.go:210. The actual crash site stack is lost from logs and telemetry.

Changes:

• Add core.PanicWithStack struct to wrap a recovered panic value with its original stack trace
• Add core.NewPanicWithStack() constructor that captures the stack and strips recovery infrastructure frames (debug.Stack, deferred recovery function, panic runtime) to avoid duplicating frames in the output
• Store *PanicWithStack in cross-project goroutine recovery instead of formatted strings
• Re-panic with the wrapped struct so the original stack propagates
• In Server.recover, unwrap PanicWithStack to use the original stack for logging and telemetry reporting
• Update sanitizeStackTrace to handle trimmed stacks that no longer start with runtime/debug.Stack()
• Add unit tests for trimPanicRecoveryFrames
• Add a stack sanitizer baseline test (TestSanitizedCrossProjectPanicStackTrace) demonstrating what the sanitized telemetry output looks like for cross-project panics

Before: telemetry/logs show stack starting at the re-panic site (handleCrossProject).
After: telemetry/logs show only the frames from the actual crash site onward, without duplicating recovery infrastructure:

typescript-go|>internal|>checker.(*Checker).checkExpression()
	typescript-go|>internal|>checker|>checker.go:5000
typescript-go|>internal|>ls.(*LanguageService).provideSymbolsAndEntries()
	typescript-go|>internal|>ls|>findallreferences.go:100
typescript-go|>internal|>ls.handleCrossProject[...].func1()
	typescript-go|>internal|>ls|>crossproject.go:105
typescript-go|>internal|>core.(*WorkGroup).worker()
	typescript-go|>internal|>core|>workgroup.go:50
typescript-go|>internal|>core.(*WorkGroup).Queue
	typescript-go|>internal|>core|>workgroup.go:35

[DanielRosenwasser]

@copilot the problem is includes what's reported in telemetry.

1. Intentionally trigger a new panic to see what a new error/stack trace would look like
2. Report back in a comment what that stack error will show up like.
3. Make sure the new stack is reported in telemetry.
4. Add a test for the stack sanitizer.

[Copilot]

@copilot the problem is includes what's reported in telemetry.

1. Intentionally trigger a new panic to see what a new error/stack trace would look like
   ...

Done in 5cdf72f. Here's what the telemetry stack trace will look like with the fix:

Before (re-panic site only, context lost):

typescript-go|>internal|>ls.handleCrossProject[...]()
	typescript-go|>internal|>ls|>crossproject.go:210
typescript-go|>internal|>ls.(*LanguageService).ProvideReferences()
	typescript-go|>internal|>ls|>findallreferences.go:659
...

After (original crash site preserved via PanicWithStack):

typescript-go|>internal|>ls.handleCrossProject[...].func1.1()
	typescript-go|>internal|>ls|>crossproject.go:88
(REDACTED FRAME)
	(REDACTED FRAME)
typescript-go|>internal|>checker.(*Checker).checkExpression()
	typescript-go|>internal|>checker|>checker.go:5000
typescript-go|>internal|>ls.(*LanguageService).provideSymbolsAndEntries()
	typescript-go|>internal|>ls|>findallreferences.go:100
typescript-go|>internal|>ls.handleCrossProject[...].func1()
	typescript-go|>internal|>ls|>crossproject.go:105
typescript-go|>internal|>core.(*WorkGroup).worker()
	typescript-go|>internal|>core|>workgroup.go:50
typescript-go|>internal|>core.(*WorkGroup).Queue
	typescript-go|>internal|>core|>workgroup.go:35

The PanicWithStack.Stack field (captured by debug.Stack() in the worker goroutine) is passed directly to sanitizeStackTrace() for telemetry, so the original stack is what gets reported. Added a baseline test demonstrating this.

[DanielRosenwasser]

@copilot can we avoid re-reporting parts of the stack? Right now the stack looks like

[panic error message]
B
A
B

[Copilot]

@copilot can we avoid re-reporting parts of the stack? Right now the stack looks like

...

Fixed in 70f8be4. NewPanicWithStack now strips the recovery infrastructure frames (debug.Stack, deferred recovery func, panic runtime) from the captured stack before storing it. The telemetry output now shows only the crash site without duplication:

typescript-go|>internal|>checker.(*Checker).checkExpression()
	typescript-go|>internal|>checker|>checker.go:5000
typescript-go|>internal|>ls.(*LanguageService).provideSymbolsAndEntries()
	typescript-go|>internal|>ls|>findallreferences.go:100
typescript-go|>internal|>ls.handleCrossProject[...].func1()
	typescript-go|>internal|>ls|>crossproject.go:105
typescript-go|>internal|>core.(*WorkGroup).worker()
	typescript-go|>internal|>core|>workgroup.go:50
typescript-go|>internal|>core.(*WorkGroup).Queue
	typescript-go|>internal|>core|>workgroup.go:35

[Copilot]

Pull request overview

This PR improves panic diagnostics in the language server’s cross-project request handling by preserving the original crash-site stack trace across goroutine recovery/re-panics, ensuring logs and telemetry reflect where the panic actually occurred (rather than the re-panic site).

Changes:

• Introduces core.PanicWithStack and stack-frame trimming (NewPanicWithStack / trimPanicRecoveryFrames) to retain the original panic stack.
• Updates cross-project goroutine recovery to capture and re-panic with the preserved stack, and updates Server.recover to unwrap and report the original stack in logs/telemetry.
• Extends stack sanitization to handle already-trimmed stacks and adds unit/baseline tests for trimming and sanitization.

Show a summary per file

File	Description
internal/core/panic.go	Adds PanicWithStack wrapper and recovery-frame trimming for preserved panic stacks.
internal/core/panic_test.go	Adds unit tests for trimPanicRecoveryFrames.
internal/ls/crossproject.go	Captures panics in worker goroutines as PanicWithStack and re-panics after RunAndWait.
internal/lsp/server.go	Unwraps PanicWithStack in request recovery to log/report the original stack trace.
internal/lsp/stack_sanitizer.go	Allows sanitizing stacks that don’t start at runtime/debug.Stack() (trimmed stacks).
internal/lsp/stack_sanitizer_test.go	Adds a baseline test covering sanitized cross-project preserved stacks.
testdata/baselines/reference/lsp/stackSanitizer/crossProjectPanicStackTrace.md	New reference baseline for cross-project preserved-stack sanitization output.

Copilot's findings

• Files reviewed: 7/7 changed files
• Comments generated: 1