Unbundle patch

.changeset/purple-teeth-look.md

@@ -0,0 +1,9 @@
+---
+'skuba': minor
+---
+
+lint: Remove `unbundle` from tsdown configs
+
+When skuba first migrated packages to tsdown, it set `unbundle: true` with a `// TODO: determine if your project can be bundled` comment as a prompt for consumers to revisit the setting. This patch now automates that follow-up.
+
+The `unbundle` field and its TODO comment will be removed from your `tsdown.config` file if your package does not appear to ship translations (`.vocab` files) or CSS (`.css` files).

src/cli/format/__snapshots__/format.int.test.ts.snap

@@ -70,6 +70,8 @@ Patch skipped: Migrate tsdown config to 0.21 - no tsdown.config files found upgr
 
 Patch skipped: Replace pnpm install --prod with pnpm prune --prod in Dockerfiles - no dockerfiles found upgrade-skuba
 
+Patch skipped: Unbundle tsdown.config files to avoid shipping translations and CSS in packages that use tsdown for non-translation purposes - no tsdown.config files found upgrade-skuba
+
 skuba update complete. upgrade-skuba
 
 Processed skuba lints in <random>s.
@@ -191,6 +193,8 @@ Patch skipped: Migrate tsdown config to 0.21 - no tsdown.config files found upgr
 
 Patch skipped: Replace pnpm install --prod with pnpm prune --prod in Dockerfiles - no dockerfiles found upgrade-skuba
 
+Patch skipped: Unbundle tsdown.config files to avoid shipping translations and CSS in packages that use tsdown for non-translation purposes - no tsdown.config files found upgrade-skuba
+
 skuba update complete. upgrade-skuba
 
 Processed skuba lints in <random>s.
@@ -305,6 +309,8 @@ Patch skipped: Migrate tsdown config to 0.21 - no tsdown.config files found upgr
 
 Patch skipped: Replace pnpm install --prod with pnpm prune --prod in Dockerfiles - no dockerfiles found upgrade-skuba
 
+Patch skipped: Unbundle tsdown.config files to avoid shipping translations and CSS in packages that use tsdown for non-translation purposes - no tsdown.config files found upgrade-skuba
+
 skuba update complete. upgrade-skuba
 
 Processed skuba lints in <random>s.
@@ -391,6 +397,8 @@ Patch skipped: Migrate tsdown config to 0.21 - no tsdown.config files found upgr
 
 Patch skipped: Replace pnpm install --prod with pnpm prune --prod in Dockerfiles - no dockerfiles found upgrade-skuba
 
+Patch skipped: Unbundle tsdown.config files to avoid shipping translations and CSS in packages that use tsdown for non-translation purposes - no tsdown.config files found upgrade-skuba
+
 skuba update complete. upgrade-skuba
 
 Processed skuba lints in <random>s.

src/cli/lint/internalLints/upgrade/patches/15.2.0/migrateTsdown.ts

@@ -25,7 +25,7 @@ interface FoundEntry {
   pairNode: SgNode;
 }
 
-const removeNodeWithComma = (node: SgNode): Edit[] => {
+export const removeNodeWithComma = (node: SgNode): Edit[] => {
   const edits: Edit[] = [node.replace('')];
   const maybeCommaAfter = node.next();
   if (maybeCommaAfter?.text().trim() === ',') {

src/cli/lint/internalLints/upgrade/patches/15.3.1/index.ts

@@ -1,11 +1,17 @@
 import type { Patches } from '../../index.js';
 
 import { tryPatchDockerfilePruneProd } from './patchDockerfilePruneProd.js';
+import { tryPatchTsdownUnbundle } from './patchTsdownUnbundle.js';
 
 export const patches: Patches = [
   {
     apply: tryPatchDockerfilePruneProd,
     description:
       'Replace pnpm install --prod with pnpm prune --prod in Dockerfiles',
   },
+  {
+    apply: tryPatchTsdownUnbundle,
+    description:
+      'Unbundle tsdown.config files to avoid shipping translations and CSS in packages that use tsdown for non-translation purposes',
+  },
 ];