pi-vim 0.3.0 → 0.3.2

package/README.md

@@ -354,9 +354,9 @@ These are **explicitly deferred** and not planned for this feature:
 - `types.ts` — shared types and escape-sequence constants.
 - `test/` — Node test runner suite; no browser / full runtime required.
 
-Run tests:
+Run checks:
 
 ```
 cd pi-vim
-npm test
+npm run check
 ```

package/index.ts

@@ -140,8 +140,8 @@ export class ModalEditor extends CustomEditor {
 
   // Unnamed register
   private unnamedRegister: string = "";
-  private clipboardFn: (text: string) => void = (text: string) => {
-    try { copyToClipboard(text); } catch { /* best effort */ }
+  private clipboardFn: (text: string) => Promise<void> = async (text: string) => {
+    await copyToClipboard(text);
   };
 
   constructor(
@@ -155,7 +155,11 @@ export class ModalEditor extends CustomEditor {
   }
 
   // Test seams
-  setClipboardFn(fn: (text: string) => void): void { this.clipboardFn = fn; }
+  setClipboardFn(fn: (text: string) => unknown): void {
+    this.clipboardFn = async (text: string) => {
+      await fn(text);
+    };
+  }
   getRegister(): string { return this.unnamedRegister; }
   setRegister(text: string): void { this.unnamedRegister = text; }
   getMode(): Mode { return this.mode; }
@@ -181,7 +185,7 @@ export class ModalEditor extends CustomEditor {
     if (!state || !Array.isArray(state.lines)) {
       throw new Error("Redo restore prerequisite: editor state unavailable");
     }
-    return state;
+    return state as { lines: string[]; cursorLine?: number; cursorCol?: number };
   }
 
   private restoreSnapshot(snapshot: EditorSnapshot): void {
@@ -1315,7 +1319,7 @@ export class ModalEditor extends CustomEditor {
   }
 
   private moveCursorToFirstNonWhitespace(): void {
-    const { line, col } = this.getCurrentLineAndCol();
+    const { line } = this.getCurrentLineAndCol();
     const targetCol = findFirstNonWhitespaceColumn(line);
     this.moveCursorToCol(targetCol);
   }
@@ -1618,7 +1622,8 @@ export class ModalEditor extends CustomEditor {
   private writeToRegister(text: string): void {
     this.unnamedRegister = text;
     if (!text) return;
-    this.clipboardFn(text);
+
+    void this.clipboardFn(text).catch(() => {});
   }
 
   private getCurrentLineAndCol(): { line: string; col: number } {
@@ -2008,7 +2013,7 @@ export class ModalEditor extends CustomEditor {
     this.writeToRegister(line.slice(start, end));
   }
 
-  private yankRangeByAbsolute(currentAbs: number, targetAbs: number, inclusive: boolean): void {
+  private yankRangeByAbsolute(currentAbs: number, targetAbs: number, inclusive: boolean = false): void {
     const text = this.getText();
     const start = Math.min(currentAbs, targetAbs);
     const rawEnd = Math.max(currentAbs, targetAbs) + (inclusive ? 1 : 0);

package/motions.ts

@@ -172,7 +172,7 @@ export function findCharMotionTarget(
     const tillRepeatOffset = isFirst && isTill && isRepeat ? 1 : 0;
 
     if (isForward) {
-      let nextIndex = currentIndex + 1 + tillRepeatOffset;
+      const nextIndex = currentIndex + 1 + tillRepeatOffset;
       let found = -1;
       for (let j = nextIndex; j < graphemes.length; j++) {
         const g = graphemes[j]!;
@@ -187,7 +187,7 @@ export function findCharMotionTarget(
       if (isFinal) return isTill ? graphemes[found - 1]!.start : graphemes[found]!.start;
       currentIndex = found;
     } else {
-      let nextIndex = currentIndex - 1 - tillRepeatOffset;
+      const nextIndex = currentIndex - 1 - tillRepeatOffset;
       let found = -1;
       for (let j = nextIndex; j >= 0; j--) {
         const g = graphemes[j]!;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-vim",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "Vim-style modal editing for Pi's TUI editor",
   "type": "module",
   "keywords": [
@@ -22,10 +22,12 @@
   ],
   "scripts": {
     "build": "echo 'nothing to build'",
+    "lint": "eslint .",
+    "typecheck": "tsc --noEmit",
     "test": "node --import tsx/esm --test 'test/**/*.test.ts'",
-    "check": "npm run test",
+    "check": "npm run lint && npm run typecheck && npm run test",
     "pack:check": "node --import tsx/esm script/pack-check.ts",
-    "prepublishOnly": "npm run pack:check && npm test"
+    "prepublishOnly": "npm run lint && npm run typecheck && npm run pack:check && npm test"
   },
   "pi": {
     "extensions": [
@@ -33,7 +35,12 @@
     ]
   },
   "devDependencies": {
-    "tsx": "^4.19.3"
+    "@eslint/js": "^9.25.1",
+    "@types/node": "^24.7.2",
+    "eslint": "^9.25.1",
+    "tsx": "^4.19.3",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.31.1"
   },
   "peerDependencies": {
     "@mariozechner/pi-coding-agent": "*",