@klitchevo/code-council 0.2.0 → 0.2.3

package/README.md

@@ -119,7 +119,7 @@ npx @klitchevo/code-council review --help
 
 ## GitHub Actions
 
-Automatically review PRs with multiple AI models. Findings appear as **inline comments** on the exact lines of code.
+Automatically review PRs with multiple AI models. Findings appear as **inline comments** on the exact lines of code. Code fixes use GitHub's **suggestion syntax** for one-click apply. Re-runs automatically clean up old comments.
 
 ### Quick Setup
 

package/dist/{chunk-IVKLQD6M.js → chunk-AEDZOTVA.js}

@@ -237,7 +237,7 @@ function toNumber(val) {
   if (typeof val === "number") return val;
   if (typeof val === "string") {
     const n = parseFloat(val);
-    return isNaN(n) ? null : n;
+    return Number.isNaN(n) ? null : n;
   }
   return null;
 }
@@ -312,4 +312,4 @@ export {
   buildSynthesisUserMessage,
   parseTpsAnalysis
 };
-//# sourceMappingURL=chunk-IVKLQD6M.js.map
+//# sourceMappingURL=chunk-AEDZOTVA.js.map

package/dist/{chunk-SYMFCPGM.js → chunk-HVF7WG6A.js}

@@ -15,7 +15,6 @@ function formatReport(report, format = "markdown") {
       return formatJson(report);
     case "html":
       return formatHtml(report);
-    case "markdown":
     default:
       return formatMarkdown(report);
   }
@@ -396,4 +395,4 @@ export {
   formatter_exports,
   init_formatter
 };
-//# sourceMappingURL=chunk-SYMFCPGM.js.map
+//# sourceMappingURL=chunk-HVF7WG6A.js.map

package/dist/{chunk-KO6UIMSL.js → chunk-ZDBY7AGR.js}

@@ -5,11 +5,11 @@ import {
   buildBatchUserMessage,
   buildSynthesisUserMessage,
   buildUserMessage
-} from "./chunk-IVKLQD6M.js";
+} from "./chunk-AEDZOTVA.js";
 import {
   formatter_exports,
   init_formatter
-} from "./chunk-SYMFCPGM.js";
+} from "./chunk-HVF7WG6A.js";
 import {
   __toCommonJS,
   init_esm_shims
@@ -929,6 +929,7 @@ You MUST respond with valid JSON matching this schema:
         "endLine": 45
       },
       "suggestion": "How to fix or improve",
+      "suggestedCode": "The actual corrected code that should replace the problematic code (if applicable)",
       "rawExcerpt": "The original text from the review that describes this finding",
       "confidence": 0.95
     }
@@ -956,6 +957,7 @@ ${Object.entries(SEVERITY_DESCRIPTIONS).map(([sev, desc]) => `- **${sev}**: ${de
 5. **Set confidence**: Higher (0.8-1.0) for clear, explicit issues; lower (0.5-0.7) for inferred or ambiguous ones
 6. **Don't duplicate**: Each distinct issue should appear once
 7. **Handle empty reviews**: If no issues found, return {"findings": []}
+8. **Include code fixes**: When a fix involves specific code changes, provide the actual corrected code in the suggestedCode field (not explanation, just the code)
 
 ## Important
 
@@ -1060,12 +1062,13 @@ var ExtractionResponseSchema = z2.object({
       description: z2.string(),
       location: z2.object({
         file: z2.string(),
-        line: z2.number().optional(),
-        endLine: z2.number().optional()
-      }).optional(),
-      suggestion: z2.string().optional(),
+        line: z2.number().nullish(),
+        endLine: z2.number().nullish()
+      }).nullish(),
+      suggestion: z2.string().nullish(),
+      suggestedCode: z2.string().nullish(),
       rawExcerpt: z2.string(),
-      confidence: z2.number().min(0).max(1).optional()
+      confidence: z2.number().min(0).max(1).nullish()
     })
   )
 });
@@ -1148,6 +1151,14 @@ function generateFindingId() {
   const random = Math.random().toString(36).substring(2, 8);
   return toFindingId(`finding-${timestamp}-${random}`);
 }
+function repairJson(jsonStr) {
+  let repaired = jsonStr;
+  repaired = repaired.replace(/,(\s*[}\]])/g, "$1");
+  repaired = repaired.replace(/"([^"]*(?:\\"[^"]*)*)"/g, (match) => {
+    return match.replace(/\n/g, "\\n").replace(/\r/g, "\\r").replace(/\t/g, "\\t");
+  });
+  return repaired;
+}
 function parseExtractionResponse(responseText, sourceModel) {
   let jsonStr = responseText.trim();
   if (jsonStr.startsWith("```json")) {
@@ -1159,6 +1170,7 @@ function parseExtractionResponse(responseText, sourceModel) {
     jsonStr = jsonStr.slice(0, -3);
   }
   jsonStr = jsonStr.trim();
+  jsonStr = repairJson(jsonStr);
   const raw = JSON.parse(jsonStr);
   const parsed = ExtractionResponseSchema.parse(raw);
   const now = (/* @__PURE__ */ new Date()).toISOString();
@@ -1172,13 +1184,14 @@ function parseExtractionResponse(responseText, sourceModel) {
       description: f.description,
       location: f.location ? {
         file: f.location.file,
-        line: f.location.line,
-        endLine: f.location.endLine
+        line: f.location.line ?? void 0,
+        endLine: f.location.endLine ?? void 0
       } : void 0,
-      suggestion: f.suggestion,
+      suggestion: f.suggestion ?? void 0,
+      suggestedCode: f.suggestedCode ?? void 0,
       rawExcerpt: f.rawExcerpt,
       extractedAt: now,
-      confidence: f.confidence
+      confidence: f.confidence ?? void 0
     })
   );
 }
@@ -2073,7 +2086,7 @@ async function buildConsensusReport(reviews, consensusClient, options = {}) {
     disagreements: disagreements.length,
     executionMs: report.executionTimeMs
   });
-  const { formatReport } = await import("./formatter-D42TRSLL.js");
+  const { formatReport } = await import("./formatter-FIH7J57R.js");
   const formatted = formatReport(report, outputFormat);
   return { report, formatted };
 }
@@ -2217,4 +2230,4 @@ export {
   gitReviewSchema,
   handleGitReview
 };
-//# sourceMappingURL=chunk-KO6UIMSL.js.map
+//# sourceMappingURL=chunk-ZDBY7AGR.js.map

package/dist/{cli-M7YIR4XR.js → cli-PHU3RI5B.js}

@@ -5,9 +5,9 @@ import {
   formatForHostExtraction,
   handleGitReview,
   initializeConfig
-} from "./chunk-KO6UIMSL.js";
-import "./chunk-IVKLQD6M.js";
-import "./chunk-SYMFCPGM.js";
+} from "./chunk-ZDBY7AGR.js";
+import "./chunk-AEDZOTVA.js";
+import "./chunk-HVF7WG6A.js";
 import {
   init_esm_shims
 } from "./chunk-UFR2SVK2.js";
@@ -43,7 +43,7 @@ function parseUnifiedDiff(diffText) {
         });
       }
       const match = line.match(/^diff --git a\/(.+) b\/(.+)$/);
-      if (match && match[1] && match[2]) {
+      if (match?.[1] && match[2]) {
         currentFile = {
           oldPath: match[1],
           newPath: match[2],
@@ -74,7 +74,7 @@ function parseUnifiedDiff(diffText) {
       const hunkMatch = line.match(
         /@@ -(\d+)(?:,(\d+))? \+(\d+)(?:,(\d+))? @@/
       );
-      if (hunkMatch && hunkMatch[1] && hunkMatch[3]) {
+      if (hunkMatch?.[1] && hunkMatch[3]) {
         const hunk = {
           oldStart: parseInt(hunkMatch[1], 10),
           oldCount: hunkMatch[2] ? parseInt(hunkMatch[2], 10) : 1,
@@ -214,7 +214,11 @@ function formatCommentBody(cluster, showModelAgreement) {
     lines.push(firstFinding.description);
     lines.push("");
   }
-  if (firstFinding?.suggestion) {
+  if (firstFinding?.suggestedCode) {
+    lines.push("```suggestion");
+    lines.push(firstFinding.suggestedCode);
+    lines.push("```");
+  } else if (firstFinding?.suggestion) {
     lines.push(`**Suggestion:** ${firstFinding.suggestion}`);
   }
   return lines.join("\n").trim();
@@ -1044,4 +1048,4 @@ export {
   processResult,
   runCli
 };
-//# sourceMappingURL=cli-M7YIR4XR.js.map
+//# sourceMappingURL=cli-PHU3RI5B.js.map

package/dist/{formatter-D42TRSLL.js → formatter-FIH7J57R.js}

@@ -1,10 +1,10 @@
 import {
   formatReport,
   init_formatter
-} from "./chunk-SYMFCPGM.js";
+} from "./chunk-HVF7WG6A.js";
 import "./chunk-UFR2SVK2.js";
 init_formatter();
 export {
   formatReport
 };
-//# sourceMappingURL=formatter-D42TRSLL.js.map
+//# sourceMappingURL=formatter-FIH7J57R.js.map

package/dist/index.js

@@ -18,9 +18,9 @@ import {
   handleGitReview,
   initializeConfig,
   logger
-} from "./chunk-KO6UIMSL.js";
-import "./chunk-IVKLQD6M.js";
-import "./chunk-SYMFCPGM.js";
+} from "./chunk-ZDBY7AGR.js";
+import "./chunk-AEDZOTVA.js";
+import "./chunk-HVF7WG6A.js";
 import {
   init_esm_shims
 } from "./chunk-UFR2SVK2.js";
@@ -1233,7 +1233,7 @@ function isSensitiveFile(filename) {
   for (const pattern of SENSITIVE_FILE_PATTERNS) {
     if (pattern.includes("*")) {
       const regex = new RegExp(
-        "^" + pattern.replace(/\./g, "\\.").replace(/\*/g, ".*") + "$",
+        `^${pattern.replace(/\./g, "\\.").replace(/\*/g, ".*")}$`,
         "i"
       );
       if (regex.test(lowerName)) {
@@ -1661,7 +1661,7 @@ async function handleTpsAudit(client, models, input) {
     });
     for (const result of results) {
       if (!result.error && result.review) {
-        const { parseTpsAnalysis } = await import("./tps-audit-DPIJH6JK.js");
+        const { parseTpsAnalysis } = await import("./tps-audit-2DYJMPJ5.js");
         analysis = parseTpsAnalysis(result.review);
         if (analysis) break;
       }
@@ -1676,7 +1676,7 @@ async function handleTpsAudit(client, models, input) {
   };
 }
 async function handleBatchedTpsAudit(client, models, scanResult, focusAreas) {
-  const { parseTpsAnalysis } = await import("./tps-audit-DPIJH6JK.js");
+  const { parseTpsAnalysis } = await import("./tps-audit-2DYJMPJ5.js");
   const repoName = scanResult.repoRoot.split("/").pop() ?? "unknown";
   const batches = createFileBatches(scanResult.files);
   const batchContents = batches.map((batch) => ({
@@ -1821,7 +1821,6 @@ function formatTpsAuditResults(auditResult) {
       );
       break;
     }
-    case "markdown":
     default: {
       const parts = [];
       parts.push("# TPS Audit Report\n");
@@ -1906,7 +1905,7 @@ ${r.review}
 var args = process.argv.slice(2);
 var command = args[0];
 if (command === "review" || command === "setup") {
-  import("./cli-M7YIR4XR.js").then(async ({ processResult, runCli }) => {
+  import("./cli-PHU3RI5B.js").then(async ({ processResult, runCli }) => {
     try {
       const result = await runCli(process.argv);
       if (result) {

package/dist/{tps-audit-DPIJH6JK.js → tps-audit-2DYJMPJ5.js}

@@ -6,7 +6,7 @@ import {
   buildSynthesisUserMessage,
   buildUserMessage,
   parseTpsAnalysis
-} from "./chunk-IVKLQD6M.js";
+} from "./chunk-AEDZOTVA.js";
 import "./chunk-UFR2SVK2.js";
 export {
   BATCH_SYSTEM_PROMPT,
@@ -17,4 +17,4 @@ export {
   buildUserMessage,
   parseTpsAnalysis
 };
-//# sourceMappingURL=tps-audit-DPIJH6JK.js.map
+//# sourceMappingURL=tps-audit-2DYJMPJ5.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@klitchevo/code-council",
-	"version": "0.2.0",
+	"version": "0.2.3",
 	"description": "Multi-model AI code review server using OpenRouter - get diverse perspectives from multiple LLMs in parallel",
 	"main": "dist/index.js",
 	"type": "module",