ux: filter findings, non-preview open, quieter copy toasts

[dmartinochoa]

Summary

Three small UX wins from the post-v0.2.0 review queue. Stacked on #20 — base is `ux-polish-2`.

What landed

#5 — Filter the Findings tree

• `Pipeline-Check: Filter Findings` command opens an InputBox; matches against rule ID, message body, and file path (case-insensitive substring).
• Re-invoking the command pre-fills the current filter so users can edit or clear it (empty input clears).
• New `$(filter)` button on the Findings view title bar, between Scan and Change Grouping.
• The badge updates to reflect the filtered count.
• `lastFindingUris` still tracks the unfiltered universe so a publish for a currently-hidden URI still wakes the tree up — fixes a subtle bug class where a CLEAR of a filtered-out finding would never refresh.

#4 — Open Finding (non-preview)

• `Pipeline-Check: Open Finding` context-menu entry on tree leaves.
• Opens with `preview: false` — pins the file as a permanent tab so triaging multiple findings side-by-side doesn't create tab clutter.
• The single-click default still uses preview, keeping the "scan through findings to triage" flow lightweight.

#7 — Quieter clipboard confirmations

• Copy Rule ID and Copy LSP Install Command switch from `showInformationMessage` (modal toast) to `setStatusBarMessage` with a 2-second TTL.
• The copy succeeds silently 95% of the time anyway; the status-bar message confirms the action without stealing focus.

Test plan

• `npm run lint` clean
• `npm run compile` clean
• `npm test` — 137 tests pass (was 129 on ux: status bar severity color + what's-new notification #20; +8 filter behaviour)
• `npm run smoke` clean
• `npm run test:integration:compile` clean
• Manual under F5: open the sample-workflow fixture; type `GHA` in Filter Findings → only GHA-* findings appear; clear filter (empty input) → all findings return.
• Manual: right-click a leaf → "Open Finding" → opens as a permanent (non-italic) tab.
• Manual: right-click a leaf → "Copy Rule ID" → status-bar message confirms.

Notes

• The two leaf-context commands (`openNonPreview`, `openRuleDocs`, `copyRuleId`) all use the same `commandPalette: when: false` gating so they don't show up palette-wide without a node argument. Standard pattern; same as before.
• The filter command's pre-fill behaviour is its own affordance: a user sees their current filter when they re-invoke. If you wanted a separate "Clear Filter" command for keybinding convenience, that's a one-liner follow-up.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4225d382-7430-4993-8ebc-c107b103e3b7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ux-polish-3

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[dmartinochoa]

Closing as duplicate of #22 (same head branch ux-polish-3; #21's base ux-polish-2 is already merged into main). #22 carries the same commits and was merged to main.