@alphatique/hono-fetch 1.0.4 → 1.0.5

package/dist/index.mjs

@@ -1,10 +1,51 @@
-//#region src/utils.ts
-function replaceUrlParam(urlString, params) {
-	for (const [k, v] of Object.entries(params)) {
-		const reg = new RegExp("/:" + k + "(?:{[^/]+})?\\??");
-		urlString = urlString.replace(reg, v ? `/${v}` : "");
+//#region src/url.ts
+const PARAM_SEGMENT = /\/:([^/{}?]+)(\{[^/]*\})?(\?)?(?=\/|$)/g;
+/**
+* baseUrl、hono の route（schema の key）、param、query から送信先の URL を作る。
+* package の entry からは export しない。
+*
+* 不変条件:
+* - 戻り値の origin は、常に new URL(baseUrl).origin と一致する。
+* - 1 つの param は、encodeURIComponent した 1 つの path segment にだけ入る。
+* - `:name` は名前が完全に一致した場合にだけ置換する。`:id` は `:idx` に一致しない。
+* - `:name?` の値が undefined か空文字なら、その segment を取り除く。
+* - query は、undefined の値と配列内の undefined を送らない。配列は同じ key で append する。
+* - baseUrl の path は、これまでどおり使わない（route は baseUrl の origin からの絶対 path として解決する）。
+*
+* エラー（同期の TypeError）:
+* - baseUrl が URL として不正
+* - 必須の param が無い、または空文字
+* - param の値が "." または ".."（encode しても dot segment として正規化されるため）
+* - param の値が対になっていない surrogate を含む（encodeURIComponent が URIError を投げるため）
+* - 組み立てた URL の origin が baseUrl と異なる（多重の防御。上の規則が正しければ起きない）
+*
+* 対象外: route 中の `*` ワイルドカード
+*/
+function buildUrl(baseUrl, route, input = {}) {
+	const base = new URL(baseUrl);
+	const param = input.param ?? {};
+	const path = route.replace(PARAM_SEGMENT, (_match, name, _pattern, optional) => {
+		const value = Object.hasOwn(param, name) ? param[name] : void 0;
+		if (value === void 0 || value === "") {
+			if (optional) return "";
+			throw new TypeError(`hono-fetch: path param "${name}" is required for ${route}`);
+		}
+		if (value === "." || value === "..") throw new TypeError(`hono-fetch: path param "${name}" must not be "." or ".."`);
+		try {
+			return "/" + encodeURIComponent(value);
+		} catch (err) {
+			throw new TypeError(`hono-fetch: path param "${name}" is not a well-formed string`, { cause: err });
+		}
+	});
+	const url = new URL(path === "" ? "/" : path, base);
+	for (const [k, v] of Object.entries(input.query ?? {})) {
+		if (v === void 0) continue;
+		if (Array.isArray(v)) {
+			for (const v2 of v) if (v2 !== void 0) url.searchParams.append(k, v2);
+		} else url.searchParams.set(k, v);
 	}
-	return urlString;
+	if (url.origin !== base.origin) throw new TypeError(`hono-fetch: built URL left the origin of baseUrl: ${url.href}`);
+	return url;
 }
 //#endregion
 //#region src/index.ts
@@ -14,12 +55,7 @@ var HonoFetch = class {
 		this.config = config;
 	}
 	async fetch(method, path, args) {
-		const url = new URL(args.param ? replaceUrlParam(path, args.param) : path, this.config.baseUrl);
-		if (args.query) for (const [k, v] of Object.entries(args.query)) {
-			if (v === void 0) continue;
-			if (Array.isArray(v)) for (const v2 of v) url.searchParams.append(k, v2);
-			else url.searchParams.set(k, v);
-		}
+		const url = buildUrl(this.config.baseUrl, path, args);
 		let contentType = null;
 		let body = null;
 		if (args.json) {
@@ -38,7 +74,7 @@ var HonoFetch = class {
 		const headers = new Headers(args.header);
 		if (contentType) headers.set("Content-Type", contentType);
 		try {
-			return (await fetch(url, {
+			return await (await fetch(url, {
 				method: method.toUpperCase(),
 				headers,
 				body

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@alphatique/hono-fetch",
-	"version": "1.0.4",
+	"version": "1.0.5",
 	"license": "ISC",
 	"repository": {
 		"type": "git",
@@ -21,7 +21,8 @@
 	},
 	"scripts": {
 		"prepare": "vp config --no-agent",
-		"build": "vp pack"
+		"build": "vp pack",
+		"test": "vp test"
 	},
 	"devDependencies": {
 		"hono": "^4.12.10",