tsl-dx 0.4.0 → 0.5.1

package/dist/index.d.ts

@@ -1,13 +1,5 @@
-import * as tsl0 from "tsl";
+import * as tsl from "tsl";
 
-//#region src/rules/consistent-nullish-comparison.d.ts
-/**
- * Rule to enforce the use of `== null` or `!= null` for nullish comparisons.
- *
- * @since 0.0.0
- */
-declare const consistentNullishComparison: (options?: "off" | undefined) => tsl0.Rule<unknown>;
-//#endregion
 //#region src/rules/no-duplicate-exports.d.ts
 /**
  * Rule to detect and merge duplicate `export from` statements from the same module.
@@ -25,7 +17,7 @@ declare const consistentNullishComparison: (options?: "off" | undefined) => tsl0
  * export { A, B } from 'module';
  * ```
  */
-declare const noDuplicateExports: (options?: "off" | undefined) => tsl0.Rule<unknown>;
+declare const noDuplicateExports: (options?: "off" | undefined) => tsl.Rule<unknown>;
 //#endregion
 //#region src/rules/no-duplicate-imports.d.ts
 /**
@@ -44,6 +36,12 @@ declare const noDuplicateExports: (options?: "off" | undefined) => tsl0.Rule<unk
  * import { A, B } from 'module';
  * ```
  */
-declare const noDuplicateImports: (options?: "off" | undefined) => tsl0.Rule<unknown>;
+declare const noDuplicateImports: (options?: "off" | undefined) => tsl.Rule<unknown>;
+//#endregion
+//#region src/rules/no-multiline-template-expressions-without-auto-dedent.d.ts
+declare const noMultilineTemplateExpressionsWithoutAutoDedent: (options?: "off" | undefined) => tsl.Rule<unknown>;
+//#endregion
+//#region src/rules/nullish.d.ts
+declare const nullish: (options?: "off" | undefined) => tsl.Rule<unknown>;
 //#endregion
-export { consistentNullishComparison, noDuplicateExports, noDuplicateImports };
+export { noDuplicateExports, noDuplicateImports, noMultilineTemplateExpressionsWithoutAutoDedent, nullish };

package/dist/index.js

@@ -1,46 +1,9 @@
-import { P, match } from "ts-pattern";
 import { defineRule } from "tsl";
 import ts, { SyntaxKind } from "typescript";
+import { P, match } from "ts-pattern";
 
-//#region src/rules/consistent-nullish-comparison.ts
-/**
-* Rule to enforce the use of `== null` or `!= null` for nullish comparisons.
-*
-* @since 0.0.0
-*/
-const consistentNullishComparison = defineRule(() => ({
-	name: "local/consistentNullishComparison",
-	visitor: { BinaryExpression(context, node) {
-		const newOperatorText = match(node.operatorToken.kind).with(SyntaxKind.EqualsEqualsEqualsToken, () => "==").with(SyntaxKind.ExclamationEqualsEqualsToken, () => "!=").otherwise(() => null);
-		if (newOperatorText == null) return;
-		const offendingChild = [node.left, node.right].find((n) => {
-			switch (n.kind) {
-				case SyntaxKind.NullKeyword: return true;
-				case SyntaxKind.Identifier: return n.escapedText === "undefined";
-				default: return false;
-			}
-		});
-		if (offendingChild == null) return;
-		context.report({
-			message: `Use '${newOperatorText}' for nullish comparison.`,
-			node,
-			suggestions: [{
-				message: offendingChild === node.left ? `Replace with 'null ${newOperatorText} ${node.right.getText()}'.` : `Replace with '${node.left.getText()} ${newOperatorText} null'.`,
-				changes: [{
-					node: node.operatorToken,
-					newText: newOperatorText
-				}, {
-					node: offendingChild,
-					newText: "null"
-				}]
-			}]
-		});
-	} }
-}));
-
-//#endregion
 //#region src/rules/no-duplicate-exports.ts
-const messages$1 = { noDuplicateExports: (p) => `Duplicate export from module ${p.source}.` };
+const messages$3 = { default: (p) => `Duplicate export from module ${p.source}.` };
 function isReExportDeclaration(node) {
 	return node.exportClause != null && node.moduleSpecifier != null;
 }
@@ -62,7 +25,7 @@ function isReExportDeclaration(node) {
 */
 const noDuplicateExports = defineRule(() => {
 	return {
-		name: "module/no-duplicate-exports",
+		name: "dx/no-duplicate-exports",
 		createData() {
 			return { exports: [] };
 		},
@@ -73,7 +36,7 @@ const noDuplicateExports = defineRule(() => {
 			if (duplicateExport != null) {
 				ctx.report({
 					node,
-					message: messages$1.noDuplicateExports({ source }),
+					message: messages$3.default({ source }),
 					suggestions: buildSuggestions(duplicateExport, node)
 				});
 				return;
@@ -224,7 +187,7 @@ const compose = dual(2, (ab, bc) => (a) => bc(ab(a)));
 
 //#endregion
 //#region src/rules/no-duplicate-imports.ts
-const messages = { noDuplicateImports: (p) => `Duplicate import from module ${p.source}.` };
+const messages$2 = { default: (p) => `Duplicate import from module ${p.source}.` };
 /**
 * Rule to detect and merge duplicate `import from` statements from the same module.
 *
@@ -243,7 +206,7 @@ const messages = { noDuplicateImports: (p) => `Duplicate import from module ${p.
 */
 const noDuplicateImports = defineRule(() => {
 	return {
-		name: "module/no-duplicate-imports",
+		name: "dx/no-duplicate-imports",
 		createData() {
 			return { imports: [
 				[],
@@ -265,7 +228,7 @@ const noDuplicateImports = defineRule(() => {
 			if (duplicateImport != null) {
 				ctx.report({
 					node,
-					message: messages.noDuplicateImports({ source: importInfo.source }),
+					message: messages$2.default({ source: importInfo.source }),
 					suggestions: importKind > 1 ? [] : [{
 						message: "Merge duplicate imports",
 						changes: [{
@@ -306,4 +269,83 @@ function buildMergedImport(a, b) {
 }
 
 //#endregion
-export { consistentNullishComparison, noDuplicateExports, noDuplicateImports };
+//#region src/utils/source-file.ts
+function getLine(node) {
+	const sourceFile = node.getSourceFile();
+	return [sourceFile.getLineAndCharacterOfPosition(node.getStart()).line, sourceFile.getLineAndCharacterOfPosition(node.getEnd()).line];
+}
+
+//#endregion
+//#region src/rules/no-multiline-template-expressions-without-auto-dedent.ts
+const messages$1 = { default: () => `Avoid using multiline template expressions without auto-dedent` };
+const noMultilineTemplateExpressionsWithoutAutoDedent = defineRule(() => ({
+	name: "dx/no-multiline-template-expressions-without-auto-dedent",
+	visitor: { NoSubstitutionTemplateLiteral(ctx, node) {
+		if (node.parent.kind === SyntaxKind.TaggedTemplateExpression) return;
+		const [startLine, endLine] = getLine(node);
+		if (startLine !== endLine) ctx.report({
+			node,
+			message: messages$1.default()
+		});
+	} }
+}));
+
+//#endregion
+//#region src/rules/nullish.ts
+const messages = {
+	useUnitForUndefined: "Use 'unit' instead of 'undefined'.",
+	useLooseNullishComparison: (p) => `Use '${p.op}' for nullish comparison.`
+};
+const suggestions = { replaceWithExpression: (p) => `Replace with '${p.expr}'.` };
+const nullish = defineRule(() => ({
+	name: "dx/nullish",
+	visitor: {
+		UndefinedKeyword(ctx, node) {
+			if (node.getSourceFile().isDeclarationFile) return;
+			ctx.report({
+				node,
+				message: messages.useUnitForUndefined,
+				suggestions: [{
+					message: suggestions.replaceWithExpression({ expr: "unit" }),
+					changes: [{
+						node,
+						newText: "unit"
+					}, {
+						start: 0,
+						end: 0,
+						newText: "import { unit } from '@local/eff';\n"
+					}]
+				}]
+			});
+		},
+		BinaryExpression(ctx, node) {
+			const newOperatorText = match(node.operatorToken.kind).with(SyntaxKind.EqualsEqualsEqualsToken, () => "==").with(SyntaxKind.ExclamationEqualsEqualsToken, () => "!=").otherwise(() => null);
+			if (newOperatorText == null) return;
+			const offendingChild = [node.left, node.right].find((n) => {
+				switch (n.kind) {
+					case SyntaxKind.NullKeyword: return true;
+					case SyntaxKind.Identifier: return n.escapedText === "undefined";
+					default: return false;
+				}
+			});
+			if (offendingChild == null) return;
+			ctx.report({
+				message: messages.useLooseNullishComparison({ op: newOperatorText }),
+				node,
+				suggestions: [{
+					message: suggestions.replaceWithExpression({ expr: offendingChild === node.left ? `null ${newOperatorText} ${node.right.getText()}` : `${node.left.getText()} ${newOperatorText} null` }),
+					changes: [{
+						node: node.operatorToken,
+						newText: newOperatorText
+					}, {
+						node: offendingChild,
+						newText: "null"
+					}]
+				}]
+			});
+		}
+	}
+}));
+
+//#endregion
+export { noDuplicateExports, noDuplicateImports, noMultilineTemplateExpressionsWithoutAutoDedent, nullish };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsl-dx",
-  "version": "0.4.0",
+  "version": "0.5.1",
   "description": "A tsl plugin for better JavaScript/TypeScript DX.",
   "keywords": [
     "tsl",
@@ -25,7 +25,7 @@
   },
   "devDependencies": {
     "dedent": "^1.7.1",
-    "tsdown": "^0.20.1",
+    "tsdown": "^0.20.3",
     "tsl": "^1.0.28",
     "vitest": "^4.0.18",
     "@local/configs": "0.0.0",