CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix: move param() to first statement in protected-paths.ps1 #4683

Open⚡ AI-generatedXSccantynz wants to mergecopilot/fix-protected-paths-script-errormain↑2 ↓9opened Mar 19, 20265/7 tasksLive: 0 editing
ccantynzcommented Mar 19, 2026

Originally written by @Copilot on GitHub.
Imported from https://github.com/ccantynz-alt/Dominat8.com/pull/19


What changed?

  • Small fix
  • Feature
  • Refactor
  • Infra / CI

PowerShell requires param() to be the first executable statement in a script. Set-StrictMode and $ErrorActionPreference preceded it, causing a parse error on every PR:

ParserError: .github/scripts/protected-paths.ps1:5
     | The assignment expression is not valid.

Swapped the order so param() comes first.

Quality gates (must be green)

  • CI build is green
  • No unexpected UI/design changes on Dominat8.com
  • If protected paths changed: override label OR ALLOW_PROTECTED_PATHS file added with reason

Notes / risks

None. Pure statement reorder, no behavioral change.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

ccantynzcommented Mar 19, 2026

Originally written by @vercel[bot] on GitHub.


The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dominat8com Canceled Canceled Mar 19, 2026 9:09pm

Request Review

gluecron[bot]🤖 botcommented 25d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 18d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 11d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.
gluecron[bot]🤖 botcommented 4d ago

This PR has gone quiet

No activity for 7+ days. Is this still in progress?

  • Author: keep working / mark draft / close.
  • Reviewers: review or unassign yourself.
  • Maintainers: close if no longer relevant.

Files changed

Side-by-side diff for copilot/fix-protected-paths-script-error → main.

View diff →

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
Risk score: LOW (2/10)2c6d6ce
Touches 1 file(s) with 3 added and 3 removed across 0 owner(s). Includes a new dependency. No tests were added for the new code.
See full signal breakdown
  • files changed: 1
  • lines added/removed: 3 / 3
  • distinct owners touched: 0
  • schema migration touched: no
  • locked / sensitive path touched: no
  • adds new dependency: yes
  • bumps major dependency: no
  • tests added for new code: no
  • diff-minus-test ratio: 1.00
How is this calculated? The score is a transparent sum of weighted signals — see src/lib/pr-risk.ts computePrRiskScore.

Gate checks

All 6 checks passed
GateTestDisabled in settingsSkipped
Secret scanNo secrets detectedPassed
Security scanStatic rules ran (8 checks, no AI required) — no static findings. AI review unavailable: The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored.Skipped
Merge checkClean merge possiblePassed
CINo CI workflows are configured for this repositorySkipped
AI ReviewAIAI review approvedPassed
c comment · e edit title · m merge · a approve · r request changes · ? shortcuts