Skip to content

ci: deterministic performance gate + image/update journeys on every push - #53

Closed
writingdeveloper wants to merge 1 commit into
mainfrom
ci/perf-gate
Closed

writingdeveloper wants to merge 1 commit into
mainfrom
ci/perf-gate

Conversation

@writingdeveloper

Copy link
Copy Markdown
Owner

Summary

  • New qa:perf-gate (Windows CI): fixture agent; fails on idle CSS animations / DOM churn, chrome churn while working, main-process stalls and long UI tasks — units independent of runner speed.
  • Each limit verified by injecting its regression (infinite CSS animation → fail; sidebar render per PTY chunk → 8,395 vs 400; 700ms main block → 710ms vs 600).
  • Windows journey job also runs qa:images, qa:images-remote, qa:update; timeout 10 → 20 min.

Test plan

  • qa:perf-gate passes locally with wide margins (idle 0/0/0, working stall 13ms, outside-terminal mutations 145, stream 0.6s / 13ms / 90ms)
  • three injected regressions each fail it
  • Windows CI job green

🤖 Generated with Claude Code

https://claude.ai/code/session_016NLGwiBY5uEivJqERQzcoY

…n every push

qa:perf-gate drives a fixture agent through idle, working (two tiles repainting at 10fps) and a
20,000-line stream, and fails on what DevDeck has actually shipped before, in units that do not
depend on the runner: running CSS animations and DOM churn on an idle deck, DOM churn outside the
terminals while working, the main process's longest event-loop stall, and long UI tasks. Each limit
was checked against its regression injected on purpose:

  infinite CSS animation          -> idle running animations 1 (limit 0)
  sidebar render per PTY chunk    -> 8,395 mutations outside terminals (limit 400; normal 145)
  700ms sync block in main        -> stream main-process stall 710ms (limit 600; normal ~13)

The Windows journey job also runs qa:images, qa:images-remote and qa:update now, and gets 20 minutes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016NLGwiBY5uEivJqERQzcoY
@writingdeveloper

Copy link
Copy Markdown
Owner Author

Folded into # the 1.39.0 branch (fix/stagger-agent-starts), which carries this commit.

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