type fixes

Author: dmonad

package.json

@@ -278,7 +278,7 @@
     "test": "c8 --check-coverage --lines 100 --branches 100 --functions 100 --statements 100 node --unhandled-rejections=strict ./src/test.js --repetition-time 50",
     "test-inspect": "node --inspect-brk --unhandled-rejections=strict ./src/test.js --repetition-time 50",
     "test-extensive": "c8 --check-coverage --lines 100 --branches 100 --functions 100 --statements 100 node ./src/test.js --repetition-time 30000 --extensive",
-    "test:bundle": "esbuild bundle.test.js --bundle --outfile=dist/bundle.test.js --analyze && wc -c dist/bundle.test.js",
+    "test:bundle": "esbuild bundle.test.js --bundle --outfile=dist/bundle.test.js --analyze --minify && wc -c dist/bundle.test.js",
     "trace-deopt": "clear && node --trace-deopt src/test.js",
     "trace-opt": "clear && node --trace-opt src/test.js",
     "lint": "standard && tsc",

src/bundle.test.js

@@ -1,3 +1,3 @@
-import { map } from './index.js'
+import * as delta from 'lib0/delta'
 
-console.log(map.all(new Map(), x => x === 1))
+console.log(delta.create().toJSON())

src/delta/delta.js

@@ -691,7 +691,7 @@ export class SetAttrOp {
 
 /**
  * @template [V=any]
- * @template {string|number} [K=string]
+ * @template {string|number} [K=string|number]
  */
 export class DeleteAttrOp {
   /**
@@ -845,7 +845,7 @@ export const $anyOp = s.$union($insertOp, $deleteOp, $textOp, $modifyOp)
 
 export const $setAttrOp = /** @type {s.Schema<SetAttrOp<any>>} */ (s.$type('setAttrOp'))
 export const $modifyAttrOp = /** @type {s.Schema<ModifyAttrOp<any,string|number>>} */ (s.$type('modifyAttrOp'))
-export const $deleteAttrOp = /** @type {s.Schema<DeleteAttrOp<{},string|number>>} */ (s.$type('deleteAttrOp'))
+export const $deleteAttrOp = /** @type {s.Schema<DeleteAttrOp<any,string|number>>} */ (s.$type('deleteAttrOp'))
 export const $anyAttrOp = s.$union($setAttrOp, $deleteAttrOp, $modifyAttrOp)
 
 /**
@@ -1000,7 +1000,7 @@ class DeltaData {
 
     /**
      * @type {list.List<
-     *   | (Text extends true ? (RetainOp|TextOp|DeleteOp<{}>) : never)
+     *   | (Text extends true ? (RetainOp|TextOp|DeleteOp<any>) : never)
      *   | (RetainOp|InsertOp<Children>|DeleteOp<any>|(Delta extends Children ? ModifyOp<Extract<Children,Delta>> : never))
      * >}
      */
@@ -1430,7 +1430,7 @@ export class DeltaBuilder extends Delta {
    * @param {Attribution?} attribution
    * @return {DeltaBuilder<DeltaConfOverwrite<
    *   DConf,
-   *   { attrs: MergeAttrs<Attrs,NewAttrs> }
+   *   { attrs: MergeAttrs<DeltaConfGetAttrs<DConf>,NewAttrs> }
    *   >>
    * }
    */

src/ts.js

@@ -29,3 +29,11 @@
  * @template B
  * @typedef {A} AssertExtends
  */
+
+/**
+ * In typescript `keyof {[k:string]:any}` is `string|number` instead of `string`. This KeyOf type returns the expected
+ * result. Use this for pretty-printing onyly.
+ *
+ * @template {{[K:string|number|symbol]:any}} Obj
+ * @typedef {{ [K in keyof Obj]: K }[keyof Obj]} KeyOf
+ */