Fix doc strings for model and mutation registry

Also:
- simplify the object namespacing used for model.data.confirmed and
  model.data.optimistic

src/Model.ts

@@ -93,16 +93,18 @@ export default class Model<T> extends EventEmitter<Record<ModelState, ModelState
     return this.currentState;
   }
 
+  /**
+   * @returns The an object giving access to the optimistic and confirmed state of the model.
+   */
   public get data() {
-    const optimistic = (() => this.optimisticData).bind(this);
-    const confirmed = (() => this.confirmedData).bind(this);
+    const self = this;
 
     return {
       get optimistic() {
-        return optimistic();
+        return self.optimisticData;
       },
       get confirmed() {
-        return confirmed();
+        return self.confirmedData;
       },
     };
   }

src/MutationsRegistry.ts

@@ -124,16 +124,6 @@ export default class MutationsRegistry {
     );
   }
 
-  /**
-   * The optimisticEvent function that handles including optimistic events in the model state.
-   * @param {OptimisticInvocationParams} params - The set of events (params.events) that should be optimistically
-   * applied to the model state. And optionally a timeout (params.options.timeout) within which a confirmed
-   * event should be received from the backend, or else the optimistic events will rollback.
-   * @returns {Promise<[Promise<void>,() => void]>} A Promise that resolves to a [confirmed, cancel] tuple
-   * when the model has successfully applied the optimistic update. The confirmed field from the tuple is a
-   * promise that resolves when the optimistic event is confirmed. The cancel field from the tuple is a
-   * function that can be used to trigger the rollback of the optimistic event.
-   */
   public async handleOptimsitic(params: OptimisticInvocationParams): Promise<[Promise<void>, () => Promise<void>]> {
     const mergedOptions = this.mergeOptions(params.options, this.options, DEFAULT_OPTIONS);