Update flags package json (#62)

* (flags) remove index from exports

* Update tsup.config, use clean:true instead of rimraf

* Add changeset

* pnpm-lock

* update tsup config, remove unused provider entrypoints

* tsup.config: update entry format

* Add attw check in CI

* Update .changeset/tidy-schools-retire.md

---------

Co-authored-by: Dominik Ferber <dominik.ferber@gmail.com>

Author: AAorris

.changeset/tidy-schools-retire.md

@@ -0,0 +1,5 @@
+---
+'@vercel/flags': patch
+---
+
+Update package.json and tsup.config.js for flags package

.github/workflows/quality.yml

@@ -74,6 +74,9 @@ jobs:
       - name: Run TypeScript type check
         run: pnpm run type-check
 
+      - name: Run `@arethetypeswrong/cli` check
+        run: pnpm run attw
+
   publint:
     name: 'publint'
     runs-on: ubuntu-latest

package.json

@@ -12,6 +12,7 @@
   },
   "license": "MIT",
   "scripts": {
+    "attw": "turbo attw",
     "build": "turbo build",
     "dev": "turbo dev",
     "lint": "turbo lint -- --max-warnings=0",

packages/flags/package.json

@@ -27,20 +27,20 @@
       "require": "./dist/index.cjs"
     },
     "./next": {
-      "import": "./dist/next/index.js",
-      "require": "./dist/next/index.cjs"
+      "import": "./dist/next.js",
+      "require": "./dist/next.cjs"
     },
     "./analytics": {
       "import": "./dist/analytics.js",
       "require": "./dist/analytics.cjs"
     },
     "./react": {
-      "import": "./dist/react/index.js",
-      "require": "./dist/react/index.cjs"
+      "import": "./dist/react.js",
+      "require": "./dist/react.cjs"
     },
     "./sveltekit": {
-      "import": "./dist/sveltekit/index.js",
-      "require": "./dist/sveltekit/index.cjs"
+      "import": "./dist/sveltekit.js",
+      "require": "./dist/sveltekit.cjs"
     }
   },
   "typesVersions": {
@@ -50,16 +50,16 @@
         "dist/*.d.cts"
       ],
       "next": [
-        "dist/next/index.d.ts",
-        "dist/next/index.d.cts"
+        "dist/next.d.ts",
+        "dist/next.d.cts"
       ],
       "react": [
-        "dist/react/index.d.ts",
-        "dist/react/index.d.cts"
+        "dist/react.d.ts",
+        "dist/react.d.cts"
       ],
       "sveltekit": [
-        "dist/sveltekit/index.d.ts",
-        "dist/sveltekit/index.d.cts"
+        "dist/sveltekit.d.ts",
+        "dist/sveltekit.d.cts"
       ]
     }
   },
@@ -68,7 +68,8 @@
     "CHANGELOG.md"
   ],
   "scripts": {
-    "build": "rimraf dist && tsup",
+    "attw": "attw --pack .",
+    "build": "tsup",
     "dev": "tsup --watch --clean=false",
     "eslint": "eslint-runner",
     "eslint:fix": "eslint-runner --fix",
@@ -81,14 +82,14 @@
     "jose": "^5.2.1"
   },
   "devDependencies": {
+    "@arethetypeswrong/cli": "0.17.3",
     "@types/node": "20.11.17",
     "@types/react": "18.2.55",
     "@vitejs/plugin-react": "4.2.1",
     "eslint-config-custom": "workspace:*",
     "msw": "2.6.4",
     "next": "15.1.4",
     "react": "canary",
-    "rimraf": "6.0.1",
     "tsconfig": "workspace:*",
     "tsup": "8.0.1",
     "typescript": "5.6.3",

packages/flags/tsup.config.js

@@ -5,26 +5,20 @@ const defaultConfig = {
   splitting: true,
   sourcemap: true,
   minify: false,
-  clean: false,
+  clean: true,
   skipNodeModulesBundle: true,
   dts: true,
-  external: ['node_modules'],
+  external: [/^node:.*/, 'node_modules'],
 };
 
 // eslint-disable-next-line import/no-default-export -- [@vercel/style-guide@5 migration]
 export default defineConfig({
-  entry: [
-    'src/next/index.ts',
-    'src/sveltekit/index.ts',
-    'src/react/index.tsx',
-    'src/index.ts',
-    'src/analytics.ts',
-    'src/providers/launchdarkly.ts',
-    'src/providers/split.ts',
-    'src/providers/statsig.ts',
-    'src/providers/optimizely.ts',
-    'src/providers/happykit.ts',
-    'src/providers/hypertune.ts',
-  ],
+  entry: {
+    index: 'src/index.ts',
+    next: 'src/next/index.ts',
+    sveltekit: 'src/sveltekit/index.ts',
+    react: 'src/react/index.tsx',
+    analytics: 'src/analytics.ts',
+  },
   ...defaultConfig,
 });