@papi-ai/server 0.7.54 → 0.7.56

package/dist/backfill-cycle-metrics.js

@@ -1065,7 +1065,7 @@ var init_install_id = __esm({
 function reportTelemetryEmitFailure(reason, ctx) {
   consecutiveTelemetryFailures += 1;
   console.error(
-    `[telemetry] emit failed (${reason}) tool=${ctx.toolName} event=${ctx.eventType} project=${ctx.projectId}`
+    `[telemetry] emit failed (${reason}) tool=${ctx.toolName} event=${ctx.eventType} project=${ctx.projectId ?? "(none)"}`
   );
   if (consecutiveTelemetryFailures >= TELEMETRY_BLACKOUT_THRESHOLD) {
     console.error(
@@ -1179,8 +1179,11 @@ var init_proxy_adapter = __esm({
       // (2) not-yet-wired hosted gaps — shrink as data-proxy handlers land (task-2390).
       // getToolCallCount + updatePhaseStatus are ABSENT: they now have edge handlers and
       // are served through the forwarder (getToolCallCount = SUP-2026-026 handler #1).
-      "createOwnerAction",
       "getContributorRole",
+      // task-2728 (C331): createOwnerAction REMOVED from NO_FORWARD — the PRODUCER half
+      // of the owner-action queue. Six readers were wired C329 (task-2412) but the
+      // producer stayed here, so the hosted Owner Action Queue was structurally empty
+      // (AD-74 inverted). Now forwards; the edge binds user_id to the bearer ([C]).
       "recordContributorReleasePr",
       "setContributorReleasePrStatus",
       "listContributorReleasePrs",
@@ -1561,18 +1564,19 @@ Check PAPI_PROJECT_ID in your .mcp.json config. Find your project ID in the PAPI
        */
       emitTelemetry(event) {
         const ctx = { projectId: event.projectId, toolName: event.toolName, eventType: event.eventType };
+        const payload = {
+          toolName: event.toolName,
+          eventType: event.eventType,
+          metadata: event.metadata ?? {}
+        };
+        if (event.projectId) payload["projectId"] = event.projectId;
         fetch(`${this.endpoint}/telemetry`, {
           method: "POST",
           headers: {
             "Content-Type": "application/json",
             "Authorization": `Bearer ${this.apiKey}`
           },
-          body: JSON.stringify({
-            projectId: event.projectId,
-            toolName: event.toolName,
-            eventType: event.eventType,
-            metadata: event.metadata ?? {}
-          }),
+          body: JSON.stringify(payload),
           signal: AbortSignal.timeout(5e3)
         }).then((res) => {
           if (res.ok) noteTelemetryEmitSuccess();

package/dist/index.js

@@ -1103,7 +1103,7 @@ function isEnabled() {
 function reportTelemetryEmitFailure(reason, ctx) {
   consecutiveTelemetryFailures += 1;
   console.error(
-    `[telemetry] emit failed (${reason}) tool=${ctx.toolName} event=${ctx.eventType} project=${ctx.projectId}`
+    `[telemetry] emit failed (${reason}) tool=${ctx.toolName} event=${ctx.eventType} project=${ctx.projectId ?? "(none)"}`
   );
   if (consecutiveTelemetryFailures >= TELEMETRY_BLACKOUT_THRESHOLD) {
     console.error(
@@ -1120,11 +1120,11 @@ function emitTelemetryEvent(event) {
   if (!apiKey) return;
   const endpoint = process.env["PAPI_DATA_ENDPOINT"] ?? DEFAULT_TELEMETRY_ENDPOINT;
   const body = {
-    projectId: event.project_id,
     toolName: event.tool_name,
     eventType: event.event_type,
     metadata: event.metadata ?? {}
   };
+  if (event.project_id) body["projectId"] = event.project_id;
   const ctx = {
     projectId: event.project_id,
     toolName: event.tool_name,
@@ -1296,8 +1296,11 @@ var init_proxy_adapter = __esm({
       // (2) not-yet-wired hosted gaps — shrink as data-proxy handlers land (task-2390).
       // getToolCallCount + updatePhaseStatus are ABSENT: they now have edge handlers and
       // are served through the forwarder (getToolCallCount = SUP-2026-026 handler #1).
-      "createOwnerAction",
       "getContributorRole",
+      // task-2728 (C331): createOwnerAction REMOVED from NO_FORWARD — the PRODUCER half
+      // of the owner-action queue. Six readers were wired C329 (task-2412) but the
+      // producer stayed here, so the hosted Owner Action Queue was structurally empty
+      // (AD-74 inverted). Now forwards; the edge binds user_id to the bearer ([C]).
       "recordContributorReleasePr",
       "setContributorReleasePrStatus",
       "listContributorReleasePrs",
@@ -1678,18 +1681,19 @@ Check PAPI_PROJECT_ID in your .mcp.json config. Find your project ID in the PAPI
        */
       emitTelemetry(event) {
         const ctx = { projectId: event.projectId, toolName: event.toolName, eventType: event.eventType };
+        const payload = {
+          toolName: event.toolName,
+          eventType: event.eventType,
+          metadata: event.metadata ?? {}
+        };
+        if (event.projectId) payload["projectId"] = event.projectId;
         fetch(`${this.endpoint}/telemetry`, {
           method: "POST",
           headers: {
             "Content-Type": "application/json",
             "Authorization": `Bearer ${this.apiKey}`
           },
-          body: JSON.stringify({
-            projectId: event.projectId,
-            toolName: event.toolName,
-            eventType: event.eventType,
-            metadata: event.metadata ?? {}
-          }),
+          body: JSON.stringify(payload),
           signal: AbortSignal.timeout(5e3)
         }).then((res) => {
           if (res.ok) noteTelemetryEmitSuccess();
@@ -8168,20 +8172,27 @@ function formatDecisionLifecycleSummary(decisions) {
   }
   return parts.join(", ");
 }
-function formatBuildReports(reports) {
+function formatBuildReports(reports, opts) {
   if (reports.length === 0) return "No build reports yet.";
-  return reports.map(
+  const capped = opts?.maxReports != null ? reports.slice(0, opts.maxReports) : reports;
+  const budget = opts?.fieldBudget;
+  const field = (v) => budget != null ? truncateNotes(v, budget) : v ?? "";
+  const body = capped.map(
     (r) => `### ${r.taskName} \u2014 ${r.date} \u2014 Cycle ${r.cycle}
 
 - **Completed:** ${r.completed}
 - **Actual Effort:** ${r.actualEffort} vs estimated ${r.estimatedEffort}
 ` + (r.correctionsCount ? `- **Corrections:** ${r.correctionsCount}
 ` : "") + (Array.isArray(r.briefImplications) && r.briefImplications.length ? `- **Brief Implications:** ${r.briefImplications.map((bi) => `[${bi.canvasSection}/${bi.type}] ${bi.detail}`).join("; ")}
-` : "") + `- **Surprises:** ${r.surprises}
-- **Discovered Issues:** ${r.discoveredIssues}
-- **Architecture Notes:** ${r.architectureNotes}` + (r.deadEnds ? `
-- **Dead Ends:** ${r.deadEnds}` : "")
+` : "") + `- **Surprises:** ${field(r.surprises)}
+- **Discovered Issues:** ${field(r.discoveredIssues)}
+- **Architecture Notes:** ${field(r.architectureNotes)}` + (r.deadEnds ? `
+- **Dead Ends:** ${field(r.deadEnds)}` : "")
   ).join("\n\n---\n\n");
+  const omitted = capped.length < reports.length ? `
+
+_\u2026and ${reports.length - capped.length} older build report(s) omitted to bound context size._` : "";
+  return body + omitted;
 }
 function formatRecentlyShippedCapabilities(reports) {
   const completed = reports.filter((r) => r.completed === "Yes" || r.completed === "Partial");
@@ -8411,6 +8422,9 @@ function computeCycleEffort(cycleTaskRows, cycleReports) {
     deliveredPoints: cycleReports.reduce((s, r) => s + effortWeight(r.actualEffort || r.estimatedEffort), 0)
   };
 }
+function velocityPoints(v) {
+  return v.deliveredPoints ?? v.effortPoints;
+}
 function computeSnapshotsFromBuildReports(reports, tasks) {
   const reportsByCycle = /* @__PURE__ */ new Map();
   for (const r of reports) {
@@ -8461,22 +8475,22 @@ function formatCycleMetrics(snapshots) {
     const avgPlanned = Math.round(
       recentVelocities.reduce((sum, v) => sum + v.effortPoints, 0) / recentVelocities.length * 10
     ) / 10;
-    const withDelivered = recentVelocities.filter((v) => v.deliveredPoints !== void 0);
-    const avgDelivered = withDelivered.length > 0 ? Math.round(withDelivered.reduce((sum, v) => sum + (v.deliveredPoints ?? 0), 0) / withDelivered.length * 10) / 10 : void 0;
-    lines.push("**Cycle Sizing \u2014 planned vs delivered effort points**");
+    const avgDelivered = Math.round(
+      recentVelocities.reduce((sum, v) => sum + velocityPoints(v), 0) / recentVelocities.length * 10
+    ) / 10;
+    const anyDelivered = recentVelocities.some((v) => v.deliveredPoints !== void 0);
+    lines.push("**Velocity \u2014 delivered effort points (size the next cycle from this)**");
     lines.push(
-      `- Last ${recentVelocities.length} cycles: ` + recentVelocities.map((v) => v.deliveredPoints !== void 0 ? `S${v.cycle}=${v.effortPoints} planned/${v.deliveredPoints} delivered` : `S${v.cycle}=${v.effortPoints} planned`).join(", ")
+      `- Last ${recentVelocities.length} cycles: ` + recentVelocities.map((v) => v.deliveredPoints !== void 0 ? `S${v.cycle}=${v.deliveredPoints} delivered/${v.effortPoints} planned` : `S${v.cycle}=${v.effortPoints} planned (no delivered data \u2014 using planned)`).join(", ")
     );
-    if (avgDelivered !== void 0) {
+    lines.push(`- Velocity (avg delivered): ${avgDelivered} pts/cycle (XS=1, S=2, M=3, L=5, XL=8)`);
+    if (anyDelivered) {
       const delta = Math.round((avgDelivered - avgPlanned) * 10) / 10;
       const sign = delta > 0 ? "+" : "";
       const read = delta === 0 ? "delivered matches planned" : delta > 0 ? "cycles cost MORE than scoped (under-scoping)" : "cycles cost LESS than scoped (over-scoping)";
-      lines.push(`- Average: ${avgPlanned} planned / ${avgDelivered} delivered (XS=1, S=2, M=3, L=5, XL=8)`);
-      lines.push(`- Scope accuracy: ${sign}${delta} pts/cycle \u2014 ${read}.`);
-    } else {
-      lines.push(`- Average: ${avgPlanned} planned effort points/cycle (XS=1, S=2, M=3, L=5, XL=8)`);
+      lines.push(`- Scope accuracy (drift vs planned): planned averaged ${avgPlanned} pts/cycle \u2014 ${sign}${delta} pts, ${read}.`);
     }
-    lines.push(`- Size cycles on what the selected tasks actually require \u2014 planned is a reference, not a target.`);
+    lines.push(`- Size cycles on what the selected tasks actually cost \u2014 delivered is the signal, planned is the estimate that was wrong.`);
   }
   return lines.join("\n");
 }
@@ -8490,7 +8504,9 @@ function formatDerivedMetrics(snapshots, backlogTasks) {
       const latest = recent[recent.length - 1];
       lines.push("**Cycle History (5-cycle avg)**");
       lines.push(`- Average: ${avg.toFixed(1)} tasks/cycle`);
-      lines.push(`- Latest (Cycle ${latest.cycle}): ${latest.completed} tasks, ${latest.effortPoints} effort points`);
+      const latestDelivered = velocityPoints(latest);
+      const plannedNote = latest.deliveredPoints !== void 0 && latest.deliveredPoints !== latest.effortPoints ? ` (planned ${latest.effortPoints})` : "";
+      lines.push(`- Latest (Cycle ${latest.cycle}): ${latest.completed} tasks, ${latestDelivered} effort points delivered${plannedNote}`);
     }
   }
   const activeTasks = backlogTasks.filter(
@@ -10419,7 +10435,9 @@ async function prepareHandoffs(adapter2, _config, taskIds, force = false) {
     // buildHandoffsOnlyUserMessage adds +1 internally
     preAssignedTasks: tasksNeedingHandoffs,
     activeDecisions: formatActiveDecisionsForPlan(decisions),
-    recentBuildReports: formatBuildReports(reports),
+    // task-2681: truncate verbose report bodies — a couple of audit reports alone
+    // (e.g. task-2413/2655) can push this payload past the overflow range.
+    recentBuildReports: formatBuildReports(reports, { maxReports: 10, fieldBudget: 280 }),
     productBrief: brief,
     northStar
   });
@@ -10940,6 +10958,7 @@ async function resolveOwnerGate(adapter2, config2) {
 }
 
 // src/services/plan.ts
+var PLAN_BUILD_REPORT_BUDGET = { maxReports: 12, fieldBudget: 280 };
 async function resolvePlanScope(adapter2, config2) {
   const gate = await resolveOwnerGate(adapter2, config2);
   return { callerUserId: gate.callerUserId, ownerUserId: gate.ownerUserId, callerIsOwner: gate.callerIsOwner };
@@ -11788,7 +11807,7 @@ ${logLines}`);
     cycleNumber: health.totalCycles,
     productBrief,
     activeDecisions: formatActiveDecisionsForPlan(decisions),
-    recentBuildReports: formatBuildReports(cappedReports),
+    recentBuildReports: formatBuildReports(cappedReports, PLAN_BUILD_REPORT_BUDGET),
     cycleLog: formatCycleLog(log2),
     board: formatBoardForPlan(plannerTasks, filters, health.totalCycles, deferredTasks),
     northStar,
@@ -12574,7 +12593,7 @@ async function preparePlan(adapter2, config2, filters, focus, force, handoffsOnl
       cycleNumber,
       preAssignedTasks: preAssigned,
       activeDecisions: formatActiveDecisionsForPlan(decisions),
-      recentBuildReports: formatBuildReports(reports),
+      recentBuildReports: formatBuildReports(reports, PLAN_BUILD_REPORT_BUDGET),
       productBrief: brief,
       northStar
     });
@@ -14072,6 +14091,7 @@ function extractTrendPoints(snapshots) {
       matchRate: accuracyRow.matchRate,
       bias: accuracyRow.bias,
       effortPoints: velocityRow.effortPoints,
+      deliveredPoints: velocityPoints(velocityRow),
       completed: velocityRow.completed
     });
   }
@@ -14085,7 +14105,8 @@ function generateValueReport(snapshots) {
   const first = recent[0];
   const last = recent[recent.length - 1];
   const matchRateDelta = last.matchRate - first.matchRate;
-  const effortDelta = last.effortPoints - first.effortPoints;
+  const effortDelta = last.deliveredPoints - first.deliveredPoints;
+  const plannedDelta = last.effortPoints - first.effortPoints;
   const completedDelta = last.completed - first.completed;
   const biasTrend = assessBiasTrend(first.bias, last.bias);
   const lines = [];
@@ -14094,7 +14115,8 @@ function generateValueReport(snapshots) {
   lines.push("");
   lines.push(`- **Scope accuracy:** ${first.matchRate}% \u2192 ${last.matchRate}% (${formatDelta(matchRateDelta, "pp")})`);
   lines.push(`- **Estimation bias:** ${formatBias(first.bias)} \u2192 ${formatBias(last.bias)} (${biasTrend})`);
-  lines.push(`- **Velocity:** ${first.effortPoints} \u2192 ${last.effortPoints} effort points/cycle (${formatDelta(effortDelta, "")})`);
+  lines.push(`- **Velocity (delivered):** ${first.deliveredPoints} \u2192 ${last.deliveredPoints} effort points/cycle (${formatDelta(effortDelta, "")})`);
+  lines.push(`- **Planned (for drift):** ${first.effortPoints} \u2192 ${last.effortPoints} effort points/cycle (${formatDelta(plannedDelta, "")})`);
   lines.push(`- **Throughput:** ${first.completed} \u2192 ${last.completed} tasks/cycle (${formatDelta(completedDelta, "")})`);
   return lines.join("\n");
 }
@@ -20909,7 +20931,9 @@ async function completeBuild(adapter2, config2, taskId, input, options = {}, cli
       }
     }
   }
+  const AUTO_TRIAGE_MIN_SEVERITY = /* @__PURE__ */ new Set(["P0", "P1", "P2"]);
   let autoTriagedCount = 0;
+  let autoTriagedSoftSkipped = 0;
   const autoTriagedIds = [];
   const autoTriagedDupes = [];
   if (input.discoveredIssues && input.discoveredIssues !== "None" && typeof adapter2.createTask === "function") {
@@ -20929,6 +20953,10 @@ async function completeBuild(adapter2, config2, taskId, input, options = {}, cli
       const sevMatch = line.match(/^(P[0-3])[\s:]+/i);
       if (!sevMatch) continue;
       const severityLabel = sevMatch[1].toUpperCase();
+      if (!AUTO_TRIAGE_MIN_SEVERITY.has(severityLabel)) {
+        autoTriagedSoftSkipped++;
+        continue;
+      }
       const priority = severityLabel === "P0" ? "P0 Critical" : severityLabel === "P1" ? "P1 High" : severityLabel === "P2" ? "P2 Medium" : "P3 Low";
       const titleRaw = line.replace(/^P[0-3][\s:]+/i, "").trim();
       const title = titleRaw.length > 120 ? titleRaw.slice(0, 120) : titleRaw;
@@ -21178,6 +21206,7 @@ async function completeBuild(adapter2, config2, taskId, input, options = {}, cli
     autoTriagedCount: autoTriagedCount > 0 ? autoTriagedCount : void 0,
     autoTriagedIds: autoTriagedIds.length > 0 ? autoTriagedIds : void 0,
     autoTriagedDupes: autoTriagedDupes.length > 0 ? autoTriagedDupes : void 0,
+    autoTriagedSoftSkipped: autoTriagedSoftSkipped > 0 ? autoTriagedSoftSkipped : void 0,
     reportWriteVerified,
     localPreview
   };
@@ -22291,7 +22320,7 @@ function formatCompleteResult(result) {
   if (result.learningsLinkedCount) {
     lines.push("", `Linked ${result.learningsLinkedCount} unactioned learning(s) to this task.`);
   }
-  if (result.autoTriagedCount || result.autoTriagedDupes?.length) {
+  if (result.autoTriagedCount || result.autoTriagedDupes?.length || result.autoTriagedSoftSkipped) {
     const parts = [];
     if (result.autoTriagedCount) {
       const ids = result.autoTriagedIds?.length ? ` (${result.autoTriagedIds.join(", ")})` : "";
@@ -22300,6 +22329,9 @@ function formatCompleteResult(result) {
     if (result.autoTriagedDupes?.length) {
       parts.push(`Skipped ${result.autoTriagedDupes.length} duplicate issue(s) already in Backlog: ${result.autoTriagedDupes.join(", ")}.`);
     }
+    if (result.autoTriagedSoftSkipped) {
+      parts.push(`${result.autoTriagedSoftSkipped} low-severity (P3) observation(s) recorded to learnings, not filed as tasks.`);
+    }
     lines.push("", parts.join(" "));
   }
   if (result.reportWriteVerified === false) {
@@ -24992,7 +25024,7 @@ function computeHealthScore(cycleNumber, snapshots, activeTasks, decisionUsage)
   const recentSnaps = snapshots.slice(-3);
   const baselineSnaps = snapshots.slice(-10);
   if (recentSnaps.length > 0 && baselineSnaps.length > 0) {
-    const avg = (snaps) => snaps.reduce((s, sn) => s + (sn.velocity[0]?.effortPoints ?? 0), 0) / snaps.length;
+    const avg = (snaps) => snaps.reduce((s, sn) => s + (sn.velocity[0] ? velocityPoints(sn.velocity[0]) : 0), 0) / snaps.length;
     const recentAvg = avg(recentSnaps);
     const baselineAvg = avg(baselineSnaps);
     const velocityScore = baselineAvg > 0 ? Math.min(100, Math.round(recentAvg / baselineAvg * 100)) : 50;
@@ -29055,16 +29087,18 @@ ${usageLine(decision.usage)}`;
       const clientName = server2.getClientVersion()?.name;
       const metric = buildMetric(name, elapsed, usage, void 0, contextBytes, contextUtilisation, clientName);
       metric.success = !isError;
-      adapter2.appendToolMetric(metric).catch(() => {
+      adapter2.appendToolMetric(metric).catch((err) => {
+        console.error(`[metrics] appendToolMetric failed (non-blocking): ${err instanceof Error ? err.message : String(err)}`);
       });
-    } catch {
+    } catch (err) {
+      console.error(`[metrics] buildMetric/appendToolMetric threw (non-blocking): ${err instanceof Error ? err.message : String(err)}`);
     }
     if (config2.adapterType === "md") {
       const mdProjectSlug = config2.projectRoot ? config2.projectRoot.split("/").pop() : void 0;
       emitMdAdapterPing(name, { duration_ms: elapsed, success: !isError }, config2.userId, mdProjectSlug);
     }
     const telemetryProjectId = resolveTelemetryProjectId(config2);
-    if (telemetryProjectId) {
+    {
       const adapterEmit = adapter2.emitTelemetry?.bind(adapter2) ?? null;
       if (adapterEmit) {
         adapterEmit({

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@papi-ai/server",
-  "version": "0.7.54",
-  "description": "PAPI MCP server \u2014 AI-powered sprint planning, build execution, and strategy review for software projects",
+  "version": "0.7.56",
+  "description": "PAPI MCP server — AI-powered sprint planning, build execution, and strategy review for software projects",
   "license": "Elastic-2.0",
   "mcpName": "io.github.getpapi/papi",
   "type": "module",