Do not export ./snippets module in browser mode (#1259)

⚠️ PR opened on top of
#1255

Related to this thread
#1255 (comment).

@coyotte508 @SBrandeis  @julien-c WDYT? 🙈 


for the record, I'm looking for a solution where:
1. we keep this public/open-source
2. we keep the structure of jinja templates
3. ideally no new package in hf.js
4. ideally no new tooling (e.g. to translate jinja into TS code)

~Solution here is simply to gracefully raise an error if environment not
supported.~
**EDIT:** based on
#1259 (comment),
goal is to not package the `./snippets` module in browser mode.

---------

Co-authored-by: coyotte508 <[email protected]>

Author: Wauplin

packages/inference/package.json

@@ -33,9 +33,16 @@
 	"main": "./dist/index.cjs",
 	"module": "./dist/index.js",
 	"exports": {
-		"types": "./dist/src/index.d.ts",
-		"require": "./dist/index.cjs",
-		"import": "./dist/index.js"
+		".": {
+			"types": "./dist/src/index.d.ts",
+			"require": "./dist/index.cjs",
+			"import": "./dist/index.js"
+		}
+	},
+	"browser": {
+		"./src/snippets/index.js": false,
+		"./dist/index.js": "./dist/browser/index.js",
+		"./dist/index.mjs": "./dist/browser/index.mjs"
 	},
 	"type": "module",
 	"scripts": {

packages/inference/src/index.ts

@@ -2,6 +2,6 @@ export { InferenceClient, InferenceClientEndpoint, HfInference } from "./Inferen
 export { InferenceOutputError } from "./lib/InferenceOutputError";
 export * from "./types";
 export * from "./tasks";
-
 import * as snippets from "./snippets/index.js";
+
 export { snippets };

packages/inference/tsup.config.ts

@@ -0,0 +1,23 @@
+import type { Options } from "tsup";
+
+const baseConfig: Options = {
+	entry: ["./index.ts"],
+	format: ["cjs", "esm"],
+	outDir: "dist",
+	clean: true,
+};
+
+const nodeConfig: Options = {
+	...baseConfig,
+	platform: "node",
+};
+
+const browserConfig: Options = {
+	...baseConfig,
+	platform: "browser",
+	target: "es2018",
+	splitting: true,
+	outDir: "dist/browser",
+};
+
+export default [nodeConfig, browserConfig];