@hybridly/core 0.10.0-beta.15 → 0.10.0-beta.17

package/dist/index.d.mts

@@ -109,7 +109,7 @@ interface RequestHooks {
    */
   fail: (error: Error, request: PendingHybridRequest, context: InternalRouterContext) => MaybePromise<any>;
   /**
-   * Called after a request has been made, even if it didn't succeed.
+   * Called after a response has been received, even if it didn't succeed in a navigation.
    */
   after: (request: PendingHybridRequest, context: InternalRouterContext) => MaybePromise<any>;
 }
@@ -271,11 +271,23 @@ interface CloseDialogOptions extends HybridRequestOptions {
   local?: boolean;
 }
 //#endregion
+//#region src/query.d.ts
+type QueryArrayFormat = 'indices' | 'brackets';
+type QueryValue = string | number | boolean | null | undefined | QueryValue[] | Set<QueryValue> | {
+  [key: string]: QueryValue;
+};
+interface StringifyQueryOptions {
+  arrayFormat?: QueryArrayFormat;
+  addQueryPrefix?: boolean;
+}
+declare function parseQueryString(query: string): Record<string, any>;
+declare function stringifyQueryString(value: QueryValue, options?: StringifyQueryOptions): string;
+//#endregion
 //#region src/url.d.ts
 type UrlResolvable = string | URL | Location;
 type UrlTransformable = BaseUrlTransformable | ((string: URL) => BaseUrlTransformable);
 type BaseUrlTransformable = Partial<Omit<URL, 'searchParams' | 'toJSON' | 'toString'>> & {
-  query?: any;
+  query?: Record<string, QueryValue>;
   trailingSlash?: boolean;
 };
 /** Normalizes the given input to an URL. */
@@ -553,7 +565,7 @@ type Validation = Record<string, Errors>;
 declare const router: {
   abort: () => void;
   navigate: (options: HybridRequestOptions) => Promise<NavigationResponse>;
-  reload: (options: HybridRequestOptions | undefined) => Promise<NavigationResponse>;
+  reload: (options?: HybridRequestOptions | undefined) => Promise<NavigationResponse>;
   get: (url: UrlResolvable, options?: Omit<HybridRequestOptions, "method" | "url"> | undefined) => Promise<NavigationResponse>;
   post: (url: UrlResolvable, options?: Omit<HybridRequestOptions, "method" | "url"> | undefined) => Promise<NavigationResponse>;
   put: (url: UrlResolvable, options?: Omit<HybridRequestOptions, "method" | "url"> | undefined) => Promise<NavigationResponse>;
@@ -561,8 +573,8 @@ declare const router: {
   delete: (url: UrlResolvable, options?: Omit<HybridRequestOptions, "method" | "url"> | undefined) => Promise<NavigationResponse>;
   local: (url: UrlResolvable, options?: ComponentNavigationOptions) => Promise<void>;
   external: (url: UrlResolvable, data?: _hybridly_utils0.RequestData) => void;
-  to: <T extends RouteName>(name: T, parameters: Record<string, any> | undefined, options: Omit<HybridRequestOptions, "url"> | undefined) => Promise<NavigationResponse>;
-  matches: <T extends RouteName>(name: T, parameters: Record<string, any> | undefined) => boolean;
+  to: <T extends RouteName>(name: T, parameters?: Record<string, any> | undefined, options?: Omit<HybridRequestOptions, "url"> | undefined) => Promise<NavigationResponse>;
+  matches: <T extends RouteName>(name: T, parameters?: Record<string, any> | undefined) => boolean;
   current: () => string | undefined;
   dialog: {
     close: (options?: CloseDialogOptions | undefined) => Promise<void | NavigationResponse>;
@@ -575,15 +587,6 @@ declare const router: {
 /** Creates the hybridly router. */
 declare function createRouter(options: RouterContextOptions): Promise<InternalRouterContext>;
 //#endregion
-//#region src/query.d.ts
-type QueryArrayFormat = 'indices' | 'brackets';
-interface StringifyQueryOptions {
-  arrayFormat?: QueryArrayFormat;
-  addQueryPrefix?: boolean;
-}
-declare function parseQueryString(query: string): Record<string, any>;
-declare function stringifyQueryString(value: unknown, options?: StringifyQueryOptions): string;
-//#endregion
 //#region src/authorization.d.ts
 interface Authorizable<Authorizations extends Record<string, boolean>> {
   authorization: Authorizations;

package/dist/index.mjs

@@ -1,7 +1,8 @@
 import { t as __exportAll } from "./_chunks/chunk.mjs";
-import { debug, getByPath, hasFiles, merge, mergeObject, objectToFormData, random, showResponseErrorModal, wrap } from "@hybridly/utils";
+import { debug, hasFiles, merge, mergeObject, objectToFormData, random, showResponseErrorModal, wrap } from "@hybridly/utils";
 import { trimEnd } from "es-toolkit/string";
-import { parse, stringify } from "picoquery";
+import { isPlainObject } from "es-toolkit/predicate";
+import { parse, stringify } from "neoqs";
 import { debounce } from "es-toolkit/function";
 import { parse as parse$1, stringify as stringify$1 } from "superjson";
 import { get, set, uniqBy } from "es-toolkit/compat";
@@ -9,35 +10,15 @@ import { get, set, uniqBy } from "es-toolkit/compat";
 function parseQueryString(query) {
 	const source = query.startsWith("?") ? query.slice(1) : query;
 	if (!source) return {};
-	return parse(source, {
-		nesting: true,
-		nestingSyntax: "index",
-		arrayRepeat: true,
-		arrayRepeatSyntax: "bracket"
-	});
+	return parse(source);
 }
 function stringifyQueryString(value, options = {}) {
-	const normalizedQuery = unescapeBracketSyntaxInKeys(stringify(normalizeQueryValue(value), {
-		nesting: true,
-		nestingSyntax: "index",
-		arrayRepeat: (options.arrayFormat ?? "brackets") === "brackets",
-		arrayRepeatSyntax: "bracket"
-	}));
-	if (!normalizedQuery) return "";
-	return options.addQueryPrefix ? `?${normalizedQuery}` : normalizedQuery;
-}
-function unescapeBracketSyntaxInKeys(query) {
-	if (!query) return query;
-	return query.split("&").map((entry) => {
-		const separator = entry.indexOf("=");
-		if (separator < 0) return decodeBracketSyntax(entry);
-		const key = entry.slice(0, separator);
-		const value = entry.slice(separator);
-		return `${decodeBracketSyntax(key)}${value}`;
-	}).join("&");
-}
-function decodeBracketSyntax(value) {
-	return value.replace(/%5B/gi, "[").replace(/%5D/gi, "]");
+	const query = stringify(normalizeQueryValue(value), {
+		arrayFormat: options.arrayFormat ?? "brackets",
+		encodeValuesOnly: true
+	});
+	if (!query) return "";
+	return options.addQueryPrefix ? `?${query}` : query;
 }
 function normalizeQueryValue(value) {
 	if (value instanceof Set) return [...value].map((entry) => normalizeQueryValue(entry));
@@ -48,11 +29,6 @@ function normalizeQueryValue(value) {
 	}), {});
 	return value;
 }
-function isPlainObject(value) {
-	if (typeof value !== "object" || value === null) return false;
-	const prototype = Object.getPrototypeOf(value);
-	return prototype === null || prototype === Object.prototype;
-}
 //#endregion
 //#region src/url.ts
 /** Normalizes the given input to an URL. */
@@ -69,9 +45,8 @@ function makeUrl(href, transformations = {}) {
 		transformations = typeof transformations === "function" ? transformations(url) ?? {} : transformations ?? {};
 		Object.entries(transformations).forEach(([key, value]) => {
 			if (key === "query") {
-				const currentQueryParameters = merge(parseQueryString(url.search), value, { mergePlainObjects: true });
 				key = "search";
-				value = stringifyQueryString(currentQueryParameters, { arrayFormat: "brackets" });
+				value = stringifyQueryString(merge(parseQueryString(url.search), value, { mergePlainObjects: true }), { arrayFormat: "brackets" });
 			}
 			Reflect.set(url, key, value);
 		});
@@ -699,8 +674,8 @@ async function transformOptions(options) {
 		debug.router("Converted data to FormData.", options.data);
 	}
 	if (!(options.data instanceof FormData) && options.method === "GET" && Object.keys(options.data ?? {}).length) {
-		debug.router("Transforming data to query parameters.", options.data);
 		options.url = makeUrl(options.url ?? context.url, { query: options.data });
+		debug.router("Transforming data to query parameters.", options.data, options.url);
 		options.data = {};
 	}
 	if ([
@@ -949,8 +924,8 @@ function isPartial(options) {
 function resolveProperties(original, payload) {
 	const mergedPayloadProperties = merge(original, payload.properties);
 	(payload.mergeable ?? []).forEach(([mergeableProperty, prepends, uniqueBy]) => {
-		const originalValue = getByPath(original, mergeableProperty);
-		const newValue = getByPath(payload.properties, mergeableProperty);
+		const originalValue = get(original, mergeableProperty);
+		const newValue = get(payload.properties, mergeableProperty);
 		const mergeArrays = (current, incoming) => {
 			const merged = prepends === true ? [...incoming, ...current] : [...current, ...incoming];
 			if (typeof uniqueBy !== "string") return merged;
@@ -1014,7 +989,12 @@ async function processNextResponse() {
 		return;
 	}
 	debug.queue("Processing response", response);
-	response.request.resolve(await handleHybridRequestResponse(response));
+	try {
+		response.request.resolve(await handleHybridRequestResponse(response));
+	} finally {
+		debug.router("Ended navigation.", response.request);
+		await runHooks("after", response.request.options.hooks, response.request, getInternalRouterContext());
+	}
 	return await processNextResponse();
 }
 //#endregion
@@ -1071,8 +1051,6 @@ async function processRequest(request, onFinally) {
 		await handleTransportError(request, error);
 	} finally {
 		request.completed = true;
-		debug.router("Ended navigation.", request);
-		await runHooks("after", request.options.hooks, request, getRouterContext());
 		onFinally();
 	}
 }
@@ -1535,7 +1513,7 @@ async function closeDialog(options) {
 const router = {
 	abort: () => cancelNavigationRequest(),
 	navigate: async (options) => await performHybridNavigation(options),
-	reload: async (options) => await performHybridNavigation({
+	reload: async (options = {}) => await performHybridNavigation({
 		preserveScroll: true,
 		preserveState: true,
 		replace: true,
@@ -1573,7 +1551,7 @@ const router = {
 	}),
 	local: async (url, options = {}) => await performLocalNavigation(url, options),
 	external: (url, data = {}) => navigateToExternalUrl(url, data),
-	to: async (name, parameters, options) => {
+	to: async (name, parameters = void 0, options = {}) => {
 		const url = generateRouteFromName(name, parameters);
 		const method = getRouteDefinition(name).method.at(0);
 		return await performHybridNavigation({
@@ -1582,7 +1560,7 @@ const router = {
 			method
 		});
 	},
-	matches: (name, parameters) => currentRouteMatches(name, parameters),
+	matches: (name, parameters = void 0) => currentRouteMatches(name, parameters),
 	current: () => getCurrentRouteName(),
 	dialog: { close: (options = {}) => closeDialog(options) },
 	history: {

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@hybridly/core",
 	"type": "module",
-	"version": "0.10.0-beta.15",
+	"version": "0.10.0-beta.17",
 	"description": "Core functionality of Hybridly",
 	"author": "Enzo Innocenzi <enzo@innocenzi.dev>",
 	"license": "MIT",
@@ -40,8 +40,8 @@
 		"build:stub": "obuild --stub"
 	},
 	"dependencies": {
-		"@hybridly/utils": "0.10.0-beta.15",
-		"picoquery": "^2.5.0",
+		"@hybridly/utils": "0.10.0-beta.17",
+		"neoqs": "^6.13.0",
 		"superjson": "^2.2.2",
 		"es-toolkit": "^1.45.1"
 	},