@dbux/babel-plugin - npm Package Compare versions

Comparing version 0.1.7 to 0.1.8

package.json

 {
   "name": "@dbux/babel-plugin",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "",
@@ -28,3 +28,3 @@ "author": "Domi",
     "@babel/types": "^7.8.3",
-    "@dbux/common": "^0.1.7",
+    "@dbux/common": "^0.1.8",
     "lodash": "4.17.19"
@@ -46,3 +46,3 @@ },
   "_moduleAliases": {},
-  "gitHead": "fab42020aa9f1914b0af362496ed930343164f78"
+  "gitHead": "4cf7c1b028f47e3bcdd538ea7e7ce3fc6bdd25ad"
 }

src/helpers/traceHelpers.js

@@ -262,2 +262,3 @@ import template from '@babel/template';
     const { loc } = path.node;
+    const oLoc = oPath.node.loc;
     const calleeLoc = calleePath.node.loc;
@@ -308,3 +309,7 @@
 
-    newCalleePath.node.loc = calleeLoc;
+    // set new callee loc, so it will still get instrumented
+    // newOPath.node.loc = oLoc;
+    newOPath.parentPath.node.loc = oLoc;
+    // newCalleePath.node.loc = calleeLoc;
+    newCalleePath.parentPath.node.loc = calleeLoc;
     newPath.node.loc = loc;
@@ -317,5 +322,6 @@
     // newPath.setData('_calleePath', calleePathId);
-    newOPath.setData('originalIsParent', false);
-    newOPath.setData('traceResultType', TraceType.ExpressionValue);
-    newCalleePath.setData('originalIsParent', false);
+    // newOPath.setData('originalIsParent', false);
+    newOPath.parentPath.setData('traceResultType', TraceType.ExpressionValue);
+    newCalleePath.parentPath.setData('traceResultType', TraceType.ExpressionValue);
+    // newCalleePath.setData('originalIsParent', false);
     newPath.setData('_bcePathId', bcePathId);
@@ -337,3 +343,4 @@
 /**
- * Convert `f(...args)` to: `traceBCE(f), f(...args)` to trace callee (`f`) and place BCE correctly
+ * Convert `f(...args)` to: `traceBCE(f), f(...args)`.
+ * Goal: trace callee `f`, and place BCE correctly
  * 
@@ -408,2 +415,3 @@ */
     if (!isSpecialCallee) {
+      // set new callee loc, so it will still get instrumented
       newCalleePath.node.loc = calleeLoc;
@@ -420,2 +428,3 @@ }
     newPath.setData('_bcePathId', bcePathId);
+    newCalleePath.setData('traceResultType', TraceType.ExpressionValue);
     newCalleePath.setData('originalIsParent', false); // override config for `AssignmentExpression.right`
@@ -422,0 +431,0 @@

src/visitors/traceVisitors.js

@@ -14,3 +14,3 @@ /**
 import { traceWrapExpression, buildTraceNoValue, traceCallExpression, instrumentCallExpressionEnter, getTracePath } from '../helpers/traceHelpers';
-import { loopVisitor } from './loopVisitors';
+// import { loopVisitor } from './loopVisitors';
 import { isCallPath } from '../helpers/functionHelpers';
@@ -660,3 +660,3 @@ import { functionVisitEnter } from './functionVisitor';
     if (direction === InstrumentationDirection.Enter) {
-      // store config on enter
+      // store config override on enter
       if (extraCfg) {
@@ -663,0 +663,0 @@ if (path.getData('visitorCfg')) {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

2151952

increased by0.06%

Lines of code

3772

increased by0.27%

Dependency changes

• Updated@dbux/common@^0.1.8