Skip to content

fix(security): remediate code scanning findings across workflows and scripts#12

Merged
ChipWolf merged 3 commits into
mainfrom
fix/code-scanning-remediation
Mar 24, 2026
Merged

fix(security): remediate code scanning findings across workflows and scripts#12
ChipWolf merged 3 commits into
mainfrom
fix/code-scanning-remediation

Conversation

@ChipWolf
Copy link
Copy Markdown
Owner

@ChipWolf ChipWolf commented Mar 23, 2026

…scripts

Pin GitHub Actions and container references, tighten workflow permissions, and harden installer/profile scripts to eliminate flagged risky patterns. Also update Dockerfile security posture and shell lint issues while preserving existing behavior.

Made-with: Cursor


Note

Medium Risk
Moderate risk because it changes CI/release workflows, the Codespaces build image base, and Windows shell bootstrap execution paths, which could break automation if pins/permissions or script invocation differs from prior behavior.

Overview
Security hardening across CI workflows and bootstrap scripts. Workflows now set explicit top-level permissions and pin third-party actions/images by commit SHA/digest (BadgeSort, MegaLinter, Release Please, Docker actions, SARIF upload), reducing supply-chain risk.

Updates the Codespaces overlay Dockerfile to use a versioned devcontainers/universal:2 base, adjust ownership handling during COPY, and add a lightweight HEALTHCHECK.

Refactors Windows PowerShell init/installer scripts to avoid direct Invoke-Expression patterns (generate init scripts then execute via ScriptBlock; Chocolatey bootstrap downloaded to disk and run in a separate process with per-process execution policy), plus a small shell output tweak in brew-review and a release-please-config.json bootstrap-sha value change.

Written by Cursor Bugbot for commit 071ffb6. This will update automatically on new commits. Configure here.


- name: Generate badges
uses: docker://ghcr.io/chipwolf/badgesort:latest
uses: docker://ghcr.io/chipwolf/badgesort@sha256:a0b74fd865d8c93040f74c865272421e6ba4cdeb4b882990548f90e2debed04a

Check notice

Code scanning / KICS (MegaLinter REPOSITORY_KICS)

Unpinned Actions Full Length Commit SHA Note

Action is not pinned to a full length commit SHA.
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

@copilot fix this

Comment thread Dockerfile Fixed
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread Dockerfile Outdated
Comment thread release-please-config.json Outdated
Comment thread home/Documents/PowerShell/Microsoft.PowerShell_profile.ps1 Outdated
Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 23, 2026

@ChipWolf I've opened a new pull request, #13, to work on those changes. Once the pull request is ready, I'll request review from you.

@cursor
Copy link
Copy Markdown

cursor Bot commented Mar 23, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@cursor cursor Bot changed the title fix(security): remediate code scanning findings across workflows and … fix(security): remediate code scanning findings across workflows and scripts Mar 23, 2026
Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 23, 2026

@ChipWolf I've opened a new pull request, #15, to work on those changes. Once the pull request is ready, I'll request review from you.

ChipWolf and others added 3 commits March 24, 2026 12:13
…scripts

Pin GitHub Actions and container references, tighten workflow permissions, and harden installer/profile scripts to eliminate flagged risky patterns. Also update Dockerfile security posture and shell lint issues while preserving existing behavior.

Made-with: Cursor
* Initial plan

* chore: remove bootstrap-sha from release-please-config

Co-authored-by: ChipWolf <3164166+ChipWolf@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ChipWolf/dotfiles/sessions/bc6eb169-ea92-4fad-b9aa-20a78a277213

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ChipWolf <3164166+ChipWolf@users.noreply.github.com>
Co-authored-by: Chip Wolf ‮ <ChipWolf@users.noreply.github.com>
@ChipWolf ChipWolf force-pushed the fix/code-scanning-remediation branch from 6a34285 to 94beb31 Compare March 24, 2026 12:14
@cursor
Copy link
Copy Markdown

cursor Bot commented Mar 24, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@ChipWolf ChipWolf merged commit c22b4de into main Mar 24, 2026
1 check failed
@ChipWolf ChipWolf deleted the fix/code-scanning-remediation branch March 24, 2026 12:14
@github-actions github-actions Bot mentioned this pull request Mar 24, 2026
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.

4 participants