Fix remaining prepublish release blockers#5697
Merged
code-yeongyu merged 12 commits intoJun 28, 2026
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes the remaining pre-publish blockers found in the release gate. It prevents CodeGraph child processes from inheriting ambient provider/API tokens, keeps Atlas from treating active full-session background output as completed work, tightens Codex CodeGraph release coverage, fixes web-terminal Authorization redaction, and gives the download-stats workflow the GitHub token it needs.
Changes
CodeGraph process safety
buildCodegraphChildEnvhelper that only preserves OS/runtime essentials and explicit CodeGraph variables..js/.cjs/.mjslauncher handling so Node script entrypoints are executed through the active Node binary.Atlas background-output handling
# Full Session Outputwith active statuses (pending,running,error,cancelled,interrupt) as incomplete, so Atlas leaves the output untouched.Release/CI gates
test:codexnow runs the CodeGraph component typecheck and runtime tests before packaged Codex checks.bun test ./testand platform-neutral assertions so Windows no longer treats test globs or escaped paths as failures.QA and workflow hardening
GH_TOKENfor both fetch and send steps while keepingPOSTHOG_KEYscoped to send only.QA & Evidence
Targeted branch tests
bun test packages/utils/src/codegraph-env.test.ts packages/omo-opencode/src/hooks/codegraph-bootstrap/codegraph-bootstrap.test.ts packages/omo-opencode/src/hooks/atlas/tool-execute-after-subagent-completion.test.ts script/codex-test-script.test.ts script/publish-workflow.test.ts script/stats.test.ts script/web-terminal-visual-qa.test.ts66 pass,0 fail.omo/evidence/20260628-prepublish-blockers-round2/targeted-bun-tests.outCodeGraph component typecheck and runtime tests
npm --prefix packages/omo-codex/plugin/components/codegraph run typecheck,npm --prefix packages/omo-codex/plugin/components/codegraph test0; runtime tests46 pass,0 fail.omo/evidence/20260628-prepublish-blockers-round2/codegraph-component-typecheck-after-review-fix.log,.omo/evidence/20260628-prepublish-blockers-round2/codegraph-component-test-after-typecheck-fix.logGenerated skill drift
node --test packages/omo-codex/plugin/test/sync-skills.test.mjs packages/omo-codex/plugin/test/sync-skills-orchestration.test.mjs23 pass,0 fail; generated skill copies have no hand-authored drift.omo/evidence/20260628-prepublish-blockers-round2/sync-skills-test.outCodex compatibility gate
bun run test:codex421 pass,0 fail.omo/evidence/20260628-prepublish-blockers-round2/test-codex-after-codegraph-typecheck-gate-fix.logCodex live harness QA
bash .agents/skills/codex-qa/scripts/lib/common.sh --self-check,bash .agents/skills/codex-qa/scripts/app-server-drive.sh --pluginCODEX_HOME, local mock model, turn completed,hook/completedforsessionStart,userPromptSubmit, andstop, real~/.codex/config.tomlunchanged.omo/evidence/20260628-prepublish-blockers-round2/codex-qa-common-self-check-after-typecheck-fix.log,.omo/evidence/20260628-prepublish-blockers-round2/codex-app-server-plugin-after-typecheck-fix.jsonOpenCode live harness QA
bash /Users/yeongyu/local-workspaces/omo/.agents/skills/opencode-qa/scripts/sse-hook-probe.sh --self-testserver.connected; live OpenCode DB session count stayed5737 -> 5737.omo/evidence/20260628-prepublish-blockers-round2/opencode-sse-self-test.outPre-push gates
bun run typecheck,bun run build,bun testtypecheckpassed,buildpassed, fullbun testreported10283 pass,2 skip,0 fail.omo/evidence/20260628-prepublish-blockers-round2/typecheck.out,.omo/evidence/20260628-prepublish-blockers-round2/build.out,.omo/evidence/20260628-prepublish-blockers-round2/bun-test.outFull evidence summary:
.omo/evidence/20260628-prepublish-blockers-round2/qa-summary.mdManual QA matrix:
.omo/evidence/20260628-prepublish-blockers-round2/manual-qa-matrix.mdRound notepad:
.omo/notepads/pr-5697-prepublish-blockers-round2.mdRisks & Residuals
.omo/evidenceartifacts and are not committed to the PR branch; they will be synced back to the main worktree after merge per the repo workflow.