• chore(skills): sprint 36 retro process improvements

    jpmschweitzer released this 2026-04-21 15:30:14 +02:00

    • pr-review: require manual smoke test for merge-path PRs (pre-game flow,
      scene transitions, save/load, connection handshake). Unchecked
      author-side smoke boxes downgrade verdict from APPROVED to
      REQUEST_CHANGES. Triggered by #872 — New Game hangs on 'connecting'
      landed through PR #134 with the exact smoke box unchecked.
    • pr-push: wipe Godot script class cache before the runtime smoke check
      so cold-start parse-order bugs surface pre-push instead of failing
      the next fresh-clone parse. Sprint 36 hit this post-merge — MetaScreen
      base class not found because the cache was warm on developer machines
      but cold in CI / post-merge. Cold-cache check stays in the skill (not
      the pre-push hook) to keep iteration fast.
    Downloads