Skip to content

Add shared Codex and Claude agent setup#136

Open
ankitvgupta wants to merge 6 commits into
mainfrom
ankitvgupta/codex-setup-docs
Open

Add shared Codex and Claude agent setup#136
ankitvgupta wants to merge 6 commits into
mainfrom
ankitvgupta/codex-setup-docs

Conversation

@ankitvgupta
Copy link
Copy Markdown
Owner

@ankitvgupta ankitvgupta commented May 22, 2026

Summary

  • Add AGENTS.md as the shared repo guidance for Codex and other agents
  • Convert CLAUDE.md into a thin Claude-specific adapter
  • Move reusable agent skills into shared skills/ with Claude and Codex symlink entrypoints
  • Remove the unused review-code skill

Verification

  • npm run build
  • node scripts/agentic-verify.mjs --mode=verify-diff
  • EXO_REAL_GMAIL_TEST=true npx playwright test --project=real-gmail
  • npm run pre-pr will be rerun after PR creation to inject the full report

Pre-PR verdict: PASS

  • mode: full
  • sha: b2e97fc
  • generated: 2026-05-22T01:51:27.630Z
Phase Status Duration
eval:analyzer ✅ exit 0 17.8s
eval:features ✅ exit 0 31.2s
agentic-verify ✅ exit 0 74.6s
real-gmail:cached ✅ exit 0 12.4s

@ankitvgupta ankitvgupta marked this pull request as ready for review May 22, 2026 00:58
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 22, 2026

Greptile Summary

This PR restructures the agent guidance and skill definitions for the repository by creating a shared AGENTS.md as the single source of truth for all coding agents and converting CLAUDE.md into a thin Claude-specific adapter. Skills previously stored only in .claude/skills/ are moved to a shared skills/ directory with symlinks from both .claude/skills/ and .codex/skills/, making them available to both Claude and Codex without duplication.

  • AGENTS.md introduced: All repo guidance (coding standards, PR workflow, git commands, architecture docs) moved from CLAUDE.md to AGENTS.md, with agent-specific language generalized.
  • Skills relocated and de-duplicated: Six shared skills moved under skills/ with the legacy review-code skill removed; the reviewloop skill gains a Codex compatibility note and push-authorization clarification.
  • clear-caches safety improvement: Default database path changed from the production ~/Library/Application Support/exo/ to the dev .dev-data/data/exo.db.

Confidence Score: 5/5

Safe to merge — all changes are documentation and tooling configuration with no production code touched.

The PR is a pure reorganization of agent guidance and skill definitions. No application logic, database schemas, IPC handlers, or React components were modified. The symlink targets resolve correctly from both .claude/skills/ and .codex/skills/. The one safety-relevant change — defaulting clear-caches to .dev-data/ instead of the production path — is a strict improvement.

No files require special attention.

Important Files Changed

Filename Overview
AGENTS.md New shared agent guidance file; content mirrors old CLAUDE.md with multi-agent language improvements and correct ### Python heading hierarchy.
CLAUDE.md Stripped to a thin adapter that delegates to AGENTS.md and adds four Claude Code-specific notes (skills path, MCP config, permissions, run_in_background).
skills/reviewloop/SKILL.md Added Codex compatibility note for allowed-tools and push-authorization clarification; otherwise unchanged from prior .claude/ version.
skills/clear-caches/SKILL.md Default DB path changed from production directory to .dev-data/data/exo.db with an explicit warning not to touch the production path.
skills/electron-devtools-testing/SKILL.md Claude Code-specific MCP tool names replaced with generic action descriptions and both Claude/Codex MCP install commands added.
skills/github-cli/SKILL.md Section renamed from Claude Code on Web to Hosted Agent Environments and auth token description softened to accommodate Codex environments.

Reviews (6): Last reviewed commit: "Merge latest origin/main into codex setu..." | Re-trigger Greptile

Comment thread AGENTS.md Outdated
@ankitvgupta
Copy link
Copy Markdown
Owner Author

Reviewloop complete on 174820e.

Fixed all Greptile feedback:

  • AGENTS.md: changed the Python heading from H1 to ###.
  • skills/reviewloop/SKILL.md: documented Codex handling for Claude-only allowed-tools metadata.
  • skills/reviewloop/SKILL.md: clarified that explicit reviewloop invocation authorizes loop pushes; ambiguous invocation still asks once.
  • skills/gh-local/SKILL.md: replaced stale tail -50 polling guidance with native long-running session output guidance.

No review issues intentionally left open. Full npm run pre-pr passed for 174820e; Greptile is 5/5 with no files requiring special attention.

@ankitvgupta
Copy link
Copy Markdown
Owner Author

Final reviewloop status on b2e97fc after merging latest origin/main into the branch:

  • Full npm run pre-pr passed and updated the PR body for b2e97fc.
  • Greptile passed at 5/5 with no files requiring special attention.
  • CI is green: lint/format, typecheck, security audit, tests, migration replay, packaged smoke, visual regression, and pre-pr marker verification all passed.
  • GitHub reports the PR as mergeable/clean.

No review issues are left open.

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