molecule-ai/molecule-core
35
0
Fork 2
Code Issues 16 Pull Requests 2 Actions 49 Packages Projects Releases Wiki Activity

fix(ci): port 3 verified-green CI fixes from staging to main (cherry-pick) #101

Closed
claude-ceo-assistant wants to merge 2 commits from fix/cherry-3-files-vitest-postgres-e2eapi into main
pull from: fix/cherry-3-files-vitest-postgres-e2eapi
merge into: molecule-ai:main
Conversation 1 Commits 2 Files Changed 6 +1321 -37
claude-ceo-assistant commented 2026-05-08 02:19:29 +00:00
Owner
Copy Link

Surgical cherry-pick of 3 verified-green CI fixes from staging tip to main, bypassing PR #99's broader staging-promote (which hit a Platform (Go) sqlmock regression in some other staging commit currently under Phase 1 investigation).

What this PR ports

  • canvas/vitest.config.ts — testTimeout 5000→30000ms on CI (PR #97 originally; unblocks Canvas vitest cold-start)
  • .github/workflows/handlers-postgres-integration.yml — parallel-safe pg containers (PR #98)
  • .github/workflows/e2e-api.yml — parallel-safe pg+redis (PR #100)
  • docs/runbooks/handlers-postgres-integration-port-collision.md — substrate runbook

Why this PR exists separately from #99

Workflow YAML + vitest config + runbook only. Zero Go code touched. Therefore Platform (Go) failure on PR #99 cannot apply here — clean cherry-pick of just the verified-safe surface.

Unblocks 4 Canvas-red PRs (#82/#81/#54/#53) once merged.

Follow-up: sister agent investigating PR #99's sqlmock regression separately. Once that lands, the residual staging→main promote can ship the rest.

Surgical cherry-pick of 3 verified-green CI fixes from staging tip to main, bypassing PR #99's broader staging-promote (which hit a Platform (Go) sqlmock regression in some other staging commit currently under Phase 1 investigation). ## What this PR ports - canvas/vitest.config.ts — testTimeout 5000→30000ms on CI (PR #97 originally; unblocks Canvas vitest cold-start) - .github/workflows/handlers-postgres-integration.yml — parallel-safe pg containers (PR #98) - .github/workflows/e2e-api.yml — parallel-safe pg+redis (PR #100) - docs/runbooks/handlers-postgres-integration-port-collision.md — substrate runbook ## Why this PR exists separately from #99 Workflow YAML + vitest config + runbook only. Zero Go code touched. Therefore Platform (Go) failure on PR #99 cannot apply here — clean cherry-pick of just the verified-safe surface. Unblocks 4 Canvas-red PRs (#82/#81/#54/#53) once merged. Follow-up: sister agent investigating PR #99's sqlmock regression separately. Once that lands, the residual staging→main promote can ship the rest.
claude-ceo-assistant added 1 commit 2026-05-08 02:19:30 +00:00
fix(ci): port 3 verified-green CI fixes from staging to main
Some checks failed
CodeQL / Analyze (${{ matrix.language }}) (go) (pull_request) Successful in 1s
Details
CodeQL / Analyze (${{ matrix.language }}) (javascript-typescript) (pull_request) Successful in 1s
Details
CodeQL / Analyze (${{ matrix.language }}) (python) (pull_request) Successful in 1s
Details
Retarget main PRs to staging / Retarget to staging (pull_request) Has been skipped
Details
Check merge_group trigger on required workflows / Required workflows have merge_group trigger (pull_request) Successful in 5s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 6s
Details
branch-protection drift check / Branch protection drift (pull_request) Successful in 8s
Details
CI / Detect changes (pull_request) Successful in 7s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 8s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 7s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 8s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 8s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 7s
Details
Harness Replays / detect-changes (pull_request) Successful in 9s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 9s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 3s
Details
CI / Platform (Go) (pull_request) Successful in 4s
Details
CI / Python Lint & Test (pull_request) Successful in 4s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 3s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 57s
Details
Harness Replays / Harness Replays (pull_request) Failing after 57s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 1m20s
Details
CI / Canvas (Next.js) (pull_request) Successful in 1m56s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 4m12s
Details
570f456436 
Files copied from staging tip (a4ab623b):
- canvas/vitest.config.ts          (vitest testTimeout 5000→30000ms on CI; was PR #97 unblocking 4 canvas-test PRs)
- .github/workflows/handlers-postgres-integration.yml  (parallel-safe pg containers; was PR #98 unblocking #84)
- .github/workflows/e2e-api.yml    (parallel-safe pg+redis; was PR #100 unblocking #84 + #99)
- docs/runbooks/handlers-postgres-integration-port-collision.md (new — substrate runbook)

Why a separate cherry-pick PR rather than promote-staging-to-main: PR #99 (full
staging→main promote) hit a Platform (Go) sqlmock regression in some other
staging commit (under Phase 1 investigation by sister agent a283d938). To
unblock prod with the verified-green CI fixes WITHOUT carrying the Go-test
regression, port these 3 workflow/config files surgically.

Verified clean: workflow-YAML + vitest config + runbook only — zero Go code
touched, so the Platform (Go) failure on PR #99 cannot apply here.

Co-authored-by: Claude (orchestrator)
Ghost added 1 commit 2026-05-08 02:26:54 +00:00
fix(eic-tunnel-pool): port race fix b6646910 from staging
Some checks failed
CodeQL / Analyze (${{ matrix.language }}) (javascript-typescript) (pull_request) Successful in 2s
Details
CodeQL / Analyze (${{ matrix.language }}) (go) (pull_request) Successful in 0s
Details
CodeQL / Analyze (${{ matrix.language }}) (python) (pull_request) Successful in 3s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 7s
Details
Check merge_group trigger on required workflows / Required workflows have merge_group trigger (pull_request) Successful in 7s
Details
pr-guards / disable-auto-merge-on-push (pull_request) Failing after 6s
Details
CI / Detect changes (pull_request) Successful in 10s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 10s
Details
branch-protection drift check / Branch protection drift (pull_request) Successful in 12s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 12s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 13s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 12s
Details
Harness Replays / detect-changes (pull_request) Successful in 13s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 14s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 14s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 4s
Details
CI / Python Lint & Test (pull_request) Successful in 6s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Failing after 46s
Details
Harness Replays / Harness Replays (pull_request) Failing after 56s
Details
CI / Platform (Go) (pull_request) Failing after 1m44s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Failing after 1m51s
Details
CI / Canvas (Next.js) (pull_request) Successful in 2m47s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 4m52s
Details
47c0a8c903 
Sister-agent investigation of PR #99's Platform (Go) regression identified
that PR #99's head pre-dates b6646910 (fix(eic-tunnel-pool): capture
poolJanitorInterval at pool construction).

Without this commit, TestPooledWithEICTunnel_PanicPoisonsEntry fails under
-race on main when the cherry-pick lands. Including it in this cherry-pick
to keep main Platform (Go) green.

Verified locally by sister agent: 3-of-3 reproducible race FAIL without fix,
3-of-3 PASS with fix. 2-of-2 full handlers go test -race PASS with this
file from staging tip.

Co-authored-by: Claude (orchestrator)
claude-ceo-assistant commented 2026-05-08 02:29:08 +00:00
Author
Owner
Copy Link

Closing — partial cherry-pick approach defeated. The 4 CI files + eic_tunnel_pool reference more staging-only code than initially scoped (e2e-api workflow expected EIC consumer code from PR #84 to be present). Switching to full staging→main promote via re-sync of PR #99, which now includes the race fix from sister-agent investigation.

Closing — partial cherry-pick approach defeated. The 4 CI files + eic_tunnel_pool reference more staging-only code than initially scoped (e2e-api workflow expected EIC consumer code from PR #84 to be present). Switching to full staging→main promote via re-sync of PR #99, which now includes the race fix from sister-agent investigation.
Some checks are pending
CodeQL / Analyze (${{ matrix.language }}) (javascript-typescript) (pull_request) Successful in 2s
Details
CodeQL / Analyze (${{ matrix.language }}) (go) (pull_request) Successful in 0s
Details
CodeQL / Analyze (${{ matrix.language }}) (python) (pull_request) Successful in 3s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 7s
Details
Check merge_group trigger on required workflows / Required workflows have merge_group trigger (pull_request) Successful in 7s
Details
pr-guards / disable-auto-merge-on-push (pull_request) Failing after 6s
Details
CI / Detect changes (pull_request) Successful in 10s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 10s
Details
branch-protection drift check / Branch protection drift (pull_request) Successful in 12s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 12s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 13s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 12s
Details
Harness Replays / detect-changes (pull_request) Successful in 13s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 14s
Required
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 14s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 4s
Details
CI / Python Lint & Test (pull_request) Successful in 6s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Failing after 46s
Details
Harness Replays / Harness Replays (pull_request) Failing after 56s
Details
CI / Platform (Go) (pull_request) Failing after 1m44s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Failing after 1m51s
Details
CI / Canvas (Next.js) (pull_request) Successful in 2m47s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 4m52s
Details
sop-tier-check / tier-check (pull_request)
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
tier:high

High risk per dev-sop §SOP-6 — ceo only, 24h cooldown

  
tier:low

Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve

  
tier:medium

Medium risk per dev-sop §SOP-6 — managers/ceo can approve

No Label
tier:high
tier:low
tier:medium
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#101
No description provided.
Delete Branch "fix/cherry-3-files-vitest-postgres-e2eapi"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?