Closed
Conversation
- Adding Agent Plugin with Skills and Custom agent orchestrator to support React 18 and 19 migration. - Introduces comprehensive React 18 and React 19 migration plugins with specialized agents and skills for code auditing, dependency upgrades, source migration, and test suite modernization. - Targets legacy class-component-heavy and modern codebases, handling breaking API removals, automatic batching, context migration, string refs, and test framework transitions (including Enzyme to RTL). - Improves upgrade safety, repeatability, and code quality by providing memory checkpointing, reference migration patterns, and strict validation gates for zero deprecation warnings and test failures.
Contributor
🔍 Skill Validator Results127 resource(s) checked | ✅ All checks passed Full output
|
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR broadly updates the repository’s plugin catalog and content by adding many new agent/skill/reference files across multiple plugins, while also normalizing plugin manifest paths (notably removing trailing slashes and switching some manifests from explicit file paths to directory-level entries).
Changes:
- Added numerous new skills, agents, reference docs, and assets across several plugins (Azure, Arize, FlowStudio, database tooling, .NET/C#, etc.).
- Updated many
plugins/**/.github/plugin/plugin.jsonmanifests to normalize path formatting and (in several cases) point to./agentsinstead of enumerating files. - Updated the marketplace catalog and the skills README entry text.
Reviewed changes
Copilot reviewed 120 out of 455 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/flowstudio-power-automate/skills/flowstudio-power-automate-build/references/trigger-types.md | Adds trigger JSON templates documentation for Power Automate. |
| plugins/flowstudio-power-automate/skills/flowstudio-power-automate-build/references/flow-schema.md | Documents expected flow definition schema and conventions. |
| plugins/flowstudio-power-automate/skills/flowstudio-power-automate-build/references/build-patterns.md | Adds sample end-to-end flow definition patterns. |
| plugins/flowstudio-power-automate/.github/plugin/plugin.json | Normalizes skill path entries (removes trailing slashes). |
| plugins/fastah-ip-geo-tools/skills/geofeed-tuner/references/snippets-python3.md | Adds Python ipaddress reference snippets and validation guidance. |
| plugins/fastah-ip-geo-tools/skills/geofeed-tuner/assets/small-territories.json | Adds a JSON list of territory/country codes used by the skill. |
| plugins/fastah-ip-geo-tools/skills/geofeed-tuner/assets/example/01-user-input-rfc8805-feed.csv | Adds example RFC8805 geofeed CSV input. |
| plugins/fastah-ip-geo-tools/.github/plugin/plugin.json | Normalizes skill path entry (removes trailing slash). |
| plugins/edge-ai-tasks/.github/plugin/plugin.json | Changes agents entry from explicit files to ./agents. |
| plugins/doublecheck/skills/doublecheck/assets/verification-report-template.md | Adds a structured verification report template asset. |
| plugins/doublecheck/agents/doublecheck.md | Adds a custom verification agent definition. |
| plugins/doublecheck/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills path. |
| plugins/devops-oncall/skills/multi-stage-dockerfile/SKILL.md | Adds a container best-practices skill. |
| plugins/devops-oncall/agents/azure-principal-architect.md | Adds an Azure principal architect agent definition. |
| plugins/devops-oncall/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/dataverse-sdk-for-python/skills/dataverse-python-usecase-builder/SKILL.md | Adds a Dataverse Python SDK use-case builder skill. |
| plugins/dataverse-sdk-for-python/skills/dataverse-python-quickstart/SKILL.md | Adds a Dataverse Python SDK quickstart skill. |
| plugins/dataverse-sdk-for-python/skills/dataverse-python-production-code/SKILL.md | Adds a Dataverse Python production-code skill. |
| plugins/dataverse-sdk-for-python/skills/dataverse-python-advanced-patterns/SKILL.md | Adds a Dataverse Python advanced patterns skill. |
| plugins/dataverse-sdk-for-python/.github/plugin/plugin.json | Normalizes skills paths for the plugin. |
| plugins/database-data-management/skills/postgresql-code-review/SKILL.md | Adds a PostgreSQL-focused code review skill. |
| plugins/database-data-management/agents/postgresql-dba.md | Adds a PostgreSQL DBA agent definition. |
| plugins/database-data-management/agents/ms-sql-dba.md | Adds an MS SQL DBA agent definition. |
| plugins/database-data-management/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/csharp-mcp-development/skills/csharp-mcp-server-generator/SKILL.md | Adds a skill for generating C# MCP server projects. |
| plugins/csharp-mcp-development/agents/csharp-mcp-expert.md | Adds a C# MCP expert agent. |
| plugins/csharp-mcp-development/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/csharp-dotnet-development/skills/dotnet-upgrade/SKILL.md | Adds a skill file with .NET upgrade prompts/checklists. |
| plugins/csharp-dotnet-development/skills/dotnet-best-practices/SKILL.md | Adds a .NET/C# best practices skill. |
| plugins/csharp-dotnet-development/skills/csharp-xunit/SKILL.md | Adds xUnit testing best practices skill. |
| plugins/csharp-dotnet-development/skills/csharp-tunit/SKILL.md | Adds TUnit testing best practices skill. |
| plugins/csharp-dotnet-development/skills/csharp-nunit/SKILL.md | Adds NUnit testing best practices skill. |
| plugins/csharp-dotnet-development/skills/csharp-async/SKILL.md | Adds async/await best practices skill. |
| plugins/csharp-dotnet-development/skills/aspnet-minimal-api-openapi/SKILL.md | Adds minimal API + OpenAPI skill guidance. |
| plugins/csharp-dotnet-development/agents/expert-dotnet-software-engineer.md | Adds an expert .NET software engineer agent definition. |
| plugins/csharp-dotnet-development/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/copilot-sdk/.github/plugin/plugin.json | Normalizes skills path entry (removes trailing slash). |
| plugins/context-matic/skills/integrate-context-matic/SKILL.md | Adds a workflow for integrating third-party APIs via context-matic. |
| plugins/context-matic/.github/plugin/plugin.json | Normalizes skills paths (removes trailing slashes). |
| plugins/context-engineering/skills/what-context-needed/SKILL.md | Adds a skill for asking what files are needed before answering. |
| plugins/context-engineering/skills/refactor-plan/SKILL.md | Adds a skill for multi-file refactor planning. |
| plugins/context-engineering/skills/context-map/SKILL.md | Adds a skill for generating a context map before changes. |
| plugins/context-engineering/agents/context-architect.md | Adds a context-architect agent definition. |
| plugins/context-engineering/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/clojure-interactive-programming/skills/remember-interactive-programming/SKILL.md | Adds a micro-skill reminder for REPL-first workflows. |
| plugins/clojure-interactive-programming/agents/clojure-interactive-programming.md | Adds a Clojure REPL-first agent definition. |
| plugins/clojure-interactive-programming/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/cast-imaging/agents/cast-imaging-structural-quality-advisor.md | Adds CAST Imaging agent definition (quality advisor). |
| plugins/cast-imaging/agents/cast-imaging-software-discovery.md | Adds CAST Imaging agent definition (software discovery). |
| plugins/cast-imaging/agents/cast-imaging-impact-analysis.md | Adds CAST Imaging agent definition (impact analysis). |
| plugins/cast-imaging/.github/plugin/plugin.json | Changes agents entry to ./agents. |
| plugins/azure-cloud-development/skills/azure-pricing/references/SERVICE-NAMES.md | Adds Azure Retail Prices API service-name reference. |
| plugins/azure-cloud-development/skills/azure-pricing/references/REGIONS.md | Adds Azure Retail Prices API region-name reference. |
| plugins/azure-cloud-development/skills/azure-pricing/references/COST-ESTIMATOR.md | Adds cost estimation formula reference. |
| plugins/azure-cloud-development/skills/azure-pricing/references/COPILOT-STUDIO-RATES.md | Adds Copilot Studio billing reference snapshot and formulas. |
| plugins/azure-cloud-development/agents/terraform-azure-planning.md | Adds Terraform planning agent definition. |
| plugins/azure-cloud-development/agents/terraform-azure-implement.md | Adds Terraform implementation specialist agent definition. |
| plugins/azure-cloud-development/agents/azure-verified-modules-terraform.md | Adds AVM Terraform agent definition and required checks guidance. |
| plugins/azure-cloud-development/agents/azure-verified-modules-bicep.md | Adds AVM Bicep agent definition. |
| plugins/azure-cloud-development/agents/azure-saas-architect.md | Adds Azure SaaS architect agent definition. |
| plugins/azure-cloud-development/agents/azure-principal-architect.md | Adds Azure principal architect agent definition. |
| plugins/azure-cloud-development/agents/azure-logic-apps-expert.md | Adds Azure Logic Apps expert agent definition. |
| plugins/azure-cloud-development/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/awesome-copilot/skills/suggest-awesome-github-copilot-skills/SKILL.md | Adds skill for suggesting/installing skills from awesome-copilot. |
| plugins/awesome-copilot/skills/suggest-awesome-github-copilot-instructions/SKILL.md | Adds skill for suggesting/installing instruction files from awesome-copilot. |
| plugins/awesome-copilot/skills/suggest-awesome-github-copilot-agents/SKILL.md | Adds skill for suggesting/installing agents from awesome-copilot. |
| plugins/awesome-copilot/agents/meta-agentic-project-scaffold.md | Adds a meta agent for pulling content from awesome-copilot. |
| plugins/awesome-copilot/.github/plugin/plugin.json | Changes agents entry to ./agents and normalizes skills paths. |
| plugins/automate-this/.github/plugin/plugin.json | Normalizes skills path entry (removes trailing slash). |
| plugins/arize-ax/skills/arize-trace/references/ax-setup.md | Adds ax CLI troubleshooting reference. |
| plugins/arize-ax/skills/arize-trace/references/ax-profiles.md | Adds ax profile setup reference. |
| plugins/arize-ax/skills/arize-prompt-optimization/references/ax-setup.md | Adds ax CLI troubleshooting reference (prompt optimization). |
| plugins/arize-ax/skills/arize-prompt-optimization/references/ax-profiles.md | Adds ax profile setup reference (prompt optimization). |
| plugins/arize-ax/skills/arize-link/references/EXAMPLES.md | Adds Arize deep-link URL examples. |
| plugins/arize-ax/skills/arize-link/SKILL.md | Adds Arize deep-link generation skill. |
| plugins/arize-ax/skills/arize-instrumentation/references/ax-profiles.md | Adds ax profile setup reference (instrumentation). |
| plugins/arize-ax/skills/arize-experiment/references/ax-setup.md | Adds ax CLI troubleshooting reference (experiments). |
| plugins/arize-ax/skills/arize-experiment/references/ax-profiles.md | Adds ax profile setup reference (experiments). |
| plugins/arize-ax/skills/arize-evaluator/references/ax-setup.md | Adds ax CLI troubleshooting reference (evaluator). |
| plugins/arize-ax/skills/arize-evaluator/references/ax-profiles.md | Adds ax profile setup reference (evaluator). |
| plugins/arize-ax/skills/arize-dataset/references/ax-setup.md | Adds ax CLI troubleshooting reference (dataset). |
| plugins/arize-ax/skills/arize-dataset/references/ax-profiles.md | Adds ax profile setup reference (dataset). |
| plugins/arize-ax/skills/arize-annotation/references/ax-setup.md | Adds ax CLI troubleshooting reference (annotation). |
| plugins/arize-ax/skills/arize-annotation/references/ax-profiles.md | Adds ax profile setup reference (annotation). |
| plugins/arize-ax/skills/arize-annotation/SKILL.md | Adds annotation config + span annotation skill. |
| plugins/arize-ax/skills/arize-ai-provider-integration/references/ax-setup.md | Adds ax CLI troubleshooting reference (provider integration). |
| plugins/arize-ax/skills/arize-ai-provider-integration/references/ax-profiles.md | Adds ax profile setup reference (provider integration). |
| plugins/arize-ax/.github/plugin/plugin.json | Normalizes skills paths across Arize skills. |
| docs/README.skills.md | Updates one skills description line. |
| .github/plugin/marketplace.json | Adds marketplace entries for react18-upgrade/react19-upgrade. |
Comment on lines
20
to
28
| "agents": [ | ||
| "./agents/azure-principal-architect.md", | ||
| "./agents/azure-saas-architect.md", | ||
| "./agents/azure-logic-apps-expert.md", | ||
| "./agents/azure-verified-modules-bicep.md", | ||
| "./agents/azure-verified-modules-terraform.md", | ||
| "./agents/terraform-azure-planning.md", | ||
| "./agents/terraform-azure-implement.md" | ||
| "./agents" | ||
| ], | ||
| "skills": [ | ||
| "./skills/azure-resource-health-diagnose/", | ||
| "./skills/az-cost-optimize/", | ||
| "./skills/import-infrastructure-as-code/", | ||
| "./skills/azure-pricing/" | ||
| "./skills/azure-resource-health-diagnose", | ||
| "./skills/az-cost-optimize", | ||
| "./skills/import-infrastructure-as-code", | ||
| "./skills/azure-pricing" | ||
| ] |
There was a problem hiding this comment.
Switching agents from explicit file paths (e.g., ./agents/foo.md) to a directory path (./agents) is a behavioral change that can break plugin discovery if the loader expects agent file entries. If directory scanning is supported, consider documenting that expectation; if it’s not supported, revert to listing individual agent markdown files under agents.
| "AD", | ||
| "AG", | ||
| "AI", | ||
| "AN", |
There was a problem hiding this comment.
This list includes ISO 3166-1 alpha-2 codes that are no longer assigned (e.g., AN for Netherlands Antilles and CS for Serbia and Montenegro). If this file is used for validation or matching, these values can cause incorrect accept/reject behavior. Update the list to current codes (e.g., replace AN with its successors like CW, SX, BQ if that’s the intent, and replace CS with RS/ME), or add an explicit note/tooling guard if legacy codes are intentionally supported.
| "BZ", | ||
| "CC", | ||
| "CK", | ||
| "CS", |
There was a problem hiding this comment.
This list includes ISO 3166-1 alpha-2 codes that are no longer assigned (e.g., AN for Netherlands Antilles and CS for Serbia and Montenegro). If this file is used for validation or matching, these values can cause incorrect accept/reject behavior. Update the list to current codes (e.g., replace AN with its successors like CW, SX, BQ if that’s the intent, and replace CS with RS/ME), or add an explicit note/tooling guard if legacy codes are intentionally supported.
| 2605:59c8:2700::/40,CA,CA-QC,"Montreal" | ||
| 150.228.170.0/24,SG,SG-01,Singapore, | ||
| # It's OK for small city states to leave state ISO2 code unspecified | ||
| 2406:2d40:8100::/42,SG,SG,Singapore, No newline at end of file |
There was a problem hiding this comment.
Line 4 states the region/state code can be left unspecified, but line 5 sets the region column to SG (the country code) instead of leaving it blank. This makes the example ambiguous and may train users to provide invalid ISO 3166-2 values. Consider changing line 5 to leave the region field empty (e.g., ...,SG,,Singapore,).
Suggested change
| 2406:2d40:8100::/42,SG,SG,Singapore, | |
| 2406:2d40:8100::/42,SG,,Singapore, |
Comment on lines
+6
to
+8
| # Project Discovery & Assessment | ||
| - name: "Project Classification Analysis" | ||
| prompt: "Identify all projects in the solution and classify them by type (`.NET Framework`, `.NET Core`, `.NET Standard`). Analyze each `.csproj` for its current `TargetFramework` and SDK usage." |
There was a problem hiding this comment.
The body content here is formatted like YAML (- name: ..., prompt: ...) but is not in a fenced code block and may be confusing to readers (and to any tooling that expects normal markdown instructions). Consider either (a) converting this into standard markdown sections/bullets, or (b) wrapping the structured prompt list in a fenced block and explicitly labeling it (e.g., as YAML) so the intended structure is unambiguous.
| --- | ||
| description: "Work with Microsoft SQL Server databases using the MS SQL extension." | ||
| name: "MS-SQL Database Administrator" | ||
| tools: ["search/codebase", "edit/editFiles", "githubRepo", "extensions", "runCommands", "database", "mssql_connect", "mssql_query", "mssql_listServers", "mssql_listDatabases", "mssql_disconnect", "mssql_visualizeSchema"] |
There was a problem hiding this comment.
The tools list includes search/codebase, while other agents in this PR use codebase and search as separate tool names. If search/codebase isn’t a valid tool identifier in this environment, the agent will fail at runtime. Recommend standardizing to the known tool naming used elsewhere in this repo (e.g., codebase + search) or confirming the exact tool name expected by the agent runtime.
Suggested change
| tools: ["search/codebase", "edit/editFiles", "githubRepo", "extensions", "runCommands", "database", "mssql_connect", "mssql_query", "mssql_listServers", "mssql_listDatabases", "mssql_disconnect", "mssql_visualizeSchema"] | |
| tools: ["search", "codebase", "edit/editFiles", "githubRepo", "extensions", "runCommands", "database", "mssql_connect", "mssql_query", "mssql_listServers", "mssql_listDatabases", "mssql_disconnect", "mssql_visualizeSchema"] |
Comment on lines
+456
to
+467
| { | ||
| "name": "react18-upgrade", | ||
| "source": "react18-upgrade", | ||
| "description": "Enterprise React 18 migration toolkit with specialized agents and skills for upgrading React 16/17 class-component codebases to React 18.3.1. Includes auditor, dependency surgeon, class component migration specialist, automatic batching fixer, and test guardian.", | ||
| "version": "1.0.0" | ||
| }, | ||
| { | ||
| "name": "react19-upgrade", | ||
| "source": "react19-upgrade", | ||
| "description": "Enterprise React 19 migration toolkit with specialized agents and skills for upgrading React 18 codebases to React 19. Includes auditor, dependency surgeon, source code migrator, and test guardian. Handles removal of deprecated APIs including ReactDOM.render, forwardRef, defaultProps, legacy context, string refs, and more.", | ||
| "version": "1.0.0" | ||
| }, |
There was a problem hiding this comment.
The PR title suggests a React upgrade-focused change, but the diff includes wide-ranging additions/manifest changes across many unrelated plugins plus marketplace additions. Please update the PR description to clearly state the actual scope (e.g., plugin manifest normalization + multiple new agents/skills + marketplace entries), so reviewers and release notes can accurately reflect what’s being shipped.
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.
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.stagedbranch for this pull request.Description
Type of Contribution
Additional Notes
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.