mirror of
https://github.com/garrytan/gstack.git
synced 2026-11-01 22:45:02 +01:00
docs(todos): close the createQAFunctionalFixture entry and file D6 (W3-13)
The git auto-maintenance entry was fixed by 0bf9d50 (#3190). D6: nested bun
test output pollutes free-runner failure attribution in
scripts/lib/shard-engine.ts (windows-free-tests 39053431430).
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
This commit is contained in:
1 parent
d9cfa873ec
commit
d9b5131807
1 file changed
+17
-5
@@ -5425,11 +5425,23 @@ Garry for each; revisit 2026-10-29 unless a date says otherwise.
|
||||
main's `flake-ledger-*` artifacts (`bun run test:health --since-days 7`) and
|
||||
check the free-suite flaky-pass rate is under 5% of main runs; a file still in
|
||||
the ledger gets a named cause or a new entry here. **Effort:** S. **Priority:** P1.
|
||||
- **`createQAFunctionalFixture` sets git auto-maintenance off itself** — its
|
||||
repos inherit the free runner's backstop keys in CI, but a direct `bun test`
|
||||
run still races `.git/objects/maintenance.lock` during observer setup.
|
||||
The helper selects 22 paid cases, so this is a priced follow-up.
|
||||
**Effort:** S. **Priority:** P2.
|
||||
- ~~**`createQAFunctionalFixture` sets git auto-maintenance off itself**~~ —
|
||||
done in v1.91.107.0 (0bf9d50, #3190): test/helpers/qa-functional-fixture.ts
|
||||
sets `gc.auto 0` and `maintenance.auto false` on its repos, as
|
||||
test/helpers/qa-callers-fixture.ts does; no recurrence of the
|
||||
`.git/objects` watch loss after it (fix wave 3 triage, W3-13). Original
|
||||
note: — its repos inherit the free runner's backstop keys in CI, but a direct
|
||||
`bun test` run still races `.git/objects/maintenance.lock` during observer
|
||||
setup. **Effort:** S. **Priority:** P2.
|
||||
- **Nested `bun test` output pollutes free-runner failure attribution** (fix
|
||||
wave 3, D6) — windows-free-tests run 39053431430 (shard 5): a free test that
|
||||
spawns a nested `bun test` echoes the child's `(fail) Review Army: Red Team >
|
||||
(unnamed)` line on failure, and scripts/lib/shard-engine.ts attributed it to
|
||||
the child's temporary file (`redteam.test.ts`), so the shard reported
|
||||
"flaky-retry skipped: failures not fully attributed" instead of retrying the
|
||||
real file. Ignore or prefix nested child result lines when attributing
|
||||
failures, with a free test that feeds the captured shard log through the
|
||||
attribution. **Effort:** S. **Priority:** P2.
|
||||
- **`test:ubicloud --diagnostic` cannot give clean counts for two ledger
|
||||
shards** — `ship-measure free` reruns set a private `HOME` under `/tmp` and
|
||||
`GSTACK_STATE_ROOT`, so `browse/test/path-validation.test.ts` (its upload
|
||||
|
||||
Reference in new issue
Block a user