Merge pull request #633 from vector-im/bwindels/fix-sdk-build1

Attempt to fix SDK build and dev server errors for consuming app

scripts/sdk/base-manifest.json

@@ -2,14 +2,6 @@
     "name": "hydrogen-view-sdk",
     "description": "Embeddable matrix client library, including view components",
     "version": "0.0.2",
-    "main": "./hydrogen.cjs.js",
+    "main": "./hydrogen.es.js",
-    "exports": {
+    "type": "module"
-        ".": {
-            "import": "./hydrogen.es.js",
-            "require": "./hydrogen.cjs.js"
-        },
-        "./paths/vite": "./paths/vite.js",
-        "./style.css": "./style.css"
-    },
-    "types": "types/lib.d.ts"
 }

scripts/sdk/create-manifest.js

@@ -1,7 +1,29 @@
 #!/usr/bin/env node
 const fs = require("fs");
-const appManifest = require("../../package.json")
+const appManifest = require("../../package.json");
-const baseSDKManifest = require("./base-manifest.json")
+const baseSDKManifest = require("./base-manifest.json");
+/*
+    need to leave exports out of base-manifest.json because of #vite-bug,
+    with the downside that we can't support environments that support
+    both esm and commonjs modules, so we pick just esm.
+    ```
+    "exports": {
+        ".": {
+            "import": "./hydrogen.es.js",
+            "require": "./hydrogen.cjs.js"
+        },
+        "./paths/vite": "./paths/vite.js",
+        "./style.css": "./style.css"
+    },
+    ```
+
+    Also need to leave typescript type definitions out until the
+    typescript conversion is complete and all imports in the d.ts files
+    exists.
+    ```
+    "types": "types/lib.d.ts"
+    ```
+*/
 const mergeOptions = require('merge-options');
 
 const manifestExtension = {