Skip to content

build(scripts): target PowerShell 7.4 and bootstrap python test deps#1842

Open
WilliamBerryiii wants to merge 1 commit into
feat/1637-d-skill-pathsfrom
feat/1637-e-ps74-pytest
Open

build(scripts): target PowerShell 7.4 and bootstrap python test deps#1842
WilliamBerryiii wants to merge 1 commit into
feat/1637-d-skill-pathsfrom
feat/1637-e-ps74-pytest

Conversation

@WilliamBerryiii

Copy link
Copy Markdown
Member

Pull Request

Description

Targeted PowerShell 7.4 and bootstrapped the Python test dependencies for local and CI runs. Updated PSScriptAnalyzer.psd1 to the 7.4 target and extended Invoke-PythonTests.ps1 to install the required test dependencies before execution.

Related Issue(s)

Closes #1827

Type of Change

Select all that apply:

Code & Documentation:

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Infrastructure & Configuration:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

AI Artifacts:

  • Reviewed contribution with prompt-builder agent and addressed all feedback
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot agent (.github/agents/*.agent.md)
  • Copilot skill (.github/skills/*/SKILL.md)

Other:

  • Script/automation (.ps1, .sh, .py)
  • Other (please describe):

Testing

Validated via npm run lint:all (exit 0), including npm run lint:ps. The updated Invoke-PythonTests.ps1 was exercised to confirm dependency bootstrapping precedes test execution.

Checklist

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)
  • Tests added for new functionality (if applicable)

AI Artifact Contributions

  • Used /prompt-analyze to review contribution
  • Addressed all feedback from prompt-builder review
  • Verified contribution follows common standards and type-specific requirements

Required Automated Checks

  • Markdown linting: npm run lint:md
  • Spell checking: npm run spell-check
  • Frontmatter validation: npm run lint:frontmatter
  • Skill structure validation: npm run validate:skills
  • Link validation: npm run lint:md-links
  • PowerShell analysis: npm run lint:ps
  • Plugin freshness: npm run plugin:generate
  • Docusaurus tests: npm run docs:test

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues
  • Security-related scripts follow the principle of least privilege

Additional Notes

Fifteenth and final PR in the #1637 stack. Base branch: feat/1637-d-skill-paths. Tooling configuration only; no functional code paths change.

- set PSUseCompatibleSyntax TargetVersions to 7.4
- run uv sync for dev and fuzz groups before python test discovery

🔧 - Generated by Copilot
@WilliamBerryiii WilliamBerryiii requested a review from a team as a code owner June 2, 2026 03:43
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.

3 participants