Skip to content

Conversation

@bdotdub
Copy link

@bdotdub bdotdub commented Jan 5, 2026

Summary

The codex installation docs were slightly confusing when I went through them. We're not actually storing the PAT in the config. As well, storing the PAT as an env var is a requirement so installation shouldn't be collapsed, where it is easily missed.

What changed

  • Updated installation docs

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@bdotdub bdotdub requested a review from a team as a code owner January 5, 2026 04:31
Copilot AI review requested due to automatic review settings January 5, 2026 04:31
@bdotdub bdotdub changed the title Update codex installation docs for clarity [docs] Update codex installation docs for clarity Jan 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the clarity of the Codex installation documentation by removing a confusing comment that implied the PAT should be stored in the config file (when it's actually stored as an environment variable), and by making the PAT security instructions more prominent by converting them from a collapsible section to a standard heading.

Key changes:

  • Removed misleading comment about replacing PAT in config file
  • Promoted PAT security instructions from collapsible details to a visible section heading

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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