Tests / fixes for moveBy (#52)

Adds tests to verify working unit motions, and simplifies and corrects
the unit-motion logic.

.eslintrc.json

@@ -37,7 +37,8 @@
             {
                 "allowModules": [
                     "lodash",
-                    "wdio-vscode-service"
+                    "wdio-vscode-service",
+                    "string.prototype.replaceall"
                 ]
             }
         ],

.gitignore

@@ -8,7 +8,7 @@ dist/*
 .DS_Store
 .cache/
 .parcel-cache
-wdio.conf.mts
 .wdio-vscode-service
 wdio.log
 coverage/**
+.package.json

.vscode/launch.json

@@ -24,6 +24,7 @@
 			"debugWebWorkerHost": true,
 			"request": "launch",
 			"args": [
+                "--profile=debug-profile",
 				"--extensionDevelopmentPath=${workspaceFolder}",
 				"--extensionDevelopmentKind=web"
 			],

.vscode/tasks.json

@@ -6,7 +6,7 @@
 		{
 			"label": "npm: compile-web",
 			"type": "shell",
-			"command": "bash ./npm-run.sh compile",
+			"command": "bash ./npm-run.sh compile-web",
 			"group": {
 				"kind": "build",
 				"isDefault": true
@@ -19,7 +19,7 @@
 		{
 			"label": "npm: watch-web",
 			"type": "shell",
-			"command": "bash ./npm-run.sh watch",
+			"command": "bash ./npm-run.sh watch-web",
 			"group": "build",
 			"isBackground": true,
 			"problemMatcher": [

debug-profile.code-profile

@@ -0,0 +1,3 @@
+{
+  "name": "debug-profile"
+}

notes.md

@@ -0,0 +1,4 @@
+Currently debugging issue with `resolveUnit` in `unitMotions`
+
+problem: when using start or end boundaries (instead of both), the selectWhole
+skips every even unit

npm-run.sh

@@ -1,5 +1,6 @@
 #!/bin/bash
-export NVM_DIR="$([ -z "${XDG_CONFIG_HOME-}" ] && printf %s "${HOME}/.nvm" || printf %s "${XDG_CONFIG_HOME}/nvm")"
+NVM_DIR="$([ -z "${XDG_CONFIG_HOME-}" ] && printf %s "${HOME}/.nvm" || printf %s "${XDG_CONFIG_HOME}/nvm")"
 [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh" # load nvm
+nvm install
 nvm use
-npm run $@
+npm run "$@"

package.json

@@ -4,7 +4,7 @@
   "publisher": "haberdashPI",
   "repository": "https://github.com/haberdashPI/vscode-selection-utilities",
   "description": "Kakaune-inspired collection of useful commands for manipulating selections.",
-  "version": "0.6.6",
+  "version": "0.6.7",
   "icon": "logo.png",
   "engines": {
     "vscode": "^1.92.0"
@@ -46,6 +46,10 @@
                 "name": "word",
                 "regex": "(_*[\\p{L}][_\\p{L}0-9]*)|(_+)|([0-9][0-9.]*)|((?<=[\\s\\r\\n])[^\\p{L}^\\s]+(?=[\\s\\r\\n]))"
               },
+              {
+                "name": "number",
+                "regex": "[0-9]+"
+              },
               {
                 "name": "subword",
                 "regex": "(_*[\\p{L}][0-9\\p{Ll}]+_*)|(_+)|(\\p{Lu}[\\p{Lu}0-9]+_*(?!\\p{Ll}))|(\\p{L})|([^\\p{L}^\\s^0-9])|([0-9][0-9.]*)"
@@ -450,6 +454,7 @@
     "nyc": "^17.0.0",
     "penv": "^0.2.0",
     "process": "^0.11.10",
+    "string.prototype.replaceall": "^1.0.10",
     "ts-loader": "^9.5.1",
     "typescript": "^5.4.3",
     "wdio-vscode-service": "^6.0.3",

src/web/activeMotions.ts

@@ -54,7 +54,9 @@ function revealActive(args: {at: 'top' | 'center' | 'bottom'} = {at: 'center'})
     }
 }
 
-function activePageMove(args: {dir?: 'up' | 'down'; count?: number, select?: boolean} = {}) {
+function activePageMove(
+    args: {dir?: 'up' | 'down'; count?: number; select?: boolean} = {}
+) {
     const editor = vscode.window.activeTextEditor;
     if (editor) {
         const heights = editor.visibleRanges.map(range => {
@@ -69,7 +71,8 @@ function activePageMove(args: {dir?: 'up' | 'down'; count?: number, select?: boo
         editor.selections = editor.selections.map(sel => {
             const active = clampedLineTranslate(sel.active, ed.document, steps);
             let anchor = sel.anchor;
-            if (args.select === false) { // args.select === undefined defaults to true
+            if (args.select === false) {
+                // args.select === undefined defaults to true
                 anchor = active;
             }
             return new vscode.Selection(anchor, active);