SponsorBlock cleanup: dedupe helpers, add filter tests, label a11y #44

Merged

365DevNet merged 0 commits from refs/pull/44/head into main 2026-07-14 01:02:10 +02:00

Conversation 0 Commits - Files changed -

365DevNet commented 2026-07-14 00:44:21 +02:00 (Migrated from codeberg.org)

Copy link

Follow-ups from the final whole-feature integration review (all 12 Phase-1 tasks already merged).

• Dedupe: removes a byte-identical duplicated block in defaults.js (sponsorblockHashPrefix + filterSponsorblockSegments were defined twice — idempotent, harmless, but dead code from a rebase).
• Tests: adds coverage for the constraint-critical poi_highlight min-duration exemption, malformed-segment rejection, and non-array input (22/22 passing).
• A11y: the category-action <select> now has an id and the <label> a matching for= (label click focuses the control).

node --check clean on defaults.js/settings.js; 22 unit tests pass.

🤖 Generated with Claude Code

Follow-ups from the final whole-feature integration review (all 12 Phase-1 tasks already merged). - **Dedupe:** removes a byte-identical duplicated block in `defaults.js` (`sponsorblockHashPrefix` + `filterSponsorblockSegments` were defined twice — idempotent, harmless, but dead code from a rebase). - **Tests:** adds coverage for the constraint-critical `poi_highlight` min-duration exemption, malformed-segment rejection, and non-array input (22/22 passing). - **A11y:** the category-action `<select>` now has an `id` and the `<label>` a matching `for=` (label click focuses the control). `node --check` clean on defaults.js/settings.js; 22 unit tests pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

365DevNet/CleanWatch!44

No description provided.