Skip to content

fix(lint): avoid false head and audio overlap findings#2348

Open
miguel-heygen wants to merge 1 commit into
mainfrom
fix/lint-comment-and-audio-duration
Open

fix(lint): avoid false head and audio overlap findings#2348
miguel-heygen wants to merge 1 commit into
mainfrom
fix/lint-comment-and-audio-duration

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

Summary

  • inspect only the actual document head so a literal <head> inside body/template script comments is not treated as leaked head markup
  • require explicit clip durations before claiming same-track audio overlap
  • add regression tests for both false positives while preserving real overlap detection

Verification

  • bun run --filter @hyperframes/parsers build
  • bun run --cwd packages/lint test --run src/rules/core.test.ts src/project.test.ts (67 tests passed)
  • bun run --cwd packages/lint typecheck
  • bunx oxfmt --check packages/lint/src/project.ts packages/lint/src/project.test.ts packages/lint/src/rules/core.ts packages/lint/src/rules/core.test.ts
  • git diff --check

Root pre-commit typecheck was also attempted but the fresh worktree lacks generated packages/producer/src/generated/runtime-inline modules; the targeted lint package typecheck passes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant