"Ignore" to unflag all occurrences

[Copilot]

Clicking "Ignore" on a flagged word only dismissed that single occurrence. Other identical flagged words remained highlighted despite the word being added to the dictionary.

• Changed ignoreDiagnostic filter predicate from positional matching (from/to) to semantic matching by problemText only (lint kind is not checked, since the word is added to the dictionary regardless of kind)
• When problemText is empty, falls back to positional matching (from/to/lintKind) to avoid over-removing unrelated diagnostics
• Added unit tests for ignoreDiagnostic covering: all same-text removal, cross-kind removal, different-text preservation, and empty-text positional fallback

- !(d.from === diag.from && d.to === diag.to && d.lintKind === diag.lintKind),
+ diag.problemText
+   ? d.problemText !== diag.problemText
+   : !(d.from === diag.from && d.to === diag.to && d.lintKind === diag.lintKind),

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

Updates the “Ignore” behavior in the diagnostic card helpers so that ignoring a flagged word can clear all matching occurrences (by text + lint kind), aligning UI highlights with dictionary additions.

Changes:

• Change ignoreDiagnostic filtering from positional matching (from/to) to semantic matching (problemText/lintKind).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[cyanzhong]

@copilot Do we need to check kind?

[Copilot]

@copilot Do we need to check kind?

No, we don't need to check kind. Since the word gets added to the dictionary, Harper won't flag it under any lint kind in the future. Updated to match by problemText only, with a positional fallback for empty problemText. See 7bf15f5.