Uh oh!
There was an error while loading. Please reload this page.
Add WinForms MCP and skills - #869
Conversation
jasonleenaylor
commented
May 8, 2026
Was removing this intentional? It seemed this would produce less churn in the repo with it in. |
johnml1135
commented
May 11, 2026
Good catch. That removal wasn't intentional. I restored the BOM-preserving behavior in |
70d6808 to
29162e5Compare
This comment has been minimized.
This comment has been minimized.
b721182 to
3aa324aComparefe6f187 to
2488da1Comparejohnml1135
commented
May 12, 2026
Follow-up after the PR split cleanup: Build/Agent/fix-whitespace.ps1 is no longer part of #869. This PR now contains only the 12 UI automation support files. The BOM-preserving whitespace-fixer change belongs with the base cleanup layer instead, so if you want to review that script change specifically, please look at the base branch/PR rather than this support-only PR. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (1)
.github/skills/fieldworks-winapp/SKILL.md:1
- The skill name
fieldworks-winappsuggests it is WinApp-only, but the skill is explicitly dual-driver (WinForms MCP + WinApp fallback). Consider renaming the skill to reflect its scope (e.g.,fieldworks-ui-automationorfieldworks-winforms-winapp) to avoid incorrect discovery/selection by authors and agents.
---
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
2488da1 to
daf921aCompare3aa324a to
bc7e8b1Comparejasonleenaylor
commented
May 13, 2026
I'm confused about this file. |
jasonleenaylor
left a comment
There was a problem hiding this comment.
@jasonleenaylor reviewed 19 files and all commit messages, and made 1 comment.
Reviewable status: 19 of 208 files reviewed, 2 unresolved discussions.
.github/skills/code-review-skill-main/SKILL.md line 1 at r6 (raw file):
---
I have questions about this skill and how/if we should integrate it. It doesn't seem closely related to the automation support yet (still reviewing)
ff11e68 to
13cdd9cCompare623fb0e to
45ed34aComparejohnml1135
commented
May 22, 2026
Previously, jasonleenaylor (Jason Naylor) wrote…
Moved to the review skills PR. |
johnml1135
commented
May 22, 2026
Previously, jasonleenaylor (Jason Naylor) wrote…
Moved to the review skills PR. |
45ed34a to
ff471a9Comparejohnml1135
commented
Jun 4, 2026
@jasonleenaylor - what else needs to be done for this one? |
jasonleenaylor
left a comment
There was a problem hiding this comment.
@jasonleenaylor reviewed 207 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on johnml1135).
Uh oh!
There was an error while loading. Please reload this page.
Summary
This PR is now scoped just to WinForms MCP and the supporting automation/screenshot skills needed for FieldWorks live UI automation work.
Included
.github/skills/fieldworks-winapp..github/skills/smart-screenshot-capture.winforms-mcpto.vscode/mcp.jsonwith headlessnet48settings.Docs/mcp.md.Excluded
.agents/skills, which remains separate from this PR.Verification
git diff --name-status cleanup/non-render-pr-split...HEADgit diff --check cleanup/non-render-pr-split...HEADpython -m json.tool .vscode/mcp.jsongit grep -n "LT-22324" -- .github/skills/fieldworks-winappreturned no matchesThis change is