SubMiner/backlog/tasks/task-322 - Fix-failing-CI-checks-on-PR-57.md at 402b58385d4cf7e2e7c8a8d5c55be74c34e5f17d - SubMiner

sudacode/SubMiner

Fork 0

mirror of https://github.com/ksyasuda/SubMiner.git synced 2026-05-04 00:41:33 -07:00

Files

T

sudacode b926f97578

fix: CI changelog, annotation options threading, and Jellyfin quit

- Add `type: fixed` / `area:` frontmatter to `changes/319` to pass `changelog:lint`
- Thread `TokenizerAnnotationOptions` through `stripSubtitleAnnotationMetadata` so `sourceText` is honored
- Include `jellyfinPlay` in `shouldQuitOnDisconnectWhenOverlayRuntimeInitialized` predicate
- Make mouse test `elementFromPoint` stubs coordinate-sensitive
- Make Lua test `.tmp` mkdir portable on Windows

2026-05-04 00:06:27 -07:00

3.4 KiB

Raw Blame History

id, title, status, assignee, created_date, updated_date, labels, dependencies, references, priority

title

status

assignee

created_date

updated_date

labels

dependencies

references

priority

TASK-322

Fix failing CI checks on PR 57

Done

codex

2026-05-03 06:27

2026-05-03 06:31

bug

https://github.com/ksyasuda/SubMiner/pull/57

high

Description

Investigate and fix failing GitHub Actions checks on PR #57 (tokenizer-updates). Scope: use CI logs to identify root cause, apply focused local fix, and verify with relevant local checks.

Acceptance Criteria

#1 Failing GitHub Actions check root cause is identified from logs.
#2 A focused code/test/docs fix is applied locally.
#3 Relevant local verification passes or blocked reason is documented.
#4 PR checks are rechecked or next CI action is documented.
#5 Actionable CodeRabbit PR comments are inspected and addressed or documented as non-actionable.

Implementation Plan

Fix CI changelog lint by adding a valid type frontmatter value to changes/319-interjection-annotation-filter.md.
Address unresolved CodeRabbit threads:
- scripts/test-plugin-session-bindings.lua: make .tmp creation portable across Unix/Windows shells.
- src/core/services/tokenizer.ts: pass TokenizerAnnotationOptions through stripSubtitleAnnotationMetadata paths so sourceText is honored.
- src/main/runtime/mpv-main-event-main-deps.ts: align overlay-runtime quit-on-disconnect predicate with hasInitialPlaybackQuitOnDisconnectArg.
- src/renderer/handlers/mouse.test.ts: make elementFromPoint stubs coordinate-sensitive.
Run focused checks: bun run changelog:lint, relevant tokenizer/main/mouse tests, and plugin Lua test path if available.
Recheck PR checks/comments after local verification.

Implementation Notes

CI root cause: GitHub Actions build-test-audit failed during bun run changelog:lint; changes/319-interjection-annotation-filter.md must declare type as one of added, changed, fixed, docs, internal. Scope expanded by user to also address CodeRabbit comments on PR #57.

Implemented CI changelog metadata fix and unresolved CodeRabbit feedback locally. Full verification run: bun run changelog:lint, focused tests, bun run typecheck, bun run test:fast, bun run test:env, bun run build, bun run test:smoke:dist, bun run format:check:src. Rechecked PR checks: remote build-test-audit still shows the old failing run until this branch is pushed; CodeRabbit remains pending remotely until review reruns.

Final Summary

Fixed PR #57 CI failure by converting changes/319-interjection-annotation-filter.md to valid changelog fragment metadata. Addressed unresolved CodeRabbit feedback by making plugin test .tmp creation portable, threading tokenizer annotation options through metadata stripping, aligning quit-on-disconnect predicates for Jellyfin playback, and strengthening mouse hit-test assertions. Also formatted two existing PR files required by the source format gate. Verification passed locally: changelog lint, focused tests, typecheck, test:fast, test:env, build, smoke dist, and format check. Remote PR checks still show the previous failed build-test-audit run until these local changes are pushed.

3.4 KiB Raw Blame History