LICENSE

@@ -1,21 +0,0 @@
-MIT License
-
-Copyright (c) 2021 Sorrel
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

packages/play-node-go/package-lock.json

@@ -8500,9 +8500,9 @@
       }
     },
     "lodash": {
-      "version": "4.17.19",
+      "version": "4.17.15",
-      "resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.19.tgz",
+      "resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.15.tgz",
-      "integrity": "sha512-JNvd8XER9GQX0v2qJgsaN/mzFCNA5BRe/j8JN9d+tWyGLSodKQHKFicdwNYzWwI3wjRnaKPsGj1XkBjx/F96DQ=="
+      "integrity": "sha512-8xOcRHvCjnocdS5cpwXQXVzmmh5e5+saE2QGoeQmbKmRS6J3VQppPOIt0MnmE+4xlZoumy0GPG0D0MVIQbNA1A=="
     },
     "lodash._reinterpolate": {
       "version": "3.0.0",

packages/play-node-go/public/reset.css

@@ -51,8 +51,3 @@ table {
 	border-collapse: collapse;
 	border-spacing: 0;
 }
-
-button {
-  background: none;
-  border: none;
-}

packages/play-node-go/src/components/Button/Guest/Guest.js

@@ -1,34 +0,0 @@
-import React from "react";
-import authServices from "../../../services/authServices";
-
-const Guest = ({ dispatch }) => {
-  const handleClick = async (e) => {
-    e.preventDefault();
-    // dispatch to guest endpoint
-    const guestResponse = await authServices.guestService();
-
-    if (guestResponse.errors) {
-      const authError = guestResponse.errors[0].auth;
-      return dispatch({
-        type: "ERR",
-        message: "AUTH_ERROR",
-        body: { authError },
-      });
-    }
-
-    return dispatch({
-      type: "AUTH",
-      message: "GUEST",
-      body: guestResponse,
-    });
-  };
-  return (
-    <>
-      <button className="nav__section__button" onClick={handleClick}>
-        Continue As Guest
-      </button>
-    </>
-  );
-};
-
-export default Guest;

packages/play-node-go/src/components/Form/Auth/Auth.js

@@ -1,49 +1,45 @@
-import React, { useState } from "react";
+import React, { useState } from 'react';
 
-import Login from "../Login/Login";
+import Login from '../Login/Login';
-import Signup from "../Signup/Signup";
+import Signup from '../Signup/Signup';
-import Guest from "../../Button/Guest/Guest";
 
 const Auth = (props) => {
-  const [showForm, setShowForm] = useState("login");
+  const [ showForm, setShowForm ] = useState('login')
   const { state, dispatch } = props;
 
   return (
     <>
       <div 
         className="nav__section nav__section--auth" 
-        onClick={() => {
+        onClick={()=>{setShowForm('login')}}
-          setShowForm("login");
-        }}
       >
-        <p className="nav__section__label">Login</p>
+        <p 
+          className="nav__section__label"
+        >Login</p>
 
-        {showForm === "login" ? (
+        {
-          <Login dispatch={dispatch} state={state} />
+          showForm === 'login' 
-        ) : (
+          ? <Login dispatch={dispatch} state={state}/> 
-          <></>
+          : <></>
-        )}
+        }
       </div>
 
       <div
         className="nav__section nav__section--auth" 
-        onClick={() => {
+        onClick={()=>{setShowForm('signup')}}
-          setShowForm("signup");
-        }}
       >
-        <p className="nav__section__label">Signup</p>
+        <p 
+          className="nav__section__label"
+        >Signup</p>
 
-        {showForm === "signup" ? (
+      {
-          <Signup dispatch={dispatch} state={state} />
+        showForm === 'signup' 
-        ) : (
+        ? <Signup dispatch={dispatch} state={state}/> 
-          <></>
+        : <></>
-        )}
+      }
-      </div>
-      <div className="nav__section nav__section--auth">
-        <Guest dispatch={dispatch} />
       </div>
     </>
   );
-};
+}
 
 export default Auth;

packages/play-node-go/src/pages/Layout/MainWrapper/MainWrapper.scss

@@ -1,5 +1,4 @@
 @import '../../../../public/stylesheets/partials/mixins';
-@import '../../../../public/stylesheets/partials/variables';
 
 div.main-wrapper {
   display: flex;
@@ -35,14 +34,3 @@ div.main-wrapper {
     }
   }
 }
-
-button {
-  color:map-get($colors, 'sidebar_link');
-  cursor: pointer;
-  font-family: inherit;
-  font-size: 110%;
-  font-weight: bold;
-  margin-bottom: .5em;
-  padding: 0;
-  text-decoration: none;
-}

packages/play-node-go/src/pages/Layout/NavBar/NavBar.js

@@ -1,36 +1,29 @@
-import React from "react";
+import React from 'react';
-import { Link } from "react-router-dom";
+import { Link } from 'react-router-dom';
-import "./NavBar.scss";
+import './NavBar.scss';
-import Logo from "../../../components/Display/Logo/Logo";
+import Logo from '../../../components/Display/Logo/Logo';
+
+const NavBar = (props) => {
 
-const NavBar = ({ state }) => {
   return (
     <div className="NavBar" data-testid="NavBar">
-      <Link to="/home">
+      <Link to="/home" >
-        <div className="NavBar__logo">
+        <div className="NavBar__logo"><Logo /></div>
-          <Logo />
-        </div>
       </Link>
 
-      <Link to="/home">
+      <Link to="/home" >
-        <div className="NavBar__menu-item NavBar__home">
+        <div className="NavBar__menu-item NavBar__home"><p className="--link">Find a Game</p></div>
-          <p className="--link">Find a Game</p>
-        </div>
       </Link>
       
       <Link to="/news">
-        <div className="NavBar__menu-item NavBar__news">
+        <div className="NavBar__menu-item NavBar__news"><p className="--link">News</p></div>
-          <p className="--link">News</p>
-        </div>
       </Link>
 
       <Link to="/account">
-        <div className="NavBar__menu-item NavBar__account">
+        <div className="NavBar__menu-item NavBar__account">{props.user ? props.user.username : <></>}</div>
-          {state.user.username ? state.user.username : <></>}
-        </div>
       </Link>
     </div>
   );
-};
+}
 
 export default NavBar;

packages/play-node-go/src/reducers/auth/reducer.auth.js

@@ -1,23 +1,20 @@
 export const authReducer = (state, action) => {
   switch (action.message) {
-    case "LOGIN":
+    case 'LOGIN':
       return loginReducer(state, action);
       
-    case "SIGNUP":
+    case 'SIGNUP':
       return loginReducer(state, action);
 
-    case "GUEST":
+    case 'LOGOUT':
-      return loginReducer(state, action);
-
-    case "LOGOUT":
       return state;
 
     default:
       return state;
   } 
-};
+}
 
 function loginReducer(state, action) {
   const newUser = action.body;
-  return { ...state, user: newUser };
+  return {...state, user: newUser };  
 }

packages/play-node-go/src/services/authServices.js

@@ -1,58 +1,43 @@
-import config from "../config";
+import config from '../config';
 
 const authEndpoint = config.authAddress;
-const signupEndpoint = `${authEndpoint}/signup`;
+const signupEndpoint = `${authEndpoint}/signup`
-const loginEndpoint = `${authEndpoint}/login`;
+const loginEndpoint = `${authEndpoint}/login`
-const guestEndpoint = `${authEndpoint}/guest`;
 
 var headers = new Headers();
-headers.append("Content-Type", "application/json");
+headers.append('Content-Type', 'application/json');
-headers.append("Accept", "application/json");
+headers.append('Accept', 'application/json');
-headers.append("Sec-Fetch-Site", "cross-site");
+headers.append('Sec-Fetch-Site', 'cross-site')
 
-const loginService = async (formData) => {
+const loginService = async(formData) => {
   const response = await fetch(loginEndpoint, {
-    method: "POST",
+    method: 'POST',
-    credentials: "include",
+    credentials: 'include',
     body: JSON.stringify(formData),
-    headers: headers,
+    headers: headers
   })
-    .then((res) => res.text())
+  .then(res => res.text())
-    .then((text) => JSON.parse(text))
+  .then(text => JSON.parse(text))
-    .catch((err) => err);
+  .catch(err => err);
   
   return response;
-};
+}
 
 const signupService = async (formData) => {
   const response = await fetch(signupEndpoint, {
-    method: "POST",
+    method: 'POST',
-    credentials: "include",
+    credentials: 'include',
     body: JSON.stringify(formData),
-    headers: headers,
+    headers: headers
   })
-    .then((res) => res.text())
+  .then(res => res.text())
-    .then((text) => JSON.parse(text))
+  .then(text => JSON.parse(text))
-    .catch((err) => err);
+  .catch(err => err);
 
   return response;
-};
+}
-
-const guestService = async () => {
-  const response = await fetch(guestEndpoint, {
-    method: "POST",
-    credentials: "include",
-    headers,
-  })
-    .then((res) => res.text())
-    .then((text) => JSON.parse(text))
-    .catch((err) => err);
-
-  return response;
-};
 
 export default {
   loginService,
-  signupService,
+  signupService
-  guestService,
+}
-};

packages/server/controllers/auth.js

@@ -1,16 +1,15 @@
-const { validationResult } = require("express-validator");
+const { validationResult } = require('express-validator');
 
-const userQueries = require("../data/queries/user");
+const userQueries = require('../data/queries/user');
-const { hashPassword, compareHash } = require("../services/bcrypt");
+const { hashPassword, compareHash } = require('../services/bcrypt');
-const signToken = require("../services/signToken");
+const signToken = require('../services/signToken');
-const guestServices = require("../services/guestServices");
 
 const checkValidationErrors = (req, res) => {
   const errors = validationResult(req);
   if (!errors.isEmpty()) {
     return res.status(422).json({ errors: errors.array() });
   }
-};
+}
 
 const signup = async (req, res, next) => {
   checkValidationErrors(req, res);
@@ -21,18 +20,18 @@ const signup = async (req, res, next) => {
     const hashedPassword = await hashPassword(user.password);
     const secureUser = { ...user, password: hashedPassword };
     if (existingUser.length) {
-      return res
+      return res.status(409).json({errors: [{auth: 'User already exists!'}]})
-        .status(409)
-        .json({ errors: [{ auth: "User already exists!" }] });
     }
 
     const newUser = await userQueries.insertUser(secureUser);
-    signToken(res, newUser);
+    signToken(res, newUser)
-    res.status(201).json({ ...newUser });
+    res.status(201).json({...newUser});
-  } catch (err) {
+  } 
+
+  catch (err) {
     res.status(500).json(err);
   }
-};
+}
 
 const login = async (req, res, next) => {
   checkValidationErrors(req, res);
@@ -43,46 +42,29 @@ const login = async (req, res, next) => {
     const savedUser = queryResults[0] || null;
     
     if (!savedUser) {
-      return res.status(401).send({ errors: "bad credentials" });
+      return res.status(401).send({errors: 'bad credentials'});
     }
     
     const hashedPassword = savedUser.password;
     const passwordMatch = await compareHash(user.password, hashedPassword);
 
     if (!passwordMatch) {
-      return res.status(401).send({ errors: "bad credentials" });
+      return res.status(401).send({errors: 'bad credentials'});
     }
     
-    const authorizedUser = { ...savedUser };
+    const authorizedUser = {...savedUser};
     delete authorizedUser.password;
     
     signToken(res, authorizedUser);
-    res.send({ ...authorizedUser }).status(200);
+    res.send({...authorizedUser}).status(200);
-  } catch (e) {
-    res.status(500).send({ errors: e });
   }
-};
   
-const guest = async (req, res, next) => {
+  catch (err) {
-  try {
+    res.status(500).send({errors: err});
-    // username generator returns `Guest-${num}`
-    const { username, password } = guestServices.generateGuest();
-    // generateGuestUser();
-    const email = null;
-    // id generator returns `
-    const id = null;
-    const user = { username, email, id, password };
-    signToken(res, user);
-    delete user.password;
-    res.send(user);
-  } catch (e) {
-    console.log(e);
-    res.status(500).send({ errors: e });
   }
-};
+}
 
 module.exports = {
   signup,
-  login,
+  login
-  guest,
+}
-};

packages/server/data/migrations/20200120224032_game.js

@@ -1,81 +1,49 @@
-const winType = ["B+R", "B+", "B+T", "W+R", "W+", "W+T", "0", "Void", "?"];
+const winType = [
+  'B+R', 'B+', 'B+T',
+  'W+R', 'W+', 'W+T',
+  '0', 'Void', '?'
+]
 
 const rankArray = [
-  "D9",
+  'D9', 'D8', 'D7', 'D6', 'D5', 'D4', 'D3', 'D2', 'D1',
-  "D8",
+  'K1', 'K2', 'K3', 'K4', 'K5', 'K6', 'K7', 'K8', 'K9', 'K10',
-  "D7",
+  'K11', 'K12', 'K13', 'K14', 'K15', 'K16', 'K17', 'K18', 'K19', 'K20',
-  "D6",
+  'K21', 'K22', 'K23', 'K24', 'K25', 'K26', 'K27', 'K28', 'K29', 'K30', 'UR'
-  "D5",
+]
-  "D4",
-  "D3",
-  "D2",
-  "D1",
-  "K1",
-  "K2",
-  "K3",
-  "K4",
-  "K5",
-  "K6",
-  "K7",
-  "K8",
-  "K9",
-  "K10",
-  "K11",
-  "K12",
-  "K13",
-  "K14",
-  "K15",
-  "K16",
-  "K17",
-  "K18",
-  "K19",
-  "K20",
-  "K21",
-  "K22",
-  "K23",
-  "K24",
-  "K25",
-  "K26",
-  "K27",
-  "K28",
-  "K29",
-  "K30",
-  "UR",
-];
 
-exports.up = function (knex) {
+exports.up = function(knex) {
-  return knex.schema.createTable("game", (table) => {
+  return knex.schema.createTable("game", table => {
-    table.increments("id").primary();
+    table.increments('id').primary();
-    table.datetime("date");
+    table.datetime('date');
-    table.float("komi").default(6.5);
+    table.float('komi').default(6.5);
-    table.integer("handicap").default(0);
+    table.integer('handicap').default(0);
-    table.integer("board_size").default(19);
+    table.integer('board_size').default(19);
-    table.boolean("open").default(true);
+    table.boolean('open').default(true);
 
-    table.string("application");
+    table.string('application');
-    table.string("application_version");
+    table.string('application_version');
     table.timestamps(true, true);
 
-    table.string("player_black");
+    table.string('player_black');
-    table.string("player_white");
+    table.string('player_white');
-    table.enu("player_black_rank", rankArray).default("UR");
+    table.enu('player_black_rank', rankArray).default('UR');
-    table.enu("player_white_rank", rankArray).default("UR");
+    table.enu('player_white_rank', rankArray).default('UR');
     
-    table.string("event");
+    table.string('event');
-    table.string("name");
+    table.string('name');
-    table.string("description");
+    table.string('description');
-    table.integer("round");
+    table.integer('round');
     
-    table.enu("win_type", winType);
+    table.enu('win_type', winType);
-    table.float("score");
+    table.float('score');
-    table.integer("captures_black");
+    table.integer('captures_black');
-    table.integer("captures_white");
+    table.integer('captures_white');
 
-    table.integer("user_black").references("id").inTable("user");
+    table.integer('user_black').references('id').inTable('user');
-    table.integer("user_white").references("id").inTable("user");
+    table.integer('user_white').references('id').inTable('user');
-    table.integer("room").references("id").inTable("room");
+    table.integer('room').references('id').inTable('room');
-    table.integer("time_setting").references("id").inTable("time_setting");
+    table.integer('time_setting').references('id').inTable('time_setting');
-  });
+  })
 };
 
-exports.down = (knex) => knex.schema.dropTableIfExists("game");
+exports.down = knex => knex.schema.dropTableIfExists("game");

packages/server/data/migrations/20200120230921_move.js

@@ -1,18 +1,17 @@
-const players = ["white", "black"];
+const players = ['white', 'black']
 
-exports.up = (knex) => {
+exports.up = knex => {
-  return knex.schema.createTable("move", (table) => {
+  return knex.schema.createTable("move", table => {
-    table.increments("id").primary();
+    table.increments('id').primary();
-    table.enu("player", players).notNullable();
+    table.enu('player', players).notNullable();
-    table.integer("point_x").notNullable();
+    table.integer('point_x').notNullable();
-    table.integer("point_y").notNullable();
+    table.integer('point_y').notNullable();
-    table.integer("number").notNullable();
+    table.integer('number').notNullable();
-    table.boolean("game_record").notNullable().default(true);
+    table.boolean('game_record').notNullable().default(true);
-    table.boolean("placement").notNullable().default(false);
 
-    table.integer("game").references("id").inTable("game").notNullable();
+    table.integer('game').references('id').inTable('game').notNullable();
-    table.integer("prior_move").references("id").inTable("move");
+    table.integer('prior_move').references('id').inTable('move');
   });
 };
 
-exports.down = (knex) => knex.schema.dropTableIfExists("move");
+exports.down = knex => knex.schema.dropTableIfExists("move");

packages/server/data/queries/game.js

@@ -70,23 +70,17 @@ const findGameByRoom = async (roomId) => {
 
 const insertGame = async (game) => {};
 
-const endGame = async ({ id, winType, score, bCaptures, wCaptures }) => {
+const endGame = async ({ id }) => {
-  try {
+  const game = await knex(game).where({ id: id }).update(
-    const game = await knex
+    {
-      .from("game")
-      .returning(gameDetailSelect)
-      .where({ id: id })
-      .update({
       win_type: winType,
       score: score,
-        captures_black: bCaptures,
+      captures_black: capturesBlack,
-        captures_white: wCaptures,
+      captures_white: capturesWhite,
-        open: false,
-      });
-    return game;
-  } catch (e) {
-    return e;
     }
+    // ["id"]
+  );
+  return game;
 };
 
 module.exports = {

packages/server/data/queries/move.js

@@ -1,47 +1,36 @@
-const knex = require("../db");
+const knex = require('../db');
 
 const findGameRecord = async (gameId) => {
-  return await knex("move")
+  return await knex('move')
-    .where({ game: gameId, game_record: true })
+  .where({ 'game': gameId, 'game_record': true })
-    .select("player", "point_x", "point_y", "number", "prior_move", "placement")
+  .select('player', 'point_x', 'point_y', 'number')
-    .orderBy("number")
+  .orderBy('number')
-    .then((record) =>
+  .then(record => record.map(({player, point_x, point_y}) => ({player, pos: { x: point_x, y: point_y } }) ))
-      record.map(({ player, point_x, point_y }) => ({
-        player,
-        pos: { x: point_x, y: point_y },
-      }))
-    );
   // .then(res => res)
-};
+}
 
 // id: 1, player: 'black', point_x: 3,  point_y: 3, number: 1, game_record: true, game: 1, prior_move: null
 
 const addMove = async ({ gameId, player, x, y, gameRecord, priorMove }) => {
-  // ! priorMove must be FK not move number
   const number = priorMove + 1;
   let result;
   try { 
-    result = await knex("move")
+  result = await knex('move')
-      .returning("*")
+    .returning('*')
-      .insert({
+    .insert({ game: gameId, player, point_x: x, point_y: y, number, game_record: gameRecord, prior_move: priorMove  })
-        game: gameId,
+    .then(res => res);
-        player,
+  }
-        point_x: x,
+  catch (e) {
-        point_y: y,
+    result = e
-        number,
+  }
-        game_record: gameRecord,
+  finally {
-        prior_move: priorMove,
-      })
-      .then((res) => res);
-  } catch (e) {
-    result = e;
-  } finally {
     console.log(result);
     return result;
   }
-};
+  
+}
 
 module.exports = {
   findGameRecord,
-  addMove,
+  addMove
-};
+}

packages/server/package-lock.json

@@ -1359,9 +1359,9 @@
       }
     },
     "lodash": {
-      "version": "4.17.19",
+      "version": "4.17.15",
-      "resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.19.tgz",
+      "resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.15.tgz",
-      "integrity": "sha512-JNvd8XER9GQX0v2qJgsaN/mzFCNA5BRe/j8JN9d+tWyGLSodKQHKFicdwNYzWwI3wjRnaKPsGj1XkBjx/F96DQ=="
+      "integrity": "sha512-8xOcRHvCjnocdS5cpwXQXVzmmh5e5+saE2QGoeQmbKmRS6J3VQppPOIt0MnmE+4xlZoumy0GPG0D0MVIQbNA1A=="
     },
     "lodash.includes": {
       "version": "4.3.0",

packages/server/routes/auth.js

@@ -1,20 +1,10 @@
-const express = require("express");
+const express = require('express');
 const router = express.Router();
-const app = require("../server");
+const app = require('../server');
-const authController = require("../controllers/auth");
+const authController = require('../controllers/auth');
-const {
+const { signupValidationRules, loginValidationRules, validate } = require('../middleware/userValidator');
-  signupValidationRules,
-  loginValidationRules,
-  validate,
-} = require("../middleware/userValidator");
 
-router.post(
+router.post('/signup', signupValidationRules(), validate, authController.signup);
-  "/signup",
+router.post('/login', loginValidationRules(), validate, authController.login);
-  signupValidationRules(),
-  validate,
-  authController.signup
-);
-router.post("/login", loginValidationRules(), validate, authController.login);
-router.post("/guest", authController.guest);
 
 module.exports = router;

packages/server/services/Game.js

@@ -181,18 +181,13 @@ const Game = ({ gameData = {}, gameRecord = [] } = {}) => {
       this.kos = [];
     },
   };
+
   if (gameRecord.length) {
     // play through all the moves
-    const game = gameRecord.reduce(
+    return gameRecord.reduce(
       (game, move) => game.makeMove(move),
       Game({ gameData }).initGame()
     );
-    // ? why is this being wrapped?
-    if (gameData && gameData.gameData && gameData.gameData.winner) {
-      const { winner, score } = gameData.gameData;
-      return game.manualEnd({ winner, score });
-    }
-    return game;
   }
   return {
     winner: gameData.winner || null,
@@ -244,58 +239,28 @@ const Game = ({ gameData = {}, gameRecord = [] } = {}) => {
       };
     },
 
-    checkMove: function ({ player, pos: { x, y } }) {
-      // if game is over
-      // TODO either change logic here or add additional method for handling moves off of record
-      if (this.pass > 1) {
-        return { ...this, success: false };
-      }
-      const point = this.boardState[`${x}-${y}`];
-      const isTurn =
-        (this.turn === 1 && player === "black") ||
-        (this.turn === -1 && player === "white");
-      if (isTurn) {
-        if (point.legal) {
-          return { ...this, success: true, move: { player, pos: { x, y } } };
-        }
-      }
-      return { ...this, success: false };
-    },
-
     makeMove: function ({ player, pos: { x, y } }) {
       if (this.pass > 1) {
         return { ...this, success: false };
       }
-      
+      if (x === 0) return this.submitPass(player);
-      let success = false;
       let game = this;
-      
+      let success = false;
-      if (x === 0) return game.submitPass(player);
-
-      // if checkMove has not been run, determine legality
-      if (!game.move) {
-        game = game.checkMove({ player, pos: { x, y } });
-      }
-      if (
-        // if move is legal
-        // ? unclear if checking move values is beneficial to prevent race conditions
-        game.move &&
-        game.move.player === player &&
-        game.move.pos.x === x &&
-        game.move.pos.y
-      ) {
       const point = game.boardState[`${x}-${y}`];
+      const isTurn =
+        (game.turn === 1 && player === "black") ||
+        (game.turn === -1 && player === "white");
+      if (isTurn) {
+        if (point.legal) {
           game.pass = 0;
-        // allows for recording of prior move on game record
+          game.addToRecord({ player, pos: { x, y } });
-        game.addToRecord(game.move);
+          if (this.kos.length) helper.clearKo.call(this);
-        if (game.kos.length) helper.clearKo.call(game);
           point.makeMove(game);
           game.turn *= -1;
           success = true;
         }
+      }
       game.boardState = getBoardState(game);
-      // remove move attribute to prevent duplicate moves
-      delete game.move;
       return { ...game, legalMoves: getLegalMoves(game), success };
     },
 
@@ -424,14 +389,6 @@ const Game = ({ gameData = {}, gameRecord = [] } = {}) => {
         this.playerState.bScore - (this.playerState.wScore + this.komi);
       return { ...this, score, winner: score > 0 ? 1 : -1 };
     },
-
-    // for playing historic games
-    manualEnd: function ({ winner, score }) {
-      this.turn = 0;
-      this.winner = winner;
-      this.score = score;
-      return this;
-    },
   };
 };
 

packages/server/services/gameServices.js

@@ -7,33 +7,11 @@ const GameService = ({ moveQueries, gameQueries }) => {
   };
   const gamesInProgress = {};
 
-  const storeMove = (gameId) => async ({ player, pos: { x, y } }) => {
-    let move = { player, pos: { x, y } };
-    try {
-      if (moveQueries) {
-        const { id } = await moveQueries.addMove({
-          gameId,
-          player,
-          x,
-          y,
-          gameRecord: true,
-          priorMove: null,
-        });
-        move.id = id;
-        move.success = true;
-      }
-    } catch (e) {
-      console.log(e);
-      move.success = false;
-    } finally {
-      return move;
-    }
-  };
-
   return {
     initGame({ id, gameRecord = [], ...gameData }) {
       if (gamesInProgress[id]) return this.getDataForUI(id);
       if (gameRecord.length) {
+        console.log("here");
         gamesInProgress[id] = Game({ gameData, gameRecord });
       } else {
         gamesInProgress[id] = Game({ gameData }).initGame();
@@ -54,7 +32,6 @@ const GameService = ({ moveQueries, gameQueries }) => {
           return { message: "error restoring game" };
         }
       }
-      gamesInProgress[id] = await gamesInProgress[id].checkMove(move);
       gamesInProgress[id] = gamesInProgress[id].makeMove(move);
       if (gamesInProgress[id].success === false)
         return { message: "illegal move" };
@@ -96,7 +73,6 @@ const GameService = ({ moveQueries, gameQueries }) => {
     },
 
     resign: ({ id, player }) => {
-      // add resign gamesQueries
       return gamesInProgress[id].submitResign(player).getMeta();
     },
 
@@ -132,19 +108,9 @@ const GameService = ({ moveQueries, gameQueries }) => {
 
     async endGame({ id }) {
       gamesInProgress[id] = gamesInProgress[id].endGame();
-      const { winner, score, playerState } = gamesInProgress[id];
-      const { bCaptures, wCaptures } = playerState;
-      const winType = winner > 0 ? "B+" : "W+";
       try {
         if (gameQueries) {
-          const result = await gameQueries.endGame({
+          // TODO add end game query
-            id,
-            winType,
-            score,
-            bCaptures,
-            wCaptures,
-          });
-          console.log(result);
         }
       } catch (e) {
         console.log(e);

packages/server/services/guestServices.js

@@ -1,25 +0,0 @@
-const generateRandomPassword = () => {
-  const minLength = 8,
-    maxLength = 16,
-    minUTF = 33,
-    maxUTF = 126;
-  const randomize = (min, max) => Math.floor(Math.random() * (max - min) + min);
-  return Array(randomize(minLength, maxLength))
-    .fill(0)
-    .map(() => String.fromCharCode(randomize(minUTF, maxUTF)))
-    .join("");
-};
-
-const guestService = {
-  currentGuest: 0,
-  generateGuest() {
-    // generate unique username
-    const username = `Guest-${String(this.currentGuest++).padStart(6, 0)}`;
-    // generate random "password"
-    // this exists solely to add extra randomness to signed token and is not validated
-    const password = generateRandomPassword();
-    return { username, password };
-  },
-};
-
-module.exports = guestService;

packages/server/socket.js

@@ -1,4 +1,4 @@
-// TODO const someSocketLogic = require('./middleware/sockets/...');
+// TODO const someSocketLogic = require('./middleware/socketssockets/...');
 const socketIO = require("socket.io");
 const io = socketIO({ cookie: false });
 
@@ -24,23 +24,8 @@ io.on("connection", async (socket) => {
       socket.on("connect_game", (data) => {
         const game = `game-${data.game.id}`;
         socket.join(game, async () => {
-          // TODO move this logic into game service
-          const gameData = await gameQueries.findGameById(data.game.id);
-          const convertWinType = (winType) => {
-            if (winType.includes("B")) return 1;
-            if (winType.includes("W")) return -1;
-            if (winType.includes("0")) return "D";
-            return "?";
-          };
-          gameData.winner = gameData.win_type
-            ? convertWinType(gameData.win_type)
-            : 0;
           const gameRecord = await moveQueries.findGameRecord(data.game.id);
-          await gameServices.initGame({
+          await gameServices.initGame({ id: data.game.id, gameRecord });
-            id: data.game.id,
-            gameRecord,
-            gameData,
-          });
           const { board, ...meta } = await gameServices.getDataForUI(
             data.game.id
           );
@@ -52,6 +37,7 @@ io.on("connection", async (socket) => {
       socket.on("make_move", async (data) => {
         const { user, move, board, game, room } = data;
         const gameNsp = `game-${data.game.id}`;
+
         try {
           const { board, message, ...meta } = await gameServices.makeMove({
             id: data.game.id,