codingbuddy 5.2.0 → 5.4.0

package/dist/src/shared/rules-core.d.ts

@@ -24,6 +24,11 @@ export declare function listSkillSummaries(rulesDir: string, deps?: FileSystemDe
     name: string;
     description: string;
     triggers?: SkillFrontmatterTrigger[];
+    userInvocable?: boolean;
+    disableModelInvocation?: boolean;
+    context?: string;
+    agent?: string;
+    allowedTools?: string[];
 }>>;
 export declare function loadSkill(rulesDir: string, name: string, deps?: FileSystemDeps): Promise<Skill>;
 export {};

package/dist/src/shared/rules-core.js

@@ -133,6 +133,11 @@ async function listSkillSummaries(rulesDir, deps) {
                         name: skill.name,
                         description: skill.description,
                         triggers: skill.triggers,
+                        userInvocable: skill.userInvocable,
+                        disableModelInvocation: skill.disableModelInvocation,
+                        context: skill.context,
+                        agent: skill.agent,
+                        allowedTools: skill.allowedTools,
                     });
                 }
                 catch {

package/dist/src/shared/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "5.2.0";
+export declare const VERSION = "5.4.0";

package/dist/src/shared/version.js

@@ -1,5 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.VERSION = void 0;
-exports.VERSION = '5.2.0';
+exports.VERSION = '5.4.0';
 //# sourceMappingURL=version.js.map

package/dist/src/ship/review-pr.service.d.ts

@@ -0,0 +1,15 @@
+import { ChecklistService } from '../checklist/checklist.service';
+import type { ReviewPrInput, ReviewPrResult } from './review-pr.types';
+export declare class ReviewPrService {
+    private readonly checklistService;
+    private readonly logger;
+    private readonly mapper;
+    constructor(checklistService: ChecklistService);
+    reviewPr(input: ReviewPrInput): Promise<ReviewPrResult>;
+    private fetchPrMeta;
+    private fetchPrDiff;
+    private generateChecklists;
+    private mapSpecialists;
+    private fetchIssueCriteria;
+    private buildReviewProtocol;
+}

package/dist/src/ship/review-pr.service.js

@@ -0,0 +1,136 @@
+"use strict";
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+var __metadata = (this && this.__metadata) || function (k, v) {
+    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
+};
+var ReviewPrService_1;
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ReviewPrService = void 0;
+const common_1 = require("@nestjs/common");
+const child_process_1 = require("child_process");
+const file_specialist_mapper_1 = require("./file-specialist-mapper");
+const checklist_service_1 = require("../checklist/checklist.service");
+let ReviewPrService = ReviewPrService_1 = class ReviewPrService {
+    constructor(checklistService) {
+        this.checklistService = checklistService;
+        this.logger = new common_1.Logger(ReviewPrService_1.name);
+        this.mapper = new file_specialist_mapper_1.FileSpecialistMapper();
+    }
+    async reviewPr(input) {
+        const start = Date.now();
+        const { prNumber, issueNumber } = input;
+        const prMeta = this.fetchPrMeta(prNumber);
+        const diff = this.fetchPrDiff(prNumber);
+        const checklists = await this.generateChecklists(prMeta.changedFiles);
+        const recommendedSpecialists = this.mapSpecialists(prMeta.changedFiles);
+        const acceptanceCriteria = issueNumber ? this.fetchIssueCriteria(issueNumber) : null;
+        const reviewProtocol = this.buildReviewProtocol(prNumber, issueNumber);
+        return {
+            prMeta,
+            diff,
+            checklists,
+            recommendedSpecialists,
+            acceptanceCriteria,
+            reviewProtocol,
+            duration: Date.now() - start,
+        };
+    }
+    fetchPrMeta(prNumber) {
+        try {
+            const raw = (0, child_process_1.execSync)(`gh pr view ${prNumber} --json title,headRefName,files,additions,deletions`, { encoding: 'utf-8', timeout: 15000 });
+            const data = JSON.parse(raw);
+            return {
+                title: data.title ?? '',
+                branch: data.headRefName ?? '',
+                changedFiles: (data.files ?? []).map((f) => f.path),
+                additions: data.additions ?? 0,
+                deletions: data.deletions ?? 0,
+            };
+        }
+        catch (error) {
+            throw new Error(`Failed to fetch PR #${prNumber}: ${error instanceof Error ? error.message : String(error)}`);
+        }
+    }
+    fetchPrDiff(prNumber) {
+        try {
+            return (0, child_process_1.execSync)(`gh pr diff ${prNumber}`, {
+                encoding: 'utf-8',
+                timeout: 15000,
+                maxBuffer: 10 * 1024 * 1024,
+            });
+        }
+        catch (error) {
+            this.logger.warn(`Failed to fetch diff for PR #${prNumber}: ${error}`);
+            return '';
+        }
+    }
+    async generateChecklists(changedFiles) {
+        if (changedFiles.length === 0)
+            return [];
+        try {
+            const result = await this.checklistService.generateChecklist({
+                files: changedFiles,
+            });
+            return result.checklists;
+        }
+        catch (error) {
+            this.logger.warn(`Checklist generation failed: ${error}`);
+            return [];
+        }
+    }
+    mapSpecialists(changedFiles) {
+        const mapped = this.mapper.mapFiles(changedFiles);
+        const specialists = [...new Set(mapped.map(m => m.specialist))];
+        return specialists;
+    }
+    fetchIssueCriteria(issueNumber) {
+        try {
+            const raw = (0, child_process_1.execSync)(`gh issue view ${issueNumber} --json body`, {
+                encoding: 'utf-8',
+                timeout: 10000,
+            });
+            const data = JSON.parse(raw);
+            const body = data.body ?? '';
+            const criteriaMatch = body.match(/## Acceptance criteria\s*\n([\s\S]*?)(?=\n## |\n---|$)/i);
+            return criteriaMatch ? criteriaMatch[1].trim() : body.slice(0, 2000);
+        }
+        catch {
+            return null;
+        }
+    }
+    buildReviewProtocol(prNumber, issueNumber) {
+        const issueCmd = issueNumber ? `\ngh issue view ${issueNumber}` : '';
+        return [
+            'Follow the canonical PR review cycle (pr-review-cycle.md):',
+            '',
+            '1. CI GATE (BLOCKING): gh pr checks ' + prNumber,
+            '2. LOCAL VERIFICATION: yarn lint && yarn type-check',
+            '3. READ THE DIFF: gh pr diff ' + prNumber,
+            '4. CODE QUALITY SCAN: unused imports, any types, missing error handling, dead code',
+            '5. SPEC COMPLIANCE: ' + (issueNumber ? `gh issue view ${issueNumber}` : 'N/A'),
+            '6. TEST COVERAGE: new logic tested? edge cases covered?',
+            '7. WRITE REVIEW: gh pr review ' + prNumber + ' --comment --body "<structured review>"',
+            '',
+            'Severity scale: critical / high / medium / low (see severity-classification.md)',
+            'Approval gate: Critical=0 AND High=0',
+            '',
+            'Commands:',
+            `gh pr checks ${prNumber}`,
+            `gh pr diff ${prNumber}`,
+            issueCmd,
+        ]
+            .filter(Boolean)
+            .join('\n');
+    }
+};
+exports.ReviewPrService = ReviewPrService;
+exports.ReviewPrService = ReviewPrService = ReviewPrService_1 = __decorate([
+    (0, common_1.Injectable)(),
+    __metadata("design:paramtypes", [checklist_service_1.ChecklistService])
+], ReviewPrService);
+//# sourceMappingURL=review-pr.service.js.map

package/dist/src/ship/review-pr.types.d.ts

@@ -0,0 +1,21 @@
+export interface ReviewPrInput {
+    prNumber: number;
+    issueNumber?: number;
+    timeout?: number;
+}
+export interface PrMeta {
+    title: string;
+    branch: string;
+    changedFiles: string[];
+    additions: number;
+    deletions: number;
+}
+export interface ReviewPrResult {
+    prMeta: PrMeta;
+    diff: string;
+    checklists: unknown[];
+    recommendedSpecialists: string[];
+    acceptanceCriteria: string | null;
+    reviewProtocol: string;
+    duration: number;
+}

package/dist/src/ship/review-pr.types.js

@@ -0,0 +1,3 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+//# sourceMappingURL=review-pr.types.js.map

package/dist/src/ship/ship.module.js

@@ -8,14 +8,17 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ShipModule = void 0;
 const common_1 = require("@nestjs/common");
+const checklist_module_1 = require("../checklist/checklist.module");
 const quality_report_service_1 = require("./quality-report.service");
+const review_pr_service_1 = require("./review-pr.service");
 let ShipModule = class ShipModule {
 };
 exports.ShipModule = ShipModule;
 exports.ShipModule = ShipModule = __decorate([
     (0, common_1.Module)({
-        providers: [quality_report_service_1.QualityReportService],
-        exports: [quality_report_service_1.QualityReportService],
+        imports: [checklist_module_1.ChecklistModule],
+        providers: [quality_report_service_1.QualityReportService, review_pr_service_1.ReviewPrService],
+        exports: [quality_report_service_1.QualityReportService, review_pr_service_1.ReviewPrService],
     })
 ], ShipModule);
 //# sourceMappingURL=ship.module.js.map