Skip to content

docs(readme): fix Copilot setup + add Vibe#824

Merged
pszymkowiak merged 5 commits intodevelopfrom
docs/readme-copilot-vibe
Mar 26, 2026
Merged

docs(readme): fix Copilot setup + add Vibe#824
pszymkowiak merged 5 commits intodevelopfrom
docs/readme-copilot-vibe

Conversation

@pszymkowiak
Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • README renders correctly
  • All 10 tools listed with correct install commands

- Split Copilot into VS Code (transparent rewrite) and CLI (deny-with-suggestion)
- Use --copilot flag (consistent with --gemini, --codex, --opencode)
- Add Mistral Vibe as planned (blocked on upstream #531)
- Fix Copilot section with VS Code vs CLI details
- Update tool count from 9 to 10
- Verified all 10 tools against actual codebase

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
@pszymkowiak pszymkowiak force-pushed the docs/readme-copilot-vibe branch from 9621149 to 19d35f5 Compare March 25, 2026 16:36
- Add `rtk init --copilot` that creates .github/hooks/rtk-rewrite.json
  and .github/copilot-instructions.md in the current project
- Hook routes through `rtk hook copilot` (auto-detects VS Code vs CLI)
- VS Code Copilot Chat: transparent rewrite via updatedInput
- Copilot CLI: deny-with-suggestion (CLI limitation)
- Update README: split Copilot VS Code/CLI, add Vibe status, 10 tools

Closes #823

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Silent truncation caused a user to make irreversible decisions on
incomplete data. rtk diff was capping at 50 changes and 70-80 char
lines without clear warning to the LLM.

- Remove all line truncation (70/80 char limits)
- Remove change count limit (was 50, now unlimited)
- Remove truncate import (no longer needed)
- Same fix for condense_unified_diff (stdin/git diff path)
- Add 3 tests verifying zero truncation on large inputs

The only compression remaining is the summary header
(+N added, -N removed, ~N modified) and stripping diff metadata.

Fixes #827

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

- Detect binary files (null bytes in first 8KB) before filtering
- Show clear message: [binary file] path (size) instead of empty output
- Fallback to raw content if filter produces empty output on non-empty file
- Prevents LLM from concluding a 70MB file is "empty" (was #822)

Fixes #822

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
Copy link
Copy Markdown
Collaborator

@FlorianBruniaux FlorianBruniaux left a comment

Choose a reason for hiding this comment

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

ok

Changed default filter level from "minimal" to "none". RTK read now
shows complete file content by default. Filtering is opt-in:

  rtk read file.rs              # full content (was: minimal filter)
  rtk read file.rs -l minimal   # light filtering (opt-in)
  rtk read file.rs -l aggressive # signatures only (opt-in)

Also adds fallback: if a filter produces empty output on non-empty
file, show raw content with a warning.

Fixes #822

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
@pszymkowiak pszymkowiak force-pushed the docs/readme-copilot-vibe branch from d871082 to 5e0f3ba Compare March 25, 2026 20:08
@pszymkowiak pszymkowiak merged commit 5e8626d into develop Mar 26, 2026
6 of 7 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.

3 participants