Transform implicit string literal insert into explicit insert actions when sibling statements are present

Author: AndreasArvidsson

out/lib.js

@@ -34,6 +34,8 @@ export interface Logger {
     log(message: string): void;
     warn(message: string): void;
     error(message: string): void;
+}
+export interface TestLogger extends Logger {
     getEntries(): readonly LoggerEntry[];
 }
 export interface DebugLogger {

out/snippetFormatter.js

@@ -0,0 +1,5 @@
+export declare class FilePatternError extends Error {
+    messages: string[];
+    name: string;
+    constructor(messages: string[]);
+}

out/talonFormatter.js

@@ -1,3 +1,4 @@
-import type { DebugLogger, Logger } from "../types.js";
+import type { DebugLogger, Logger, TestLogger } from "../types.js";
 export declare function createLogger(quiet?: boolean): Logger;
+export declare function createTestLogger(): TestLogger;
 export declare function createDebugLogger(debug: boolean): DebugLogger;

out/treeSitterFormatter.js

@@ -1,6 +1,6 @@
 {
     "name": "@cursorless/talon-tools",
-    "version": "0.4.1",
+    "version": "0.5.0",
     "description": "Linting and formatting tools for Talon and Cursorless",
     "author": "Cursorless Dev",
     "license": "MIT",

out/types.d.ts

@@ -223,6 +223,9 @@ class TalonFormatter {
             case "choice":
                 return node.children.map((n) => this.getNodeText(n)).join(" ");
 
+            case "string":
+                return formatString(node);
+
             case "tag_binding":
             case "settings_binding":
             case "capture":
@@ -237,7 +240,6 @@ class TalonFormatter {
             case "variable":
             case "word":
             case "binary_operator":
-            case "string":
             case "integer":
             case "float":
             case "start_anchor":
@@ -278,3 +280,18 @@ function rangeEqual(a: SyntaxNode, b: SyntaxNode): boolean {
 function isFirstChild(node: SyntaxNode): boolean {
     return node.id === node.parent?.children?.[0]?.id;
 }
+
+function formatString(node: SyntaxNode): string {
+    // A single string literal is allowed as syntactic sugar for the insert
+    // action, but not in combination with other sibling statements.
+    if (
+        node.parent?.type === "expression_statement" &&
+        node.parent.parent?.type === "block" &&
+        rangeEqual(node, node.parent) &&
+        node.parent.parent.children.length > 1
+    ) {
+        return `insert(${node.text})`;
+    }
+
+    return node.text;
+}