@screenbook/cli 1.7.0 → 1.7.1

package/dist/index.mjs

@@ -827,23 +827,22 @@ async function generateCoverageData(config, cwd, screens) {
 		cwd,
 		ignore: config.ignore
 	});
-	new Set(screens.map((s) => {
-		const parts = s.id.split(".");
-		return parts.slice(0, -1).join("/") || parts[0];
-	}));
+	const metaDirs = /* @__PURE__ */ new Set();
+	for (const screen of screens) if (screen.filePath) {
+		const absoluteDir = dirname(screen.filePath);
+		const relativeDir = absoluteDir.startsWith(cwd) ? absoluteDir.slice(cwd.length + 1) : absoluteDir;
+		metaDirs.add(relativeDir);
+	}
 	const missing = [];
 	for (const routeFile of routeFiles) {
 		const routeDir = dirname(routeFile);
-		if (!screens.some((s) => {
-			const screenDir = s.id.replace(/\./g, "/");
-			return routeDir.includes(screenDir) || screenDir.includes(routeDir.replace(/^src\/pages\//, "").replace(/^app\//, ""));
-		})) missing.push({
+		if (!metaDirs.has(routeDir)) missing.push({
 			route: routeFile,
 			suggestedPath: join(dirname(routeFile), "screen.meta.ts")
 		});
 	}
 	const total = routeFiles.length > 0 ? routeFiles.length : screens.length;
-	const covered = screens.length;
+	const covered = routeFiles.length > 0 ? routeFiles.length - missing.length : screens.length;
 	const percentage = total > 0 ? Math.round(covered / total * 100) : 100;
 	const byOwner = {};
 	for (const screen of screens) {
@@ -3326,6 +3325,83 @@ function walkTemplateNodes(nodes, callback) {
 
 //#endregion
 //#region src/commands/generate.ts
+/**
+* Common paths where Vue Router configuration files are typically located
+*/
+const VUE_ROUTER_CONFIG_PATHS = [
+	"src/router/routes.ts",
+	"src/router/index.ts",
+	"src/router.ts",
+	"src/routes.ts",
+	"router/routes.ts",
+	"router/index.ts"
+];
+/**
+* Default patterns to exclude from screen.meta.ts generation.
+* These directories typically contain reusable components, not navigable screens.
+* @see https://github.com/wadakatu/screenbook/issues/170
+*/
+const DEFAULT_EXCLUDE_PATTERNS = [
+	"**/components/**",
+	"**/shared/**",
+	"**/utils/**",
+	"**/hooks/**",
+	"**/composables/**",
+	"**/stores/**",
+	"**/services/**",
+	"**/helpers/**",
+	"**/lib/**",
+	"**/common/**"
+];
+/**
+* Check if a path matches any of the exclude patterns
+*/
+function matchesExcludePattern(filePath, excludePatterns) {
+	for (const pattern of excludePatterns) {
+		const cleanPattern = pattern.replace(/\*\*/g, "").replace(/\*/g, "").replace(/^\//, "").replace(/\/$/, "");
+		if (cleanPattern && filePath.includes(`/${cleanPattern}/`)) return true;
+		if (cleanPattern && filePath.startsWith(`${cleanPattern}/`)) return true;
+	}
+	return false;
+}
+/**
+* Detect Vue Router configuration file in the project
+*/
+function detectVueRouterConfigFile(cwd) {
+	for (const configPath of VUE_ROUTER_CONFIG_PATHS) {
+		const absolutePath = join(cwd, configPath);
+		if (existsSync(absolutePath)) return absolutePath;
+	}
+	return null;
+}
+/**
+* Build a map from component file paths to their route information.
+* Normalizes paths for reliable matching.
+*/
+function buildRouteComponentMap(flatRoutes, cwd) {
+	const map = /* @__PURE__ */ new Map();
+	for (const route of flatRoutes) if (route.componentPath) {
+		const componentDir = dirname(relative(cwd, route.componentPath));
+		const componentNameWithoutExt = basename(route.componentPath).replace(/\.[^.]+$/, "");
+		map.set(componentDir, route);
+		map.set(basename(componentDir), route);
+		map.set(componentNameWithoutExt, route);
+	}
+	return map;
+}
+/**
+* Find matching route from the component map for a given route file
+*/
+function findMatchingRoute(routeFile, routeComponentMap) {
+	const routeDir = dirname(routeFile);
+	const routeDirName = basename(routeDir);
+	if (routeComponentMap.has(routeDir)) return routeComponentMap.get(routeDir) ?? null;
+	if (routeComponentMap.has(routeDirName)) return routeComponentMap.get(routeDirName) ?? null;
+	for (const [, route] of routeComponentMap) if (route.componentPath) {
+		if (basename(dirname(route.componentPath)) === routeDirName) return route;
+	}
+	return null;
+}
 const generateCommand = define({
 	name: "generate",
 	description: "Auto-generate screen.meta.ts files from route files",
@@ -3463,6 +3539,10 @@ async function generateFromRoutesFile(routesFile, cwd, options) {
 	for (const route of flatRoutes) {
 		const metaPath = determineMetaPath(route, cwd);
 		const absoluteMetaPath = resolve(cwd, metaPath);
+		if (matchesExcludePattern(metaPath, DEFAULT_EXCLUDE_PATTERNS)) {
+			skipped++;
+			continue;
+		}
 		if (!force && existsSync(absoluteMetaPath)) {
 			if (!interactive) {
 				skipped++;
@@ -3557,13 +3637,29 @@ async function generateFromRoutesPattern(routesPattern, cwd, options) {
 	logger.blank();
 	const routeFiles = await glob(routesPattern, {
 		cwd,
-		ignore
+		ignore: [...ignore, ...DEFAULT_EXCLUDE_PATTERNS]
 	});
 	if (routeFiles.length === 0) {
 		logger.warn(`No route files found matching: ${routesPattern}`);
 		return;
 	}
 	logger.log(`Found ${routeFiles.length} route files`);
+	let routeComponentMap = null;
+	if (routesPattern.includes(".vue")) {
+		const vueRouterConfig = detectVueRouterConfigFile(cwd);
+		if (vueRouterConfig) try {
+			const parseResult = parseVueRouterConfig(vueRouterConfig);
+			const flatRoutes = flattenRoutes(parseResult.routes);
+			if (flatRoutes.length > 0) {
+				routeComponentMap = buildRouteComponentMap(flatRoutes, cwd);
+				logger.log(`  ${logger.dim(`(using routes from ${relative(cwd, vueRouterConfig)})`)}`);
+			}
+			for (const warning of parseResult.warnings) logger.warn(warning);
+		} catch (error) {
+			const message = error instanceof Error ? error.message : String(error);
+			logger.warn(`Could not parse Vue Router config: ${message}`);
+		}
+	}
 	logger.blank();
 	let created = 0;
 	let skipped = 0;
@@ -3580,7 +3676,14 @@ async function generateFromRoutesPattern(routesPattern, cwd, options) {
 			skipped++;
 			continue;
 		}
-		const screenMeta = inferScreenMeta(routeDir, routesPattern);
+		let screenMeta;
+		const matchedRoute = routeComponentMap ? findMatchingRoute(routeFile, routeComponentMap) : null;
+		if (matchedRoute) screenMeta = {
+			id: matchedRoute.screenId,
+			title: matchedRoute.screenTitle,
+			route: matchedRoute.fullPath
+		};
+		else screenMeta = inferScreenMeta(routeDir, routesPattern);
 		let detectedApis = [];
 		if (detectApi && apiIntegration) {
 			const absoluteRouteFile = join(cwd, routeFile);