Skip to content

Commit ffe8a8d

Browse files
committed
chore(lefthook): grandfather 3 pre-existing over-cap engine files
screenshotService.ts (516 LoC), frameCapture.ts (791 LoC), and videoFrameInjector.ts (610 LoC) are pre-existing over the 500-line lefthook cap added in #748. Any PR touching them currently must ship with --no-verify (see PR #787 which hit exactly this). This change extends the filesize hook's case-allowlist (already used for useTimelinePlayer.ts + App.tsx) to cover the three engine files so future PRs to them can land cleanly. Splitting them properly is a separate effort tracked as tech debt; this unblocks the hf#677 perf chain in the meantime.
1 parent 0340774 commit ffe8a8d

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎lefthook.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ pre-commit:
1717
exclude: "(\\.test\\.(ts|tsx)$|\\.generated\\.)"
1818
run: |
1919
for f in {staged_files}; do
20-
case "$f" in *useTimelinePlayer.ts|*App.tsx) continue ;; esac
20+
case "$f" in *useTimelinePlayer.ts|*App.tsx|*screenshotService.ts|*frameCapture.ts|*videoFrameInjector.ts) continue ;; esac
2121
lines=$(wc -l < "$f")
2222
if [ "$lines" -gt 500 ]; then
2323
echo "ERROR: $f has $lines lines (max 500)"

0 commit comments

Comments
 (0)