can1357
fbdc064186
- Added `DiagnosticsLedger` to track diagnostics already surfaced per file, suppressing repeats within a session. - Wired dedup into both edit and write tools via a `transformDiagnostics` hook on the writethrough pipeline. - Added `lsp.diagnosticsDeduplicate` setting (default: true) to control the behavior.
52 lines
1.3 KiB
TypeScript
52 lines
1.3 KiB
TypeScript
import type { FileDiagnosticsResult } from "./index";
|
|
import { summarizeDiagnosticMessages } from "./utils";
|
|
|
|
const DIAGNOSTIC_LOCATION_PREFIX_RE = /^.*?:\d+:\d+\s+/;
|
|
|
|
export function diagnosticIdentity(message: string): string {
|
|
return message.replace(DIAGNOSTIC_LOCATION_PREFIX_RE, "");
|
|
}
|
|
|
|
export class DiagnosticsLedger {
|
|
readonly #seen = new Map<string, Set<string>>();
|
|
|
|
reduce(absPath: string, result: FileDiagnosticsResult): FileDiagnosticsResult {
|
|
const previous = this.#seen.get(absPath);
|
|
const currentIdentities = new Set<string>();
|
|
const fresh: string[] = [];
|
|
|
|
for (const message of result.messages) {
|
|
const identity = diagnosticIdentity(message);
|
|
currentIdentities.add(identity);
|
|
if (!previous?.has(identity)) {
|
|
fresh.push(message);
|
|
}
|
|
}
|
|
|
|
if (currentIdentities.size === 0) {
|
|
this.#seen.delete(absPath);
|
|
} else {
|
|
this.#seen.set(absPath, currentIdentities);
|
|
}
|
|
|
|
if (fresh.length === result.messages.length) {
|
|
return result;
|
|
}
|
|
|
|
return {
|
|
...result,
|
|
messages: fresh,
|
|
...summarizeDiagnosticMessages(fresh),
|
|
};
|
|
}
|
|
}
|
|
|
|
export interface DiagnosticsLedgerOwner {
|
|
diagnosticsLedger?: DiagnosticsLedger;
|
|
}
|
|
|
|
export function getDiagnosticsLedger(owner: DiagnosticsLedgerOwner): DiagnosticsLedger {
|
|
owner.diagnosticsLedger ??= new DiagnosticsLedger();
|
|
return owner.diagnosticsLedger;
|
|
}
|