Skip to content

Fix GitHub Copilot apply_patch hook handling#819

Merged
svarlamov merged 1 commit intomainfrom
codex/fix-copilot-apply-patch-hooks
Mar 27, 2026
Merged

Fix GitHub Copilot apply_patch hook handling#819
svarlamov merged 1 commit intomainfrom
codex/fix-copilot-apply-patch-hooks

Conversation

@svarlamov
Copy link
Copy Markdown
Member

@svarlamov svarlamov commented Mar 27, 2026

Fixes #814.

Summary

  • treat VS Code Copilot's apply_patch tool name as a supported edit tool
  • extract edited file paths from raw apply-patch text payloads used in Copilot pre/post hook events
  • add focused Copilot integration tests for apply_patch pre and post hooks

Testing

  • cargo test --test integration copilot_preset_vscode -- --nocapture

Recognize VS Code Copilot's apply_patch tool name as an edit tool and extract file paths from raw patch-text payloads for pre/post hooks.

Refs #814.
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@svarlamov svarlamov merged commit 10cfc12 into main Mar 27, 2026
21 checks passed
@svarlamov svarlamov deleted the codex/fix-copilot-apply-patch-hooks branch March 27, 2026 03:37
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.

[Bug]: Some tools (all?) not supported for GitHub Copilot

1 participant