fix: copy src into dist in w3up-client (#1239)

I ran into an issue in a Next.js app that seems to be a result of having sourcecode and type declarations in separate directories: ``` ./node_modules/.pnpm/@web3-storage [email protected]/node_modules/@web3-storage/w3up-client/src/types.ts:15:29 Type error: Could not find a declaration file for module './client.js'. '/Users/travis/dev/pl/fetch-test/node_modules/.pnpm/@web3-storage [email protected]/node_modules/@web3-storage/w3up-client/src/client.js' implicitly has an 'any' type. 13 | Unit, 14 | } from '@ucanto/interface' > 15 | import { type Client } from './client.js' | ^ ``` Copying the sourcecode into `dist` and updating exports to point to it seems to fix this.
storacha · Dec 13, 2023 · 468bb79 · 468bb79
1 parent bb5235f
commit 468bb79
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 16 deletions.
diff --git a/packages/w3up-client/package.json b/packages/w3up-client/package.json
@@ -4,7 +4,7 @@
   "description": "Client for the web3.storage w3up api",
   "license": "Apache-2.0 OR MIT",
   "type": "module",
-  "main": "src/index.js",
+  "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",
   "typesVersions": {
     "*": {
@@ -25,56 +25,55 @@
   "exports": {
     ".": {
       "types": "./dist/src/index.d.ts",
-      "node": "./src/index.node.js",
-      "import": "./src/index.js"
+      "node": "./dist/src/index.node.js",
+      "import": "./dist/src/index.js"
     },
     "./account": {
       "types": "./dist/src/account.d.ts",
-      "import": "./src/account.js"
+      "import": "./dist/src/account.js"
     },
     "./space": {
       "types": "./dist/src/space.d.ts",
-      "import": "./src/space.js"
+      "import": "./dist/src/space.js"
     },
     "./result": {
       "types": "./dist/src/result.d.ts",
-      "import": "./src/result.js"
+      "import": "./dist/src/result.js"
     },
     "./client": {
       "types": "./dist/src/client.d.ts",
-      "import": "./src/client.js"
+      "import": "./dist/src/client.js"
     },
     "./capability/access": {
       "types": "./dist/src/capability/access.d.ts",
-      "import": "./src/capability/access.js"
+      "import": "./dist/src/capability/access.js"
     },
     "./capability/space": {
       "types": "./dist/src/capability/space.d.ts",
-      "import": "./src/capability/space.js"
+      "import": "./dist/src/capability/space.js"
     },
     "./capability/store": {
       "types": "./dist/src/capability/store.d.ts",
-      "import": "./src/capability/store.js"
+      "import": "./dist/src/capability/store.js"
     },
     "./capability/subscription": {
       "types": "./dist/src/capability/subscription.d.ts",
-      "import": "./src/capability/subscription.js"
+      "import": "./dist/src/capability/subscription.js"
     },
     "./capability/upload": {
       "types": "./dist/src/capability/upload.d.ts",
-      "import": "./src/capability/upload.js"
+      "import": "./dist/src/capability/upload.js"
     },
     "./capability/usage": {
       "types": "./dist/src/capability/usage.d.ts",
-      "import": "./src/capability/usage.js"
+      "import": "./dist/src/capability/usage.js"
     },
-    "./types": "./src/types.js"
+    "./types": "./dist/src/types.js"
   },
   "publishConfig": {
     "access": "public"
   },
   "files": [
-    "src",
     "dist"
   ],
   "scripts": {

diff --git a/packages/w3up-client/src/delegation.js b/packages/w3up-client/src/delegation.js
@@ -1,10 +1,12 @@
 import { Delegation as CoreDelegation } from '@ucanto/core/delegation'
 
+/* c8 ignore start */
 /**
  * @template {import('./types.js').Capabilities} C
  * @extends {CoreDelegation<C>}
  */
 export class Delegation extends CoreDelegation {
+  /* c8 ignore stop */
   /** @type {Record<string, any>} */
   #meta
 

diff --git a/packages/w3up-client/tsconfig.json b/packages/w3up-client/tsconfig.json
@@ -2,7 +2,6 @@
   "extends": "../../tsconfig.json",
   "compilerOptions": {
     "outDir": "dist",
-    "emitDeclarationOnly": true,
     "composite": true,
     "noUnusedParameters": false
   },