Skip to content

feat(tools): advertise one posix wire set - #1182

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-8400-advertise-one-tool-set-11-names-hide-corbits-names-as-grant
Sep 27, 2026
Merged

TheGreatAxios merged 4 commits into
mainfrom
cl-8400-advertise-one-tool-set-11-names-hide-corbits-names-as-grant

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Models see one posix tool menu (read / write / edit / delete / bash / grep / glob); engine names stay hidden aliases for dispatch and grants.
  • Codex no longer advertises apply_patch / shell / update_plan beside those engines. Hidden shell argv/workdir/timeout coerce onto run_shell before authorize, so command arrays do not auto-allow.
  • Worker prompt tool names match the advertised wire set.

Verification

  • bun run check passes (lint, typecheck, dead-exports, build, guarded test suite)
  • Hidden shell with command: ["bash","-lc", script] asks on the unwrapped script rather than allowing an empty command

Fixes CL-8400

@linear-code

linear-code Bot commented Sep 26, 2026

Copy link
Copy Markdown

CL-8400

@TheGreatAxios TheGreatAxios changed the title CL-8400: advertise one tool set, hide engine aliases as grants feat(tools): advertise one posix wire set Sep 26, 2026
Advertise read/write/edit/delete/bash/grep/glob plus the control-plane.
Registry engines stay posix-named. Hidden aliases dispatch; grants
canonicalize both sides. Codex does not advertise apply_patch, shell,
or update_plan.

Fixes CL-8400
Authorize classified Codex command arrays as empty strings, so
reactor-gated hidden shell auto-allowed and the unwrapped script ran.
Stored update_plan grants canonicalized onto manage_tasks, so a
create-only plan approval auto-allowed full task lifecycle calls.
Coverage is now one-directional and seeders drop the narrow key.
Live requests never present as aliases (coerced before matching), so
tests presenting update_plan requests exercise an unreachable path.
Fail-closed pins and seeder tests remain.
@TheGreatAxios
TheGreatAxios force-pushed the cl-8400-advertise-one-tool-set-11-names-hide-corbits-names-as-grant branch from 1a6c171 to 93ed58a Compare September 27, 2026 22:33
@TheGreatAxios
TheGreatAxios enabled auto-merge (squash) September 27, 2026 22:34
@TheGreatAxios
TheGreatAxios merged commit 9920046 into main Sep 27, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant