'Oracle-to-PostgreSQL Migration Expert' Custom Agent: Update Custom Agent & Plugin & Skills - #2566
Open
PrimedPaul wants to merge 14 commits into
Open
Conversation
- Update migration agent guidelines to prioritize extension tool usage for code migration. - Refine migration phases with detailed steps for pre-migration review and schema migration. - Add new reviewing skill references for PostgreSQL materialized view refresh and UNION ALL planner risks. - Ensure consistency in collation handling and testing strategies across skills.
- Add exit criteria to each migration phase - Remove invocation of `ms-ossdata.vscode-pgsql` extension due to dependency on VSCode - Update README.md to reflect changes in migration phases and exit criteria - Update broken reference to custom agent in plugin.json
…ements - Added new skill for migrating .NET data access code from Oracle to PostgreSQL. - Updated README to include new migration skill. - Improved existing skills with clearer file naming conventions and migration actions. - Added reference documents for handling Oracle-specific functions and pagination. - Created detailed guides for NVL, DECODE, ROWNUM, SYSDATE, and DUAL replacements.
Both Oracle test skills were ambiguously worded in ways that could cause a model to invoke them during Phase 6 (PostgreSQL test migration) instead of using them exclusively in Phase 3. Key changes: - Rewrite descriptions to explicitly state Phase 3-only usage and warn against invoking during Phase 6 - Replace "scaffold for Oracle first" with "Oracle only" to remove the false implication of a second PostgreSQL scaffolding step - Replace "Tests validate behavior consistency when running against Oracle or PostgreSQL" with clear Oracle-only framing - Rename "DB-agnostic assertions" → "Assertion portability" and explain the why (survive Phase 6 migration without rewrites) - Fix datetime bullet in integration tests skill to use generic Oracle column precision language instead of PostgreSQL type syntax - Name Oracle NuGet package explicitly (Oracle.ManagedDataAccess.Core)
…to Phase 2 risk analysis
- Phase 1 success criteria now requires Oracle DDL artifacts to be
confirmed present at the recorded location before proceeding.
If missing, the agent stops and prompts the user to provide them.
- Phase 2 risk analysis now explicitly scans DDL/Oracle/{ProjectName}/
as supplemental context, summarising procedure complexity indicators
(dynamic SQL, DBMS_* / UTL_* references, autonomous transactions,
pipelined functions, BULK COLLECT/FORALL, REF CURSOR, TYPE bodies)
rather than ingesting DDL files wholesale. This ensures schema-level
migration risk is captured even when it isn't visible in application
code alone.
PrimedPaul
marked this pull request as draft
August 6, 2026 18:04
PrimedPaul
marked this pull request as ready for review
August 6, 2026 18:06
PrimedPaul
marked this pull request as draft
August 6, 2026 18:08
- Update custom agent plugin (eg resolve conflict and add new skill) - Validate skills - Run build
PrimedPaul
force-pushed
the
update-custom-agent/oracle-to-postgres-migration-expert
branch
from
August 6, 2026 18:15
3db8792 to
fa41b4b
Compare
PrimedPaul
marked this pull request as ready for review
August 6, 2026 18:16
aaronpowell
requested changes
Aug 7, 2026
Contributor
🔒 PR Risk Scan ResultsScanned 14 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
Contributor
🔍 Vally Lint Results
Summary
Full linter output |
PrimedPaul
marked this pull request as draft
August 7, 2026 12:01
…' of https://github.com/PrimedPaul/awesome-copilot into update-custom-agent/oracle-to-postgres-migration-expert
PrimedPaul
marked this pull request as ready for review
August 7, 2026 12:02
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.mainbranch for this pull request.Description
This iteration brings many benefits to the custom agent plugin resulting in better alignment with custom agent's goal, which is to guide and advise:
ms-ossdata.vscode-pgsqlextension because it disrupted the migration process flow and it also creates a dependency on VSCode.Type of Contribution
Additional Notes
For your information, I will continue looking for improvements on this custom agent as my work continues on enterprise legacy application migration effort. I am honestly impressed how much more helpful it has become and continue to strive to make it better during my work on the migration project.
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.