trickle-observe 0.2.80 → 0.2.82

package/dist/observe-register.js

@@ -182,6 +182,18 @@ function findVarDeclarations(source, lineOffset = 0) {
             continue;
         if (varName === '_a' || varName === '_b' || varName === '_c')
             continue; // TS compiled vars
+        // Skip TS compiler helpers and module internals
+        if (varName === '__createBinding' || varName === '__setModuleDefault' || varName === '__importStar' || varName === '__importDefault')
+            continue;
+        if (varName === '__decorate' || varName === '__metadata' || varName === '__param' || varName === '__awaiter')
+            continue;
+        if (varName === 'ownKeys' || varName === 'desc' || varName === '_')
+            continue;
+        // Skip React Refresh / HMR internals
+        if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker' || varName === 'invalidateMessage')
+            continue;
+        if (varName === '_s' || varName === '_c2' || varName === '_s2')
+            continue;
         // Check if this is a require() call — skip those (they're imports, not interesting values)
         const restOfLine = source.slice(vmatch.index + vmatch[0].length - 1, vmatch.index + vmatch[0].length + 200);
         if (/^\s*require\s*\(/.test(restOfLine))
@@ -396,9 +408,10 @@ function transformCjsSource(source, filename, moduleName, env) {
         // Skip common false positives
         if (name === 'require' || name === 'exports' || name === 'module')
             continue;
-        // Find the opening brace of the function body
+        // Find the opening brace of the function body, correctly skipping
+        // default parameter values like `headers = {}` which contain braces
         const afterMatch = match.index + match[0].length;
-        const openBrace = source.indexOf('{', afterMatch);
+        const openBrace = (0, vite_plugin_1.findFunctionBodyBrace)(source, afterMatch);
         if (openBrace === -1)
             continue;
         // Extract parameter names from the source between ( and {

package/dist/vite-plugin.d.ts

@@ -40,6 +40,13 @@ export declare function tricklePlugin(options?: TricklePluginOptions): {
         map: null;
     } | null;
 };
+/**
+ * Find the opening brace of a function body, skipping the parameter list.
+ * Starting from the character right after the opening `(` of the parameter list,
+ * scans forward matching parens to find the closing `)`, then finds the `{` after it.
+ * Returns -1 if not found.
+ */
+export declare function findFunctionBodyBrace(source: string, afterOpenParen: number): number;
 /**
  * Find variable reassignments (not declarations) and return insertions for tracing.
  * Handles: x = newValue; x += 1; x ||= fallback; etc.

package/dist/vite-plugin.js

@@ -23,6 +23,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.tricklePlugin = tricklePlugin;
+exports.findFunctionBodyBrace = findFunctionBodyBrace;
 exports.findReassignments = findReassignments;
 exports.findCatchVars = findCatchVars;
 exports.findForLoopVars = findForLoopVars;
@@ -262,9 +263,15 @@ function findVarDeclarations(source) {
         // Skip trickle internals
         if (varName.startsWith('__trickle'))
             continue;
-        // Skip TS compiled vars
+        // Skip TS compiled vars and helpers
         if (varName === '_a' || varName === '_b' || varName === '_c')
             continue;
+        if (varName === '__createBinding' || varName === '__setModuleDefault' || varName === '__importStar' || varName === '__importDefault')
+            continue;
+        if (varName === '__decorate' || varName === '__metadata' || varName === '__param' || varName === '__awaiter')
+            continue;
+        if (varName === 'ownKeys' || varName === 'desc')
+            continue;
         // Skip React Refresh / HMR internals (Vite, webpack, Next.js inject these)
         if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker' || varName === 'invalidateMessage')
             continue;
@@ -517,6 +524,9 @@ function findReassignments(source) {
         // Skip 'this', 'self', 'super' (not reassignable in practice)
         if (varName === 'this' || varName === 'super')
             continue;
+        // Skip TS compiler helpers and module internals
+        if (varName === 'ownKeys' || varName === 'desc')
+            continue;
         // Skip React Refresh / HMR internals and discard variables
         if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker')
             continue;

package/dist-esm/vite-plugin.js

@@ -114,7 +114,7 @@ export function tricklePlugin(options = {}) {
  * scans forward matching parens to find the closing `)`, then finds the `{` after it.
  * Returns -1 if not found.
  */
-function findFunctionBodyBrace(source, afterOpenParen) {
+export function findFunctionBodyBrace(source, afterOpenParen) {
     let depth = 1;
     let pos = afterOpenParen;
     // Skip the parameter list (matching parens)
@@ -251,9 +251,15 @@ function findVarDeclarations(source) {
         // Skip trickle internals
         if (varName.startsWith('__trickle'))
             continue;
-        // Skip TS compiled vars
+        // Skip TS compiled vars and helpers
         if (varName === '_a' || varName === '_b' || varName === '_c')
             continue;
+        if (varName === '__createBinding' || varName === '__setModuleDefault' || varName === '__importStar' || varName === '__importDefault')
+            continue;
+        if (varName === '__decorate' || varName === '__metadata' || varName === '__param' || varName === '__awaiter')
+            continue;
+        if (varName === 'ownKeys' || varName === 'desc')
+            continue;
         // Skip React Refresh / HMR internals (Vite, webpack, Next.js inject these)
         if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker' || varName === 'invalidateMessage')
             continue;
@@ -506,6 +512,9 @@ export function findReassignments(source) {
         // Skip 'this', 'self', 'super' (not reassignable in practice)
         if (varName === 'this' || varName === 'super')
             continue;
+        // Skip TS compiler helpers and module internals
+        if (varName === 'ownKeys' || varName === 'desc')
+            continue;
         // Skip React Refresh / HMR internals and discard variables
         if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker')
             continue;

package/package.json

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

package/src/observe-register.ts

@@ -40,6 +40,7 @@ import {
   findReassignments,
   findForLoopVars,
   findCatchVars,
+  findFunctionBodyBrace,
 } from './vite-plugin';
 
 const M = Module as any;
@@ -173,6 +174,13 @@ function findVarDeclarations(source: string, lineOffset: number = 0): Array<{ li
     if (varName === '__trickle_mod' || varName === '__trickle_wrap' || varName === '__trickle_tv') continue;
     if (varName.startsWith('__trickle')) continue;
     if (varName === '_a' || varName === '_b' || varName === '_c') continue; // TS compiled vars
+    // Skip TS compiler helpers and module internals
+    if (varName === '__createBinding' || varName === '__setModuleDefault' || varName === '__importStar' || varName === '__importDefault') continue;
+    if (varName === '__decorate' || varName === '__metadata' || varName === '__param' || varName === '__awaiter') continue;
+    if (varName === 'ownKeys' || varName === 'desc' || varName === '_') continue;
+    // Skip React Refresh / HMR internals
+    if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker' || varName === 'invalidateMessage') continue;
+    if (varName === '_s' || varName === '_c2' || varName === '_s2') continue;
 
     // Check if this is a require() call — skip those (they're imports, not interesting values)
     const restOfLine = source.slice(vmatch.index + vmatch[0].length - 1, vmatch.index + vmatch[0].length + 200);
@@ -370,9 +378,10 @@ function transformCjsSource(source: string, filename: string, moduleName: string
     // Skip common false positives
     if (name === 'require' || name === 'exports' || name === 'module') continue;
 
-    // Find the opening brace of the function body
+    // Find the opening brace of the function body, correctly skipping
+    // default parameter values like `headers = {}` which contain braces
     const afterMatch = match.index + match[0].length;
-    const openBrace = source.indexOf('{', afterMatch);
+    const openBrace = findFunctionBodyBrace(source, afterMatch);
     if (openBrace === -1) continue;
 
     // Extract parameter names from the source between ( and {

package/src/vite-plugin.ts

@@ -134,7 +134,7 @@ export function tricklePlugin(options: TricklePluginOptions = {}) {
  * scans forward matching parens to find the closing `)`, then finds the `{` after it.
  * Returns -1 if not found.
  */
-function findFunctionBodyBrace(source: string, afterOpenParen: number): number {
+export function findFunctionBodyBrace(source: string, afterOpenParen: number): number {
   let depth = 1;
   let pos = afterOpenParen;
   // Skip the parameter list (matching parens)
@@ -255,8 +255,11 @@ function findVarDeclarations(source: string): Array<{ lineEnd: number; varName:
 
     // Skip trickle internals
     if (varName.startsWith('__trickle')) continue;
-    // Skip TS compiled vars
+    // Skip TS compiled vars and helpers
     if (varName === '_a' || varName === '_b' || varName === '_c') continue;
+    if (varName === '__createBinding' || varName === '__setModuleDefault' || varName === '__importStar' || varName === '__importDefault') continue;
+    if (varName === '__decorate' || varName === '__metadata' || varName === '__param' || varName === '__awaiter') continue;
+    if (varName === 'ownKeys' || varName === 'desc') continue;
     // Skip React Refresh / HMR internals (Vite, webpack, Next.js inject these)
     if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker' || varName === 'invalidateMessage') continue;
     if (varName === '_s' || varName === '_c2' || varName === '_s2') continue;
@@ -491,6 +494,8 @@ export function findReassignments(source: string): Array<{ lineEnd: number; varN
     if (varName === '_a' || varName === '_b' || varName === '_c') continue;
     // Skip 'this', 'self', 'super' (not reassignable in practice)
     if (varName === 'this' || varName === 'super') continue;
+    // Skip TS compiler helpers and module internals
+    if (varName === 'ownKeys' || varName === 'desc') continue;
     // Skip React Refresh / HMR internals and discard variables
     if (varName === 'prevRefreshReg' || varName === 'prevRefreshSig' || varName === 'inWebWorker') continue;
     if (varName === '_s' || varName === '_c2' || varName === '_s2' || varName === '_') continue;