Fix/dead code and doc accuracy

[ionutcnu]

No description provided.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

• New Features

  • "Ignore this" button on toast to dismiss errors per session or permanently
  • Max toasts cap prevents UI flooding
  • Ignored errors management page to view and remove permanently ignored errors
  • Error deduplication grouped by type, message, file, and line
  • Visual customization options
  • Export history functionality

• Changes

  • Error history drawer now caps at 50 errors (configurable 5–50)
  • Resource and network error tracking removed

Walkthrough

Documentation and code cleanup removing deprecated features: IgnoreRule interface, rateLimit configuration, and resource/network error type support. Terminology updated from "ignore rules" to "ignored error signatures" and "error ignore" across documentation.

Changes

Cohort / File(s)	Summary
Documentation Updates CHANGELOG.md, PRIVACY_POLICY.md, README.md	Terminology shifts from "ignore rules" to "ignored error signatures" and "error ignore"; updated feature descriptions reflecting removal of rateLimit and resource/network error types; adjusted permission descriptions for activeTab and storage references.
Type Definitions src/types/index.ts	Removed IgnoreRule interface and CatchySettings.ignoreRules property; removed CatchySettings.rateLimit block with maxPerInterval and intervalMs; eliminated associated comments and DEFAULT_SETTINGS entries.
Content Script src/content/index.ts	Removed IgnoreRule from imports; removed ignoreRules and rateLimit from CatchySettings interface and DEFAULT_SETTINGS; removed resource/network from isErrorTypeAllowed type mapping; migrated color properties (backgroundColor/textColor → backgroundColors/textColors) with migration logic adjustments.
Settings Hook src/options/hooks/useSettings.ts	Removed rateLimit merging logic from merged settings object during load from storage.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• PR #14: Removes IdentifyRule, rateLimit, and resource/network error types—identical code-level changes across src/types, src/content, and useSettings.
• PR #12: Modifies error-ignore behavior, toast UI, and related types (IgnoreRule, CatchySettings, ToastManager) with overlapping codepaths.
• PR #4: Content-script toast changes (shadow-DOM flow, max-toasts cap, position handling) that interact with removed rateLimit functionality.

Poem

🧹 Deprecated dust swept away,
Rules and limits—yesterday's way,
Clean slate awaits, code takes flight,
Ignored errors, burning bright! 🔥

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/dead-code-and-doc-accuracy

---

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.}