🚨 update linter with some more rules.

Author: loucyx

@coven/iterables/async/take.ts

@@ -28,7 +28,7 @@ export const take = (
 						yield item;
 						count += 1n;
 					} else {
-						break;
+						return;
 					}
 				}
 			}

@coven/iterables/async/zip.ts

@@ -32,7 +32,7 @@ export const zip = <ItemFirst>(
 					.next();
 
 				if (done) {
-					break;
+					return;
 				}
 
 				yield [itemFirst as ItemFirst, value] as const;

@coven/iterables/zip.ts

@@ -28,7 +28,7 @@ export const zip = <ItemFirst>(
 			const { done = false, value } = iteratorSecond.next();
 
 			if (done) {
-				break;
+				return;
 			}
 
 			yield [itemFirst, value as ItemSecond];

lint/mod.ts

@@ -1,16 +1,37 @@
+import { maxLines } from "./rules/max-lines.ts";
+import { noBreak } from "./rules/no-break.ts";
 import { noClass } from "./rules/no-class.ts";
+import { noContinue } from "./rules/no-continue.ts";
+import { noDefaultExport } from "./rules/no-default-export.ts";
+import { noDoWhile } from "./rules/no-do-while.ts";
+import { noEnum } from "./rules/no-enum.ts";
+import { noForIn } from "./rules/no-for-in.ts";
+import { noFunction } from "./rules/no-function.ts";
 import { noNull } from "./rules/no-null.ts";
+import { noSwitch } from "./rules/no-switch.ts";
 import { noThis } from "./rules/no-this.ts";
 import { noThrow } from "./rules/no-throw.ts";
 import { noTry } from "./rules/no-try.ts";
+import { noWhile } from "./rules/no-while.ts";
 
+// deno-lint-ignore coven/no-default-export
 export default {
 	name: "coven",
 	rules: {
+		"max-lines": maxLines,
+		"no-break": noBreak,
 		"no-class": noClass,
+		"no-continue": noContinue,
+		"no-default-export": noDefaultExport,
+		"no-do-while": noDoWhile,
+		"no-enum": noEnum,
+		"no-for-in": noForIn,
+		"no-function": noFunction,
 		"no-null": noNull,
+		"no-switch": noSwitch,
 		"no-this": noThis,
 		"no-throw": noThrow,
 		"no-try": noTry,
+		"no-while": noWhile,
 	},
 } satisfies Deno.lint.Plugin;

lint/no.ts

@@ -0,0 +1,26 @@
+/**
+ * Utility type to figure out what the Node type is based on the Visitor.
+ */
+type FindContext<
+	Visitor extends keyof Deno.lint.LintVisitor,
+	Node extends Deno.lint.Node = Deno.lint.Node,
+> = Node extends { type: Visitor } ? Node : never;
+
+/**
+ * Shorthand for common "Avoid this, do that." lintinrg rules.
+ */
+export const no = <Visitor extends keyof Deno.lint.LintVisitor>(
+	visitor: Visitor,
+	message: string,
+	condition?: (
+		data: { context: Deno.lint.RuleContext; node: FindContext<Visitor> },
+	) => boolean,
+): Deno.lint.Rule => ({
+	create: (context): Deno.lint.LintVisitor => ({
+		[visitor]: (node: FindContext<Visitor>): void => {
+			(condition?.({ context, node }) ?? true)
+				? context.report({ message, node })
+				: undefined;
+		},
+	}),
+});

lint/rules/max-lines.ts

@@ -0,0 +1,9 @@
+import { no } from "../no.ts";
+
+const LINE_LIMIT = 300;
+
+export const maxLines: Deno.lint.Rule = no(
+	"Program",
+	`Avoid large files. Split this into smaller files (${LINE_LIMIT} lines or less).`,
+	({ context }) => context.sourceCode.text.split("\n").length > LINE_LIMIT,
+);

lint/rules/no-break.ts

@@ -0,0 +1,6 @@
+import { no } from "../no.ts";
+
+export const noBreak: Deno.lint.Rule = no(
+	"BreakStatement",
+	"Avoid using `break`. Use explicit control flow.",
+);

lint/rules/no-class.ts

@@ -1,10 +1,6 @@
-export const noClass: Deno.lint.Rule = {
-	create: (context): Deno.lint.LintVisitor => ({
-		ClassExpression: (node) => {
-			context.report({
-				node,
-				message: "Avoid using `class`. Use a function instead.",
-			});
-		},
-	}),
-};
+import { no } from "../no.ts";
+
+export const noClass: Deno.lint.Rule = no(
+	"ClassExpression",
+	"Avoid using `class`. Use a function instead.",
+);

lint/rules/no-continue.ts

@@ -0,0 +1,6 @@
+import { no } from "../no.ts";
+
+export const noContinue: Deno.lint.Rule = no(
+	"ContinueStatement",
+	"Avoid using `continue`. Use explicit control flow.",
+);

lint/rules/no-default-export.ts

@@ -0,0 +1,6 @@
+import { no } from "../no.ts";
+
+export const noDefaultExport: Deno.lint.Rule = no(
+	"ExportDefaultDeclaration",
+	"Avoid using default exports. Use named exports instead.",
+);