Skip to content

chore: prune legacy config switches#998

Merged
jackwener merged 1 commit into
mainfrom
codex-mini1/config-prune
Apr 13, 2026
Merged

chore: prune legacy config switches#998
jackwener merged 1 commit into
mainfrom
codex-mini1/config-prune

Conversation

@jackwener

Copy link
Copy Markdown
Owner

Summary

  • remove OUTPUT env override and DEBUG=opencli fallback from runtime behavior
  • stop exposing dead or internal-only config names in public docs and workflow examples
  • drop stale OPENCLI_BROWSER_EXECUTABLE_PATH injections from CI workflows

Verification

  • npm run typecheck
  • npx vitest run src/output.test.ts --reporter=verbose
  • manual logger checks for OPENCLI_VERBOSE vs DEBUG fallback removal

@jackwener jackwener left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Reviewed the diff — all five ghost/obsolete configs are cleanly removed from runtime code, CI workflows, docs, and tests. Verified no remaining references to OPENCLI_BROWSER_EXECUTABLE_PATH, OPENCLI_DAEMON_TIMEOUT, or OPENCLI_HEADLESS in the codebase. Ready to merge.

@jackwener jackwener merged commit 9ebb921 into main Apr 13, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant