Skip to content

feat: integrate batpipe for archive and file preprocessing - #24

Merged
menil merged 1 commit into
mainfrom
feat/batpipe-integration
Sep 29, 2026
Merged

menil merged 1 commit into
mainfrom
feat/batpipe-integration

Conversation

@menil

@menil menil commented Sep 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add bat-extras.batpipe: Included pkgs.bat-extras.batpipe in modules/shell.nix to enable archive/compressed file preprocessing with bat styling.
  • Configure .zshrc: Initialize batpipe (eval "$(batpipe)") with fallback handling to standalone bat and src-hilite when batpipe is not present.
  • Resolves: less and bat viewing issues on .zip, .tar*, and other compressed formats.

Closes dotfiles-36d, dotfiles-ad4, dotfiles-9dz.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Summary

Needs Attention (0 issues)

Suggestions (0 items)

All Clear

  • Correctness & Logic: Proper fallback chain, no bugs
  • Static Analysis: Clean syntax
  • Security: eval is standard for batpipe, trusted source
  • Quality & Style: Clear, well-commented
  • Dependencies: Safe addition from nixpkgs, backward compatible
  • Performance: Negligible overhead
  • PII: None detected

Verdict: Ready to Merge

The PR adds batpipe for enhanced less/cat syntax highlighting with archive support, with a clean fallback to bat. No issues requiring changes.

@menil
menil force-pushed the feat/batpipe-integration branch from c00bf05 to 614aeea Compare September 29, 2026 22:45
@menil
menil merged commit e89fc8b into main Sep 29, 2026
2 of 3 checks passed
@menil
menil deleted the feat/batpipe-integration branch September 29, 2026 22:46
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