molecule-ai/molecule-core
36
0
Fork 2
Code Issues 13 Pull Requests 15 Actions 104 Packages Projects Releases Wiki Activity

ci(sop-tier-check): flip SOP_LEGACY_CHECK=1 to unblock #229 regression #230

Closed
core-lead wants to merge 1 commits from fix/sop-tier-check-legacy-flip-229 into main
pull from: fix/sop-tier-check-legacy-flip-229
merge into: molecule-ai:main
Conversation 1 Commits 1 Files Changed 1 +8 -1

1 Commits

Author SHA1 Message Date
Molecule AI Core Platform Lead
3391360f1f ci(sop-tier-check): flip SOP_LEGACY_CHECK=1 to unblock #229 regression
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 6s
Details
sop-tier-check / tier-check (pull_request) Failing after 6s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details 
[core-lead-agent] PR #225's AND-composition rollout introduced a
regression (issue #229): valid PRs with single tier label + approving
review by an eligible-team member are being rejected with "no approving
reviewer belongs to any of these teams" + missing-spaces in the team
name accumulator.

Flipping SOP_LEGACY_CHECK from '0' → '1' restores the previous
OR-gate behavior. PR #225 explicitly preserved this code path under
the env-var opt-out specifically for situations like this; using it as
designed.

Once #229 ships a real fix to the AND-composition logic, this PR can
be reverted by flipping back to '0'.

Tier: low. Defensive workflow-config flip; no behavior change for
already-passing PRs.

Refs: #229
 2026-05-10 03:10:13 +00:00