chore: fix lint issues

Author: jerrythomas

packages/actions/src/utils.js

@@ -20,7 +20,7 @@ import * as R from 'ramda'
  * @returns {string|null} - The event name or null if no match is found.
  */
 export const getEventForKey = (keyMapping, key) => {
-	// eslint-disable-next-line no-unused-vars
+	 
 	const matchEvent = ([eventName, keys]) =>
 		(Array.isArray(keys) && keys.includes(key)) || (keys instanceof RegExp && keys.test(key))
 

packages/atoms/spec/ListItems.spec.svelte.js

@@ -1,6 +1,5 @@
-import { describe, expect, it, beforeEach, vi } from 'vitest'
-import { cleanup, render, fireEvent } from '@testing-library/svelte'
-import { tick } from 'svelte'
+import { describe, expect, it, beforeEach } from 'vitest'
+import { cleanup, render } from '@testing-library/svelte'
 import { MockItem } from '@rokkit/helpers/components'
 import ListItems from '../src/ListItems.svelte'
 import Item from '../src/Item.svelte'

packages/core/src/key-event-map.js

@@ -26,7 +26,7 @@ export class KeyEventMap {
 	 * @returns {string|null} - The event name or null if no match is found.
 	 */
 	getEventForKey(key) {
-		// eslint-disable-next-line no-unused-vars
+		 
 		const matchEvent = ([_, keys]) =>
 			(Array.isArray(keys) && keys.includes(key)) || (keys instanceof RegExp && keys.test(key))
 

packages/state/src/view.svelte.js

@@ -2,35 +2,35 @@ import { equals } from 'ramda'
 import { FieldMapper } from '@rokkit/core'
 import { getTree } from './view/tree'
 import {
-	getList,
-	collapse,
-	collapseAll,
-	expand,
-	expandAll,
-	moveByOffset,
-	moveTo,
-	select,
-	selectAll,
-	selectRange,
-	toggleExpansion,
-	toggleSelection,
-	unselect,
-	unselectAll,
-	invertSelection
+	getList
+	// collapse,
+	// collapseAll,
+	// expand,
+	// expandAll,
+	// moveByOffset,
+	// moveTo,
+	// select,
+	// selectAll,
+	// selectRange,
+	// toggleExpansion,
+	// toggleSelection,
+	// unselect,
+	// unselectAll,
+	// invertSelection
 } from './view/primitives'
 
-/**
- * Gets the current item from the view store.
- *
- * @param {import('svelte/store').Writable} view - the view store.
- * @returns {Object}                               The current item.
- */
-function getCurrentItem(view) {
-	const state = get(view)
-	const currentItem =
-		state.data.length > 0 && state.currentIndex >= 0 ? state.data[state.currentIndex] : null
-	return currentItem
-}
+// /**
+//  * Gets the current item from the view store.
+//  *
+//  * @param {import('svelte/store').Writable} view - the view store.
+//  * @returns {Object}                               The current item.
+//  */
+// function getCurrentItem(view) {
+// 	const state = get(view)
+// 	const currentItem =
+// 		state.data.length > 0 && state.currentIndex >= 0 ? state.data[state.currentIndex] : null
+// 	return currentItem
+// }
 
 /**
  * Validates the index.

packages/state/src/view/primitives.js

@@ -1,4 +1,24 @@
 import { equals, has } from 'ramda'
+
+/**
+ * Add an event to the event stack
+ * @param {Object} state
+ * @param {Object} event
+ * @param {Number} index
+ * @returns {Object} The updated state.
+ */
+export function addEvent(state, event, index = -1) {
+	// state.events = state.events ?? []
+
+	if (event === 'select') {
+		const detail = state.selectedItems.map((i) => state.data[i].value)
+		state.events.push({ type: event, detail })
+	} else if (event === 'move' || index > -1) {
+		const { value = null, indexPath = null } = index >= 0 ? state.data[index] : {}
+		state.events.push({ type: event, detail: { value, path: indexPath } })
+	}
+}
+
 /**
  * Creates a list view data store using the given items.
  *
@@ -191,6 +211,47 @@ export function selectRange(state, offset) {
 	return state
 }
 
+/**
+ * Recursively update visibility for all children of the given parent
+ * based on the parent's expanded state. Stops recursion if a child's expanded
+ * state does not match the parent's state.
+ *
+ * @param {Array} children - The children of the parent to update visibility for.
+ * @param {Boolean} isVisible - Indicates whether children should be visible or hidden.
+ */
+function updateChildVisibility(children, isVisible) {
+	children.forEach((child) => {
+		// Update visibility only if child's expanded state mismatches with the desired visibility
+		if (child.isExpanded === isVisible) {
+			child.isHidden = !isVisible // Update based on parent's state
+			if (child.children) {
+				updateChildVisibility(child.children, isVisible) // Recursive call for further children
+			}
+		}
+	})
+}
+
+/**
+ * Toggles the expansion of the item at the given index.
+ *
+ * @param {Object} state
+ * @param {Number} index
+ * @param {Boolean} expand - the state
+ * @param {Boolean} events - Whether to emit event
+ * @returns {Object} The updated state.
+ */
+function toggle(state, index, events = true) {
+	const item = state.data[index]
+	item.isExpanded = !item.isExpanded
+
+	updateChildVisibility(item.children, item.isExpanded)
+	if (events) {
+		const event = item.isExpanded ? 'expand' : 'collapse'
+		addEvent(state, event, index)
+	}
+	return state
+}
+
 /**
  * Expands the item at the given index.
  *
@@ -230,28 +291,6 @@ export function collapse(state, pathIndex, emit = true) {
 
 	return state
 }
-
-/**
- * Toggles the expansion of the item at the given index.
- *
- * @param {Object} state
- * @param {Number} index
- * @param {Boolean} expand - the state
- * @param {Boolean} events - Whether to emit event
- * @returns {Object} The updated state.
- */
-function toggle(state, index, events = true) {
-	const item = state.data[index]
-	item.isExpanded = !item.isExpanded
-
-	updateChildVisibility(item.children, item.isExpanded)
-	if (events) {
-		const event = item.isExpanded ? 'expand' : 'collapse'
-		addEvent(state, event, index)
-	}
-	return state
-}
-
 /**
  * Toggles the expansion of the item at the given index.
  *
@@ -294,42 +333,3 @@ export function collapseAll(state) {
 	addEvent(state, 'collapse')
 	return state
 }
-
-/**
- * Recursively update visibility for all children of the given parent
- * based on the parent's expanded state. Stops recursion if a child's expanded
- * state does not match the parent's state.
- *
- * @param {Array} children - The children of the parent to update visibility for.
- * @param {Boolean} isVisible - Indicates whether children should be visible or hidden.
- */
-function updateChildVisibility(children, isVisible) {
-	children.forEach((child) => {
-		// Update visibility only if child's expanded state mismatches with the desired visibility
-		if (child.isExpanded === isVisible) {
-			child.isHidden = !isVisible // Update based on parent's state
-			if (child.children) {
-				updateChildVisibility(child.children, isVisible) // Recursive call for further children
-			}
-		}
-	})
-}
-
-/**
- * Add an event to the event stack
- * @param {Object} state
- * @param {Object} event
- * @param {Number} index
- * @returns {Object} The updated state.
- */
-export function addEvent(state, event, index = -1) {
-	// state.events = state.events ?? []
-
-	if (event === 'select') {
-		const detail = state.selectedItems.map((i) => state.data[i].value)
-		state.events.push({ type: event, detail })
-	} else if (event === 'move' || index > -1) {
-		const { value = null, indexPath = null } = index >= 0 ? state.data[index] : {}
-		state.events.push({ type: event, detail: { value, path: indexPath } })
-	}
-}