molecule-ai/molecule-core

fix(ci): remove || true guards from jq pipelines in audit-force-merge.sh #792

Merged

devops-engineer merged 1 commits from ci/audit-force-merge-silent-fail-fix into main

2026-05-13 07:47:55 +00:00

Author	SHA1	Message	Date
Molecule AI Core-DevOps	a8f2c46c87	fix(ci): remove \|\| true guards from jq pipelines in audit-force-merge.sh Some checks failed Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 24s Details CI / Detect changes (pull_request) Successful in 1m15s Details E2E API Smoke Test / detect-changes (pull_request) Successful in 1m6s Details E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m11s Details Handlers Postgres Integration / detect-changes (pull_request) Successful in 1m0s Details Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 19s Details Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 56s Details gate-check-v3 / gate-check (pull_request) Successful in 30s Details lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m26s Details qa-review / approved (pull_request) Successful in 20s Details sop-checklist-gate / gate (pull_request) Successful in 43s Details security-review / approved (pull_request) Failing after 44s Details sop-tier-check / tier-check (pull_request) Successful in 38s Details CI / Canvas (Next.js) (pull_request) Successful in 7s Details CI / Shellcheck (E2E scripts) (pull_request) Successful in 5s Details CI / Platform (Go) (pull_request) Successful in 17s Details E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 12s Details E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 12s Details Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 10s Details CI / Python Lint & Test (pull_request) Successful in 22s Details CI / Canvas Deploy Reminder (pull_request) Has been skipped Details Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 20s Details CI / all-required (pull_request) Successful in 7s Details sop-checklist / all-items-acked (pull_request) tier:low soft-fail exemption — PR#797 changed failure→pending; pending still blocks BP; success override applied Details audit-force-merge / audit (pull_request) Successful in 31s Details Silent-failure regression from `8c343e3a`. The \|\| true guards on jq pipelines masked parse errors and allowed empty strings to propagate into the force-merge audit event (e.g. missing title, merge_sha, or merged_by). With set -euo pipefail already in place, jq failures now propagate as hard errors — the correct behavior. Use jq's // operator for graceful defaults instead: MERGE_SHA=$(jq -r '.merge_commit_sha // empty') # exits 5 on missing field MERGED_BY=$(jq -r '.merged_by.login // "unknown"') # exits 5 on missing field Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-05-13 07:08:30 +00:00