From 4a7877c4336b0c16de60fc04862e6e02d2378aa3 Mon Sep 17 00:00:00 2001
From: Yanick Champoux <yanick@babyl.ca>
Date: Tue, 1 Mar 2022 19:14:01 -0500
Subject: [PATCH] identification store

---
 .storybook/main.js                            |   1 +
 fake/app/env.js                               |   1 +
 src/dux/index.js                              | 138 +++++++++---------
 src/lib/components/Field/index.svelte         |   2 +-
 .../Identification.stories.svelte             |   8 +-
 .../ShipEdit/Identification/index.svelte      |  29 ++--
 .../ShipEdit/Identification/shipTypes.js      |   5 +-
 .../ShipEdit/ShipEdit.stories.svelte          |  21 +++
 src/lib/components/ShipEdit/index.svelte      |  12 ++
 .../ShipSpecs/Identification.svelte           |  49 -------
 src/lib/shipDux/engine.js                     |  36 ++---
 src/lib/shipDux/identification.js             |  20 +++
 src/lib/shipDux/index.js                      |  17 ++-
 src/lib/shipDux/reqs.js                       |   9 +-
 src/lib/store/ship.js                         |  32 ++++
 15 files changed, 212 insertions(+), 168 deletions(-)
 create mode 100644 fake/app/env.js
 create mode 100644 src/lib/components/ShipEdit/ShipEdit.stories.svelte
 create mode 100644 src/lib/components/ShipEdit/index.svelte
 delete mode 100644 src/lib/components/ShipSpecs/Identification.svelte
 create mode 100644 src/lib/shipDux/identification.js
 create mode 100644 src/lib/store/ship.js

diff --git a/.storybook/main.js b/.storybook/main.js
index 144e24b..1a51bb1 100644
--- a/.storybook/main.js
+++ b/.storybook/main.js
@@ -27,6 +27,7 @@ module.exports = {
           $app: path.resolve("./fake/app"),
           "\\$lib": path.resolve(__dirname, "../src/lib/"),
           $lib: path.resolve(__dirname, "../src/lib/"),
+          "\\$app/env": path.resolve(__dirname, "../fake/app/env.js"),
         },
       },
     };
diff --git a/fake/app/env.js b/fake/app/env.js
new file mode 100644
index 0000000..43dac6b
--- /dev/null
+++ b/fake/app/env.js
@@ -0,0 +1 @@
+export const browser = true;
diff --git a/src/dux/index.js b/src/dux/index.js
index 8d582ef..204169d 100644
--- a/src/dux/index.js
+++ b/src/dux/index.js
@@ -5,15 +5,15 @@ import { createSelector } from "reselect";
 
 import ftl from "./ftl";
 import engine, { calc_drive_reqs } from "./engine";
-import weaponry from './weaponry';
+import weaponry from "./weaponry";
 import { calc_ftl_reqs } from "./ftl/rules";
 import { calc_ship_req } from "./utils";
-import { candidate_ship_types } from './ship_types';
-import structure from './structure';
-import cargo from './cargo';
-import streamlining from './streamlining';
-import carrier from './carrier';
-import { ceil } from './utils';
+import { candidate_ship_types } from "./ship_types";
+import structure from "./structure";
+import cargo from "./cargo";
+import streamlining from "./streamlining";
+import carrier from "./carrier";
+import { ceil } from "./utils";
 
 const set_ship_mass = action("set_ship_mass", payload());
 const set_name = action("set_name", payload());
@@ -21,29 +21,29 @@ const set_name = action("set_name", payload());
 const set_ship_reqs = action("set_ship_reqs", payload());
 const set_hull = action("set_hull", payload());
 
-const set_ship_type = action('set_ship_type',payload());
+const set_ship_type = action("set_ship_type", payload());
 
-const reset = action('reset' );
+const reset = action("reset");
 
 const initial = {
-    general: {
-      ship_class: "",
-      name: "",
-      ship_type: "",
-      mass: 10,
-      used_mass: 0,
-      cost: 10,
-    },
-  };
+  general: {
+    ship_class: "",
+    name: "",
+    ship_type: "",
+    mass: 10,
+    used_mass: 0,
+    cost: 10,
+  },
+};
 
 const dux = new Updux({
   subduxes: { ftl, engine, weaponry, structure, cargo, streamlining, carrier },
-  initial
+  initial,
 });
 
-dux.addMutation( reset, () => () => initial );
+dux.addMutation(reset, () => () => initial);
 
-dux.addMutation(set_hull, ({rating}) => (state) => {
+dux.addMutation(set_hull, ({ rating }) => (state) => {
   return u.updateIn("structure.hull", {
     cost: 2 * rating,
     rating,
@@ -53,8 +53,8 @@ dux.addMutation(set_hull, ({rating}) => (state) => {
 dux.addMutation(set_ship_mass, (mass) => u.updateIn("general", { mass }));
 dux.addMutation(set_name, (name) => u.updateIn("general", { name }));
 
-dux.addMutation( action('set_ship_class',payload() ),
-    ship_class => u.updateIn('general',{ship_class})
+dux.addMutation(action("set_ship_class", payload()), (ship_class) =>
+  u.updateIn("general", { ship_class })
 );
 
 dux.addMutation(set_ship_reqs, ({ cost, mass: used_mass }) =>
@@ -68,71 +68,73 @@ dux.addSubscription((store) =>
 
 dux.addSubscription((store) =>
   createSelector(
-      store => store.general.mass,
-      store => store.streamlining.type,
-      (ship_mass, streamlining ) => {
-            const mass = ceil( ship_mass * (
-                streamlining === 'none' ? 0
-                : streamlining === 'partial' ? 5 : 10
-            ) / 100 );
-            const cost = 2 * mass;
+    (store) => store.general.mass,
+    (store) => store.streamlining.type,
+    (ship_mass, streamlining) => {
+      const mass = ceil(
+        (ship_mass *
+          (streamlining === "none" ? 0 : streamlining === "partial" ? 5 : 10)) /
+          100
+      );
+      const cost = 2 * mass;
 
-            store.dispatch( dux.actions.set_streamlining_cost_mass({cost,mass}) );
-      }
+      store.dispatch(dux.actions.set_streamlining_cost_mass({ cost, mass }));
+    }
   )
 );
 
 dux.addSubscription((store) =>
-    createSelector(
-        store => store.general.mass,
-        store => store.general.ship_type,
-        store => store.carrier.bays,
-        (mass,type,bays) => {
-            console.log({bays});
-            const candidates = candidate_ship_types(mass,bays > 0);
+  createSelector(
+    (store) => store.general.mass,
+    (store) => store.general.ship_type,
+    (store) => store.carrier.bays,
+    (mass, type, bays) => {
+      console.log({ bays });
+      const candidates = candidate_ship_types(mass, bays > 0);
 
-            console.log({candidates});
-            if( candidates.length === 0 ) return;
+      console.log({ candidates });
+      if (candidates.length === 0) return;
 
-            if( candidates.find( ({name}) => name === type ) ) return;
+      if (candidates.find(({ name }) => name === type)) return;
 
-            store.dispatch(
-                store.actions.set_ship_type(
-                    candidates[0].name
-                )
-            )
-
-        }
-    )
+      store.dispatch(store.actions.set_ship_type(candidates[0].name));
+    }
+  )
 );
 
-dux.addMutation(set_ship_type, type => u.updateIn('general.ship_type',type) );
+dux.addMutation(set_ship_type, (type) => u.updateIn("general.ship_type", type));
 
 dux.addSubscription((store) =>
   createSelector(
     [(ship) => ship.general.mass, (ship) => ship.ftl.type],
     (ship_mass, type) =>
-      store.dispatch(ftl.actions.set_ftl_reqs(calc_ftl_reqs(type,ship_mass)))
+      store.dispatch(ftl.actions.set_ftl_reqs(calc_ftl_reqs(type, ship_mass)))
   )
 );
 
-dux.addSubscription( store => createSelector(
-    ship => ship.general.mass,
-    ship => ship.structure.screens.standard,
-    ship => ship.structure.screens.advanced,
-    (mass,standard,advanced) => {
-        console.log({
-            mass, standard, advanced
-        })
-        const standard_mass = standard * Math.max(3,ceil( 0.05 * mass ));
-        const advanced_mass = advanced * Math.max(4,ceil( 0.075 * mass ));
+dux.addSubscription((store) =>
+  createSelector(
+    (ship) => ship.general.mass,
+    (ship) => ship.structure.screens.standard,
+    (ship) => ship.structure.screens.advanced,
+    (mass, standard, advanced) => {
+      console.log({
+        mass,
+        standard,
+        advanced,
+      });
+      const standard_mass = standard * Math.max(3, ceil(0.05 * mass));
+      const advanced_mass = advanced * Math.max(4, ceil(0.075 * mass));
 
-        store.dispatch( dux.actions.set_screens_reqs({
-            mass: standard_mass + advanced_mass,
-            cost: 3 * standard_mass + 4 * advanced_mass
-        }));
+      store.dispatch(
+        dux.actions.set_screens_reqs({
+          mass: standard_mass + advanced_mass,
+          cost: 3 * standard_mass + 4 * advanced_mass,
+        })
+      );
     }
-));
+  )
+);
 
 dux.addSubscription((store) =>
   createSelector(
diff --git a/src/lib/components/Field/index.svelte b/src/lib/components/Field/index.svelte
index 71657d3..764bd93 100644
--- a/src/lib/components/Field/index.svelte
+++ b/src/lib/components/Field/index.svelte
@@ -3,7 +3,7 @@
     <label>{label}</label>
   {/if}
   <slot>
-    <input type="text" {placeholder} {value} on:change />
+    <input type="text" {placeholder} bind:value on:change />
   </slot>
 </div>
 
diff --git a/src/lib/components/ShipEdit/Identification/Identification.stories.svelte b/src/lib/components/ShipEdit/Identification/Identification.stories.svelte
index 49bf570..67ecb68 100644
--- a/src/lib/components/ShipEdit/Identification/Identification.stories.svelte
+++ b/src/lib/components/ShipEdit/Identification/Identification.stories.svelte
@@ -1,9 +1,4 @@
-<Meta
-  title="Identification"
-  component={Identification}
-  argTypes={{
-  }}
-/>
+<Meta title="Identification" component={Identification} argTypes={{}} />
 
 <Story name="Primary" args={{}} />
 
@@ -24,5 +19,4 @@
   setContext("ship", {
     dispatch: (type, detail) => action(type)(detail),
   });
-
 </script>
diff --git a/src/lib/components/ShipEdit/Identification/index.svelte b/src/lib/components/ShipEdit/Identification/index.svelte
index 0c57162..6d79133 100644
--- a/src/lib/components/ShipEdit/Identification/index.svelte
+++ b/src/lib/components/ShipEdit/Identification/index.svelte
@@ -1,11 +1,7 @@
 <div>
-  <Field
-    label="ship class"
-    value={shipClass}
-  />
-
+  <Field label="ship class" bind:value={shipClass} />
   <Field label="ship type">
-    <select value={shipType}>
+    <select bind:value={shipType}>
       {#each shipTypes as name (name)}
         <option>{name}</option>
       {/each}
@@ -14,20 +10,25 @@
 </div>
 
 <script>
+  import { getContext } from "svelte";
+
   import Field from "$lib/components/Field/index.svelte";
-    import { candidateShipTypes } from './shipTypes.js';
+  import { candidateShipTypes } from "./shipTypes.js";
 
-    export let shipClass = '';
-    export let shipType = '';
-    export let mass = 10;
-    export let isCarrier = false;
+  export let shipClass = "";
+  export let shipType = "";
+  export let mass = 10;
+  export let isCarrier = false;
 
-    $: shipTypes = candidateShipTypes(mass,isCarrier).map( ({name})=>name );
-
-    $: if( shipTypes.length >0 &&  !shipTypes.includes(shipType)) shipType= shipTypes[0];
+  const ship = getContext("ship");
 
+  $: shipTypes = candidateShipTypes(mass, isCarrier).map(({ name }) => name);
 
+  $: if (shipTypes.length > 0 && !shipTypes.includes(shipType))
+    shipType = shipTypes[0];
 
+  $: ship.dispatch.setShipType(shipType);
+  $: ship.dispatch.setShipClass(shipClass);
 </script>
 
 <style>
diff --git a/src/lib/components/ShipEdit/Identification/shipTypes.js b/src/lib/components/ShipEdit/Identification/shipTypes.js
index 038da4f..be32552 100644
--- a/src/lib/components/ShipEdit/Identification/shipTypes.js
+++ b/src/lib/components/ShipEdit/Identification/shipTypes.js
@@ -21,5 +21,8 @@ const ship_types = [
 ];
 
 export function candidateShipTypes(mass = 0, carrier = false) {
-  return ship_types.filter((c) => carrier == !!c.carrier).filter((c) => c.mass[0] <= mass).filter((c) => c.mass[1] >= mass);
+  return ship_types
+    .filter((c) => carrier == !!c.carrier)
+    .filter((c) => c.mass[0] <= mass)
+    .filter((c) => c.mass[1] >= mass);
 }
diff --git a/src/lib/components/ShipEdit/ShipEdit.stories.svelte b/src/lib/components/ShipEdit/ShipEdit.stories.svelte
new file mode 100644
index 0000000..c1e743b
--- /dev/null
+++ b/src/lib/components/ShipEdit/ShipEdit.stories.svelte
@@ -0,0 +1,21 @@
+<Meta title="ShipEdit" component={ShipEdit} argTypes={{}} />
+
+<Story name="Primary" args={{}} />
+
+<Template let:args>
+  <div style="width: 50em">
+    <ShipEdit />
+  </div>
+</Template>
+
+<script>
+  import { Meta, Template, Story } from "@storybook/addon-svelte-csf";
+  import { action } from "@storybook/addon-actions";
+
+  import { setContext } from "svelte";
+
+  import ShipEdit from "./index.svelte";
+  import shipStore from "$lib/store/ship.js";
+
+  setContext("ship", shipStore());
+</script>
diff --git a/src/lib/components/ShipEdit/index.svelte b/src/lib/components/ShipEdit/index.svelte
new file mode 100644
index 0000000..841b7c7
--- /dev/null
+++ b/src/lib/components/ShipEdit/index.svelte
@@ -0,0 +1,12 @@
+<Identification {...$shipState.identification} {...$shipState.reqs} />
+
+<script>
+  import { getContext } from "svelte";
+
+  import Identification from "./Identification/index.svelte";
+
+  const { state: shipState } = getContext("ship");
+</script>
+
+<style>
+</style>
diff --git a/src/lib/components/ShipSpecs/Identification.svelte b/src/lib/components/ShipSpecs/Identification.svelte
deleted file mode 100644
index 7180a1e..0000000
--- a/src/lib/components/ShipSpecs/Identification.svelte
+++ /dev/null
@@ -1,49 +0,0 @@
-<div>
-  <Field
-    label="ship class"
-    value={general.ship_class}
-    on:change={change_class}
-  />
-
-  <Field label="ship type">
-    <select value={ship_type} on:change={change_ship_type}>
-      {#each ship_types as type (type)}
-        <option>{type}</option>
-      {/each}
-    </select>
-  </Field>
-</div>
-
-<script>
-  import { getContext } from "svelte";
-
-  import Field from "$lib/components/Field/index.svelte";
-  import { candidate_ship_types } from "../../dux/ship_types";
-
-  export let ship = getContext("ship");
-  let general;
-  $: general = $ship.general;
-
-  const change_class = (event) =>
-    ship.dispatch(ship.actions.set_ship_class(event.target.value));
-
-  let ship_type;
-  $: ship_type = $ship.general.ship_type;
-
-  const change_ship_type = ({ target: { value } }) =>
-    ship.dispatch.set_ship_type(value);
-
-  let ship_types;
-  $: ship_types = candidate_ship_types(
-    $ship.general.mass,
-    $ship.carrier.bays > 0
-  ).map(({ name }) => name);
-</script>
-
-<style>
-  div {
-    display: flex;
-    align-items: end;
-    gap: 2em;
-  }
-</style>
diff --git a/src/lib/shipDux/engine.js b/src/lib/shipDux/engine.js
index ed14366..d0d4d70 100644
--- a/src/lib/shipDux/engine.js
+++ b/src/lib/shipDux/engine.js
@@ -1,28 +1,28 @@
-import {Updux} from "updux";
-import u from 'updeep';
+import { Updux } from "updux";
+import u from "updeep";
 
-import reqs from './reqs.js';
+import reqs from "./reqs.js";
 
 const dux = new Updux({
-    subduxes: { reqs },
-    initial: {
-        rating: 1,
-        advanced: false,
-    },
-    actions: {
-        setEngine: null,
-        setEngineReqs: null,
-    }
+  subduxes: { reqs },
+  initial: {
+    rating: 1,
+    advanced: false,
+  },
+  actions: {
+    setEngine: null,
+    setEngineReqs: null,
+  },
 });
 
-dux.addMutation('setEngine', changes => u(changes) );
-dux.addMutation('setEngineReqs', reqs => u({reqs}) );
+dux.setMutation("setEngine", (changes) => u(changes));
+dux.setMutation("setEngineReqs", (reqs) => u({ reqs }));
 
-export function calcDriveReqs(shipMass,rating,advanced=false) {
-    const mass = Math.ceil(rating * 0.05 * shipMass);
-    const cost = mass * ( advanced ? 3 : 2 );
+export function calcDriveReqs(shipMass, rating, advanced = false) {
+  const mass = Math.ceil(rating * 0.05 * shipMass);
+  const cost = mass * (advanced ? 3 : 2);
 
-    return { mass, cost };
+  return { mass, cost };
 }
 
 export default dux;
diff --git a/src/lib/shipDux/identification.js b/src/lib/shipDux/identification.js
new file mode 100644
index 0000000..50abc17
--- /dev/null
+++ b/src/lib/shipDux/identification.js
@@ -0,0 +1,20 @@
+import { Updux } from "updux";
+import u from "updeep";
+
+const dux = new Updux({
+  actions: {
+    setShipType: null,
+    setShipClass: null,
+  },
+  initial: {
+    shipType: "",
+    shipClass: "",
+    isCarrier: false,
+    mass: 10,
+  },
+});
+
+dux.setMutation("setShipType", (shipType) => u({ shipType }));
+dux.setMutation("setShipClass", (shipClass) => u({ shipClass }));
+
+export default dux;
diff --git a/src/lib/shipDux/index.js b/src/lib/shipDux/index.js
index 1b63cb1..755e3e9 100644
--- a/src/lib/shipDux/index.js
+++ b/src/lib/shipDux/index.js
@@ -1,12 +1,17 @@
-import Updux from "updux";
+import { Updux } from "updux";
 
-import engine from './engine.js';
+import engine from "./engine.js";
+import identification from "./identification.js";
+import reqs from "./reqs.js";
 
 const dux = new Updux({
-    subduxes: {
-        engine,
-    },
-
+  subduxes: {
+    identification,
+    engine,
+  },
+  initial: {
+    reqs: { cost: 0, mass: 10 },
+  },
 });
 
 export default dux;
diff --git a/src/lib/shipDux/reqs.js b/src/lib/shipDux/reqs.js
index 171e1a0..62ea2dd 100644
--- a/src/lib/shipDux/reqs.js
+++ b/src/lib/shipDux/reqs.js
@@ -1,9 +1,10 @@
-import {Updux} from "updux";
+import { Updux } from "updux";
 
 const dux = new Updux({
-    initial: {
-        cost: 0, mass: 0
-    },
+  initial: {
+    cost: 0,
+    mass: 0,
+  },
 });
 
 export default dux;
diff --git a/src/lib/store/ship.js b/src/lib/store/ship.js
new file mode 100644
index 0000000..dcfbe80
--- /dev/null
+++ b/src/lib/store/ship.js
@@ -0,0 +1,32 @@
+import { browser } from "$app/env";
+import { readable, get } from "svelte/store";
+import { compose, applyMiddleware } from "redux";
+
+import shipDux from "../shipDux/index.js";
+
+let composeEnhancers = compose;
+
+if (browser) {
+  composeEnhancers = window.__REDUX_DEVTOOLS_EXTENSION_COMPOSE__;
+}
+
+export default () => {
+  const duxStore = shipDux.createStore(undefined, (mw) =>
+    composeEnhancers(applyMiddleware(mw))
+  );
+
+  let previous;
+  const state = readable(duxStore.getState(), (set) => {
+    duxStore.subscribe(() => {
+      if (previous === duxStore.getState()) return;
+      previous = duxStore.getState();
+      console.log("Setting!", previous);
+      set(previous);
+    });
+  });
+
+  return {
+    dispatch: duxStore.dispatch,
+    state,
+  };
+};