feat: add coreex-db-migration skill#152
Merged
Merged
Conversation
Adds the coreex-db-migration L1 skill covering all database schema changes for a CoreEx domain: - New transactional table (script create) - New reference-data table (script refdata) - Alter existing table (script alter) — transactional and ref-data - Non-entity schema changes (bare script with renamed suffix) Skill includes: - SKILL.md with quick-reference command table and polyglot note - references/workflow.md with 6-phase decision tree, SQL column templates (SQL Server + PostgreSQL, aggregate + ref-data), dbex.yaml registration guidance, and guardrails - 255-character filename limit documented for bare scripts - Links to coreex-tooling.instructions.md and canonical sample Database projects Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…n convention - Add dotnet run -- dropandall --accept-prompts to Quick Reference and Guardrails; documents that --accept-prompts bypasses DbEx DROP prompt and must only be used after explicit user consent - Add JSON column convention: column names ending in Json/json default to NVARCHAR(MAX) (SQL Server) or jsonb (PostgreSQL); documents the NoSQL-within-SQL intent, .NET class/record type expectation, and note that value converter wiring is a hand-written concern Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…plate - Add .github/prompts/coreex-db-migration.prompt.md (VS Code / VS / GitHub.com / Claude-as-Copilot bridge) - Update CoreEx.Template.csproj CopyTemplateAiContext target: - Add prompt wrapper to _AiFile ItemGroup (CoreEx.Bootstrap) - Add skill SKILL.md + references/workflow.md to _AiFile ItemGroup (CoreEx.Bootstrap) - Add dedicated Copy block for CoreEx.Ai (prompt + skill files) Each future skill follows the same one-block-per-skill pattern in the csproj. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the first atomic L1 skill in the CoreEx skills inventory by introducing a reusable, end-to-end database migration workflow (prompt wrapper + skill + detailed workflow reference) and wiring it into the CoreEx.Template AI-context packaging so consuming projects receive it.
Changes:
- Introduces the new
coreex-db-migrationskill (SKILL.md) plus a detailed procedural workflow reference (references/workflow.md). - Adds a thin prompt wrapper (
coreex-db-migration.prompt.md) to enable/coreex-db-migrationacross Copilot surfaces while deferring to the skill as source-of-truth. - Updates
CoreEx.Template.csprojto package/copy the new prompt + skill assets into bothCoreEx.BootstrapandCoreEx.Ai.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/CoreEx.Template/CoreEx.Template.csproj |
Packages the new prompt + skill assets into generated AI context (Bootstrap + Ai packs). |
.github/skills/coreex-db-migration/SKILL.md |
Defines the new L1 skill entry point with scope boundaries and a quick command reference. |
.github/skills/coreex-db-migration/references/workflow.md |
Provides the detailed 6-phase DB migration workflow, decision tree, and guardrails. |
.github/prompts/coreex-db-migration.prompt.md |
Adds the prompt wrapper that routes users to the skill/workflow as the authoritative contract. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Eric Sibly [chullybun] <eric@thesiblys.com>
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.
Closes #151 (partial — first skill in the inventory)
Summary
Adds the
coreex-db-migrationskill — the first atomic L1 skill in the CoreEx skills inventory. Encodes the full database schema change workflow that was previously scattered acrosscoreex-tooling.instructions.mdas prose and "Agent instruction:" blocks.What's included
.github/skills/coreex-db-migration/SKILL.md— lean entry point (<70 lines); frontmatter, when to use/not use, quick-reference command table, polyglot note, workflow pointerreferences/workflow.md— full 6-phase procedural workflow:NVARCHAR(MAX)/jsonb)dbex.yamldotnet run -- all(ordropandall --accept-promptsfor recovery)*.g.*files, etc.).github/prompts/coreex-db-migration.prompt.mdThin bridge wrapper enabling
/coreex-db-migrationin VS Code Copilot Chat, Visual Studio, and GitHub.com. Points toSKILL.mdas authoritative source; the skill always wins on conflict.src/CoreEx.Template/CoreEx.Template.csprojWires the new skill and prompt wrapper into the
CopyTemplateAiContextpack target:_AiFileItemGroup →CoreEx.Bootstrapreceives the full setCopyblock →CoreEx.Aialso receives skill + prompt wrapperThe one-block-per-skill pattern establishes the convention for all future skills.
Compatibility
SKILL.mdloaded at session startSKILL.mdformatcoreex-db-migration.prompt.mdwrapperKey conventions encoded
script create/script refdata/script alter/ barescriptcommand matrixdotnet run -- dropandall --accept-promptsfor DB recovery (user must initiate)Jsonsuffix →NVARCHAR(MAX)/jsonb; .NET type is class/record).Migrations.prefix counts)