Skip to content

[codex] address docs review cleanup notes#39

Merged
charlesrhoward merged 1 commit into
mainfrom
codex/docs-review-cleanups
May 18, 2026
Merged

[codex] address docs review cleanup notes#39
charlesrhoward merged 1 commit into
mainfrom
codex/docs-review-cleanups

Conversation

@charlesrhoward
Copy link
Copy Markdown
Contributor

@charlesrhoward charlesrhoward commented May 18, 2026

Summary

Follow-up to the docs platform overhaul review notes.

  • document the optional AGENTS_HOME slash-command scope in both the custom-command guide and CLI environment table
  • replace the hardcoded run branch date in the API quickstart with a stable YYYY-MM-DD placeholder
  • explain why allowedDevOrigins includes 127.0.0.1 in the Next dev config

Validation

  • git diff --check
  • pnpm lint
  • pnpm types:check
  • pnpm build

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

@charlesrhoward charlesrhoward enabled auto-merge (squash) May 18, 2026 18:57
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mogplex-docs Ready Ready Preview, Comment May 18, 2026 6:58pm

Request Review

@mogplex
Copy link
Copy Markdown

mogplex Bot commented May 18, 2026

Mogplex PR Review

Status: No material issues found

Summary

Clean, tightly-scoped follow-up. All four changes are accurate, internally consistent, and match the PR description exactly. No issues found.

configuration-and-flags.mdx — The new AGENTS_HOME row is correctly placed in the Storage table (between MOGPLEX_HOME and the deprecated CODEX_HOME) and the description accurately matches the precedence semantics documented in the custom-slash-commands guide.

custom-slash-commands.mdx — The rewrite clearly communicates optionality and lookup order. Both docs now agree on the behaviour.

quickstart.mdx — Replacing the hardcoded 2026-05-17 with YYYY-MM-DD is the right call; it prevents the example from silently rotting and misleading readers.

next.config.mjsallowedDevOrigins is a Next.js dev-only knob (ignored in production builds), so this carries no production security risk. The comment is clear and sufficient.

Verdict

APPROVE — No issues. Ready to merge.

Affected files:

  • content/docs/cli/guides/configuration-and-flags.mdx
  • content/docs/configure-and-extend/custom-slash-commands.mdx
  • content/docs/web/api/quickstart.mdx
  • next.config.mjs

View check run

@charlesrhoward charlesrhoward merged commit 05df5f5 into main May 18, 2026
6 checks passed
@charlesrhoward charlesrhoward deleted the codex/docs-review-cleanups branch May 18, 2026 18:58
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