@oqtopus-team/pi-qdash 0.1.10-beta.11 → 0.1.10-beta.13

package/extensions/lib/results.ts

@@ -28,3 +28,26 @@ export function toTextToolResult(text: string, data: unknown, details: Record<st
     details: { ...details, data: safeData },
   };
 }
+
+/**
+ * Result of a tool that fetched a figure: the text summary plus, for an image,
+ * the image itself as model-visible content. Without the image block a
+ * vision model only reads that a figure exists; with it, it can judge the
+ * curve. Non-image figures (JSON) keep a text-only result.
+ */
+export function toFigureToolResult(
+  text: string,
+  figure: { base64?: string; mediaType: string },
+  data: unknown,
+  details: Record<string, unknown> = {},
+) {
+  const result = toTextToolResult(text, data, details);
+  if (!figure.base64 || !figure.mediaType.startsWith("image/")) return result;
+  return {
+    ...result,
+    content: [
+      ...result.content,
+      { type: "image" as const, data: figure.base64, mimeType: figure.mediaType },
+    ],
+  };
+}

package/extensions/qdash.ts

@@ -21,7 +21,7 @@ import { forumDetailComponent, forumDetailLines, forumListLines, forumPostTitle,
 import { investigationContextSummary, validateInvestigationContext, type TimeseriesInvestigationContext } from "./lib/investigation-context.js";
 import { qdashObjectLinks, qdashWebBaseUrl, qdashWebUrl, safeConfig, withQDashLinks } from "./lib/links.js";
 import { arrayFromPayload, compactItems, firstNumber, firstString, formatItem, payloadTotal, statusIcon } from "./lib/payload.js";
-import { toTextToolResult, toToolResult } from "./lib/results.js";
+import { toFigureToolResult, toTextToolResult, toToolResult } from "./lib/results.js";
 import { ansi, boxLinesToWidth, boxed, formatNumber, textComponent } from "./lib/render.js";
 import { plotSeriesLines, timeseriesPlotComponent, timeseriesPoints } from "./lib/timeseries-plot.js";
 import { installQDashWriteGate } from "./lib/write-gate.js";
@@ -2543,7 +2543,7 @@ export default function qdashExtension(pi: ExtensionAPI) {
       const details: FigureDetails = { tool: "qdash_recent_calibration_figure", taskId, path: file.path ?? "", mediaType, sizeBytes: bytes.byteLength, figurePaths: file.figurePaths ?? [], jsonFigurePaths: file.jsonFigurePaths ?? [] };
       if (mediaType.startsWith("image/")) details.base64 = bytes.toString("base64");
       else details.text = bytes.toString("utf8");
-      return toTextToolResult(figureResultText(details), { task, selectedPath: details.path }, details);
+      return toFigureToolResult(figureResultText(details), details, { task, selectedPath: details.path }, details);
     },
     renderResult(result, _options, theme) {
       return figureComponent(result.details as unknown as FigureDetails, theme);
@@ -2563,7 +2563,7 @@ export default function qdashExtension(pi: ExtensionAPI) {
     async execute(_toolCallId, params: { profile?: string; configPath?: string; useEnv?: boolean; path: string }) {
       const client = await makeClient(params);
       const details = await fetchFigureDetails(client, params.path);
-      return toTextToolResult(figureResultText(details), { path: params.path }, details);
+      return toFigureToolResult(figureResultText(details), details, { path: params.path }, details);
     },
     renderResult(result, _options, theme) {
       return figureComponent(result.details as unknown as FigureDetails, theme);
@@ -2600,7 +2600,7 @@ export default function qdashExtension(pi: ExtensionAPI) {
         };
         if (mediaType.startsWith("image/")) details.base64 = bytes.toString("base64");
         else details.text = bytes.toString("utf8");
-        return toTextToolResult(figureResultText(details), { taskId: params.taskId, figurePaths: details.figurePaths, jsonFigurePaths: details.jsonFigurePaths, selectedPath: details.path }, details);
+        return toFigureToolResult(figureResultText(details), details, { taskId: params.taskId, figurePaths: details.figurePaths, jsonFigurePaths: details.jsonFigurePaths, selectedPath: details.path }, details);
       } catch (error) {
         return toTextToolResult(boxed("QDash Task Figures", [
           `task ${params.taskId}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oqtopus-team/pi-qdash",
-  "version": "0.1.10-beta.11",
+  "version": "0.1.10-beta.13",
   "description": "Pi extension for querying QDash via @oqtopus-team/qdash-client",
   "type": "module",
   "license": "Apache-2.0",
@@ -27,7 +27,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@oqtopus-team/qdash-client": "1.9.21-beta.168"
+    "@oqtopus-team/qdash-client": "1.9.21-beta.171"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",

package/skills/qdash/SKILL.md

@@ -58,7 +58,7 @@ Workflow:
 2. Turn the request into steps. Start from the catalog defaults and the flow templates (`qdash_query` with `flow_templates`, then `flow_template` for one template's task list): `OneQubitCheck` and `OneQubitFineTune` carry the standard one-qubit sequences, `TwoQubitCalibration` the two-qubit chain. Use `CustomOneQubit` / `CustomTwoQubit` only where the user wants a different task list, and keep the task order the templates use.
 3. `qdash_plan_pipeline` — validate. A result with problems is not an error: each problem carries a path into the spec (`steps[2].tasks[0]`, `targets.qids[1]`); fix it and plan again until it is valid.
 4. Show the user the resolved steps and task counts from the plan, then `qdash_run_pipeline` with the same `chipId` and `spec`. The execution is confirmation-gated.
-5. `qdash_wait_execution` with the returned `execution_id`, then summarize which targets passed and which failed, and read `qdash_get_task_knowledge` for any task whose result needs interpretation.
+5. `qdash_wait_execution` with the returned `execution_id`. It returns when the whole run has finished, not just its first step. Then summarize which targets passed and which failed, fetch the figures of failed tasks with `qdash_get_task_figures` so the user sees them, and read `qdash_get_task_knowledge` for any task whose result needs interpretation.
 
 Rules: