Skip to content

feat(copilot): cross-platform PreToolUse hook via rtk hook (VS Code + Copilot CLI + Windows)#605

Merged
pszymkowiak merged 1 commit intortk-ai:developfrom
jeziellopes:feat(copilot)/copilot-cli-integration
Mar 18, 2026
Merged

feat(copilot): cross-platform PreToolUse hook via rtk hook (VS Code + Copilot CLI + Windows)#605
pszymkowiak merged 1 commit intortk-ai:developfrom
jeziellopes:feat(copilot)/copilot-cli-integration

Conversation

@jeziellopes
Copy link
Contributor

@jeziellopes jeziellopes commented Mar 15, 2026

Closes #593

What this adds

Cross-platform PreToolUse hook for GitHub Copilot (VS Code Copilot Chat + Copilot CLI) via rtk hook — a native Rust binary. No bash scripts, no jq, works on Windows natively.

How it works

rtk hook reads PreToolUse JSON from stdin, detects the agent format, responds appropriately:

VS Code Copilot Chat (updatedInput — transparent rewrite):

git status → rtk hook → updatedInput: "rtk git status"  (silent, no denial)

GitHub Copilot CLI (deny-with-suggestion — CLI ignores updatedInput today, copilot-cli#2013):

git status → rtk hook → deny: "Token savings: use `rtk git status` instead"
             Copilot reads reason → re-runs rtk git status

When Copilot CLI adds updatedInput support, only rtk hook needs a one-line change.

Also fixed

The old .github/hooks/rtk-rewrite.json had wrong key names (preToolUsePreToolUse, bash/timeoutSeccommand/timeout) that prevented it from loading in VS Code.

Measured gains (validated Copilot CLI session)

Command Savings Tokens saved
rtk cargo test 99.9% 15.8K
rtk cargo clippy --all-targets 93.3% 5.9K
rtk git diff HEAD~1 81.2% 2.6K
rtk gh pr list 67.0% 0.8K
Session total (24 commands) 83.5% avg ~23.3K

Files

File Role
src/hook_cmd.rs New rtk hook subcommand (13 unit tests)
.github/hooks/rtk-rewrite.json Fixed key casing, points to rtk hook
hooks/copilot-rtk-rewrite.sh Deleted — replaced by rtk hook
hooks/test-copilot-rtk-rewrite.sh Rewritten to test rtk hook
hooks/copilot-rtk-awareness.md Updated: VS Code row, both output paths
src/init.rs Remove bash script checks, update status message
README.md VS Code Copilot Chat support, Windows note
CHANGELOG.md Updated unreleased entry

Testing

# Unit tests (13 tests)
cargo test hook_cmd

# Manual: Copilot CLI deny path
echo '{"toolName":"bash","toolArgs":"{\"command\":\"git status\"}"}' | rtk hook

# Manual: VS Code updatedInput path
echo '{"tool_name":"Bash","tool_input":{"command":"git status"}}' | rtk hook

# Verify rtk init --show
rtk init --show

@vrdons
Copy link

vrdons commented Mar 17, 2026

You should have to do windows support too, also his maybe works on vscode copilot integration. I'm planning to add many extensions to the VS Code chat integration.

@vrdons
Copy link

vrdons commented Mar 17, 2026

Also rewrite script should be automated. Maybe adding rust code should fix that

@jeziellopes
Copy link
Contributor Author

@vrdons Great suggestions — this PR has been expanded to implement exactly what you described.

Rust binary instead of bash script
Replaced hooks/copilot-rtk-rewrite.sh with rtk hook, a native Rust subcommand. No bash, no jq, no shell dependency. Works on Windows natively.

VS Code Copilot Chat support
VS Code reads .github/hooks/*.json automatically (same dir, same format). I investigated the VS Code hooks docs and found that VS Code also supports updatedInput — which enables a transparent rewrite instead of a deny. So now rtk hook handles both agents differently:

  • VS Code Copilot Chat → returns updatedInput (silent rewrite, no denial, no retry)
  • GitHub Copilot CLI → returns permissionDecision: deny with suggestion (CLI ignores updatedInput today — tracked in copilot-cli#2013)

When Copilot CLI adds updatedInput support, only rtk hook needs a one-line change.

The old bash hook also had wrong key casing (preToolUse → should be PreToolUse, bash/timeoutSec → should be command/timeout). All fixed now.

@jeziellopes jeziellopes changed the title feat(copilot): GitHub Copilot CLI integration feat(copilot): cross-platform PreToolUse hook via rtk hook (VS Code + Copilot CLI + Windows) Mar 18, 2026
@olivier-Attestis
Copy link

Can't wait to have this feature ! my premium credits are exploding !
Thanks a lot for the hard work guys !

Add `rtk hook copilot` command that handles both VS Code Copilot Chat
(updatedInput rewrite) and GitHub Copilot CLI (deny-with-suggestion).

- Auto-detects format: snake_case (VS Code) vs camelCase (Copilot CLI)
- Delegates to `rtk rewrite` (single source of truth)
- 14 hook tests (format detection, rewrite gating, output shape)
- .github/hooks/rtk-rewrite.json for repo-scoped hook config
- .github/copilot-instructions.md for RTK awareness
- Test script: hooks/test-copilot-rtk-rewrite.sh

Rebased on develop (includes Gemini rtk-ai#573, Codex rtk-ai#377, OpenClaw rtk-ai#358).

Original work by @jeziellopes, cleaned up and rebased by maintainer.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
@pszymkowiak pszymkowiak force-pushed the feat(copilot)/copilot-cli-integration branch from 4eeace1 to 1b44eb5 Compare March 18, 2026 15:01
@pszymkowiak
Copy link
Collaborator

@jeziellopes — Rebased your PR on develop (which now includes Gemini #573, Codex #377, and OpenClaw #358).

Changes made during cleanup:

  • Merged Copilot hook functions into the existing hook_cmd.rs (which already had Gemini support)
  • Added HookCommands::Copilot variant in main.rsrtk hook copilot
  • Removed conflicting code from the original merge
  • Kept all your Copilot-specific files (.github/hooks, copilot-instructions.md, test script, awareness doc)
  • 14 hook tests passing (9 Copilot + 5 Gemini)

Tested manually:

  • VS Code format: {"tool_name":"Bash","tool_input":{"command":"git status"}} → rewrite with updatedInput
  • Copilot CLI format: {"toolName":"bash","toolArgs":...} → deny-with-suggestion
  • Non-bash tools: passthrough (silent)
  • Gemini hook still works alongside Copilot

Ready to merge into develop. Thanks for the solid work on cross-platform Copilot support!

@pszymkowiak
Copy link
Collaborator

@jeziellopes — Tested the Copilot hook end-to-end on macOS with Copilot CLI 1.0.7:

$ copilot -p "Run git status" --yolo

Copilot correctly executed via the RTK hook — output was filtered (compact 4-line format instead of raw git output). The rtk hook copilot command auto-detected the Copilot format and rewrote git statusrtk git status transparently.

Also verified:

  • rtk hook copilot (VS Code format): updatedInput rewrite works
  • rtk hook copilot (Copilot CLI format): deny-with-suggestion works
  • rtk hook gemini still works alongside Copilot (no regression)
  • 14 hook tests passing

Ready to merge. Thanks for the great contribution — Copilot support was a top request!

@pszymkowiak pszymkowiak merged commit 0800bbe into rtk-ai:develop Mar 18, 2026
9 checks passed
pszymkowiak added a commit that referenced this pull request Mar 19, 2026
* fix: remove all decorative emojis from CLI output (#687)

* fix: remove decorative emojis from CLI output (#511)

Replace decorative emojis with plain text to reduce token waste.
Keep functional symbols (⚠️ ✓ ❌ ✅ ℹ️) that convey meaning in fewer tokens.

Signed-off-by: Patrick Szymkowiak <patrick@rtk.ai>
Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix: remove remaining decorative emojis from find_cmd and formatter

Missed in initial emoji cleanup pass: 📁 in find_cmd.rs and parser/formatter.rs

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix: remove all decorative emojis from CLI output (#511)

Replace emojis with plain text tokens across all production files
for better LLM compatibility. Test fixtures and external tool
detection patterns (e.g. Black's "All done!") are preserved.

Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app>
Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix: remove last decorative emoji from next_cmd.rs

Remove ⚡ from Next.js Build header, missed in previous passes.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix: remove remaining emojis from gh_cmd.rs and init.rs

Replace production emojis:
- gh_cmd.rs: 🟣→[merged], ⚪→[unknown]/[pending], ⭐→removed, 🔱→removed
- init.rs: ⚪→[--] for "not found" status indicators

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix: remove all checkmark emojis from CLI output

Replace ✓ (U+2713) with plain text across 19 files:
- "ok ✓" → "ok" (git add/commit/push/pull)
- "✓ cargo test: ..." → "cargo test: ..." (all tool summaries)
- Preserved ✓ in input detection patterns and test fixtures

LLMs cannot interpret emoji semantics; plain text is clearer.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

---------

Signed-off-by: Patrick Szymkowiak <patrick@rtk.ai>
Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app>

* feat: OpenClaw plugin for transparent exec rewriting (#358)

* feat: add OpenClaw plugin for transparent exec rewriting

Adds an OpenClaw plugin that intercepts exec tool calls via the
before_tool_call hook and rewrites commands to their RTK equivalents.

This is the OpenClaw equivalent of hooks/rtk-rewrite.sh for Claude Code.

The plugin:
- Registers a before_tool_call hook on the exec tool
- Rewrites git, grep, find, ls, gh, docker, kubectl, and test commands
- Guards against rewriting piped/compound commands and heredocs
- Returns properly typed PluginHookBeforeToolCallResult
- Supports enabled/verbose config options

Measured savings: 48-87% token reduction on common commands.

Files:
- openclaw/index.ts — plugin source
- openclaw/openclaw.plugin.json — plugin manifest
- openclaw/README.md — installation and usage docs

* refactor: delegate OpenClaw plugin to rtk rewrite

Replace 60+ hardcoded regex rules with a single call to `rtk rewrite`,
matching the OpenCode plugin pattern (hooks/opencode-rtk.ts).

Benefits:
- Zero maintenance: new RTK filters work automatically
- Single source of truth: rewrite logic in Rust (src/discover/registry.rs)
- 122 → 73 lines, no rule duplication

Also: rebase on develop, fix homepage URL, bump version to 1.0.0.
Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* feat: add package.json for npm publishing

Enables `openclaw plugins install @rtk-ai/rtk-rewrite` for OpenClaw users.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

---------

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Co-authored-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* feat: add Gemini CLI support via rtk init --gemini (#573)

- Add `rtk hook gemini` command: native Rust hook processor for Gemini CLI
  BeforeTool hooks. Reads JSON from stdin, delegates to `rewrite_command()`
  (single source of truth), outputs Gemini-format JSON response.
- Add `--gemini` flag to `rtk init`: installs hook wrapper script, GEMINI.md,
  and patches ~/.gemini/settings.json with BeforeTool hook entry.
- Add `rtk init -g --gemini --uninstall`: clean removal of all Gemini artifacts.
- 6 unit tests covering hook format, rewrite delegation, and exclusions.

Replaces PR #174 which had too many conflicts after upstream restructuring.

Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

* feat(init): add Codex CLI support via AGENTS.md + RTK.md workflow (#377)

* feat(init): add Codex CLI support via AGENTS.md + RTK.md workflow

Add --codex mode to rtk init for Codex CLI integration using AGENTS.md + RTK.md, while keeping the newer develop init/opencode flow intact. Includes Codex install/show/uninstall handling, ASCII status output, stricter flag validation, and expanded tests for Codex AGENTS lifecycle and patch-mode rejection.

Signed-off-by: Zacaria <havesomecode@gmail.com>

* docs: fix validation metadata

Signed-off-by: Zacaria <havesomecode@gmail.com>

---------

Signed-off-by: Zacaria <havesomecode@gmail.com>

* feat(copilot): add Copilot hook support (VS Code + CLI) (#605)

Add `rtk hook copilot` command that handles both VS Code Copilot Chat
(updatedInput rewrite) and GitHub Copilot CLI (deny-with-suggestion).

- Auto-detects format: snake_case (VS Code) vs camelCase (Copilot CLI)
- Delegates to `rtk rewrite` (single source of truth)
- 14 hook tests (format detection, rewrite gating, output shape)
- .github/hooks/rtk-rewrite.json for repo-scoped hook config
- .github/copilot-instructions.md for RTK awareness
- Test script: hooks/test-copilot-rtk-rewrite.sh

Rebased on develop (includes Gemini #573, Codex #377, OpenClaw #358).

Original work by @jeziellopes, cleaned up and rebased by maintainer.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Co-authored-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* feat: add Cursor Agent support via --agent flag (#595)

Add `rtk init -g --agent cursor` to install RTK hooks for Cursor Agent.
Cursor's preToolUse hook supports command rewriting via updated_input,
functionally identical to Claude Code's PreToolUse. Works with both the
Cursor editor and cursor-cli (they share ~/.cursor/hooks.json).

Changes:
- New `--agent <name>` flag (claude|cursor) on `rtk init`, extensible
  for future agents. Default is claude (backward compatible).
- Cursor hook script (hooks/cursor-rtk-rewrite.sh) outputs Cursor's
  JSON format: {permission, updated_input} vs Claude's hookSpecificOutput.
- `rtk init --show` reports Cursor hook and hooks.json status.
- `rtk init -g --uninstall` removes Cursor artifacts.
- `rtk discover` notes that Cursor sessions are tracked via `rtk gain`
  (Cursor transcripts lack structured tool_use/tool_result blocks).
- Unit tests for Cursor hooks.json patching, detection, and removal.

Made-with: Cursor

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Co-authored-by: Moisei <1199723+moisei@users.noreply.github.com>

* feat: add Windsurf support via rtk init --agent windsurf (#695) (#697)

Install RTK rules in .windsurfrules (project-scoped) so Cascade
prefixes shell commands with rtk for token savings.

Windsurf hooks don't support command rewriting (only blocking),
so RTK uses the rules-based approach (like Codex with AGENTS.md).

Tested: Windsurf Cascade correctly uses rtk git status after install.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* feat: add Cline/Roo Code support via rtk init --agent cline (#701) (#702)

Install RTK rules in .clinerules (project-scoped) so Cline
prefixes shell commands with rtk for token savings.

Same rules-based approach as Windsurf and Codex.

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

* fix(skill/rtk-triage): increase PR/issue limit to 200 with pagination hint (#717)

* fix(skill/rtk-triage): increase PR/issue limit to 200 with pagination hint

Raise gh pr list limit from 60 to 200 to match gh's max per call.
Add inline comment explaining how to paginate for repos with >200 open PRs.
Update threshold warning from >60 to >200 PRs/issues.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>

* docs(architecture): update module count to 67 (hook_cmd added in #573)

hook_cmd.rs was added in feat: add Gemini CLI support (#573) but
ARCHITECTURE.md was not updated. Fixes pre-push validation failure.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>

---------

Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Signed-off-by: Patrick Szymkowiak <patrick@rtk.ai>
Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app>
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>
Signed-off-by: Zacaria <havesomecode@gmail.com>
Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com>
Co-authored-by: Alex <a.neyman17@gmail.com>
Co-authored-by: Ben Younes <benyounes.ousama@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Zacaria Chtatar <Zacaria@users.noreply.github.com>
Co-authored-by: Jeziel Lopes <jeziellcarvalho@gmail.com>
Co-authored-by: Moisei Rabinovich <moisei@users.noreply.github.com>
Co-authored-by: Moisei <1199723+moisei@users.noreply.github.com>
Co-authored-by: Florian BRUNIAUX <florian@bruniaux.com>
jeziellopes added a commit to jeziellopes/rtk that referenced this pull request Mar 19, 2026
Add GitHub Copilot as a target agent for rtk init:
- Add `Copilot` variant to `AgentTarget` enum
- Add `hooks/copilot-rtk-rules.md` with compact RTK instructions
- Add `run_copilot_mode()` that writes `.github/copilot-instructions.md`
  (idempotent: skips if RTK already detected in the file)
- Add `--uninstall --agent copilot` to remove the instructions file
- Route `install_copilot` through `init::run()` and `init::uninstall()`

Usage:
  rtk init --agent copilot          # install in current project
  rtk init --uninstall --agent copilot  # remove

Closes the gap from PR rtk-ai#605 / rtk-ai#704 where rtk hook copilot was added
but rtk init had no copilot counterpart.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: jeziellopes <jeziellopes@gmail.com>
jeziellopes added a commit to jeziellopes/rtk that referenced this pull request Mar 19, 2026
Add GitHub Copilot as a target agent for rtk init:
- Add `Copilot` variant to `AgentTarget` enum
- Add `hooks/copilot-rtk-rules.md` with compact RTK instructions
- Add `run_copilot_mode()` that writes `.github/copilot-instructions.md`
  (idempotent: skips if RTK already detected in the file)
- Add `--uninstall --agent copilot` to remove the instructions file
- Route `install_copilot` through `init::run()` and `init::uninstall()`

Usage:
  rtk init --agent copilot          # install in current project
  rtk init --uninstall --agent copilot  # remove

Closes the gap from PR rtk-ai#605 / rtk-ai#704 where rtk hook copilot was added
but rtk init had no copilot counterpart.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: jeziellopes <jeziellopes@gmail.com>
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.

4 participants