@sanity/pkg-utils 3.0.0 → 3.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/pkg-utils",
-  "version": "3.0.0",
+  "version": "3.1.1",
   "description": "Simple utilities for modern npm packages.",
   "keywords": [
     "sanity-io",
@@ -46,25 +46,6 @@
     "!dist/stats.html",
     "src"
   ],
-  "scripts": {
-    "build": "run-s clean build:pkg check:pkg",
-    "build:pkg": "node --loader esbuild-register/loader -r esbuild-register scripts/build",
-    "check:pkg": "node --loader esbuild-register/loader -r esbuild-register scripts/check",
-    "check:types": "tsc --build",
-    "clean": "rimraf dist",
-    "commit": "cz",
-    "extract": "sanity-tsdoc etl --outDir etc/api/@sanity/pkg-utils --tsconfig tsconfig.dist.json",
-    "format": "prettier --write --cache --ignore-unknown . && eslint . --ext .cjs,.js,.ts,.tsx --fix --quiet",
-    "lint": "eslint . --ext .cjs,.js,.ts,.tsx",
-    "playground:build": "pnpm --filter './playground/**' build",
-    "playground:clean": "pnpm --filter './playground/**' --silent clean",
-    "prepublishOnly": "pnpm run build",
-    "release": "semantic-release",
-    "test": "run-s playground:clean test:clean vitest:run",
-    "test:clean": "rimraf test/env/__tmp__",
-    "vitest:run": "vitest run",
-    "watch": "node --loader esbuild-register/loader -r esbuild-register scripts/watch"
-  },
   "commitlint": {
     "extends": [
       "@commitlint/config-conventional"
@@ -105,40 +86,24 @@
       }
     ]
   },
-  "release": {
-    "branches": [
-      "+([0-9])?(.{+([0-9]),x}).x",
-      "main",
-      "next",
-      {
-        "name": "beta",
-        "prerelease": true
-      },
-      {
-        "name": "alpha",
-        "prerelease": true
-      }
-    ],
-    "extends": "@sanity/semantic-release-preset"
-  },
   "dependencies": {
-    "@babel/core": "^7.23.0",
-    "@babel/preset-env": "^7.22.20",
+    "@babel/core": "^7.23.2",
+    "@babel/preset-env": "^7.23.2",
     "@babel/types": "^7.23.0",
     "@microsoft/api-extractor": "^7.38.0",
     "@microsoft/tsdoc-config": "^0.16.2",
     "@rollup/plugin-alias": "^5.0.1",
     "@rollup/plugin-babel": "^6.0.4",
-    "@rollup/plugin-commonjs": "^25.0.5",
+    "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-json": "^6.0.1",
     "@rollup/plugin-node-resolve": "^15.2.3",
-    "@rollup/plugin-replace": "^5.0.3",
+    "@rollup/plugin-replace": "^5.0.4",
     "@rollup/plugin-terser": "^0.4.4",
     "browserslist": "^4.22.1",
     "cac": "^6.7.14",
     "chalk": "^4.1.2",
     "chokidar": "^3.5.3",
-    "esbuild": "^0.19.4",
+    "esbuild": "^0.19.5",
     "esbuild-register": "^3.5.0",
     "find-config": "^1.0.0",
     "get-latest-version": "^5.1.0",
@@ -154,65 +119,73 @@
     "pretty-bytes": "^5.6.0",
     "prompts": "^2.4.2",
     "rimraf": "^4.4.1",
-    "rollup": "^4.0.2",
+    "rollup": "^4.1.4",
     "rollup-plugin-esbuild": "^6.1.0",
-    "rollup-plugin-preserve-directives": "^0.2.0",
     "rxjs": "^7.8.1",
     "treeify": "^1.1.0",
     "uuid": "^9.0.1",
     "zod": "^3.22.4"
   },
   "devDependencies": {
-    "@commitlint/cli": "^17.7.2",
-    "@commitlint/config-conventional": "^17.7.0",
-    "@sanity/pkg-utils": "workspace:*",
-    "@sanity/semantic-release-preset": "^4.1.4",
-    "@types/babel__core": "^7.20.2",
-    "@types/cpx": "^1.5.3",
-    "@types/find-config": "^1.0.2",
-    "@types/git-url-parse": "^9.0.1",
-    "@types/node": "^20.8.3",
-    "@types/parse-git-config": "^3.0.2",
-    "@types/prompts": "^2.4.5",
+    "@commitlint/cli": "^17.8.0",
+    "@commitlint/config-conventional": "^17.8.0",
+    "@types/babel__core": "^7.20.3",
+    "@types/cpx": "^1.5.4",
+    "@types/find-config": "^1.0.3",
+    "@types/git-url-parse": "^9.0.2",
+    "@types/node": "^20.8.7",
+    "@types/parse-git-config": "^3.0.3",
+    "@types/prompts": "^2.4.7",
     "@types/rimraf": "^3.0.2",
-    "@types/treeify": "^1.0.1",
-    "@types/uuid": "^9.0.4",
-    "@typescript-eslint/eslint-plugin": "^6.7.4",
-    "@typescript-eslint/parser": "^6.7.4",
+    "@types/treeify": "^1.0.2",
+    "@types/uuid": "^9.0.5",
+    "@typescript-eslint/eslint-plugin": "^6.8.0",
+    "@typescript-eslint/parser": "^6.8.0",
     "commitizen": "^4.3.0",
     "cpx": "^1.5.0",
     "cz-conventional-changelog": "^3.3.0",
     "eslint": "^8.51.0",
     "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-import": "^2.28.1",
-    "eslint-plugin-prettier": "^5.0.0",
+    "eslint-plugin-prettier": "^5.0.1",
     "eslint-plugin-simple-import-sort": "10.0.0",
     "husky": "^8.0.3",
-    "lint-staged": "^14.0.1",
+    "lint-staged": "^15.0.1",
     "npm-run-all": "^4.1.5",
     "outdent": "^0.8.0",
     "recast": "^0.23.4",
     "rollup-plugin-visualizer": "^5.9.2",
-    "semantic-release": "^21.1.2",
     "ts-node": "^10.9.1",
     "typescript": "^5.2.2",
     "vitest": "^0.34.6",
-    "vitest-github-actions-reporter": "^0.10.0"
+    "vitest-github-actions-reporter": "^0.10.0",
+    "@sanity/pkg-utils": "3.1.1"
   },
   "peerDependencies": {
     "typescript": "^4.7 || ^5"
   },
-  "packageManager": "pnpm@8.8.0",
+  "packageManager": "pnpm@8.9.2",
   "engines": {
     "node": ">=16.0.0"
   },
   "publishConfig": {
-    "access": "public",
-    "provenance": true
+    "access": "public"
   },
-  "pnpm": {
-    "overrides": {
-      "@sanity/pkg-utils": "workspace:*"
-    }
+  "scripts": {
+    "build": "run-s clean build:pkg check:pkg",
+    "build:pkg": "node --loader esbuild-register/loader -r esbuild-register scripts/build",
+    "check:pkg": "node --loader esbuild-register/loader -r esbuild-register scripts/check",
+    "check:types": "tsc --build",
+    "clean": "rimraf dist",
+    "commit": "cz",
+    "extract": "sanity-tsdoc etl --outDir etc/api/@sanity/pkg-utils --tsconfig tsconfig.dist.json",
+    "format": "prettier --write --cache --ignore-unknown . && eslint . --ext .cjs,.js,.ts,.tsx --fix --quiet",
+    "lint": "eslint . --ext .cjs,.js,.ts,.tsx",
+    "playground:build": "pnpm --filter './playground/**' build",
+    "playground:clean": "pnpm --filter './playground/**' --silent clean",
+    "test": "run-s playground:clean test:clean vitest:run",
+    "test:clean": "rimraf test/env/__tmp__",
+    "vitest:run": "vitest run",
+    "watch": "node --loader esbuild-register/loader -r esbuild-register scripts/watch"
   }
-}
+}

package/src/node/core/config/types.ts

@@ -114,11 +114,6 @@ export interface PkgConfigOptions {
    * Build package with support for legacy exports (writes root `<export>.js` files)
    */
   legacyExports?: boolean
-  /**
-   * Preserves module directives, such as `'use client'`
-   * @alpha
-   */
-  preserveModuleDirectives?: boolean
   minify?: boolean
   /** @alpha */
   rollup?: {

package/src/node/tasks/rollup/resolveRollupConfig.ts

@@ -8,7 +8,6 @@ import terser from '@rollup/plugin-terser'
 import path from 'path'
 import {InputOptions, OutputOptions, Plugin} from 'rollup'
 import esbuild from 'rollup-plugin-esbuild'
-import preserveDirectives from 'rollup-plugin-preserve-directives'
 
 import {BuildContext, DEFAULT_BROWSERSLIST_QUERY, resolveConfigProperty} from '../../core'
 import {RollupTask, RollupWatchTask} from '../types'
@@ -112,12 +111,7 @@ export function resolveRollupConfig(
     }),
     minify &&
       terser({
-        compress: config?.preserveModuleDirectives
-          ? {
-              // Needed until https://github.com/terser/terser/issues/1320 is fixed
-              directives: false,
-            }
-          : true,
+        compress: true,
         output: {
           comments: (_node, comment) => {
             const text = comment.value
@@ -133,8 +127,6 @@ export function resolveRollupConfig(
           },
         },
       }),
-    // Allows marking export files with `'use client'` at the top
-    config?.preserveModuleDirectives && preserveDirectives(),
   ].filter(Boolean) as Plugin[]
 
   const userPlugins = config?.rollup?.plugins
@@ -184,6 +176,12 @@ export function resolveRollupConfig(
         return {...acc, [entry.name]: entry.source}
       }, {}),
 
+      watch: {
+        chokidar: {
+          usePolling: true,
+        },
+      },
+
       plugins,
 
       treeshake: {
@@ -217,7 +215,6 @@ export function resolveRollupConfig(
       minifyInternalExports: minify,
       sourcemap: config?.sourcemap ?? true,
       hoistTransitiveImports: false,
-      preserveModules: config?.preserveModuleDirectives ? true : false,
     },
   }
 }

package/src/node/tasks/rollup/rollupTask.ts

@@ -114,26 +114,6 @@ async function execPromise(ctx: BuildContext, task: RollupTask) {
     const bundle = await rollup({
       ...inputOptions,
       onwarn(warning) {
-        if (
-          // Ignore the directive warning until the preserveDirectives plugin does it automatically: https://github.com/Ephem/rollup-plugin-preserve-directives#rollup-plugin-preserve-directives-warning
-          warning.message.includes(
-            'Module level directives cause errors when bundled, "use client" in',
-          )
-        ) {
-          // If the preserve option isn't enabled, then extend the message with a note about how to enable
-          if (!ctx.config?.preserveModuleDirectives) {
-            consoleSpy.messages.push({
-              type: 'warn',
-              code: warning.code,
-              args: [
-                `${warning.message} You can preserve it by setting "preserveModuleDirectives: true" in your "package.config.ts"`,
-              ],
-            })
-          }
-
-          return
-        }
-
         consoleSpy.messages.push({
           type: 'warn',
           code: warning.code,