Merge pull request #710 from streamich/production-build

Remove usage of `constructor.name` property

Author: streamich

src/json-crdt-extensions/peritext/Peritext.ts

@@ -315,7 +315,7 @@ export class Peritext<T = string> implements Printable {
     const nl = () => '';
     const {savedSlices, extraSlices, localSlices} = this;
     return (
-      this.constructor.name +
+      'Peritext' +
       printTree(tab, [
         (tab) => this.str.toString(tab),
         nl,

src/json-crdt-extensions/peritext/PeritextApi.ts

@@ -29,8 +29,7 @@ export class PeritextApi extends NodeApi<PeritextNode> implements ExtApi<Peritex
 
   public toString(tab?: string): string {
     return (
-      this.constructor.name +
-      printTree(tab, [(tab) => this.node.toString(tab), () => '', (tab) => this.txt.toString(tab)])
+      'PeritextApi' + printTree(tab, [(tab) => this.node.toString(tab), () => '', (tab) => this.txt.toString(tab)])
     );
   }
 }

src/json-crdt-extensions/peritext/block/Block.ts

@@ -139,10 +139,13 @@ export class Block<Attr = unknown> implements IBlock, Printable, Stateful {
 
   // ---------------------------------------------------------------- Printable
 
+  protected toStringName(): string {
+    return 'Block';
+  }
   protected toStringHeader(): string {
     const hash = `#${this.hash.toString(36).slice(-4)}`;
     const tag = `<${this.path.join('.')}>`;
-    const header = `${this.constructor.name} ${hash} ${tag}`;
+    const header = `${this.toStringName()} ${hash} ${tag}`;
     return header;
   }
 

src/json-crdt-extensions/peritext/block/Blocks.ts

@@ -18,7 +18,7 @@ export class Blocks implements Printable, Stateful {
   // ---------------------------------------------------------------- Printable
 
   public toString(tab: string = ''): string {
-    return this.constructor.name + printTree(tab, [(tab) => this.root.toString(tab)]);
+    return 'Blocks' + printTree(tab, [(tab) => this.root.toString(tab)]);
   }
 
   // ----------------------------------------------------------------- Stateful

src/json-crdt-extensions/peritext/block/Inline.ts

@@ -149,14 +149,18 @@ export class Inline extends Range implements Printable {
 
   // ---------------------------------------------------------------- Printable
 
+  public toStringName(): string {
+    return 'Inline';
+  }
+
   public toString(tab: string = ''): string {
     const str = this.text();
     const truncate = str.length > 32;
     const text = JSON.stringify(truncate ? str.slice(0, 32) : str) + (truncate ? ' …' : '');
     const startFormatted = this.start.toString(tab, true);
     const range =
       this.start.cmp(this.end) === 0 ? startFormatted : `${startFormatted} ↔ ${this.end.toString(tab, true)}`;
-    const header = `${this.constructor.name} ${range} ${text}`;
+    const header = `Inline ${range} ${text}`;
     const marks = this.attr();
     const markKeys = Object.keys(marks);
     return (

src/json-crdt-extensions/peritext/block/LeafBlock.ts

@@ -10,6 +10,9 @@ export interface IBlock<Attr = unknown> {
 
 export class LeafBlock<Attr = unknown> extends Block<Attr> {
   // ---------------------------------------------------------------- Printable
+  protected toStringName(): string {
+    return 'LeafBlock';
+  }
 
   protected toStringHeader(): string {
     const str = this.text();

src/json-crdt-extensions/peritext/editor/Cursor.ts

@@ -100,7 +100,11 @@ export class Cursor<T = string> extends PersistedSlice<T> {
   // ---------------------------------------------------------------- Printable
 
   public toStringName(): string {
+    return 'Cursor';
+  }
+
+  public toStringHeaderName(): string {
     const focusIcon = this.anchorSide === CursorAnchor.Start ? '.→|' : '|←.';
-    return `${super.toStringName()}, ${focusIcon}`;
+    return `${super.toStringHeaderName()}, ${focusIcon}`;
   }
 }

src/json-crdt-extensions/peritext/overlay/MarkerOverlayPoint.ts

@@ -32,15 +32,19 @@ export class MarkerOverlayPoint<T = string> extends OverlayPoint<T> implements H
 
   // ---------------------------------------------------------------- Printable
 
-  public toStringName(tab: string, lite?: boolean): string {
+  public toStringName(): string {
+    return 'OverlayPoint';
+  }
+
+  public toStringHeader(tab: string, lite?: boolean): string {
     const hash = lite ? '' : `#${this.textHash.toString(36).slice(-4)}`;
     const tag = lite ? '' : `, type = ${JSON.stringify(this.type() as any)}`;
-    return `${super.toStringName(tab, lite)}${lite ? '' : ' '}${hash}${tag}`;
+    return `${super.toStringHeader(tab, lite)}${lite ? '' : ' '}${hash}${tag}`;
   }
 
   public toString(tab: string = '', lite?: boolean): string {
     return (
-      this.toStringName(tab, lite) +
+      this.toStringHeader(tab, lite) +
       (lite
         ? ''
         : printTree(tab, [

src/json-crdt-extensions/peritext/overlay/Overlay.ts

@@ -530,7 +530,7 @@ export class Overlay<T = string> implements Printable, Stateful {
       );
     };
     return (
-      `${this.constructor.name} #${this.hash.toString(36)}` +
+      `Overlay #${this.hash.toString(36)}` +
       printTree(tab, [
         !this.root ? null : (tab) => printPoint(tab, this.root!),
         !this.root2 ? null : (tab) => printMarkerPoint(tab, this.root2!),

src/json-crdt-extensions/peritext/overlay/OverlayPoint.ts

@@ -205,13 +205,17 @@ export class OverlayPoint<T = string> extends Point<T> implements Printable, Hea
 
   // ---------------------------------------------------------------- Printable
 
-  public toStringName(tab: string, lite?: boolean): string {
+  public toStringName(): string {
+    return 'OverlayPoint';
+  }
+
+  public toStringHeader(tab: string = '', lite?: boolean): string {
     return super.toString(tab, lite);
   }
 
   public toString(tab: string = '', lite?: boolean): string {
     const refs = lite ? '' : `, refs = ${this.refs.length}`;
-    const header = this.toStringName(tab, lite) + refs;
+    const header = this.toStringHeader(tab, lite) + refs;
     if (lite) return header;
     const children: PrintChild[] = [];
     const layers = this.layers;

src/json-crdt-extensions/peritext/rga/Point.ts

@@ -449,8 +449,12 @@ export class Point<T = string> implements Pick<Stateful, 'refresh'>, Printable {
 
   // ---------------------------------------------------------------- Printable
 
+  public toStringName(): string {
+    return 'Point';
+  }
+
   public toString(tab: string = '', lite?: boolean): string {
-    const name = lite ? '' : this.constructor.name + ' ';
+    const name = lite ? '' : this.toStringName() + ' ';
     const pos = this.pos();
     const id = printTs(this.id);
     const anchor = this.anchor === Anchor.Before ? '.▢' : '▢.';

src/json-crdt-extensions/peritext/rga/Range.ts

@@ -231,8 +231,12 @@ export class Range<T = string> implements Pick<Stateful, 'refresh'>, Printable {
 
   // ---------------------------------------------------------------- Printable
 
+  public toStringName(): string {
+    return 'Range';
+  }
+
   public toString(tab: string = '', lite: boolean = true): string {
-    const name = lite ? '' : `${this.constructor.name} `;
+    const name = lite ? '' : this.toStringName();
     const start = this.start.toString(tab, lite);
     const end = this.end.toString(tab, lite);
     let text = this.text();

src/json-crdt-extensions/peritext/slice/ExtraSlices.ts

@@ -1,3 +1,7 @@
 import {Slices} from './Slices';
 
-export class ExtraSlices<T = string> extends Slices<T> {}
+export class ExtraSlices<T = string> extends Slices<T> {
+  public toStringName(): string {
+    return 'ExtraSlices';
+  }
+}

src/json-crdt-extensions/peritext/slice/LocalSlices.ts

@@ -6,4 +6,8 @@ export class LocalSlices<T = string> extends Slices<T> {
     super.del(id);
     if (Math.random() < 0.1) this.set.removeTombstones();
   }
+
+  public toStringName(): string {
+    return 'LocalSlices';
+  }
 }

src/json-crdt-extensions/peritext/slice/PersistedSlice.ts

@@ -164,22 +164,26 @@ export class PersistedSlice<T = string> extends Range<T> implements MutableSlice
 
   // ---------------------------------------------------------------- Printable
 
-  protected toStringName(): string {
+  public toStringName(): string {
+    return 'Range';
+  }
+
+  protected toStringHeaderName(): string {
     const data = this.data();
     const dataFormatted = data ? prettyOneLine(data) : '∅';
     const dataLengthBreakpoint = 32;
-    const header = `${this.constructor.name} ${super.toString('', true)}, ${
+    const header = `${this.toStringName()} ${super.toString('', true)}, ${
       SliceBehaviorName[this.behavior]
     }, ${JSON.stringify(this.type)}${dataFormatted.length < dataLengthBreakpoint ? `, ${dataFormatted}` : ''}`;
     return header;
   }
 
-  public toString(tab: string = ''): string {
+  public toStringHeader(tab: string = ''): string {
     const data = this.data();
     const dataFormatted = data ? prettyOneLine(data) : '';
     const dataLengthBreakpoint = 32;
     return (
-      this.toStringName() +
+      this.toStringHeaderName() +
       printTree(tab, [dataFormatted.length < dataLengthBreakpoint ? null : (tab) => dataFormatted])
     );
   }

src/json-crdt-extensions/peritext/slice/Slices.ts

@@ -201,9 +201,13 @@ export class Slices<T = string> implements Stateful, Printable {
 
   // ---------------------------------------------------------------- Printable
 
+  public toStringName(): string {
+    return 'Slices';
+  }
+
   public toString(tab: string = ''): string {
     return (
-      this.constructor.name +
+      this.toString() +
       printTree(
         tab,
         [...this.list.entries()].map(

src/json-crdt-extensions/peritext/util/ChunkSlice.ts

@@ -56,7 +56,7 @@ export class ChunkSlice<T = string> implements IChunkSlice<T>, Stateful, Printab
   // ---------------------------------------------------------------- Printable
 
   public toString(tab: string = ''): string {
-    const name = this.constructor.name;
+    const name = 'ChunkSlice';
     const off = this.off;
     const len = this.len;
     const str = this.view() + '';

src/json-crdt-patch/Batch.ts

@@ -30,7 +30,7 @@ export class Batch {
 
   public toString(tab: string = ''): string {
     const id = this.getId();
-    let out = `${this.constructor.name} ${id ? printTs(id) : '(nil)'}\n`;
+    let out = `Batch ${id ? printTs(id) : '(nil)'}\n`;
     for (let i = 0; i < this.patches.length; i++) {
       const patch = this.patches[i];
       const isLast = i === this.patches.length - 1;

src/json-crdt-patch/Patch.ts

@@ -214,7 +214,7 @@ export class Patch implements Printable {
    */
   public toString(tab: string = ''): string {
     const id = this.getId();
-    const header = `${this.constructor.name} ${id ? printTs(id) : '(nil)'}!${this.span()}`;
+    const header = `Patch ${id ? printTs(id) : '(nil)'}!${this.span()}`;
     return (
       header +
       printTree(

src/json-crdt/model/api/nodes.ts

@@ -180,7 +180,7 @@ export class NodeApi<N extends JsonNode = JsonNode> implements Printable {
   }
 
   public toString(tab: string = ''): string {
-    return this.constructor.name + printTree(tab, [(tab) => this.node.toString(tab)]);
+    return 'api' + printTree(tab, [(tab) => this.node.toString(tab)]);
   }
 }
 

src/json-crdt/nodes/rga/AbstractRga.ts

@@ -879,7 +879,7 @@ export abstract class AbstractRga<T> {
   // ---------------------------------------------------------------- Printable
 
   protected toStringName(): string {
-    return this.constructor.name;
+    return 'AbstractRga';
   }
 
   public toString(tab: string = ''): string {
@@ -904,7 +904,7 @@ export abstract class AbstractRga<T> {
 
   protected formatChunk(chunk: Chunk<T>): string {
     const id = printTs(chunk.id);
-    let str = `${chunk.constructor.name} ${id}!${chunk.span} len:${chunk.len}`;
+    let str = `chunk ${id}!${chunk.span} len:${chunk.len}`;
     if (chunk.del) str += ` [${chunk.span}]`;
     else {
       if (isUint8Array(chunk.data)) str += ` { ${printOctets(chunk.data) || '∅'} }`;

src/json-crdt/nodes/str/__tests__/StrNode.spec.ts

@@ -154,21 +154,21 @@ describe('binary tree', () => {
     const tree = createTree();
     expect(tree.toString()).toMatchInlineSnapshot(`
       "str .0 { "a1a2a3a4a5a6a7a8a9a10a11a12a13a1" … }
-      └─ StrChunk .8!2 len:36 { "a8" }
-         ← StrChunk .4!2 len:14 { "a4" }
-           ← StrChunk .2!2 len:6 { "a2" }
-             ← StrChunk .1!2 len:2 { "a1" }
-             → StrChunk .3!2 len:2 { "a3" }
-           → StrChunk .6!2 len:6 { "a6" }
-             ← StrChunk .5!2 len:2 { "a5" }
-             → StrChunk .7!2 len:2 { "a7" }
-         → StrChunk .12!3 len:20 { "a12" }
-           ← StrChunk .10!3 len:8 { "a10" }
-             ← StrChunk .9!2 len:2 { "a9" }
-             → StrChunk .11!3 len:3 { "a11" }
-           → StrChunk .14!3 len:9 { "a14" }
-             ← StrChunk .13!3 len:3 { "a13" }
-             → StrChunk .15!3 len:3 { "a15" }"
+      └─ chunk .8!2 len:36 { "a8" }
+         ← chunk .4!2 len:14 { "a4" }
+           ← chunk .2!2 len:6 { "a2" }
+             ← chunk .1!2 len:2 { "a1" }
+             → chunk .3!2 len:2 { "a3" }
+           → chunk .6!2 len:6 { "a6" }
+             ← chunk .5!2 len:2 { "a5" }
+             → chunk .7!2 len:2 { "a7" }
+         → chunk .12!3 len:20 { "a12" }
+           ← chunk .10!3 len:8 { "a10" }
+             ← chunk .9!2 len:2 { "a9" }
+             → chunk .11!3 len:3 { "a11" }
+           → chunk .14!3 len:9 { "a14" }
+             ← chunk .13!3 len:3 { "a13" }
+             → chunk .15!3 len:3 { "a15" }"
     `);
   });
 

src/json-crdt/nodes/str/__tests__/StrNodeFuzzer.ts

@@ -87,7 +87,7 @@ class StrNodeSite implements Printable {
 
   public toString(tab?: string): string {
     return (
-      this.constructor.name +
+      'StrNodeSite' +
       printTree(tab, [
         (tab) => 'clock ' + this.clock.toString(tab),
         (tab) =>
@@ -244,7 +244,7 @@ export class StrNodeFuzzer extends Fuzzer implements Printable {
 
   public toString(tab?: string): string {
     return (
-      this.constructor.name +
+      'StrNodeFuzzer' +
       printTree(tab, [
         (tab) => this.sites[0].rga.toString(tab),
         (tab) => 'prelude ' + printPatch(tab, this.prelude),

src/json-ot/types/ot-json/tree.ts

@@ -112,7 +112,7 @@ export class Register {
       ? '/' + this.pick.path.slice(0, this.pick.pathLength).join('/')
       : `{ ${JSON.stringify(this.data)} }`;
     const dst = drops ? `{ ${drops} }` : '∅';
-    return `${this.id}: ${this.constructor.name} ${src} ┈┈┈→ ${dst}`;
+    return `${this.id}: Register ${src} ┈┈┈→ ${dst}`;
   }
 }
 
@@ -408,6 +408,6 @@ export class OpTree {
       i++;
     }
     const drops = this.drop ? this.drop.toString(tab + '   ') : ' ∅';
-    return `${this.constructor.name}\n${tab}├─ ${picks}\n${tab}│\n${tab}├─ ${registers}\n${tab}│\n${tab}└─ ${drops}`;
+    return `OpTree\n${tab}├─ ${picks}\n${tab}│\n${tab}├─ ${registers}\n${tab}│\n${tab}└─ ${drops}`;
   }
 }

src/json-type-value/ObjectValue.ts

@@ -159,6 +159,6 @@ export class ObjectValue<T extends classes.ObjectType<any>> extends Value<T> imp
   }
 
   public toString(tab: string = ''): string {
-    return this.constructor.name + printTree(tab, [(tab) => this.type.toString(tab)]);
+    return 'ObjectValue' + printTree(tab, [(tab) => this.type.toString(tab)]);
   }
 }

src/json-type/system/TypeSystem.ts

@@ -74,7 +74,7 @@ export class TypeSystem implements Printable {
   public toString(tab: string = '') {
     const nl = () => '';
     return (
-      this.constructor.name +
+      'TypeSystem' +
       printTree(tab, [
         (tab) =>
           'aliases' +