Description
docs/codex.md documents enabling Codex auto-review via Helm with:
--set-json 'agents.codex.extraConfig={"sandbox":{"approval_policy":"auto-review"}}'
However, the published Helm charts I tested do not render agents.codex.extraConfig into the Kubernetes manifest or into /home/node/.codex/config.toml.
Tested chart versions:
- openab-0.8.2
- openab-0.8.3-beta.4
In both versions, helm show values openab/openab --version does not list extraConfig, and rendering the chart with agents.codex.extraConfig.sandboxapproval_policy=auto-review does not produce
approval_policy or auto-review in the manifest.
Current working deployments therefore need to use agents.codex.extraInitContainers to mutate /home/node/.codex/config.toml, for example by writing:
[sandbox]
approval_policy = "auto-review"
This makes the docs/codex.md Helm example misleading: it suggests a declarative Helm values path that does not appear to be implemented in the released chart.
Suggested Change
Either:
- Implement agents..extraConfig in the Helm chart so the documented example works for Codex, or
- Update docs/codex.md to document the currently supported Helm approach using agents.codex.extraInitContainers.
If extraConfig is planned but not released yet, the docs should mention the minimum chart version that supports it.
Description
docs/codex.mddocuments enabling Codex auto-review via Helm with:--set-json 'agents.codex.extraConfig={"sandbox":{"approval_policy":"auto-review"}}'However, the published Helm charts I tested do not render agents.codex.extraConfig into the Kubernetes manifest or into /home/node/.codex/config.toml.
Tested chart versions:
In both versions, helm show values openab/openab --version does not list extraConfig, and rendering the chart with agents.codex.extraConfig.sandboxapproval_policy=auto-review does not produce
approval_policy or auto-review in the manifest.
Current working deployments therefore need to use agents.codex.extraInitContainers to mutate /home/node/.codex/config.toml, for example by writing:
This makes the docs/codex.md Helm example misleading: it suggests a declarative Helm values path that does not appear to be implemented in the released chart.
Suggested Change
Either:
If extraConfig is planned but not released yet, the docs should mention the minimum chart version that supports it.