feat: fix git window and add pretty diff viewer #178

2026-03-05T17:25:59-08:00

hikari commented

2026-03-05 17:25:59 -08:00

Summary

Fix git window "Not a git repository" error — The working directory received from Claude Code is a WSL Linux path (e.g. /home/naomi/...), but git commands were being run as native Windows processes with .current_dir(). Windows can't resolve WSL paths, causing git rev-parse --git-dir to fail. Fixed by routing git commands through wsl -- git -C <path> when the working directory starts with /.
Add syntax highlighting and line numbers to diff view — Replaced the raw <pre> block with a proper DiffViewer component featuring:
- Old/new line number columns with correct tracking across hunks
- Colour-coded gutter (+/-) with green/red row backgrounds
- Syntax highlighting via highlight.js using the detected file language, respecting all app themes via --hljs-* CSS variables
- Styled hunk headers and file headers

New files

src/lib/utils/diffParser.ts — pure diff parsing logic
src/lib/utils/diffParser.test.ts — 30 tests covering all line types, line number tracking, and language detection
src/lib/components/DiffViewer.svelte — the pretty diff viewer component

✨ This pull request was created with help from Hikari~ 🌸

## Summary - **Fix git window "Not a git repository" error** — The working directory received from Claude Code is a WSL Linux path (e.g. `/home/naomi/...`), but git commands were being run as native Windows processes with `.current_dir()`. Windows can't resolve WSL paths, causing `git rev-parse --git-dir` to fail. Fixed by routing git commands through `wsl -- git -C <path>` when the working directory starts with `/`. - **Add syntax highlighting and line numbers to diff view** — Replaced the raw `<pre>` block with a proper `DiffViewer` component featuring: - Old/new line number columns with correct tracking across hunks - Colour-coded gutter (`+`/`-`) with green/red row backgrounds - Syntax highlighting via `highlight.js` using the detected file language, respecting all app themes via `--hljs-*` CSS variables - Styled hunk headers and file headers ## New files - `src/lib/utils/diffParser.ts` — pure diff parsing logic - `src/lib/utils/diffParser.test.ts` — 30 tests covering all line types, line number tracking, and language detection - `src/lib/components/DiffViewer.svelte` — the pretty diff viewer component ✨ This pull request was created with help from Hikari~ 🌸

hikari added 2 commits 2026-03-05 17:25:59 -08:00

fix: run git commands via WSL for Linux paths on Windows 8972194a29

feat: add syntax highlighting and line numbers to diff view

Security Scan and Upload / Security & DefectDojo Upload (pull_request) Successful in 1m0s

Details

CI / Lint & Test (pull_request) Successful in 16m27s

Details

CI / Build Linux (pull_request) Successful in 20m22s

Details

CI / Build Windows (cross-compile) (pull_request) Successful in 30m28s

Details

9e04875144