Skip to content

feat: add GitHub CLI integration (depends on #9) #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pszymkowiak merged 3 commits into from
Jan 29, 2026
Merged

feat: add GitHub CLI integration (depends on #9) #10

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0f7418e
feat: add GitHub CLI integration with token optimizations
FlorianBruniaux Jan 29, 2026
05be911
docs: update CLAUDE.md with PR #9 features
FlorianBruniaux Jan 29, 2026
9526471
fix(ci): correct rust-toolchain action name
FlorianBruniaux Jan 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/benchmark.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- uses: actions/checkout@v4

- name: Install Rust
uses: dtolnay/rust-action@stable
uses: dtolnay/rust-toolchain@stable

@FlorianBruniaux FlorianBruniaux Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@pszymkowiak ici ;)


- name: Build rtk
run: cargo build --release
Expand Down
28 changes: 28 additions & 0 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,12 @@ main.rs (CLI entry)
- Reads ~/.config/rtk/config.toml for user preferences
- `rtk init` command bootstraps LLM integration

**5. Shared Utilities** (src/utils.rs)
- Common functions for command modules: truncate, strip_ansi, execute_command
- Package manager auto-detection (pnpm/yarn/npm/npx)
- Consistent error handling and output formatting
- Used by all modern JavaScript/TypeScript tooling commands

### Command Routing Flow

All commands follow this pattern:
Expand Down Expand Up @@ -144,6 +150,16 @@ main.rs:Commands enum
| runner.rs | Command execution | Stderr only (err), failures only (test) |
| log_cmd.rs | Log parsing | Deduplication with counts |
| json_cmd.rs | JSON inspection | Structure without values |
| lint_cmd.rs | ESLint/Biome linting | Group by rule, file summary (84% reduction) |
| tsc_cmd.rs | TypeScript compiler | Group by file/error code (83% reduction) |
| next_cmd.rs | Next.js build/dev | Route metrics, bundle stats only (87% reduction) |
| prettier_cmd.rs | Format checking | Files needing changes only (70% reduction) |
| playwright_cmd.rs | E2E test results | Failures only, grouped by suite (94% reduction) |
| prisma_cmd.rs | Prisma CLI | Strip ASCII art and verbose output (88% reduction) |
| gh_cmd.rs | GitHub CLI | Compact PR/issue/run views (26-87% reduction) |
| vitest_cmd.rs | Vitest test runner | Failures only with ANSI stripping (99.5% reduction) |
| pnpm_cmd.rs | pnpm package manager | Compact dependency trees (70-90% reduction) |
| utils.rs | Shared utilities | Package manager detection, common formatting |

## Fork-Specific Features

Expand All @@ -157,6 +173,18 @@ main.rs:Commands enum
- **Token Savings**: 70-90% reduction on package manager operations
- **Security**: Package name validation prevents command injection

### PR #9: Modern JavaScript/TypeScript Tooling (2026-01-29)
- **New Commands**: 6 commands for T3 Stack workflows
- `rtk lint`: ESLint/Biome with grouped rule violations (84% reduction)
- `rtk tsc`: TypeScript compiler errors grouped by file/code (83% reduction)
- `rtk next`: Next.js build with route/bundle metrics (87% reduction)
- `rtk prettier`: Format checker showing files needing changes (70% reduction)
- `rtk playwright`: E2E test results showing failures only (94% reduction)
- `rtk prisma`: Prisma CLI without ASCII art (88% reduction)
- **Shared Infrastructure**: utils.rs module for package manager auto-detection
- **Features**: Exit code preservation, error grouping, consistent formatting
- **Testing**: Validated on production T3 Stack project (methode-aristote/app)

## Testing Strategy

Tests are embedded in modules using `#[cfg(test)] mod tests`:
Expand Down
Loading