feat(cargo-heather): recognize .psd1 and .psm1 as PowerShell files

[Sander Saares (sandersaares)]

Recognize .psd1 (PowerShell data) and .psm1 (PowerShell module) files as PowerShell sources in FileKind::detect so the scanner checks/fixes their license headers instead of silently skipping them.

Motivation

.psd1 and .psm1 use the same # line-comment syntax as .ps1 scripts and routinely appear alongside them — module manifests (MyMod.psd1), data files driving PowerShell-based tooling (e.g. Pester scenario fixtures), and module implementations (MyMod.psm1). Pre-fix, FileKind::detect matched only ps1 so these files were silently dropped during scanning. Running cargo heather against a directory containing .psd1 fixtures with no header reports All N file(s) have correct license headers even though those fixtures were never inspected.

This was observed in microsoft/oxidizer#436 (a new Pester scenario suite landed ~25 .psd1 fixtures without license headers and CI did not catch it).

Change

• FileKind::detect in crates/cargo-heather/src/comment.rs: extend the PowerShell match arm to additionally accept psd1 and psm1 (case-insensitive, matching existing arms).
• Doc comments on FileKind::PowerShell and lib.rs module docs updated to enumerate the supported PowerShell extensions.
• crates/cargo-heather/README.md brought up to date — it previously claimed only .rs and .toml were supported (pre-dated even .ps1, .just, and constants.env).

The downstream scanner in src/bin/cargo-heather/scanner.rs filters via CommentStyle::from_path(path).is_some(), which delegates to FileKind::detect; comment-style mapping for FileKind::PowerShell was already CommentStyle::Hash. So extending detect is sufficient — no scanner or fix-mode changes needed.

Tests

• 4 unit tests added in comment::tests exercising FileKind::detect for .ps1, .psd1, .psm1 (mixed case), plus a negative test asserting we did not start matching on substrings like .ps, .psd, .psm.
• scanner_includes_legacy_hash_comment_file_types (integration) now includes Settings.psd1 and Module.psm1 fixtures and asserts Checking 6 file(s).
• 2 new integration tests (scanner_reports_missing_header_in_powershell_data_file and …_module_file) lock in the regression: a missing header on a .psd1 / .psm1 is detected and reported, not silently skipped.

cargo test -p cargo-heather passes (25 unit + 19 integration + 5 ms-mit + 3 toml + 3 script + 7 etc.; no failures). cargo clippy -p cargo-heather --all-targets -- -D warnings clean. just format-check and the workspace cargo heather run both clean.

Backwards compatibility

Purely additive. Existing .ps1, .toml, .rs, justfile, constants.env behaviour is unchanged. Users with .psd1 / .psm1 files in their tree that previously had no header will see new MISSING header diagnostics on their next run — that is the desired behaviour (the previous silent skip was the defect).

[Copilot]

Pull request overview

Expands cargo-heather’s PowerShell support so .psd1 (data/module manifests) and .psm1 (modules) are treated as PowerShell sources during scanning, ensuring license headers are validated/fixed rather than silently skipped.

Changes:

• Extend FileKind::detect to classify .psd1 and .psm1 (case-insensitive) as FileKind::PowerShell.
• Add unit + integration test coverage to prevent regressions and to assert the scanner reports missing headers for these file types.
• Update crate docs/README to reflect the broader set of supported source file kinds.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
crates/cargo-heather/tests/cli.rs	Extends integration tests to include .psd1/.psm1 scanning and missing-header reporting.
crates/cargo-heather/src/lib.rs	Updates library docs to enumerate supported PowerShell extensions.
crates/cargo-heather/src/comment.rs	Updates PowerShell detection logic and adds unit tests for new extensions.
crates/cargo-heather/README.md	Updates README to list supported file kinds and scanning behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[Codecov Comments Bot (codecov-commenter)]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.2%. Comparing base (48e192e) to head (17f9709).

Additional details and impacted files

@@           Coverage Diff           @@
##            main     #38     +/-   ##
=======================================
+ Coverage   93.1%   93.2%   +0.1%     
=======================================
  Files         28      28             
  Lines       1241    1262     +21     
=======================================
+ Hits        1156    1177     +21     
  Misses        85      85             

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.