ripple 0.2.139 → 0.2.141

package/src/runtime/set.js

@@ -15,183 +15,183 @@ let init = false;
  * @returns {TrackedSet<T>}
  */
 export class TrackedSet extends Set {
-  /** @type {Tracked} */
-  #tracked_size;
-  /** @type {Map<T, Tracked>} */
-  #tracked_items = new Map();
-  /** @type {Block} */
-  #block;
-
-  /**
-   * @param {Iterable<T>} [iterable]
-   */
-  constructor(iterable) {
-    super();
-
-    var block = this.#block = safe_scope();
-
-    if (iterable) {
-      for (var item of iterable) {
-        super.add(item);
-        this.#tracked_items.set(item, tracked(0, block));
-      }
-    }
-
-    this.#tracked_size = tracked(super.size, block);
-
-    if (!init) {
-      init = true;
-      this.#init();
-    }
-  }
-
-  /**
-   * @returns {void}
-   */
-  #init() {
-    var proto = TrackedSet.prototype;
-    var set_proto = Set.prototype;
-
-    for (const method of introspect_methods) {
-      if (!(method in set_proto)) {
-        continue;
-      }
+	/** @type {Tracked} */
+	#tracked_size;
+	/** @type {Map<T, Tracked>} */
+	#tracked_items = new Map();
+	/** @type {Block} */
+	#block;
+
+	/**
+	 * @param {Iterable<T>} [iterable]
+	 */
+	constructor(iterable) {
+		super();
+
+		var block = this.#block = safe_scope();
+
+		if (iterable) {
+			for (var item of iterable) {
+				super.add(item);
+				this.#tracked_items.set(item, tracked(0, block));
+			}
+		}
+
+		this.#tracked_size = tracked(super.size, block);
+
+		if (!init) {
+			init = true;
+			this.#init();
+		}
+	}
+
+	/**
+	 * @returns {void}
+	 */
+	#init() {
+		var proto = TrackedSet.prototype;
+		var set_proto = Set.prototype;
+
+		for (const method of introspect_methods) {
+			if (!(method in set_proto)) {
+				continue;
+			}
 
       /** @type {any} */ (proto)[method] = function (/** @type {...any} */ ...v) {
-        this.size;
+				this.size;
 
-        return /** @type {any} */ (set_proto)[method].apply(this, v);
-      };
-    }
+				return /** @type {any} */ (set_proto)[method].apply(this, v);
+			};
+		}
 
-    for (const method of compare_other_methods) {
-      if (!(method in set_proto)) {
-        continue;
-      }
+		for (const method of compare_other_methods) {
+			if (!(method in set_proto)) {
+				continue;
+			}
 
       /** @type {any} */ (proto)[method] = function (/** @type {any} */ other, /** @type {...any} */ ...v) {
-        this.size;
+				this.size;
 
-        if (other instanceof TrackedSet) {
-          other.size;
-        }
+				if (other instanceof TrackedSet) {
+					other.size;
+				}
 
-        return /** @type {any} */ (set_proto)[method].apply(this, [other, ...v]);
-      };
-    }
+				return /** @type {any} */ (set_proto)[method].apply(this, [other, ...v]);
+			};
+		}
 
-    for (const method of new_other_methods) {
-      if (!(method in set_proto)) {
-        continue;
-      }
+		for (const method of new_other_methods) {
+			if (!(method in set_proto)) {
+				continue;
+			}
 
       /** @type {any} */ (proto)[method] = function (/** @type {any} */ other, /** @type {...any} */ ...v) {
-        this.size;
-
-        if (other instanceof TrackedSet) {
-          other.size;
-        }
-
-        return new TrackedSet(/** @type {any} */ (set_proto)[method].apply(this, [other, ...v]));
-      };
-    }
-  }
-
-  /**
-   * @param {T} value
-   * @returns {this}
-   */
-  add(value) {
-    var block = this.#block;
-
-    if (!super.has(value)) {
-      super.add(value);
-      this.#tracked_items.set(value, tracked(0, block));
-      set(this.#tracked_size, super.size, block);
-    }
-
-    return this;
-  }
-
-  /**
-   * @param {T} value
-   * @returns {boolean}
-   */
-  delete(value) {
-    var block = this.#block;
-
-    if (!super.delete(value)) {
-      return false;
-    }
-
-    var t = this.#tracked_items.get(value);
-
-    if (t) {
-      increment(t, block);
-    }
-    this.#tracked_items.delete(value);
-    set(this.#tracked_size, super.size, block);
-
-    return true;
-  }
-
-  /**
-   * @param {T} value
-   * @return {boolean}
-  */
-  has(value) {
-
-    var has = super.has(value);
-    var tracked_items = this.#tracked_items;
-    var t = tracked_items.get(value);
-
-    if (t === undefined) {
-      // if no tracked it also means super didn't have it
-      // It's not possible to have a disconnect, we track each value
-      // If the value doesn't exist, track the size in case it's added later
-      // but don't create tracked entries willy-nilly to track all possible values
-      this.size;
-    } else {
-      get(t);
-    }
-
-    return has;
-  }
-
-  /**
-   * @returns {void}
-   */
-  clear() {
-    var block = this.#block;
-
-    if (super.size === 0) {
-      return;
-    }
-
-    for (var [_, t] of this.#tracked_items) {
-      increment(t, block);
-    }
-
-    super.clear();
-    this.#tracked_items.clear();
-    set(this.#tracked_size, 0, block);
-  }
-
-  /**
-   * @returns {number}
-   */
-  get size() {
-    return get(this.#tracked_size);
-  }
-
-  /**
-   * @returns {T[]}
-   */
-  toJSON() {
-    this.size;
-
-    return [...this];
-  }
+				this.size;
+
+				if (other instanceof TrackedSet) {
+					other.size;
+				}
+
+				return new TrackedSet(/** @type {any} */(set_proto)[method].apply(this, [other, ...v]));
+			};
+		}
+	}
+
+	/**
+	 * @param {T} value
+	 * @returns {this}
+	 */
+	add(value) {
+		var block = this.#block;
+
+		if (!super.has(value)) {
+			super.add(value);
+			this.#tracked_items.set(value, tracked(0, block));
+			set(this.#tracked_size, super.size);
+		}
+
+		return this;
+	}
+
+	/**
+	 * @param {T} value
+	 * @returns {boolean}
+	 */
+	delete(value) {
+		var block = this.#block;
+
+		if (!super.delete(value)) {
+			return false;
+		}
+
+		var t = this.#tracked_items.get(value);
+
+		if (t) {
+			increment(t);
+		}
+		this.#tracked_items.delete(value);
+		set(this.#tracked_size, super.size);
+
+		return true;
+	}
+
+	/**
+	 * @param {T} value
+	 * @return {boolean}
+	*/
+	has(value) {
+
+		var has = super.has(value);
+		var tracked_items = this.#tracked_items;
+		var t = tracked_items.get(value);
+
+		if (t === undefined) {
+			// if no tracked it also means super didn't have it
+			// It's not possible to have a disconnect, we track each value
+			// If the value doesn't exist, track the size in case it's added later
+			// but don't create tracked entries willy-nilly to track all possible values
+			this.size;
+		} else {
+			get(t);
+		}
+
+		return has;
+	}
+
+	/**
+	 * @returns {void}
+	 */
+	clear() {
+		var block = this.#block;
+
+		if (super.size === 0) {
+			return;
+		}
+
+		for (var [_, t] of this.#tracked_items) {
+			increment(t);
+		}
+
+		super.clear();
+		this.#tracked_items.clear();
+		set(this.#tracked_size, 0);
+	}
+
+	/**
+	 * @returns {number}
+	 */
+	get size() {
+		return get(this.#tracked_size);
+	}
+
+	/**
+	 * @returns {T[]}
+	 */
+	toJSON() {
+		this.size;
+
+		return [...this];
+	}
 }
 
 /**
@@ -201,5 +201,5 @@ export class TrackedSet extends Set {
  * @returns {TrackedSet<V>}
  */
 export function tracked_set(block, ...args) {
-  return with_scope(block, () => new TrackedSet(...args));
-}
+	return with_scope(block, () => new TrackedSet(...args));
+}

package/src/runtime/url-search-params.js

@@ -37,7 +37,7 @@ export class TrackedURLSearchParams extends URLSearchParams {
 			super.append(key, value);
 		}
 
-		increment(this.#version, this.#block);
+		increment(this.#version);
 		this.#updating = false;
 	}
 
@@ -49,7 +49,7 @@ export class TrackedURLSearchParams extends URLSearchParams {
 	append(name, value) {
 		super.append(name, value);
 		this.#update_url();
-		increment(this.#version, this.#block);
+		increment(this.#version);
 	}
 
 	/**
@@ -62,7 +62,7 @@ export class TrackedURLSearchParams extends URLSearchParams {
 		super.delete(name, value);
 		if (has_value) {
 			this.#update_url();
-			increment(this.#version, this.#block);
+			increment(this.#version);
 		}
 	}
 
@@ -111,14 +111,14 @@ export class TrackedURLSearchParams extends URLSearchParams {
 		// if you set `foo` to 1, then foo=3 gets deleted whilst `has("foo", "1")` returns true
 		if (previous !== super.getAll(name).join('')) {
 			this.#update_url();
-			increment(this.#version, this.#block);
+			increment(this.#version);
 		}
 	}
 
 	sort() {
 		super.sort();
 		this.#update_url();
-		increment(this.#version, this.#block);
+		increment(this.#version);
 	}
 
 	toString() {

package/src/runtime/url.js

@@ -39,7 +39,7 @@ export class TrackedURL extends URL {
 
 	set hash(value) {
 		super.hash = value;
-		set(this.#hash, super.hash, this.#block);
+		set(this.#hash, super.hash);
 	}
 
 	get host() {
@@ -50,8 +50,8 @@ export class TrackedURL extends URL {
 
 	set host(value) {
 		super.host = value;
-		set(this.#hostname, super.hostname, this.#block);
-		set(this.#port, super.port, this.#block);
+		set(this.#hostname, super.hostname);
+		set(this.#port, super.port);
 	}
 
 	get hostname() {
@@ -60,7 +60,7 @@ export class TrackedURL extends URL {
 
 	set hostname(value) {
 		super.hostname = value;
-		set(this.#hostname, super.hostname, this.#block);
+		set(this.#hostname, super.hostname);
 	}
 
 	get href() {
@@ -77,14 +77,14 @@ export class TrackedURL extends URL {
 
 	set href(value) {
 		super.href = value;
-		set(this.#protocol, super.protocol, this.#block);
-		set(this.#username, super.username, this.#block);
-		set(this.#password, super.password, this.#block);
-		set(this.#hostname, super.hostname, this.#block);
-		set(this.#port, super.port, this.#block);
-		set(this.#pathname, super.pathname, this.#block);
-		set(this.#hash, super.hash, this.#block);
-		set(this.#search, super.search, this.#block);
+		set(this.#protocol, super.protocol);
+		set(this.#username, super.username);
+		set(this.#password, super.password);
+		set(this.#hostname, super.hostname);
+		set(this.#port, super.port);
+		set(this.#pathname, super.pathname);
+		set(this.#hash, super.hash);
+		set(this.#search, super.search);
 		this.#searchParams[REPLACE](super.searchParams);
 	}
 
@@ -94,7 +94,7 @@ export class TrackedURL extends URL {
 
 	set password(value) {
 		super.password = value;
-		set(this.#password, super.password, this.#block);
+		set(this.#password, super.password);
 	}
 
 	get pathname() {
@@ -103,7 +103,7 @@ export class TrackedURL extends URL {
 
 	set pathname(value) {
 		super.pathname = value;
-		set(this.#pathname, super.pathname, this.#block);
+		set(this.#pathname, super.pathname);
 	}
 
 	get port() {
@@ -112,7 +112,7 @@ export class TrackedURL extends URL {
 
 	set port(value) {
 		super.port = value;
-		set(this.#port, super.port, this.#block);
+		set(this.#port, super.port);
 	}
 
 	get protocol() {
@@ -121,7 +121,7 @@ export class TrackedURL extends URL {
 
 	set protocol(value) {
 		super.protocol = value;
-		set(this.#protocol, super.protocol, this.#block);
+		set(this.#protocol, super.protocol);
 	}
 
 	get search() {
@@ -130,7 +130,7 @@ export class TrackedURL extends URL {
 
 	set search(value) {
 		super.search = value;
-		set(this.#search, value, this.#block);
+		set(this.#search, value);
 		this.#searchParams[REPLACE](super.searchParams);
 	}
 
@@ -140,7 +140,7 @@ export class TrackedURL extends URL {
 
 	set username(value) {
 		super.username = value;
-		set(this.#username, super.username, this.#block);
+		set(this.#username, super.username);
 	}
 
 	get origin() {

package/src/utils/builders.js

@@ -714,6 +714,7 @@ export function jsx_attribute(name, value = null) {
 		type: 'JSXAttribute',
 		name,
 		value,
+		shorthand: false,
 	};
 }
 

package/tests/client/basic/basic.get-set.test.ripple

@@ -39,11 +39,15 @@ describe('basic client > get/set functions', () => {
 			let count = track(0);
 
 			<p>{get(count)}</p>
-			<button onClick={() => {
-				@count++;
-				@count++;
-				@count++;
-			}}>{'increment'}</button>
+			<button
+				onClick={() => {
+					@count++;
+					@count++;
+					@count++;
+				}}
+			>
+				{'increment'}
+			</button>
 		}
 
 		render(Test);
@@ -83,11 +87,15 @@ describe('basic client > get/set functions', () => {
 			let count = track(0);
 
 			<p>{get(count)}</p>
-			<button onClick={() => {
-				set(count, 5);
-				set(count, 15);
-				set(count, 25);
-			}}>{'set multiple times'}</button>
+			<button
+				onClick={() => {
+					set(count, 5);
+					set(count, 15);
+					set(count, 25);
+				}}
+			>
+				{'set multiple times'}
+			</button>
 		}
 
 		render(Test);
@@ -133,11 +141,15 @@ describe('basic client > get/set functions', () => {
 			let count = track(0);
 
 			<p>{get(count)}</p>
-			<button onClick={() => {
-				set(count, get(count) + 5);
-				set(count, get(count) + 15);
-				set(count, get(count) + 25);
-			}}>{'add multiple times'}</button>
+			<button
+				onClick={() => {
+					set(count, get(count) + 5);
+					set(count, get(count) + 15);
+					set(count, get(count) + 25);
+				}}
+			>
+				{'add multiple times'}
+			</button>
 		}
 
 		render(Test);
@@ -158,11 +170,11 @@ describe('basic client > get/set functions', () => {
 		expect(p.textContent).toBe('90');
 	});
 
-	function store() {
-		return track(0);
-	}
-
 	it('gets value declared outside Ripple component', () => {
+		function store() {
+			return track(0);
+		}
+
 		component Test() {
 			let count = store();
 			<p>{get(count)}</p>
@@ -175,6 +187,10 @@ describe('basic client > get/set functions', () => {
 	});
 
 	it('sets value declared outside Ripple component', () => {
+		function store() {
+			return track(0);
+		}
+
 		component Test() {
 			let count = store();
 
@@ -257,35 +273,7 @@ describe('basic client > get/set functions', () => {
 		expect(p.textContent).toBe('2');
 	});
 
-	it("get isn't reactive when declared outside Ripple context", () => {
-		let count = store();
-
-		component Test() {
-			<p>{get(count)}</p>
-			<button onClick={() => { set(count, get(count) + 1) }}>{'increment'}</button>
-		}
-
-		expect(get(count)).toBe(0);
-
-		render(Test);
-
-		const p = container.querySelector('p');
-		expect(p.textContent).toBe('0');
-		expect(get(count)).toBe(0);
-
-		const button = container.querySelector('button');
-		button.click();
-		flushSync();
-
-		expect(p.textContent).toBe('0');
-		expect(get(count)).toBe(1);
-	});
-
-	it('throws on trying to set a value outside Ripple component', () => {
-		let count = store();
-
-		expect(get(count)).toBe(0);
-		expect(() => set(count, 1)).toThrow();
-		expect(get(count)).toBe(0);
+	it('throws on trying to create tracked Ripple component', () => {
+		expect(() => track(0)).toThrow();
 	});
 });

package/tests/client/compiler/__snapshots__/compiler.assignments.test.ripple.snap

@@ -4,9 +4,9 @@ exports[`compiler > assignments > compiles tracked values in effect with assignm
 
 exports[`compiler > assignments > compiles tracked values in effect with update expressions 1`] = `
 "_$_.with_scope(__block, () => untrack(() => {
-			state.preIncrement = _$_.update_pre(count, __block);
-			state.postIncrement = _$_.update(count, __block);
-			state.preDecrement = _$_.update_pre(count, __block, -1);
-			state.postDecrement = _$_.update(count, __block, -1);
+			state.preIncrement = _$_.update_pre(count);
+			state.postIncrement = _$_.update(count);
+			state.preDecrement = _$_.update_pre(count, -1);
+			state.postDecrement = _$_.update(count, -1);
 		}));"
 `;