Merge branch 'main' into copilot/consolidate-playgrounds-examples

Author: yamcodes

.github/actions/size-limit/bun.lock

@@ -11,7 +11,7 @@
 	},
 	"packageManager": "bun@1.2.22",
 	"dependencies": {
-		"arkregex": "0.0.3",
+		"arkregex": "0.0.4",
 		"typescript": "5.9.3"
 	}
 }

.github/actions/size-limit/package.json

@@ -60,9 +60,17 @@ await installAndBuild(config, config.isReleasePR);
 // Run size-limit on current branch
 const { results, hasErrors } = await runSizeLimit(config.filter);
 
+// Debug: Log results before filtering
+console.log(`🔍 Found ${results.length} total results before filtering`);
+console.log(
+	`🔍 Changed packages: ${changedPackages ? Array.from(changedPackages).join(", ") || "(none)" : "(all)"}`,
+);
+
 // Filter results to only include changed packages (if in PR context)
 const filteredResults = filterChangedPackages(results, changedPackages);
 
+console.log(`🔍 ${filteredResults.length} results after filtering`);
+
 // Log baseline and current sizes for debugging (especially for release PRs)
 logDebugInfo(filteredResults, baselineSizes, config.isReleasePR);
 
@@ -71,6 +79,7 @@ calculateDiffs(filteredResults, baselineSizes, config.isReleasePR);
 
 // Create the table
 const result = createTable(filteredResults);
+console.log(result);
 if (filteredResults.length === 0 && results.length > 0) {
 	console.log("⚠️ Could not parse size-limit output");
 }

.github/actions/size-limit/src/index.ts

@@ -62,34 +62,51 @@ export const getChangedPackages = async (
 		// Map changed files to packages
 		for (const file of changedFiles) {
 			// Check if file is in packages directory
-			const packagesMatch = file.match(
-				/^packages\/([@a-z0-9][@a-z0-9/_-]*)\//i,
-			);
-			if (packagesMatch?.[1]) {
-				const packageDir = packagesMatch[1];
+			if (file.startsWith("packages/")) {
+				// Find the nearest package.json
+				const parts = file.split("/");
+				// Start from the directory containing the file, go up until we hit "packages"
+				// parts[0] is "packages"
+				let packageFound = false;
+
+				// Iterate from deep to shallow, but stop before "packages" (index 0)
+				for (let i = parts.length - 1; i > 0; i--) {
+					const potentialPackageDir = parts.slice(1, i).join("/");
+					if (!potentialPackageDir) continue;
+
+					const packageJsonPath = join(
+						process.cwd(),
+						"packages",
+						potentialPackageDir,
+						"package.json",
+					);
 
-				// Get actual package name from package.json
-				const packageJsonPath = join(
-					process.cwd(),
-					"packages",
-					packageDir,
-					"package.json",
-				);
-				if (existsSync(packageJsonPath)) {
-					try {
-						const packageJson = JSON.parse(
-							readFileSync(packageJsonPath, "utf-8"),
-						);
-						if (packageJson.name) {
-							changedPackages.add(packageJson.name);
-						} else {
-							changedPackages.add(packageDir);
+					if (existsSync(packageJsonPath)) {
+						try {
+							const packageJson = JSON.parse(
+								readFileSync(packageJsonPath, "utf-8"),
+							);
+							if (packageJson.name) {
+								changedPackages.add(packageJson.name);
+								packageFound = true;
+								break;
+							}
+						} catch {
+							// Ignore invalid package.json
 						}
-					} catch {
-						changedPackages.add(packageDir);
 					}
-				} else {
-					changedPackages.add(packageDir);
+				}
+
+				// Fallback: if we couldn't find a package.json, try to guess the package directory
+				// This handles cases where package.json might be deleted or we can't read it
+				if (!packageFound) {
+					// Regex that handles scoped and unscoped packages
+					const packagesMatch = file.match(
+						/^packages\/((?:@[^/]+\/)?[^/]+)\//i,
+					);
+					if (packagesMatch?.[1]) {
+						changedPackages.add(packagesMatch[1]);
+					}
 				}
 			}
 		}

.github/actions/size-limit/src/package/changes.ts

@@ -94,8 +94,7 @@ export const runSizeLimitOnPackage = async (
 
 		return {
 			...config,
-			webpack: false, // Use esbuild instead of webpack
-			// Configure esbuild to mark Node.js built-in modules as external
+			// Use esbuild (default in preset-small-lib) and configure it to mark Node.js built-in modules as external
 			// This tells esbuild not to try to bundle these modules
 			// size-limit passes this to esbuild's external option
 			ignore: [
@@ -106,6 +105,12 @@ export const runSizeLimitOnPackage = async (
 					: []),
 				...nodeBuiltinModules.map((m) => `node:${m}`),
 				...nodeBuiltinModules,
+				// Automatically ignore peerDependencies
+				// This prevents bundling large peer dependencies like arktype or vite
+				// which should be external in the final bundle anyway
+				...(packageJson.peerDependencies
+					? Object.keys(packageJson.peerDependencies)
+					: []),
 			],
 		};
 	});

.github/actions/size-limit/src/size-limit/package.ts

@@ -5,8 +5,6 @@ on:
     branches:
       - main
 
-concurrency: ${{ github.workflow }}-${{ github.ref }}
-
 jobs:
   release:
     name: Release

.github/workflows/release.yml

@@ -5,6 +5,8 @@ on:
       - main
     paths:
       - "packages/**"
+  workflow_dispatch:
+
 jobs:
   changes:
     if: github.repository == 'yamcodes/arkenv'
@@ -23,7 +25,7 @@ jobs:
               - 'packages/**'
 
   size-limit:
-    if: github.repository == 'yamcodes/arkenv' && needs.changes.outputs.packages == 'true'
+    if: github.repository == 'yamcodes/arkenv' && (needs.changes.outputs.packages == 'true' || github.event_name == 'workflow_dispatch')
     needs: changes
     runs-on: ubuntu-latest
     env:

.github/workflows/size-limit.yml

@@ -33,6 +33,8 @@ jobs:
           cache: pnpm
       - name: Install dependencies
         run: pnpm install
+      - name: Build packages
+        run: pnpm run build --filter=./packages/*
       - run: pnpm run test
 
   test-typesafety:

.github/workflows/test.yml

@@ -41,3 +41,6 @@ pnpm-debug.log*
 
 # coverage
 coverage/
+
+# size-limit
+esbuild-why*.html

.gitignore

@@ -20,7 +20,8 @@
 	"devDependencies": {
 		"@types/bun": "catalog:",
 		"@types/react": "catalog:",
-		"@types/react-dom": "catalog:"
+		"@types/react-dom": "catalog:",
+		"rimraf": "catalog:"
 	},
 	"arkenvExamples": [
 		{