Unify agent config via lnai and auto-generate on setup#2750
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1dea0dbb33
ℹ️ 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".
| node-tools: | ||
| build: | ||
| context: . | ||
| dockerfile: node.dockerfile | ||
| volumes: |
There was a problem hiding this comment.
Add SELinux label disable for node-tools mount
On SELinux-enforcing hosts, bind-mounted project directories are not writable/readable from containers unless they are relabeled or the container runs with label:disable. This file already applies security_opt: [label:disable] to other bind-mounted services (php, node-docs), but node-tools omits it, so make setup/make ai-sync can fail with permission errors when lnai reads or writes files under /workdir.
Useful? React with 👍 / 👎.
Summary
.ai/AGENTS.mdand manage Claude/Codex via lnai.ai/config.jsonand.ai/settings.jsonfor shared tool configuration and permissionsmake setupvia newai-synctargetnode-docs(docs workspace) andnode-tools(repo-root tools)Details
.claude/CLAUDE.mdinto.ai/AGENTS.md.ai/settings.jsondocker compose exec ...usage and usenode-toolsfor lnai syncnode-docswith existing port mappingVerification
make ai-synccompletes successfully with Docker running and generates/updates.claude/settings.jsonandAGENTS.mdsymlinks