feat(scripts): add structured JSON log output to Validate-Marketplace.ps1

[MauroDruwel]

Pull Request

Description

Add structured JSON log output support to Validate-Marketplace.ps1 so marketplace validation results are persisted to logs/ in a machine-readable format, consistent with other linting scripts.

This update now also includes review-requested help documentation improvements:

• script-level .EXAMPLE shows -OutputPath usage
• Write-MarketplaceValidationReport now includes .OUTPUTS and .EXAMPLE

Related Issue(s)

Closes #991

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

• npm run lint:ps
• npm run test:ps -- -TestPath "scripts/tests/plugins/"

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

The following validation commands must pass before merging:

• 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

JSON output is additive. Existing console output and failure behavior remain unchanged.

[katriendg]

Thanks for this contribution, @MauroDruwel! The implementation is looking good.

Before merge, please update the PR description to follow the PR template. You can generate one automatically by checking out your branch locally and running the /pull-request slash command in Copilot Chat, agent mode:

/pull-request issue https://github.com/microsoft/hve-core/issues/991

This will generate the correctly formatted description with the Type of Change checklist, Testing section, Required Automated Checks, and Security Considerations sections.
Then copy the output of the pr.md file into this PR manually, since the PR already exists.

Additional note: also update your other PRs with this approach?

Optional improvements:

1. Update the script-level .EXAMPLE in the comment-based help to show the new -OutputPath parameter usage.
2. Add .OUTPUTS and .EXAMPLE to the Write-MarketplaceValidationReport function's comment-based help, consistent with the other helper functions in the file.

[MauroDruwel]

Thanks @katriendg — updated.

I applied the requested changes:

1. PR description now follows .github/PULL_REQUEST_TEMPLATE.md sections and checklists.
2. Script-level .EXAMPLE now shows -OutputPath usage.
3. Added .OUTPUTS and .EXAMPLE to Write-MarketplaceValidationReport comment-based help.

Latest commit: 97491a3.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 98.30508% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.43%. Comparing base (d51b4d3) to head (ac1b3c6).

Files with missing lines	Patch %	Lines
scripts/plugins/Validate-Marketplace.ps1	98.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1430      +/-   ##
==========================================
- Coverage   87.64%   87.43%   -0.22%     
==========================================
  Files          65       64       -1     
  Lines       10152     9930     -222     
==========================================
- Hits         8898     8682     -216     
+ Misses       1254     1248       -6     

Flag	Coverage Δ
pester	85.12% <98.30%> (+0.08%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
scripts/plugins/Validate-Marketplace.ps1	92.59% <98.30%> (+3.82%)	⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:

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

[katriendg]

Thank you Mauro for the fast update, LGTM!