Add validation scripts for all sample apps#248
Open
dluces wants to merge 2 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a consistent, root-level PowerShell validation entrypoint (validate-sample.ps1) across sample apps, backed by shared tooling to perform local builds and lightweight runtime/browser smoke checks.
Changes:
- Added shared Playwright-based browser smoke tooling under
Tools/sample-validation(Node + Playwright). - Added shared PowerShell helper functions (
Tools/powershell/SampleValidation.ps1) and per-samplevalidate-sample.ps1scripts to standardize validation flows. - Documented the new validation approach in
README.mdand ignored generated validation logs via.gitignore.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| Tools/sample-validation/package.json | Introduces Node tooling dependency for shared browser smoke checks. |
| Tools/sample-validation/package-lock.json | Locks Playwright dependency tree for the shared validation tooling. |
| Tools/sample-validation/browser-smoke.mjs | Implements the Playwright-driven browser smoke test runner. |
| Tools/powershell/SampleValidation.ps1 | Adds shared PowerShell helpers for running commands, managing processes, and HTTP/browser smoke checks. |
| Custom Apps/webhook/validate-sample.ps1 | Adds validation script for the webhook sample (runtime echo check). |
| Custom Apps/project-management/validate-sample.ps1 | Adds validation script for the project-management sample (build + optional browser smoke). |
| Custom Apps/legal-docs/validate-sample.ps1 | Adds validation script for the legal-docs sample (build + optional browser smoke). |
| Custom Apps/boilerplate-typescript-react/validate-sample.ps1 | Adds validation script for the TypeScript React boilerplate (build + optional runtime + optional browser smoke). |
| Custom Apps/boilerplate-react-azurefunction/validate-sample.ps1 | Adds validation script for the React + Azure Functions boilerplate (build + optional runtime + optional browser smoke). |
| Custom Apps/boilerplate-aspnet-webservice/validate-sample.ps1 | Adds validation script for the ASP.NET web service sample (build + optional runtime smoke). |
| AI/ocr/validate-sample.ps1 | Adds validation script for OCR sample (build/test + optional runtime + optional browser smoke). |
| AI/mcp-server/validate-sample.ps1 | Adds validation script for MCP server sample (build + optional runtime health check). |
| README.md | Documents how to run sample validation scripts and their downgrade behavior. |
| .gitignore | Ignores generated local .validation log folders. |
Files not reviewed (1)
- Tools/sample-validation/package-lock.json: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Validation scripts for all sample apps to help ensure dependency auto-upgrades don't break them