From 54820797faf0d3570db579dc1f6e9d3f8c8e07cf Mon Sep 17 00:00:00 2001 From: Julian lechner Date: Fri, 11 Sep 2026 14:42:47 +0200 Subject: [PATCH] feat(skills): add shared task workflows --- shared/skills/debugging/SKILL.md | 17 +++++++++++++++++ shared/skills/git/answer-pull-request/SKILL.md | 10 ++++++++++ shared/skills/git/review-pull-request/SKILL.md | 10 ++++++++++ .../integration-test-write/SKILL.md | 10 ++++++++++ shared/skills/implementation/sql-write/SKILL.md | 12 ++++++++++++ .../implementation/unit-test-write/SKILL.md | 10 ++++++++++ .../planning/change-impact-analysis/SKILL.md | 12 ++++++++++++ .../planning/implementation-planner/SKILL.md | 12 ++++++++++++ .../planning/work-item-definition/SKILL.md | 12 ++++++++++++ .../research/repository-research/SKILL.md | 12 ++++++++++++ .../skills/research/technical-research/SKILL.md | 12 ++++++++++++ shared/skills/reviews/api-review/SKILL.md | 10 ++++++++++ shared/skills/reviews/business-review/SKILL.md | 10 ++++++++++ shared/skills/reviews/code-review/SKILL.md | 12 ++++++++++++ shared/skills/reviews/efcore-review/SKILL.md | 10 ++++++++++ .../skills/reviews/performance-review/SKILL.md | 10 ++++++++++ shared/skills/reviews/security-review/SKILL.md | 10 ++++++++++ shared/skills/reviews/ui-ux-review/SKILL.md | 10 ++++++++++ shared/skills/reviews/wpf-review/SKILL.md | 10 ++++++++++ .../verification/facts-verification/SKILL.md | 10 ++++++++++ .../regression-verification/SKILL.md | 10 ++++++++++ .../work-item-verification/SKILL.md | 10 ++++++++++ 22 files changed, 241 insertions(+) create mode 100644 shared/skills/debugging/SKILL.md create mode 100644 shared/skills/git/answer-pull-request/SKILL.md create mode 100644 shared/skills/git/review-pull-request/SKILL.md create mode 100644 shared/skills/implementation/integration-test-write/SKILL.md create mode 100644 shared/skills/implementation/sql-write/SKILL.md create mode 100644 shared/skills/implementation/unit-test-write/SKILL.md create mode 100644 shared/skills/planning/change-impact-analysis/SKILL.md create mode 100644 shared/skills/planning/implementation-planner/SKILL.md create mode 100644 shared/skills/planning/work-item-definition/SKILL.md create mode 100644 shared/skills/research/repository-research/SKILL.md create mode 100644 shared/skills/research/technical-research/SKILL.md create mode 100644 shared/skills/reviews/api-review/SKILL.md create mode 100644 shared/skills/reviews/business-review/SKILL.md create mode 100644 shared/skills/reviews/code-review/SKILL.md create mode 100644 shared/skills/reviews/efcore-review/SKILL.md create mode 100644 shared/skills/reviews/performance-review/SKILL.md create mode 100644 shared/skills/reviews/security-review/SKILL.md create mode 100644 shared/skills/reviews/ui-ux-review/SKILL.md create mode 100644 shared/skills/reviews/wpf-review/SKILL.md create mode 100644 shared/skills/verification/facts-verification/SKILL.md create mode 100644 shared/skills/verification/regression-verification/SKILL.md create mode 100644 shared/skills/verification/work-item-verification/SKILL.md diff --git a/shared/skills/debugging/SKILL.md b/shared/skills/debugging/SKILL.md new file mode 100644 index 0000000..1a0e0e9 --- /dev/null +++ b/shared/skills/debugging/SKILL.md @@ -0,0 +1,17 @@ +--- +name: debugging +description: Investigate and resolve software defects using evidence. +--- + +# Debugging + +Use when behavior is incorrect, failing, or unexpectedly slow. + +1. Reproduce or isolate the problem. +2. Gather evidence and identify the likely cause. +3. Apply the smallest complete fix. +4. Run relevant verification. + +Return the cause, change, and verification result. + +Do not guess when evidence can be collected. diff --git a/shared/skills/git/answer-pull-request/SKILL.md b/shared/skills/git/answer-pull-request/SKILL.md new file mode 100644 index 0000000..11aa3f1 --- /dev/null +++ b/shared/skills/git/answer-pull-request/SKILL.md @@ -0,0 +1,10 @@ +--- +name: answer-pull-request +description: Prepare a clear response to pull request feedback. +--- + +# Answer Pull Request + +Understand each comment, make authorized changes, and respond with evidence. + +Separate completed actions, explanations, and unresolved questions. diff --git a/shared/skills/git/review-pull-request/SKILL.md b/shared/skills/git/review-pull-request/SKILL.md new file mode 100644 index 0000000..3413c63 --- /dev/null +++ b/shared/skills/git/review-pull-request/SKILL.md @@ -0,0 +1,10 @@ +--- +name: review-pull-request +description: Review a pull request for correctness, risk, and readiness. +--- + +# Review Pull Request + +Inspect the change and its context independently. + +Report actionable findings first, followed by verification status and open risks. diff --git a/shared/skills/implementation/integration-test-write/SKILL.md b/shared/skills/implementation/integration-test-write/SKILL.md new file mode 100644 index 0000000..05980ff --- /dev/null +++ b/shared/skills/implementation/integration-test-write/SKILL.md @@ -0,0 +1,10 @@ +--- +name: integration-test-write +description: Add focused integration tests for connected system behavior. +--- + +# Integration Test Write + +Identify the integration boundary, required setup, and meaningful success and failure cases. + +Keep the test deterministic and limited to the behavior under change. diff --git a/shared/skills/implementation/sql-write/SKILL.md b/shared/skills/implementation/sql-write/SKILL.md new file mode 100644 index 0000000..5ad09ef --- /dev/null +++ b/shared/skills/implementation/sql-write/SKILL.md @@ -0,0 +1,12 @@ +--- +name: sql-write +description: Implement a focused SQL change safely. +--- + +# SQL Write + +Understand the schema and affected data before writing SQL. + +Use the smallest complete change, preserve data safety, and run relevant verification. + +Do not execute destructive operations without explicit authorization. diff --git a/shared/skills/implementation/unit-test-write/SKILL.md b/shared/skills/implementation/unit-test-write/SKILL.md new file mode 100644 index 0000000..020ab53 --- /dev/null +++ b/shared/skills/implementation/unit-test-write/SKILL.md @@ -0,0 +1,10 @@ +--- +name: unit-test-write +description: Add focused unit tests for isolated behavior. +--- + +# Unit Test Write + +Identify the behavior and important boundary cases, then add focused deterministic tests. + +Prefer observable behavior over implementation details. Report the checks that were run. diff --git a/shared/skills/planning/change-impact-analysis/SKILL.md b/shared/skills/planning/change-impact-analysis/SKILL.md new file mode 100644 index 0000000..0d5d103 --- /dev/null +++ b/shared/skills/planning/change-impact-analysis/SKILL.md @@ -0,0 +1,12 @@ +--- +name: change-impact-analysis +description: Determine what a proposed change may affect. +--- + +# Change Impact Analysis + +Use when the blast radius or compatibility impact is uncertain. + +Inspect callers, dependencies, configuration, tests, and affected workflows. + +Return affected areas, risks, and recommended verification. Separate facts from assumptions. diff --git a/shared/skills/planning/implementation-planner/SKILL.md b/shared/skills/planning/implementation-planner/SKILL.md new file mode 100644 index 0000000..31be6ba --- /dev/null +++ b/shared/skills/planning/implementation-planner/SKILL.md @@ -0,0 +1,12 @@ +--- +name: implementation-planner +description: Create a concise plan for implementing an understood change. +--- + +# Implementation Planner + +Use before implementation when a change has multiple meaningful steps. + +Identify the goal, affected areas, sequence, risks, and verification. + +Return a short actionable plan. Do not implement the change. diff --git a/shared/skills/planning/work-item-definition/SKILL.md b/shared/skills/planning/work-item-definition/SKILL.md new file mode 100644 index 0000000..099d69e --- /dev/null +++ b/shared/skills/planning/work-item-definition/SKILL.md @@ -0,0 +1,12 @@ +--- +name: work-item-definition +description: Turn an unclear request into a bounded technical work item. +--- + +# Work Item Definition + +Use when requirements, scope, or acceptance criteria are incomplete. + +Clarify the objective, boundaries, assumptions, acceptance criteria, and open questions. + +Do not invent business requirements. diff --git a/shared/skills/research/repository-research/SKILL.md b/shared/skills/research/repository-research/SKILL.md new file mode 100644 index 0000000..d9b1045 --- /dev/null +++ b/shared/skills/research/repository-research/SKILL.md @@ -0,0 +1,12 @@ +--- +name: repository-research +description: Investigate an existing repository to find relevant code and established patterns. +--- + +# Repository Research + +Use before changing an unfamiliar repository area. + +Inspect relevant files, callers, tests, configuration, and established patterns. + +Return evidence, affected areas, unknowns, and useful implementation context. Avoid modifying implementation code. diff --git a/shared/skills/research/technical-research/SKILL.md b/shared/skills/research/technical-research/SKILL.md new file mode 100644 index 0000000..9ec02b9 --- /dev/null +++ b/shared/skills/research/technical-research/SKILL.md @@ -0,0 +1,12 @@ +--- +name: technical-research +description: Investigate technical options and relevant evidence before implementation. +--- + +# Technical Research + +Use when an implementation depends on unfamiliar technology, APIs, tools, or trade-offs. + +Gather relevant evidence, compare viable options, and identify unknowns. + +Return concise findings with sources, assumptions, and a recommendation when appropriate. Do not implement the change. diff --git a/shared/skills/reviews/api-review/SKILL.md b/shared/skills/reviews/api-review/SKILL.md new file mode 100644 index 0000000..f82d242 --- /dev/null +++ b/shared/skills/reviews/api-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: api-review +description: Review API changes for compatibility and contract clarity. +--- + +# API Review + +Inspect request and response contracts, validation, errors, authorization, compatibility, and documentation. + +Return concrete findings and affected consumers. Keep detailed API standards for future customization. diff --git a/shared/skills/reviews/business-review/SKILL.md b/shared/skills/reviews/business-review/SKILL.md new file mode 100644 index 0000000..6ad7ef5 --- /dev/null +++ b/shared/skills/reviews/business-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: business-review +description: Review a change against stated business intent and workflows. +--- + +# Business Review + +Check actors, workflows, domain concepts, invariants, edge cases, and acceptance criteria. + +Separate stated requirements, evidence, and assumptions. Do not invent business requirements. diff --git a/shared/skills/reviews/code-review/SKILL.md b/shared/skills/reviews/code-review/SKILL.md new file mode 100644 index 0000000..01e5250 --- /dev/null +++ b/shared/skills/reviews/code-review/SKILL.md @@ -0,0 +1,12 @@ +--- +name: code-review +description: Review code for correctness, maintainability, and regressions. +--- + +# Code Review + +Inspect the change independently and report concrete findings with severity and evidence. + +Check behavior, regressions, maintainability, unnecessary complexity, and verification. + +Do not modify code unless explicitly asked. diff --git a/shared/skills/reviews/efcore-review/SKILL.md b/shared/skills/reviews/efcore-review/SKILL.md new file mode 100644 index 0000000..effbe07 --- /dev/null +++ b/shared/skills/reviews/efcore-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: efcore-review +description: Review Entity Framework Core changes for correctness and data access risks. +--- + +# EF Core Review + +Inspect queries, relationships, tracking, migrations, transactions, and performance-sensitive access. + +Return concrete findings and relevant verification. Keep detailed EF Core standards for future customization. diff --git a/shared/skills/reviews/performance-review/SKILL.md b/shared/skills/reviews/performance-review/SKILL.md new file mode 100644 index 0000000..0403631 --- /dev/null +++ b/shared/skills/reviews/performance-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: performance-review +description: Review changes for meaningful performance risks. +--- + +# Performance Review + +Identify measurable or plausible bottlenecks in the changed paths. + +Return evidence, expected impact, and proportionate recommendations. Avoid speculative optimization. diff --git a/shared/skills/reviews/security-review/SKILL.md b/shared/skills/reviews/security-review/SKILL.md new file mode 100644 index 0000000..a38a8f1 --- /dev/null +++ b/shared/skills/reviews/security-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: security-review +description: Review changes for concrete security weaknesses. +--- + +# Security Review + +Inspect trust boundaries, input handling, authorization, secrets, permissions, and sensitive data. + +Return evidence, impact, and actionable findings. Do not invent vulnerabilities without evidence. diff --git a/shared/skills/reviews/ui-ux-review/SKILL.md b/shared/skills/reviews/ui-ux-review/SKILL.md new file mode 100644 index 0000000..df7fcb9 --- /dev/null +++ b/shared/skills/reviews/ui-ux-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: ui-ux-review +description: Review user interface changes for usability and consistency. +--- + +# UI UX Review + +Inspect states, feedback, accessibility basics, consistency, and error handling. + +Return concrete findings with affected user flows. Keep detailed design standards for future customization. diff --git a/shared/skills/reviews/wpf-review/SKILL.md b/shared/skills/reviews/wpf-review/SKILL.md new file mode 100644 index 0000000..3423004 --- /dev/null +++ b/shared/skills/reviews/wpf-review/SKILL.md @@ -0,0 +1,10 @@ +--- +name: wpf-review +description: Review WPF changes for correctness and maintainability. +--- + +# WPF Review + +Inspect the changed WPF views, bindings, resources, commands, and lifecycle behavior. + +Return concrete findings and relevant verification. Keep detailed WPF standards for future customization. diff --git a/shared/skills/verification/facts-verification/SKILL.md b/shared/skills/verification/facts-verification/SKILL.md new file mode 100644 index 0000000..539e51e --- /dev/null +++ b/shared/skills/verification/facts-verification/SKILL.md @@ -0,0 +1,10 @@ +--- +name: facts-verification +description: Verify factual claims against available evidence. +--- + +# Facts Verification + +Check important claims against source code, documentation, commands, or other appropriate evidence. + +Return the claim, evidence, and confidence. Mark unknowns explicitly. diff --git a/shared/skills/verification/regression-verification/SKILL.md b/shared/skills/verification/regression-verification/SKILL.md new file mode 100644 index 0000000..040e401 --- /dev/null +++ b/shared/skills/verification/regression-verification/SKILL.md @@ -0,0 +1,10 @@ +--- +name: regression-verification +description: Check that existing behavior was not unintentionally broken. +--- + +# Regression Verification + +Identify the affected existing behavior and run proportionate regression checks. + +Report results and any unverified areas. Do not claim checks that were not run. diff --git a/shared/skills/verification/work-item-verification/SKILL.md b/shared/skills/verification/work-item-verification/SKILL.md new file mode 100644 index 0000000..7a26cdc --- /dev/null +++ b/shared/skills/verification/work-item-verification/SKILL.md @@ -0,0 +1,10 @@ +--- +name: work-item-verification +description: Verify that a work item meets its stated acceptance criteria. +--- + +# Work Item Verification + +Compare the completed change with its requirements and acceptance criteria. + +Return passed criteria, failed criteria, checks run, and remaining limitations.