@orangepro/orangepro-mcp 0.2.31 → 0.2.32

package/dist/local/generate/draftGuidance.js

@@ -10,7 +10,7 @@ export function classifyGeneratedDraftBlocker(reason) {
     if (/no required module provides package|cannot find module|module not found|unresolved import|imports module-path package/i.test(text)) {
         return "unresolved_import";
     }
-    if (/syntax check failed|undefined:|unknown field|redeclared in this block|expected (?:declaration|operand|'[^']+'|"[^"]+"|[^ ]+),? found|literal not terminated|cannot assign/i.test(text))
+    if (/syntax check failed|undefined:|unknown field|redeclared in this block|expected (?:declaration|operand|'[^']+'|"[^"]+"|[^ ]+),? found|literal not terminated|cannot assign|cannot use .+ as |missing ['",].+argument list/i.test(text))
         return "generated_code";
     return "unknown";
 }

package/dist/local/generate/generator.js

@@ -1515,7 +1515,13 @@ function commandAvailable(command) {
     }
 }
 function shortStaticDiag(message) {
-    return message.replace(/\s+/g, " ").trim().slice(0, 240);
+    const compact = message.replace(/\s+/g, " ").trim();
+    if (compact.length <= 240)
+        return compact;
+    // Compiler output starts with package/build boilerplate and puts the useful
+    // file:line diagnostic at the end. Preserve both ends so classification and
+    // remediation do not degrade to "unknown" on long package names.
+    return `${compact.slice(0, 72)} … ${compact.slice(-165)}`;
 }
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
@@ -2158,6 +2164,7 @@ export async function generateTests(graph, opts, provider, fileReader, clock = s
     const framework = pickFramework(graph, opts, targets, fileReader);
     const runSelection = targetsForFramework(graph, targets, framework);
     let runTargets = runSelection.targets;
+    const explicitMulti = Boolean(opts.target_ids && opts.target_ids.length > 1);
     warnings.push(...runSelection.warnings);
     const systemPrompt = opts.systemPrompt ?? buildSystemPrompt();
     const promptVersion = inputMode === "graph_grounded" && opts.prompt_version === "v5" ? PROMPT_VERSION_V5 : PROMPT_VERSION;
@@ -2275,7 +2282,8 @@ export async function generateTests(graph, opts, provider, fileReader, clock = s
     }
     else if (opts.prompt_version === "v5") {
         const declaredDeps = readDeclaredDeps(graph.workspace.root);
-        for (const behavior of runTargets) {
+        for (let targetIndex = 0; targetIndex < runTargets.length; targetIndex++) {
+            const behavior = runTargets[targetIndex];
             if (generated.length >= limit)
                 break;
             const gc = gatherContext(graph, behavior, framework, fileReader);
@@ -2378,7 +2386,13 @@ export async function generateTests(graph, opts, provider, fileReader, clock = s
                 });
                 continue;
             }
-            const selected = scenarios.slice(0, Math.max(1, limit - generated.length));
+            const remainingSlots = Math.max(1, limit - generated.length);
+            const remainingTargets = Math.max(1, runTargets.length - targetIndex);
+            // In explicit multi-target mode, reserve a fair share for every remaining
+            // target. Previously the first behavior could consume the entire batch
+            // with several scenarios, leaving later high-risk behaviors untouched.
+            const targetLimit = explicitMulti ? Math.max(1, Math.floor(remainingSlots / remainingTargets)) : remainingSlots;
+            const selected = scenarios.slice(0, targetLimit);
             const completions = [];
             try {
                 reportProgress(`Generating "${gc.ctx.behavior_title}" [v5 batch: ${selected.length}]…`);
@@ -2591,7 +2605,6 @@ export async function generateTests(graph, opts, provider, fileReader, clock = s
     }
     else {
         // Default: target-focused, bucket-diverse generation (one test per local bucket).
-        const explicitMulti = Boolean(opts.target_ids && opts.target_ids.length > 1);
         const plan = planGroundedBuckets(graph, runTargets, framework, fileReader, limit, explicitMulti, missing, warnings);
         // Repo dependency names (read once) — used by the runnable check to tell a missing
         // baseUrl-local import from a genuine external package the agent has installed.

package/dist/local/operations.js

@@ -19,7 +19,7 @@ import { enrichFromContent } from "./enrich/index.js";
 import { scoreGraph } from "./score/score.js";
 import { doctorGraph } from "./score/doctor.js";
 import { findGaps } from "./gaps/gaps.js";
-import { rankRiskGaps } from "./score/risk.js";
+import { rankPriorityGaps, rankRiskGaps } from "./score/risk.js";
 import { generateTests } from "./generate/generator.js";
 import { classifyGeneratedDraftBlocker } from "./generate/draftGuidance.js";
 import { autoProve, NO_KEY_MESSAGE, isEligibleProvableTarget } from "./autoProve.js";
@@ -1631,7 +1631,7 @@ export async function opStart(root, opts = {}, deps = defaultDeps()) {
         try {
             const graphForGeneration = loadGraph(workspacePaths(root).graphPath);
             const generatedTargets = new Set((graphForGeneration.generated_tests ?? []).map((t) => t.target_symbol_external_id).filter((id) => Boolean(id)));
-            const targetIds = rankRiskGaps(graphForGeneration, {
+            const targetIds = rankPriorityGaps(graphForGeneration, {
                 repoRoot: root,
                 limit: generationLimit,
                 provenIds: provenSymbolIds(graphForGeneration, loadLedger(root))

package/dist/local/score/risk.js

@@ -580,3 +580,11 @@ export function rankRiskGaps(graph, opts = {}) {
         .sort((a, b) => b.risk_score - a.risk_score || b.incoming_refs - a.incoming_refs || b.git_churn - a.git_churn || a.id.localeCompare(b.id))
         .slice(0, limit);
 }
+/**
+ * Canonical priority-gap portfolio shown to a local user and used for automatic
+ * generation. Keeping this policy in one function prevents `opro start` from
+ * generating for a different "top N" than behavior-coverage.html displays.
+ */
+export function rankPriorityGaps(graph, opts = {}) {
+    return rankRiskGaps(graph, { ...opts, maxPerFile: 3, maxPerTitle: 1 });
+}

package/dist/local/viz/behaviorReportData.js

@@ -1,7 +1,7 @@
 import { createHash } from "node:crypto";
 import path from "node:path";
 import { buildRtm } from "../rtm.js";
-import { inspectRiskInputHealth, isEntryPoint, rankRiskGaps } from "../score/risk.js";
+import { inspectRiskInputHealth, isEntryPoint, rankPriorityGaps } from "../score/risk.js";
 import { ORANGEPRO_VERSION } from "../version.js";
 import { PROOF_BLOCKER_GUIDE } from "../proofDoctor.js";
 import { classifyGeneratedDraftBlocker } from "../generate/draftGuidance.js";
@@ -892,7 +892,10 @@ export function buildBehaviorReportData(graph, ledger, opts = {}) {
     const flowIds = flowSymbolIds(graph);
     const summary = summaryFromRows(rows, flowIds);
     const repoRoot = opts.repoRoot ?? graph.workspace.root;
-    const riskGaps = rankRiskGaps(graph, { repoRoot, limit: opts.riskLimit ?? 20, maxPerFile: 3, maxPerTitle: 1 });
+    const provenIds = new Set(rows
+        .filter((row) => row.evidence_tier === "proven" && Boolean(row.code_symbol))
+        .map((row) => row.code_symbol));
+    const riskGaps = rankPriorityGaps(graph, { repoRoot, limit: opts.riskLimit ?? 20, provenIds });
     const riskHealth = inspectRiskInputHealth(repoRoot);
     const churnAvailable = riskHealth.churnAvailable && riskGaps.every((risk) => risk.churn_available !== false);
     const provenance = {

package/dist/local/viz/behaviorReportHtml.js

@@ -911,16 +911,20 @@ if(D.viewMeta){
   if(rm&&rm.scored>rm.shown)$("#risk-cap-note").textContent="Showing the top "+rm.shown+" of "+rm.scored.toLocaleString()+" scored behaviors — every behavior is scored; only the highest-risk are surfaced here. Full ranking: opro gaps --limit N, or .orangepro/graph.json.";
   if(fm&&fm.shown>0&&fm.prunedByCaps>0)$("#flow-cap-note").textContent="Showing "+fm.shown.toLocaleString()+" flows, endpoint-anchored first. "+fm.prunedByCaps.toLocaleString()+" additional branch expansions were pruned by depth/branch/global rendering caps — pruning affects display only, not scoring.";
 }
-// Platform CTA: top banner in risk panel
-const riskTopBanner=el("div","platform-top-banner",
-  \`<span class="platform-top-banner-text">Local scan shows <b>\${D.risks.length}</b> priority gaps. Full ranked list, incident correlation, and CI merge gate on Platform.</span>
-   <a class="platform-footer-btn" href="https://orangepro.ai/get-started" target="_blank">Unlock Full Analysis &rarr;</a>\`);
-riskList.before(riskTopBanner);
 const generatedRiskCount=D.risks.filter(r=>r.generatedTests&&r.generatedTests.length).length;
 const generatedOutputCopy=[
   D.generatedRunnableTotal?\`\${D.generatedRunnableTotal} runnable generated test\${D.generatedRunnableTotal===1?"":"s"}\`:'',
   D.generatedDraftTotal?\`\${D.generatedDraftTotal} grounded draft\${D.generatedDraftTotal===1?"":"s"} with code withheld\`:'',
 ].filter(Boolean).join(' and ');
+const generationSummary=D.generatedTotal
+  ?\` Generated output: <b>\${generatedOutputCopy}</b>; <b>\${D.shownCount}</b> shown inline across <b>\${generatedRiskCount} of \${D.risks.length}</b> priority flows.\`
+  :'';
+// Platform CTA: top banner in risk panel. Keep generated-output totals beside
+// the flow count so a user cannot mistake "5 flows" for "5 generated tests".
+const riskTopBanner=el("div","platform-top-banner",
+  \`<span class="platform-top-banner-text">Local scan shows <b>\${D.risks.length}</b> priority gaps.\${generationSummary} Full ranked list, incident correlation, and CI merge gate on Platform.</span>
+   <a class="platform-footer-btn" href="https://orangepro.ai/get-started" target="_blank">Unlock Full Analysis &rarr;</a>\`);
+riskList.before(riskTopBanner);
 let activeRiskFilter=generatedRiskCount?"generated":"all";
 function riskMatchesFilter(r){
   const hasGenerated=Boolean(r.generatedTests&&r.generatedTests.length);
@@ -931,8 +935,8 @@ function riskMatchesFilter(r){
 function renderRiskFilters(){
   const options=[
     ["all","All",D.risks.length],
-    ["generated","Flows with generated output",generatedRiskCount],
-    ["missing","No generated tests",D.risks.filter(r=>!(r.generatedTests&&r.generatedTests.length)).length]
+    ["generated","Flows with generated output",generatedRiskCount+"/"+D.risks.length],
+    ["missing","Flows without generated output",D.risks.filter(r=>!(r.generatedTests&&r.generatedTests.length)).length+"/"+D.risks.length]
   ];
   riskTools.innerHTML=options.map(([key,label,count])=>\`<button class="risk-filter" type="button" data-risk-filter="\${key}" aria-pressed="\${key===activeRiskFilter}">\${label} <span class="gc">\${count}</span></button>\`).join("");
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orangepro/orangepro-mcp",
-  "version": "0.2.31",
+  "version": "0.2.32",
   "private": false,
   "description": "OrangePro (`opro`) — a local-first, BYOK CLI + MCP server that builds an evidence graph from a local checkout, ingests runtime coverage, and generates grounded tests. Metadata-only exports; no source upload; generated tests stay local.",
   "license": "MIT",