simframe 0.15.0 → 0.15.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "simframe",
-  "version": "0.15.0",
+  "version": "0.15.1",
   "mcpName": "io.github.lvlrSajjad/simframe",
   "description": "Always-warm iOS Simulator and Android emulator frames: agents read the screen in ~20ms instead of waiting on screenshots. MCP server + CLI.",
   "keywords": [

package/scripts/bench-hpi.mjs

@@ -186,11 +186,7 @@ const report = {
 
 console.log('\nflow                      runs  agent p50   human p50   HPI_time  step_ratio');
 for (const f of report.flows) {
-  console.log(
-    `${f.flow.padEnd(24)} ${String(f.runs).padStart(5)}  ${`${f.agent_ms.p50}ms`.padStart(9)}   ` +
-      `${(f.human_median_ms ? `${f.human_median_ms}ms` : '—').padStart(9)}   ` +
-      `${String(f.hpi_time ?? '—').padStart(8)}  ${String(f.step_ratio ?? '—').padStart(10)}`,
-  );
+  console.log(metrics.flowRow(f));
 }
 report.overall.hpi_time_median_of_passes = passTimes.length ? Number(metrics.median(passTimes).toFixed(3)) : null;
 const o = report.overall;

package/src/cli.js

@@ -1234,11 +1234,7 @@ async function main() {
       emit(flags, report, [
         flags.last ? `the last ${num(flags.last)} run(s) of each flow, of ${all.filter((f) => f.flow_name).length} named runs in the log` : null,
         'flow                      runs  agent p50   human p50   HPI_time  step_ratio  turns  esc',
-        ...report.flows.map((f) =>
-          `${f.flow.padEnd(24)} ${String(f.runs).padStart(5)}  ${`${f.agent_ms.p50}ms`.padStart(9)}   ` +
-          `${(f.human_median_ms ? `${f.human_median_ms}ms` : '—').padStart(9)}   ` +
-          `${(f.hpi_time ?? '—').toString().padStart(8)}  ${(f.step_ratio ?? '—').toString().padStart(10)}  ` +
-          `${(f.model_turns ?? '—').toString().padStart(5)}  ${String(f.escalations).padStart(3)}`),
+        ...report.flows.map((f) => metrics.flowRow(f, { wide: true })),
         '',
         `HPI_accuracy ${report.overall.hpi_accuracy} (${report.overall.runs} runs, ` +
           `${report.overall.runs - runs.filter((r) => r.completed && !r.wrong_action_taken).length} not clean)`,

package/src/metrics.js

@@ -601,6 +601,30 @@ export function harmonicMean(xs) {
  * a human baseline. A flow with no human baseline gets no HPI_time — reported
  * as null, never as 1.0, because a missing denominator is not parity.
  */
+/**
+ * One flow's row in the HPI table, shared by `simframe hpi` and the bench
+ * script because they had the same row duplicated byte for byte.
+ *
+ * It lives here, next to the report it renders, for a reason the v0.15.0 tag
+ * paid for: making `agent_ms` null for a flow that timed nothing was correct,
+ * and both printers dereferenced `.p50` on it. `bench` died with exit 1 and no
+ * hpi.json, and `simframe hpi` would have done the same for any user whose
+ * flow never completed. The metric had a test; nothing rendered it. A format
+ * duplicated in two files is a format that gets fixed in one.
+ *
+ * `—` means "not measured", never zero.
+ */
+export function flowRow(f, { wide = false } = {}) {
+  const cell = (v, unit = '') => (v === null || v === undefined ? '—' : `${v}${unit}`);
+  const row = `${f.flow.padEnd(24)} ${String(f.runs).padStart(5)}  `
+    + `${cell(f.agent_ms?.p50, 'ms').padStart(9)}   `
+    + `${cell(f.human_median_ms, 'ms').padStart(9)}   `
+    + `${cell(f.hpi_time).padStart(8)}  ${cell(f.step_ratio).padStart(10)}`;
+  return wide
+    ? `${row}  ${cell(f.model_turns).padStart(5)}  ${String(f.escalations).padStart(3)}`
+    : row;
+}
+
 export function hpi({ flows, baselines = {} }) {
   const byName = new Map();
   for (const f of flows) {