fix(cli): suppress ANSI escape codes in non-TTY progress output

[miguel-heygen]

PR Stack

#	PR	Status
1	#122 — fix: info width/height swap	← base
2	#123 — fix: include all 9 templates in build
3	#124 — fix: suppress ANSI in non-TTY
4	#125 — fix: lint --json error paths
5	#126 — fix: separate info/warning counts
6	#127 — fix: lint severity display
7	#128 — fix: render output path error
8	#129 — fix: zero-duration error message	← top

---

Summary

• renderProgress() was writing \r\x1b[2K escape codes even when stdout is not a TTY, corrupting output for CI pipelines and AI agents
• Now checks stdout.isTTY and writes clean text lines at 10% intervals when piped/redirected

Reproducer

cd my-video
npx hyperframes render --output out.mp4 2>&1 | cat
# Output contained raw escape codes visible as garbled text
# Now outputs clean: "  50%  Capturing frames"

Stack

3/8 — Depends on #123

🤖 Generated with Claude Code

[miguel-heygen]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix(engine,cli): download headless-shell instead of using system Chrome for local renders #131
• fix(cli): improve error message for zero-duration composition renders #129
• fix(cli): catch output directory creation errors in render #130
• fix(cli): distinguish info vs warning severity in lint output #127
• fix(core): separate info count from warning count in linter results #126
• fix(cli): respect --json flag on lint error paths #125
• fix(cli): suppress ANSI escape codes in non-TTY progress output #124 👈 (View in Graphite)
• fix(cli): include all 9 templates in build output #123
• fix(cli): read actual data-width/data-height in info command #122
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[miguel-heygen]

Duplicate output lines at 10% boundaries

There's no dedup guard — multiple calls rounding to the same threshold will print duplicate lines. For example, if the producer calls with 49.7 then 50.1, both round to 50 and both satisfy rounded % 10 === 0:

  50%  encoding
  50%  encoding

Fix with a module-level guard:

let lastPrintedThreshold = -1;

// inside the non-TTY branch:
const rounded = Math.round(percent);
if ((rounded % 10 === 0 || rounded === 100) && rounded !== lastPrintedThreshold) {
  lastPrintedThreshold = rounded;
  stdout.write(`  ${rounded}%  ${stage}\n`);
}

Also worth considering: early-return for the non-TTY path to avoid computing the full bar/color line variable that's never used in that branch.

[miguel-heygen]

Addressed: added lastPrintedThreshold dedup guard to prevent duplicate lines. Also moved non-TTY check to early-return at top to skip unused bar/color computation.

[miguel-heygen]

Consolidated into fix/cli-polish.