molecule-ai/molecule-core
41
0
Fork 2
Code Issues 46 Pull Requests 23 Actions 36 Packages Projects Releases Wiki Activity

fix(infra): status-reaper + watchdog read per-entry status not state (Gitea 1.22.6) #654

Closed
core-devops wants to merge 1 commits from ci/652-gitea-1-22-status-key into main
pull from: ci/652-gitea-1-22-status-key
merge into: molecule-ai:main
Conversation 2 Commits 1 Files Changed 4 +211 -4

1 Commits

Author SHA1 Message Date
Molecule AI Core-DevOps
a2701450c8 fix(infra): status-reaper + watchdog read per-entry status not state (Gitea 1.22.6)
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 17s
Details
Harness Replays / detect-changes (pull_request) Successful in 14s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 36s
Details
CI / Detect changes (pull_request) Successful in 40s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 43s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 44s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 11s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 35s
Details
gate-check-v3 / gate-check (pull_request) Successful in 26s
Details
qa-review / approved (pull_request) Failing after 14s
Details
security-review / approved (pull_request) Failing after 16s
Details
Harness Replays / Harness Replays (pull_request) Successful in 7s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 10s
Details
CI / Platform (Go) (pull_request) Successful in 9s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 8s
Details
CI / Python Lint & Test (pull_request) Successful in 14s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 8s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 6s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
CI / Canvas (Next.js) (pull_request) Successful in 6m21s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 1s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7m24s
Details 
Gitea 1.22.6 combined-status API returns per-entry objects with a
`status` key (NOT `state`). The top-level `state` field is the
aggregate; per-entry has only `status`.

Revisions 1–4 of status-reaper read `s.get("state")` per-entry and
got None for every entry — their compensation logic was unreachable.
The watchdog's `is_red()` and `render_body()` had the same bug at
3 sites, silently missing all per-context failures.

Fix: `s.get("status") or s.get("state") or ""` at every per-entry
read site. Top-level combined-state reads are unaffected.

Tests: 8 new cases (4 reaper + 4 watchdog) covering canonical
`status`-key shape, `status`-over-`state` precedence, backward
compat with `state`-only shape, and non-failure passthrough.
91/91 tests pass.

Refs: molecule-core#652
 2026-05-12 03:52:28 +00:00