@rokkit/states 1.0.0-next.149 → 1.0.0-next.151

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rokkit/states",
-  "version": "1.0.0-next.149",
+  "version": "1.0.0-next.151",
   "description": "Contains generic data manipulation functions that can be used in various components.",
   "repository": {
     "type": "git",

package/src/alerts.svelte.js

@@ -13,13 +13,17 @@ class AlertsStore {
 	 * @param {{ type?: string, text?: string, dismissible?: boolean, timeout?: number, actions?: unknown }} alert
 	 * @returns {string}
 	 */
+	#scheduleTimer(id, timeout) {
+		if (timeout > 0) {
+			this.#timers.set(id, setTimeout(() => this.dismiss(id), timeout))
+		}
+	}
+
+	// eslint-disable-next-line complexity
 	push({ type = 'info', text, dismissible = false, timeout = dismissible ? 0 : 4000, actions } = {}) {
 		const id = crypto.randomUUID()
 		this.#items = [...this.#items, { id, type, text, dismissible, timeout, actions }]
-		if (timeout > 0) {
-			const timer = setTimeout(() => this.dismiss(id), timeout)
-			this.#timers.set(id, timer)
-		}
+		this.#scheduleTimer(id, timeout)
 		return id
 	}
 

package/src/proxy-item.svelte.js

@@ -92,8 +92,12 @@ export class ProxyItem {
 	 * @param {string} [key]    Path-based key assigned by ProxyTree
 	 * @param {number} [level]  Nesting depth (1 = root)
 	 */
+	#mergeFields(fields) {
+		return { ...BASE_FIELDS, ...(fields && typeof fields === 'object' ? fields : {}) }
+	}
+
 	constructor(raw, fields = {}, key = '', level = 0) {
-		this.#fields = { ...BASE_FIELDS, ...(fields && typeof fields === 'object' ? fields : {}) }
+		this.#fields = this.#mergeFields(fields)
 		this.#raw = raw
 		this.#key = key
 		this.#level = level

package/src/proxy-tree.svelte.js

@@ -136,8 +136,12 @@ export class ProxyTree {
 	 * @param {Partial<typeof BASE_FIELDS>} [fields]
 	 * @param {{ createProxy?: (raw: *, fields: object, key: string, level: number) => ProxyItem }} [options]
 	 */
+	#mergeFields(fields) {
+		return { ...BASE_FIELDS, ...(fields && typeof fields === 'object' ? fields : {}) }
+	}
+
 	constructor(items = [], fields = {}, options = {}) {
-		this.#fields = { ...BASE_FIELDS, ...(fields && typeof fields === 'object' ? fields : {}) }
+		this.#fields = this.#mergeFields(fields)
 		this.#factory = this.#resolveFactory(options.createProxy)
 		this.#rootProxies = (items ?? []).map((raw, i) =>
 			this.#factory(raw, this.#fields, String(i), 1)

package/src/wrapper.svelte.js

@@ -125,11 +125,13 @@ export class Wrapper {
 	 * At root level with no parent: no-op.
 	 * When collapsible=false, skips closing the group but still moves focus to parent.
 	 */
+	// eslint-disable-next-line complexity
 	collapse(_path) {
 		if (!this.#focusedKey) return
 		const node = this.flatView.find((n) => n.key === this.#focusedKey)
 		if (!node) return
-		if (node.hasChildren && node.proxy.expanded && this.#collapsible) {
+		const canCollapse = node.hasChildren && node.proxy.expanded && this.#collapsible
+		if (canCollapse) {
 			node.proxy.expanded = false
 		} else {
 			this.#focusParent()
@@ -154,12 +156,7 @@ export class Wrapper {
 	 * Select item at path (or focusedKey when path is null).
 	 * Groups toggle expanded (only when collapsible=true). Leaves fire onchange and onselect callbacks.
 	 */
-	select(path) {
-		const key = path ?? this.#focusedKey
-		if (!key) return
-		this.#focusedKey = key
-		const proxy = this.#proxyTree.lookup.get(key)
-		if (!proxy) return
+	#selectProxy(proxy) {
 		if (proxy.hasChildren) {
 			if (this.#collapsible) proxy.expanded = !proxy.expanded
 		} else {
@@ -167,17 +164,27 @@ export class Wrapper {
 		}
 	}
 
+	select(path) {
+		const key = path ?? this.#focusedKey
+		if (!key) return
+		this.#focusedKey = key
+		const proxy = this.#proxyTree.lookup.get(key)
+		if (proxy) this.#selectProxy(proxy)
+	}
+
 	/**
 	 * Toggle expansion of group at path — called by Navigator for accordion-trigger clicks.
 	 * Unlike select(), this only applies to groups and never fires onselect.
 	 * No-op when collapsible=false.
 	 */
+	// eslint-disable-next-line complexity
 	toggle(path) {
 		if (!this.#collapsible) return
 		const key = path ?? this.#focusedKey
-		if (!key) return
-		const proxy = this.#proxyTree.lookup.get(key)
-		if (proxy?.hasChildren) proxy.expanded = !proxy.expanded
+		if (key) {
+			const proxy = this.#proxyTree.lookup.get(key)
+			if (proxy?.hasChildren) proxy.expanded = !proxy.expanded
+		}
 	}
 
 	/**