Skip to content

feat(agent): mount apply_patch tool for the gpt family - #1254

Open
TheGreatAxios wants to merge 1 commit into
cl-9530-per-model-family-wire-projection-for-tool-namesfrom
cl-9532-apply_patch-tool-for-the-gpt-family
Open

TheGreatAxios wants to merge 1 commit into
cl-9530-per-model-family-wire-projection-for-tool-namesfrom
cl-9532-apply_patch-tool-for-the-gpt-family

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Implements apply_patch tool in src/agent/apply-patch-tool.ts using the existing Codex patch parser and hunk applier.
  • Plans all operations (add, update, move, delete) upfront to prevent half-applied patches on hunk errors, and rejects workspace escapes.
  • Mounts apply_patch across sessions and subagents while advertising it specifically for the gpt profile in place of individual file mutation tools.

Verification

  • bun run typecheck, bun run build, and bun test src/agent/apply-patch-tool.test.ts pass cleanly.
  • Unit tests verify add/update/delete envelope execution, atomic rollback on hunk failure, and containment enforcement.

Fixes CL-9532

@linear-code

linear-code Bot commented Sep 30, 2026

Copy link
Copy Markdown

CL-9532

@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-9530-per-model-family-wire-projection-for-tool-names September 30, 2026 16:30
@TheGreatAxios
TheGreatAxios added this pull request to stack #1260 September 30, 2026 16:30
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1260 September 30, 2026 17:49
@TheGreatAxios
TheGreatAxios added this pull request to stack #1263 September 30, 2026 17:49
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1263 September 30, 2026 18:12
@TheGreatAxios
TheGreatAxios added this pull request to stack #1266 September 30, 2026 18:13
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1266 September 30, 2026 20:26
@TheGreatAxios
TheGreatAxios added this pull request to stack #1273 September 30, 2026 20:27
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1273 September 30, 2026 20:41
@TheGreatAxios
TheGreatAxios added this pull request to stack #1277 September 30, 2026 20:42
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1277 September 30, 2026 20:56
@TheGreatAxios
TheGreatAxios added this pull request to stack #1279 September 30, 2026 20:57
@TheGreatAxios
TheGreatAxios removed this pull request from stack #1279 September 30, 2026 23:49
@TheGreatAxios
TheGreatAxios added this pull request to stack #1284 September 30, 2026 23:50
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