@oscarpalmer/abydon 0.15.0 → 0.16.0

package/src/helpers/dom.ts

@@ -1,6 +1,5 @@
 import {getString} from '@oscarpalmer/atoms/string';
-import {isChildNode, isHTMLOrSVGElement} from '@oscarpalmer/toretto/is';
-import {removeEvents} from '../node/event';
+import {isChildNode} from '@oscarpalmer/toretto/is';
 import {isFragment} from './index';
 
 export function createNodes(value: unknown): ChildNode[] {
@@ -16,8 +15,6 @@ export function createNodes(value: unknown): ChildNode[] {
 }
 
 export function removeNodes(nodes: ChildNode[]): void {
-	sanitizeNodes(nodes);
-
 	const {length} = nodes;
 
 	for (let index = 0; index < length; index += 1) {
@@ -34,19 +31,3 @@ export function replaceNodes(from: ChildNode[], to: ChildNode[]): void {
 		from[index].remove();
 	}
 }
-
-function sanitizeNodes(nodes: ChildNode[]): void {
-	const {length} = nodes;
-
-	for (let index = 0; index < length; index += 1) {
-		const node = nodes[index];
-
-		if (isHTMLOrSVGElement(node)) {
-			removeEvents(node);
-		}
-
-		if (node.hasChildNodes()) {
-			sanitizeNodes([...node.childNodes] as ChildNode[]);
-		}
-	}
-}

package/src/helpers/index.ts

@@ -1,3 +1,5 @@
+import type {PlainObject} from '@oscarpalmer/atoms/models';
+import {NAME_FRAGMENT, NAME_FRAGMENTS} from '../constants';
 import type {Fragment} from '../fragment';
 import type {Fragments} from '../fragments';
 
@@ -9,31 +11,32 @@ export function compareArrays(
 	const from = firstIsLarger ? first : second;
 	const to = firstIsLarger ? second : first;
 
-	if (
-		!from
-			.filter(key => to.includes(key))
-			.every((key, index) => to[index] === key)
-	) {
-		return 'dissimilar';
+	if (!from.filter(key => to.includes(key)).every((key, index) => to[index] === key)) {
+		return COMPARISON_DISSIMILAR;
 	}
 
-	return firstIsLarger ? 'removed' : 'added';
+	return firstIsLarger ? COMPARISON_REMOVED : COMPARISON_ADDED;
 }
 
 export function isFragment(value: unknown): value is Fragment {
-	return (
-		typeof value === 'object' &&
-		value != null &&
-		'$fragment' in value &&
-		value.$fragment === true
-	);
+	return isNamed(value, NAME_FRAGMENT);
 }
 
 export function isFragments(value: unknown): value is Fragments {
+	return isNamed(value, NAME_FRAGMENTS);
+}
+
+function isNamed(value: unknown, name: string): boolean {
 	return (
 		typeof value === 'object' &&
 		value != null &&
-		'$fragments' in value &&
-		value.$fragments === true
+		name in value &&
+		(value as PlainObject)[name] === true
 	);
 }
+
+const COMPARISON_ADDED = 'added';
+
+const COMPARISON_DISSIMILAR = 'dissimilar';
+
+const COMPARISON_REMOVED = 'removed';

package/src/index.ts

@@ -3,23 +3,27 @@ import type {ReactiveArray} from '@oscarpalmer/mora';
 import {Fragment} from './fragment';
 import {Fragments} from './fragments';
 
+/**
+ * Create Fragments from a reactive array
+ * @param array Reactive array
+ * @param identify Function to identify item
+ * @param fragment Function to create fragment from item
+ * @returns Fragments
+ */
 export function fragments<Item>(
 	array: ReactiveArray<Item>,
 	identify: (item: Item) => unknown,
 	fragment: (item: Item) => Fragment,
 ): Fragments {
-	return new Fragments(
-		array as ReactiveArray<unknown>,
-		identify as never,
-		fragment as never,
-	);
+	return new Fragments(array as ReactiveArray<unknown>, identify as never, fragment as never);
 }
 
-export function html(
-	strings: TemplateStringsArray,
-	...values: unknown[]
-): unknown {
-	return new Fragment(strings, values);
+/**
+ * Create Fragment from a template
+ * @returns Fragment
+ */
+export function html(template: TemplateStringsArray, ...values: unknown[]): Fragment {
+	return new Fragment(template, values);
 }
 
 export * from '@oscarpalmer/mora';

package/src/models.ts

@@ -2,8 +2,17 @@ import type {Reactive, ReactiveArray, Unsubscribe} from '@oscarpalmer/mora';
 import type {Fragment} from './fragment';
 
 export type FragmentConfiguration = {
+	/**
+	 * Should the template be cached? _(defaults to `true`)_
+	 */
+	cache?: boolean;
+	/**
+	 * Identifier for the fragment
+	 *
+	 * _(An identifier can be used to uniquely identify a fragment,
+	 * which helps prevent re-rendering in certain scenarios)_
+	 */
 	identifier?: unknown;
-	ignoreCache?: boolean;
 };
 
 export type FragmentData = {

package/src/node/attribute/index.ts

@@ -1,43 +1,44 @@
 import type {GenericCallback} from '@oscarpalmer/atoms/models';
 import {computed, isReactive} from '@oscarpalmer/mora';
-import {isBooleanAttribute, setProperty} from '@oscarpalmer/toretto/attribute';
-import type {HTMLOrSVGElement} from '@oscarpalmer/toretto/models';
-import {EXPRESSION_COMMENT_FULL} from '../../constants';
+import {
+	EXPRESSION_ABYDON_ATTRIBUTE_PREFIX,
+	EXPRESSION_ABYDON_CONTENT,
+	EXPRESSION_EVENT_PREFIX,
+	EXPRESSION_PERIOD,
+} from '../../constants';
 import type {FragmentData} from '../../models';
 import {mapEvent} from '../event';
 import {setAttribute} from './value';
 
 function getValue(data: FragmentData, original: string): unknown {
-	const matches = EXPRESSION_COMMENT_FULL.exec(original ?? '');
+	const matches = EXPRESSION_ABYDON_CONTENT.exec(original ?? '');
 
 	return matches == null ? original : data.values[+matches[1]];
 }
 
-export function mapAttributes(
-	data: FragmentData,
-	element: HTMLOrSVGElement,
-): void {
+export function mapAttributes(data: FragmentData, element: HTMLElement | SVGElement): void {
 	const attributes = [...element.attributes];
 	const {length} = attributes;
 
 	for (let index = 0; index < length; index += 1) {
 		const {name, value} = attributes[index];
 
-		const actual = getValue(data, value);
+		const actualName = name.replace(EXPRESSION_ABYDON_ATTRIBUTE_PREFIX, '');
+		const actualValue = getValue(data, value) as never;
 
-		switch (true) {
-			case name.startsWith('@'):
-				mapEvent(element, name, actual);
-				break;
+		if (actualName !== name) {
+			element.removeAttribute(name);
+		}
 
-			case name.includes('.') ||
-				typeof actual === 'function' ||
-				isReactive(actual):
-				mapValue(data, element, name, actual);
+		switch (true) {
+			case EXPRESSION_EVENT_PREFIX.test(actualName):
+				mapEvent(element, actualName, actualValue);
 				break;
 
-			case isBooleanAttribute(name):
-				setProperty(element, name, value);
+			case EXPRESSION_PERIOD.test(actualName):
+			case typeof actualValue === 'function':
+			case isReactive(actualValue):
+				mapValue(data, element, actualName, actualValue);
 				break;
 
 			default:
@@ -48,7 +49,7 @@ export function mapAttributes(
 
 function mapValue(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
 ): void {
@@ -61,7 +62,7 @@ function mapValue(
 
 function setComputedAttribute(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	callback: GenericCallback,
 ): void {

package/src/node/attribute/value.ts

@@ -1,11 +1,10 @@
 import type {PlainObject} from '@oscarpalmer/atoms/models';
-import {getString} from '@oscarpalmer/atoms/string';
 import {isReactive} from '@oscarpalmer/mora';
 import {
 	isBooleanAttribute,
 	setAttribute as setAttr,
+	setProperty,
 } from '@oscarpalmer/toretto/attribute';
-import type {HTMLOrSVGElement} from '@oscarpalmer/toretto/models';
 import {
 	ATTRIBUTE_CLASS_PREFIX_LENGTH,
 	EXPRESSION_ATTRIBUTE_CLASS,
@@ -14,14 +13,20 @@ import {
 } from '../../constants';
 import type {FragmentData} from '../../models';
 
+function getCallback(element: HTMLElement | SVGElement, name: string) {
+	if (isBooleanAttribute(name) && name in element) {
+		return name === 'checked' ? updateChecked : updateProperty;
+	}
+
+	return name === 'value' ? updateValue : setAttr;
+}
+
 export function setAttribute(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
 ): void {
-	element.removeAttribute(name);
-
 	switch (true) {
 		case EXPRESSION_ATTRIBUTE_CLASS.test(name):
 			setClasses(data, element, name, value);
@@ -39,7 +44,7 @@ export function setAttribute(
 
 function setClasses(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
 ): void {
@@ -62,7 +67,7 @@ function setClasses(
 
 function setStyle(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
 ): void {
@@ -76,10 +81,7 @@ function setStyle(
 		if (value == null || value === false || (value === true && unit == null)) {
 			element.style.removeProperty(property);
 		} else {
-			element.style.setProperty(
-				property,
-				value === true ? unit : getString(value),
-			);
+			element.style.setProperty(property, value === true ? unit : String(value));
 		}
 	}
 
@@ -92,21 +94,11 @@ function setStyle(
 
 function setValue(
 	data: FragmentData,
-	element: HTMLOrSVGElement,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
 ): void {
-	let callback: (
-		element: HTMLOrSVGElement,
-		name: string,
-		value: unknown,
-	) => void;
-
-	if (isBooleanAttribute(name) && name in element) {
-		callback = name === 'selected' ? updateSelected : updateProperty;
-	} else {
-		callback = setAttr;
-	}
+	const callback = getCallback(element, name);
 
 	if (isReactive(value)) {
 		data.mora.subscribers.add(
@@ -119,25 +111,40 @@ function setValue(
 	}
 }
 
-function updateProperty(
-	element: HTMLOrSVGElement,
-	name: string,
-	value: unknown,
-): void {
-	(element as unknown as PlainObject)[name] = value === true;
+function updateChecked(element: HTMLElement | SVGElement, name: string, value: unknown): void {
+	updateElement('change', 'checked', element, name, value, value === true);
 }
 
-function updateSelected(
-	element: HTMLOrSVGElement,
+function updateElement(
+	event: string,
+	property: string,
+	element: HTMLElement | SVGElement,
 	name: string,
 	value: unknown,
+	next: unknown,
 ): void {
-	const select = element.closest('select') as HTMLSelectElement;
-	const options = [...(select?.options ?? [])];
-
-	if (select != null && options.includes(element as HTMLOptionElement)) {
-		select.dispatchEvent(new Event('change', {bubbles: true}));
+	if (!(property in element) || (element as unknown as PlainObject)[property] === next) {
+		return;
 	}
 
-	updateProperty(element, name, value);
+	setAttr(element, name, value);
+
+	(element as unknown as PlainObject)[property] = next;
+
+	element.dispatchEvent(new Event(event, {bubbles: true}));
+}
+
+function updateProperty(element: HTMLElement | SVGElement, name: string, value: unknown): void {
+	setProperty(element, name, value === true);
+}
+
+function updateValue(element: HTMLElement | SVGElement, name: string, value: unknown): void {
+	updateElement(
+		element instanceof HTMLSelectElement ? 'change' : 'input',
+		'value',
+		element,
+		name,
+		value,
+		String(value),
+	);
 }

package/src/node/event.ts

@@ -1,50 +1,43 @@
-import type {HTMLOrSVGElement} from '@oscarpalmer/toretto/models';
+import {on} from '@oscarpalmer/toretto/event';
 import {
-	ABORT_CONTROLLERS,
+	EVENT_DEFAULTS,
+	EXPRESSION_EVENT_CHANGE_TYPES,
 	EXPRESSION_EVENT_NAME,
-	REASON_EVENT_REMOVED,
+	EXPRESSION_EVENT_OPTIONS_ACTIVE,
+	EXPRESSION_EVENT_OPTIONS_CAPTURE,
+	EXPRESSION_EVENT_OPTIONS_ONCE,
 } from '../constants';
 
-function getController(element: HTMLOrSVGElement): AbortController {
-	let controller = ABORT_CONTROLLERS.get(element);
-
-	if (controller == null) {
-		controller = new AbortController();
-
-		ABORT_CONTROLLERS.set(element, controller);
-	}
-
-	return controller;
-}
-
 function getOptions(options: string): AddEventListenerOptions {
 	const parts = options.split(':');
 
 	return {
-		capture: parts.includes('c') || parts.includes('capture'),
-		once: parts.includes('o') || parts.includes('once'),
-		passive: !(parts.includes('a') || parts.includes('active')),
+		capture: parts.some(part => EXPRESSION_EVENT_OPTIONS_CAPTURE.test(part)),
+		once: parts.some(part => EXPRESSION_EVENT_OPTIONS_ONCE.test(part)),
+		passive: !parts.some(part => EXPRESSION_EVENT_OPTIONS_ACTIVE.test(part)),
 	};
 }
 
-export function mapEvent(
-	element: HTMLOrSVGElement,
-	name: string,
-	value: unknown,
-): void {
-	element.removeAttribute(name);
+function getType(element: HTMLElement | SVGElement, type: string): string {
+	if (type !== 'on') {
+		return type;
+	}
 
-	const [, type, options] = EXPRESSION_EVENT_NAME.exec(name) ?? [];
+	if (element instanceof HTMLInputElement) {
+		if (EXPRESSION_EVENT_CHANGE_TYPES.test(element.type)) {
+			return 'change';
+		}
 
-	if (typeof value === 'function' && type != null) {
-		element.addEventListener(type, value as EventListener, {
-			...getOptions(options ?? ''),
-			signal: getController(element).signal,
-		});
+		return element.type === 'submit' ? 'submit' : 'input';
 	}
+
+	return EVENT_DEFAULTS[element.tagName] ?? type;
 }
 
-export function removeEvents(element: HTMLOrSVGElement): void {
-	ABORT_CONTROLLERS.get(element)?.abort(REASON_EVENT_REMOVED);
-	ABORT_CONTROLLERS.delete(element);
+export function mapEvent(element: HTMLElement | SVGElement, name: string, value: unknown): void {
+	const [, type, options] = EXPRESSION_EVENT_NAME.exec(name) ?? [];
+
+	if (type != null && typeof value === 'function') {
+		on(element, getType(element, type), value as EventListener, getOptions(options ?? ''));
+	}
 }

package/src/node/index.ts

@@ -1,7 +1,7 @@
 import type {GenericCallback} from '@oscarpalmer/atoms/models';
-import {computed, isReactive, Reactive} from '@oscarpalmer/mora';
+import {computed, isReactive, type Reactive} from '@oscarpalmer/mora';
 import {isHTMLOrSVGElement} from '@oscarpalmer/toretto/is';
-import {EXPRESSION_COMMENT_CONTENT} from '../constants';
+import {EXPRESSION_ABYDON_CONTENT} from '../constants';
 import {handleFragments} from '../fragments';
 import {isFragment, isFragments} from '../helpers';
 import {createNodes} from '../helpers/dom';
@@ -10,7 +10,7 @@ import {mapAttributes} from './attribute';
 import {setReactiveValue} from './value';
 
 function mapNode(data: FragmentData, comment: Comment): void {
-	const matches = EXPRESSION_COMMENT_CONTENT.exec(comment.textContent ?? '');
+	const matches = EXPRESSION_ABYDON_CONTENT.exec(comment.textContent ?? '');
 	const value = matches == null ? null : data.values[+matches[1]];
 
 	if (value != null) {
@@ -48,11 +48,7 @@ function mapValue(data: FragmentData, comment: Comment, value: unknown): void {
 
 		case isFragments(value):
 			handleFragments(value, false);
-			setReactiveValue(
-				data,
-				comment,
-				value.items as Reactive<unknown[], unknown>,
-			);
+			setReactiveValue(data, comment, value.items as Reactive<unknown[], unknown>);
 			break;
 
 		case isReactive(value):
@@ -65,11 +61,7 @@ function mapValue(data: FragmentData, comment: Comment, value: unknown): void {
 	}
 }
 
-function replaceComment(
-	data: FragmentData,
-	comment: Comment,
-	value: unknown,
-): void {
+function replaceComment(data: FragmentData, comment: Comment, value: unknown): void {
 	const item = data.items.find(item => item.nodes?.includes(comment));
 	const nodes = createNodes(value);
 
@@ -81,11 +73,7 @@ function replaceComment(
 	comment.replaceWith(...nodes);
 }
 
-function setComputedValue(
-	data: FragmentData,
-	comment: Comment,
-	callback: GenericCallback,
-): void {
+function setComputedValue(data: FragmentData, comment: Comment, callback: GenericCallback): void {
 	const value = computed(callback);
 
 	data.mora.values.add(value);

package/src/node/value.ts

@@ -10,8 +10,8 @@ import type {FragmentData, FragmentItem} from '../models';
 //
 
 type Identifiers = {
-	next: unknown[];
-	previous: (unknown | undefined)[];
+	next: Set<unknown>;
+	previous: Set<unknown>;
 };
 
 type BaseItems = {
@@ -33,8 +33,7 @@ function addToArray(
 ): void {
 	let position = nodes[0];
 
-	const before =
-		added && !identifiers.previous.includes(items.templates[0].identifier);
+	const before = added && !identifiers.previous.has(items.templates[0].identifier);
 
 	const next = items.next.flatMap(fragment =>
 		fragment.get().flatMap(node => ({
@@ -48,7 +47,7 @@ function addToArray(
 	for (let index = 0; index < length; index += 1) {
 		const node = next[index];
 
-		if (!(added && identifiers.previous.includes(node.identifier))) {
+		if (!(added && identifiers.previous.has(node.identifier))) {
 			if (index === 0 && before) {
 				position.before(node.value);
 			} else {
@@ -67,9 +66,7 @@ function handleArray(
 ): Partial<FragmentItem> {
 	const next = items.templates.map(
 		template =>
-			items.fragments?.find(
-				fragment => fragment.identifier === template.identifier,
-			) ?? template,
+			items.fragments?.find(fragment => fragment.identifier === template.identifier) ?? template,
 	);
 
 	const comparison = compareArrays(items.fragments ?? [], items.templates);
@@ -79,9 +76,7 @@ function handleArray(
 	}
 
 	const toRemove =
-		items.fragments?.filter(
-			fragment => !identifiers.next.includes(fragment.identifier),
-		) ?? [];
+		items.fragments?.filter(fragment => !identifiers.next.has(fragment.identifier)) ?? [];
 
 	const {length} = toRemove;
 
@@ -105,11 +100,7 @@ function removeFragments(fragments: Fragment[] | undefined): void {
 	}
 }
 
-function replaceText(
-	item: FragmentItem,
-	comment: Comment,
-	isNullable: boolean,
-): void {
+function replaceText(item: FragmentItem, comment: Comment, isNullable: boolean): void {
 	let to: ChildNode[];
 
 	if (isNullable) {
@@ -132,9 +123,7 @@ function setArray(
 		};
 	}
 
-	let templates = value.filter(
-		item => isFragment(item) && item.identifier != null,
-	) as Fragment[];
+	let templates = value.filter(item => isFragment(item) && item.identifier != null) as Fragment[];
 
 	const next = templates.map(fragment => fragment.identifier) as unknown[];
 	const previous = item.fragments?.map(fragment => fragment.identifier) ?? [];
@@ -145,11 +134,7 @@ function setArray(
 
 	const noTemplates = templates.length === 0;
 
-	if (
-		noTemplates ||
-		item.nodes == null ||
-		previous.some(identifier => identifier == null)
-	) {
+	if (noTemplates || item.nodes == null || previous.some(identifier => identifier == null)) {
 		return {
 			fragments: noTemplates ? undefined : templates,
 			nodes: setNodes(
@@ -164,8 +149,8 @@ function setArray(
 
 	return handleArray(
 		{
-			next,
-			previous,
+			next: new Set(next),
+			previous: new Set(previous),
 		},
 		{
 			templates,
@@ -175,11 +160,7 @@ function setArray(
 	);
 }
 
-function setNodes(
-	item: FragmentItem,
-	comment: Comment,
-	next: ChildNode[],
-): ChildNode[] {
+function setNodes(item: FragmentItem, comment: Comment, next: ChildNode[]): ChildNode[] {
 	if (item.nodes == null) {
 		if (comment.parentNode != null) {
 			replaceNodes([comment], next);
@@ -219,11 +200,7 @@ export function setReactiveValue(
 	);
 }
 
-function setReactiveValueForArray(
-	item: FragmentItem,
-	comment: Comment,
-	value: unknown[],
-): void {
+function setReactiveValueForArray(item: FragmentItem, comment: Comment, value: unknown[]): void {
 	const result = setArray(item, comment, value);
 
 	item.fragments = typeof result === 'boolean' ? undefined : result?.fragments;
@@ -239,11 +216,7 @@ function setReactiveValueForArray(
 	}
 }
 
-function setReactiveValueForSingle(
-	item: FragmentItem,
-	comment: Comment,
-	value: unknown,
-): void {
+function setReactiveValueForSingle(item: FragmentItem, comment: Comment, value: unknown): void {
 	const valueIsFragment = isFragment(value);
 
 	item.fragments = valueIsFragment ? [value] : undefined;
@@ -255,11 +228,7 @@ function setReactiveValueForSingle(
 	}
 }
 
-function setText(
-	item: FragmentItem,
-	comment: Comment,
-	value: unknown,
-): ChildNode[] | undefined {
+function setText(item: FragmentItem, comment: Comment, value: unknown): ChildNode[] | undefined {
 	const isNullable = isNullableOrWhitespace(value);
 
 	if (item.text != null) {