codex: address PR review feedback (#29497)

This commit is contained in:
Adam Perry
2026-06-22 23:28:00 +00:00
parent 2437634691
commit 4bf950af4f
2 changed files with 6 additions and 5 deletions
+4 -3
View File
@@ -1,6 +1,9 @@
# Shared rules only allow commands that are generally safe for every project
# developer. Prompt and denial policies belong in user config so the repository
# does not impose those flows globally.
# Cargo commands and argument-forwarding recipes may use `--manifest-path` to
# target manifests outside this checkout. That flexibility is an accepted
# tradeoff for development.
prefix_rule(
pattern = [
@@ -25,12 +28,10 @@ prefix_rule(
decision = "allow",
justification = """
Needed: host caches or nested sandbox capabilities.
Safe: fixed non-publishing recipes only operate on the trusted checkout.
Safe: these non-publishing recipes are standard development operations.
""",
)
# Cargo accepts `--manifest-path`, so these rules may target manifests outside
# this checkout. That flexibility is an accepted tradeoff for development.
prefix_rule(
pattern = ["cargo", ["build", "check", "clippy", "test"]],
decision = "allow",
+2 -2
View File
@@ -1,6 +1,6 @@
# Pin Codex development tool paths
Pinning prevents trusted rules from matching an unexpected executable with the same name.
Pins constrain absolute-path fallback to binaries trusted by each developer.
From the development shell's repository root:
@@ -10,6 +10,6 @@ python3 scripts/setup_dev_execpolicy.py \
> ~/.codex/rules/codex_dev_tool_paths.rules
```
This pins binaries for the repository's
They constrain absolute executable paths for the repository's
[development rules](../../.codex/rules/development.rules). Rules load when a
Codex session starts.