fix: new build import bugs and types noop (#91)

Co-authored-by: Florian Lefebvre <[email protected]>

.changeset/four-bags-mix.md

@@ -0,0 +1,5 @@
+---
+"astro-integration-kit": patch
+---
+
+Fixes runtime error when importing the new DB type extension

.changeset/short-rice-return.md

@@ -0,0 +1,5 @@
+---
+"astro-integration-kit": patch
+---
+
+Fixes ESM import resolution after new build step

package/package.json

@@ -33,7 +33,8 @@
 			"default": "./dist/plugins/index.js"
 		},
 		"./types/db": {
-			"types": "./types/db.d.ts"
+			"types": "./types/db.d.ts",
+			"default": "./types/noop.js"
 		}
 	},
 	"files": [

package/src/utilities/add-dts.ts

@@ -3,7 +3,7 @@ import { dirname, relative } from "node:path";
 import { fileURLToPath } from "node:url";
 import type { AstroIntegrationLogger } from "astro";
 import { parse, prettyPrint } from "recast";
-import typescriptParser from "recast/parsers/typescript";
+import typescriptParser from "recast/parsers/typescript.js";
 
 const injectEnvDTS = ({
 	srcDir,

package/src/utilities/inject-dev-route.ts

@@ -1,5 +1,5 @@
 import type { InjectedRoute } from "astro";
-import type { HookParameters } from "../core/types";
+import type { HookParameters } from "../core/types.js";
 
 type HookParameterProperties = Pick<
 	HookParameters<"astro:config:setup">,

package/tsconfig.json

@@ -1,6 +1,8 @@
 {
   "extends": "astro/tsconfigs/strictest",
   "compilerOptions": {
+    "module": "Node16",
+    "moduleResolution": "Node16",
     "jsx": "preserve"
   },
   "exclude": ["dist"]

package/types/noop.js

@@ -0,0 +1,18 @@
+/**
+ * Type-only import is not supported with the nice syntax for modules across packages.
+ * This requires libraries using the extended types provided by AIK with one of the syntaxes below:
+ * - `import type {} from 'astro-integration-kit/types/db';` with the empty `{}`
+ * - `/// <reference types="astro-integration-kit/types/db" />` with the triple-slash directive`
+ *
+ * None of those are as nice as simply `import 'astro-integration-kit/types/db';`.
+ *
+ * Using the plain import means it will attempt to import a JS file during module resolution,
+ * which would fail at runtime because the type extensions are only available as types.
+ *
+ * As a workaround for that, this file exists. The type extensions are declared in the package.json
+ * with this file as their runtime counterpart, so the import will succeed and not throw at runtime.
+ *
+ * This file is a no-op, it does nothing. It is just a placeholder to make the nice import work.
+ *
+ * @see https://discord.com/channels/830184174198718474/1197638002764152843/1220528418035208212
+ */