mapshaper 0.6.32 → 0.6.34

package/mapshaper.js

@@ -1,6 +1,6 @@
 (function () {
 
-  var VERSION = "0.6.32";
+  var VERSION = "0.6.34";
 
 
   var utils = /*#__PURE__*/Object.freeze({
@@ -11665,8 +11665,16 @@
       }
       cands.forEach(function(cand) {
         var p = getTestPoint(cand.ids);
-        var isEnclosed = b.containsPoint(p[0], p[1]) && (index ?
-          index.pointInPolygon(p[0], p[1]) : geom.testPointInRing(p[0], p[1], pathIds, arcs));
+        var isEnclosed = b.containsPoint(p[0], p[1]) &&
+          // added a bounds-in-bounds test to handle a case where the test point
+          // fell along the shared boundary of two rings, but the rings did no overlap
+          // (this gave a false positive for the enclosure test)
+          // (for speed, the midpoint of an arc is used as the test point; this
+          // works well in the typical case where rings to not share an edge.
+          // Finding an internal test point would be better, we just need a fast
+          // function to find internal points)
+          b.contains(cand.bounds) &&
+          (index ? index.pointInPolygon(p[0], p[1]) : geom.testPointInRing(p[0], p[1], pathIds, arcs));
         if (isEnclosed) {
           paths.push(cand.ids);
         }
@@ -11674,6 +11682,7 @@
       return paths.length > 0 ? paths : null;
     };
 
+    // return array of indexed paths within a given shape
     this.findPathsInsideShape = function(shape) {
       var paths = []; // list of enclosed paths
       shape.forEach(function(ids) {
@@ -12075,7 +12084,6 @@
         yy = coords.yy,
         ids = nodes.getConnectedArcs(fromArcId),
         toArcId = fromArcId; // initialize to fromArcId -- an error condition
-
     if (filter) {
       ids = ids.filter(filter);
     }
@@ -12104,14 +12112,12 @@
         continue;
       }
       icand = arcs.indexOfVertex(candId, -2);
-
       if (toArcId == fromArcId) {
         // first valid candidate
         ito = icand;
         toArcId = candId;
         continue;
       }
-
       code = chooseRighthandPath(fromX, fromY, nodeX, nodeY, xx[ito], yy[ito], xx[icand], yy[icand]);
       if (code == 2) {
         ito = icand;
@@ -12119,6 +12125,7 @@
       }
     }
 
+
     if (toArcId == fromArcId) {
       // This shouldn't occur, assuming that other arcs are present
       error("Pathfinder error");
@@ -12183,6 +12190,9 @@
     chooseRighthandVector: chooseRighthandVector
   });
 
+  var FWD_USED = 0x8;
+  var REV_USED = 0x80;
+
   function setBits(bits, arcBits, mask) {
     return (bits & ~mask) | (arcBits & mask);
   }
@@ -12210,6 +12220,16 @@
     return bits & 7;
   }
 
+  function markPathsAsUsed(paths, routesArr) {
+    forEachArcId(paths, function(arcId) {
+      if (arcId < 0) {
+        routesArr[~arcId] |= REV_USED;
+      } else {
+        routesArr[arcId] |= FWD_USED;
+      }
+    });
+  }
+
   // Open arc pathways in a single shape or array of shapes
   //
   function openArcRoutes(paths, arcColl, routesArr, fwd, rev, dissolve, orBits) {
@@ -12400,6 +12420,7 @@
     andBits: andBits,
     setRouteBits: setRouteBits,
     getRouteBits: getRouteBits,
+    markPathsAsUsed: markPathsAsUsed,
     openArcRoutes: openArcRoutes,
     closeArcRoutes: closeArcRoutes,
     getPathFinder: getPathFinder,
@@ -18192,6 +18213,7 @@
     dy: 'measure',
     fill: 'color',
     'fill-pattern': 'pattern',
+    'fill-effect': null, // todo: validate effect names
     'font-family': null,
     'font-size': null,
     'font-style': null,
@@ -18240,7 +18262,7 @@
   var commonProperties = 'css,class,opacity,stroke,stroke-width,stroke-dasharray,stroke-opacity,fill-opacity,vector-effect'.split(',');
 
   var propertiesBySymbolType = {
-    polygon: utils.arrayToIndex(commonProperties.concat('fill', 'fill-pattern')),
+    polygon: utils.arrayToIndex(commonProperties.concat('fill', 'fill-pattern', 'fill-effect')),
     polyline: utils.arrayToIndex(commonProperties.concat('stroke-linecap', 'stroke-linejoin', 'stroke-miterlimit')),
     point: utils.arrayToIndex(commonProperties.concat('fill', 'r')),
     label: utils.arrayToIndex(commonProperties.concat(
@@ -19125,6 +19147,9 @@
       if (obj.tag == 'path' && obj.properties['fill-pattern']) {
         convertFillPattern(obj.properties, defs);
       }
+      if (obj.tag == 'path' && obj.properties['fill-effect'] == 'sphere') {
+        convertSphereEffect(obj.properties, defs);
+      }
       if (obj.tag == 'image') {
         if (/\.svg/.test(obj.properties.href || '')) {
           convertSvgImage(obj, defs);
@@ -19135,6 +19160,25 @@
     }
   }
 
+  function convertSphereEffect(obj, defs) {
+    var id = 'mapshaper_sphere_effect';
+    var href = `url(#${ id })`;
+    var svg =
+`<radialGradient id="${id}" cx="0.5" cy="0.5" r=".56" fx="0.4" fy="0.35">
+  <stop offset=".35" stop-opacity="0"/>
+  <stop offset=".65" stop-opacity="0.1" />
+  <stop offset=".85" stop-opacity="0.45" />
+  <stop offset=".95"  stop-opacity="1" />
+</radialGradient>`  ;
+    if (!utils.find(defs, function(o) { return o.id == id; })) {
+      defs.push({svg, id, href});
+    }
+    obj.fill = href;
+    if ('opacity' in obj === false && 'fill-opacity' in obj === false) {
+      obj['fill-opacity'] = 0.35;
+    }
+  }
+
   function convertSvgImage(obj, defs) {
     // Same-origin policy prevents embedding images in the web UI
     var href = obj.properties.href;
@@ -23896,7 +23940,9 @@ ${svg}
       .option('polygon', {
         describe: 'create a polygon to match the outline of the graticule',
         type: 'flag'
-      });
+      })
+      .option('name', nameOpt);
+
 
     parser.command('grid')
       .describe('create a grid of square or hexagonal polygons')
@@ -24426,6 +24472,9 @@ ${svg}
       .option('fill-pattern', {
         describe: 'pattern fill, ex: "hatches 2px grey 2px blue"'
       })
+      .option('fill-effect', {
+        describe: 'use "sphere" on a circle for a 3d globe effect'
+      })
       .option('fill-opacity', {
         describe: 'fill opacity'
       })
@@ -33401,7 +33450,7 @@ ${svg}
     var clipArcTouches = 0;
     var clipArcUses = 0;
     var usedClipArcs = [];
-    var dividePath = getPathFinder(nodes, useRoute, routeIsActive);
+    var findPath = getPathFinder(nodes, useRoute, routeIsActive);
     var dissolvePolygon = getPolygonDissolver(nodes);
 
     // The following cleanup step is a performance bottleneck (it often takes longer than
@@ -33434,12 +33483,16 @@ ${svg}
       return null;
     });
 
+    markPathsAsUsed(clippedShapes, routeFlags); // to help us find unused paths later
+
+
     // add clip/erase polygons that are fully contained in a target polygon
     // need to index only non-intersecting clip shapes
     // (Intersecting shapes have one or more arcs that have been scanned)
 
     // first, find shapes that do not intersect the target layer
     // (these could be inside or outside the target polygons)
+
     var undividedClipShapes = findUndividedClipShapes(clipShapes);
 
     closeArcRoutes(clipShapes, arcs, routeFlags, true, true); // not needed?
@@ -33468,7 +33521,7 @@ ${svg}
         for (var i=0, n=ids.length; i<n; i++) {
           clipArcTouches = 0;
           clipArcUses = 0;
-          path = dividePath(ids[i]);
+          path = findPath(ids[i]);
           if (path) {
             // if ring doesn't touch/intersect a clip/erase polygon, check if it is contained
             // if (clipArcTouches === 0) {
@@ -33487,6 +33540,7 @@ ${svg}
         }
       });
 
+
       // Clear pathways of current target shape to hidden/closed
       closeArcRoutes(shape, arcs, routeFlags, true, true, true);
       // Also clear pathways of any clip arcs that were used
@@ -33557,8 +33611,9 @@ ${svg}
       return usable;
     }
 
-    // Filter a collection of shapes to exclude paths that contain clip/erase arcs
-    // and paths that are hidden (e.g. internal boundaries)
+
+    // Filter a collection of shapes to exclude paths that incorporate parts of
+    // clip/erase polygons and paths that are hidden (e.g. internal boundaries)
     function findUndividedClipShapes(clipShapes) {
       return clipShapes.map(function(shape) {
         var usableParts = [];
@@ -33582,12 +33637,12 @@ ${svg}
       });
     }
 
-    // Test if arc is unused in both directions
-    // (not testing open/closed or visible/hidden)
+
     function arcIsUnused(id, flags) {
       var abs = absArcId(id),
           flag = flags[abs];
-          return (flag & 0x44) === 0;
+          return (flag & 0x88) === 0;
+          // return id < 0 ? (flag & 0x80) === 0 : (flag & 0x8) === 0;
     }
 
     function arcIsVisible(id, flags) {
@@ -33610,7 +33665,7 @@ ${svg}
         enclosedPaths.forEach(function(ids) {
           var path;
           for (var j=0; j<ids.length; j++) {
-            path = dividePath(ids[j]);
+            path = findPath(ids[j]);
             if (path) {
               dissolvedPaths.push(path);
             }
@@ -37902,7 +37957,8 @@ ${svg}
     return {
       gnom: 60,
       laea: 179,
-      ortho: 89.9, // TODO: investigate projection errors closer to 90
+      //ortho: 89.9, // projection errors betwen lat +/-35 to 55
+      ortho: 89.85, // TODO: investigate
       stere: 142,
       sterea: 142,
       ups: 10.5 // TODO: should be 6.5 deg at north pole
@@ -38281,7 +38337,7 @@ ${svg}
   });
 
   cmd.graticule = function(dataset, opts) {
-    var name = opts.polygon ? 'polygon' : 'graticule';
+    var name = opts.name || opts.polygon && 'polygon' || 'graticule';
     var graticule, destInfo;
     if (dataset && !isLatLngDataset(dataset)) {
       // project graticule to match dataset
@@ -38347,7 +38403,7 @@ ${svg}
   //
   function createGraticule(P, outlined, opts) {
     var interval = opts.interval || 10;
-    if (![5,10,15,30,45].includes(interval)) stop('Invalid interval:', interval);
+    if (![5,10,15,20,30,45].includes(interval)) stop('Invalid interval:', interval);
     P.lam0 * 180 / Math.PI;
     var precision = interval > 10 ? 1 : 0.5; // degrees between each vertex
     var xstep = interval;
@@ -40935,6 +40991,7 @@ ${svg}
     }
     commandStr = runGlobalExpression(opts.expression, targets);
     if (commandStr) {
+      message(`command: [${commandStr}]`);
       commands = parseCommands(commandStr);
       runParsedCommands(commands, job, cb);
     } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mapshaper",
-  "version": "0.6.32",
+  "version": "0.6.34",
   "description": "A tool for editing vector datasets for mapping and GIS.",
   "keywords": [
     "shapefile",

package/www/mapshaper-gui.js

@@ -4038,8 +4038,11 @@
     }
 
     function updateMenuBtn() {
-      var name = model.getActiveLayer().layer.name || "[unnamed layer]";
-      btn.classed('active', 'true').findChild('.layer-name').html(name + "  ▼");
+      var lyrName = model.getActiveLayer().layer.name || '';
+      var menuTitle = lyrName || '[unnamed layer]';
+      var pageTitle = lyrName || 'mapshaper';
+      btn.classed('active', 'true').findChild('.layer-name').html(menuTitle + "  ▼");
+      window.document.title = pageTitle;
     }
 
     function render() {
@@ -10172,12 +10175,12 @@
       return action == 'hover' && _overlayCanv.visible();
     }
 
-    function drawCanvasLayer(target, canv) {
-      if (!target) return;
-      if (target.style.type == 'outline') {
-        drawOutlineLayerToCanvas(target, canv, ext);
+    function drawCanvasLayer(lyr, canv) {
+      if (!lyr) return;
+      if (lyr.style.type == 'outline') {
+        drawOutlineLayerToCanvas(lyr, canv, ext);
       } else {
-        drawStyledLayerToCanvas(target, canv, ext);
+        drawStyledLayerToCanvas(lyr, canv, ext);
       }
     }
 
@@ -11039,7 +11042,9 @@
       } else {
         _overlayLyr = null;
       }
-      drawLayers('hover');
+      // 'hover' bypasses style creation in drawLayers2()... sometimes we need that
+      // drawLayers('hover');
+      drawLayers();
     }
 
     function getDisplayOptions() {
@@ -11054,12 +11059,14 @@
       return flags.simplify_method || flags.simplify || flags.proj ||
         flags.arc_count || flags.repair || flags.clip || flags.erase ||
         flags.slice || flags.affine || flags.rectangle || flags.buffer ||
-        flags.union || flags.mosaic || flags.snap || flags.clean || false;
+        flags.union || flags.mosaic || flags.snap || flags.clean || flags.drop || false;
     }
 
     // Test if an update allows hover popup to stay open
     function popupCanStayOpen(flags) {
-      // if (arcsMayHaveChanged(flags)) return false;
+      // keeping popup open after -drop geometry causes problems...
+      // // if (arcsMayHaveChanged(flags)) return false;
+      if (arcsMayHaveChanged(flags)) return false;
       if (flags.points || flags.proj) return false;
       if (!flags.same_table) return false;
       return true;

package/www/mapshaper.js

@@ -1,6 +1,6 @@
 (function () {
 
-  var VERSION = "0.6.32";
+  var VERSION = "0.6.34";
 
 
   var utils = /*#__PURE__*/Object.freeze({
@@ -11665,8 +11665,16 @@
       }
       cands.forEach(function(cand) {
         var p = getTestPoint(cand.ids);
-        var isEnclosed = b.containsPoint(p[0], p[1]) && (index ?
-          index.pointInPolygon(p[0], p[1]) : geom.testPointInRing(p[0], p[1], pathIds, arcs));
+        var isEnclosed = b.containsPoint(p[0], p[1]) &&
+          // added a bounds-in-bounds test to handle a case where the test point
+          // fell along the shared boundary of two rings, but the rings did no overlap
+          // (this gave a false positive for the enclosure test)
+          // (for speed, the midpoint of an arc is used as the test point; this
+          // works well in the typical case where rings to not share an edge.
+          // Finding an internal test point would be better, we just need a fast
+          // function to find internal points)
+          b.contains(cand.bounds) &&
+          (index ? index.pointInPolygon(p[0], p[1]) : geom.testPointInRing(p[0], p[1], pathIds, arcs));
         if (isEnclosed) {
           paths.push(cand.ids);
         }
@@ -11674,6 +11682,7 @@
       return paths.length > 0 ? paths : null;
     };
 
+    // return array of indexed paths within a given shape
     this.findPathsInsideShape = function(shape) {
       var paths = []; // list of enclosed paths
       shape.forEach(function(ids) {
@@ -12075,7 +12084,6 @@
         yy = coords.yy,
         ids = nodes.getConnectedArcs(fromArcId),
         toArcId = fromArcId; // initialize to fromArcId -- an error condition
-
     if (filter) {
       ids = ids.filter(filter);
     }
@@ -12104,14 +12112,12 @@
         continue;
       }
       icand = arcs.indexOfVertex(candId, -2);
-
       if (toArcId == fromArcId) {
         // first valid candidate
         ito = icand;
         toArcId = candId;
         continue;
       }
-
       code = chooseRighthandPath(fromX, fromY, nodeX, nodeY, xx[ito], yy[ito], xx[icand], yy[icand]);
       if (code == 2) {
         ito = icand;
@@ -12119,6 +12125,7 @@
       }
     }
 
+
     if (toArcId == fromArcId) {
       // This shouldn't occur, assuming that other arcs are present
       error("Pathfinder error");
@@ -12183,6 +12190,9 @@
     chooseRighthandVector: chooseRighthandVector
   });
 
+  var FWD_USED = 0x8;
+  var REV_USED = 0x80;
+
   function setBits(bits, arcBits, mask) {
     return (bits & ~mask) | (arcBits & mask);
   }
@@ -12210,6 +12220,16 @@
     return bits & 7;
   }
 
+  function markPathsAsUsed(paths, routesArr) {
+    forEachArcId(paths, function(arcId) {
+      if (arcId < 0) {
+        routesArr[~arcId] |= REV_USED;
+      } else {
+        routesArr[arcId] |= FWD_USED;
+      }
+    });
+  }
+
   // Open arc pathways in a single shape or array of shapes
   //
   function openArcRoutes(paths, arcColl, routesArr, fwd, rev, dissolve, orBits) {
@@ -12400,6 +12420,7 @@
     andBits: andBits,
     setRouteBits: setRouteBits,
     getRouteBits: getRouteBits,
+    markPathsAsUsed: markPathsAsUsed,
     openArcRoutes: openArcRoutes,
     closeArcRoutes: closeArcRoutes,
     getPathFinder: getPathFinder,
@@ -18192,6 +18213,7 @@
     dy: 'measure',
     fill: 'color',
     'fill-pattern': 'pattern',
+    'fill-effect': null, // todo: validate effect names
     'font-family': null,
     'font-size': null,
     'font-style': null,
@@ -18240,7 +18262,7 @@
   var commonProperties = 'css,class,opacity,stroke,stroke-width,stroke-dasharray,stroke-opacity,fill-opacity,vector-effect'.split(',');
 
   var propertiesBySymbolType = {
-    polygon: utils.arrayToIndex(commonProperties.concat('fill', 'fill-pattern')),
+    polygon: utils.arrayToIndex(commonProperties.concat('fill', 'fill-pattern', 'fill-effect')),
     polyline: utils.arrayToIndex(commonProperties.concat('stroke-linecap', 'stroke-linejoin', 'stroke-miterlimit')),
     point: utils.arrayToIndex(commonProperties.concat('fill', 'r')),
     label: utils.arrayToIndex(commonProperties.concat(
@@ -19125,6 +19147,9 @@
       if (obj.tag == 'path' && obj.properties['fill-pattern']) {
         convertFillPattern(obj.properties, defs);
       }
+      if (obj.tag == 'path' && obj.properties['fill-effect'] == 'sphere') {
+        convertSphereEffect(obj.properties, defs);
+      }
       if (obj.tag == 'image') {
         if (/\.svg/.test(obj.properties.href || '')) {
           convertSvgImage(obj, defs);
@@ -19135,6 +19160,25 @@
     }
   }
 
+  function convertSphereEffect(obj, defs) {
+    var id = 'mapshaper_sphere_effect';
+    var href = `url(#${ id })`;
+    var svg =
+`<radialGradient id="${id}" cx="0.5" cy="0.5" r=".56" fx="0.4" fy="0.35">
+  <stop offset=".35" stop-opacity="0"/>
+  <stop offset=".65" stop-opacity="0.1" />
+  <stop offset=".85" stop-opacity="0.45" />
+  <stop offset=".95"  stop-opacity="1" />
+</radialGradient>`  ;
+    if (!utils.find(defs, function(o) { return o.id == id; })) {
+      defs.push({svg, id, href});
+    }
+    obj.fill = href;
+    if ('opacity' in obj === false && 'fill-opacity' in obj === false) {
+      obj['fill-opacity'] = 0.35;
+    }
+  }
+
   function convertSvgImage(obj, defs) {
     // Same-origin policy prevents embedding images in the web UI
     var href = obj.properties.href;
@@ -23896,7 +23940,9 @@ ${svg}
       .option('polygon', {
         describe: 'create a polygon to match the outline of the graticule',
         type: 'flag'
-      });
+      })
+      .option('name', nameOpt);
+
 
     parser.command('grid')
       .describe('create a grid of square or hexagonal polygons')
@@ -24426,6 +24472,9 @@ ${svg}
       .option('fill-pattern', {
         describe: 'pattern fill, ex: "hatches 2px grey 2px blue"'
       })
+      .option('fill-effect', {
+        describe: 'use "sphere" on a circle for a 3d globe effect'
+      })
       .option('fill-opacity', {
         describe: 'fill opacity'
       })
@@ -33401,7 +33450,7 @@ ${svg}
     var clipArcTouches = 0;
     var clipArcUses = 0;
     var usedClipArcs = [];
-    var dividePath = getPathFinder(nodes, useRoute, routeIsActive);
+    var findPath = getPathFinder(nodes, useRoute, routeIsActive);
     var dissolvePolygon = getPolygonDissolver(nodes);
 
     // The following cleanup step is a performance bottleneck (it often takes longer than
@@ -33434,12 +33483,16 @@ ${svg}
       return null;
     });
 
+    markPathsAsUsed(clippedShapes, routeFlags); // to help us find unused paths later
+
+
     // add clip/erase polygons that are fully contained in a target polygon
     // need to index only non-intersecting clip shapes
     // (Intersecting shapes have one or more arcs that have been scanned)
 
     // first, find shapes that do not intersect the target layer
     // (these could be inside or outside the target polygons)
+
     var undividedClipShapes = findUndividedClipShapes(clipShapes);
 
     closeArcRoutes(clipShapes, arcs, routeFlags, true, true); // not needed?
@@ -33468,7 +33521,7 @@ ${svg}
         for (var i=0, n=ids.length; i<n; i++) {
           clipArcTouches = 0;
           clipArcUses = 0;
-          path = dividePath(ids[i]);
+          path = findPath(ids[i]);
           if (path) {
             // if ring doesn't touch/intersect a clip/erase polygon, check if it is contained
             // if (clipArcTouches === 0) {
@@ -33487,6 +33540,7 @@ ${svg}
         }
       });
 
+
       // Clear pathways of current target shape to hidden/closed
       closeArcRoutes(shape, arcs, routeFlags, true, true, true);
       // Also clear pathways of any clip arcs that were used
@@ -33557,8 +33611,9 @@ ${svg}
       return usable;
     }
 
-    // Filter a collection of shapes to exclude paths that contain clip/erase arcs
-    // and paths that are hidden (e.g. internal boundaries)
+
+    // Filter a collection of shapes to exclude paths that incorporate parts of
+    // clip/erase polygons and paths that are hidden (e.g. internal boundaries)
     function findUndividedClipShapes(clipShapes) {
       return clipShapes.map(function(shape) {
         var usableParts = [];
@@ -33582,12 +33637,12 @@ ${svg}
       });
     }
 
-    // Test if arc is unused in both directions
-    // (not testing open/closed or visible/hidden)
+
     function arcIsUnused(id, flags) {
       var abs = absArcId(id),
           flag = flags[abs];
-          return (flag & 0x44) === 0;
+          return (flag & 0x88) === 0;
+          // return id < 0 ? (flag & 0x80) === 0 : (flag & 0x8) === 0;
     }
 
     function arcIsVisible(id, flags) {
@@ -33610,7 +33665,7 @@ ${svg}
         enclosedPaths.forEach(function(ids) {
           var path;
           for (var j=0; j<ids.length; j++) {
-            path = dividePath(ids[j]);
+            path = findPath(ids[j]);
             if (path) {
               dissolvedPaths.push(path);
             }
@@ -37902,7 +37957,8 @@ ${svg}
     return {
       gnom: 60,
       laea: 179,
-      ortho: 89.9, // TODO: investigate projection errors closer to 90
+      //ortho: 89.9, // projection errors betwen lat +/-35 to 55
+      ortho: 89.85, // TODO: investigate
       stere: 142,
       sterea: 142,
       ups: 10.5 // TODO: should be 6.5 deg at north pole
@@ -38281,7 +38337,7 @@ ${svg}
   });
 
   cmd.graticule = function(dataset, opts) {
-    var name = opts.polygon ? 'polygon' : 'graticule';
+    var name = opts.name || opts.polygon && 'polygon' || 'graticule';
     var graticule, destInfo;
     if (dataset && !isLatLngDataset(dataset)) {
       // project graticule to match dataset
@@ -38347,7 +38403,7 @@ ${svg}
   //
   function createGraticule(P, outlined, opts) {
     var interval = opts.interval || 10;
-    if (![5,10,15,30,45].includes(interval)) stop('Invalid interval:', interval);
+    if (![5,10,15,20,30,45].includes(interval)) stop('Invalid interval:', interval);
     P.lam0 * 180 / Math.PI;
     var precision = interval > 10 ? 1 : 0.5; // degrees between each vertex
     var xstep = interval;
@@ -40935,6 +40991,7 @@ ${svg}
     }
     commandStr = runGlobalExpression(opts.expression, targets);
     if (commandStr) {
+      message(`command: [${commandStr}]`);
       commands = parseCommands(commandStr);
       runParsedCommands(commands, job, cb);
     } else {