@atlashub/smartstack-cli 4.80.0 → 5.0.0

package/templates/skills/migrate/references/v3.34-to-v3.46.md

@@ -1,289 +0,0 @@
-# Migration rules: v3.34 → v3.46
-
-> Used by `/migrate --from=3.34 --to=3.46`.
-> Covers the breaking changes shipped during the v3.46 alignment work
-> (ApplicationZone removal, layout consolidation, NavigationApplication.Create
-> signature change, Tailwind tokenisation, mergeRoutes deprecation, AI patterns).
-
-## Rule list (machine-readable)
-
-```yaml
-rules:
-  # ──────────────────────────────────────────────────────────────────
-  # BACKEND — Domain / Seed / API
-  # ──────────────────────────────────────────────────────────────────
-
-  - id: R-BE-001
-    category: backend
-    description: "ApplicationZone enum and Zone column removed (replaced by IsPersonal + IsOpen flags on NavigationApplication)."
-    detect_pattern: 'ApplicationZone\.\w+|Zone\s*=\s*ApplicationZone\.\w+|public\s+ApplicationZone\s+Zone'
-    glob: "src/**/*.cs"
-    severity: BLOCKING
-    requires_confirmation: false
-    references:
-      - templates/skills/apex/references/core-seed-data.md
-      - templates/skills/application/references/provider-template.md
-      - templates/skills/apex/references/checks/seed-checks.sh   # check C66 enforces this
-
-  - id: R-BE-002
-    category: backend
-    description: "NavigationApplication.Create() signature lost the first 'zone' parameter; gained isOpen + isPersonal at the end (both default false)."
-    detect_pattern: 'NavigationApplication\.Create\(\s*ApplicationZone\.\w+'
-    glob: "src/**/*.cs"
-    severity: BLOCKING
-    requires_confirmation: true   # contextual — review each call site
-    references:
-      - D:/01 - projets/SmartStack.app/features/IA-Workflow/src/SmartStack.Domain/Navigation/NavigationApplication.cs
-
-  - id: R-BE-003
-    category: backend
-    description: "ApplicationMenuDto.Zone (string) field removed; consume IsPersonal + IsOpen instead."
-    detect_pattern: '\.Zone\b|"Zone"'
-    glob: "src/SmartStack.Application/Navigation/**/*.cs"
-    severity: BLOCKING
-    requires_confirmation: true
-
-  - id: R-BE-004
-    category: backend
-    description: "Migration to drop the 'Zone' column and add IsPersonal/IsOpen columns. Generate via dotnet ef if missing."
-    detect_pattern: 'Zone'
-    glob: "src/SmartStack.Infrastructure/Persistence/Migrations/*.cs"
-    severity: WARNING
-    requires_confirmation: true
-    note: "If the project's last migration is older than core_v3.46.0, prompt user to run /efcore:migration RemoveApplicationZone."
-
-  - id: R-BE-005
-    category: backend
-    description: "Domain Hooks moved to SmartStack.Application.Common.Interfaces.Hooks namespace (previously: scattered)."
-    detect_pattern: 'SmartStack\.(?:Domain|Application)\.(?!Common\.Interfaces\.Hooks)\w+\.(?:IBeforeCreate|IAfterCreate|IBeforeUpdate|IAfterUpdate|IBeforeDelete|IAfterDelete)'
-    glob: "src/**/*.cs"
-    severity: WARNING
-    requires_confirmation: false
-    references:
-      - templates/skills/apex/references/smartstack-api.md   # Entity Lifecycle Hooks section
-
-  - id: R-BE-006
-    category: backend
-    description: "BaseEntity gained an ExtensionData (string JSON) property + 7 typed methods. EF configuration must include builder.Property(x => x.ExtensionData).HasColumnType(\"nvarchar(max)\").HasDefaultValue(\"{}\")."
-    detect_pattern: 'class\s+\w+Configuration\s*:\s*IEntityTypeConfiguration<\w+>'
-    glob: "src/SmartStack.Infrastructure/Persistence/Configurations/**/*.cs"
-    severity: WARNING
-    requires_confirmation: true
-    note: "Inspect each EF config file; only add the ExtensionData mapping where the entity inherits BaseEntity AND the table is meant to store extension data."
-
-  # ──────────────────────────────────────────────────────────────────
-  # FRONTEND — Layouts / Routing / Theme
-  # ──────────────────────────────────────────────────────────────────
-
-  - id: R-FE-001
-    category: frontend
-    description: "Legacy domain-specific layouts (AdminLayout / BusinessLayout / UserLayout / HRLayout / SalesLayout) consolidated into AppLayout (sole authenticated shell)."
-    detect_pattern: '<\s*(AdminLayout|BusinessLayout|UserLayout|HRLayout|SalesLayout)\b|from\s+["\x27][^"\x27]*\/(AdminLayout|BusinessLayout|UserLayout|HRLayout|SalesLayout)["\x27]'
-    glob: "web/**/*.tsx"
-    severity: BLOCKING
-    requires_confirmation: false
-    references:
-      - templates/skills/application/references/frontend-route-naming.md
-      - templates/skills/apex/references/checks/seed-checks.sh   # check C67
-
-  - id: R-FE-002
-    category: frontend
-    description: "Hardcoded Tailwind colors must be replaced by CSS variables. Mapping table in themes-db-driven.md (24 rows: bg-blue-* → var(--info-bg), text-gray-900 → var(--text-primary), etc.)."
-    detect_pattern: '(bg|text|border)-(?!\[)(red|blue|green|gray|amber|yellow|orange|purple|indigo|violet|cyan|pink|emerald|rose|sky|teal|lime|fuchsia|stone|slate|zinc|neutral)-\d'
-    glob: "web/**/*.tsx"
-    severity: WARNING   # was BLOCKING in iteration 2 — but for a migration we relax to WARNING and let the user batch-fix
-    requires_confirmation: true
-    references:
-      - templates/skills/application/references/themes-db-driven.md   # Section 5: Migration map
-      - templates/skills/apex/references/checks/frontend-checks.sh    # check C9
-
-  - id: R-FE-003
-    category: frontend
-    description: "Tailwind dark: prefix removed. SmartStack toggles a .dark class on root; CSS vars adapt automatically."
-    detect_pattern: '\bdark:(bg|text|border)-'
-    glob: "web/**/*.tsx"
-    severity: WARNING
-    requires_confirmation: false
-
-  - id: R-FE-004
-    category: frontend
-    description: "mergeRoutes() and getStaticAppRoutes() removed in v3.7+. Pages must register via PageRegistry.register() in componentRegistry.generated.ts."
-    detect_pattern: 'mergeRoutes\s*\(|getStaticAppRoutes\s*\('
-    glob: "web/**/*.{ts,tsx}"
-    severity: BLOCKING
-    requires_confirmation: true
-    references:
-      - templates/skills/audit-route/references/routing-pattern.md
-      - templates/skills/application/references/smartstack-provider.md
-
-  - id: R-FE-005
-    category: frontend
-    description: "RouteGuard.tsx replaced by ProtectedRoute.tsx (API-driven permissions)."
-    detect_pattern: 'RouteGuard|from\s+["\x27][^"\x27]*\/RouteGuard["\x27]'
-    glob: "web/**/*.{ts,tsx}"
-    severity: BLOCKING
-    requires_confirmation: false
-
-  - id: R-FE-006
-    category: frontend
-    description: "useParams must use generic { id: string } (not custom names like { ticketId, userId }). Custom names go in the destructuration."
-    detect_pattern: 'useParams<\{\s*(?!id\s*:)\w+Id\s*:'
-    glob: "web/**/*.tsx"
-    severity: WARNING
-    requires_confirmation: true
-    note: "Refactor: const { id: ticketId } = useParams<{ id: string }>(). Param URL must be 'id' for DynamicRouter."
-
-  - id: R-FE-007
-    category: frontend
-    description: "Client App.tsx must wrap with <SmartStackProvider config={...}> (replaces ad-hoc provider stacks)."
-    detect_pattern: 'AuthProvider|TenantProvider|ThemeProvider'   # if present in App.tsx without SmartStackProvider above
-    glob: "web/**/{App,main}.tsx"
-    severity: WARNING
-    requires_confirmation: true
-    references:
-      - templates/skills/application/references/smartstack-provider.md
-
-  # ──────────────────────────────────────────────────────────────────
-  # SEED — Navigation & Roles
-  # ──────────────────────────────────────────────────────────────────
-
-  - id: R-SEED-001
-    category: seed
-    description: "NavigationApplicationSeedEntry must include IsPersonal + IsOpen (both bool, default false). Replaces the Zone field."
-    detect_pattern: 'NavigationApplicationSeedEntry'
-    glob: "src/**/*NavigationApplicationSeedData.cs"
-    severity: BLOCKING
-    requires_confirmation: false
-
-  - id: R-SEED-002
-    category: seed
-    description: "MySpace and Personal-scope apps should set IsPersonal = true."
-    detect_pattern: 'NavigationApplicationSeedEntry'
-    glob: "src/**/*MySpace*SeedData.cs"
-    severity: WARNING
-    requires_confirmation: true
-    note: "Manual judgement: only set IsPersonal=true on apps that genuinely belong to user space."
-
-  # ──────────────────────────────────────────────────────────────────
-  # CONFIG — appsettings / package.json
-  # ──────────────────────────────────────────────────────────────────
-
-  - id: R-CFG-001
-    category: config
-    description: "@atlashub/smartstack peer must be at least 3.46.0."
-    detect_pattern: '"@atlashub/smartstack":\s*"\^?[0-2]\.|^"@atlashub/smartstack":\s*"\^?3\.[0-3][0-9]\.|^"@atlashub/smartstack":\s*"\^?3\.4[0-5]\.'
-    glob: "web/smartstack-web/package.json"
-    severity: BLOCKING
-    requires_confirmation: false
-    note: "Bump @atlashub/smartstack to ^3.46.0 (or latest)."
-
-  - id: R-CFG-002
-    category: config
-    description: "Tailwind v4 + @tailwindcss/vite required. Old @tailwindcss/postcss configuration deprecated."
-    detect_pattern: '"@tailwindcss/postcss"'
-    glob: "web/smartstack-web/package.json"
-    severity: WARNING
-    requires_confirmation: false
-    note: "Install @tailwindcss/vite, remove @tailwindcss/postcss, update vite.config.ts plugins list."
-
-  - id: R-CFG-003
-    category: config
-    description: "appsettings.json should include the new sections : SlaMonitoring, SolutionAutoClose, AutoAssignment, WorkflowExecution (added in v3.45/v3.46)."
-    detect_pattern: '^\s*"(?:Logging|Email)\s*"'
-    glob: "src/SmartStack.Api/appsettings.json"
-    severity: WARNING
-    requires_confirmation: true
-    note: "Compare against templates/project/appsettings.json.template ; add missing sections (defaults are safe)."
-```
-
-## Implementation recipes
-
-The rules above are declarative ; some need a step-by-step recipe when a single regex isn't enough. The skill's `--apply` mode falls back to these recipes for any rule with `requires_confirmation: true`.
-
-### R-BE-001 — `ApplicationZone` removal
-
-For each file matched :
-
-1. Remove every line that matches `^\s*Zone\s*=\s*ApplicationZone\.\w+,?\s*$`
-2. After the line `Id = ApplicationId,` (or the first property of the SeedEntry), insert :
-   ```
-   IsPersonal = false,
-   IsOpen = false,
-   ```
-3. Replace `public ApplicationZone Zone { get; init; }` (DTO) by :
-   ```csharp
-   public bool IsPersonal { get; init; }
-   public bool IsOpen { get; init; }
-   ```
-4. Remove `using` directives for `SmartStack.Domain.Navigation` if `ApplicationZone` was the only reason they were imported.
-
-### R-BE-002 — `NavigationApplication.Create()` signature
-
-For each call site :
-
-- Old signature : `NavigationApplication.Create(ApplicationZone.X, code, label, description, icon, iconType, route, displayOrder)`
-- New signature : `NavigationApplication.Create(code, label, description, icon, iconType, route, displayOrder, isOpen: bool, isPersonal: bool)`
-
-**Recipe** :
-1. Remove the first argument (`ApplicationZone.X`).
-2. Append two named arguments at the end : `isOpen: false, isPersonal: false` (or `true` based on context — see R-SEED-002).
-3. **Manual review** : if the original call passed `Personal` zone, propose `isPersonal: true`. If `Admin` zone (system app), propose `isOpen: true`.
-
-### R-FE-001 — Layout consolidation
-
-For each `.tsx` file containing `<AdminLayout>`, `<BusinessLayout>`, `<UserLayout>`, `<HRLayout>`, or `<SalesLayout>` :
-
-1. Replace the JSX tag (open + close) by `<AppLayout>` / `</AppLayout>`.
-2. Update the import : `import { AppLayout } from '@/layouts/AppLayout'`.
-3. Remove the old import line.
-
-If the file was `src/layouts/AdminLayout.tsx` (or similar), DELETE the file — it no longer has a purpose.
-
-### R-FE-002 — Tailwind tokenisation
-
-This rule is **bulk** but **noisy**. Apply only if the user accepts (default WARNING in migration ; was BLOCKING in iteration 2).
-
-Use the migration map in `templates/skills/application/references/themes-db-driven.md` Section 5. For each `(bg|text|border)-{color}-{shade}` occurrence, propose the canonical CSS-var replacement based on context :
-- Status badges (success/warning/error/info) → `var(--{status}-bg)` / `var(--{status}-text)` / `var(--{status}-border)`
-- Action buttons → `var(--color-accent-600)` / `hover:bg-[var(--color-accent-700)]`
-- Body text → `var(--text-primary)` / `var(--text-secondary)`
-- Card backgrounds → `var(--bg-card)`
-- Borders → `var(--border-color)`
-
-When the context is ambiguous (e.g. random `bg-blue-100`), prompt for the intent (info badge ? accent ? branding ?) and apply accordingly.
-
-### R-FE-004 — `mergeRoutes` / `getStaticAppRoutes` removal
-
-These calls usually live in `src/main.tsx` or `src/App.tsx`. Recipe :
-
-1. Open the file.
-2. Find the call to `mergeRoutes(...)` or `getStaticAppRoutes(...)`.
-3. For each route registered :
-   - Convert to `PageRegistry.register('module.section', lazy(() => import(...)))`
-   - Place those calls in `src/extensions/componentRegistry.generated.ts` (create the file if missing).
-4. Remove the original `mergeRoutes` call ; remove its imports.
-5. Ensure `main.tsx` imports `./extensions/componentRegistry.generated` early (before render).
-
-This is a refactor — the skill SHOULD prompt the user to confirm and provide a diff preview.
-
-### R-CFG-002 — Tailwind v4 migration
-
-```bash
-# Inside web/smartstack-web/
-npm uninstall @tailwindcss/postcss
-npm install --save-dev @tailwindcss/vite
-
-# Edit vite.config.ts:
-#  + import tailwindcss from '@tailwindcss/vite';
-#  + plugins: [react(), tailwindcss()],
-
-# Edit index.css to use the new @theme {} syntax (see themes-db-driven.md Section 3.3)
-```
-
-The skill prompts and applies one step at a time, since these touch different files.
-
-## Notes
-
-- This rule file is the single source of truth for `v3.34 → v3.46`. Add new rules here as breaking changes are discovered.
-- For future migrations (e.g. `v3.46 → v3.47`), create `references/v3.46-to-v3.47.md` and register it in the SKILL.md mapping table.
-- Always run with `--dry-run` first. Always commit before `--apply`.

package/templates/skills/notification/SKILL.md

@@ -1,173 +0,0 @@
----
-name: notification
-description: |
-  Integrates SmartStack notification system into development.
-  Use this skill when:
-  - User wants to send notifications from a feature
-  - User mentions "notify", "alert", "notification", "in-app"
-  - Creating a module that requires user notifications
-  - SignalR integration for real-time
-  Types: In-App, Email (via Workflow), Push (future)
----
-
-# Skill Notification SmartStack
-
-> **Architecture:** In-App (DB + SignalR) + Email (via Workflows)
-
-**Reference:** [_shared.md](../_shared.md) for common services
-
-## WHEN THIS SKILL ACTIVATES
-
-| Trigger | Example |
-|---------|---------|
-| Explicit request | "Notify the user when a ticket is created" |
-| Event-driven | "When SLA expires, alert the user" |
-| Real-time | "Display new notifications in real-time" |
-| Keywords | "notify", "alert", "SignalR", "real-time" |
-
-## FLOW
-
-```
-EVENT → INotificationService.SendNotificationAsync()
-    ↓
-DB (store) + SignalR Hub (real-time)
-    ↓
-Frontend: useSignalR() → NotificationBell
-```
-
-## NOTIFICATION TYPES
-
-| Type | Usage |
-|------|-------|
-| `TicketCreated/Assigned/StatusChanged` | Support |
-| `RoleAssigned/Removed/PermissionsChanged` | Admin |
-| `SystemAnnouncement` | Global |
-| `SlaWarning/ResponseBreached/ResolutionBreached` | SLA |
-
-### Add a New Type
-```csharp
-// 1. Domain/Support/Enums/NotificationType.cs
-public enum NotificationType { ..., $NEW_TYPE = XX }
-
-// 2. Usage
-await _notificationService.SendNotificationAsync(userId, NotificationType.$NEW_TYPE, title, message,
-    relatedEntityType: "Entity", relatedEntityId: id, actionUrl: "/path");
-```
-
-## WORKFLOW INTEGRATION
-
-### 1. Identify the Need
-| Question | Action |
-|----------|--------|
-| Real-time? | → In-App + SignalR |
-| Email? | → Workflow + EmailTemplate |
-| Related entity? | → relatedEntityType/Id |
-| Navigation? | → actionUrl |
-
-### 2. Send Notification
-```csharp
-// Simple
-await _notificationService.SendNotificationAsync(userId, NotificationType.TicketCreated,
-    "New ticket", $"Ticket #{ticket.Number} created", ct);
-
-// With related entity
-await _notificationService.SendNotificationAsync(userId, NotificationType.TicketAssigned,
-    "Ticket assigned", message,
-    relatedEntityType: "Ticket", relatedEntityId: ticket.Id,
-    actionUrl: $"/support/tickets/{ticket.Id}", ct);
-
-// Multi-user
-await _notificationService.SendNotificationsAsync(userIds, type, title, message, ct);
-
-// By role
-await _notificationService.SendNotificationToRoleAsync("Admin", type, title, message, ct);
-```
-
-### 3. User Preferences
-```csharp
-var (shouldEmail, shouldInApp, shouldPush) = await _notificationService.ShouldNotifyAsync(userId, type);
-if (shouldInApp) await _notificationService.SendNotificationAsync(...);
-if (shouldEmail) await _workflowService.TriggerAsync("ticket.created", variables);
-```
-
-## FRONTEND
-
-### Hook useSignalR
-```typescript
-useSignalR({
-  onNotification: (notification) => { toast.info(notification.title); },
-  onUnreadCountUpdate: (count) => { setUnreadCount(count); },
-  onPermissionsChanged: () => { queryClient.invalidateQueries(['permissions']); }
-});
-```
-
-### API Notifications
-```typescript
-notificationsApi.getUnread(limit)
-notificationsApi.getAll(page, pageSize, isRead?)
-notificationsApi.getUnreadCount()
-notificationsApi.markAsRead(id) / markAllAsRead()
-notificationsApi.delete(id) / deleteAll()
-```
-
-### NotificationBell Component
-```tsx
-<NotificationBell />  // Manages useSignalR, badge unreadCount, dropdown
-```
-
-## ADVANCED PATTERNS
-
-### SLA Warning (Hangfire Job)
-```csharp
-var tickets = await _context.Tickets
-    .Where(t => t.SlaDeadline <= DateTime.UtcNow.AddMinutes(30) && !t.SlaWarningNotificationSent)
-    .ToListAsync();
-foreach (var ticket in tickets) {
-    await _notificationService.SendNotificationAsync(..., NotificationType.SlaWarning, ...);
-    ticket.SlaWarningNotificationSent = true;
-}
-```
-
-### Grouped Notification
-```csharp
-if (entities.Count == 1) await SendSingle(...);
-else await SendBatch($"{entities.Count} new elements", ...);
-```
-
-## CHECKLIST
-
-```
-□ Notification type identified (existing or new enum)
-□ INotificationService injected
-□ Send with: userId, type, title, message, relatedEntityType?, relatedEntityId?, actionUrl?
-□ Frontend: useSignalR configured, toast/UI on receipt
-□ If email: workflow configured
-```
-
-## ABSOLUTE RULES
-
-| DO | DON'T |
-|----|-------|
-| INotificationService | Direct DB access |
-| NotificationType enum | Hardcoded types |
-| relatedEntityType/Id + actionUrl | Notifications without context |
-| Respect user preferences | Spam notifications |
-| CancellationToken | Hardcoded messages (use i18n) |
-
-## KEY FILES
-
-| File | Role |
-|------|------|
-| `Domain/Support/Notification.cs` | Entity |
-| `Domain/Support/Enums/NotificationType.cs` | Types |
-| `Application/Common/Interfaces/INotificationService.cs` | Interface |
-| `Infrastructure/Services/SignalR/NotificationHubService.cs` | Real-time |
-| `web/src/hooks/useSignalR.ts` | Frontend hook |
-| `web/src/components/notifications/NotificationBell.tsx` | UI |
-
-<success_criteria>
-- INotificationService integration wired with correct NotificationType
-- relatedEntityType/Id and actionUrl set for navigation context
-- SignalR real-time delivery working via NotificationHubService
-- User preference filtering respected
-</success_criteria>

package/templates/skills/refactor/SKILL.md

@@ -1,56 +0,0 @@
----
-name: refactor
-description: Refactor code by finding files, grouping them, and launching parallel Snipper agents
-argument-hint: <search-pattern-or-description>
-disable-model-invocation: true
----
-
-<objective>
-Refactor code matching the given pattern across the codebase using parallel Snipper agents for maximum speed.
-
-This skill finds all relevant files, creates ONE instruction file, then launches Snipper agents in parallel with batches of max 3 files each.
-</objective>
-
-<quick_start>
-```bash
-/refactor rename getUserData to fetchUserProfile
-/refactor replace console.log with logger.info
-/refactor convert class components to functional
-/refactor update deprecated API calls
-```
-</quick_start>
-
-<workflow>
-
-## Phase 1: Discovery
-1. Parse the refactor request (method name, pattern, code smell)
-2. Find all affected files (Grep for code, Glob for file patterns)
-3. Analyze scope — if >15 files, confirm with user
-
-See [steps/step-01-discover.md](steps/step-01-discover.md) for search strategy and instruction file template.
-
-## Phase 2: Create Instructions
-4. Create task folder: `.claude/tasks/refactor-{timestamp}/`
-5. Create ONE `instructions.md` with: Objective, Pattern, Transformation, Examples, Constraints
-
-## Phase 3: Group and Execute
-6. Group files into batches (max 3 per batch)
-7. Launch ALL Snipper agents in parallel (single message, multiple Task calls)
-8. Wait for completion
-
-## Phase 4: Verification
-9. Validate changes (lint/typecheck/build)
-10. Provide summary report
-
-See [steps/step-02-execute.md](steps/step-02-execute.md) for execution details and report format.
-
-</workflow>
-
-<success_criteria>
-- All target files identified
-- ONE instruction file created in `.claude/tasks/refactor-{id}/instructions.md`
-- Snipper agents launched in parallel (max 3 files per agent)
-- All batches completed successfully
-- Lint/type checks pass
-- Summary provided to user
-</success_criteria>

package/templates/skills/refactor/steps/step-01-discover.md

@@ -1,60 +0,0 @@
-# Phase 1: Discovery & Instructions
-
-## 1. Parse the Refactor Request
-
-Understand what the pattern means:
-- Could be: method name, component name, pattern, code smell, etc.
-- Identify the search strategy (Grep for code patterns, Glob for file patterns)
-
-## 2. Find All Affected Files
-
-```
-Use Grep to search for the pattern in the codebase
-Use Glob if searching by file name patterns
-Exclude: node_modules, .git, dist, build, bin, obj
-```
-
-## 3. Analyze Scope
-
-- Count total files found
-- If more than 15 files, ask user to confirm or narrow scope
-- Show preview of files to refactor
-
-## 4. Create Task Folder
-
-```
-Generate unique ID: refactor-{timestamp}
-Create folder: .claude/tasks/refactor-{timestamp}/
-```
-
-## 5. Create ONE Instruction File
-
-Create `.claude/tasks/refactor-{id}/instructions.md`:
-
-```markdown
-# Refactor: {title}
-
-## Objective
-{What needs to be refactored - derived from user request}
-
-## Pattern to Find
-{Exact code pattern, method name, or structure to locate}
-
-## Transformation
-{How to transform the found pattern - be specific and adaptive}
-
-## Examples
-Before:
-{example of current code}
-
-After:
-{example of refactored code}
-
-## Constraints
-- Only modify code matching the pattern
-- Preserve all existing functionality
-- Follow codebase conventions
-- No comments unless necessary
-```
-
-**IMPORTANT**: Make instructions adaptive - they should work for ANY file in the list.

package/templates/skills/refactor/steps/step-02-execute.md

@@ -1,67 +0,0 @@
-# Phase 3-4: Execute & Verify
-
-## Group Files into Batches
-
-- Maximum 3 files per batch
-- Group by related functionality when possible
-
-## Launch Snipper Agents in Parallel
-
-For EACH batch, use Task tool with `subagent_type='Snipper'`:
-
-```
-Using the instructions in .claude/tasks/refactor-{id}/instructions.md, refactor these files:
-- {file_1}
-- {file_2}
-- {file_3}
-
-Read the instructions file first, then apply the refactor to each file.
-```
-
-**CRITICAL**: Launch ALL batches in a SINGLE message with multiple Task calls.
-
-## Wait for Completion
-
-- All Snipper agents run in parallel
-- Collect results from each
-
-## Validate Changes
-
-```bash
-# For TypeScript/JavaScript
-pnpm lint || npm run lint
-pnpm tsc || npx tsc
-
-# For .NET
-dotnet build
-```
-
-Fix any errors immediately.
-
-## Summary Report
-
-```
-Refactor Complete: {title}
-
-Files modified: {count}
-Batches executed: {count}
-Errors: {count}
-
-Modified files:
-- {file1}
-- {file2}
-...
-
-Next steps:
-- Review changes: git diff
-- Run tests: pnpm test
-- Commit: /gitflow commit
-```
-
-## Execution Rules
-
-- **PARALLEL EXECUTION**: Launch all Snipper batches simultaneously
-- **SINGLE INSTRUCTION FILE**: One instructions.md for all batches
-- **MAX 3 FILES PER BATCH**: Keep Snipper agents focused
-- **VALIDATE AFTER**: Run lint/typecheck after all batches complete
-- **PRESERVE FUNCTIONALITY**: Never break existing behavior