@expressive/mvc 0.6.1 → 0.7.0

package/lib/index.d.ts

@@ -53,11 +53,6 @@ declare namespace Model {
     export type Methods<T> = {
         [K in keyof T]: T[K] extends Ref ? never : T[K] extends Function ? K : never;
     }[keyof T];
-    export type Refs<T extends Model> = {
-        <K extends Field<T>>(key: K, value: T[K]): void;
-    } & {
-        [P in Field<T>]: (next: any) => void;
-    };
     /**
      * Subset of `keyof T` which are not methods or defined by base Model U.
      *
@@ -92,13 +87,6 @@ interface Model extends Stateful {
      * Useful to obtain full reference where one has already destructured.
      */
     is: this;
-    /**
-    * Setter utility
-    *
-    * Can be used to set (and force-update) values.
-    * Shortcut to update values, contains refs for all controlled values.
-    */
-    set: Model.Refs<this>;
 }
 declare class Model {
     static [WHY]: readonly string[];
@@ -145,7 +133,7 @@ declare class Model {
      *
      * @param args - arguments sent to constructor
      */
-    static create<T extends Class>(this: T, ...args: any[]): InstanceOf<T>;
+    static create<T extends Class>(this: T, ...args: ConstructorParameters<T>): InstanceOf<T>;
     /**
      * Static equivalent of `x instanceof this`.
      *
@@ -301,6 +289,15 @@ declare namespace ref {
         [P in Model.Field<T>]: Model.Ref<T[P]>;
     };
 }
+/**
+ * Creates an object with references to all managed values.
+ * Each property will set value in state when invoked.
+ *
+ * *Properties are simultaneously a ref-function and ref-object, use as needed.*
+ *
+ * @param target - Source model from which to reference values.
+ */
+declare function ref<T extends Model>(target: T): ref.Proxy<T>;
 /**
  * Creates a ref-compatible property.
  * Will persist value, and updates to this are made part of controller event-stream.
@@ -533,13 +530,13 @@ declare namespace Provider {
 declare function Provider<T extends Provider.Item>(props: Provider.Props<T>): React.FunctionComponentElement<React.ProviderProps<Lookup>>;
 
 declare class Global extends MVC {
-    static create<T extends Class>(this: T, ...args: any[]): InstanceOf<T>;
+    static create<T extends Class>(this: T, ...args: ConstructorParameters<T>): InstanceOf<T>;
     /**
      * Update the active instance of this class.
      * Returns a thenable; resolves after successful update.
      * If instance does not already exist, one will be created.
      */
-    static set<T extends typeof Global>(this: T, updates: Partial<InstanceOf<T>>): Promise<false>;
+    static set<T extends typeof Global>(this: T, updates: Partial<InstanceOf<T>>): any;
     /**
      * **React Hook** - Fetch most instance of this controller from context, if it exists.
      *

package/lib/index.esm.js

@@ -196,22 +196,6 @@ class Model {
         defineProperty(this, "is", {
             value: this
         });
-        defineProperty(this, "set", {
-            configurable: true,
-            get: () => {
-                const controller = ensure(this);
-                const value = (key, value) => {
-                    controller.state.set(key, value);
-                    controller.update(key);
-                };
-                for (const key of controller.state.keys())
-                    defineProperty(value, key, {
-                        value: controller.ref(key)
-                    });
-                defineProperty(this, "set", { value });
-                return value;
-            }
-        });
     }
     get [STATE]() {
         return this.export();
@@ -474,8 +458,11 @@ class Controller {
             return;
         if (typeof value == "symbol") {
             const instruction = PENDING.get(value);
-            if (instruction)
+            if (instruction) {
+                delete subject[key];
+                PENDING.delete(value);
                 instruction.call(this, key, this);
+            }
             return;
         }
         state.set(key, value);
@@ -579,9 +566,6 @@ function apply(fn, label) {
     const placeholder = Symbol(`${name} instruction`);
     function setup(key) {
         const { subject, state } = this;
-        const control = this;
-        PENDING.delete(placeholder);
-        delete subject[key];
         let output = fn.call(this, key, this);
         if (typeof output == "function")
             output = { get: output };
@@ -604,6 +588,7 @@ function apply(fn, label) {
                         destroy();
                 });
         }
+        const control = this;
         defineProperty(subject, key, {
             enumerable: output.enumerable,
             set: onSet !== false
@@ -752,7 +737,8 @@ function factoryMode(self, output, key, required) {
 }
 
 const Oops$7 = issues({
-    BadSource: (model, property, got) => `Bad from-instruction provided to ${model}.${property}. Expects an arrow-function or a Model as source. Got ${got}.`,
+    // BadSource: (model, property, got) =>
+    //   `Bad from-instruction provided to ${model}.${property}. Expects an arrow-function or a Model as source. Got ${got}.`,
     PeerNotAllowed: (model, property) => `Attempted to use an instruction result (probably use or tap) as computed source for ${model}.${property}. This is not possible.`,
     ComputeFailed: (model, key) => `Generating initial value for ${model}.${key} failed.`,
     NotReady: (model, key) => `Value ${model}.${key} value is not yet available.`,
@@ -768,8 +754,8 @@ function get(arg0, arg1, arg2) {
         // Easy mistake, using a peer, will always be unresolved.
         if (typeof arg0 == "symbol")
             throw Oops$7.PeerNotAllowed(subject, key);
-        if (typeof arg0 == "function" && /^[A-Z]/.test(arg0.name))
-            throw Oops$7.BadSource(subject, key, arg0);
+        // if(typeof arg0 == "function" && /^[A-Z]/.test(arg0.name))
+        //   throw Oops.BadSource(subject, key, arg0);
         this.state.set(key, undefined);
         const init = () => {
             let source = this;
@@ -796,7 +782,7 @@ function get(arg0, arg1, arg2) {
                 throw new Error(`Factory argument cannot be ${arg1}`);
             getter = computeMode(this, source, setter, key, computeRequired);
         };
-        if (typeof arg0 == "function" && factoryRequired)
+        if (typeof arg0 == "function" && computeRequired)
             try {
                 init();
             }
@@ -812,18 +798,28 @@ function get(arg0, arg1, arg2) {
     });
 }
 
-function ref(callback) {
+function ref(arg) {
     return apply(function ref(key) {
-        const value = this.ref(key, callback && createValueEffect(callback));
-        defineProperty(value, "current", {
-            set: value,
-            get: () => this.state.get(key)
-        });
-        return {
-            value,
-            explicit: true
-        };
+        let value = {};
+        if (typeof arg == "object") {
+            const source = ensure(arg);
+            for (const key of source.state.keys())
+                defineProperty(value, key, {
+                    value: createRef(this, key)
+                });
+        }
+        else
+            value = createRef(this, key, arg);
+        return { value, explicit: true };
     });
+}
+function createRef(src, key, cb) {
+    const refObjectFunction = src.ref(key, cb && createValueEffect(cb));
+    defineProperty(refObjectFunction, "current", {
+        set: refObjectFunction,
+        get: () => src.state.get(key)
+    });
+    return refObjectFunction;
 }
 
 function set(value, argument) {
@@ -998,9 +994,11 @@ function use$1(input, argument) {
         let pending;
         let error;
         let get = (_local) => state.get(key);
-        if (typeof input === "function")
-            input = /^[A-Z]/.test(input.name)
-                ? new input() : input;
+        if (typeof input === "function") {
+            if ("prototype" in input &&
+                input === input.prototype.constructor)
+                input = new input();
+        }
         else if (input && typeof input !== "object")
             throw Oops$6.BadArgument(typeof input);
         if (input instanceof Map || input instanceof Set)
@@ -1469,11 +1467,11 @@ const Oops = issues({
 const Active = new WeakMap();
 class Global extends MVC {
     static create(...args) {
-        const Type = this;
-        if (Active.has(Type))
+        if (Active.has(this))
             throw Oops.AlreadyExists(this.name);
-        const instance = super.create(...args);
-        Active.set(Type, instance);
+        const instance = new this(...args);
+        ensure(instance);
+        Active.set(this, instance);
         return instance;
     }
     /**