Add webchat-html-tests skill#5840
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new contributor-facing webchat-html-tests skill to streamline running and debugging the __tests__/html2 Selenium Grid HTML test harness, with operational steps moved into bundled scripts and longer troubleshooting guidance moved into reference docs.
Changes:
- Added the
webchat-html-testsskill entry and procedure doc. - Introduced helper scripts to start the Docker/Selenium Grid stack, wait for readiness, run HTML tests, and clean leaked sessions.
- Added reference docs describing the HTML test architecture and common failure/snapshot recovery workflows.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
.github/skills/webchat-html-tests/SKILL.md |
Skill definition and streamlined procedure for running/debugging HTML tests. |
.github/skills/webchat-html-tests/scripts/start-grid.sh |
Scripted Docker Compose build/up workflow for Selenium Grid + webchat2. |
.github/skills/webchat-html-tests/scripts/wait-for-grid.py |
Polls Selenium Grid status until ready. |
.github/skills/webchat-html-tests/scripts/run-html-tests.sh |
Wrapper for running Jest HTML tests (optionally scoped + snapshot update). |
.github/skills/webchat-html-tests/scripts/cleanup-grid-sessions.py |
Attempts to delete busy Selenium sessions to prevent grid stalls. |
.github/skills/webchat-html-tests/references/architecture.md |
Reference doc on test layout and regex targeting patterns. |
.github/skills/webchat-html-tests/references/failure-modes.md |
Reference doc on common failures and snapshot update workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
compulim
approved these changes
Jun 9, 2026
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.
Changelog Entry
Not required for this PR because it only adds contributor tooling.
Description
Adds a new
webchat-html-testsskill for Bot Framework Web Chat contributors working on the HTML Selenium Grid test harness. The skill is a renamed copy of the earlier run-and-debug workflow, but now splits the operational commands into bundled scripts and moves the long-form troubleshooting guidance into reference docs.Design
The skill body stays small for progressive loading, while the scripts and references carry the heavier operational details. The validation pass used the new scripts to start Docker, wait for Selenium Grid readiness, run a focused HTML test, and clean leaked sessions.
Specific Changes
Added the
webchat-html-testsskill with a more specific name and a keyword-rich description.Split Docker, Jest, and Selenium session management into bundled shell and Python scripts.
Moved test architecture notes and failure recovery guidance into separate reference docs.
Hardened the Selenium readiness script after a real startup
ConnectionResetErrorduring validation.I have added tests and executed them locally
I have updated
CHANGELOG.mdI have updated documentation
Review Checklist
z-index)package.jsonandpackage-lock.jsonreviewed