startx 0.7.2 → 0.9.0

package/apps/startx-cli/src/configs/scripts.ts

@@ -1,5 +1,3 @@
-/* eslint-disable @typescript-eslint/naming-convention */
-
 import type { SCRIPT } from "../types";
 
 export const scripts: SCRIPT = {
@@ -10,7 +8,7 @@ export const scripts: SCRIPT = {
 		},
 		{
 			script: "tsx watch src/index.ts",
-			tags: ["runnable", "node", "backend", "dev"],
+			tags: ["runnable", "node", "backend", "express"],
 		},
 		{
 			script: "email dev --port 3014 --dir ./src",
@@ -24,7 +22,7 @@ export const scripts: SCRIPT = {
 		},
 		{
 			script: "tsx watch --inspect src/index.ts",
-			tags: ["backend", "node", "runnable", "dev"],
+			tags: ["backend", "node", "runnable", "express"],
 		},
 	],
 	"bun:dev": [
@@ -34,7 +32,7 @@ export const scripts: SCRIPT = {
 		},
 		{
 			script: "bun --watch src/index.ts",
-			tags: ["backend", "node", "runnable", "dev"],
+			tags: ["backend", "node", "runnable", "express"],
 		},
 	],
 	"build": [
@@ -44,7 +42,7 @@ export const scripts: SCRIPT = {
 		},
 		{
 			script: "tsdown --config-loader unrun",
-			tags: ["runnable", "node"],
+			tags: ["runnable", "node", "tsdown"],
 		},
 	],
 	"cli": [
@@ -69,7 +67,7 @@ export const scripts: SCRIPT = {
 	],
 	"lint": [
 		{
-			script: "turbo run eslint",
+			script: "turbo run lint",
 			tags: ["node", "eslint", "root"],
 		},
 		{
@@ -107,6 +105,26 @@ export const scripts: SCRIPT = {
 			tags: ["node"],
 		},
 	],
+	"db:push": [
+		{
+			script: "drizzle-kit push",
+			tags: ["drizzle", "db"],
+		},
+		{
+			script: "turbo run db:push",
+			tags: ["db", "root"],
+		},
+	],
+	"db:studio": [
+		{
+			script: "drizzle-kit studio",
+			tags: ["drizzle", "db"],
+		},
+		{
+			script: "turbo run db:studio",
+			tags: ["db", "root"],
+		},
+	],
 	"typecheck": [
 		{
 			script: "turbo run typecheck",

package/apps/startx-cli/src/types.ts

@@ -2,25 +2,37 @@
 import type { PackageJson } from "type-fest";
 
 export type TAGS =
-	| "required"
+	// Always installs
 	| "common"
+	// Used for apps
 	| "runnable"
+	// Formatter tags
 	| "biome"
 	| "prettier"
-	| "vitest"
 	| "eslint"
+	// Testing tags
+	| "vitest"
+	// Build tags
 	| "tsdown"
+	// Backend tags
 	| "node"
 	| "backend"
-	| "dev"
+	| "express"
+	// Frontend tags
 	| "frontend"
+	| "react"
+	// Cli tags
 	| "cli"
 	| "commander"
-	| "react"
-	| "extra"
+	// Mail
 	| "mail"
+	// Never installs or ignore
 	| "never"
-	| "root";
+	// For handling workspace only files and scripts
+	| "root"
+	// Database tags
+	| "db"
+	| "drizzle";
 
 export type SCRIPT = Record<string, Array<{ script: string; tags: TAGS[] }>>;
 

package/apps/startx-cli/src/utils/cli-utils.ts

@@ -6,84 +6,94 @@ import z from "zod";
 
 import type { PnpmWorkspace, StartXPackageJson } from "../types";
 
-export type RawPackageItem = {
+export type PackageItem = {
 	type: "apps" | "configs" | "packages";
 	path: string;
+	relativePath: string;
 	name: string;
+	packageJson: StartXPackageJson;
 };
 
 const ENV = defineEnv({
 	STARTX_ENV: z.enum(["development", "production", "test", "staging"]).default("production"),
 });
+
 export class CliUtils {
 	static getDirectory() {
 		const __filename = fileURLToPath(import.meta.url);
 		let template = path.dirname(__filename);
 		const workspace = process.cwd();
+
 		if (ENV.STARTX_ENV === "development") {
 			template = path.resolve(template, "../../../../");
-		} else template = path.resolve(template, "../../../");
+		} else {
+			template = path.resolve(template, "../../../");
+		}
 
 		return {
 			template,
 			workspace,
 		};
 	}
-	static async getPackageList() {
-		const packages: RawPackageItem[] = [];
+
+	static async getPackageList(): Promise<PackageItem[]> {
 		const cliDirectory = this.getDirectory().template;
-		const safeListDir = async (dir: string): Promise<string[]> => {
+
+		const fetchPackages = async (
+			subPath: string,
+			type: "apps" | "configs" | "packages",
+			namePrefix = "",
+			filterFn?: (name: string) => boolean
+		): Promise<PackageItem[]> => {
+			const dirPath = path.join(cliDirectory, ...subPath.split("/"));
+
 			try {
-				return await fsTool.listDirectories({ dir });
+				let names = await fsTool.listDirectories({ dir: dirPath });
+				if (filterFn) names = names.filter(filterFn);
+
+				const packages = await Promise.all(
+					names.map(async name => {
+						const pkgPath = path.join(dirPath, name);
+						const relativePath = path.relative(cliDirectory, pkgPath);
+						const pkgName = namePrefix ? `${namePrefix}${name}` : name;
+						let packageJson;
+
+						try {
+							packageJson = await this.parsePackageJson({ dir: pkgPath });
+						} catch {
+							packageJson = null;
+						}
+
+						if (!packageJson) {
+							console.error(`Ignoring this package failed to read package.json: ${pkgName}`);
+							return null;
+						}
+
+						return {
+							type,
+							path: pkgPath,
+							relativePath,
+							name: pkgName,
+							packageJson,
+						};
+					})
+				);
+
+				return packages.filter((pkg): pkg is PackageItem => pkg !== null);
 			} catch (error) {
-				console.error("Error listing directory:", error);
+				console.error(`Error reading directory ${dirPath}:`, error);
 				return [];
 			}
 		};
 
-		const [availableApps, availableConfigs, availablePackages, availableRepoPackages] =
-			await Promise.all([
-				safeListDir(path.join(cliDirectory, "apps")),
-				safeListDir(path.join(cliDirectory, "configs")),
-				safeListDir(path.join(cliDirectory, "packages")),
-				safeListDir(path.join(cliDirectory, "packages", "@repo")),
-			]);
-
-		// 2. Map Apps
-		availableApps.forEach(name => {
-			packages.push({
-				type: "apps",
-				path: path.join(cliDirectory, "apps", name),
-				name,
-			});
-		});
+		const results = await Promise.all([
+			fetchPackages("apps", "apps"),
+			fetchPackages("configs", "configs"),
+			fetchPackages("packages", "packages", "", name => name !== "@repo"),
+			fetchPackages("packages/@repo", "packages", "@repo/"),
+		]);
 
-		availableConfigs.forEach(name => {
-			packages.push({
-				type: "configs",
-				path: path.join(cliDirectory, "configs", name),
-				name,
-			});
-		});
-
-		availablePackages.forEach(name => {
-			if (name === "@repo") return;
-			packages.push({
-				type: "packages",
-				path: path.join(cliDirectory, "packages", name),
-				name,
-			});
-		});
-
-		availableRepoPackages.forEach(name => {
-			packages.push({
-				type: "packages",
-				path: path.join(cliDirectory, "packages", "@repo", name),
-				name: `@repo/${name}`,
-			});
-		});
-
-		return packages;
+		return results.flat();
 	}
 
 	static async parsePackageJson({ dir, file = "package" }: { dir: string; file?: string }) {
@@ -93,6 +103,7 @@ export class CliUtils {
 		});
 		return packageJson;
 	}
+
 	static async parsePnpmWorkspace({ dir }: { dir: string }) {
 		const yaml = await fsTool.readYamlFile({
 			file: "pnpm-workspace",

package/apps/startx-cli/src/utils/file-handler.ts

@@ -5,9 +5,7 @@ import type { StartXPackageJson, TAGS } from "../types";
 
 export class FileHandler {
 	private static objSorter(obj: Record<string, unknown>, sorter: string[] = []) {
-		const cleaned = Object.fromEntries(
-			Object.entries(obj).filter(([, v]) => v !== null && v !== undefined)
-		);
+		const cleaned = Object.fromEntries(Object.entries(obj).filter(([, v]) => v !== null && v !== undefined));
 
 		const sortedEntries: Array<[string, unknown]> = [];
 
@@ -63,13 +61,20 @@ export class FileHandler {
 		const devDependencies = filterDeps(props.app.devDependencies as Record<string, string>);
 
 		// Removing all workspace dependencies
+		for (const [key, value] of Object.entries(dependencies)) {
+			if (value.includes("workspace:")) {
+				delete dependencies[key];
+			}
+		}
+
+		// Removing all workspace dev dependencies
 		for (const [key, value] of Object.entries(devDependencies)) {
 			if (value.includes("workspace:")) {
 				delete devDependencies[key];
 			}
 		}
 
-		// Adding required
+		// Adding props required dependencies
 		if (props.dependencies) {
 			for (const [key, value] of Object.entries(props.dependencies)) {
 				if (!dependencies[key]) {
@@ -78,6 +83,10 @@ export class FileHandler {
 			}
 		}
 
+		// Adding required dev & devDeps
+		props.app.startx?.requiredDevDeps?.forEach(e => (devDependencies[e] = "workspace:^"));
+		props.app.startx?.requiredDeps?.forEach(e => (dependencies[e] = "workspace:^"));
+
 		// Adding rest
 		for (const [key, value] of Object.entries(DepCheck)) {
 			if (!value.tags.every(tag => tags.includes(tag))) continue;
@@ -85,7 +94,7 @@ export class FileHandler {
 			const isDev = value.isDevDependency;
 			if (isDev && !devDependencies[key]) {
 				devDependencies[key] = value.version;
-			} else if (!dependencies[key]) {
+			} else if (!isDev && !dependencies[key]) {
 				dependencies[key] = value.version;
 			}
 		}

package/biome.json

@@ -28,7 +28,7 @@
 		"indentStyle": "tab",
 		"indentWidth": 2,
 		"lineEnding": "lf",
-		"lineWidth": 100,
+		"lineWidth": 120,
 		"attributePosition": "auto"
 	},
 	"assist": { "actions": { "source": { "organizeImports": "off" } } },

package/configs/eslint-config/src/configs/base.ts

@@ -1,20 +1,15 @@
 import { fixupPluginRules } from "@eslint/compat";
-import js from "@eslint/js";
-import stylisticPlugin from "@stylistic/eslint-plugin";
-import { defineConfig } from "eslint/config";
 import eslintConfigPrettier from "eslint-config-prettier";
-import { createTypeScriptImportResolver } from "eslint-import-resolver-typescript";
 import importPlugin from "eslint-plugin-import-x";
 import lodashPlugin from "eslint-plugin-lodash";
-import reactPlugin from "eslint-plugin-react";
-import reactHooksPlugin from "eslint-plugin-react-hooks";
 import unicornPlugin from "eslint-plugin-unicorn";
 import unusedImportsPlugin from "eslint-plugin-unused-imports";
 import globals from "globals";
 import tseslint from "typescript-eslint";
 import { localRulesPlugin } from "../plugin.js";
 
-export const baseConfig = defineConfig(
+// Exported as a standard array for Flat Config
+export const baseConfig = tseslint.config(
 	// 1. Global Ignores
 	{
 		ignores: [
@@ -29,11 +24,11 @@ export const baseConfig = defineConfig(
 			"vite.config.ts",
 			"vitest.config.ts",
 			"tsdown.config.ts",
+			"drizzle.config.ts",
 		],
 	},
 
 	// 2. Base Configurations
-	js.configs.recommended,
 	...tseslint.configs.recommended,
 	...tseslint.configs.recommendedTypeChecked,
 	importPlugin.flatConfigs.recommended as any,
@@ -43,17 +38,14 @@ export const baseConfig = defineConfig(
 	{
 		plugins: {
 			"unused-imports": fixupPluginRules(unusedImportsPlugin),
-			"react-hooks": fixupPluginRules(reactHooksPlugin as any),
 			lodash: fixupPluginRules(lodashPlugin),
-			"@stylistic": stylisticPlugin as any,
 			unicorn: unicornPlugin,
-			react: reactPlugin,
+			"local-rules": localRulesPlugin,
 		},
 		languageOptions: {
-			ecmaVersion: 2022,
+			ecmaVersion: "latest",
 			sourceType: "module",
 			globals: {
-				...globals.browser,
 				...globals.node,
 				...globals.es2021,
 			},
@@ -62,25 +54,16 @@ export const baseConfig = defineConfig(
 				tsconfigRootDir: import.meta.dirname,
 			},
 		},
-		settings: {
-			"import-x/resolver-next": [createTypeScriptImportResolver()],
-			react: {
-				version: "detect",
-			},
-		},
 		rules: {
 			// Core Rules
 			"no-void": ["error", { allowAsStatement: true }],
 			"no-constant-binary-expression": "error",
-			"no-console": ["warn", { allow: ["warn", "error"] }],
-			eqeqeq: "warn",
+			"no-console": ["warn", { allow: ["warn", "error", "info"] }], // Allowed info for debugging
+			eqeqeq: ["warn", "always", { null: "ignore" }],
 			"object-shorthand": "warn",
 			"prefer-const": "warn",
 
-			"id-denylist": "off",
-			"import-x/no-named-as-default": "off",
-
-			// TypeScript
+			// TypeScript Quality of Life
 			"@typescript-eslint/await-thenable": "error",
 			"@typescript-eslint/no-floating-promises": ["error", { ignoreVoid: true }],
 			"@typescript-eslint/no-misused-promises": ["error", { checksVoidReturn: false }],
@@ -90,75 +73,60 @@ export const baseConfig = defineConfig(
 			"@typescript-eslint/no-unsafe-argument": "warn",
 			"@typescript-eslint/no-unsafe-call": "warn",
 			"@typescript-eslint/no-unsafe-return": "off",
-			"@typescript-eslint/no-unsafe-member-access": ["error", { "allowOptionalChaining": true }],
+			"@typescript-eslint/no-unsafe-member-access": ["warn", { allowOptionalChaining: true }],
 			"@typescript-eslint/array-type": ["warn", { default: "array-simple" }],
-			"@typescript-eslint/consistent-type-assertions": "warn",
-			"@typescript-eslint/consistent-type-imports": "warn",
-			"@typescript-eslint/consistent-type-exports": "warn",
 			"@typescript-eslint/return-await": ["warn", "always"],
+			"@typescript-eslint/no-empty-object-type": ["warn"],
 
-			"@typescript-eslint/explicit-member-accessibility": "off",
-
+			// Naming Conventions (Relaxed for APIs and strict for standard code)
 			"@typescript-eslint/naming-convention": [
 				"warn",
 				{ selector: "default", format: ["camelCase"] },
-
 				{ selector: "import", format: ["camelCase", "PascalCase"] },
-
 				{
 					selector: "variable",
 					format: ["camelCase", "snake_case", "UPPER_CASE", "PascalCase"],
-					leadingUnderscore: "allowSingleOrDouble",
-					trailingUnderscore: "allowSingleOrDouble",
+					leadingUnderscore: "allow",
 				},
-
 				{
 					selector: "parameter",
 					format: ["camelCase"],
-					leadingUnderscore: "allowSingleOrDouble",
+					leadingUnderscore: "allow",
 				},
-
 				{ selector: "typeLike", format: ["PascalCase"] },
-				{ selector: "enumMember", format: ["UPPER_CASE", "PascalCase"] },
-			],
-			"@typescript-eslint/no-restricted-types": [
-				"warn",
 				{
-					types: {
-						Object: { message: "Use object instead", fixWith: "object" },
-						String: { message: "Use string instead", fixWith: "string" },
-						Boolean: { message: "Use boolean instead", fixWith: "boolean" },
-						Number: { message: "Use number instead", fixWith: "number" },
-						Symbol: { message: "Use symbol instead", fixWith: "symbol" },
-					},
+					selector: ["objectLiteralProperty", "typeProperty"],
+					format: null, // Critical: Allows API payloads with snake_case or headers
 				},
 			],
-			"@typescript-eslint/no-unused-vars": "warn",
-			"@typescript-eslint/no-empty-object-type": "warn",
-			// Stylistic
-			"@stylistic/member-delimiter-style": [
+
+			// Handled entirely by unused-imports
+			"@typescript-eslint/no-unused-vars": "off",
+			"unused-imports/no-unused-imports": "warn",
+			"unused-imports/no-unused-vars": [
 				"warn",
 				{
-					multiline: { delimiter: "semi", requireLast: true },
-					singleline: { delimiter: "semi", requireLast: false },
+					vars: "all",
+					varsIgnorePattern: "^_",
+					args: "after-used",
+					argsIgnorePattern: "^_",
 				},
 			],
 
+			// Imports
 			"import-x/no-cycle": "off",
-			"import-x/no-default-export": "off",
-
+			"import-x/no-named-as-default": "off",
 			"import-x/no-duplicates": "warn",
 			"import-x/order": [
 				"warn",
 				{
 					alphabetize: { order: "asc", caseInsensitive: true },
 					groups: [["builtin", "external"], "internal", ["parent", "index", "sibling"], "object"],
-					"newlines-between": "always",
+					// "newlines-between": "always",
 				},
 			],
 
 			// Plugins
-			"unused-imports/no-unused-imports": "warn",
 			"unicorn/no-unnecessary-await": "warn",
 			"unicorn/no-useless-promise-resolve-reject": "warn",
 			"lodash/path-style": ["warn", "as-needed"],
@@ -166,32 +134,17 @@ export const baseConfig = defineConfig(
 		},
 	},
 
-	// 4. React Specific Overrides
+	// 4. Test Overrides
 	{
-		files: ["**/*.tsx", "**/*.jsx"],
-		...reactPlugin.configs.flat.recommended,
-		...reactPlugin.configs.flat["jsx-runtime"],
-		rules: {
-			...reactHooksPlugin.configs.recommended.rules,
-			"react/prop-types": "off",
-			"react/react-in-jsx-scope": "off",
-			"react/jsx-no-leaked-render": "error", // Security/Perf rule (Keep as error)
-		},
-	},
-
-	// 5. Test Overrides
-	{
-		files: ["test/**/*.ts", "**/__tests__/*.ts", "**/*.test.ts", "**/*.cy.ts"],
+		files: ["test/**/*.ts", "**/__tests__/*.ts", "**/*.test.ts", "**/*.cy.ts", "**/*.spec.ts"],
 		rules: {
 			"local-rules/no-plain-errors": "off",
 			"@typescript-eslint/unbound-method": "off",
-			"import-x/no-default-export": "off",
+			"@typescript-eslint/no-unsafe-call": "off",
+			"@typescript-eslint/no-unsafe-member-access": "off",
 		},
 	},
 
-	// 6. Local Rules
-	localRulesPlugin.configs.recommended,
-
-	// 7. Prettier (Must be last to cleanly override stylistic rules)
+	// 5. Prettier (Must be absolutely last)
 	eslintConfigPrettier
 );

package/configs/eslint-config/src/configs/extend.ts

@@ -1,3 +1,3 @@
-import { defineConfig } from "eslint/config";
+import tseslint from "typescript-eslint";
 
-export const extend = defineConfig;
+export const extend = tseslint.config;

package/configs/eslint-config/src/configs/frontend.ts

@@ -1,23 +1,21 @@
 import { fixupPluginRules } from "@eslint/compat";
-import { defineConfig } from "eslint/config";
+import eslintConfigPrettier from "eslint-config-prettier";
 import jsxA11yPlugin from "eslint-plugin-jsx-a11y";
 import reactPlugin from "eslint-plugin-react";
 import reactHooksPlugin from "eslint-plugin-react-hooks";
 import globals from "globals";
+import tseslint from "typescript-eslint";
+
 import { baseConfig } from "./base.js";
-export const frontendConfig = defineConfig(
-	...baseConfig,
+
+const baseWithoutPrettier = baseConfig.filter(config => config !== eslintConfigPrettier);
+
+export const frontendConfig = tseslint.config(
+	...baseWithoutPrettier,
 
 	// 1. Frontend Ignores
 	{
-		ignores: [
-			"**/dist/**",
-			"**/coverage/**",
-			"**/storybook-static/**",
-			"**/*.snap",
-			"**/*.d.ts",
-			"vite.config.ts",
-		],
+		ignores: ["**/coverage/**", "**/storybook-static/**", "**/*.snap", "**/*.d.ts"],
 	},
 
 	// 2. Browser/React Globals & Settings
@@ -37,8 +35,8 @@ export const frontendConfig = defineConfig(
 	},
 
 	// 3. React Recommended Configs
-	reactPlugin.configs.flat.recommended,
-	reactPlugin.configs.flat["jsx-runtime"],
+	reactPlugin.configs.flat.recommended as any,
+	reactPlugin.configs.flat["jsx-runtime"] as any,
 
 	// 4. React, Hooks, and Accessibility Rules
 	{
@@ -52,6 +50,7 @@ export const frontendConfig = defineConfig(
 			...jsxA11yPlugin.configs.recommended.rules,
 
 			// --- Naming Convention Override for React ---
+			// Allows PascalCase for functional components while keeping variables camelCase
 			"@typescript-eslint/naming-convention": [
 				"warn",
 				{ selector: "default", format: ["camelCase"] },
@@ -60,17 +59,18 @@ export const frontendConfig = defineConfig(
 				{
 					selector: "variable",
 					format: ["camelCase", "snake_case", "UPPER_CASE", "PascalCase"],
-					leadingUnderscore: "allowSingleOrDouble",
-					trailingUnderscore: "allowSingleOrDouble",
+					leadingUnderscore: "allow",
 				},
 				{ selector: "typeLike", format: ["PascalCase"] },
-				{ selector: "enumMember", format: ["UPPER_CASE", "PascalCase"] },
 			],
 
 			// --- React Specifics ---
-			"react/prop-types": "off",
+			"react/prop-types": "off", // Using TS interfaces instead
+			"react/react-in-jsx-scope": "off", // Not needed for React 17+
+			"react/jsx-uses-react": "off", // Not needed for React 17+
 			"react/display-name": "warn",
 			"react/no-unescaped-entities": "warn",
+			"react/jsx-no-leaked-render": ["error", { validStrategies: ["ternary", "coerce"] }],
 
 			// --- React Hooks Specifics ---
 			"react-hooks/rules-of-hooks": "error",
@@ -81,7 +81,17 @@ export const frontendConfig = defineConfig(
 			"jsx-a11y/click-events-have-key-events": "warn",
 			"jsx-a11y/no-static-element-interactions": "warn",
 			"jsx-a11y/anchor-is-valid": "warn",
-			"jsx-a11y/no-onchange": "off",
+			"jsx-a11y/no-onchange": "off", // Deprecated rule, fine to turn off
+		},
+	},
+
+	{
+		files: ["**/emails/**/*.tsx", "**/emails/**/*.jsx"],
+		rules: {
+			"react/react-in-jsx-scope": "error",
+			"react/jsx-uses-react": "error",
 		},
-	}
+	},
+
+	eslintConfigPrettier
 );