@juicesharp/rpiv-pi 0.4.0

package/scripts/migrate.js

@@ -0,0 +1,242 @@
+import { existsSync, readFileSync, writeFileSync, mkdirSync, unlinkSync, readdirSync, statSync } from 'fs';
+import { join, dirname, relative, sep } from 'path';
+import { execSync } from 'child_process';
+// --- CLI Argument Parsing ---
+function parseArgs(argv) {
+    let projectDir = process.cwd();
+    let deleteOriginals = false;
+    let dryRun = false;
+    let force = false;
+    for (let i = 2; i < argv.length; i++) {
+        if (argv[i] === '--project-dir' && argv[i + 1]) {
+            projectDir = argv[++i];
+        }
+        else if (argv[i] === '--delete-originals') {
+            deleteOriginals = true;
+        }
+        else if (argv[i] === '--dry-run') {
+            dryRun = true;
+        }
+        else if (argv[i] === '--force') {
+            force = true;
+        }
+    }
+    return { projectDir, deleteOriginals, dryRun, force };
+}
+// --- Discovery ---
+const HARDCODED_EXCLUDES = new Set([
+    'node_modules', 'dist', 'build', '.git', 'vendor', '.rpiv',
+    '.next', '.nuxt', '.output', 'coverage', '__pycache__', '.venv',
+]);
+function discoverClaudeMdFiles(projectDir) {
+    const gitDir = join(projectDir, '.git');
+    if (existsSync(gitDir)) {
+        return discoverViaGit(projectDir);
+    }
+    return discoverViaWalk(projectDir);
+}
+function discoverViaGit(projectDir) {
+    try {
+        const output = execSync('git ls-files --cached --others --exclude-standard', {
+            cwd: projectDir,
+            encoding: 'utf-8',
+            maxBuffer: 10 * 1024 * 1024,
+        });
+        return output
+            .split('\n')
+            .filter((f) => f.endsWith('/CLAUDE.md') || f === 'CLAUDE.md')
+            .filter((f) => !f.startsWith('.rpiv/'));
+    }
+    catch {
+        // git command failed — fall back to walk
+        return discoverViaWalk(projectDir);
+    }
+}
+function discoverViaWalk(projectDir) {
+    const results = [];
+    function walk(dir) {
+        let entries;
+        try {
+            entries = readdirSync(dir);
+        }
+        catch {
+            return; // permission error, skip
+        }
+        for (const entry of entries) {
+            if (HARDCODED_EXCLUDES.has(entry))
+                continue;
+            const fullPath = join(dir, entry);
+            let stat;
+            try {
+                stat = statSync(fullPath);
+            }
+            catch {
+                continue;
+            }
+            if (stat.isDirectory()) {
+                walk(fullPath);
+            }
+            else if (entry === 'CLAUDE.md') {
+                const rel = relative(projectDir, fullPath).split(sep).join('/');
+                if (!rel.startsWith('.rpiv/')) {
+                    results.push(rel);
+                }
+            }
+        }
+    }
+    walk(projectDir);
+    return results;
+}
+// --- Path Mapping ---
+function computeTargetPath(claudeMdRelative) {
+    const dir = dirname(claudeMdRelative);
+    if (dir === '.') {
+        return '.rpiv/guidance/architecture.md';
+    }
+    return join('.rpiv', 'guidance', dir, 'architecture.md').split(sep).join('/');
+}
+function transformContent(content, targetPath) {
+    let refsTransformed = 0;
+    const warnings = [];
+    // Pattern 1: Backtick-wrapped path references like `src/core/CLAUDE.md`
+    let transformed = content.replace(/`((?:[\w][\w./-]*\/)?CLAUDE\.md)`/g, (_match, claudePath) => {
+        const replacement = claudePathToGuidancePath(claudePath);
+        refsTransformed++;
+        return '`' + replacement + '`';
+    });
+    // Pattern 2: Bare path references (with directory prefix) not inside backticks
+    // Match things like "src/core/CLAUDE.md" but not already-backtick-wrapped
+    transformed = transformed.replace(/(?<!`)([\w][\w./-]*\/CLAUDE\.md)(?!`)/g, (_match, claudePath) => {
+        const replacement = claudePathToGuidancePath(claudePath);
+        refsTransformed++;
+        return replacement;
+    });
+    // Pattern 3: Standalone "CLAUDE.md" that references the root file
+    // Only match when it looks like a file reference (not part of a longer word)
+    // Avoid matching inside paths already transformed above
+    transformed = transformed.replace(/(?<![/\w`])CLAUDE\.md(?![/\w`])/g, () => {
+        refsTransformed++;
+        return '.rpiv/guidance/architecture.md';
+    });
+    // Scan for remaining prose references that might need manual attention
+    const lines = transformed.split('\n');
+    for (let i = 0; i < lines.length; i++) {
+        // Look for prose patterns like "see X CLAUDE.md" or "X layer CLAUDE.md"
+        if (/\b\w+\s+CLAUDE\.md\b/i.test(content.split('\n')[i] ?? '') &&
+            !/(src|lib|app|packages|apps)\//.test(content.split('\n')[i] ?? '')) {
+            // Check if this line still has an untransformed prose reference
+            if (/CLAUDE\.md/i.test(lines[i])) {
+                warnings.push({
+                    file: targetPath,
+                    line: i + 1,
+                    message: `Prose reference to CLAUDE.md may need manual update: "${lines[i].trim()}"`,
+                });
+            }
+        }
+    }
+    return { content: transformed, refsTransformed, warnings };
+}
+function claudePathToGuidancePath(claudePath) {
+    const dir = dirname(claudePath);
+    if (dir === '.') {
+        return '.rpiv/guidance/architecture.md';
+    }
+    return '.rpiv/guidance/' + dir + '/architecture.md';
+}
+// --- Main ---
+function main() {
+    const { projectDir, deleteOriginals, dryRun, force } = parseArgs(process.argv);
+    process.stderr.write(`[rpiv:migrate] scanning ${projectDir} for CLAUDE.md files\n`);
+    const claudeFiles = discoverClaudeMdFiles(projectDir);
+    if (claudeFiles.length === 0) {
+        const report = {
+            migrated: [],
+            conflicts: [],
+            warnings: [],
+            originalsDeleted: false,
+            dryRun,
+        };
+        process.stdout.write(JSON.stringify(report, null, 2));
+        return;
+    }
+    process.stderr.write(`[rpiv:migrate] found ${claudeFiles.length} CLAUDE.md file(s)\n`);
+    const migrated = [];
+    const conflicts = [];
+    const allWarnings = [];
+    const writtenFiles = [];
+    for (const source of claudeFiles) {
+        const target = computeTargetPath(source);
+        const targetAbs = join(projectDir, target);
+        // Check for conflicts
+        if (existsSync(targetAbs) && !force) {
+            conflicts.push(target);
+            continue;
+        }
+        // Read source content
+        const sourceAbs = join(projectDir, source);
+        let content;
+        try {
+            content = readFileSync(sourceAbs, 'utf-8');
+        }
+        catch (err) {
+            allWarnings.push({
+                file: source,
+                line: 0,
+                message: `Failed to read: ${err instanceof Error ? err.message : String(err)}`,
+            });
+            continue;
+        }
+        if (content.trim().length === 0) {
+            allWarnings.push({
+                file: source,
+                line: 0,
+                message: 'Empty file, skipped',
+            });
+            continue;
+        }
+        // Transform content
+        const { content: transformed, refsTransformed, warnings } = transformContent(content, target);
+        const lines = transformed.split('\n').length;
+        migrated.push({ source, target, lines, refsTransformed });
+        allWarnings.push(...warnings);
+        if (!dryRun) {
+            writtenFiles.push({ targetAbs, content: transformed });
+        }
+    }
+    // Write all files (all-or-nothing approach for safety)
+    if (!dryRun) {
+        for (const { targetAbs, content } of writtenFiles) {
+            mkdirSync(dirname(targetAbs), { recursive: true });
+            writeFileSync(targetAbs, content, 'utf-8');
+        }
+        process.stderr.write(`[rpiv:migrate] wrote ${writtenFiles.length} file(s)\n`);
+    }
+    // Delete originals only after all writes succeed
+    let originalsDeleted = false;
+    if (!dryRun && deleteOriginals && writtenFiles.length > 0) {
+        for (const entry of migrated) {
+            const sourceAbs = join(projectDir, entry.source);
+            try {
+                unlinkSync(sourceAbs);
+            }
+            catch (err) {
+                allWarnings.push({
+                    file: entry.source,
+                    line: 0,
+                    message: `Failed to delete original: ${err instanceof Error ? err.message : String(err)}`,
+                });
+            }
+        }
+        originalsDeleted = true;
+        process.stderr.write(`[rpiv:migrate] deleted ${migrated.length} original CLAUDE.md file(s)\n`);
+    }
+    const report = {
+        migrated,
+        conflicts,
+        warnings: allWarnings,
+        originalsDeleted,
+        dryRun,
+    };
+    process.stdout.write(JSON.stringify(report, null, 2));
+}
+main();

package/scripts/types.js

@@ -0,0 +1 @@
+export {};

package/skills/annotate-guidance/SKILL.md

@@ -0,0 +1,303 @@
+---
+name: annotate-guidance
+description: Generate architecture.md guidance files in .rpiv/guidance/ by analyzing architecture and patterns in parallel. Auto-detects architecture, proposes locations, and batch-writes compact documentation. Use for onboarding or improving AI assistant context.
+argument-hint: [target-directory]
+allowed-tools: Agent, Read, Write, Glob, Grep
+---
+
+# Annotate Project
+
+You are tasked with generating architecture guidance files for a brownfield project. You will map the project structure, auto-detect its architecture, analyze each architectural layer, and batch-write compact architecture.md files under `.rpiv/guidance/` mirroring the project's directory structure.
+
+## Initial Setup:
+
+Use the current working directory as the target project by default. If the user provides a specific directory path as an argument, use that instead.
+
+## Steps to follow:
+
+1. **Read any directly mentioned files first:**
+   - If the user mentions specific files (existing architecture.md, CLAUDE.md, architecture docs, READMEs), read them FULLY first
+   - **IMPORTANT**: Use the Read tool WITHOUT limit/offset parameters to read entire files
+   - **CRITICAL**: Read these files yourself in the main context before invoking any skills
+   - This ensures you have full context before decomposing the work
+
+2. **Pass 1 — Map the project (parallel agents):**
+   - Spawn the following agents in parallel using the Agent tool:
+
+   **Agent A — Project tree mapping:**
+   - subagent_type: `codebase-locator`
+   - Prompt: "Map the full project tree structure for [target directory]. List all directories and their contents, respecting .gitignore. Focus on source code directories, configuration files, and build artifacts. Return a complete tree view."
+
+   **Agent B — Architecture and conventions:**
+   - subagent_type: `codebase-locator`
+   - Prompt: "Identify the architectural layers, folder roles, and framework conventions in [target directory]. Determine: What architecture pattern is used (clean architecture, MVC, monorepo, microservices, hexagonal, etc.)? What are the main layers/modules? What frameworks and languages are present? What build system is used? For each main layer/module, check if it contains internal sub-layers with distinct architectural roles. If it does, explicitly flag each sub-layer as a guidance target candidate with: (a) path, (b) role/responsibility, (c) approximate file count, (d) how its patterns differ from siblings. Frontend frameworks (Angular, React, Vue, etc.) almost always have distinct sub-layers — components, services, shared/base classes, state management — report each separately."
+
+   - While agents run, read .gitignore yourself to understand exclusion rules
+
+3. **Wait for Pass 1 and determine guidance targets:**
+   - IMPORTANT: Wait for ALL agents from Pass 1 to complete before proceeding
+   - Synthesize the tree structure and architecture findings
+   - Auto-detect the architecture pattern (clean architecture, MVC, monorepo, microservices, etc.)
+   - Determine guidance targets using a two-pass process:
+
+     **Initial pass — identify top-level targets:**
+     - Apply the Guidance Depth Rules (see below) to top-level architectural layers
+     - This produces the initial target list (one per distinct layer/project)
+
+     **Decomposition pass — expand composite targets (ADD, never REPLACE):**
+     - For EACH initial target, review Agent B's sub-layer candidates
+     - If Agent B flagged sub-layers with distinct roles and file counts >10, ADD them as separate guidance targets alongside the parent — the parent stays in the list as an overview, sub-layers are added beneath it
+     - NEVER remove the parent when promoting sub-layers — decomposition expands the target list, it does not substitute entries
+     - Do NOT apply a blanket "sub-folders same as parent" skip — evaluate each sub-layer Agent B flagged individually against the Depth Rules
+     - Common decompositions: Angular/React/Vue apps → components/, services/, shared/; monorepo packages → per-package; large shared libraries → per-concern
+
+   - Present the proposed guidance locations to the user:
+     ```
+     ## Proposed Guidance Locations
+
+     Architecture detected: [pattern name]
+
+     Files will be written to `.rpiv/guidance/` mirroring the project structure.
+
+     ### Folders that need architectural guidance:
+     - `/` (root) — Project overview (compact)
+     - `src/core/` — Core domain layer
+     - `src/services/` — Service layer
+     - [etc.]
+
+     ### Folders to skip:
+     - `src/core/entities/` — Entity grouping, same pattern as parent
+     - [etc.]
+
+     Does this look right? Should I add or remove any locations?
+     ```
+   - Use the `ask_user_question` tool with the following question: "[N] guidance targets across [M] layers. Proceed with analysis?". Options: "Proceed (Recommended)" (Analyze all proposed folders and write architecture.md files); "Add folders" (I want to add more folders to the target list); "Remove folders" (Some proposed folders should be skipped).
+   - Adjust the target list based on user feedback
+
+4. **Pass 2 — Analyze each layer (parallel analyzer agents):**
+   - For each confirmed target folder, spawn agents in parallel:
+
+   **For each target folder, spawn TWO agents:**
+
+   **Analyzer agent:**
+   - subagent_type: `codebase-analyzer`
+   - Prompt: "Analyze [folder path] in detail. Determine: 1) What is this layer's responsibility? 2) What are its dependencies (what does it import/use)? 3) Who consumes it (what imports/uses it)? 4) What are the key architectural boundaries and constraints? 5) What is the module structure — list DIRECTORIES with their roles, base types, and naming conventions. Use architectural annotations (e.g., 'one repo per entity', 'one controller per resource') instead of listing individual filenames. The structure should remain valid when non-architectural files are added. 6) What naming conventions are used (prefixes, suffixes, base classes)?"
+
+   **Pattern finder agent:**
+   - subagent_type: `codebase-pattern-finder`
+   - Prompt: "Find all distinct code patterns used in [folder path]. For each pattern found: 1) Name the pattern with a descriptive heading (e.g., 'Repository Boundary (CRITICAL: Plain Types, NOT Result<T>)'). 2) Provide an IDIOMATIC code example — a generalized, representative version that shows the pattern's essential shape (constructor, key method signatures, return types, error handling). Do NOT copy-paste a single file verbatim; instead synthesize the typical usage across the layer. 3) Add inline comments highlighting important conventions (e.g., '// DB int → boolean', '// throws on error — service wraps in Result'). 4) If the pattern involves a boundary between layers, show both sides. 5) Identify any repeatable workflows for adding new elements to this layer — backend entities (repositories, services, controllers) AND frontend elements (components, services, pages/routes, directives). For example: creating a new repository requires extending BaseRepository + registering in factory; adding a new Angular component requires extending BaseComponent + adding to routes + creating the template. Return these as step-by-step checklists. Return patterns with full code block examples."
+
+   - Spawn 1 analyzer + 1 pattern finder per folder, all in parallel
+   - For the root architecture.md, use findings from ALL folders to create the overview
+
+5. **Wait for Pass 2 and synthesize:**
+   - IMPORTANT: Wait for ALL agents from Pass 2 to complete before proceeding
+   - Compile all agent findings per folder
+   - **Do NOT draft architecture.md content yet** — proceed to developer checkpoint first (Step 6)
+
+6. **Developer checkpoint — validate findings before drafting:**
+
+   Present a per-folder findings summary, then ask grounded questions. This pulls domain knowledge that agents can't discover from code alone — deprecated patterns, undocumented conventions, migration-in-progress situations, or cross-layer rules that only the developer knows.
+
+   **Findings summary** — one block per target folder, 2-3 lines each:
+   ```
+   ## Findings Summary
+
+   ### src/core/
+   Patterns: Repository base class, Entity base with soft-delete, Value Objects
+   Dependencies: Database layer (outbound), Services layer (inbound)
+   Workflows detected: "Add new entity" (5 steps), "Add new value object" (2 steps)
+
+   ### src/services/
+   Patterns: Result<T> wrapping, Transaction scope per operation
+   Dependencies: Core (outbound), Controllers (inbound)
+   Workflows detected: "Add new service" (4 steps)
+
+   [etc.]
+   ```
+
+   Then ask grounded questions — **one at a time**, waiting for the developer's answer before asking the next. Ask as many as the findings warrant — one per significant ambiguity or discovery gap. Use a **❓ Question:** prefix. Each question must reference real findings and pull NEW information from the developer — not confirm what you already found, and not ask about cosmetic issues (typos, formatting) or absences the developer can't add context to.
+
+   Only ask questions whose answer would change what gets written in an architecture.md file. Focus on:
+   - Competing patterns that need a canonical vs. legacy designation (which style should new code follow?)
+   - Cross-layer dependencies that look like violations but might be design decisions
+   - Undocumented architectural constraints not visible in code (ordering, idempotency, invariants)
+
+   Example grounded questions:
+   - "Found two different mapping approaches in `src/services/`: manual mapping in `OrderService` and AutoMapper in `UserService`. Which is the current convention, or is there a migration in progress I should document?"
+   - "The analyzer found no event/message patterns in `src/core/`. Is domain event publishing handled elsewhere, or is it not used in this project?"
+   - "Detected 3 different error-handling styles across layers. Is there a canonical approach, or are these intentional per-layer differences?"
+
+   **CRITICAL**: Ask ONE question at a time. Wait for the answer before asking the next. Lead with your most significant finding. The developer will redirect you if needed.
+
+   **Choosing question format:**
+
+   - **`ask_user_question` tool** — when your question has 2-4 concrete options from code analysis (pattern conflicts, integration choices, scope boundaries, priority overrides). The user can always pick "Other" for free-text. Example: Use the `ask_user_question` tool with the question "Found 2 mapping approaches — which should new code follow?". Options: "Manual mapping (Recommended)" (Used in OrderService (src/services/OrderService.ts:45) — 8 occurrences); "AutoMapper" (Used in UserService (src/services/UserService.ts:12) — 2 occurrences).
+
+   - **Free-text with ❓ Question: prefix** — when the question is open-ended and options can't be predicted (discovery, "what am I missing?", corrections). Example:
+     "❓ Question: Integration scanner found no background job registration for this area. Is that expected, or is there async processing I'm not seeing?"
+
+   **Batching**: When you have 2-4 independent questions (answers don't depend on each other), you MAY batch them in a single `ask_user_question` call. Keep dependent questions sequential.
+
+   **Incorporate developer input:**
+
+   **Corrections** ("that pattern is deprecated", "wrong — we use X"):
+   - Update synthesis. If the correction reveals a pattern that needs fresh analysis, re-prompt a targeted **codebase-analyzer** or **codebase-pattern-finder** (max 2 agents).
+
+   **Missing conventions** ("you missed the soft-delete convention", "all handlers must be idempotent"):
+   - Add directly to synthesis for the relevant folder.
+
+   **Migration context** ("we're moving from X to Y", "old pattern in these files, new pattern in those"):
+   - Record both old and new approaches in synthesis — architecture.md should document the canonical (new) way with a note about the legacy approach still present in specific areas.
+
+   **Scope adjustments** ("skip that layer, it's being rewritten", "add src/shared/"):
+   - Update target list. For new targets, run a targeted Pass 2 (analyzer + pattern-finder, max 2 agents), then fold results into synthesis.
+
+   **Confirmations** ("looks right", "yes that's correct"):
+   - Proceed to drafting.
+
+   After incorporating all input, proceed to Step 7.
+
+7. **Draft architecture.md content:**
+   - Draft architecture.md content in this order — **subfolder files first, root last**:
+     - Subfolder: Use the **Subfolder Architecture Template** (detailed, max 100 lines)
+     - Root folder (LAST): Use the **Root Architecture Template** (compact overview). Draft root only after all subfolder files are finalized — this ensures the deduplication rule can be applied and cross-layer checklists can accurately reference subfolder content
+   - **Output directory convention:** All architecture.md files are written under `.rpiv/guidance/` at the project root, mirroring the project's directory structure. For a target folder at `src/core/`, the output path is `.rpiv/guidance/src/core/architecture.md`. For the root target, the output path is `.rpiv/guidance/architecture.md`. Create intermediate directories as needed.
+   - Enforce the 100-line limit on subfolder files — code examples are essential but keep them concise
+   - If the pattern-finder identified repeatable "add new entity" workflows, include them as `<important if="you are adding a new [entity] to this layer">` conditional sections
+   - If testing patterns were detected, include them as `<important if="you are writing or modifying tests for this layer">` conditional sections
+   - Conditional sections are optional — only include when the pattern-finder found clear evidence of a repeatable workflow
+   - Conditions must be narrow and action-specific (NOT "you are writing code" — too broad)
+   - Do NOT include conventions enforceable by linters, formatters, or pre-commit hooks (e.g., naming conventions, import ordering, indentation) — these add noise without value
+   - Do NOT include patterns easily discoverable from existing code — LLMs are in-context learners and will follow patterns after a few file reads. Only document conventions that are surprising, non-obvious, or span multiple layers
+   - If a pattern section would contain only prose or comments with no code example, either expand it with a real idiomatic example or omit it and reference the source file (e.g., "see `BaseModalComponent` for the modal pattern")
+   - Before writing, verify: no root conditional block duplicates content from a subfolder architecture.md. If a layer has its own subfolder file, remove its summary from root
+   - For cross-layer vertical-slice checklists in root, each step should reference the relevant subfolder architecture.md (e.g., "see `.rpiv/guidance/src/data/architecture.md`") rather than inlining the full procedure
+   - If an existing root architecture.md or CLAUDE.md was found:
+     - Review its content
+     - Redistribute any detailed layer-specific content to the appropriate subfolder architecture.md files
+     - Rewrite the root as a compact overview
+
+8. **Self-review pass — verify every drafted file before writing:**
+   Walk through each drafted architecture.md and check every item below. Fix violations in-place before proceeding to writing.
+
+   **Dependencies** — for each listed dependency, ask: "does this library impose patterns, constraints, or conventions on the code?" If the answer is no (utility libraries like lodash, moment, xlsx, FontAwesome), remove it. Only frameworks and libraries that shape how you write code survive.
+
+   **Module Structure** — count top-level entries. If more than 7, group related directories on one line (e.g., `guards/, interceptors/, pipes/ — cross-cutting plumbing`). Target 4-7 entries.
+
+   **Pattern sections** — every pattern H2 must contain a fenced code block with an idiomatic example. If a section is prose-only or comment-only, either expand it with a real code example or replace the section with a one-line file reference (e.g., "see `TradeDeskMapping.cs` for the mapping pattern").
+
+   **Root deduplication** — for each root conditional block, verify it is NOT summarizing a layer that has its own subfolder architecture.md. If it is, remove the block. For cross-layer vertical-slice checklists, verify each step references the relevant subfolder file (e.g., "see `.rpiv/guidance/X/architecture.md`") rather than inlining the procedure.
+
+   **Frontend/UI conditional coverage** — for each frontend/UI layer, list every repeatable workflow the pattern-finder reported (components, services, pages/routes, directives, pipes, hooks, stores — whatever was detected). Then compare that list against the drafted `<important if>` conditional sections. Any workflow on the list without a matching conditional is a gap — draft and add the missing section before proceeding.
+
+   After fixing all violations, re-scan the corrected drafts to confirm every check passes. Only proceed to writing when all checks are clean. Present a brief summary of what was fixed:
+   ```
+   ## Self-review results
+   - [file]: removed 2 utility deps (moment, xlsx-js-style)
+   - [file]: grouped Module Structure from 11 → 6 entries
+   - [file]: added "Adding a New Service" conditional
+   - Root: no violations found
+   ```
+
+9. **Pass 3 — Write all architecture.md files:**
+   - Write each file to `.rpiv/guidance/{relative_path}/architecture.md`. For the root file, write to `.rpiv/guidance/architecture.md`. Create any intermediate directories that do not exist.
+   - Write ALL files at once using the Write tool
+   - Do NOT ask for confirmation before each file — batch mode
+   - After writing, present a summary:
+     ```
+     ## Architecture Guidance Files Created
+
+     | File | Lines | Description |
+     |------|-------|-------------|
+     | .rpiv/guidance/architecture.md | 45 | Root project overview |
+     | .rpiv/guidance/src/core/architecture.md | 78 | Core domain layer |
+     | .rpiv/guidance/src/services/architecture.md | 65 | Service layer |
+     | [etc.] | | |
+
+     Total: [N] files created/updated
+
+     Please review the files and let me know if you'd like any adjustments.
+     ```
+
+10. **Handle follow-up adjustments:**
+   - If the user requests changes to specific files, edit them directly using the Edit tool
+   - If the user wants additional folders annotated, run a targeted Pass 2 (analyzer + pattern finder) for those folders, then write
+   - If the user wants a file removed, note that they can delete it themselves
+
+## Root Architecture Template (compact):
+
+Read the full template at `templates/root-architecture.md`.
+
+Key principles:
+- Bare sections (Overview, Architecture, Commands, Business Context) are foundational — always included
+- Cross-cutting patterns go in `<important if>` blocks with narrow conditions
+- Deduplication rule: if a layer has a subfolder architecture.md, don't summarize it in root
+- Root MAY include cross-layer vertical-slice checklists referencing subfolder files
+
+### Root Architecture Reference Examples
+
+See `examples/root-nodejs-monorepo.md` (Node.js monorepo) and `examples/root-dotnet-clean-arch.md` (.NET Clean Architecture) for well-formed root architecture.md examples.
+
+What makes these examples good:
+- **Bare sections** (Overview, Project map, Commands) are relevant to nearly every task — no wrapper needed
+- **Each `<important if>` has a narrow trigger** — "adding a new API endpoint" not "writing backend code"
+- **No linter territory** — formatting rules left to tooling
+- **No code snippets** — uses file path references since patterns are better shown in subfolder architecture.md files
+- **Same structure, different ecosystems** — the pattern works identically for Node.js and .NET
+
+## Subfolder Architecture Template (max 100 lines):
+
+Read the full template at `templates/subfolder-architecture.md`.
+
+Key principles:
+- Each distinct pattern gets its own H2 section with a fenced code block
+- Module Structure: aim for 4-7 top-level entries, use architectural annotations
+- Conditional sections (`<important if>`) are optional — only for detected repeatable workflows
+- Conditional sections do NOT count toward the 100-line budget
+
+### Reference Examples
+
+See the following for well-formed subfolder architecture.md examples:
+- `examples/subfolder-database-layer.md` — Database layer (~80 lines)
+- `examples/subfolder-schemas-layer.md` — Schemas layer (~70 lines)
+- `examples/subfolder-dotnet-application.md` — .NET Application layer (~65 lines)
+
+### What makes these examples good:
+- **Module Structure**: Compact, uses architectural annotations, groups related files on one line
+- **Patterns as H2 sections**: Each pattern has a descriptive name, NOT a generic umbrella
+- **Code examples are idiomatic**: Generalized to show the pattern's shape
+- **Cross-boundary patterns**: Shows both sides of layer boundaries
+- **Concise**: All fit well within 100 lines
+- **Conditional blocks**: Wrap scenario-specific recipes with narrow conditions
+
+## Guidance Depth Rules:
+
+**CREATE architecture.md when:**
+- Folder represents a distinct **architectural layer** (core, services, database, redis, ipc)
+- Folder contains **unique organizational logic** not captured by parent
+- Subfolder has **different patterns/constraints** than parent (e.g., `database/repositories/` vs `database/`)
+- Folder has **its own responsibility** (e.g., `database/migrations/`)
+- Folder is a **composite application root** (e.g., SPA, monorepo package) whose children represent distinct sub-layers with different patterns — apply Depth Rules recursively to its children
+
+**SKIP architecture.md when:**
+- Folder only groups entities/DTOs by domain boundary following the same pattern
+- Folder content is fully described by parent architecture.md
+- Folder is a simple grouping without unique constraints
+
+## Important notes:
+- Always use parallel Agent tool calls to maximize efficiency and minimize context usage
+- **File reading**: Always read mentioned files FULLY (no limit/offset) before invoking skills
+- **Critical ordering**: Follow the numbered steps exactly
+  - ALWAYS read mentioned files first before invoking skills (step 1)
+  - ALWAYS wait for all skills in a pass to complete before proceeding to the next step
+  - NEVER write architecture.md files with placeholder values — all content must come from skill findings
+  - NEVER proceed to Pass 2 without user confirmation of target locations
+  - NEVER skip the developer checkpoint (step 6) — developer input is the highest-value signal for architecture.md quality
+  - NEVER draft architecture.md content before completing the developer checkpoint
+- **.gitignore compliance**: Skip directories excluded by .gitignore (node_modules, dist, build, .git, vendor, etc.)
+- **Batch output mode**: Write all architecture.md files at once in Pass 3, do not ask for per-file confirmation
+- **Existing file handling**: If an architecture.md already exists at any target location in `.rpiv/guidance/`, replace it entirely using the Write tool
+- **Line budget**: Subfolder architecture.md files must not exceed 100 lines — code examples in Key Patterns are mandatory, keep them idiomatic and concise
+- **No frontmatter**: architecture.md files are pure markdown, no YAML frontmatter
+- Keep the main agent focused on synthesis, not deep file reading — delegate analysis to sub-agents

package/skills/annotate-guidance/examples/root-dotnet-clean-arch.md

@@ -0,0 +1,38 @@
+# Project Overview
+
+ASP.NET Core 8 Web API with Clean Architecture (CQRS + MediatR).
+
+## Project map
+
+- `src/Api/` - ASP.NET Core controllers, middleware, DI setup
+- `src/Application/` - MediatR handlers, validators, DTOs
+- `src/Domain/` - Entities, value objects, domain events
+- `src/Infrastructure/` - EF Core, external services, file storage
+- `tests/` - Unit and integration tests
+
+## Commands
+
+| Command | What it does |
+|---|---|
+| `dotnet build` | Build solution |
+| `dotnet test` | Run all tests |
+| `dotnet run --project src/Api` | Start API locally |
+| `dotnet ef migrations add <Name> -p src/Infrastructure` | Create EF migration |
+| `dotnet ef database update -p src/Infrastructure` | Apply migrations |
+
+<important if="you are adding a new API endpoint">
+- Add controller in `Api/Controllers/` inheriting `BaseApiController`
+- Add command/query + handler + validator in `Application/Features/`
+- See `Application/Features/Orders/Commands/CreateOrder/` for the pattern
+</important>
+
+<important if="you are adding or modifying EF Core migrations or database schema">
+- Entities configured via `IEntityTypeConfiguration<T>` in `Infrastructure/Persistence/Configurations/`
+- Always create a migration after schema changes — never modify existing migrations
+</important>
+
+<important if="you are writing or modifying tests">
+- Unit tests: xUnit + NSubstitute, one test class per handler
+- Integration tests: `WebApplicationFactory<Program>` with test database
+- See `tests/Application.IntegrationTests/TestBase.cs` for setup
+</important>

package/skills/annotate-guidance/examples/root-nodejs-monorepo.md

@@ -0,0 +1,42 @@
+# Project Overview
+
+Express API + React frontend in a Turborepo monorepo.
+
+## Project map
+
+- `apps/api/` - Express REST API
+- `apps/web/` - React SPA
+- `packages/db/` - Prisma schema and client
+- `packages/ui/` - Shared component library
+- `packages/config/` - Shared configuration
+
+## Commands
+
+| Command | What it does |
+|---|---|
+| `turbo build` | Build all packages |
+| `turbo test` | Run all tests |
+| `turbo lint` | Lint all packages |
+| `turbo dev` | Start dev server |
+| `turbo db:generate` | Regenerate Prisma client after schema changes |
+| `turbo db:migrate` | Run database migrations |
+
+<important if="you are adding or modifying API routes">
+- All routes go in `apps/api/src/routes/`
+- Use Zod for request validation — see `apps/api/src/routes/connections.ts` for the pattern
+- Error responses follow RFC 7807 format
+- Authentication via JWT middleware
+</important>
+
+<important if="you are writing or modifying tests">
+- API: Jest + Supertest, Frontend: Vitest + Testing Library
+- Test fixtures in `__fixtures__/` directories
+- Use `createTestClient()` helper for API integration tests
+- Mock database with `prismaMock` from `packages/db/test`
+</important>
+
+<important if="you are working with client-side state, stores, or data fetching">
+- Zustand for global client state
+- React Query for server state
+- URL state via `nuqs`
+</important>