@staff0rd/assist 0.108.2 → 0.110.0

package/README.md

@@ -86,10 +86,12 @@ After installation, the `assist` command will be available globally. You can als
 - `assist verify all` - Run all checks, ignoring diff-based filters
 - `assist verify init` - Add verify scripts to a project
 - `assist verify hardcoded-colors` - Check for hardcoded hex colors in src/ (supports `hardcodedColors.ignore` globs in config)
-- `assist lint` - Run lint checks for conventions not enforced by biomejs
+- `assist lint [-f, --fix]` - Run lint checks for conventions not enforced by biomejs (use `-f` to auto-fix)
 - `assist lint init` - Initialize Biome with standard linter config
 - `assist refactor check [pattern]` - Check for files that exceed the maximum line count
 - `assist refactor ignore <file>` - Add a file to the refactor ignore list
+- `assist refactor rename file <source> <destination>` - Rename/move a TypeScript file and update all imports (dry-run by default, use `--apply` to execute)
+- `assist refactor rename symbol <file> <oldName> <newName>` - Rename a variable, function, class, or type across the project (dry-run by default, use `--apply` to execute)
 - `assist refactor restructure [pattern]` - Analyze import graph and restructure tightly-coupled files into nested directories
 - `assist devlog list` - Group git commits by date
 - `assist devlog next` - Show commits for the day after the last versioned entry

package/claude/CLAUDE.md

@@ -1,3 +1,8 @@
 After any code change, run `/verify` to ensure all checks pass.
 
 The tool is invoked using the `assist` command and is installed globally. Use it directly (e.g., `assist commit "message"`). Do NOT try to invoke it via `npx tsx src/index.ts` or guess at the entry point.
+
+When renaming TypeScript files or symbols, use the refactor commands instead of doing it manually:
+- `assist refactor rename file <source> <destination>` — rename/move a file and update all imports
+- `assist refactor rename symbol <file> <oldName> <newName>` — rename a variable, function, class, or type across the project
+Both default to dry-run; add `--apply` to execute.

package/dist/commands/lint/lint/applyMoves.ts

@@ -0,0 +1,50 @@
+import fs from "node:fs";
+import path from "node:path";
+import type { Project } from "ts-morph";
+import { renameExports } from "./renameExports";
+
+type Move = { sourcePath: string; destPath: string };
+
+function isCaseOnly(a: string, b: string): boolean {
+	return a.toLowerCase() === b.toLowerCase();
+}
+
+function moveCaseInsensitive(absSource: string, absDest: string): void {
+	const tmp = `${absSource}.tmp`;
+	fs.renameSync(absSource, tmp);
+	fs.renameSync(tmp, absDest);
+}
+
+export function applyMoves(
+	project: Project,
+	moves: Move[],
+	cwd: string,
+	emit: (line: string) => void,
+): void {
+	for (const { sourcePath, destPath } of moves) {
+		const start = performance.now();
+		const absSource = path.resolve(sourcePath);
+		const absDest = path.resolve(destPath);
+
+		for (const r of renameExports(project, absSource, absDest)) {
+			emit(`  Renamed export ${r} in ${sourcePath}`);
+		}
+
+		const sourceFile = project.getSourceFile(absSource);
+		if (sourceFile) sourceFile.move(absDest);
+
+		const ms = (performance.now() - start).toFixed(0);
+		const rel = `${path.relative(cwd, absSource)} → ${path.relative(cwd, absDest)}`;
+		emit(`  Renamed ${rel} (${ms}ms)`);
+	}
+
+	project.saveSync();
+
+	for (const { sourcePath, destPath } of moves) {
+		const absSource = path.resolve(sourcePath);
+		const absDest = path.resolve(destPath);
+		if (isCaseOnly(absSource, absDest) && fs.existsSync(absSource)) {
+			moveCaseInsensitive(absSource, absDest);
+		}
+	}
+}

package/dist/commands/lint/lint/checkFileNames.ts

@@ -0,0 +1,81 @@
+import fs from "node:fs";
+import path from "node:path";
+import { findSourceFiles } from "../../../shared/findSourceFiles";
+
+export type FileNameViolation = {
+	filePath: string;
+	fileName: string;
+	suggestedName: string;
+};
+
+function hasClassOrComponent(content: string): boolean {
+	const classPattern = /^(export\s+)?(abstract\s+)?class\s+\w+/m;
+	const functionComponentPattern =
+		/^(export\s+)?(default\s+)?function\s+[A-Z]\w*\s*\(/m;
+	const arrowComponentPattern = /^(export\s+)?(const|let)\s+[A-Z]\w*\s*=.*=>/m;
+
+	return (
+		classPattern.test(content) ||
+		functionComponentPattern.test(content) ||
+		arrowComponentPattern.test(content)
+	);
+}
+
+function hasMatchingTypeExport(
+	content: string,
+	nameWithoutExt: string,
+): boolean {
+	const typePattern = new RegExp(
+		`^export\\s+type\\s+${nameWithoutExt}\\b`,
+		"m",
+	);
+	const interfacePattern = new RegExp(
+		`^export\\s+interface\\s+${nameWithoutExt}\\b`,
+		"m",
+	);
+	return typePattern.test(content) || interfacePattern.test(content);
+}
+
+function suggestName(fileName: string): string {
+	const nameWithoutExt = fileName.replace(/\.(ts|tsx)$/, "");
+	const ext = fileName.slice(nameWithoutExt.length);
+
+	// SCREAMING_SNAKE_CASE → camelCase (e.g. OC_COLORS → ocColors)
+	if (/^[A-Z][A-Z0-9]*(_[A-Z0-9]+)+$/.test(nameWithoutExt)) {
+		const camel = nameWithoutExt
+			.toLowerCase()
+			.replace(/_([a-z0-9])/g, (_, c: string) => c.toUpperCase());
+		return `${camel}${ext}`;
+	}
+
+	return `${nameWithoutExt.charAt(0).toLowerCase()}${nameWithoutExt.slice(1)}${ext}`;
+}
+
+export function checkFileNames(): FileNameViolation[] {
+	const sourceFiles = findSourceFiles("src");
+	const violations: FileNameViolation[] = [];
+
+	for (const filePath of sourceFiles) {
+		const fileName = path.basename(filePath);
+		const nameWithoutExt = fileName.replace(/\.(ts|tsx)$/, "");
+
+		// Skip .stories and .test files — they mirror the component/module name
+		if (/\.(stories|test)\.(ts|tsx)$/.test(fileName)) continue;
+
+		if (/^[A-Z]/.test(nameWithoutExt)) {
+			const content = fs.readFileSync(filePath, "utf-8");
+			if (
+				!hasClassOrComponent(content) &&
+				!hasMatchingTypeExport(content, nameWithoutExt)
+			) {
+				violations.push({
+					filePath,
+					fileName,
+					suggestedName: suggestName(fileName),
+				});
+			}
+		}
+	}
+
+	return violations;
+}

package/dist/commands/lint/lint/createLintProject.ts

@@ -0,0 +1,16 @@
+import fs from "node:fs";
+import path from "node:path";
+import { Project } from "ts-morph";
+
+export function createLintProject(): Project {
+	const tsConfigPath = path.resolve("tsconfig.json");
+	const project = fs.existsSync(tsConfigPath)
+		? new Project({
+				tsConfigFilePath: tsConfigPath,
+				skipAddingFilesFromTsConfig: true,
+			})
+		: new Project();
+
+	project.addSourceFilesAtPaths("src/**/*.{ts,tsx}");
+	return project;
+}

package/dist/commands/lint/lint/fixFileNameViolations.ts

@@ -0,0 +1,16 @@
+import chalk from "chalk";
+import { applyMoves } from "./applyMoves";
+import { createLintProject } from "./createLintProject";
+
+type Move = { sourcePath: string; destPath: string };
+
+export function fixFileNameViolations(moves: Move[]): void {
+	const start = performance.now();
+	const project = createLintProject();
+	const cwd = process.cwd();
+
+	applyMoves(project, moves, cwd, (line) => console.log(chalk.green(line)));
+
+	const ms = (performance.now() - start).toFixed(0);
+	console.log(chalk.dim(`  Done in ${ms}ms`));
+}

package/dist/commands/lint/lint/index.ts

@@ -3,8 +3,12 @@ import { runFileNameCheck } from "./runFileNameCheck";
 import { runImportExtensionCheck } from "./runImportExtensionCheck";
 import { runStaticImportCheck } from "./runStaticImportCheck";
 
-export function lint(): void {
-	const fileNamePassed = runFileNameCheck();
+type LintOptions = {
+	fix?: boolean;
+};
+
+export function lint(options: LintOptions = {}): void {
+	const fileNamePassed = runFileNameCheck(options.fix);
 	const staticImportPassed = runStaticImportCheck();
 	const importExtensionPassed = runImportExtensionCheck();
 

package/dist/commands/lint/lint/renameExports.ts

@@ -0,0 +1,37 @@
+import path from "node:path";
+import type { Project } from "ts-morph";
+import { SyntaxKind } from "ts-morph";
+
+function nameWithoutExtension(filePath: string): string {
+	return path.basename(filePath).replace(/\.(ts|tsx)$/, "");
+}
+
+export function renameExports(
+	project: Project,
+	absSource: string,
+	absDest: string,
+): string[] {
+	const oldName = nameWithoutExtension(absSource);
+	const newName = nameWithoutExtension(absDest);
+	const sourceFile = project.getSourceFile(absSource);
+	if (!sourceFile) return [];
+
+	const renamed: string[] = [];
+	for (const [, declarations] of sourceFile.getExportedDeclarations()) {
+		for (const decl of declarations) {
+			// Skip type aliases and interfaces — they should stay PascalCase
+			const kind = decl.getKind();
+			if (
+				kind === SyntaxKind.TypeAliasDeclaration ||
+				kind === SyntaxKind.InterfaceDeclaration
+			) {
+				continue;
+			}
+			const nameNode = decl.getFirstChildByKind(SyntaxKind.Identifier);
+			if (!nameNode || nameNode.getText() !== oldName) continue;
+			nameNode.rename(newName);
+			renamed.push(`${oldName} → ${newName}`);
+		}
+	}
+	return renamed;
+}

package/dist/commands/lint/lint/runFileNameCheck.ts

@@ -1,69 +1,45 @@
-import fs from "node:fs";
 import path from "node:path";
 import chalk from "chalk";
-import { findSourceFiles } from "../../../shared/findSourceFiles";
-
-type FileNameViolation = {
-	filePath: string;
-	fileName: string;
-};
-
-function hasClassOrComponent(content: string): boolean {
-	const classPattern = /^(export\s+)?(abstract\s+)?class\s+\w+/m;
-	const functionComponentPattern =
-		/^(export\s+)?(default\s+)?function\s+[A-Z]\w*\s*\(/m;
-	const arrowComponentPattern = /^(export\s+)?(const|let)\s+[A-Z]\w*\s*=.*=>/m;
-
-	return (
-		classPattern.test(content) ||
-		functionComponentPattern.test(content) ||
-		arrowComponentPattern.test(content)
+import { checkFileNames, type FileNameViolation } from "./checkFileNames";
+import { fixFileNameViolations } from "./fixFileNameViolations";
+
+function reportViolations(violations: FileNameViolation[]): void {
+	console.error(chalk.red("\nFile name check failed:\n"));
+	console.error(
+		chalk.red(
+			"  Files without classes or React components should not start with a capital letter.\n",
+		),
 	);
-}
-
-function checkFileNames(): FileNameViolation[] {
-	const sourceFiles = findSourceFiles("src");
-	const violations: FileNameViolation[] = [];
-
-	for (const filePath of sourceFiles) {
-		const fileName = path.basename(filePath);
-		const nameWithoutExt = fileName.replace(/\.(ts|tsx)$/, "");
-
-		if (/^[A-Z]/.test(nameWithoutExt)) {
-			const content = fs.readFileSync(filePath, "utf-8");
-			if (!hasClassOrComponent(content)) {
-				violations.push({ filePath, fileName });
-			}
-		}
+	for (const violation of violations) {
+		console.error(chalk.red(`  ${violation.filePath}`));
+		console.error(chalk.gray(`    Rename to: ${violation.suggestedName}\n`));
 	}
-
-	return violations;
+	console.error(chalk.dim("  Run with -f to auto-fix.\n"));
 }
 
-export function runFileNameCheck(): boolean {
+export function runFileNameCheck(fix = false): boolean {
 	const violations = checkFileNames();
-	if (violations.length > 0) {
-		console.error(chalk.red("\nFile name check failed:\n"));
-		console.error(
-			chalk.red(
-				"  Files without classes or React components should not start with a capital letter.\n",
-			),
-		);
-		for (const violation of violations) {
-			console.error(chalk.red(`  ${violation.filePath}`));
-			console.error(
-				chalk.gray(
-					`    Rename to: ${violation.fileName.charAt(0).toLowerCase()}${violation.fileName.slice(1)}\n`,
-				),
+
+	if (violations.length === 0) {
+		if (!process.env.CLAUDECODE) {
+			console.log(
+				"File name check passed. All PascalCase files contain classes or components.",
 			);
 		}
-		return false;
+		return true;
 	}
 
-	if (!process.env.CLAUDECODE) {
-		console.log(
-			"File name check passed. All PascalCase files contain classes or components.",
-		);
+	if (!fix) {
+		reportViolations(violations);
+		return false;
 	}
+
+	fixFileNameViolations(
+		violations.map((v) => ({
+			sourcePath: v.filePath,
+			destPath: path.join(path.dirname(v.filePath), v.suggestedName),
+		})),
+	);
+
 	return true;
 }