chore: standardize agent skills - #93
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fe05e36f72
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| echo "ELIXIR_DIR=${ELIXIR_DIR:-none} ASTRO_DIR=${ASTRO_DIR:-none}" | ||
| ``` | ||
|
|
||
| - **`ELIXIR_DIR` only** → Phoenix/Ash track (with `APP_DIR=$ELIXIR_DIR`). |
There was a problem hiding this comment.
Handle plain Mix projects before Phoenix setup
When this new worktrees skill is invoked in this repo, Step 1 sets ELIXIR_DIR=. just because the root mix.exs exists, but ports is a plain Elixir library and has no Phoenix config/dev.exs or config/test.exs; the next Phoenix/Ash step then greps those missing files and the setup path fails before producing a usable isolated checkout. Please distinguish Phoenix/Ash apps from plain Mix projects here, or add a non-Phoenix Elixir track for repos like this one.
Useful? React with 👍 / 👎.
Standardize agent skills: AGENTS.md canonical (+ CLAUDE.md symlink), full elixir skill template (baseline + worktrees + engineering suite), /.worktrees/ gitignored. The earlier PR was closed unmerged; this reopens the change against current main.