Skip to content

fix: keep permissions profiles forward compatible#14107

Merged
viyatb-oai merged 4 commits intomainfrom
codex/viyatb/permissions-profile-forward-compat
Mar 10, 2026
Merged

fix: keep permissions profiles forward compatible#14107
viyatb-oai merged 4 commits intomainfrom
codex/viyatb/permissions-profile-forward-compat

Conversation

@viyatb-oai
Copy link
Copy Markdown
Collaborator

Summary

  • preserve unknown :special_path tokens, including nested entries, so older Codex builds warn and ignore instead of failing config load
  • fail closed with a startup warning when a permissions profile has missing or empty filesystem entries instead of aborting profile compilation
  • normalize Windows verbatim paths like \?\C:\... before absolute-path validation while keeping explicit errors for truly invalid paths

Testing

  • just fmt
  • cargo test -p codex-core permissions_profiles_allow
  • cargo test -p codex-core normalize_absolute_path_for_platform_simplifies_windows_verbatim_paths
  • cargo test -p codex-protocol unknown_special_paths_are_ignored_by_legacy_bridge
  • cargo clippy -p codex-core -p codex-protocol --all-targets -- -D warnings
  • cargo clean
@viyatb-oai viyatb-oai merged commit 1165a16 into main Mar 10, 2026
74 of 83 checks passed
@viyatb-oai viyatb-oai deleted the codex/viyatb/permissions-profile-forward-compat branch March 10, 2026 01:43
@github-actions github-actions bot locked and limited conversation to collaborators Mar 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants