trickle-observe 0.2.79 → 0.2.80

package/dist/observe-register.js

@@ -539,12 +539,13 @@ function transformCjsSource(source, filename, moduleName, env) {
             destructInsertions = findDestructuredDeclarations(source);
         }
     }
-    // Additional variable patterns: reassignments, for-loops, catch clauses, function params
+    // Additional variable patterns: reassignments, for-loops, catch clauses
+    // Note: function params are NOT traced here because observe-register already
+    // wraps functions with __trickle_wrap which captures param types via wrapFunction.
     const reassignInsertions = (0, vite_plugin_1.findReassignments)(source);
     const forLoopInsertions = (0, vite_plugin_1.findForLoopVars)(source);
     const catchInsertions = (0, vite_plugin_1.findCatchVars)(source);
-    const funcParamInsertions = (0, vite_plugin_1.findFunctionParams)(source, false);
-    if (insertions.length === 0 && varInsertions.length === 0 && destructInsertions.length === 0 && reassignInsertions.length === 0 && forLoopInsertions.length === 0 && catchInsertions.length === 0 && funcParamInsertions.length === 0 && classInsertions.length === 0)
+    if (insertions.length === 0 && varInsertions.length === 0 && destructInsertions.length === 0 && reassignInsertions.length === 0 && forLoopInsertions.length === 0 && catchInsertions.length === 0 && classInsertions.length === 0)
         return source;
     // Resolve the path to the wrap helper (compiled JS)
     const wrapHelperPath = path_1.default.join(__dirname, 'wrap.js');
@@ -567,7 +568,7 @@ function transformCjsSource(source, filename, moduleName, env) {
         `};`,
     ];
     // Add variable tracing helper if we have var insertions
-    if (varInsertions.length > 0 || destructInsertions.length > 0 || reassignInsertions.length > 0 || forLoopInsertions.length > 0 || catchInsertions.length > 0 || funcParamInsertions.length > 0) {
+    if (varInsertions.length > 0 || destructInsertions.length > 0 || reassignInsertions.length > 0 || forLoopInsertions.length > 0 || catchInsertions.length > 0) {
         const traceVarPath = path_1.default.join(__dirname, 'trace-var.js');
         prefixLines.push(`var __trickle_tv_mod = require(${JSON.stringify(traceVarPath)});`, `var __trickle_tv = function(v, n, l) { try { __trickle_tv_mod.traceVar(v, n, l, ${JSON.stringify(moduleName)}, ${JSON.stringify(filename)}); } catch(e){} };`);
     }
@@ -617,14 +618,6 @@ function transformCjsSource(source, filename, moduleName, env) {
             code: `\ntry{${calls}}catch(__e2){}\n`,
         });
     }
-    // Function parameter insertions
-    for (const { bodyStart, paramNames, lineNo } of funcParamInsertions) {
-        const calls = paramNames.map(n => `__trickle_tv(${n},${JSON.stringify(n)},${lineNo})`).join(';');
-        allInsertions.push({
-            position: bodyStart,
-            code: `\ntry{${calls}}catch(__e){}\n`,
-        });
-    }
     // Add class method wrappings
     for (const ci of classInsertions) {
         allInsertions.push(ci);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "trickle-observe",
-  "version": "0.2.79",
+  "version": "0.2.80",
   "description": "Runtime type observability for JavaScript applications",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/observe-register.ts

@@ -40,7 +40,6 @@ import {
   findReassignments,
   findForLoopVars,
   findCatchVars,
-  findFunctionParams,
 } from './vite-plugin';
 
 const M = Module as any;
@@ -514,13 +513,14 @@ function transformCjsSource(source: string, filename: string, moduleName: string
     }
   }
 
-  // Additional variable patterns: reassignments, for-loops, catch clauses, function params
+  // Additional variable patterns: reassignments, for-loops, catch clauses
+  // Note: function params are NOT traced here because observe-register already
+  // wraps functions with __trickle_wrap which captures param types via wrapFunction.
   const reassignInsertions = findReassignments(source);
   const forLoopInsertions = findForLoopVars(source);
   const catchInsertions = findCatchVars(source);
-  const funcParamInsertions = findFunctionParams(source, false);
 
-  if (insertions.length === 0 && varInsertions.length === 0 && destructInsertions.length === 0 && reassignInsertions.length === 0 && forLoopInsertions.length === 0 && catchInsertions.length === 0 && funcParamInsertions.length === 0 && classInsertions.length === 0) return source;
+  if (insertions.length === 0 && varInsertions.length === 0 && destructInsertions.length === 0 && reassignInsertions.length === 0 && forLoopInsertions.length === 0 && catchInsertions.length === 0 && classInsertions.length === 0) return source;
 
   // Resolve the path to the wrap helper (compiled JS)
   const wrapHelperPath = path.join(__dirname, 'wrap.js');
@@ -545,7 +545,7 @@ function transformCjsSource(source: string, filename: string, moduleName: string
   ];
 
   // Add variable tracing helper if we have var insertions
-  if (varInsertions.length > 0 || destructInsertions.length > 0 || reassignInsertions.length > 0 || forLoopInsertions.length > 0 || catchInsertions.length > 0 || funcParamInsertions.length > 0) {
+  if (varInsertions.length > 0 || destructInsertions.length > 0 || reassignInsertions.length > 0 || forLoopInsertions.length > 0 || catchInsertions.length > 0) {
     const traceVarPath = path.join(__dirname, 'trace-var.js');
     prefixLines.push(
       `var __trickle_tv_mod = require(${JSON.stringify(traceVarPath)});`,
@@ -609,15 +609,6 @@ function transformCjsSource(source: string, filename: string, moduleName: string
     });
   }
 
-  // Function parameter insertions
-  for (const { bodyStart, paramNames, lineNo } of funcParamInsertions) {
-    const calls = paramNames.map(n => `__trickle_tv(${n},${JSON.stringify(n)},${lineNo})`).join(';');
-    allInsertions.push({
-      position: bodyStart,
-      code: `\ntry{${calls}}catch(__e){}\n`,
-    });
-  }
-
   // Add class method wrappings
   for (const ci of classInsertions) {
     allInsertions.push(ci);