Dev

.github/copilot-instructions.md

@@ -0,0 +1,44 @@
+# Unity Portal – Copilot Instructions (Repository Root)
+
+> This file is used by **Copilot on GitHub.com** (PR reviews, web chat). For local IDE instructions, see `applications/Unity.GrantManager/.github/copilot-instructions.md`.
+
+## Project Summary
+
+Unity is a **grant management portal** for the Province of British Columbia, built on **ABP Framework 9.1.3** with **.NET 9.0**, targeting **PostgreSQL 17**. The primary application code lives in `applications/Unity.GrantManager/`.
+
+**Key stack:** .NET 9 · ABP 9.1.3 · EF Core 9.0 · PostgreSQL 17 · Redis · RabbitMQ · xUnit · Shouldly · NSubstitute · AutoMapper · Cypress (E2E)
+
+## Repository Structure
+
+```
+applications/Unity.GrantManager/   ← Main .NET solution (developers open this)
+applications/Unity.AutoUI/        ← Cypress E2E tests (TypeScript)
+database/scripts/                  ← SQL seed/migration scripts
+documentation/                     ← Technical docs
+.github/workflows/                 ← GitHub Actions CI/CD
+```
+
+## Key Conventions
+
+- **ABP Framework** modular monolith with DDD layered architecture
+- **AutoMapper** for DTO mapping (not Mapperly)
+- **Razor Pages** UI with custom ABP theme (Unity.Theme.UX2)
+- Multi-tenant architecture with separate host/tenant database contexts
+- `dev` → `test` → `main` branch promotion flow
+- PRs to `dev` from `feature/*`, `hotfix/*`, `bugfix/*`; PRs to `main` from `test` or `hotfix/*` only
+
+## Build & Test (from `applications/Unity.GrantManager/`)
+
+```bash
+dotnet restore Unity.GrantManager.sln
+dotnet build Unity.GrantManager.sln --no-restore
+dotnet test Unity.GrantManager.sln --no-build
+```
+
+All PRs must pass `dotnet build` and `dotnet test` before merge. The CI runs all `*Tests.csproj` in a parallel matrix.
+
+## Do NOT
+
+- Use Mapperly patterns — this project uses AutoMapper
+- Create repositories for child entities — only aggregate roots get repositories
+- Put business logic in application services — use domain entities/services

.github/workflows/docker-build-dev.yml

@@ -63,7 +63,7 @@ jobs:
     environment: dev
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Get short commitId
@@ -93,7 +93,7 @@ jobs:
       actions: write
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Set repository version variables
@@ -112,7 +112,7 @@ jobs:
     runs-on: ubuntu-latest
     environment: dev
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v6
     - name: Build Docker images
       run: |
         rm -f ./docker-compose.override.yml

.github/workflows/docker-build-main.yml

@@ -63,7 +63,7 @@ jobs:
     environment: main
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Get short commitId
@@ -93,7 +93,7 @@ jobs:
       contents: write
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Generate Next Git Tag
@@ -146,7 +146,7 @@ jobs:
       actions: write
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Set repository version variables
@@ -169,7 +169,7 @@ jobs:
     runs-on: ubuntu-latest
     environment: main
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v6
     - name: Build Docker images
       run: |
         rm -f ./docker-compose.override.yml

.github/workflows/docker-build-test.yml

@@ -63,7 +63,7 @@ jobs:
     environment: test
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Get short commitId
@@ -91,7 +91,7 @@ jobs:
     environment: test
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Generate Next Git Tag
@@ -118,7 +118,7 @@ jobs:
     environment: test
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Set repository version variables
@@ -145,7 +145,7 @@ jobs:
     runs-on: ubuntu-latest
     environment: test
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v6
     - name: Build Docker images
       run: |
         rm -f ./docker-compose.override.yml

.github/workflows/manual-trigger.yml

@@ -59,7 +59,7 @@ jobs:
     environment: ${{ inputs.name }}
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Get short commitId
@@ -88,7 +88,7 @@ jobs:
       actions: write
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v6
       with:
           fetch-depth: '1'
     - name: Set repository version variables
@@ -107,7 +107,7 @@ jobs:
     runs-on: ubuntu-latest
     environment: ${{ inputs.name }}
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v6
     - name: Build Docker images
       run: |
         rm -f ./docker-compose.override.yml

.github/workflows/pr-check-dev-branch.yml

@@ -44,7 +44,7 @@ jobs:
     outputs:
       matrix: ${{ steps.discover.outputs.matrix }}
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - id: discover
         run: |
@@ -67,7 +67,7 @@ jobs:
         project: ${{ fromJson(needs.discover-test-projects.outputs.matrix) }}
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - uses: actions/setup-dotnet@v4
         with:

.github/workflows/pr-check-main-branch.yml

@@ -40,7 +40,7 @@ jobs:
     outputs:
       matrix: ${{ steps.discover.outputs.matrix }}
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - id: discover
         run: |
@@ -63,7 +63,7 @@ jobs:
         project: ${{ fromJson(needs.discover-test-projects.outputs.matrix) }}
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - uses: actions/setup-dotnet@v4
         with:

.github/workflows/pr-check-test-branch.yml

@@ -42,7 +42,7 @@ jobs:
     outputs:
       matrix: ${{ steps.discover.outputs.matrix }}
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - id: discover
         run: |
@@ -65,7 +65,7 @@ jobs:
         project: ${{ fromJson(needs.discover-test-projects.outputs.matrix) }}
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - uses: actions/setup-dotnet@v4
         with:

applications/Unity.GrantManager/.github/AGENTS.md

@@ -0,0 +1,24 @@
+# ABP Workflow Agents
+
+This folder contains reusable Copilot agent definitions tailored to Unity Grant Manager ABP workflows.
+
+## Agent Catalog
+
+- `feature-planner.agent.md` - Breaks a feature request into ABP-aligned implementation steps.
+- `ddd-modeler.agent.md` - Designs aggregate boundaries, invariants, repositories, and domain managers.
+- `application-service-designer.agent.md` - Produces application contract and service design with DTO and mapping plans.
+- `efcore-migration-planner.agent.md` - Plans host vs tenant EF Core changes and migration steps.
+- `permissions-localization-auditor.agent.md` - Audits changes for missing permissions and localization compliance.
+- `test-strategy.agent.md` - Generates ABP test plans with unit/integration split and scenario coverage.
+- `test-triage.agent.md` - Diagnoses failing tests and proposes minimal-risk fix sequences.
+- `pr-readiness.agent.md` - Runs a final ABP policy and quality gate before PR creation.
+
+## Usage
+
+Pick the agent that matches your workflow stage and provide:
+
+1. Feature or bug context.
+2. Target module(s) and files.
+3. Constraints (tenant scope, security, deadline, non-functional requirements).
+
+Each agent enforces ABP layering, AutoMapper usage, localization, and test conventions from repository instructions and skills.

applications/Unity.GrantManager/.github/agents/application-service-designer.agent.md

@@ -0,0 +1,43 @@
+---
+name: application-service-designer
+description: Designs application contracts, DTOs, authorization, and AutoMapper mapping plans.
+---
+
+# ABP Application Service Designer Agent
+
+You are the application-layer design specialist for Unity Grant Manager.
+
+## Mission
+
+Produce ABP-compliant service contracts and implementation plans using DTO-first design.
+
+## Inputs
+
+- Use cases and API behavior.
+- Existing service interfaces and DTOs.
+- Target module and permissions.
+
+## Process
+
+1. Propose or update `I*AppService` method signatures.
+2. Define DTOs per method intent (create, update, get, list).
+3. Identify authorization requirements and permission constants.
+4. Define AutoMapper profile changes.
+5. Define validation and business-exception boundaries.
+
+## Output Format
+
+1. Contract changes.
+2. DTO matrix.
+3. Authorization matrix.
+4. Mapping profile changes.
+5. Service implementation checklist.
+6. Test targets.
+
+## Guardrails
+
+- Apply `.github/skills/unity-application-layer/SKILL.md`.
+- Follow `.github/instructions/csharp.instructions.md`.
+- Methods must be async and end with `Async`.
+- Accept/return DTOs only, never entities.
+- Use AutoMapper with `ObjectMapper.Map<>()`, never Mapperly.

applications/Unity.GrantManager/.github/agents/ddd-modeler.agent.md

@@ -0,0 +1,44 @@
+---
+name: ddd-modeler
+description: Designs and reviews ABP DDD models, aggregates, repositories, and domain managers.
+---
+
+# ABP DDD Modeler Agent
+
+You are the DDD modeling specialist for Unity Grant Manager.
+
+## Mission
+
+Design or review domain models so business invariants are enforced in the correct ABP layer.
+
+## Inputs
+
+- Business rules and scenarios.
+- Existing entities and repository interfaces.
+- Target module.
+
+## Process
+
+1. Define aggregate boundaries and ownership rules.
+2. Identify entity/value object responsibilities.
+3. Propose behavior methods that enforce invariants.
+4. Define repository contract additions only for aggregate roots.
+5. Define domain service responsibilities (`*Manager`) where orchestration is needed.
+6. Propose business error codes and exception points.
+
+## Output Format
+
+1. Aggregate model proposal.
+2. Invariants and rule enforcement table.
+3. Repository contract changes.
+4. Domain manager methods.
+5. Error code list.
+6. Anti-pattern checks.
+
+## Guardrails
+
+- Apply `.github/skills/unity-domain-driven-design/SKILL.md`.
+- Follow `.github/instructions/csharp.instructions.md`.
+- Do not generate GUIDs in entity constructors.
+- Reference external aggregates by Id only.
+- Keep app-service logic out of the domain model design.

applications/Unity.GrantManager/.github/agents/efcore-migration-planner.agent.md

@@ -0,0 +1,43 @@
+---
+name: efcore-migration-planner
+description: Plans EF Core model updates and host versus tenant migrations safely.
+---
+
+# ABP EF Core Migration Planner Agent
+
+You are the EF Core migration planning specialist for Unity Grant Manager.
+
+## Mission
+
+Plan schema changes, mapping updates, and migration execution for the correct database context.
+
+## Inputs
+
+- Proposed entity/model changes.
+- Whether data is host-wide or tenant-scoped.
+- Existing migrations and repository code.
+
+## Process
+
+1. Classify each change as host, tenant, or both.
+2. Propose `ModelBuilder` mapping updates.
+3. Verify repository impact and query behavior.
+4. Produce migration commands and ordering.
+5. Identify rollback and data backfill considerations.
+
+## Output Format
+
+1. Context classification.
+2. Mapping change checklist.
+3. Migration command plan.
+4. Data safety notes.
+5. Repository update checklist.
+6. Validation tests.
+
+## Guardrails
+
+- Apply `.github/skills/unity-ef-core/SKILL.md`.
+- Follow `.github/instructions/efcore.instructions.md`.
+- Always call `ConfigureByConvention()` for mapped entities.
+- Do not use `includeAllEntities: true` with default repositories.
+- Always specify context for migration commands.