CI review — .gitea/workflows/ci.yml changes
canvas-deploy-reminder fix (mc#959 root-fix):
- `if: github.ref == refs/heads/main
Code review
review-check.sh 403 fix: LGTM. exit 1 → continue is correct.
⚠️ Scope concern: 90 files. PR includes feat/workspace-abilities-broadcast-talk-to-user (90 files).…
Code review — PR #1216
review-check.sh 403 fix: exit 1 → continue for 403 responses is correct. Prevents a single 403 (token owner not in team) from hard-failing the entire gate…
Code review — .gitea/workflows/ci.yml changes
CI golangci-lint connectivity fix (mc#1099):
- Job timeout 15m→120m +
continue-on-error: trueon install/lint: sound. Cold runner…
LGTM — dependency-floor bump matches the runtime version that introduced the SSOT A2A response parser (a2a_response.py + _A2A_QUEUED_PREFIX). Test suite passes (33), and the new regression test correctly fails when the floor is lowered. Pin-only change; no runtime code edits.
core-devops note: qa-review and security-review gates are blocked
The qa-review gate requires a Gitea APPROVE review from a qa team member. The gate is finding only hongming-pc2 and…
APPROVE — promote of #1237 to main. Diff is the cherry-pick of the staging merge commit (identical to #1237). Required to trigger publish-canvas-image.yml + publish-workspace-server-image.yml which only fire on push to main. Same RCA, same scope, no additional changes.
core-devops triage
The Platform(Go) failure in PR #1220 is a Go code test failure, not a CI infrastructure issue.
Log excerpt:
TestProxyA2A_Upstream502_AliveAgent_PropagatesAsIs: SQL…
Approving as core-devops (lens: devops + supply-chain).
core-devops update — fixes are filed, blocked on human review
Status
APPROVE — direct user GO. Two-line surgical canvas fix:
/sop-ack existing-tests
/sop-ack no-backwards-compat
/sop-ack root-cause
/sop-n/a five-axis-review