measure-fn 3.8.1 → 3.10.0

package/index.test.ts

@@ -385,6 +385,46 @@ describe("createMeasure (scoped)", () => {
         expect(out.logs[2]).toBe("[b:a] ... y");
         expect(out.logs[4]).toBe("[a:b] ... z");
     });
+
+    test("scoped maxResultLength truncates results", () => {
+        const s = createMeasure("s", { maxResultLength: 15 });
+        s.resetCounter();
+        const out = captureConsole();
+        s.measureSync("op", () => ({ data: "x".repeat(50) }));
+        out.restore();
+        expect(out.logs[0]).toContain("…");
+    });
+
+    test("scoped maxResultLength inherits to children", () => {
+        const s = createMeasure("s", { maxResultLength: 15 });
+        s.resetCounter();
+        const out = captureConsole();
+        s.measureSync("parent", (m) => {
+            m("child", () => ({ data: "x".repeat(50) }));
+            return 1;
+        });
+        out.restore();
+        const childLine = out.logs[1];
+        expect(childLine).toContain("…");
+    });
+
+    test("per-label maxResultLength overrides scoped", () => {
+        const s = createMeasure("s", { maxResultLength: 15 });
+        s.resetCounter();
+        const out = captureConsole();
+        s.measureSync({ label: "op", maxResultLength: 500 }, () => ({ data: "x".repeat(50) }));
+        out.restore();
+        expect(out.logs[0]).not.toContain("…");
+    });
+
+    test("scoped maxResultLength works with async measure", async () => {
+        const s = createMeasure("s", { maxResultLength: 15 });
+        s.resetCounter();
+        const out = captureConsole();
+        await s.measure("op", async () => ({ data: "x".repeat(50) }));
+        out.restore();
+        expect(out.logs[1]).toContain("…");
+    });
 });
 
 // ─── measure.retry ───────────────────────────────────────────────────

package/index.ts

@@ -273,10 +273,13 @@ export const resetCounter = () => {
   globalRootCounter = 0;
 };
 
-// ─── Core Implementation ─────────────────────────────────────────────
+export type ScopeOpts = {
+  maxResultLength?: number;
+};
 
-const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
+const createMeasureImpl = (prefix?: string, counterRef?: { value: number }, scopeOpts?: ScopeOpts) => {
   const counter = counterRef ?? { get value() { return globalRootCounter; }, set value(v) { globalRootCounter = v; } };
+  const scopeMaxLen = scopeOpts?.maxResultLength;
   let _lastError: unknown = null;
 
   const _measureInternal = async <U>(
@@ -295,8 +298,8 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     const localMaxLen = extractMaxResultLength(actionInternal);
     const effectiveMaxLen = localMaxLen ?? inheritedMaxLen;
 
-    const currentId = toAlpha(parentIdChain.pop() ?? 0);
-    const fullIdChain = [...parentIdChain, currentId];
+    const currentId = toAlpha(Number(parentIdChain.pop() ?? 0));
+    const fullIdChain: string[] = [...parentIdChain.map(String), currentId];
     const idStr = fullIdChain.join('-');
 
     emit({
@@ -346,7 +349,7 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     actionInternal: string | object,
     parentIdChain: (string | number)[],
     depth: number,
-    _onError?: undefined,
+    _onError?: ((error: unknown) => any) | undefined,
     inheritedMaxLen?: number
   ): U | null => {
     const start = performance.now();
@@ -357,8 +360,8 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     const localMaxLen = extractMaxResultLength(actionInternal);
     const effectiveMaxLen = localMaxLen ?? inheritedMaxLen;
 
-    const currentId = toAlpha(parentIdChain.pop() ?? 0);
-    const fullIdChain = [...parentIdChain, currentId];
+    const currentId = toAlpha(Number(parentIdChain.pop() ?? 0));
+    const fullIdChain: string[] = [...parentIdChain.map(String), currentId];
     const idStr = fullIdChain.join('-');
 
     if (hasNested) {
@@ -394,7 +397,7 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     arg3?: (error: unknown) => any
   ): Promise<T | null> => {
     if (typeof arg2 === 'function') {
-      return _measureInternal(arg2 as any, arg1, [counter.value++], 0, arg3) as Promise<T | null>;
+      return _measureInternal(arg2 as any, arg1, [counter.value++], 0, arg3, scopeMaxLen) as Promise<T | null>;
     } else {
       const currentId = toAlpha(counter.value++);
       emit({
@@ -502,7 +505,7 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     const results: (R | null)[] = [];
     for (let i = 0; i < items.length; i++) {
       try {
-        results.push(await fn(items[i], i));
+        results.push(await fn(items[i]!, i));
       } catch {
         results.push(null);
       }
@@ -539,7 +542,7 @@ const createMeasureImpl = (prefix?: string, counterRef?: { value: number }) => {
     arg2?: ((measure: MeasureSyncFn) => T)
   ): T | null => {
     if (typeof arg2 === 'function') {
-      return _measureInternalSync(arg2, arg1, [counter.value++], 0) as T | null;
+      return _measureInternalSync(arg2, arg1, [counter.value++], 0, undefined, scopeMaxLen) as T | null;
     } else {
       const currentId = toAlpha(counter.value++);
       emit({
@@ -595,9 +598,9 @@ export const measureSync = globalInstance.measureSync;
 
 // ─── Scoped instances ────────────────────────────────────────────────
 
-export const createMeasure = (scopePrefix: string) => {
+export const createMeasure = (scopePrefix: string, opts?: ScopeOpts) => {
   const scopeCounter = { value: 0 };
-  const scoped = createMeasureImpl(scopePrefix, scopeCounter);
+  const scoped = createMeasureImpl(scopePrefix, scopeCounter, opts);
   return {
     ...scoped,
     resetCounter: () => { scopeCounter.value = 0; },

package/package.json

@@ -1,33 +1,33 @@
-{
-  "name": "measure-fn",
-  "module": "index.ts",
-  "main": "./index.ts",
-  "types": "./index.ts",
-  "version": "3.8.1",
-  "type": "module",
-  "private": false,
-  "description": "Zero-dependency function performance measurement with hierarchical logging",
-  "keywords": [
-    "measure",
-    "performance",
-    "timing",
-    "logging",
-    "tracing",
-    "hierarchical"
-  ],
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/7flash/measure-fn"
-  },
-  "devDependencies": {
-    "@types/bun": "latest"
-  },
-  "peerDependencies": {
-    "typescript": "^5"
-  },
-  "scripts": {
-    "test": "bun test",
-    "example": "bun run example.ts"
-  }
-}
+{
+  "name": "measure-fn",
+  "module": "index.ts",
+  "main": "./index.ts",
+  "types": "./index.ts",
+  "version": "3.10.0",
+  "type": "module",
+  "private": false,
+  "description": "Zero-dependency function performance measurement with hierarchical logging",
+  "keywords": [
+    "measure",
+    "performance",
+    "timing",
+    "logging",
+    "tracing",
+    "hierarchical"
+  ],
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/7flash/measure-fn"
+  },
+  "devDependencies": {
+    "@types/bun": "latest"
+  },
+  "peerDependencies": {
+    "typescript": "^5"
+  },
+  "scripts": {
+    "test": "bun test",
+    "example": "bun run example.ts"
+  }
+}