Skip to content

harden export filtering and release versioning#34

Merged
hyle merged 1 commit intomainfrom
misc-fixing
Mar 17, 2026
Merged

harden export filtering and release versioning#34
hyle merged 1 commit intomainfrom
misc-fixing

Conversation

@hyle
Copy link
Copy Markdown
Owner

@hyle hyle commented Mar 17, 2026

Summary

Fix the issues around sensitive-file detection, recursive filesystem robustness, absolute stdin tree rendering, and release version stamping.

Changes

  • Broaden sensitive filename protection to match the documented wildcard-style behavior for:
    • .env*
    • credentials*
    • secret*
    • id_rsa* and related key-name prefixes
  • Make recursive filesystem export skip unreadable descendant directories with a warning instead of aborting the entire walk
  • Preserve the leading / for absolute stdin paths in the project tree so the tree matches absolute file headings
  • Pass the tag name into make in the release workflow so release binaries report the actual version instead of dev
  • Extend CLI coverage for:
    • additional sensitive filename variants
    • unreadable descendant directories
    • absolute stdin paths in the tree
    • root-only skipping of the unreadable-file permission test

Why

These changes bring the implementation in line with the documented behavior, make recursive exports more resilient to partial filesystem access problems, keep the tree view consistent with exported headings, and ensure release artifacts report the correct version.

Verification

  • Ran make test successfully
  • Release-version wiring to try in CI

@hyle hyle self-assigned this Mar 17, 2026
@hyle hyle added bug Something isn't working enhancement New feature or request labels Mar 17, 2026
@hyle hyle merged commit a69710e into main Mar 17, 2026
@hyle hyle deleted the misc-fixing branch March 17, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant