Broaden empty-catch intent comment recognition

[Copilot]

no-empty-catch-block was only accepting a narrow set of intent-signaling comments, causing documented no-op catches such as non-fatal cleanup and JSON parse fallthrough handling to be reported.

• Intent vocabulary

  • Recognizes additional explicit phrases: non-fatal, ignore/ignored, fall through, no-op, safe to ignore, and constrained swallow phrasing.
  • Keeps vague comments rejected instead of treating any comment as sufficient.

• Negation safeguards

  • Rejects comments that mention accepted phrases in a negative sense, such as do not ignore, can't ignore, do not fall through, or not a no-op.

• Regression coverage

  • Adds valid cases for the live comments that motivated the issue:

try {
  cleanup();
} catch {
  // Cleanup failure is non-fatal.
}

try {
  parseAwContext();
} catch (_parseError) {
  // aw_context is not valid JSON – ignore and fall through
}

• Fixes eslint-factory: no-empty-catch-block comment regex misses common real-world phrasings (grounded FPs) #52844

[Copilot]

Pull request overview

Broadens ESLint empty-catch intent detection while preserving rejection of vague or negated comments.

Changes:

• Adds recognized intent phrases and negation safeguards.
• Adds positive and negative regression cases.

Show a summary per file

File	Description
no-empty-catch-block.ts	Expands comment recognition logic.
no-empty-catch-block.test.ts	Covers new intent and negation phrases.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Balanced

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.87.0