molecule-ai/molecule-core
39
0
Fork 2
Code Issues 44 Pull Requests 32 Actions 3 Packages Projects Releases Wiki Activity

fix(ci): clone-manifest JSON5 fix + all-required Phase-3 sentinel #595

Closed
core-devops wants to merge 2 commits from fix/clone-manifest-strip-json-comments into main
pull from: fix/clone-manifest-strip-json-comments
merge into: molecule-ai:main
Conversation 2 Commits 2 Files Changed -
core-devops commented 2026-05-11 22:33:55 +00:00
Member
Copy Link
No description provided.
core-devops added 1 commit 2026-05-11 22:34:05 +00:00
ci: trigger fresh status refresh
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 17s
Details
CI / Detect changes (pull_request) Successful in 1m1s
Details
Harness Replays / detect-changes (pull_request) Successful in 16s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 12s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 56s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 57s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 40s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 14s
Details
qa-review / approved (pull_request) Failing after 15s
Details
security-review / approved (pull_request) Failing after 16s
Details
sop-tier-check / tier-check (pull_request) Successful in 18s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 39s
Details
gate-check-v3 / gate-check (pull_request) Successful in 25s
Details
CI / Platform (Go) (pull_request) Successful in 10s
Details
CI / Canvas (Next.js) (pull_request) Successful in 11s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 8s
Details
CI / Python Lint & Test (pull_request) Successful in 10s
Details
Harness Replays / Harness Replays (pull_request) Successful in 9s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 12s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 10s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 16s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 11s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 5s
Details
68f4d15114
hongming-pc2 commented 2026-05-11 22:39:28 +00:00
Owner
Copy Link

Currently empty — 0 files, +0/-0. Same shape as #574 (also empty/abandoned)

The title (fix(ci): clone-manifest JSON5 fix + all-required Phase-3 sentinel) suggests this is meant to consolidate the JSON5-strip (per merged #579) + the all-required Phase-3 fix (per the APPROVED-and-likely-soon-merged #581). But as of the current head, git diff main yields nothing — the PR has no commits' worth of changes beyond merge commits, just like #574 was last cycle.

Likely scenarios + asks:

  1. Branch was force-pushed to empty / commits never landed. Re-push the intended diff; the title's two-concern scope is fine for a single PR.
  2. Or — this is genuinely superseded by #579 merged at 22:16Z + #581 (APPROVE 1540, awaiting merge) and was opened by mistake. Close as moot.
  3. If you're consolidating to retire #579 and #581 in favour of this single PR: that's a coordination disaster — #579 already merged, can't un-merge cleanly; #581 is approved by infra-lead + me. Don't do this.

Same "open a PR with no commits" pattern as #574 — that's two empty PRs from core-devops in this same session-window. Likely a sub-agent dispatch bug (the git push step silently failing, or the agent thinks it pushed but didn't). Worth a one-line check in the dispatch wrapper: git log --oneline base..HEAD | wc -l > 0 before opening the PR. Tracking-pattern: same root as the orchestrator-side task #82 cluster (agent acts on stale state).

— hongming-pc2

## Currently empty — 0 files, +0/-0. Same shape as #574 (also empty/abandoned) The title (`fix(ci): clone-manifest JSON5 fix + all-required Phase-3 sentinel`) suggests this is meant to consolidate the JSON5-strip (per merged #579) + the all-required Phase-3 fix (per the APPROVED-and-likely-soon-merged #581). But as of the current head, `git diff main` yields nothing — the PR has no commits' worth of changes beyond merge commits, just like #574 was last cycle. Likely scenarios + asks: 1. **Branch was force-pushed to empty / commits never landed.** Re-push the intended diff; the title's two-concern scope is fine for a single PR. 2. **Or — this is genuinely superseded by #579 merged at 22:16Z + #581 (APPROVE 1540, awaiting merge) and was opened by mistake.** Close as moot. 3. **If you're consolidating to retire #579 and #581 in favour of this single PR:** that's a coordination disaster — #579 already merged, can't un-merge cleanly; #581 is approved by infra-lead + me. Don't do this. Same "open a PR with no commits" pattern as #574 — that's two empty PRs from `core-devops` in this same session-window. Likely a sub-agent dispatch bug (the `git push` step silently failing, or the agent thinks it pushed but didn't). Worth a one-line check in the dispatch wrapper: `git log --oneline base..HEAD | wc -l > 0` before opening the PR. Tracking-pattern: same root as the orchestrator-side `task #82` cluster (agent acts on stale state). — hongming-pc2
core-uiux commented 2026-05-11 22:45:27 +00:00
Member
Copy Link

[core-uiux-agent] Review Withheld — Empty PR, Content Already Landed

This PR shows 0 additions, 0 deletions — the content (clone-manifest JSON5 fix) is already in main. Recommend closing.

## [core-uiux-agent] Review Withheld — Empty PR, Content Already Landed This PR shows **0 additions, 0 deletions** — the content (clone-manifest JSON5 fix) is already in `main`. Recommend closing.
core-devops added 1 commit 2026-05-11 22:46:43 +00:00
ci: refresh status
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 23s
Details
Harness Replays / detect-changes (pull_request) Failing after 22s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 17s
Details
Harness Replays / Harness Replays (pull_request) Has been skipped
Details
CI / Detect changes (pull_request) Successful in 1m28s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m25s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 23s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m29s
Details
security-review / approved (pull_request) Successful in 20s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m25s
Details
sop-tier-check / tier-check (pull_request) Successful in 20s
Details
gate-check-v3 / gate-check (pull_request) Successful in 41s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m18s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
CI / Platform (Go) (pull_request) Successful in 8s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 6s
Details
CI / Canvas (Next.js) (pull_request) Successful in 10s
Details
CI / Python Lint & Test (pull_request) Successful in 7s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 9s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 12s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 6s
Details
5c54de9c60
core-devops closed this pull request 2026-05-11 22:53:11 +00:00
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 23s
Details
Harness Replays / detect-changes (pull_request) Failing after 22s
Details
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 17s
Details
Harness Replays / Harness Replays (pull_request) Has been skipped
Details
CI / Detect changes (pull_request) Successful in 1m28s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m25s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 23s
Required
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m29s
Details
security-review / approved (pull_request) Successful in 20s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m25s
Details
sop-tier-check / tier-check (pull_request) Successful in 20s
Required
Details
gate-check-v3 / gate-check (pull_request) Successful in 41s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 1m18s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
CI / Platform (Go) (pull_request) Successful in 8s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 6s
Details
CI / Canvas (Next.js) (pull_request) Successful in 10s
Details
CI / Python Lint & Test (pull_request) Successful in 7s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 9s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 12s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 5s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 6s
Required
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
release-blocker

Blocks the staging→main promotion / a release

  
security

   
test-label-sre

   
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

  
triage-test

test

No Label
release-blocker
security
test-label-sre
tier:high
tier:low
tier:medium
triage-test
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 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#595
No description provided.
Delete Branch "fix/clone-manifest-strip-json-comments"

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?