Remove unused browser polyfill dependencies

[rak-phillip]

[github-actions]

🎯 Agentic Issue Triage

This is a dependency cleanup task (sub-issue of epic #17622) to remove two browser polyfill packages — element-matches and custom-event-polyfill — that are no longer needed in modern browsers.

Both polyfills are currently imported in shell/plugins/shortkey.js:

import 'element-matches';
import 'custom-event-polyfill';

📋 Analysis

• element-matches polyfills Element.matches(), which has been natively supported in all major browsers since 2015 (Chrome 33, Firefox 34, Safari 9, Edge 12). It is safe to remove.
• custom-event-polyfill polyfills CustomEvent, natively available in all modern browsers and Node ≥ 18. Safe to remove.
• These are the only import sites for both packages in the codebase.
• Removing the imports from shortkey.js and the entries from package.json is the full scope of this task.

✅ Suggested Sub-tasks

• Remove import 'element-matches'; from shell/plugins/shortkey.js
• Remove import 'custom-event-polyfill'; from shell/plugins/shortkey.js
• Remove element-matches from package.json dependencies
• Remove custom-event-polyfill from package.json dependencies
• Run yarn install --frozen-lockfile to update lock file
• Run yarn test:ci and yarn lint to confirm nothing is broken
• Verify shortkey functionality still works (keyboard shortcuts in the UI)

🔍 Debugging / Verification Tips

• After removal, manually test keyboard shortcuts in the dashboard (the shortkey plugin handles global keyboard bindings).
• element-matches was used internally by the shortkey library to match DOM elements — confirm the shortkey plugin itself doesn't rely on the polyfill at runtime via a bundled dependency rather than a direct import. Check node_modules/vue-shortkey (or whichever shortkey lib is used) to see if it bundles its own polyfill.
• Run a quick browser smoke test to ensure no Element.matches is not a function or CustomEvent is not a constructor errors in the console.

📚 Relevant Resources

• [MDN: Element.matches() — browser compatibility]((developer.mozilla.org/redacted)
• [MDN: CustomEvent — browser compatibility]((developer.mozilla.org/redacted)
• Parent Epic #17622 — Remove unused and misplaced dependencies

Generated by Agentic Triage for issue #17625 · ● 230.9K · ◷

[github-actions]

🤖 Issue grooming analysis

This issue meets all four primary grooming criteria and is ready for triage. While the issue body is empty, the triage analysis comment provides sufficient technical context for an engineer to scope and complete the work.

Criterion	Status	Notes
Clear Problem Statement	✅	Identifies element-matches and custom-event-polyfill as browser polyfills no longer needed in modern browsers; only import site is shell/plugins/shortkey.js
Acceptance Criteria	✅	Triage comment provides a detailed sub-task checklist covering import removal, package.json cleanup, and verification steps
Technical Details	✅	Specific packages, versions, and the exact file containing the imports are identified; browser compatibility rationale is provided
Reproducibility	✅	Tech-debt cleanup task with clearly bounded scope

This analysis was generated automatically based on the repository's grooming checklist.

Generated by Daily Issue Grooming · ● 584.1K · ◷