Skip to content

fix(term): do not capture bat, delta, glow and git diff/log/show/blame in PowerShell - #63

Open
NaNIsaac wants to merge 2 commits into
mainfrom
fix/hook-skip-colour-pagers
Open

NaNIsaac wants to merge 2 commits into
mainfrom
fix/hook-skip-colour-pagers

Conversation

@NaNIsaac

@NaNIsaac NaNIsaac commented Nov 5, 2026

Copy link
Copy Markdown
Owner

In PowerShell the hook runs a typed line as . $block *>&1 | __cdgRecord (hook.ps1:118), so a captured program's stdout is a pipe, not a terminal. bat, delta and git's pager then print plain text without colour or paging. #62 did not change this; it only retries programs that failed to start.

Change: the default skip regex now also matches bat, batcat, delta, glow and git diff/log/show/blame or git -p/--paginate, as it already does for less. README (en, ko) lists them and points to CODIGEST_CAP_SKIP for other tools. Capture of all other commands is unchanged.

Test: tests/test_hook_wants.py checks __cdgWants for 12 skipped and 4 captured lines in pwsh and Windows PowerShell (2 passed locally). Full suite not run here.

…e in PowerShell

The PowerShell hook pipes a captured command's output through __cdgRecord, so the program's stdout is not a terminal and bat, delta and git's pager drop colour and paging. These commands now bypass capture like less does; CODIGEST_CAP_SKIP covers other tools.

This branch has not been deployed

No deployments
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.

1 participant