synstate 6.1.1 → 6.2.1

package/src/core/class/propagation-state.mts

@@ -18,8 +18,8 @@ import type { UpdateToken } from '../types/index.mjs';
 import type { ChildNode } from './observable-node.mjs';
 
 /**
- * How many updates are propagating, across every observable, and the token
- * of the outermost one. A write nothing observes opens none
+ * How many updates are propagating, across every observable, and the token of
+ * the outermost one. A write nothing observes opens none
  * (`setNextIfUnobserved`).
  */
 const mut_propagation: {
@@ -28,15 +28,14 @@ const mut_propagation: {
 } = { depth: 0, outermostToken: undefined };
 
 /**
- * Work deferred by {@link runAfterPropagation}, run once the outermost
- * update has reached every descendant.
+ * Work deferred by {@link runAfterPropagation}, run once the outermost update
+ * has reached every descendant.
  */
 const mut_afterPropagation: (() => void)[] = [];
 
 /**
- * Whether some observable's update is propagating right now. A root written
- * to while one is, is written to from inside a subscriber or a projection
- * (#2336).
+ * Whether some observable's update is propagating right now. A root written to
+ * while one is, is written to from inside a subscriber or a projection (#2336).
  */
 export const isPropagating = (): boolean => mut_propagation.depth > 0;
 
@@ -71,8 +70,7 @@ export const beginPropagation = (updateToken: UpdateToken): void => {
 };
 
 /**
- * Ends one update, and runs the deferred work once the outermost one has
- * ended.
+ * Ends one update, and runs the deferred work once the outermost one has ended.
  */
 export const endPropagation = (): void => {
   mut_propagation.depth -= 1;
@@ -91,11 +89,11 @@ export const endPropagation = (): void => {
 };
 
 /**
- * The buckets of this copy's open updates, one set per nesting level: a
- * bucket per depth below the node that started the update, holding the
- * queued nodes and how many of its slots are in use. Kept from update to
- * update, so that an update allocates nothing but its token; emptying a
- * bucket is setting its length to 0.
+ * The buckets of this copy's open updates, one set per nesting level: a bucket
+ * per depth below the node that started the update, holding the queued nodes
+ * and how many of its slots are in use. Kept from update to update, so that an
+ * update allocates nothing but its token; emptying a bucket is setting its
+ * length to 0.
  */
 const mut_bucketSets: Bucket[][] = [];
 
@@ -125,9 +123,9 @@ export const openUpdate = (depth: number): UpdateToken => {
 
 /**
  * How many updates of one copy may be open at once, one inside another. An
- * update opens inside another only when a subscriber or projection writes to
- * a root while the outer one is propagating; a chain of such writes this deep
- * is a loop that would otherwise run until the stack overflows.
+ * update opens inside another only when a subscriber or projection writes to a
+ * root while the outer one is propagating; a chain of such writes this deep is
+ * a loop that would otherwise run until the stack overflows.
  */
 export const maxUpdateNesting = 100;
 
@@ -173,10 +171,10 @@ type Bucket = {
  *
  * A queued node is marked with the token of the update that queued it
  * (`queuedFor`), which is how an update queues a node once. The mark is not
- * cleared after the visit: a later update has another token. An update does
- * not overwrite the mark of an update that is still open — one it is nested
- * in, or one of another copy of synstate — and remembers the nodes it
- * queues despite such a mark in a set of its own instead.
+ * cleared after the visit: a later update has another token. An update does not
+ * overwrite the mark of an update that is still open — one it is nested in, or
+ * one of another copy of synstate — and remembers the nodes it queues despite
+ * such a mark in a set of its own instead.
  */
 class UpdateQueue {
   readonly #base: number;
@@ -252,8 +250,8 @@ class UpdateQueue {
       }
 
       if (mark !== undefined && toQueue(mark).isOpen) {
-        // Another open update has queued `child` and may not have visited
-        // it yet: leave its mark alone.
+        // Another open update has queued `child` and may not have visited it
+        // yet: leave its mark alone.
         if (this.#mut_queuedDespiteMark === undefined) {
           this.#mut_queuedDespiteMark = new Set<ChildNode>();
         } else if (this.#mut_queuedDespiteMark.has(child)) {
@@ -274,8 +272,8 @@ class UpdateQueue {
 
     const mut_buckets = this.#mut_buckets;
 
-    // `size` grows while the buckets are visited, always past `index`, which
-    // a `range` built at the start would miss.
+    // `size` grows while the buckets are visited, always past `index`, which a
+    // `range` built at the start would miss.
     // eslint-disable-next-line ts-data-forge/prefer-range-for-loop
     for (let mut_index = 0; mut_index < this.#mut_size; mut_index += 1) {
       const mut_bucket = mut_buckets[mut_index];
@@ -284,9 +282,9 @@ class UpdateQueue {
         continue;
       }
 
-      // A node visited here queues only deeper nodes, so this bucket does
-      // not grow while it is visited. Its length is kept until the end, so
-      // that `close` can clear what an exception leaves.
+      // A node visited here queues only deeper nodes, so this bucket does not
+      // grow while it is visited. Its length is kept until the end, so that
+      // `close` can clear what an exception leaves.
       const { nodes: mut_nodes, length } = mut_bucket;
 
       // eslint-disable-next-line ts-data-forge/prefer-range-for-loop

package/src/core/class/remove-completed.test.mts

@@ -202,8 +202,8 @@ describe('a completed observable leaves the propagation graph', () => {
 
     const mut_got: number[] = [];
 
-    // Subscribed on a child, so the node is created while the update of `s`
-    // is under way.
+    // Subscribed on a child, so the node is created while the update of `s` is
+    // under way.
     s.pipe(map((x) => x)).subscribe((x) => {
       if (x === 1) {
         s.pipe(

package/src/core/class/root-observable-class.mts

@@ -16,11 +16,11 @@ export class RootObservableClass<A>
   implements RootObservable<A>
 {
   /**
-   * The outermost update propagating when this root was created. A root
-   * created while an update propagates (inside a `switchMap` projection,
-   * say) may emit while it is created, as `fromSubscribable` does for a
-   * synchronous source. Nothing is derived from it yet, so that is its
-   * initial value rather than a write (#2336).
+   * The outermost update propagating when this root was created. A root created
+   * while an update propagates (inside a `switchMap` projection, say) may emit
+   * while it is created, as `fromSubscribable` does for a synchronous source.
+   * Nothing is derived from it yet, so that is its initial value rather than a
+   * write (#2336).
    */
   readonly #createdDuring: UpdateToken | undefined;
   #mut_hasWarned: boolean;
@@ -64,10 +64,10 @@ export class RootObservableClass<A>
 
 /**
  * Shown once per root observable, the first time it is written to (`next`,
- * `setState`, `dispatch`, …) while another update is still propagating —
- * from a subscriber or a projection. The write still propagates immediately,
- * as it always has; the warning is there because the pattern is wrong, not
- * because anything was refused.
+ * `setState`, `dispatch`, …) while another update is still propagating — from a
+ * subscriber or a projection. The write still propagates immediately, as it
+ * always has; the warning is there because the pattern is wrong, not because
+ * anything was refused.
  */
 export const writeDuringPropagationWarning =
   'synstate: an observable was written to (next / setState / dispatch) while an update was propagating, so subscribers can see intermediate states. Derive the value with map / combine instead of writing from a subscriber or a projection.';

package/src/core/class/unobserved-write.test.mts

@@ -6,8 +6,8 @@ import { debounce, map, switchMap } from '../operators/index.mjs';
 import { writeDuringPropagationWarning } from './root-observable-class.mjs';
 
 // A write to an observable with no child and no subscriber stores the value
-// without opening an update (`ManagerObservableClass.startUpdate`). These
-// pin what that must not change.
+// without opening an update (`ManagerObservableClass.startUpdate`). These pin
+// what that must not change.
 describe('a write nothing observes', () => {
   test('is held, and a later subscriber receives the last one', () => {
     const s$ = source<number>(0);
@@ -98,8 +98,8 @@ describe('a write nothing observes', () => {
 
     a$.next(6);
 
-    // merge passes on the parents that emitted in an update: only b$, then
-    // only a$, never the value a$ was given while nothing observed it.
+    // merge passes on the parents that emitted in an update: only b$, then only
+    // a$, never the value a$ was given while nothing observed it.
     assert.deepStrictEqual(mut_merged, [1, 6]);
 
     assert.deepStrictEqual(mut_derived, [

package/src/core/combine/derive.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -20,14 +20,13 @@ import type {
 
 /**
  * Derives one value from the latest values of several observables: one
- * observable that calls `project` with them, as arguments, and emits the
- * result unless it equals the one already held.
+ * observable that calls `project` with them, as arguments, and emits the result
+ * unless it equals the one already held.
  *
- * It emits once every parent has a value, and then whenever any of them
- * emits, as `combine` does; but where `combine(parents).pipe(map(fn))` is two
- * observables and a tuple per update, `derive` is one, and passes the values
- * as arguments (a rest parameter in `project` collects them into an array
- * again).
+ * It emits once every parent has a value, and then whenever any of them emits,
+ * as `combine` does; but where `combine(parents).pipe(map(fn))` is two
+ * observables and a tuple per update, `derive` is one, and passes the values as
+ * arguments (a rest parameter in `project` collects them into an array again).
  * The comparison is `Object.is` unless `equals` says otherwise.
  *
  * @template OS - Tuple type of the parent observables
@@ -83,8 +82,8 @@ export const derive = <const OS extends NonEmptyTuple<Observable<unknown>>, B>(
   // @sumi-expect-error banned-syntax/no-unsafe-type-assertion
   new DeriveObservableClass(
     parents,
-    // `project` is called with the values of `parents`, position by
-    // position, which the checker cannot follow through an index.
+    // `project` is called with the values of `parents`, position by position,
+    // which the checker cannot follow through an index.
     // @sumi-expect-error banned-syntax/no-unsafe-type-assertion
     project as (...values: readonly unknown[]) => B,
     options?.equals ?? Object.is,

package/src/core/combine/merge.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/combine/zip.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -26,9 +26,9 @@ import type {
 /**
  * Combines multiple observables by pairing their emissions by index.
  * Waits for all sources to emit their nth value before emitting the nth tuple.
- * Completes once a completed source has no value left to pair, as RxJS's
- * `zip` does: at once if it completes with nothing queued, otherwise after
- * the tuple that takes its last queued value.
+ * Completes once a completed source has no value left to pair, as RxJS's `zip`
+ * does: at once if it completes with nothing queued, otherwise after the tuple
+ * that takes its last queued value.
  *
  * @template OS - Tuple type of source observables
  * @param parents - Array of observables to zip
@@ -156,8 +156,8 @@ class ZipObservableClass<const A extends NonEmptyUnknownList>
   }
 
   /**
-   * No tuple can be completed any more once a completed source has nothing
-   * left to pair (#2343).
+   * No tuple can be completed any more once a completed source has nothing left
+   * to pair (#2343).
    */
   override canEmitNoMore(): boolean {
     return this.parents.some(

package/src/core/create/counter.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -12,9 +12,9 @@ import type { CounterObservable, TimerId } from '../types/index.mjs';
 
 /**
  * Creates an observable that emits incremental numbers at a specified interval.
- * Emits 0 in the macrotask right after `start()` (which runs at creation
- * unless `startManually` is set), then 1, 2, 3, ... every interval. A
- * subscriber added after that 0 has been emitted does not receive it.
+ * Emits 0 in the macrotask right after `start()` (which runs at creation unless
+ * `startManually` is set), then 1, 2, 3, ... every interval. A subscriber added
+ * after that 0 has been emitted does not receive it.
  *
  * @param intervalMilliSeconds - The interval duration in milliseconds
  * @param options - Optional settings

package/src/core/create/from-abortable-promise.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/create/from-promise.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/create/from-subscribable.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/create/source.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -79,9 +79,9 @@ class SourceObservableClass<A>
     });
   }
 
-  // Returns the value held after the call: `nextValue`, or, once completed,
-  // the value already there — `undefined` only on a source that never had
-  // one, whose `next` is typed `void`.
+  // Returns the value held after the call: `nextValue`, or, once completed, the
+  // value already there — `undefined` only on a source that never had one,
+  // whose `next` is typed `void`.
   next(nextValue: A): A | undefined {
     if (this.isCompleted) {
       return Optional.unwrap(this.getSnapshot());

package/src/core/create/timer.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/audit.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/debounce.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/filter.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -175,8 +175,8 @@ if (import.meta.vitest !== undefined) {
 
     expectType<typeof _inferred, Observable<number>>('=');
 
-    // A generic guard is what a single signature with a union parameter
-    // loses (it infers `unknown`), and why this set stays overloaded.
+    // A generic guard is what a single signature with a union parameter loses
+    // (it infers `unknown`), and why this set stays overloaded.
     const _generic = s.pipe(filter(isNotUndefined));
 
     expectType<typeof _generic, Observable<number>>('=');

package/src/core/operators/keep-equal-parts.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -20,19 +20,19 @@ import type {
 
 /**
  * Passes on each value with every part deeply equal to the same part of the
- * value passed on before replaced by that part, and skips a value deeply
- * equal to it altogether (structural sharing). A reader of a part that did
- * not change keeps the object it has, so `skipIfNoChange()` with `Object.is`
- * stops it. Superset of `skipIfNoChange(fastDeepEqual)`.
+ * value passed on before replaced by that part, and skips a value deeply equal
+ * to it altogether (structural sharing). A reader of a part that did not change
+ * keeps the object it has, so `skipIfNoChange()` with `Object.is` stops it.
+ * Superset of `skipIfNoChange(fastDeepEqual)`.
  *
  * **Prefer keeping references where the value is written** — compare before
- * `setState`, and skip it when nothing changed — so that nothing downstream
- * has anything to undo. The `keepEqualParts` option of `createState` and
- * `createReducer` does it for a state written whole, and `replaceEqualDeep`
- * of ts-data-forge does it for any other write. This operator lets every
- * update that changes nothing reach the source and everything up to it, and
- * walks the whole value again on each one; reach for it when the source is
- * not yours to change (a stream that arrives whole from outside).
+ * `setState`, and skip it when nothing changed — so that nothing downstream has
+ * anything to undo. The `keepEqualParts` option of `createState` and
+ * `createReducer` does it for a state written whole, and `replaceEqualDeep` of
+ * ts-data-forge does it for any other write. This operator lets every update
+ * that changes nothing reach the source and everything up to it, and walks the
+ * whole value again on each one; reach for it when the source is not yours to
+ * change (a stream that arrives whole from outside).
  *
  * Array items are matched by index; pass `{ key }` to match them by key, an
  * item for which `key` returns `undefined` being matched by index. Only plain
@@ -51,10 +51,9 @@ import type {
  *   cards: { id: number; text: string }[];
  * }>;
  *
- * // A stream that arrives whole from outside (a WebSocket, say), each
- * // message a new object even when nothing changed. A state you write
- * // yourself takes `createState(initial, { keepEqualParts: true })`
- * // instead.
+ * // A stream that arrives whole from outside (a WebSocket, say), each message
+ * // a new object even when nothing changed. A state you write yourself takes
+ * // `createState(initial, { keepEqualParts: true })` instead.
  * const message$ = source<Board>();
  *
  * const board$ = message$.pipe(keepEqualParts());

package/src/core/operators/map.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/merge-map.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -22,9 +22,9 @@ import type {
  * Unlike `switchMap`, does not cancel previous inner observables.
  *
  * Each projection runs in a scope of its own (see `createScope`): what it
- * creates is disposed when `mergeMap` completes. An observable it only
- * returns — one created outside the projection and shared with others — is
- * unsubscribed and left running.
+ * creates is disposed when `mergeMap` completes. An observable it only returns
+ * — one created outside the projection and shared with others — is unsubscribed
+ * and left running.
  *
  * @template A - The type of values from the source
  * @template B - The type of values from the projected observable

package/src/core/operators/pairwise.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -90,8 +90,8 @@ class PairwiseObservableClass<A>
       initialValue: Optional.none,
     });
 
-    // parentObservable.snapshot has value
-    // if parentObservable is InitializedObservable
+    // parentObservable.snapshot has value if parentObservable is
+    // InitializedObservable
     this.#mut_previousValue = parentObservable.getSnapshot();
 
     this.attach();

package/src/core/operators/scan.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/skip-if-no-change.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -104,8 +104,8 @@ class SkipIfNoChangeObservableClass<A>
       initialValue: parentObservable.getSnapshot(),
     });
 
-    // parentObservable.snapshot has value
-    // if parentObservable is InitializedObservable
+    // parentObservable.snapshot has value if parentObservable is
+    // InitializedObservable
     this.#mut_previousValue = parentObservable.getSnapshot();
 
     this.#eq = eq;

package/src/core/operators/skip-until.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -83,8 +83,8 @@ class SkipUntilObservableClass<A>
       initialValue: Optional.none,
     });
 
-    // A notifier that already holds a value, or has completed, would stop
-    // the skipping inside `subscribe`, before the subscription is stored.
+    // A notifier that already holds a value, or has completed, would stop the
+    // skipping inside `subscribe`, before the subscription is stored.
     this.#mut_isSkipping =
       Optional.isNone(notifier.getSnapshot()) && !notifier.isCompleted;
 

package/src/core/operators/skip-while.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/switch-map.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function
@@ -126,9 +126,9 @@ class SwitchMapObservableClass<A, B>
   readonly #mapToObservable: (curr: A) => Observable<B>;
 
   /**
-   * The scope the operator was created in, so that each inner scope is
-   * disposed with it; the projection itself runs during propagation, when the
-   * current scope is whatever happens to be running.
+   * The scope the operator was created in, so that each inner scope is disposed
+   * with it; the projection itself runs during propagation, when the current
+   * scope is whatever happens to be running.
    */
   readonly #ownerScope: Scope | undefined;
 

package/src/core/operators/take-until.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/take-while.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/take.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function

package/src/core/operators/throttle.mts

@@ -1,6 +1,6 @@
 // Sumi bans classes, `this`, method syntax and writes to `this` outside a
-// `mut_` binding; an observable is a class (`ObservableBaseClass` says why),
-// so those rules are waived here (D-63).
+// `mut_` binding; an observable is a class (`ObservableBaseClass` says why), so
+// those rules are waived here (D-63).
 // @sumi-expect-error-file classes/no-class
 // @sumi-expect-error-file banned-syntax/no-this
 // @sumi-expect-error-file functions/prefer-arrow-function