@gradio/client 2.3.0 → 2.4.0

package/src/utils/submit.ts

@@ -19,11 +19,13 @@ import {
 } from "../helpers/api_info";
 import {
 	BROKEN_CONNECTION_MSG,
+	NO_API_INFO_MSG,
 	QUEUE_FULL_MSG,
 	SSE_URL,
 	SSE_DATA_URL,
 	RESET_URL,
-	CANCEL_URL
+	CANCEL_URL,
+	WS_PROTOCOL_MSG
 } from "../constants";
 import { apply_diff_stream, close_stream } from "./stream";
 import { Client } from "../client";
@@ -60,7 +62,7 @@ export function submit(
 
 		const that = this;
 
-		if (!api_info) throw new Error("No API found");
+		if (!api_info) throw new Error(NO_API_INFO_MSG);
 		if (!config) throw new Error("Could not resolve app config");
 
 		let { fn_index, endpoint_info, dependency } = get_endpoint_info(
@@ -75,7 +77,7 @@ export function submit(
 		let stream: EventSource | null;
 		let protocol = config.protocol ?? "ws";
 		if (protocol === "ws") {
-			throw new Error("WebSocket protocol is not supported in this version");
+			throw new Error(WS_PROTOCOL_MSG);
 		}
 		let event_id_final = "";
 		let event_id_cb: () => string = () => event_id_final;
@@ -182,7 +184,10 @@ export function submit(
 				data: input_data || [],
 				event_data,
 				fn_index,
-				trigger_id
+				trigger_id,
+				...(options.oauth_token && endpoint_info?.oauth_token
+					? { oauth_token: options.oauth_token }
+					: {})
 			};
 			if (skip_queue(fn_index, config)) {
 				fire_event({
@@ -630,6 +635,22 @@ export function submit(
 			}
 		});
 
+		// Surface internal failures (e.g. malformed payloads or configs) as an
+		// error event instead of leaving the returned iterator hanging forever
+		// with an unhandled promise rejection.
+		job.catch((e) => {
+			fire_event({
+				type: "status",
+				stage: "error",
+				message: e instanceof Error ? e.message : String(e),
+				queue: !skip_queue(fn_index, config),
+				endpoint: _endpoint,
+				fn_index,
+				time: new Date()
+			});
+			close();
+		});
+
 		let done = false;
 		const values: (IteratorResult<GradioEvent> | PromiseLike<never>)[] = [];
 		const resolvers: ((
@@ -735,25 +756,37 @@ function get_endpoint_info(
 } {
 	let fn_index: number;
 	let endpoint_info: EndpointInfo<JsApiData>;
-	let dependency: Dependency;
+	let dependency: Dependency | undefined;
 
 	if (typeof endpoint === "number") {
 		fn_index = endpoint;
 		endpoint_info = api_info.unnamed_endpoints[fn_index];
-		dependency = config.dependencies.find((dep) => dep.id == endpoint)!;
+		dependency = config.dependencies.find((dep) => dep.id == endpoint);
 	} else {
 		const trimmed_endpoint = endpoint.replace(/^\//, "");
 
 		fn_index = api_map[trimmed_endpoint];
-		endpoint_info = api_info.named_endpoints[endpoint.trim()];
+		// named endpoints are keyed with a leading slash in the API info, but
+		// accept endpoint names passed without one (e.g. "predict")
+		endpoint_info =
+			api_info.named_endpoints[endpoint.trim()] ??
+			api_info.named_endpoints[`/${trimmed_endpoint}`];
 		dependency = config.dependencies.find(
 			(dep) => dep.id == api_map[trimmed_endpoint]
-		)!;
+		);
 	}
 
-	if (typeof fn_index !== "number") {
+	if (typeof fn_index !== "number" || !dependency) {
+		const valid_endpoints = config.dependencies
+			.filter((dep) => dep.api_name)
+			.map((dep) => `"/${dep.api_name}"`)
+			.join(", ");
 		throw new Error(
-			"There is no endpoint matching that name of fn_index matching that number."
+			`No endpoint matching ${JSON.stringify(endpoint)} was found. ` +
+				(valid_endpoints
+					? `Valid named endpoints are: ${valid_endpoints}. `
+					: "This app exposes no named endpoints. ") +
+				"An fn_index (number) of an existing dependency can also be used."
 		);
 	}
 	return { fn_index, endpoint_info, dependency };

package/src/utils/upload_files.ts

@@ -34,7 +34,7 @@ export async function upload_files(
 				method: "POST",
 				body: formData,
 				headers,
-				credentials: "include"
+				credentials: this.options.credentials ?? "same-origin"
 			});
 		} catch (e) {
 			throw new Error(BROKEN_CONNECTION_MSG + (e as Error).message);

package/src/utils/view_api.ts

@@ -12,8 +12,7 @@ export async function view_api(this: Client): Promise<any> {
 
 	const headers: {
 		Authorization?: string;
-		"Content-Type": "application/json";
-	} = { "Content-Type": "application/json" };
+	} = {};
 
 	if (token) {
 		headers.Authorization = `Bearer ${token}`;
@@ -32,7 +31,7 @@ export async function view_api(this: Client): Promise<any> {
 			const url = join_urls(config.root, this.api_prefix, API_INFO_URL);
 			response = await this.fetch(url, {
 				headers,
-				credentials: "include"
+				credentials: this.options.credentials ?? "same-origin"
 			});
 			if (!response.ok) {
 				throw new Error(BROKEN_CONNECTION_MSG);

package/vite.config.ts

@@ -38,7 +38,12 @@ logger.info = (log, options) => {
 };
 export default defineConfig(({ mode }) => {
 	const production = mode === "production";
-	const isBrowserBuild = process.env.BROWSER_BUILD === "true";
+	// Single-file, minified browser bundle served from CDNs at
+	// dist/index.min.js (see https://github.com/gradio-app/gradio/issues/10028)
+	const isCdnBuild = process.env.CDN_BUILD === "true";
+	// a CDN build is always a browser build (guards against CDN_BUILD=true
+	// without BROWSER_BUILD=true emitting es+cjs into the same file name)
+	const isBrowserBuild = process.env.BROWSER_BUILD === "true" || isCdnBuild;
 
 	if (mode === "preview") {
 		return {
@@ -50,15 +55,26 @@ export default defineConfig(({ mode }) => {
 		customLogger: logger,
 		build: {
 			emptyOutDir: false,
+			...(isCdnBuild ? { minify: true } : {}),
 			lib: {
 				entry: "src/index.ts",
-				formats: ["es"],
-				fileName: isBrowserBuild ? `browser` : `index`
+				// the node build also emits a CommonJS bundle (dist/index.cjs)
+				// so that require("@gradio/client") works on all supported
+				// node versions (see https://github.com/gradio-app/gradio/issues/9214)
+				formats: isBrowserBuild ? ["es"] : ["es", "cjs"],
+				fileName: (format) => {
+					if (isCdnBuild) return "index.min.js";
+					if (isBrowserBuild) return "browser.js";
+					return format === "cjs" ? "index.cjs" : "index.js";
+				}
 			},
 			rollupOptions: {
 				input: "src/index.ts",
 				output: {
-					dir: "dist"
+					dir: "dist",
+					// keep the CDN bundle as a single file so the documented
+					// <script type="module"> import works without extra chunks
+					inlineDynamicImports: isCdnBuild
 				}
 			}
 		},

package/dist/__vite-browser-external-DYxpcVy9.js

@@ -1,4 +0,0 @@
-const e = {};
-export {
-  e as default
-};