@haoyiyin/workflow 0.2.0

package/dist/src/skills/review-diff/index.js

@@ -0,0 +1,316 @@
+/**
+ * Review Diff Skill - Reviews code changes via reviewer subagents
+ *
+ * Use case: Review PR, branch, commit, or diff-file
+ * Pattern: Get diff → Inspect surface → Check requirements → Check tests →
+ *          Check quality → Return structured review report
+ *
+ * Spawns reviewer subagent(s) to review diffs.
+ * Returns structured review report with severity-classified issues.
+ */
+import { z } from 'zod';
+import { Skill } from '../skill.js';
+import { createDispatcher } from '../../agents/dispatcher.js';
+import { createMainAgentGuard } from '../../guard/main-agent.js';
+// ---------------------------------------------------------------------------
+// Schemas
+// ---------------------------------------------------------------------------
+const ReviewDiffInputSchema = z.object({
+    target: z.string().min(1),
+    targetType: z
+        .enum(['pr', 'branch', 'commit', 'diff-file'])
+        .default('branch'),
+    planPath: z.string().optional(),
+    focus: z
+        .array(z.enum(['security', 'performance', 'correctness', 'style', 'tests']))
+        .optional(),
+    model: z.string().optional(),
+});
+const ReviewDiffOutputSchema = z.object({
+    decision: z.enum(['approve', 'changes', 'blocked', 'needs-evidence']),
+    issues: z.array(z.object({
+        severity: z.enum(['blocker', 'major', 'minor', 'note']),
+        file: z.string().optional(),
+        line: z.number().optional(),
+        description: z.string(),
+        action: z.string(),
+    })),
+    specCompliance: z.enum(['pass', 'partial', 'fail']),
+    testAssessment: z.enum(['adequate', 'missing', 'failing']),
+    strengths: z.array(z.string()),
+    summary: z.string(),
+    tokensUsed: z.number(),
+});
+// ---------------------------------------------------------------------------
+// Prompt builders (pure functions)
+// ---------------------------------------------------------------------------
+function buildDiffPrompt(targetType, target) {
+    const commands = {
+        pr: `gh pr diff ${target}`,
+        branch: `git diff origin/${target}...HEAD`,
+        commit: `git show ${target}`,
+        'diff-file': `cat ${target}`,
+    };
+    const cmd = commands[targetType] ?? `git diff ${target}`;
+    return [
+        `Get the diff for ${targetType}: ${target}`,
+        '',
+        `Run: \`${cmd}\``,
+        '',
+        'Return the full diff content. Include changed file paths and line numbers.',
+    ].join('\n');
+}
+function buildSurfacePrompt(diff) {
+    return [
+        'Analyze the changed surface from this diff:',
+        '',
+        diff.slice(0, 5000),
+        '',
+        'Report:',
+        '- Files changed (list each path)',
+        '- Lines added / removed per file',
+        '- Scope assessment (localized vs wide-ranging)',
+    ].join('\n');
+}
+function buildSpecCompliancePrompt(diff, _planPath) {
+    return [
+        'Compare implementation to requirements.',
+        '',
+        'Read the plan file to understand what was requested.',
+        '',
+        '## Diff',
+        diff.slice(0, 4000),
+        '',
+        'Check: Does the implementation satisfy the plan requirements?',
+        'Return: pass, partial, or fail with specific violations listed.',
+    ].join('\n');
+}
+function buildQualityReviewPrompt(diff, focusAreas) {
+    const focusText = focusAreas.length > 0 ? focusAreas.join(', ') : 'all aspects';
+    return [
+        `Review code quality focusing on: ${focusText}`,
+        '',
+        '## Diff',
+        diff.slice(0, 5000),
+        '',
+        '## Check Each Category',
+        '',
+        '- **Correctness**: Logic errors, edge cases, off-by-one',
+        '- **Security**: Injection risks, auth bypass, secrets in code, input validation',
+        '- **Performance**: N+1 queries, unnecessary allocations, blocking calls',
+        '- **Style**: Naming conventions, consistency with codebase, immutability',
+        '- **Error Handling**: Caught errors, graceful degradation, user-facing messages',
+        '',
+        '## Output Format',
+        '',
+        'For each issue found, output one line:',
+        '`SEVERITY|FILE|LINE|DESCRIPTION|SUGGESTED_ACTION`',
+        '',
+        'Severity values: BLOCKER, MAJOR, MINOR, NOTE',
+        'BLOCKER = must fix before merge (security hole, broken build, data loss)',
+        'MAJOR = should fix (bug, performance problem, missing error handling)',
+        'MINOR = nice to fix (style, naming, minor duplication)',
+        'NOTE = observation, not a problem',
+    ].join('\n');
+}
+function parseIssues(output) {
+    const issues = [];
+    const lines = output.split('\n');
+    for (const line of lines) {
+        const match = line.match(/^(BLOCKER|MAJOR|MINOR|NOTE)\s*\|\s*([^|]*)\s*\|\s*(\d+)\s*\|\s*([^|]+)\s*\|\s*(.+)$/i);
+        if (match) {
+            const file = match[2]?.trim();
+            issues.push({
+                severity: match[1].toLowerCase(),
+                file: file || undefined,
+                line: parseInt(match[3], 10),
+                description: match[4].trim(),
+                action: match[5].trim(),
+            });
+        }
+    }
+    return issues;
+}
+function determineDecision(issues) {
+    if (issues.some((i) => i.severity === 'blocker'))
+        return 'blocked';
+    if (issues.some((i) => i.severity === 'major'))
+        return 'changes';
+    return 'approve';
+}
+function assessTests(output) {
+    const lower = output.toLowerCase();
+    if (lower.includes('missing') || lower.includes('no test'))
+        return 'missing';
+    if (lower.includes('fail'))
+        return 'failing';
+    return 'adequate';
+}
+function assessSpecCompliance(output) {
+    const lower = output.toLowerCase();
+    if (lower.includes('fail'))
+        return 'fail';
+    if (lower.includes('partial'))
+        return 'partial';
+    return 'pass';
+}
+// ---------------------------------------------------------------------------
+// Skill class
+// ---------------------------------------------------------------------------
+export class ReviewDiffSkill extends Skill {
+    constructor() {
+        super({
+            name: 'review-diff',
+            description: 'Review code changes (diff/branch/PR) with reviewer subagents. Returns structured report.',
+            requires: [],
+            inputSchema: ReviewDiffInputSchema,
+            outputSchema: ReviewDiffOutputSchema,
+        });
+    }
+    async execute(input, context) {
+        const { config, logger } = context;
+        const dispatcher = createDispatcher(logger);
+        const guard = createMainAgentGuard({}, logger);
+        guard.activateEmbargo();
+        let totalTokens = 0;
+        try {
+            // Step 1: Get diff
+            logger.info(`[review-diff] Getting diff for ${input.targetType}: ${input.target}`);
+            const diffResult = await dispatcher.dispatch({
+                role: 'explorer',
+                model: input.model || config.defaultModel,
+            }, {
+                permissions: {
+                    readFiles: true,
+                    searchCode: false,
+                    runCommands: true,
+                    writeFiles: false,
+                    gitOperations: true,
+                },
+                prompt: buildDiffPrompt(input.targetType, input.target),
+                owns: [],
+                reads: [],
+            });
+            totalTokens += diffResult.tokensUsed;
+            if (diffResult.status !== 'success') {
+                return {
+                    decision: 'needs-evidence',
+                    issues: [
+                        {
+                            severity: 'blocker',
+                            description: 'Cannot access diff',
+                            action: 'Verify target exists and is accessible',
+                        },
+                    ],
+                    specCompliance: 'fail',
+                    testAssessment: 'missing',
+                    strengths: [],
+                    summary: 'Cannot retrieve diff for review',
+                    tokensUsed: totalTokens,
+                };
+            }
+            const diffContent = diffResult.output;
+            // Step 2: Inspect changed surface
+            logger.info('[review-diff] Inspecting changed surface');
+            const surfaceResult = await dispatcher.dispatch({
+                role: 'explorer',
+                model: input.model || config.defaultModel,
+            }, {
+                permissions: {
+                    readFiles: true,
+                    searchCode: false,
+                    runCommands: false,
+                    writeFiles: false,
+                    gitOperations: false,
+                },
+                prompt: buildSurfacePrompt(diffContent),
+                owns: [],
+                reads: [],
+            });
+            totalTokens += surfaceResult.tokensUsed;
+            // Step 3: Check requirement compliance (if plan provided)
+            let specOutput = 'pass';
+            if (input.planPath) {
+                logger.info('[review-diff] Checking spec compliance');
+                const specResult = await dispatcher.dispatch({
+                    role: 'reviewer',
+                    model: input.model || config.defaultModel,
+                }, {
+                    permissions: {
+                        readFiles: true,
+                        searchCode: false,
+                        runCommands: false,
+                        writeFiles: false,
+                        gitOperations: false,
+                    },
+                    prompt: buildSpecCompliancePrompt(diffContent, input.planPath),
+                    owns: [],
+                    reads: [input.planPath],
+                });
+                totalTokens += specResult.tokensUsed;
+                specOutput = specResult.output;
+            }
+            // Step 4: Check tests
+            logger.info('[review-diff] Assessing test coverage');
+            const testResult = await dispatcher.dispatch({
+                role: 'reviewer',
+                model: input.model || config.defaultModel,
+            }, {
+                permissions: {
+                    readFiles: true,
+                    searchCode: false,
+                    runCommands: true,
+                    writeFiles: false,
+                    gitOperations: false,
+                },
+                prompt: [
+                    'Check test coverage in the changed code:',
+                    '',
+                    diffContent.slice(0, 4000),
+                    '',
+                    'Do changed files have corresponding test changes?',
+                    'Are new functions/classes covered by tests?',
+                    'Return: adequate, missing, or failing with details.',
+                ].join('\n'),
+                owns: [],
+                reads: [],
+            });
+            totalTokens += testResult.tokensUsed;
+            // Step 5: Review code quality
+            logger.info('[review-diff] Reviewing code quality');
+            const qualityResult = await dispatcher.dispatch({
+                role: 'reviewer',
+                model: input.model || config.defaultModel,
+            }, {
+                permissions: {
+                    readFiles: true,
+                    searchCode: true,
+                    runCommands: false,
+                    writeFiles: false,
+                    gitOperations: false,
+                },
+                prompt: buildQualityReviewPrompt(diffContent, input.focus ?? []),
+                owns: [],
+                reads: [],
+            });
+            totalTokens += qualityResult.tokensUsed;
+            // Step 6: Compile final report
+            const issues = parseIssues(qualityResult.output);
+            return {
+                decision: determineDecision(issues),
+                issues,
+                specCompliance: assessSpecCompliance(specOutput),
+                testAssessment: assessTests(testResult.output),
+                strengths: ['Reviewed all changed files'],
+                summary: `Review complete: ${issues.length} issue(s) found (${issues.filter((i) => i.severity === 'blocker').length} blocker, ${issues.filter((i) => i.severity === 'major').length} major)`,
+                tokensUsed: totalTokens,
+            };
+        }
+        finally {
+            guard.deactivateEmbargo();
+        }
+    }
+}
+export const reviewDiffSkill = new ReviewDiffSkill();
+export default reviewDiffSkill;
+//# sourceMappingURL=index.js.map

package/dist/src/skills/review-diff/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../src/skills/review-diff/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAA;AAEnC,OAAO,EAAE,gBAAgB,EAAE,MAAM,4BAA4B,CAAA;AAC7D,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAA;AAEhE,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,MAAM,qBAAqB,GAAG,CAAC,CAAC,MAAM,CAAC;IACrC,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IACzB,UAAU,EAAE,CAAC;SACV,IAAI,CAAC,CAAC,IAAI,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC;SAC7C,OAAO,CAAC,QAAQ,CAAC;IACpB,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,KAAK,EAAE,CAAC;SACL,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,aAAa,EAAE,aAAa,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;SAC3E,QAAQ,EAAE;IACb,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;CAC7B,CAAC,CAAA;AAEF,MAAM,sBAAsB,GAAG,CAAC,CAAC,MAAM,CAAC;IACtC,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,SAAS,EAAE,SAAS,EAAE,gBAAgB,CAAC,CAAC;IACrE,MAAM,EAAE,CAAC,CAAC,KAAK,CACb,CAAC,CAAC,MAAM,CAAC;QACP,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,CAAC,CAAC;QACvD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAC3B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAC3B,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE;QACvB,MAAM,EAAE,CAAC,CAAC,MAAM,EAAE;KACnB,CAAC,CACH;IACD,cAAc,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC;IACnD,cAAc,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,SAAS,EAAE,SAAS,CAAC,CAAC;IAC1D,SAAS,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC;IAC9B,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE;IACnB,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE;CACvB,CAAC,CAAA;AAKF,8EAA8E;AAC9E,mCAAmC;AACnC,8EAA8E;AAE9E,SAAS,eAAe,CAAC,UAAkB,EAAE,MAAc;IACzD,MAAM,QAAQ,GAA2B;QACvC,EAAE,EAAE,cAAc,MAAM,EAAE;QAC1B,MAAM,EAAE,mBAAmB,MAAM,SAAS;QAC1C,MAAM,EAAE,YAAY,MAAM,EAAE;QAC5B,WAAW,EAAE,OAAO,MAAM,EAAE;KAC7B,CAAA;IACD,MAAM,GAAG,GAAG,QAAQ,CAAC,UAAU,CAAC,IAAI,YAAY,MAAM,EAAE,CAAA;IACxD,OAAO;QACL,oBAAoB,UAAU,KAAK,MAAM,EAAE;QAC3C,EAAE;QACF,UAAU,GAAG,IAAI;QACjB,EAAE;QACF,4EAA4E;KAC7E,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACd,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY;IACtC,OAAO;QACL,6CAA6C;QAC7C,EAAE;QACF,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC;QACnB,EAAE;QACF,SAAS;QACT,kCAAkC;QAClC,kCAAkC;QAClC,gDAAgD;KACjD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACd,CAAC;AAED,SAAS,yBAAyB,CAChC,IAAY,EACZ,SAAiB;IAEjB,OAAO;QACL,yCAAyC;QACzC,EAAE;QACF,sDAAsD;QACtD,EAAE;QACF,SAAS;QACT,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC;QACnB,EAAE;QACF,+DAA+D;QAC/D,iEAAiE;KAClE,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACd,CAAC;AAED,SAAS,wBAAwB,CAC/B,IAAY,EACZ,UAAoB;IAEpB,MAAM,SAAS,GACb,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,aAAa,CAAA;IAE/D,OAAO;QACL,oCAAoC,SAAS,EAAE;QAC/C,EAAE;QACF,SAAS;QACT,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC;QACnB,EAAE;QACF,wBAAwB;QACxB,EAAE;QACF,yDAAyD;QACzD,iFAAiF;QACjF,yEAAyE;QACzE,0EAA0E;QAC1E,iFAAiF;QACjF,EAAE;QACF,kBAAkB;QAClB,EAAE;QACF,wCAAwC;QACxC,mDAAmD;QACnD,EAAE;QACF,8CAA8C;QAC9C,0EAA0E;QAC1E,uEAAuE;QACvE,wDAAwD;QACxD,mCAAmC;KACpC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACd,CAAC;AAcD,SAAS,WAAW,CAAC,MAAc;IACjC,MAAM,MAAM,GAAkB,EAAE,CAAA;IAChC,MAAM,KAAK,GAAG,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;IAEhC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CACtB,sFAAsF,CACvF,CAAA;QACD,IAAI,KAAK,EAAE,CAAC;YACV,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAA;YAC7B,MAAM,CAAC,IAAI,CAAC;gBACV,QAAQ,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAA6B;gBAC3D,IAAI,EAAE,IAAI,IAAI,SAAS;gBACvB,IAAI,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC;gBAC5B,WAAW,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE;gBAC5B,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE;aACxB,CAAC,CAAA;QACJ,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC;AAED,SAAS,iBAAiB,CAAC,MAAqB;IAC9C,IAAI,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,SAAS,CAAC;QAAE,OAAO,SAAS,CAAA;IAClE,IAAI,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,OAAO,CAAC;QAAE,OAAO,SAAS,CAAA;IAChE,OAAO,SAAS,CAAA;AAClB,CAAC;AAED,SAAS,WAAW,CAAC,MAAc;IACjC,MAAM,KAAK,GAAG,MAAM,CAAC,WAAW,EAAE,CAAA;IAClC,IAAI,KAAK,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,SAAS,CAAC;QAAE,OAAO,SAAS,CAAA;IAC5E,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC;QAAE,OAAO,SAAS,CAAA;IAC5C,OAAO,UAAU,CAAA;AACnB,CAAC;AAED,SAAS,oBAAoB,CAAC,MAAc;IAC1C,MAAM,KAAK,GAAG,MAAM,CAAC,WAAW,EAAE,CAAA;IAClC,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC;QAAE,OAAO,MAAM,CAAA;IACzC,IAAI,KAAK,CAAC,QAAQ,CAAC,SAAS,CAAC;QAAE,OAAO,SAAS,CAAA;IAC/C,OAAO,MAAM,CAAA;AACf,CAAC;AAED,8EAA8E;AAC9E,cAAc;AACd,8EAA8E;AAE9E,MAAM,OAAO,eAAgB,SAAQ,KAAwC;IAC3E;QACE,KAAK,CAAC;YACJ,IAAI,EAAE,aAAa;YACnB,WAAW,EACT,0FAA0F;YAC5F,QAAQ,EAAE,EAAE;YACZ,WAAW,EAAE,qBAAmD;YAChE,YAAY,EAAE,sBAAsB;SACrC,CAAC,CAAA;IACJ,CAAC;IAED,KAAK,CAAC,OAAO,CACX,KAAsB,EACtB,OAAqB;QAErB,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAA;QAClC,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC,CAAA;QAC3C,MAAM,KAAK,GAAG,oBAAoB,CAAC,EAAE,EAAE,MAAM,CAAC,CAAA;QAE9C,KAAK,CAAC,eAAe,EAAE,CAAA;QACvB,IAAI,WAAW,GAAG,CAAC,CAAA;QAEnB,IAAI,CAAC;YACH,mBAAmB;YACnB,MAAM,CAAC,IAAI,CACT,kCAAkC,KAAK,CAAC,UAAU,KAAK,KAAK,CAAC,MAAM,EAAE,CACtE,CAAA;YACD,MAAM,UAAU,GAAG,MAAM,UAAU,CAAC,QAAQ,CAC1C;gBACE,IAAI,EAAE,UAAU;gBAChB,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,MAAM,CAAC,YAAY;aAC1C,EACD;gBACE,WAAW,EAAE;oBACX,SAAS,EAAE,IAAI;oBACf,UAAU,EAAE,KAAK;oBACjB,WAAW,EAAE,IAAI;oBACjB,UAAU,EAAE,KAAK;oBACjB,aAAa,EAAE,IAAI;iBACpB;gBACD,MAAM,EAAE,eAAe,CAAC,KAAK,CAAC,UAAU,EAAE,KAAK,CAAC,MAAM,CAAC;gBACvD,IAAI,EAAE,EAAE;gBACR,KAAK,EAAE,EAAE;aACV,CACF,CAAA;YAED,WAAW,IAAI,UAAU,CAAC,UAAU,CAAA;YAEpC,IAAI,UAAU,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;gBACpC,OAAO;oBACL,QAAQ,EAAE,gBAAgB;oBAC1B,MAAM,EAAE;wBACN;4BACE,QAAQ,EAAE,SAAS;4BACnB,WAAW,EAAE,oBAAoB;4BACjC,MAAM,EAAE,wCAAwC;yBACjD;qBACF;oBACD,cAAc,EAAE,MAAM;oBACtB,cAAc,EAAE,SAAS;oBACzB,SAAS,EAAE,EAAE;oBACb,OAAO,EAAE,iCAAiC;oBAC1C,UAAU,EAAE,WAAW;iBACxB,CAAA;YACH,CAAC;YAED,MAAM,WAAW,GAAG,UAAU,CAAC,MAAM,CAAA;YAErC,kCAAkC;YAClC,MAAM,CAAC,IAAI,CAAC,0CAA0C,CAAC,CAAA;YACvD,MAAM,aAAa,GAAG,MAAM,UAAU,CAAC,QAAQ,CAC7C;gBACE,IAAI,EAAE,UAAU;gBAChB,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,MAAM,CAAC,YAAY;aAC1C,EACD;gBACE,WAAW,EAAE;oBACX,SAAS,EAAE,IAAI;oBACf,UAAU,EAAE,KAAK;oBACjB,WAAW,EAAE,KAAK;oBAClB,UAAU,EAAE,KAAK;oBACjB,aAAa,EAAE,KAAK;iBACrB;gBACD,MAAM,EAAE,kBAAkB,CAAC,WAAW,CAAC;gBACvC,IAAI,EAAE,EAAE;gBACR,KAAK,EAAE,EAAE;aACV,CACF,CAAA;YAED,WAAW,IAAI,aAAa,CAAC,UAAU,CAAA;YAEvC,0DAA0D;YAC1D,IAAI,UAAU,GAAG,MAAM,CAAA;YACvB,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;gBACnB,MAAM,CAAC,IAAI,CAAC,wCAAwC,CAAC,CAAA;gBACrD,MAAM,UAAU,GAAG,MAAM,UAAU,CAAC,QAAQ,CAC1C;oBACE,IAAI,EAAE,UAAU;oBAChB,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,MAAM,CAAC,YAAY;iBAC1C,EACD;oBACE,WAAW,EAAE;wBACX,SAAS,EAAE,IAAI;wBACf,UAAU,EAAE,KAAK;wBACjB,WAAW,EAAE,KAAK;wBAClB,UAAU,EAAE,KAAK;wBACjB,aAAa,EAAE,KAAK;qBACrB;oBACD,MAAM,EAAE,yBAAyB,CAAC,WAAW,EAAE,KAAK,CAAC,QAAQ,CAAC;oBAC9D,IAAI,EAAE,EAAE;oBACR,KAAK,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC;iBACxB,CACF,CAAA;gBAED,WAAW,IAAI,UAAU,CAAC,UAAU,CAAA;gBACpC,UAAU,GAAG,UAAU,CAAC,MAAM,CAAA;YAChC,CAAC;YAED,sBAAsB;YACtB,MAAM,CAAC,IAAI,CAAC,uCAAuC,CAAC,CAAA;YACpD,MAAM,UAAU,GAAG,MAAM,UAAU,CAAC,QAAQ,CAC1C;gBACE,IAAI,EAAE,UAAU;gBAChB,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,MAAM,CAAC,YAAY;aAC1C,EACD;gBACE,WAAW,EAAE;oBACX,SAAS,EAAE,IAAI;oBACf,UAAU,EAAE,KAAK;oBACjB,WAAW,EAAE,IAAI;oBACjB,UAAU,EAAE,KAAK;oBACjB,aAAa,EAAE,KAAK;iBACrB;gBACD,MAAM,EAAE;oBACN,0CAA0C;oBAC1C,EAAE;oBACF,WAAW,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC;oBAC1B,EAAE;oBACF,mDAAmD;oBACnD,6CAA6C;oBAC7C,qDAAqD;iBACtD,CAAC,IAAI,CAAC,IAAI,CAAC;gBACZ,IAAI,EAAE,EAAE;gBACR,KAAK,EAAE,EAAE;aACV,CACF,CAAA;YAED,WAAW,IAAI,UAAU,CAAC,UAAU,CAAA;YAEpC,8BAA8B;YAC9B,MAAM,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAA;YACnD,MAAM,aAAa,GAAG,MAAM,UAAU,CAAC,QAAQ,CAC7C;gBACE,IAAI,EAAE,UAAU;gBAChB,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,MAAM,CAAC,YAAY;aAC1C,EACD;gBACE,WAAW,EAAE;oBACX,SAAS,EAAE,IAAI;oBACf,UAAU,EAAE,IAAI;oBAChB,WAAW,EAAE,KAAK;oBAClB,UAAU,EAAE,KAAK;oBACjB,aAAa,EAAE,KAAK;iBACrB;gBACD,MAAM,EAAE,wBAAwB,CAC9B,WAAW,EACX,KAAK,CAAC,KAAK,IAAI,EAAE,CAClB;gBACD,IAAI,EAAE,EAAE;gBACR,KAAK,EAAE,EAAE;aACV,CACF,CAAA;YAED,WAAW,IAAI,aAAa,CAAC,UAAU,CAAA;YAEvC,+BAA+B;YAC/B,MAAM,MAAM,GAAG,WAAW,CAAC,aAAa,CAAC,MAAM,CAAC,CAAA;YAEhD,OAAO;gBACL,QAAQ,EAAE,iBAAiB,CAAC,MAAM,CAAC;gBACnC,MAAM;gBACN,cAAc,EAAE,oBAAoB,CAAC,UAAU,CAAC;gBAChD,cAAc,EAAE,WAAW,CAAC,UAAU,CAAC,MAAM,CAAC;gBAC9C,SAAS,EAAE,CAAC,4BAA4B,CAAC;gBACzC,OAAO,EAAE,oBAAoB,MAAM,CAAC,MAAM,oBAAoB,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,MAAM,aAAa,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,MAAM,SAAS;gBAC5L,UAAU,EAAE,WAAW;aACxB,CAAA;QACH,CAAC;gBAAS,CAAC;YACT,KAAK,CAAC,iBAAiB,EAAE,CAAA;QAC3B,CAAC;IACH,CAAC;CACF;AAED,MAAM,CAAC,MAAM,eAAe,GAAG,IAAI,eAAe,EAAE,CAAA;AACpD,eAAe,eAAe,CAAA"}

package/dist/src/skills/skill.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Base skill class with validation and error handling
+ */
+import { z } from 'zod';
+import type { SkillDefinition, SkillContext } from './types.js';
+export declare abstract class Skill<Input, Output> {
+    protected name: string;
+    protected description: string;
+    protected requires: string[];
+    protected inputSchema?: z.ZodSchema<Input>;
+    protected outputSchema?: z.ZodSchema<Output>;
+    constructor(options: {
+        name: string;
+        description: string;
+        requires?: string[];
+        inputSchema?: z.ZodSchema<Input>;
+        outputSchema?: z.ZodSchema<Output>;
+    });
+    abstract execute(input: Input, context: SkillContext): Promise<Output>;
+    validateInput(input: unknown): Promise<Input>;
+    validateOutput(output: unknown): Promise<Output>;
+    toDefinition(): SkillDefinition<any, any>;
+}
+//# sourceMappingURL=skill.d.ts.map

package/dist/src/skills/skill.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"skill.d.ts","sourceRoot":"","sources":["../../../src/skills/skill.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,KAAK,EACV,eAAe,EACf,YAAY,EACb,MAAM,YAAY,CAAA;AAEnB,8BAAsB,KAAK,CAAC,KAAK,EAAE,MAAM;IACvC,SAAS,CAAC,IAAI,EAAE,MAAM,CAAA;IACtB,SAAS,CAAC,WAAW,EAAE,MAAM,CAAA;IAC7B,SAAS,CAAC,QAAQ,EAAE,MAAM,EAAE,CAAA;IAC5B,SAAS,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAA;IAC1C,SAAS,CAAC,YAAY,CAAC,EAAE,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAA;gBAEhC,OAAO,EAAE;QACnB,IAAI,EAAE,MAAM,CAAA;QACZ,WAAW,EAAE,MAAM,CAAA;QACnB,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAA;QACnB,WAAW,CAAC,EAAE,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAA;QAChC,YAAY,CAAC,EAAE,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAA;KACnC;IAQD,QAAQ,CAAC,OAAO,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,MAAM,CAAC;IAEhE,aAAa,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAAC,KAAK,CAAC;IAO7C,cAAc,CAAC,MAAM,EAAE,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC;IAOtD,YAAY,IAAI,eAAe,CAAC,GAAG,EAAE,GAAG,CAAC;CAY1C"}

package/dist/src/skills/skill.js

@@ -0,0 +1,39 @@
+export class Skill {
+    name;
+    description;
+    requires;
+    inputSchema;
+    outputSchema;
+    constructor(options) {
+        this.name = options.name;
+        this.description = options.description;
+        this.requires = options.requires || [];
+        this.inputSchema = options.inputSchema;
+        this.outputSchema = options.outputSchema;
+    }
+    async validateInput(input) {
+        if (this.inputSchema) {
+            return this.inputSchema.parse(input);
+        }
+        return input;
+    }
+    async validateOutput(output) {
+        if (this.outputSchema) {
+            return this.outputSchema.parse(output);
+        }
+        return output;
+    }
+    toDefinition() {
+        return {
+            name: this.name,
+            description: this.description,
+            requires: this.requires,
+            execute: async (input, context) => {
+                const validatedInput = await this.validateInput(input);
+                const result = await this.execute(validatedInput, context);
+                return await this.validateOutput(result);
+            },
+        };
+    }
+}
+//# sourceMappingURL=skill.js.map

package/dist/src/skills/skill.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"skill.js","sourceRoot":"","sources":["../../../src/skills/skill.ts"],"names":[],"mappings":"AASA,MAAM,OAAgB,KAAK;IACf,IAAI,CAAQ;IACZ,WAAW,CAAQ;IACnB,QAAQ,CAAU;IAClB,WAAW,CAAqB;IAChC,YAAY,CAAsB;IAE5C,YAAY,OAMX;QACC,IAAI,CAAC,IAAI,GAAG,OAAO,CAAC,IAAI,CAAA;QACxB,IAAI,CAAC,WAAW,GAAG,OAAO,CAAC,WAAW,CAAA;QACtC,IAAI,CAAC,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,EAAE,CAAA;QACtC,IAAI,CAAC,WAAW,GAAG,OAAO,CAAC,WAAW,CAAA;QACtC,IAAI,CAAC,YAAY,GAAG,OAAO,CAAC,YAAY,CAAA;IAC1C,CAAC;IAID,KAAK,CAAC,aAAa,CAAC,KAAc;QAChC,IAAI,IAAI,CAAC,WAAW,EAAE,CAAC;YACrB,OAAO,IAAI,CAAC,WAAW,CAAC,KAAK,CAAC,KAAK,CAAC,CAAA;QACtC,CAAC;QACD,OAAO,KAAc,CAAA;IACvB,CAAC;IAED,KAAK,CAAC,cAAc,CAAC,MAAe;QAClC,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;YACtB,OAAO,IAAI,CAAC,YAAY,CAAC,KAAK,CAAC,MAAM,CAAC,CAAA;QACxC,CAAC;QACD,OAAO,MAAgB,CAAA;IACzB,CAAC;IAED,YAAY;QACV,OAAO;YACL,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,WAAW,EAAE,IAAI,CAAC,WAAW;YAC7B,QAAQ,EAAE,IAAI,CAAC,QAAQ;YACvB,OAAO,EAAE,KAAK,EAAE,KAAU,EAAE,OAAqB,EAAE,EAAE;gBACnD,MAAM,cAAc,GAAG,MAAM,IAAI,CAAC,aAAa,CAAC,KAAK,CAAC,CAAA;gBACtD,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,OAAO,CAAC,cAAc,EAAE,OAAO,CAAC,CAAA;gBAC1D,OAAO,MAAM,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,CAAA;YAC1C,CAAC;SACF,CAAA;IACH,CAAC;CACF"}

package/dist/src/skills/systematic-debugging/index.d.ts

@@ -0,0 +1,90 @@
+/**
+ * Systematic Debugging Skill - Find root cause before fixing
+ *
+ * Use case: Bug investigation, test failure, unexpected behavior
+ * Pattern: Reproduce → Gather Evidence → Diagnose (hypothesize + test) →
+ *          Apply Fix → Verify
+ *
+ * CRITICAL RULE: No fix before root cause is confirmed.
+ * Spawns debugger subagent(s) for each phase.
+ * Prevents the "fix-first-ask-questions-later" anti-pattern.
+ */
+import { z } from 'zod';
+import { Skill } from '../skill.js';
+import type { SkillContext } from '../types.js';
+declare const DebugInputSchema: z.ZodObject<{
+    symptom: z.ZodString;
+    reproduceCommand: z.ZodOptional<z.ZodString>;
+    logs: z.ZodOptional<z.ZodString>;
+    recentChanges: z.ZodOptional<z.ZodString>;
+    model: z.ZodOptional<z.ZodString>;
+}, "strip", z.ZodTypeAny, {
+    symptom: string;
+    model?: string | undefined;
+    reproduceCommand?: string | undefined;
+    logs?: string | undefined;
+    recentChanges?: string | undefined;
+}, {
+    symptom: string;
+    model?: string | undefined;
+    reproduceCommand?: string | undefined;
+    logs?: string | undefined;
+    recentChanges?: string | undefined;
+}>;
+declare const DebugOutputSchema: z.ZodObject<{
+    rootCause: z.ZodString;
+    confidence: z.ZodEnum<["high", "medium", "low"]>;
+    location: z.ZodObject<{
+        file: z.ZodOptional<z.ZodString>;
+        line: z.ZodOptional<z.ZodNumber>;
+        function: z.ZodOptional<z.ZodString>;
+    }, "strip", z.ZodTypeAny, {
+        function?: string | undefined;
+        file?: string | undefined;
+        line?: number | undefined;
+    }, {
+        function?: string | undefined;
+        file?: string | undefined;
+        line?: number | undefined;
+    }>;
+    fixApplied: z.ZodBoolean;
+    fixDescription: z.ZodOptional<z.ZodString>;
+    verification: z.ZodString;
+    risks: z.ZodArray<z.ZodString, "many">;
+    tokensUsed: z.ZodNumber;
+}, "strip", z.ZodTypeAny, {
+    tokensUsed: number;
+    rootCause: string;
+    confidence: "low" | "medium" | "high";
+    verification: string;
+    risks: string[];
+    location: {
+        function?: string | undefined;
+        file?: string | undefined;
+        line?: number | undefined;
+    };
+    fixApplied: boolean;
+    fixDescription?: string | undefined;
+}, {
+    tokensUsed: number;
+    rootCause: string;
+    confidence: "low" | "medium" | "high";
+    verification: string;
+    risks: string[];
+    location: {
+        function?: string | undefined;
+        file?: string | undefined;
+        line?: number | undefined;
+    };
+    fixApplied: boolean;
+    fixDescription?: string | undefined;
+}>;
+type DebugInput = z.infer<typeof DebugInputSchema>;
+type DebugOutput = z.infer<typeof DebugOutputSchema>;
+export declare class SystematicDebuggingSkill extends Skill<DebugInput, DebugOutput> {
+    constructor();
+    execute(input: DebugInput, context: SkillContext): Promise<DebugOutput>;
+}
+export declare const systematicDebuggingSkill: SystematicDebuggingSkill;
+export default systematicDebuggingSkill;
+//# sourceMappingURL=index.d.ts.map

package/dist/src/skills/systematic-debugging/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/skills/systematic-debugging/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,EAAE,KAAK,EAAE,MAAM,aAAa,CAAA;AACnC,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,aAAa,CAAA;AAS/C,QAAA,MAAM,gBAAgB;;;;;;;;;;;;;;;;;;EAMpB,CAAA;AAEF,QAAA,MAAM,iBAAiB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAarB,CAAA;AAEF,KAAK,UAAU,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,gBAAgB,CAAC,CAAA;AAClD,KAAK,WAAW,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,iBAAiB,CAAC,CAAA;AA6KpD,qBAAa,wBAAyB,SAAQ,KAAK,CAAC,UAAU,EAAE,WAAW,CAAC;;IAYpE,OAAO,CACX,KAAK,EAAE,UAAU,EACjB,OAAO,EAAE,YAAY,GACpB,OAAO,CAAC,WAAW,CAAC;CAqKxB;AAED,eAAO,MAAM,wBAAwB,0BAAiC,CAAA;AACtE,eAAe,wBAAwB,CAAA"}