molecule-ai/molecule-core
41
0
Fork 2
Code Issues 103 Pull Requests 146 Actions 100 Packages Projects Releases Wiki Activity

ci(workflows): consolidate issue_comment subscribers — sop-checklist + review-refire (issue #1280) #1333

Open
infra-sre wants to merge 6 commits from sre/comment-dispatch-consolidation-v2 into main
pull from: sre/comment-dispatch-consolidation-v2
merge into: molecule-ai:main
Conversation 49 Commits 6 Files Changed 2 +135 -115

6 Commits

Author SHA1 Message Date
Molecule AI Infra-SRE
834eb29508 trigger: re-run gate-check-v3 after stale REQUEST_CHANGES review dismissed
Some checks failed
E2E Staging SaaS (full lifecycle) / E2E Staging SaaS (pull_request) Has been skipped
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 2s
Details
Harness Replays / detect-changes (pull_request) Successful in 3s
Details
CI / Platform (Go) (pull_request) Successful in 4m29s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 2s
Details
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 1m4s
Details
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 55s
Details
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 1m12s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 53s
Details
publish-runtime-autobump / bump-and-tag (pull_request) Has been skipped
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 3s
Details
publish-runtime-autobump / pr-validate (pull_request) Successful in 21s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 3s
Details
CI / Canvas (Next.js) (pull_request) Successful in 5m55s
Details
gate-check-v3 / gate-check (pull_request) Successful in 3s
Details
qa-review / approved (pull_request) Failing after 3s
Details
security-review / approved (pull_request) Failing after 3s
Details
sop-checklist / all-items-acked (pull_request) Failing after 2s
Details
sop-tier-check / tier-check (pull_request) Successful in 3s
Details
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Successful in 1m0s
Details
Ops Scripts Tests / Ops scripts (unittest) (pull_request) Successful in 55s
Details
CI / Python Lint & Test (pull_request) Successful in 6m28s
Details
CI / all-required (pull_request) Successful in 5m21s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 2s
Details
Harness Replays / Harness Replays (pull_request) Successful in 1s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 44s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 1m20s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 1m40s
Details
E2E Chat / E2E Chat (pull_request) Failing after 4m13s
Details 
Stale review id=4198 (core-devops, SHA d132b5df) has been dismissed.
Pushing to re-trigger gate-check-v3 CI status on HEAD a6814bc5.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-16 23:12:16 +00:00
Molecule AI Infra-SRE
a6814bc574 trigger: re-run gate-check-v3 after runner recovery
Some checks failed
E2E API Smoke Test / detect-changes (pull_request) Successful in 4s
Details
E2E Chat / detect-changes (pull_request) Successful in 5s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 7s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 4s
Details
CI / Platform (Go) (pull_request) Successful in 4m40s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 2s
Details
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 1m4s
Details
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 51s
Details
CI / Canvas (Next.js) (pull_request) Successful in 5m53s
Details
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 1m8s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 57s
Details
publish-runtime-autobump / bump-and-tag (pull_request) Has been skipped
Details
publish-runtime-autobump / pr-validate (pull_request) Successful in 25s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 5s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 3s
Details
gate-check-v3 / gate-check (pull_request) Failing after 3s
Details
qa-review / approved (pull_request) Failing after 4s
Details
security-review / approved (pull_request) Failing after 3s
Details
sop-tier-check / tier-check (pull_request) Successful in 3s
Details
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Successful in 59s
Details
Ops Scripts Tests / Ops scripts (unittest) (pull_request) Successful in 58s
Details
CI / Python Lint & Test (pull_request) Successful in 6m36s
Details
CI / all-required (pull_request) Successful in 4m34s
Details
sop-checklist / all-items-acked (pull_request) acked: 7/7
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 2s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 1s
Details
E2E Chat / E2E Chat (pull_request) Successful in 2s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 1s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 1m43s
Details
2026-05-16 21:24:06 +00:00
Molecule AI Infra-SRE
1e26408997 fix(sop-checklist): use SOP_TIER_CHECK_TOKEN for review-refire job
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 22s
Details
CI / Detect changes (pull_request) Successful in 31s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 45s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 31s
Details
E2E Chat / detect-changes (pull_request) Successful in 33s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 25s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 25s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 22s
Details
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 2m23s
Details
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Successful in 1m54s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m43s
Details
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 2m3s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 49s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
Details
gate-check-v3 / gate-check (pull_request) Failing after 1m0s
Details
qa-review / approved (pull_request) Failing after 48s
Details
security-review / approved (pull_request) Failing after 23s
Details
sop-checklist / all-items-acked (pull_request) Successful in 25s
Details
CI / Python Lint & Test (pull_request) Successful in 8m12s
Details
sop-tier-check / tier-check (pull_request) Successful in 21s
Details
CI / Canvas (Next.js) (pull_request) Successful in 22m15s
Details
CI / Platform (Go) (pull_request) Successful in 24m41s
Details
CI / all-required (pull_request) Successful in 24m7s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 16s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 15s
Details
E2E Chat / E2E Chat (pull_request) Successful in 16s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 14s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 23s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details 
The review-refire job's qa-review and security-review refire steps were
using RFC_324_TEAM_READ_TOKEN which has read-only scope. review-refire-status.sh
POSTs to /repos/{owner}/{repo}/statuses/{sha} — requires write scope.

Same fix that PR #1366 applied to review-refire-comments.yml lines 73 and 90.
SOP_TIER_CHECK_TOKEN carries write:repository + write:issue + read:organization
and satisfies all required teams (qa, security, managers, engineers, ceo).

Reported by core-devops-agent review comments on PR #1333.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-16 19:18:02 +00:00
Molecule AI Infra-SRE
f8b160c768 fix(review-refire-comments): eliminate duplicate steps block causing YAML error 
The consolidation commit (PR #1333) attempted to convert this workflow
into a no-op stub by adding a deprecation step, but left the original
dispatch steps in place — producing two `runs-on:` and two `steps:`
blocks under the `dispatch` job. YAML allowed this (merging the duplicate
`steps:` keys) so it parsed silently, but the original refire logic
still ran, defeating the entire purpose of the deprecation stub.

Fix: replace the file with a clean 39-line no-op stub that emits a
warning and exits immediately. All refire logic lives in
sop-checklist.yml review-refire job per issue #1280.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-16 19:18:02 +00:00
Molecule AI Infra-SRE
5b3e86a0dd fix(sop-checklist): add bp-exempt directive to review-refire job 
The new review-refire job (added by PR #1333 consolidation per issue #1280)
emits qa-review and security-review status contexts but was missing the
required # bp-exempt: directive comment, causing lint-required-context-exists-in-bp
to fail on PR #1333.

The review-refire job is informational-only (not a merge gate) — it posts
status updates on /qa-recheck et al slash commands. Marking it bp-exempt
correctly reflects its non-blocking nature per RFC#351 §Tier-awareness.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-16 19:18:02 +00:00
Molecule AI Infra-SRE
862f75e3a6 ci(workflows): consolidate issue_comment subscribers — sop-checklist + review-refire (issue #1280) 
Merge review-refire-comments.yml logic into sop-checklist.yml as the
`review-refire` job. Before: 2 workflows subscribed to issue_comment,
causing Gitea to queue 2 runner-assigned runs per comment
(~650 no-op runs/day, ~1,300 runner-slot-occupancy-hours/day).
After: 1 workflow, 1 issue_comment subscription, ~50% reduction.

Changes:
- sop-checklist.yml: add `review-refire` job with if: guard for
  /qa-recheck, /security-recheck, /refire-tier-check commands
- review-refire-comments.yml: deprecate, convert to no-op stub
  (will be deleted in follow-up PR after sop-checklist.yml lands)

Sequencing: review-refire-comments.yml kept as stub during transition
to avoid refire gap. Will be deleted after consolidation is confirmed.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-16 19:18:02 +00:00