Rhizamonitor
live

Run

CI takes 11-18 minutes per PR

done ship-change

Steps

  1. build/builddone31m 17s

    Nov 6 02:03 · 4 of 4 leaves passed

  2. build/renderdone3s

    Nov 6 02:03 · exit 0

  3. build/checkdone6s

    Nov 6 02:03 · exit 0

  4. review/codedone18s

    Nov 6 02:03 · exit 0

  5. review/reviewdone32m 5s

    Nov 6 02:35 · 5 of 5 checks passed

  6. review/renderdone1s

    Nov 6 02:35 · exit 0

  7. render-prdone2s

    Nov 6 02:35 · exit 0

  8. open-prdone6s

    Nov 6 02:35 · exit 0

  9. wait-prdone18m 2s

    Nov 6 02:53 · held after 35 checks

  10. pr-greenfail3s

    Nov 6 02:53 · exit 1 PR #322 at fc1cdb6: failed: rest, check; failing tests: TestMakeTestShardTargets

  11. ci-triagedone0s

    Nov 6 02:53 · exit 0

  12. ci-fixdone1m 41s

    Nov 6 02:55 · said done and until held

  13. render-prdone2s

    Nov 6 02:55 · exit 0

  14. open-prdone5s

    Nov 6 02:55 · exit 0

  15. wait-prdone6m 41s

    Nov 6 03:02 · held after 14 checks

  16. pr-greendone0s

    Nov 6 03:02 · exit 0

  17. signoffdone0s

    Nov 6 03:02 · approved by rule autonomous

  18. rebasedone2s

    Nov 6 03:02 · exit 0

  19. wait-rebaseddone0s

    Nov 6 03:02 · held after 1 checks

  20. rebased-greendone1s

    Nov 6 03:02 · exit 0

  21. landdone13s

    Nov 6 03:02 · exit 0

  22. wait-landeddone5m 41s

    Nov 6 03:08 · held after 12 checks

  23. landed-greendone1s

    Nov 6 03:08 · exit 0

  24. rebase-mergedone1s

    Nov 6 03:08 · exit 0

  25. wait-rebased-mergedone1s

    Nov 6 03:08 · held after 1 checks

  26. rebased-merge-greendone1s

    Nov 6 03:08 · exit 0

  27. mergedone4s

    Nov 6 03:08 · exit 0

  28. wait-maindone1s

    Nov 6 03:08 · held after 1 checks

  29. main-greendone1s

    Nov 6 03:08 · exit 0

  30. close-outdone4s

    Nov 6 03:08 · exit 0

  31. shipdone2s

    Nov 6 03:08 · exit 0

Review

compliance passed

  • minor The acceptance's last point, that the change set's own PR CI run completes in about 5 minutes or less (read from gh run view), cannot be met at review: no PR exists for this change set yet, so there is no run to read. Every other acceptance point of this leaf and of the other three leaves is met as written (the scoped tests in internal/ciworkflow, internal/api -run 'TestParsePRView|TestChecksVerdict' and cmd/rhiza -run 'TestMake|TestScriptShard' all pass at 2f747105). The duration should be read from the ship run's PR CI. .github/workflows/ci.yml:23

quality passed

  • minor test-api's go test -list runs in a pipeline whose status is sort's, so a compile error or TestMain failure in internal/api leaves names empty and the shard prints 'has no tests' and exits 0: both api jobs go green. CI still fails through vet and the budgets job (which compile internal/api), but the api shards themselves hide the failure; set pipefail or check go test -list's status separately. Makefile:132
  • minor The check target's comment still says it is 'the single entry point .github/workflows/ci.yml runs, so local and CI runs cannot drift apart'; CI now runs lint/test-scripts/test-api/test-rest/test-budgets instead, so the comment is stale. Makefile:173
  • nit shardFiles globs only *.txtar, while testscript's Dir mode also runs *.txt scripts; a .txt script would run locally (unsharded) but in no CI shard. None exist today. cmd/rhiza/main_test.go:55
  • nit '_ = err' after Sscanf is noise; also Sscanf's %d accepts '+1/4' and leading spaces. Harmless. cmd/rhiza/main_test.go:69
  • nit Local func named make shadows the builtin. cmd/rhiza/makeupdate_test.go:160
  • nit TestScriptShardEnvError duplicates the 'malformed 0/4' subtest of TestScriptShard. cmd/rhiza/shard_test.go:67

qa passed

No findings.

scope passed

No findings.

model passed

  • minor Capability budgets-measure-in-quiet (decided by budgets-run-alone) says the timing budgets run 'on their own after the rest of the test suite', and its acceptance ends 'CI's make check passes'. In CI the budgets now run as their own job, at the same time as the other jobs but on a separate runner, and CI no longer runs make check (make test locally still runs them after the suite). The purpose still holds, since the budgets measure without their neighbours' load, but the wording no longer matches CI. ci-speed-shape could refine that capability's statement. .github/workflows/ci.yml:110
  • minor Still open from cycle 1: CLAUDE.md says the PR's GitHub CI runs `make check`. CI now runs make check's slices as parallel jobs (make lint, test-scripts, test-api, test-rest, test-budgets). The same tests run, but the wording is stale. CLAUDE.md:38

Log

log · tail
- render-pr (visit 2): pass, then open-pr: exit 0
- open-pr (visit 2): pass, then wait-pr: exit 0
- wait-pr (visit 2): pass, then pr-green: held after 14 checks
- pr-green (visit 2): pass, then signoff: exit 0
- signoff (visit 1): pass, then rebase: approved by rule autonomous
- rebase (visit 1): pass, then wait-rebased: exit 0
- wait-rebased (visit 1): pass, then rebased-green: held after 1 checks
- rebased-green (visit 1): pass, then land: exit 0
- land (visit 1): pass, then wait-landed: exit 0
- wait-landed (visit 1): pass, then landed-green: held after 12 checks
- landed-green (visit 1): pass, then rebase-merge: exit 0
- rebase-merge (visit 1): pass, then wait-rebased-merge: exit 0
- wait-rebased-merge (visit 1): pass, then rebased-merge-green: held after 1 checks
- rebased-merge-green (visit 1): pass, then merge: exit 0
- merge (visit 1): pass, then wait-main: exit 0
- wait-main (visit 1): pass, then main-green: held after 1 checks
- main-green (visit 1): pass, then close-out: exit 0
- close-out (visit 1): pass, then ship: exit 0
- ship (visit 1): pass, then end: exit 0

## Leaves

- test-scripts-sharded (step build/build): passed: attempt 1: verifier passed
  - attempt 1 builder, session 2a6ef5c0-eb42-412d-a788-01a1ae7c1c1f: pass (said done)
  - attempt 1 verifier, session e8b74584-361f-4864-9503-01a1ae8b315c: pass (Evidence written by the verifier)
- ship-check-multi-job-verdict (step build/build): passed: attempt 1: verifier passed
  - attempt 1 builder, session 19ce5fb3-4f41-4568-b6a1-01a1ae8ba8a3: pass (said done)
  - attempt 1 verifier, session 43f7013f-1434-403e-abe5-01a1ae8e24a8: pass (Evidence written by the verifier)
- make-test-shard-targets (step build/build): passed: attempt 1: verifier passed
  - attempt 1 builder, session a2e7b890-d635-441a-8973-01a1ae8f657f: pass (said done)
  - attempt 1 verifier, session 3a599979-5d89-4a7e-ab97-01a1ae94999b: pass (Evidence written by the verifier)
- ci-yml-parallel-jobs (step build/build): passed: attempt 1: verifier passed
  - attempt 1 builder, session 0eeaef88-1eac-43d6-b6a6-01a1ae9612f5: pass (said done)
  - attempt 1 verifier, session bad97f9a-7582-4ae8-a0c2-01a1ae97641e: pass (Evidence written by the verifier)
- ci-yml-parallel-jobs (step review/review): open
  - attempt 2 fixer in cycle 2, session 701011da-86fe-49ce-9b74-01a1aeb2152b: pass (said done)
  - attempt 2 verifier in cycle 2, session 6e32a8c4-df14-45ae-9272-01a1aeb32ce3: pass (Evidence written by the verifier)

## Checks

- compliance (step review/review): passed after 1 cycle
  - cycle 1: passed, check session 7cbeaf77-ff21-4dd3-ac12-01a1ae990f04 pass (said done)
  - round 1 (cycle 1): 1 finding, 0 confirmed
    - f1 minor at .github/workflows/ci.yml:23 on ci-yml-parallel-jobs: The acceptance's last point, that the change set's own PR CI run completes in about 5 minutes or less (read from gh run view), cannot be met at review: no PR exists for this change set yet, so there is no run to read. Every other acceptance point of this leaf and of the other three leaves is met as written (the scoped tests in internal/ciworkflow, internal/api -run 'TestParsePRView|TestChecksVerdict' and cmd/rhiza -run 'TestMake|TestScriptShard' all pass at 2f747105). The duration should be read from the ship run's PR CI.
- quality (step review/review): passed after 1 cycle
  - cycle 1: passed, check session 30ff6d81-ace8-4507-bb8d-01a1ae995f25 pass (said done)
  - round 1 (cycle 1): 6 findings, 0 confirmed
    - f1 minor at Makefile:132: test-api's go test -list runs in a pipeline whose status is sort's, so a compile error or TestMain failure in internal/api leaves names empty and the shard prints 'has no tests' and exits 0: both api jobs go green. CI still fails through vet and the budgets job (which compile internal/api), but the api shards themselves hide the failure; set pipefail or check go test -list's status separately.
    - f2 minor at Makefile:173: The check target's comment still says it is 'the single entry point .github/workflows/ci.yml runs, so local and CI runs cannot drift apart'; CI now runs lint/test-scripts/test-api/test-rest/test-budgets instead, so the comment is stale.
    - f3 nit at cmd/rhiza/main_test.go:55: shardFiles globs only *.txtar, while testscript's Dir mode also runs *.txt scripts; a .txt script would run locally (unsharded) but in no CI shard. None exist today.
    - f4 nit at cmd/rhiza/main_test.go:69: '_ = err' after Sscanf is noise; also Sscanf's %d accepts '+1/4' and leading spaces. Harmless.
    - f5 nit at cmd/rhiza/makeupdate_test.go:160: Local func named make shadows the builtin.
    - f6 nit at cmd/rhiza/shard_test.go:67: TestScriptShardEnvError duplicates the 'malformed 0/4' subtest of TestScriptShard.
- qa (step review/review): passed after 1 cycle
  - cycle 1: passed, check session 49d75309-b42b-4037-811b-01a1ae99b342 pass (said done)
  - round 1 (cycle 1): no findings
- scope (step review/review): passed after 1 cycle
  - cycle 1: passed, check session 8655f2a2-704d-4c7d-9465-01a1ae99fd4c pass (said done)
  - round 1 (cycle 1): no findings
- model (step review/review): passed after 2 cycles
  - cycle 1: failed, check session 64b92334-4f92-475b-b21e-01a1ae9a570e pass (said done), critic session 9be6f184-150b-4092-abe1-01a1aeb167a8 pass (said done)
  - cycle 2: passed, check session 9dd2870f-0945-4b8a-85e8-01a1aeb4ee94 pass (said done)
  - round 1 (cycle 1): 3 findings, 1 confirmed
    - f1 major at .github/workflows/ci.yml:25 on ci-yml-parallel-jobs: The model still holds Decision t1-d12-ci-one-job (01M5SWRRKB, 'CI is one job on ubuntu-26.04 ... setup-go@v7 (go-version-file: go.mod), make check'), linked from self-check, and nothing retires it: ci-speed-shape has no supersedes link to it. The code now runs six jobs (lint, scripts x4, api x2, rest, budgets, check) calling per-job make targets with the setup-go cache, so the model holds two live, contradicting decisions about CI's shape. Add 'ci-speed-shape supersedes t1-d12-ci-one-job' in this change set. — confirmed: rhiza show t1-d12-ci-one-job: unretired Decision stating 'CI is one job ... make check', decided-by self-check, with no supersedes link in (ci-speed-shape links out only answers and derived-from). ci.yml at HEAD runs lint, scripts x4, api x2, rest, budgets and an aggregate check job calling per-slice make targets, so the two live decisions contradict; a supersedes link in this change set retires the stale one.
    - f2 minor at CLAUDE.md:38: CLAUDE.md still says the PR's GitHub CI runs `make check` (lines 29 and 38-39); CI now runs make check's slices as parallel jobs (make lint, test-scripts, test-api, test-rest, test-budgets). The meaning holds (the same tests run), but the wording no longer matches ci.yml.
    - f3 minor at Makefile:130 on make-test-shard-targets: test-api's `names=$(go test -list ... | grep | grep -Ev | sort) || exit 1` takes sort's status, so a compile error in internal/api makes a shard print 'no tests' and pass. The leaf says the targets cover what make test runs; here that holds only because lint (vet) and budgets also fail. No constraint in the model states that a CI shard job must fail when its package does not build.
  - round 2 (cycle 2): 2 findings, 0 confirmed
    - f1 minor at .github/workflows/ci.yml:110: Capability budgets-measure-in-quiet (decided by budgets-run-alone) says the timing budgets run 'on their own after the rest of the test suite', and its acceptance ends 'CI's make check passes'. In CI the budgets now run as their own job, at the same time as the other jobs but on a separate runner, and CI no longer runs make check (make test locally still runs them after the suite). The purpose still holds, since the budgets measure without their neighbours' load, but the wording no longer matches CI. ci-speed-shape could refine that capability's statement.
    - f2 minor at CLAUDE.md:38: Still open from cycle 1: CLAUDE.md says the PR's GitHub CI runs `make check`. CI now runs make check's slices as parallel jobs (make lint, test-scripts, test-api, test-rest, test-budgets). The same tests run, but the wording is stale.

## Code checks

- step review/code: pass

## Sign-offs

- signoff (visit 1): approved by rule autonomous

## Sessions

- build/build, session 2a6ef5c0-eb42-412d-a788-01a1ae7c1c1f: tokens input 38, output 8169, cache creation 42851, cache read 1157163; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session e8b74584-361f-4864-9503-01a1ae8b315c: tokens input 16, output 2817, cache creation 26932, cache read 398306; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session 19ce5fb3-4f41-4568-b6a1-01a1ae8ba8a3: tokens input 12, output 2173, cache creation 29835, cache read 295465; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session 43f7013f-1434-403e-abe5-01a1ae8e24a8: tokens input 12, output 2184, cache creation 27694, cache read 292112; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session a2e7b890-d635-441a-8973-01a1ae8f657f: tokens input 24, output 9814, cache creation 39277, cache read 697401; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session 3a599979-5d89-4a7e-ab97-01a1ae94999b: tokens input 14, output 3864, cache creation 39716, cache read 383086; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session 0eeaef88-1eac-43d6-b6a6-01a1ae9612f5: tokens input 8, output 3226, cache creation 28938, cache read 182559; helper tokens input 0, output 0, cache creation 0, cache read 0
- build/build, session bad97f9a-7582-4ae8-a0c2-01a1ae97641e: tokens input 14, output 2733, cache creation 28865, cache read 351170; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 7cbeaf77-ff21-4dd3-ac12-01a1ae990f04: tokens input 20, output 3280, cache creation 44346, cache read 576463; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 30ff6d81-ace8-4507-bb8d-01a1ae995f25: tokens input 14, output 7201, cache creation 46233, cache read 411282; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 49d75309-b42b-4037-811b-01a1ae99b342: tokens input 42, output 8330, cache creation 59770, cache read 1416089; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 8655f2a2-704d-4c7d-9465-01a1ae99fd4c: tokens input 14, output 1862, cache creation 40290, cache read 375233; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 64b92334-4f92-475b-b21e-01a1ae9a570e: tokens input 28, output 5859, cache creation 53507, cache read 954140; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 9be6f184-150b-4092-abe1-01a1aeb167a8: tokens input 10, output 1537, cache creation 27741, cache read 235067; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 701011da-86fe-49ce-9b74-01a1aeb2152b: tokens input 12, output 1442, cache creation 27660, cache read 294799; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 6e32a8c4-df14-45ae-9272-01a1aeb32ce3: tokens input 22, output 2954, cache creation 29890, cache read 574001; helper tokens input 0, output 0, cache creation 0, cache read 0
- review/review, session 9dd2870f-0945-4b8a-85e8-01a1aeb4ee94: tokens input 24, output 4937, cache creation 54354, cache read 812877; helper tokens input 0, output 0, cache creation 0, cache read 0
- ci-fix, session 58f5529d-569f-4077-93ff-01a1aec70d57: tokens input 18, output 2980, cache creation 33477, cache read 490666; helper tokens input 0, output 0, cache creation 0, cache read 0

Totals: tokens input 342, output 75362, cache creation 681376, cache read 9897879; helper tokens input 0, output 0, cache creation 0, cache read 0