@flint.fyi/ts 0.20.0 → 0.22.0

package/lib/rules/variableDeletions.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Reports attempting to delete variables with the delete operator.";
-    readonly id: "variableDeletions";
-    readonly presets: readonly ["javascript"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "noDeleteVar", undefined>;
-export default _default;
-//# sourceMappingURL=variableDeletions.d.ts.map

package/lib/rules/variableDeletions.js

@@ -1,35 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { typescriptLanguage } from "@flint.fyi/typescript-language";
-import { SyntaxKind } from "typescript";
-//#region src/rules/variableDeletions.ts
-var variableDeletions_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Reports attempting to delete variables with the delete operator.",
-		id: "variableDeletions",
-		presets: ["javascript"]
-	},
-	messages: { noDeleteVar: {
-		primary: "Deleting a variable with the delete operator outside of strict mode will silently fail and return false.",
-		secondary: [
-			"The delete operator is only meant to remove properties from objects, not variables.",
-			"Attempting to delete a variable in strict mode will cause a syntax error.",
-			"In non-strict mode, it will silently fail and return false without actually deleting the variable."
-		],
-		suggestions: ["Remove the delete statement, or if you need to unset the value, assign undefined instead."]
-	} },
-	setup(context) {
-		return { visitors: { DeleteExpression: (node, { sourceFile }) => {
-			if (node.expression.kind === SyntaxKind.Identifier) context.report({
-				message: "noDeleteVar",
-				range: {
-					begin: node.getStart(sourceFile),
-					end: node.getEnd()
-				}
-			});
-		} } };
-	}
-});
-//#endregion
-export { variableDeletions_default as default };
-
-//# sourceMappingURL=variableDeletions.js.map

package/lib/rules/voidOperator.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Reports using the void operator.";
-    readonly id: "voidOperator";
-    readonly presets: readonly ["stylistic", "stylisticStrict"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "noVoid", undefined>;
-export default _default;
-//# sourceMappingURL=voidOperator.d.ts.map

package/lib/rules/voidOperator.js

@@ -1,34 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/voidOperator.ts
-var voidOperator_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Reports using the void operator.",
-		id: "voidOperator",
-		presets: ["stylistic", "stylisticStrict"]
-	},
-	messages: { noVoid: {
-		primary: "Prefer an explicit value over using the void operator to produce undefined.",
-		secondary: [
-			"The void operator evaluates an expression and returns undefined, regardless of the expression's value.",
-			"This is often confusing and can be replaced with more explicit code.",
-			"Instead of using void, return or assign undefined directly."
-		],
-		suggestions: ["Replace void expressions with undefined or restructure the code to be more explicit."]
-	} },
-	setup(context) {
-		return { visitors: { VoidExpression: (node, { sourceFile }) => {
-			context.report({
-				message: "noVoid",
-				range: {
-					begin: node.getStart(sourceFile),
-					end: node.getEnd()
-				}
-			});
-		} } };
-	}
-});
-//#endregion
-export { voidOperator_default as default };
-
-//# sourceMappingURL=voidOperator.js.map

package/lib/rules/withStatements.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Reports using with statements";
-    readonly id: "withStatements";
-    readonly presets: readonly ["javascript"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "withStatement", undefined>;
-export default _default;
-//# sourceMappingURL=withStatements.d.ts.map

package/lib/rules/withStatements.js

@@ -1,30 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { typescriptLanguage } from "@flint.fyi/typescript-language";
-//#region src/rules/withStatements.ts
-var withStatements_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Reports using with statements",
-		id: "withStatements",
-		presets: ["javascript"]
-	},
-	messages: { withStatement: {
-		primary: "`with` statements are deprecated, unreliable, and difficult to reason about.",
-		secondary: ["The with statement is problematic because it adds members of an object to the current scope, making it impossible to tell what a variable inside the block actually refers to.", "If an object doesn't have an expected variable, you'll end up modifying a global instead of the expected property."],
-		suggestions: ["Create a variable with the content of the with expression and access its properties instead."]
-	} },
-	setup(context) {
-		return { visitors: { WithStatement: (node, { sourceFile }) => {
-			context.report({
-				message: "withStatement",
-				range: {
-					begin: node.getStart(sourceFile),
-					end: node.getStart(sourceFile) + 4
-				}
-			});
-		} } };
-	}
-});
-//#endregion
-export { withStatements_default as default };
-
-//# sourceMappingURL=withStatements.js.map

package/lib/rules/wrapperObjects.d.ts

@@ -1,10 +0,0 @@
-declare const _default: import("@flint.fyi/core").Rule<{
-    readonly description: "Flags using `new` with `Boolean`, `Number`, or `String`, which creates wrapper objects instead of primitives.";
-    readonly id: "wrapperObjects";
-    readonly presets: readonly ["logical", "logicalStrict"];
-} & {
-    readonly pluginId: string;
-    readonly url: string;
-}, "wrapperObject", undefined>;
-export default _default;
-//# sourceMappingURL=wrapperObjects.d.ts.map

package/lib/rules/wrapperObjects.js

@@ -1,36 +0,0 @@
-import { ruleCreator } from "./ruleCreator.js";
-import { getTSNodeRange, isGlobalDeclarationOfName, typescriptLanguage } from "@flint.fyi/typescript-language";
-import * as ts$1 from "typescript";
-//#region src/rules/wrapperObjects.ts
-const wrapperConstructors = new Set([
-	"Boolean",
-	"Number",
-	"String"
-]);
-var wrapperObjects_default = ruleCreator.createRule(typescriptLanguage, {
-	about: {
-		description: "Flags using `new` with `Boolean`, `Number`, or `String`, which creates wrapper objects instead of primitives.",
-		id: "wrapperObjects",
-		presets: ["logical", "logicalStrict"]
-	},
-	messages: { wrapperObject: {
-		primary: "{{ name }} wrapper objects are rarely intended and can cause unexpected behavior.",
-		secondary: ["Using `new {{ name }}()` creates an object wrapper around a primitive value, not the primitive itself.", "Object wrappers behave differently from primitives in comparisons: `new String('a') !== 'a'`."],
-		suggestions: ["Use `{{ name }}()` without `new` to coerce values to primitives."]
-	} },
-	setup(context) {
-		return { visitors: { NewExpression: (node, { sourceFile, typeChecker }) => {
-			if (node.expression.kind !== ts$1.SyntaxKind.Identifier || !wrapperConstructors.has(node.expression.text) || !isGlobalDeclarationOfName(node.expression, node.expression.text, typeChecker)) return;
-			const range = getTSNodeRange(node, sourceFile);
-			context.report({
-				data: { name: node.expression.text },
-				message: "wrapperObject",
-				range
-			});
-		} } };
-	}
-});
-//#endregion
-export { wrapperObjects_default as default };
-
-//# sourceMappingURL=wrapperObjects.js.map

package/lib/type-utils/getSpecifierNames.d.ts

@@ -1,3 +0,0 @@
-import type { TypeOrValueSpecifier } from "./schemas.ts";
-export declare function getSpecifierNames(specifier: TypeOrValueSpecifier): string[] | undefined;
-//# sourceMappingURL=getSpecifierNames.d.ts.map

package/lib/type-utils/getSpecifierNames.js

@@ -1,9 +0,0 @@
-//#region src/type-utils/getSpecifierNames.ts
-function getSpecifierNames(specifier) {
-	if (specifier.name === void 0) return;
-	return Array.isArray(specifier.name) ? specifier.name : [specifier.name];
-}
-//#endregion
-export { getSpecifierNames };
-
-//# sourceMappingURL=getSpecifierNames.js.map

package/lib/type-utils/isDeclaredInModuleBlock.d.ts

@@ -1,3 +0,0 @@
-import ts from "typescript";
-export declare function isDeclaredInModuleBlock(declaration: ts.Declaration, packageName: string): boolean;
-//# sourceMappingURL=isDeclaredInModuleBlock.d.ts.map

package/lib/type-utils/isDeclaredInModuleBlock.js

@@ -1,14 +0,0 @@
-import ts from "typescript";
-//#region src/type-utils/isDeclaredInModuleBlock.ts
-function isDeclaredInModuleBlock(declaration, packageName) {
-	let current = declaration;
-	while (!ts.isSourceFile(current)) {
-		if (ts.isModuleDeclaration(current) && !(current.flags & ts.NodeFlags.Namespace) && ts.isStringLiteral(current.name) && current.name.text === packageName) return true;
-		current = current.parent;
-	}
-	return false;
-}
-//#endregion
-export { isDeclaredInModuleBlock };
-
-//# sourceMappingURL=isDeclaredInModuleBlock.js.map

package/lib/type-utils/isFromFile.d.ts

@@ -1,3 +0,0 @@
-import ts from "typescript";
-export declare function isFromFile(sourceFile: ts.SourceFile, specifiedPath: string | undefined, program: ts.Program): boolean;
-//# sourceMappingURL=isFromFile.d.ts.map

package/lib/type-utils/isFromFile.js

@@ -1,13 +0,0 @@
-import ts from "typescript";
-import { pathKey } from "@flint.fyi/utils";
-import path from "node:path";
-//#region src/type-utils/isFromFile.ts
-function isFromFile(sourceFile, specifiedPath, program) {
-	if (specifiedPath === void 0) return !sourceFile.fileName.includes("/node_modules/") && !program.isSourceFileDefaultLibrary(sourceFile);
-	const caseSensitive = ts.sys.useCaseSensitiveFileNames;
-	return pathKey(sourceFile.fileName, caseSensitive) === pathKey(path.resolve(program.getCurrentDirectory(), specifiedPath), caseSensitive);
-}
-//#endregion
-export { isFromFile };
-
-//# sourceMappingURL=isFromFile.js.map

package/lib/type-utils/isFromPackage.d.ts

@@ -1,3 +0,0 @@
-import ts from "typescript";
-export declare function isFromPackage(declaration: ts.Declaration, packageName: string, program: ts.Program): boolean;
-//# sourceMappingURL=isFromPackage.d.ts.map

package/lib/type-utils/isFromPackage.js

@@ -1,16 +0,0 @@
-import { isDeclaredInModuleBlock } from "./isDeclaredInModuleBlock.js";
-import "typescript";
-//#region src/type-utils/isFromPackage.ts
-function isFromPackage(declaration, packageName, program) {
-	if (isDeclaredInModuleBlock(declaration, packageName)) return true;
-	const sourceFile = declaration.getSourceFile();
-	if (!program.isSourceFileFromExternalLibrary(sourceFile)) return false;
-	const resolvedName = program.sourceFileToPackageName.get(sourceFile.path);
-	if (resolvedName === packageName || sourceFile.fileName.includes(`/node_modules/${packageName}/`)) return true;
-	const typesPackageName = packageName.replace(/^@([^/]+)\//, "$1__");
-	return resolvedName === typesPackageName || sourceFile.fileName.includes(`/node_modules/@types/${typesPackageName}/`);
-}
-//#endregion
-export { isFromPackage };
-
-//# sourceMappingURL=isFromPackage.js.map

package/lib/type-utils/matchesSpecifier.d.ts

@@ -1,4 +0,0 @@
-import type ts from "typescript";
-import type { TypeOrValueSpecifier } from "./schemas.ts";
-export declare function matchesSpecifier(importedName: string | undefined, declarations: ts.Declaration[], specifier: TypeOrValueSpecifier, program: ts.Program): boolean;
-//# sourceMappingURL=matchesSpecifier.d.ts.map

package/lib/type-utils/matchesSpecifier.js

@@ -1,20 +0,0 @@
-import { getSpecifierNames } from "./getSpecifierNames.js";
-import { isFromFile } from "./isFromFile.js";
-import { isFromPackage } from "./isFromPackage.js";
-import { declarationIncludesGlobal } from "@flint.fyi/typescript-language";
-//#region src/type-utils/matchesSpecifier.ts
-function matchesSpecifier(importedName, declarations, specifier, program) {
-	const names = getSpecifierNames(specifier);
-	if (names !== void 0 && (importedName === void 0 || !names.includes(importedName))) return false;
-	return declarations.some((declaration) => {
-		switch (specifier.from) {
-			case "file": return isFromFile(declaration.getSourceFile(), specifier.path, program);
-			case "lib": return declarationIncludesGlobal(declaration);
-			case "package": return isFromPackage(declaration, specifier.package, program);
-		}
-	});
-}
-//#endregion
-export { matchesSpecifier };
-
-//# sourceMappingURL=matchesSpecifier.js.map

package/lib/type-utils/schemas.d.ts

@@ -1,15 +0,0 @@
-import { z } from "zod/v4";
-export declare const typeOrValueSpecifierSchema: z.ZodUnion<readonly [z.ZodObject<{
-    from: z.ZodLiteral<"file">;
-    name: z.ZodOptional<z.ZodUnion<readonly [z.ZodString, z.ZodArray<z.ZodString>]>>;
-    path: z.ZodOptional<z.ZodString>;
-}, z.core.$strip>, z.ZodObject<{
-    from: z.ZodLiteral<"lib">;
-    name: z.ZodOptional<z.ZodUnion<readonly [z.ZodString, z.ZodArray<z.ZodString>]>>;
-}, z.core.$strip>, z.ZodObject<{
-    from: z.ZodLiteral<"package">;
-    name: z.ZodOptional<z.ZodUnion<readonly [z.ZodString, z.ZodArray<z.ZodString>]>>;
-    package: z.ZodString;
-}, z.core.$strip>]>;
-export type TypeOrValueSpecifier = z.infer<typeof typeOrValueSpecifierSchema>;
-//# sourceMappingURL=schemas.d.ts.map

package/lib/type-utils/schemas.js

@@ -1,25 +0,0 @@
-import { z } from "zod/v4";
-//#region src/type-utils/schemas.ts
-const fileSpecifierSchema = z.object({
-	from: z.literal("file"),
-	name: z.union([z.string(), z.array(z.string())]).optional(),
-	path: z.string().optional()
-});
-const libSpecifierSchema = z.object({
-	from: z.literal("lib"),
-	name: z.union([z.string(), z.array(z.string())]).optional()
-});
-const packageSpecifierSchema = z.object({
-	from: z.literal("package"),
-	name: z.union([z.string(), z.array(z.string())]).optional(),
-	package: z.string()
-});
-const typeOrValueSpecifierSchema = z.union([
-	fileSpecifierSchema,
-	libSpecifierSchema,
-	packageSpecifierSchema
-]);
-//#endregion
-export { typeOrValueSpecifierSchema };
-
-//# sourceMappingURL=schemas.js.map