@tikoci/rosetta 0.8.6 → 0.8.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tikoci/rosetta",
-  "version": "0.8.6",
+  "version": "0.8.7",
   "description": "RouterOS documentation as SQLite FTS5 — RAG search + command glossary via MCP",
   "type": "module",
   "license": "MIT",

package/src/browse.ts

@@ -51,6 +51,7 @@ import {
   searchDude,
   searchProperties,
   searchVideos,
+  truncateDeviceTestResultsPrefer512,
 } from "./query.ts";
 
 // ── ANSI utilities (zero deps) ──
@@ -785,15 +786,16 @@ function renderDeviceCard(d: DeviceResult): string {
 
   // Test results (attached for exact matches)
   if (d.test_results && d.test_results.length > 0) {
+    const truncated = truncateDeviceTestResultsPrefer512(d.test_results, 12);
     out.push("");
     out.push(`  ${bold("Benchmarks:")}  ${dim(`(${d.test_results.length} tests)`)}`);
-    for (const t of d.test_results.slice(0, 12)) {
+    for (const t of truncated.rows) {
       const mbps = t.throughput_mbps ? `${fmt(t.throughput_mbps)} Mbps` : "";
       const kpps = t.throughput_kpps ? `${fmt(t.throughput_kpps)} Kpps` : "";
       out.push(`    ${dim(pad(t.test_type, 9))} ${pad(t.mode, 16)} ${dim(pad(t.configuration, 28))} ${pad(`${t.packet_size}B`, 6)} ${bold(mbps)} ${dim(kpps)}`);
     }
-    if (d.test_results.length > 12) {
-      out.push(`    ${dim(`... and ${d.test_results.length - 12} more (use`)} ${cyan("tests")} ${dim("for full listing)")}`);
+    if (truncated.omitted > 0) {
+      out.push(`    ${dim(`... and ${truncated.omitted} more (use`)} ${cyan("tests")} ${dim("for full listing)")}`);
     }
   }
 

package/src/query.test.ts

@@ -43,6 +43,7 @@ const {
   searchDeviceTests,
   getTestResultMeta,
   normalizeDeviceQuery,
+  truncateDeviceTestResultsPrefer512,
   searchVideos,
   searchDude,
   getDudePage,
@@ -1426,6 +1427,48 @@ describe("dataset CSV exports", () => {
   });
 });
 
+describe("truncateDeviceTestResultsPrefer512", () => {
+  test("keeps all 512B rows when truncating", () => {
+    const rows = [
+      { packet_size: 1518, id: "a" },
+      { packet_size: 512, id: "b" },
+      { packet_size: 64, id: "c" },
+      { packet_size: 512, id: "d" },
+    ];
+    const res = truncateDeviceTestResultsPrefer512(rows, 2);
+
+    expect(res.rows.map((r) => r.id)).toEqual(["b", "d"]);
+    expect(res.omitted).toBe(2);
+  });
+
+  test("fills remaining slots with non-512 rows", () => {
+    const rows = [
+      { packet_size: 1518, id: "a" },
+      { packet_size: 512, id: "b" },
+      { packet_size: 64, id: "c" },
+      { packet_size: 512, id: "d" },
+    ];
+    const res = truncateDeviceTestResultsPrefer512(rows, 3);
+
+    expect(res.rows.map((r) => r.id)).toEqual(["b", "d", "a"]);
+    expect(res.omitted).toBe(1);
+  });
+
+  test("returns all 512B rows even if they exceed maxRows", () => {
+    const rows = [
+      { packet_size: 512, id: "a" },
+      { packet_size: 512, id: "b" },
+      { packet_size: 1518, id: "c" },
+      { packet_size: 512, id: "d" },
+      { packet_size: 64, id: "e" },
+    ];
+    const res = truncateDeviceTestResultsPrefer512(rows, 2);
+
+    expect(res.rows.map((r) => r.id)).toEqual(["a", "b", "d"]);
+    expect(res.omitted).toBe(2);
+  });
+});
+
 describe("getTestResultMeta", () => {
   test("returns distinct values", () => {
     const meta = getTestResultMeta();

package/src/query.ts

@@ -298,8 +298,8 @@ export function searchAll(query: string, limit = DEFAULT_LIMIT): SearchAllRespon
         path: node.path,
         type: node.type,
         description: node.description,
-        ...(node.page_id
-          ? { linked_page: { id: node.page_id, title: node.page_title!, url: node.page_url! } }
+        ...(node.page_id && node.page_title && node.page_url
+          ? { linked_page: { id: node.page_id, title: node.page_title, url: node.page_url } }
           : {}),
       };
     }
@@ -1330,6 +1330,43 @@ export type DeviceTestResult = {
   throughput_mbps: number | null;
 };
 
+/**
+ * Truncate benchmark rows for display while always keeping all 512-byte rows.
+ *
+ * Used by both TUI and MCP-facing adapters when they need a compact view.
+ * If the 512B bucket itself exceeds maxRows, return all 512B rows anyway.
+ */
+export function truncateDeviceTestResultsPrefer512<T extends { packet_size: number }>(
+  rows: T[],
+  maxRows: number,
+): { rows: T[]; omitted: number } {
+  if (rows.length === 0) return { rows: [], omitted: 0 };
+
+  const safeMax = Math.max(0, Math.floor(maxRows));
+  const rows512 = rows.filter((r) => r.packet_size === 512);
+  const non512 = rows.filter((r) => r.packet_size !== 512);
+
+  if (safeMax === 0) {
+    return { rows: rows512, omitted: rows.length - rows512.length };
+  }
+
+  if (rows.length <= safeMax) {
+    return { rows, omitted: 0 };
+  }
+
+  if (rows512.length === 0) {
+    return { rows: rows.slice(0, safeMax), omitted: rows.length - safeMax };
+  }
+
+  if (rows512.length >= safeMax) {
+    return { rows: rows512, omitted: rows.length - rows512.length };
+  }
+
+  const takeNon512 = safeMax - rows512.length;
+  const selected = [...rows512, ...non512.slice(0, takeNon512)];
+  return { rows: selected, omitted: rows.length - selected.length };
+}
+
 /** Map Unicode superscript digits to ASCII equivalents for product name matching.
  *  MikroTik uses ² and ³ in product names (hAP ax³, hAP ac²), but users type ASCII. */
 const SUPERSCRIPT_TO_ASCII: [string, string][] = [