























@@ -251,6 +251,31 @@ Keep it efficient:
251251- Include `--timing-json` on broad or flaky runs when command duration or sync
252252 behavior matters.
253253254+Before/after PR proof on delegated Testbox:
255+256+- For PRs that should prove "broken before, fixed after", compare base and PR
257+ on the same Testbox when practical. Fetch both refs, create detached temp
258+ worktrees under `/tmp`, install in each, then run the same harness twice.
259+- Do not checkout base/PR refs in the synced repo root. Delegated Testbox sync
260+ may leave the root dirty with local files; `git checkout` can abort or mix
261+ proof state.
262+- Temp harness files under `/tmp` do not resolve repo packages by default. Put
263+ the harness inside the worktree, or in ESM use
264+`createRequire(path.join(process.cwd(), "package.json"))` before requiring
265+ workspace deps such as `@lydell/node-pty`.
266+- For full-screen TUI/CLI bugs, a PTY harness is stronger than helper-only
267+ assertions. Use a real PTY, wait for visible lifecycle markers, send input,
268+ then send control keys and assert process exit/stuck behavior.
269+- When validating a rebased local branch before push, remember delegated sync
270+ usually validates synced file content on a detached dirty checkout, not a
271+ remote commit object. Record the local head SHA, changed files, Testbox id,
272+ and final success markers; after pushing, ensure the pushed SHA has the same
273+ file content.
274+- If GitHub CI is still queued but the exact changed content passed Testbox
275+`pnpm check:changed`, `pnpm check:test-types`, and the real E2E proof, it is
276+ reasonable to merge once required checks allow it. Note any still-running
277+ unrelated shards in the proof comment instead of waiting forever.
278+254279Interactive CLI/onboarding:
255280256281- For full-screen or prompt-heavy CLI flows, run the target command inside tmux
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。