Merge pull request #117 from timlrx/v2

Update build process to work around nextjs issue

.changeset/dry-dolphins-burn.md

@@ -0,0 +1,5 @@
+---
+'pliny': patch
+---
+
+Split all files except ui components so that they are default exported

.changeset/pre.json

@@ -6,6 +6,7 @@
     "pliny": "0.0.10"
   },
   "changesets": [
+    "dry-dolphins-burn",
     "fuzzy-apples-judge",
     "giant-geckos-dress",
     "good-cheetahs-tap",

packages/pliny/CHANGELOG.md

@@ -1,5 +1,11 @@
 # pliny
 
+## 0.1.0-beta.9
+
+### Patch Changes
+
+- e669506: Split all files except ui components so that they are default exported
+
 ## 0.1.0-beta.8
 
 ### Patch Changes

packages/pliny/add-use-client.mjs

@@ -15,4 +15,12 @@ import globby from 'globby'
       fs.writeFileSync(path, insert + data)
     }
   }
+  // Handle ui differently as they are not split
+  const clientPaths = await globby(['ui/NewsletterForm.js', 'ui/Pre.js'])
+  for (const path of clientPaths) {
+    console.log(path)
+    const data = fs.readFileSync(path)
+    const insert = Buffer.from('"use client"\n')
+    fs.writeFileSync(path, insert + data)
+  }
 })()

packages/pliny/package.json

@@ -2,7 +2,7 @@
   "name": "pliny",
   "description": "Main entry point for pliny components",
   "homepage": "https://github.com/timlrx/pliny",
-  "version": "0.1.0-beta.8",
+  "version": "0.1.0-beta.9",
   "type": "module",
   "exports": {
     "./*": "./*",
@@ -18,8 +18,8 @@
     "**"
   ],
   "scripts": {
-    "dev": "tsup-node --watch && cp -fR dist/* ./ && rimraf dist && node add-use-client.mjs && yarn copyfiles",
-    "build": "rimraf dist && tsup && cp -fR dist/* ./ && rimraf dist && node add-use-client.mjs && yarn copyfiles",
+    "dev": "tsup && cp -fR dist/* ./ && tsup --config tsup.ui.config.ts && rimraf dist && node add-use-client.mjs",
+    "build": "tsup && cp -fR dist/* ./ && tsup --config tsup.ui.config.ts && rimraf dist && node add-use-client.mjs && yarn copyfiles",
     "copyfiles": "copyfiles -f public/algolia.css search"
   },
   "author": "Timothy Lin <[email protected]> (https://timlrx.com)",

packages/pliny/src/analytics/GoogleAnalytics.tsx

@@ -26,6 +26,7 @@ export const GA = ({ googleAnalyticsId }: GoogleAnalyticsProps) => {
 
 // https://developers.google.com/analytics/devguides/collection/gtagjs/events
 export const logEvent = (action, category, label, value) => {
+  // @ts-ignore
   window.gtag?.('event', action, {
     event_category: category,
     event_label: label,

packages/pliny/src/comments/Disqus.tsx

@@ -15,10 +15,12 @@ export const Disqus = ({ shortname, slug }: DisqusProps) => {
   const COMMENTS_ID = 'disqus_thread'
 
   const LoadComments = useCallback(() => {
+    //@ts-ignore
     window.disqus_config = function () {
       this.page.url = window.location.href
       this.page.identifier = slug
     }
+    //@ts-ignore
     if (window.DISQUS === undefined) {
       const script = document.createElement('script')
       script.src = 'https://' + shortname + '.disqus.com/embed.js'

packages/pliny/tsup.config.ts

@@ -3,6 +3,7 @@ import { defineConfig } from 'tsup'
 export default defineConfig({
   entry: ['src/**/*'],
   format: 'esm',
+  clean: true,
   splitting: true,
   treeshake: true,
   dts: true,

packages/pliny/tsup.ui.config.ts

@@ -0,0 +1,14 @@
+import { defineConfig } from 'tsup'
+
+// Do not split UI files as we want to use default export
+// https://github.com/vercel/next.js/issues/52415
+export default defineConfig({
+  entry: ['src/ui/*'],
+  format: 'esm',
+  outDir: 'ui',
+  clean: true,
+  splitting: false,
+  treeshake: true,
+  dts: true,
+  silent: true,
+})