espolar 0.3.1 → 0.4.0

package/dist/index.d.ts

@@ -94,7 +94,15 @@ interface PrinterContext<Data = any> {
   writeNode(node: AST.Node | null | undefined): void;
   writeNodeList(nodes: readonly (AST.Node | null)[], separator: string): void;
   writeExpressionListWithCommaSep(nodes: readonly (AST.Expression | AST.SpreadElement | null)[]): void;
-  writeNodeListWithNewLineSep(nodes: readonly (AST.ProgramStatement | AST.ClassElement)[]): void;
+  /**
+   * Write `nodes` with "newline" separator, but if the node is ranged and either:
+   * - it is the first node with `lastRangeEnd` provided, or
+   * - the previous adjacent node is ranged too,
+   * Then the source text ranging between the two adjacent nodes will be preserved instead of printing a newline character.
+   * @param nodes
+   * @param lastRangeEnd
+   */
+  writeNodeListWithNewLineSep(nodes: readonly (AST.ProgramStatement | AST.ClassElement)[], lastRangeEnd?: number): void;
   writeSource(start: number, end: number, data?: Data): void;
   writePreservedNode(node: AST.Node): void;
   appendMapping(sourceRange: SourceRange, generatedStart: number, generatedEnd: number, data?: Data): void;

package/dist/index.js

@@ -1,4 +1,8 @@
 //#region src/printers.ts
+/**
+* Precedence table for expressions.
+* @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Operator_Precedence#table
+*/
 const EXPRESSIONS_PRECEDENCE = {
 	ArrayPattern: 20,
 	ObjectPattern: 20,
@@ -274,7 +278,7 @@ const defaultPrinters = {
 	TSIntrinsicKeyword: printKeywordType
 };
 function printProgram(program, context) {
-	context.writeNodeListWithNewLineSep(program.body);
+	context.writeNodeListWithNewLineSep(program.body, program.range?.[0]);
 }
 function canStartExpressionStatement(node) {
 	let lhs;
@@ -335,8 +339,11 @@ function printVariableDeclaration(declaration, context) {
 	context.write(";");
 }
 function printVariableDeclarator(declarator, context) {
-	context.writeNode(declarator.id);
-	if (declarator.definite === true) context.write("!");
+	if (declarator.definite === true) {
+		context.write(String(declarator.id.name));
+		context.write("!");
+		writeOptionalTypeAnnotation(declarator.id, context);
+	} else context.writeNode(declarator.id);
 	if (declarator.init) {
 		context.write(" = ");
 		if (expectAssignmentExprNeedsParen(declarator.init)) {
@@ -350,8 +357,10 @@ function printBlockStatement(block, context) {
 	const body = block.body;
 	context.write("{");
 	if (body.length > 0) {
-		context.write("\n");
-		context.writeNodeListWithNewLineSep(body);
+		let lastRangeEnd;
+		if (block.range) lastRangeEnd = block.range[0] + 1;
+		else context.write("\n");
+		context.writeNodeListWithNewLineSep(body, lastRangeEnd);
 		context.write("\n");
 	}
 	context.write("}");
@@ -870,8 +879,10 @@ function printClassBody(node, context) {
 	context.write("{");
 	const body = node.body;
 	if (body.length > 0) {
-		context.write("\n");
-		context.writeNodeListWithNewLineSep(body);
+		let lastRangeEnd;
+		if (node.range) lastRangeEnd = node.range[0] + 1;
+		else context.write("\n");
+		context.writeNodeListWithNewLineSep(body, lastRangeEnd);
 		context.write("\n");
 	}
 	context.write("}");
@@ -880,8 +891,10 @@ function printStaticBlock(node, context) {
 	context.write("static {");
 	const body = node.body;
 	if (body.length > 0) {
-		context.write("\n");
-		context.writeNodeListWithNewLineSep(body);
+		let lastRangeEnd;
+		if (node.range) lastRangeEnd = node.range[0] + 1;
+		else context.write("\n");
+		context.writeNodeListWithNewLineSep(body, lastRangeEnd);
 		context.write("\n");
 	}
 	context.write("}");
@@ -1425,8 +1438,11 @@ function printTSModuleDeclaration(node, context) {
 	}
 }
 function printTSModuleBlock(node, context) {
-	context.write("{\n");
-	context.writeNodeListWithNewLineSep(node.body);
+	context.write("{");
+	let lastRangeEnd;
+	if (node.range) lastRangeEnd = node.range[0] + 1;
+	else context.write("\n");
+	context.writeNodeListWithNewLineSep(node.body, lastRangeEnd);
 	context.write("\n}");
 }
 function printTSDeclareFunction(node, context) {
@@ -1639,14 +1655,12 @@ function createPrinterContext(options) {
 				needsSeparator = true;
 			}
 		},
-		writeNodeListWithNewLineSep(nodes) {
-			let lastRangeEnd;
+		writeNodeListWithNewLineSep(nodes, lastRangeEnd) {
 			let wroteNode = false;
 			for (const node of nodes) {
-				if (!node) continue;
 				const range = getNodeRange(node);
-				if (wroteNode) if (lastRangeEnd !== void 0 && range && range.start >= lastRangeEnd) context.writeSource(lastRangeEnd, range.start, getMappingData(null));
-				else context.write("\n");
+				if (lastRangeEnd !== void 0 && range && range.start >= lastRangeEnd) context.writeSource(lastRangeEnd, range.start, getMappingData(null));
+				else if (wroteNode) context.write("\n");
 				context.writeNode(node);
 				wroteNode = true;
 				if (range) lastRangeEnd = range.end;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "espolar",
-  "version": "0.3.1",
+  "version": "0.4.0",
   "type": "module",
   "types": "./dist/index.d.ts",
   "exports": {

package/src/api.ts

@@ -57,8 +57,17 @@ export interface PrinterContext<Data = any> {
   writeExpressionListWithCommaSep(
     nodes: readonly (AST.Expression | AST.SpreadElement | null)[],
   ): void;
+  /**
+   * Write `nodes` with "newline" separator, but if the node is ranged and either:
+   * - it is the first node with `lastRangeEnd` provided, or
+   * - the previous adjacent node is ranged too,
+   * Then the source text ranging between the two adjacent nodes will be preserved instead of printing a newline character.
+   * @param nodes 
+   * @param lastRangeEnd 
+   */
   writeNodeListWithNewLineSep(
     nodes: readonly (AST.ProgramStatement | AST.ClassElement)[],
+    lastRangeEnd?: number,
   ): void;
   writeSource(start: number, end: number, data?: Data): void;
   writePreservedNode(node: AST.Node): void;

package/src/printer.ts

@@ -220,35 +220,21 @@ function createPrinterContext<Data>(
         needsSeparator = true;
       }
     },
-    writeNodeListWithNewLineSep(nodes) {
-      let lastRangeEnd: number | undefined;
+    writeNodeListWithNewLineSep(nodes, lastRangeEnd) {
       let wroteNode = false;
-
       for (const node of nodes) {
-        if (!node) {
-          continue;
-        }
-
         const range = getNodeRange(node);
-        if (wroteNode) {
-          if (
-            lastRangeEnd !== undefined &&
-            range &&
-            range.start >= lastRangeEnd
-          ) {
-            context.writeSource(
-              lastRangeEnd,
-              range.start,
-              getMappingData(null),
-            );
-          } else {
-            context.write("\n");
-          }
+        if (
+          lastRangeEnd !== undefined &&
+          range &&
+          range.start >= lastRangeEnd
+        ) {
+          context.writeSource(lastRangeEnd, range.start, getMappingData(null));
+        } else if (wroteNode) {
+          context.write("\n");
         }
-
         context.writeNode(node);
         wroteNode = true;
-
         if (range) {
           lastRangeEnd = range.end;
         } else {

package/src/printers.ts

@@ -1,6 +1,10 @@
 import type { PrinterContext, Printers } from "./api.ts";
 import type { AST, AST_NODE_TYPES, Comment } from "./types.ts";
 
+/**
+ * Precedence table for expressions.
+ * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Operator_Precedence#table
+ */
 const EXPRESSIONS_PRECEDENCE = {
   // LHS of =, must NOT parenthesized
   ArrayPattern: 20,
@@ -48,7 +52,9 @@ const EXPRESSIONS_PRECEDENCE = {
 
   // ranges from 13-5 depending on operator
   BinaryExpression: 13,
-  // as/satisfies have same precedence as relational operators
+  // as/satisfies have same precedence as relational operators, e.g.
+  // `1 < 2 as number` is `(1 < 2) as number`, but
+  // `1 === 2 as number` is `1 === (2 as number)`
   TSAsExpression: 9,
   TSSatisfiesExpression: 9,
   // ranges from 4-3 depending on operator
@@ -426,7 +432,7 @@ export const defaultPrinters = {
 // JS – Statements
 
 function printProgram(program: AST.Program, context: PrinterContext): void {
-  context.writeNodeListWithNewLineSep(program.body);
+  context.writeNodeListWithNewLineSep(program.body, program.range?.[0]);
 }
 
 function canStartExpressionStatement(
@@ -518,9 +524,18 @@ function printVariableDeclarator(
   declarator: AST.VariableDeclarator,
   context: PrinterContext,
 ): void {
-  context.writeNode(declarator.id);
   if (declarator.definite === true) {
+    context.write(String((declarator.id as AST.Identifier).name));
     context.write("!");
+    writeOptionalTypeAnnotation(
+      declarator.id as unknown as {
+        optional?: boolean;
+        typeAnnotation?: AST.Node | null;
+      },
+      context,
+    );
+  } else {
+    context.writeNode(declarator.id);
   }
   if (declarator.init) {
     context.write(" = ");
@@ -542,8 +557,13 @@ function printBlockStatement(
   const body = block.body;
   context.write("{");
   if (body.length > 0) {
-    context.write("\n");
-    context.writeNodeListWithNewLineSep(body);
+    let lastRangeEnd: number | undefined;
+    if (block.range) {
+      lastRangeEnd = block.range[0] + 1;
+    } else {
+      context.write("\n");
+    }
+    context.writeNodeListWithNewLineSep(body, lastRangeEnd);
     context.write("\n");
   }
   context.write("}");
@@ -1457,8 +1477,13 @@ function printClassBody(node: AST.ClassBody, context: PrinterContext): void {
   context.write("{");
   const body = node.body;
   if (body.length > 0) {
-    context.write("\n");
-    context.writeNodeListWithNewLineSep(body);
+    let lastRangeEnd: number | undefined;
+    if (node.range) {
+      lastRangeEnd = node.range[0] + 1;
+    } else {
+      context.write("\n");
+    }
+    context.writeNodeListWithNewLineSep(body, lastRangeEnd);
     context.write("\n");
   }
   context.write("}");
@@ -1471,8 +1496,13 @@ function printStaticBlock(
   context.write("static {");
   const body = node.body;
   if (body.length > 0) {
-    context.write("\n");
-    context.writeNodeListWithNewLineSep(body);
+    let lastRangeEnd: number | undefined;
+    if (node.range) {
+      lastRangeEnd = node.range[0] + 1;
+    } else {
+      context.write("\n");
+    }
+    context.writeNodeListWithNewLineSep(body, lastRangeEnd);
     context.write("\n");
   }
   context.write("}");
@@ -2512,8 +2542,14 @@ function printTSModuleBlock(
   node: AST.TSModuleBlock,
   context: PrinterContext,
 ): void {
-  context.write("{\n");
-  context.writeNodeListWithNewLineSep(node.body);
+  context.write("{");
+  let lastRangeEnd: number | undefined;
+  if (node.range) {
+    lastRangeEnd = node.range[0] + 1;
+  } else {
+    context.write("\n");
+  }
+  context.writeNodeListWithNewLineSep(node.body, lastRangeEnd);
   context.write("\n}");
 }