fix: Skip PSScriptAnalyzer install on cache hit

[tablackburn]

Summary

• The lint job was unconditionally running Install-Module even when PSScriptAnalyzer was already restored from cache
• This caused Install-Module -Force to conflict with the already-loaded cached module, which can trigger a NullReferenceException in PSScriptAnalyzer
• Added id to the cache step and an if condition on the install step so it only runs on cache miss
• Added workflow_dispatch trigger to allow manual CI runs

Test Plan

Note: This is a template repository — CI workflows are not intended to run here. When using this template to create a new repository, verify the following:

• CI lint job passes with a cold cache (no prior cache entry)
• On a subsequent run with a warm cache, the Install-Module step is skipped
• workflow_dispatch trigger allows manual CI runs from the Actions tab

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds workflow_dispatch manual trigger to the CI workflow, assigns an explicit step ID (cache-lint-modules) to the PowerShell module cache step, and makes the Install PSScriptAnalyzer step conditional on steps.cache-lint-modules.outputs.cache-hit != 'true'.

Changes

Cohort / File(s)	Summary
CI Workflow /.github/workflows/CI.yaml	Added workflow_dispatch manual trigger; set explicit step ID cache-lint-modules for PowerShell module cache; made PSScriptAnalyzer installation run only when cache miss (if: steps.cache-lint-modules.outputs.cache-hit != 'true').

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐇 I named the cache, I set the rule,
Skip installs when modules stay cool.
A tiny tweak, a faster run,
CI hops forward—job well done! 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title clearly and concisely summarizes the main fix: adding a conditional check to skip PSScriptAnalyzer installation when the cache is already hit.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bugfix/ci-lint-cache-conditional-install

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Updates the CI lint workflow to avoid reinstalling PSScriptAnalyzer when it’s already restored from the GitHub Actions cache, preventing install/load conflicts during lint runs.

Changes:

• Adds a step id to the PowerShell modules cache step.
• Gates the Install PSScriptAnalyzer step to only run on cache misses.

---

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