From 127dac9d91a07c48d0173c6b6f4f7418347a9687 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 10:59:23 -0700
Subject: [PATCH 01/36] refactor: Extract Checkbox.jsx

---
 frontend/src/components/Checkbox.jsx | 27 +++++++++++++++++++++++++++
 frontend/src/components/README.md    |  5 +++++
 frontend/src/game/GameSettings.jsx   |  2 +-
 frontend/src/game/StartPanel.jsx     |  4 ++--
 frontend/src/lobby/CreatePanel.jsx   |  2 +-
 frontend/src/lobby/GameOptions.jsx   |  3 ++-
 frontend/src/utils.jsx               | 21 ---------------------
 7 files changed, 38 insertions(+), 26 deletions(-)
 create mode 100644 frontend/src/components/Checkbox.jsx
 create mode 100644 frontend/src/components/README.md

diff --git a/frontend/src/components/Checkbox.jsx b/frontend/src/components/Checkbox.jsx
new file mode 100644
index 00000000..25dfa104
--- /dev/null
+++ b/frontend/src/components/Checkbox.jsx
@@ -0,0 +1,27 @@
+import React from "react";
+import PropTypes from "prop-types";
+
+import App from "../app";
+
+const Checkbox = ({link, text, side, onChange, ...rest}) => (
+  <div>
+    {side === "right" ? text : ""}
+    <input
+      {...rest}
+      type="checkbox"
+      onChange={onChange || function (e) {
+        App.save(link, e.currentTarget.checked);
+      }}
+      checked={App.state[link]}/>
+    {side === "left" ? text : ""}
+  </div>
+);
+
+Checkbox.propTypes = {
+  link: PropTypes.string,
+  text: PropTypes.string,
+  side: PropTypes.string,
+  onChange: PropTypes.func
+};
+
+export default Checkbox;
diff --git a/frontend/src/components/README.md b/frontend/src/components/README.md
new file mode 100644
index 00000000..ca327996
--- /dev/null
+++ b/frontend/src/components/README.md
@@ -0,0 +1,5 @@
+# Components
+
+A set of general UI components.
+
+Many are "connected" tools which use a "link" prop to connect to the app state.
diff --git a/frontend/src/game/GameSettings.jsx b/frontend/src/game/GameSettings.jsx
index 007fc6de..610a4716 100644
--- a/frontend/src/game/GameSettings.jsx
+++ b/frontend/src/game/GameSettings.jsx
@@ -1,7 +1,7 @@
 import React from "react";
 
 import App from "../app";
-import {Checkbox} from "../utils";
+import Checkbox from "../components/Checkbox";
 
 const GameSettings = () => (
   <div className='game-settings'>
diff --git a/frontend/src/game/StartPanel.jsx b/frontend/src/game/StartPanel.jsx
index 79038ae1..5b5c482f 100644
--- a/frontend/src/game/StartPanel.jsx
+++ b/frontend/src/game/StartPanel.jsx
@@ -1,8 +1,8 @@
 import React from "react";
 
 import App from "../app";
-import { Select, Checkbox, toTitleCase } from "../utils";
-
+import Checkbox from "../components/Checkbox";
+import { Select, toTitleCase } from "../utils";
 
 const StartPanel = () => {
   const gameType = toTitleCase(App.state.game.type);
diff --git a/frontend/src/lobby/CreatePanel.jsx b/frontend/src/lobby/CreatePanel.jsx
index 5c90d89d..e26c1c4f 100644
--- a/frontend/src/lobby/CreatePanel.jsx
+++ b/frontend/src/lobby/CreatePanel.jsx
@@ -2,7 +2,7 @@ import React from "react";
 
 import _ from "utils/utils";
 import App from "../app";
-import {Checkbox} from "../utils";
+import Checkbox from "../components/Checkbox";
 
 import GameTypes from "./GameTypes";
 import GameOptions from "./GameOptions";
diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 8e8ed17b..3f93eb91 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -3,7 +3,8 @@ import PropTypes from "prop-types";
 
 import _ from "utils/utils";
 import App from "../app";
-import { Checkbox, Select } from "../utils";
+import Checkbox from "../components/Checkbox"
+import { Select } from "../utils";
 
 import Set from "./Set";
 import CubeList from "./CubeList";
diff --git a/frontend/src/utils.jsx b/frontend/src/utils.jsx
index 6f6d0881..2376dcb4 100644
--- a/frontend/src/utils.jsx
+++ b/frontend/src/utils.jsx
@@ -8,27 +8,6 @@ import App from "./app";
  * through a "link" prop to connect to the app state
  */
 
-export const Checkbox = ({link, text, side, onChange, ...rest}) => (
-  <div>
-    {side === "right" ? text : ""}
-    <input
-      {...rest}
-      type="checkbox"
-      onChange={onChange || function (e) {
-        App.save(link, e.currentTarget.checked);
-      }}
-      checked={App.state[link]}/>
-    {side === "left" ? text : ""}
-  </div>
-);
-
-Checkbox.propTypes = {
-  link: PropTypes.string,
-  text: PropTypes.string,
-  side: PropTypes.string,
-  onChange: PropTypes.func
-};
-
 export const Spaced = ({elements}) => (
   elements
     .map((x, index) => <span key={index}>{x}</span>)

From 86cadbf885b9d55aed041b9b9bbcb1a785b3b154 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 11:02:18 -0700
Subject: [PATCH 02/36] refactor: Extract Spaced.jsx

---
 frontend/src/components/Spaced.jsx | 13 +++++++++++++
 frontend/src/game/Cols.jsx         |  2 +-
 frontend/src/game/Grid.jsx         |  2 +-
 frontend/src/lobby/Header.jsx      |  2 +-
 frontend/src/utils.jsx             | 10 ----------
 5 files changed, 16 insertions(+), 13 deletions(-)
 create mode 100644 frontend/src/components/Spaced.jsx

diff --git a/frontend/src/components/Spaced.jsx b/frontend/src/components/Spaced.jsx
new file mode 100644
index 00000000..e729ae4e
--- /dev/null
+++ b/frontend/src/components/Spaced.jsx
@@ -0,0 +1,13 @@
+import React from "react";
+
+const Spaced = ({elements}) => (
+  elements
+    .map((x, index) => <span key={index}>{x}</span>)
+    .reduce((prev, curr) => [
+      prev,
+      <span key = {prev+curr} className = 'spacer-dot' />,
+      curr
+    ])
+);
+
+export default Spaced;
diff --git a/frontend/src/game/Cols.jsx b/frontend/src/game/Cols.jsx
index d05ac470..15196b21 100644
--- a/frontend/src/game/Cols.jsx
+++ b/frontend/src/game/Cols.jsx
@@ -3,7 +3,7 @@ import PropTypes from "prop-types";
 
 import App from "../app";
 import {getZoneDisplayName} from "../zones";
-import {Spaced} from "../utils.jsx";
+import Spaced from "../components/Spaced";
 import {getCardSrc, getFallbackSrc} from "../cardimage";
 
 class Cols extends Component {
diff --git a/frontend/src/game/Grid.jsx b/frontend/src/game/Grid.jsx
index cac13620..f689a54b 100644
--- a/frontend/src/game/Grid.jsx
+++ b/frontend/src/game/Grid.jsx
@@ -4,7 +4,7 @@ import PropTypes from "prop-types";
 import _ from "utils/utils";
 import App from "../app";
 import {getCardSrc, getFallbackSrc} from "../cardimage";
-import {Spaced} from "../utils";
+import Spaced from "../components/Spaced";
 import {ZONE_PACK, getZoneDisplayName} from "../zones";
 
 const Grid = ({zones}) => (
diff --git a/frontend/src/lobby/Header.jsx b/frontend/src/lobby/Header.jsx
index 47fd8166..c621fd95 100644
--- a/frontend/src/lobby/Header.jsx
+++ b/frontend/src/lobby/Header.jsx
@@ -1,7 +1,7 @@
 import React from "react";
 
 import { STRINGS } from "../config";
-import { Spaced } from "../utils";
+import Spaced from "../components/Spaced";
 import App from "../app";
 
 const Header = () => (
diff --git a/frontend/src/utils.jsx b/frontend/src/utils.jsx
index 2376dcb4..dd461469 100644
--- a/frontend/src/utils.jsx
+++ b/frontend/src/utils.jsx
@@ -8,16 +8,6 @@ import App from "./app";
  * through a "link" prop to connect to the app state
  */
 
-export const Spaced = ({elements}) => (
-  elements
-    .map((x, index) => <span key={index}>{x}</span>)
-    .reduce((prev, curr) => [
-      prev,
-      <span key = {prev+curr} className = 'spacer-dot' />,
-      curr
-    ])
-);
-
 export const Select = ({
   link,
   opts,

From 4fc39723dc3bd739a59342d413e4dd5623a76b0d Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 11:05:21 -0700
Subject: [PATCH 03/36] refactor: Extract Select.jsx

---
 frontend/src/components/Select.jsx | 29 +++++++++++++++++++++++++++++
 frontend/src/game/DeckSettings.jsx |  2 +-
 frontend/src/game/StartPanel.jsx   |  3 ++-
 frontend/src/lobby/GameOptions.jsx |  4 ++--
 frontend/src/utils.jsx             | 23 -----------------------
 5 files changed, 34 insertions(+), 27 deletions(-)
 create mode 100644 frontend/src/components/Select.jsx

diff --git a/frontend/src/components/Select.jsx b/frontend/src/components/Select.jsx
new file mode 100644
index 00000000..a98482ec
--- /dev/null
+++ b/frontend/src/components/Select.jsx
@@ -0,0 +1,29 @@
+import React from "react";
+import PropTypes from "prop-types";
+
+import App from "../app";
+
+const Select = ({
+  link,
+  opts,
+  onChange = (e) => { App.save(link, e.currentTarget.value); },
+  value = App.state[link],
+  ...rest}) => (
+  <select
+    onChange={onChange}
+    value={value}
+    {...rest}>
+    {opts.map((opt, index) =>
+      <option key={index}>{opt}</option>
+    )}
+  </select>
+);
+
+Select.propTypes = {
+  link: PropTypes.string,
+  onChange: PropTypes.func,
+  value: PropTypes.any,
+  opts: PropTypes.array
+};
+
+export default Select;
diff --git a/frontend/src/game/DeckSettings.jsx b/frontend/src/game/DeckSettings.jsx
index 191cfe96..d788b550 100644
--- a/frontend/src/game/DeckSettings.jsx
+++ b/frontend/src/game/DeckSettings.jsx
@@ -4,7 +4,7 @@ import PropTypes from "prop-types";
 import App from "../app";
 import {getZoneDisplayName, ZONE_MAIN, ZONE_SIDEBOARD} from "../zones";
 import {COLORS_TO_LANDS_NAME} from "../gamestate";
-import {Select} from "../utils";
+import Select from "../components/Select";
 
 const DeckSettings = () => (
   (App.state.isGameFinished || App.state.didGameStart)
diff --git a/frontend/src/game/StartPanel.jsx b/frontend/src/game/StartPanel.jsx
index 5b5c482f..83669392 100644
--- a/frontend/src/game/StartPanel.jsx
+++ b/frontend/src/game/StartPanel.jsx
@@ -2,7 +2,8 @@ import React from "react";
 
 import App from "../app";
 import Checkbox from "../components/Checkbox";
-import { Select, toTitleCase } from "../utils";
+import Select from "../components/Select";
+import { toTitleCase } from "../utils";
 
 const StartPanel = () => {
   const gameType = toTitleCase(App.state.game.type);
diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 3f93eb91..bb4709b8 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -3,8 +3,8 @@ import PropTypes from "prop-types";
 
 import _ from "utils/utils";
 import App from "../app";
-import Checkbox from "../components/Checkbox"
-import { Select } from "../utils";
+import Checkbox from "../components/Checkbox";
+import Select from "../components/Select";
 
 import Set from "./Set";
 import CubeList from "./CubeList";
diff --git a/frontend/src/utils.jsx b/frontend/src/utils.jsx
index dd461469..69d71b65 100644
--- a/frontend/src/utils.jsx
+++ b/frontend/src/utils.jsx
@@ -8,29 +8,6 @@ import App from "./app";
  * through a "link" prop to connect to the app state
  */
 
-export const Select = ({
-  link,
-  opts,
-  onChange = (e) => { App.save(link, e.currentTarget.value); },
-  value = App.state[link],
-  ...rest}) => (
-  <select
-    onChange={onChange}
-    value={value}
-    {...rest}>
-    {opts.map((opt, index) =>
-      <option key={index}>{opt}</option>
-    )}
-  </select>
-);
-
-Select.propTypes = {
-  link: PropTypes.string,
-  onChange: PropTypes.func,
-  value: PropTypes.any,
-  opts: PropTypes.array
-};
-
 export const Textarea = ({link, ...rest}) => (
   <textarea
     style={ {"overflowY": "scroll", "height": "150px"}}

From 2da45194239b92d201e947288e34d9d92d050a54 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 11:08:04 -0700
Subject: [PATCH 04/36] refactor: Extract TextArea.jsx

---
 frontend/src/components/TextArea.jsx | 19 +++++++++++++++++++
 frontend/src/lobby/CubeList.jsx      |  4 ++--
 frontend/src/utils.jsx               | 23 -----------------------
 3 files changed, 21 insertions(+), 25 deletions(-)
 create mode 100644 frontend/src/components/TextArea.jsx

diff --git a/frontend/src/components/TextArea.jsx b/frontend/src/components/TextArea.jsx
new file mode 100644
index 00000000..dbb29b27
--- /dev/null
+++ b/frontend/src/components/TextArea.jsx
@@ -0,0 +1,19 @@
+import React from "react";
+import PropTypes from "prop-types";
+
+import App from "../app";
+
+const TextArea = ({link, ...rest}) => (
+  <textarea
+    style={ {"overflowY": "scroll", "height": "150px"}}
+    onChange=
+      { (e) => { App.save("list", e.currentTarget.value); } }
+    value={App.state[link]}
+    {...rest} />
+);
+
+TextArea.propTypes = {
+  link: PropTypes.string
+};
+
+export default TextArea;
diff --git a/frontend/src/lobby/CubeList.jsx b/frontend/src/lobby/CubeList.jsx
index e7afa188..11d824c2 100644
--- a/frontend/src/lobby/CubeList.jsx
+++ b/frontend/src/lobby/CubeList.jsx
@@ -1,7 +1,7 @@
 import React, { useState, Fragment } from "react";
 import axios from "axios";
 
-import { Textarea } from "../utils";
+import TextArea from "../components/TextArea";
 import App from "../app";
 
 const CubeList = () => {
@@ -13,7 +13,7 @@ const CubeList = () => {
   return (<div id='cube-list'>
     <div className='column'>
       <div>{`one card per line (${cubeListLength} cards)`}</div>
-      <Textarea
+      <TextArea
         placeholder='cube list'
         link='list'
       />
diff --git a/frontend/src/utils.jsx b/frontend/src/utils.jsx
index 69d71b65..4f6bb80c 100644
--- a/frontend/src/utils.jsx
+++ b/frontend/src/utils.jsx
@@ -1,26 +1,3 @@
-import React from "react";
-import PropTypes from "prop-types";
-
-import App from "./app";
-
-/**
- * Utils offers a list of "connected" tools
- * through a "link" prop to connect to the app state
- */
-
-export const Textarea = ({link, ...rest}) => (
-  <textarea
-    style={ {"overflowY": "scroll", "height": "150px"}}
-    onChange=
-      { (e) => { App.save("list", e.currentTarget.value); } }
-    value={App.state[link]}
-    {...rest} />
-);
-
-Textarea.propTypes = {
-  link: PropTypes.string
-};
-
 //TODO: check if lodash can do it
 export const toTitleCase = (string="", separator=" ") =>
   string.split(separator)

From b3620cc34db3b55244531f0c466ffb3f6197d28c Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 11:19:28 -0700
Subject: [PATCH 05/36] refactor: Clean and re-use toTitleCase()

---
 frontend/src/lobby/GameTypes.jsx |  4 ++-
 frontend/src/utils.js            | 21 +++++++++++++++
 frontend/src/utils.jsx           |  4 ---
 frontend/test/utils.spec.js      | 44 ++++++++++++++++++++++++++++++++
 4 files changed, 68 insertions(+), 5 deletions(-)
 create mode 100644 frontend/src/utils.js
 delete mode 100644 frontend/src/utils.jsx
 create mode 100644 frontend/test/utils.spec.js

diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 6d7c3a73..043ca362 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -3,6 +3,8 @@ import PropTypes from "prop-types";
 
 import App from "../app";
 
+import { toTitleCase } from "../utils";
+
 const GameTypes = () => {
   const types = ["draft", "sealed"];
   const subtypes = ["regular", "cube", "chaos"];
@@ -42,7 +44,7 @@ const GameType = ({ name, type, isChecked, onChange}) => (
       type='radio'
       value={type}
       onChange={onChange}
-      checked={isChecked}/> {type.toLowerCase().split(" ").map((s) => s.charAt(0).toUpperCase() + s.substring(1)).join(" ")}
+      checked={isChecked}/> {toTitleCase(type)}
   </label>
 );
 
diff --git a/frontend/src/utils.js b/frontend/src/utils.js
new file mode 100644
index 00000000..3c7d9b3d
--- /dev/null
+++ b/frontend/src/utils.js
@@ -0,0 +1,21 @@
+const capitalize = (word) => {
+  if (!word || typeof word !== "string") {
+    return "";
+  }
+  return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
+};
+
+const toTitleCase = (sentence, separator=" ") => {
+  if (!sentence || typeof sentence !== "string") {
+    return "";
+  }
+
+  const words = sentence.split(separator);
+  const capitalized = words.map(capitalize);
+  return capitalized.join(separator);
+};
+
+module.exports = {
+  capitalize,
+  toTitleCase
+};
diff --git a/frontend/src/utils.jsx b/frontend/src/utils.jsx
deleted file mode 100644
index 4f6bb80c..00000000
--- a/frontend/src/utils.jsx
+++ /dev/null
@@ -1,4 +0,0 @@
-//TODO: check if lodash can do it
-export const toTitleCase = (string="", separator=" ") =>
-  string.split(separator)
-    .reduce((result, word) => `${result} ${word.charAt(0).toUpperCase()+word.slice(1).toLowerCase()}`, "");
diff --git a/frontend/test/utils.spec.js b/frontend/test/utils.spec.js
new file mode 100644
index 00000000..59587c27
--- /dev/null
+++ b/frontend/test/utils.spec.js
@@ -0,0 +1,44 @@
+const {describe, it} = require("mocha");
+const assert = require("assert");
+const {capitalize, toTitleCase} = require("../src/utils");
+
+describe("Acceptance tests for frontend utils", () => {
+  it("capitalize() should capitalize a single word", () => {
+    assert(capitalize("") === "");
+    assert(capitalize(0) === "");
+    assert(capitalize({}) === "");
+
+    assert(capitalize("draft") === "Draft");
+    assert(capitalize("Draft") === "Draft");
+    assert(capitalize("DRAFT") === "Draft");
+    assert(capitalize("draft") === "Draft");
+
+    assert(capitalize("cube draft") === "Cube draft");
+    assert(capitalize("CUBE DRAFT") === "Cube draft");
+  });
+  it("toTitleCase() should capitalize each word", () => {
+    assert(toTitleCase("") === "");
+    assert(toTitleCase(0) === "");
+    assert(toTitleCase({}) === "");
+
+    assert(toTitleCase("draft") === "Draft");
+    assert(toTitleCase("Draft") === "Draft");
+    assert(toTitleCase("DRAFT") === "Draft");
+    assert(toTitleCase("draft") === "Draft");
+
+    assert(toTitleCase("cube draft") === "Cube Draft");
+    assert(toTitleCase("CUBE DRAFT") === "Cube Draft");
+
+    assert(toTitleCase("cube_draft") === "Cube_draft");
+    assert(toTitleCase("CUBE_DRAFT") === "Cube_draft");
+    assert(toTitleCase("cube_draft", "_") === "Cube_Draft");
+    assert(toTitleCase("CUBE_DRAFT", "_") === "Cube_Draft");
+
+    assert(toTitleCase("longer sentence of wordiness") === "Longer Sentence Of Wordiness");
+    assert(toTitleCase("LONGER SENTENCE OF WORDINESS") === "Longer Sentence Of Wordiness");
+    assert(toTitleCase("longer_sentence_of_wordiness") === "Longer_sentence_of_wordiness");
+    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS") === "Longer_sentence_of_wordiness");
+    assert(toTitleCase("longer_sentence_of_wordiness", "_") === "Longer_Sentence_Of_Wordiness");
+    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS", "_") === "Longer_Sentence_Of_Wordiness");
+  });
+});

From c448abb579b02faf89b993b2439f3654845fdbca Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 12:10:43 -0700
Subject: [PATCH 06/36] feat: Enable debugging in VSCode

---
 README.md    | 27 ++++++++++++++++++++++++++-
 package.json |  2 ++
 2 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 62d1b705..c86a1590 100644
--- a/README.md
+++ b/README.md
@@ -83,7 +83,7 @@ You can also create a Docker image and run the app in a container:
 
 ## Usage
 
-### Start server
+### Start Server
 
 `npm start`
 
@@ -98,6 +98,31 @@ This command downloads integrates all files previously downloaded from MTGJson.
 `npm run download_booster_rules`
  download and parse booster generation rules from [magic-sealed-data](https://github.com/taw/magic-sealed-data)
 
+## Development Notes
+
+### VSCode
+
+You can debug this application by adding the following configuration to your `launch.json`:
+
+```json
+{
+  "name": "Launch via NPM",
+  "type": "node",
+  "request": "launch",
+  "cwd": "${workspaceFolder}",
+  "runtimeExecutable": "npm",
+  "runtimeArgs": [
+      "run", "start-debug"
+  ],
+  "port": 1338
+}
+```
+
+You should now be able to set breakpoints in `backend/` and hit them when you start the debugger.
+This relies on the `--inspect-brk=1338` flag to open port 1338 for the debugger to attach to.
+
+Breakpoints for the frontend should be set in your browser console.
+
 ### Contributors
 
 THANK YOU!
diff --git a/package.json b/package.json
index af8371de..9bb22cd5 100644
--- a/package.json
+++ b/package.json
@@ -25,9 +25,11 @@
   "scripts": {
     "start": "node scripts/run",
     "start-dev": "npm run watch & npm run nodemon",
+    "start-debug": "npm run watch & npm run nodemon:debug",
     "build": "webpack --config webpack.prod.js",
     "watch": "webpack --config webpack.dev.js --watch -d --display-error-details",
     "nodemon": "nodemon ./app.js -e html,js --ignore built --ignore frontend",
+    "nodemon:debug": "nodemon --inspect-brk=1338 ./app.js -e html,js --ignore built --ignore frontend",
     "lint": "eslint --ignore-path .gitignore .",
     "pretest": "npm run lint",
     "test": "mocha --reporter spec --exit \"./{,!(node_modules)/**/}*.spec.js\"",

From 91ff1c5062a6acbcd2bae65e8cdac40115c7d57c Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 12:12:22 -0700
Subject: [PATCH 07/36] feat: Lobby UI for decadent draft

---
 backend/util.js                    | 13 +++++++++++--
 frontend/src/events.js             | 14 +++++++++++---
 frontend/src/lobby/GameOptions.jsx | 26 ++++++++++++++++++++++++--
 frontend/src/lobby/GameTypes.jsx   |  2 +-
 frontend/src/lobby/Set.jsx         | 15 ++++++++++++---
 5 files changed, 59 insertions(+), 11 deletions(-)

diff --git a/backend/util.js b/backend/util.js
index 3b3087e9..cfdaee7e 100644
--- a/backend/util.js
+++ b/backend/util.js
@@ -66,8 +66,17 @@ module.exports = {
     return true;
   },
   game({ seats, type, sets, cube, isPrivate, modernOnly = true, chaosPacksNumber, totalChaos = true }) {
-    assert(["draft", "sealed", "cube draft", "cube sealed", "chaos draft", "chaos sealed"].includes(type),
-      "type can be draft, sealed, chaos draft, chaos sealed, cube draft or cube sealed");
+    const acceptableGameTypes = [
+      "draft",
+      "sealed",
+      "cube draft",
+      "cube sealed",
+      "chaos draft",
+      "chaos sealed",
+      "decadent draft"
+    ];
+    assert(acceptableGameTypes.includes(type),
+      `type can be one of: ${acceptableGameTypes.join(", ")}`);
     assert(typeof isPrivate === "boolean", "isPrivate must be a boolean");
     assert(typeof seats === "number", "seats must be a number");
     assert(seats >= 1 && seats <= 100, "seats' number must be between 1 and 100");
diff --git a/frontend/src/events.js b/frontend/src/events.js
index 592f9635..2533965b 100644
--- a/frontend/src/events.js
+++ b/frontend/src/events.js
@@ -110,7 +110,8 @@ const events = {
     let options = {type, seats, title, isPrivate, modernOnly, totalChaos};
 
     switch (gamesubtype) {
-    case "regular": {
+    case "regular":
+    case "decadent": {
       const {setsDraft, setsSealed} = App.state;
       options.sets = gametype === "sealed" ? setsSealed : setsDraft;
       break;
@@ -124,10 +125,10 @@ const events = {
     }
     App.send("create", options);
   },
-  changeSetsNumber(type, event) {
+  changeSetsNumber(type, ensureAllSetsIdentical, event) {
     event.preventDefault();
     const packsNumber = event.currentTarget.value;
-    const sets = App.state[type];
+    let sets = App.state[type];
 
     if (sets.length < packsNumber) {
       const toAdd = packsNumber - sets.length;
@@ -137,6 +138,13 @@ const events = {
       sets.splice(packsNumber);
     }
 
+    if (ensureAllSetsIdentical) {
+      const firstSet = sets[0];
+      for (let i = 1; i < sets.length; i++) {
+        sets[i] = firstSet;
+      }
+    }
+
     App.save(type, sets);
   },
   pool(cards) {
diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index bb4709b8..80bfe408 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -17,6 +17,8 @@ const GameOptions = () => {
     return <Regular sets={setsDraft} type={"setsDraft"} />;
   case "regular sealed":
     return <Regular sets={setsSealed} type={"setsSealed"} />;
+  case "decadent draft":
+    return <Decadent sets={setsDraft} type={"setsDraft"} />;
   case "cube draft":
     return <CubeDraft />;
   case "cube sealed":
@@ -31,12 +33,12 @@ const GameOptions = () => {
 };
 
 const Regular = ({ sets, type }) => (
-  <Fragment >
+  <Fragment>
     <div>
       Number of packs:{" "}
       <Select
         value={sets.length}
-        onChange={App._emit("changeSetsNumber", type)}
+        onChange={App._emit("changeSetsNumber", type, false)}
         opts={_.seq(12, 1)} />
     </div>
     <div className="wrapper">
@@ -55,6 +57,26 @@ const Sets = ({ sets, type }) => (
     .map((set, i) => <Set type={type} selectedSet={set} index={i} key={i} />)
 );
 
+const Decadent = ({ sets, type }) => (
+  <Fragment>
+    <div>
+      Number of packs:{" "}
+      <Select
+        value={sets.length}
+        onChange={App._emit("changeSetsNumber", type, true)}
+        opts={_.seq(60, 36)} />
+    </div>
+    <div className="wrapper">
+      <Set type={type} selectedSet={sets[0]} index={0} key={0} useForAllSets={true} />
+    </div>
+  </Fragment>
+);
+
+Decadent.propTypes = {
+  sets: PropTypes.array,
+  type: PropTypes.string
+};
+
 const CubeDraft = () => (
   <div>
     <CubeList />
diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 043ca362..98fa742b 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -7,7 +7,7 @@ import { toTitleCase } from "../utils";
 
 const GameTypes = () => {
   const types = ["draft", "sealed"];
-  const subtypes = ["regular", "cube", "chaos"];
+  const subtypes = ["regular", "cube", "chaos", "decadent"];
   return (
     <div>
       <p>Game type:{" "}
diff --git a/frontend/src/lobby/Set.jsx b/frontend/src/lobby/Set.jsx
index e42d4bb7..3eb3ef2d 100644
--- a/frontend/src/lobby/Set.jsx
+++ b/frontend/src/lobby/Set.jsx
@@ -4,9 +4,17 @@ import PropTypes from "prop-types";
 import App from "../app";
 import { toTitleCase } from "../utils";
 
-const Set = ({ index, selectedSet, type }) => {
+const Set = ({ index, selectedSet, type, useForAllSets=false }) => {
   const onSetChange = (e) => {
-    App.state[type][index] = e.currentTarget.value;
+    const chosenSet = e.currentTarget.value;
+    let sets = App.state[type];
+    if (!useForAllSets) {
+      sets[index] = chosenSet;
+    } else {
+      for (let i = 0; i < sets.length; i++) {
+        sets[i] = chosenSet;
+      }
+    }
     App.save(type, App.state[type]);
   };
   let groups = [];
@@ -32,7 +40,8 @@ const Set = ({ index, selectedSet, type }) => {
 Set.propTypes = {
   index: PropTypes.number,
   selectedSet: PropTypes.string,
-  type: PropTypes.string
+  type: PropTypes.string,
+  useForAllSets: PropTypes.bool
 };
 
 export default Set;

From 65b8328344bf96611f3665a057d39acb9bf8e91f Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 12:13:12 -0700
Subject: [PATCH 08/36] feat: Handle pack passing for decadent draft

---
 backend/game.js | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/backend/game.js b/backend/game.js
index f901bee3..c120a4d4 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -25,6 +25,7 @@ module.exports = class Game extends Room {
       round: 0,
       bots: 0,
       sets: sets || [],
+      isDecadent: false,
       secret: uuid.v4(),
       logger: logger.child({ id: gameID })
     });
@@ -36,6 +37,11 @@ module.exports = class Game extends Room {
       this.packsInfo = this.sets.join(" / ");
       this.rounds = this.sets.length;
       break;
+    case "decadent draft":
+      this.packsInfo = this.sets.join(" / ");
+      this.rounds = this.sets.length;
+      this.isDecadent = true;
+      break;
     case "cube draft":
       this.packsInfo = `${cube.packs} packs with ${cube.cards} cards from a pool of ${cube.list.length} cards`;
       this.rounds = this.cube.packs;
@@ -342,12 +348,18 @@ module.exports = class Game extends Room {
   }
 
   pass(p, pack) {
-    if (!pack.length) {
-      if (!--this.packCount)
-        this.startRound();
-      else
-        this.meta();
+    if (this.isDecadent) {
+      // Decadent starts new round after each pick.
+      this.startRound();
       return;
+    } else {
+      if (!pack.length) {
+        if (!--this.packCount)
+          this.startRound();
+        else
+          this.meta();
+        return;
+      }
     }
 
     const index = this.players.indexOf(p) + this.delta;
@@ -450,7 +462,8 @@ module.exports = class Game extends Room {
       });
       break;
     }
-    case "draft": {
+    case "draft": 
+    case "decadent draft": {
       this.pool = Pool.DraftNormal({
         playersLength: this.players.length,
         sets: this.sets

From 6c8fa21b9cca1acd72bf0d0c7ba037ebfcc73110 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 12:50:06 -0700
Subject: [PATCH 09/36] fix: Exclude *.spec.js files from webpack build

---
 webpack.common.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/webpack.common.js b/webpack.common.js
index a734b67a..683af76e 100644
--- a/webpack.common.js
+++ b/webpack.common.js
@@ -43,7 +43,7 @@ module.exports = {
     rules: [
       {
         test: /\.(js|jsx)$/,
-        exclude: /node_modules/,
+        exclude: /(node_modules|\.spec\.js$)/,
         use: {
           loader: "babel-loader",
           options: {

From a6ac3f924f14da670c4c634b9b3b595f96180faf Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 12:50:22 -0700
Subject: [PATCH 10/36] fix: Decadent draft should show Pick 1 / 1

---
 frontend/src/app.js        |  3 +++
 frontend/src/game/Grid.jsx | 25 +++++++++++++++++++------
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git a/frontend/src/app.js b/frontend/src/app.js
index c2e7795d..93359acb 100644
--- a/frontend/src/app.js
+++ b/frontend/src/app.js
@@ -82,6 +82,9 @@ let App = {
     get isGameFinished() {
       return App.state.round === -1;
     },
+    get isDecadentDraft() {
+      return /decadent draft/.test(App.state.game.type);
+    },
 
     get notificationBlocked() {
       return ["denied", "notsupported"].includes(App.state.notificationResult);
diff --git a/frontend/src/game/Grid.jsx b/frontend/src/game/Grid.jsx
index f689a54b..d55c097c 100644
--- a/frontend/src/game/Grid.jsx
+++ b/frontend/src/game/Grid.jsx
@@ -17,6 +17,23 @@ Grid.propTypes = {
   zones: PropTypes.array.isRequired
 };
 
+const getZoneDetails = (appState, zoneName, cards) => {
+  if (!appState.didGameStart) {
+    return 0;
+  }
+
+  if (zoneName === ZONE_PACK) {
+    if (appState.isDecadentDraft) {
+      // Only 1 pick in decadent draft.
+      return `Pick 1 / 1`;
+    } else {
+      return `Pick ${appState.pickNumber} / ${appState.packSize}`
+    }
+  } else {
+    return cards.length;
+  }
+}
+
 const zone = (zoneName, index) => {
   const zone = App.getSortedZone(zoneName);
   const zoneDisplayName = getZoneDisplayName(zoneName);
@@ -24,16 +41,12 @@ const zone = (zoneName, index) => {
   const cards = _.flat(values);
 
   const zoneTitle = zoneDisplayName + (zoneName === ZONE_PACK ? " " + App.state.round : "");
-  const zoneHelper = App.state.didGameStart
-    ? zoneName === ZONE_PACK
-      ? `Pick ${App.state.pickNumber} / ${App.state.packSize}`
-      : cards.length
-    : 0;
+  const zoneDetails = getZoneDetails(App.state, zoneName, cards);
 
   return (
     <div className='zone' key={index}>
       <h1>
-        <Spaced elements={[zoneTitle, zoneHelper]}/>
+        <Spaced elements={[zoneTitle, zoneDetails]}/>
       </h1>
       {cards.map((card, i) =>
         <Card key={i+zoneName+card.name+card.foil} card={card} zoneName={zoneName} />

From df88748b31a4e0c26fc5206ebf759070ce59573a Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 13:25:23 -0700
Subject: [PATCH 11/36] wip: Temporarily disable frontend tests

---
 frontend/test/utils.spec.js | 74 ++++++++++++++++++-------------------
 1 file changed, 37 insertions(+), 37 deletions(-)

diff --git a/frontend/test/utils.spec.js b/frontend/test/utils.spec.js
index 59587c27..56e76f7c 100644
--- a/frontend/test/utils.spec.js
+++ b/frontend/test/utils.spec.js
@@ -1,44 +1,44 @@
-const {describe, it} = require("mocha");
-const assert = require("assert");
-const {capitalize, toTitleCase} = require("../src/utils");
+// const {describe, it} = require("mocha");
+// const assert = require("assert");
+// const {capitalize, toTitleCase} = require("./../src/utils");
 
-describe("Acceptance tests for frontend utils", () => {
-  it("capitalize() should capitalize a single word", () => {
-    assert(capitalize("") === "");
-    assert(capitalize(0) === "");
-    assert(capitalize({}) === "");
+// describe("Acceptance tests for frontend utils", () => {
+//   it("capitalize() should capitalize a single word", () => {
+//     assert(capitalize("") === "");
+//     assert(capitalize(0) === "");
+//     assert(capitalize({}) === "");
 
-    assert(capitalize("draft") === "Draft");
-    assert(capitalize("Draft") === "Draft");
-    assert(capitalize("DRAFT") === "Draft");
-    assert(capitalize("draft") === "Draft");
+//     assert(capitalize("draft") === "Draft");
+//     assert(capitalize("Draft") === "Draft");
+//     assert(capitalize("DRAFT") === "Draft");
+//     assert(capitalize("draft") === "Draft");
 
-    assert(capitalize("cube draft") === "Cube draft");
-    assert(capitalize("CUBE DRAFT") === "Cube draft");
-  });
-  it("toTitleCase() should capitalize each word", () => {
-    assert(toTitleCase("") === "");
-    assert(toTitleCase(0) === "");
-    assert(toTitleCase({}) === "");
+//     assert(capitalize("cube draft") === "Cube draft");
+//     assert(capitalize("CUBE DRAFT") === "Cube draft");
+//   });
+//   it("toTitleCase() should capitalize each word", () => {
+//     assert(toTitleCase("") === "");
+//     assert(toTitleCase(0) === "");
+//     assert(toTitleCase({}) === "");
 
-    assert(toTitleCase("draft") === "Draft");
-    assert(toTitleCase("Draft") === "Draft");
-    assert(toTitleCase("DRAFT") === "Draft");
-    assert(toTitleCase("draft") === "Draft");
+//     assert(toTitleCase("draft") === "Draft");
+//     assert(toTitleCase("Draft") === "Draft");
+//     assert(toTitleCase("DRAFT") === "Draft");
+//     assert(toTitleCase("draft") === "Draft");
 
-    assert(toTitleCase("cube draft") === "Cube Draft");
-    assert(toTitleCase("CUBE DRAFT") === "Cube Draft");
+//     assert(toTitleCase("cube draft") === "Cube Draft");
+//     assert(toTitleCase("CUBE DRAFT") === "Cube Draft");
 
-    assert(toTitleCase("cube_draft") === "Cube_draft");
-    assert(toTitleCase("CUBE_DRAFT") === "Cube_draft");
-    assert(toTitleCase("cube_draft", "_") === "Cube_Draft");
-    assert(toTitleCase("CUBE_DRAFT", "_") === "Cube_Draft");
+//     assert(toTitleCase("cube_draft") === "Cube_draft");
+//     assert(toTitleCase("CUBE_DRAFT") === "Cube_draft");
+//     assert(toTitleCase("cube_draft", "_") === "Cube_Draft");
+//     assert(toTitleCase("CUBE_DRAFT", "_") === "Cube_Draft");
 
-    assert(toTitleCase("longer sentence of wordiness") === "Longer Sentence Of Wordiness");
-    assert(toTitleCase("LONGER SENTENCE OF WORDINESS") === "Longer Sentence Of Wordiness");
-    assert(toTitleCase("longer_sentence_of_wordiness") === "Longer_sentence_of_wordiness");
-    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS") === "Longer_sentence_of_wordiness");
-    assert(toTitleCase("longer_sentence_of_wordiness", "_") === "Longer_Sentence_Of_Wordiness");
-    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS", "_") === "Longer_Sentence_Of_Wordiness");
-  });
-});
+//     assert(toTitleCase("longer sentence of wordiness") === "Longer Sentence Of Wordiness");
+//     assert(toTitleCase("LONGER SENTENCE OF WORDINESS") === "Longer Sentence Of Wordiness");
+//     assert(toTitleCase("longer_sentence_of_wordiness") === "Longer_sentence_of_wordiness");
+//     assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS") === "Longer_sentence_of_wordiness");
+//     assert(toTitleCase("longer_sentence_of_wordiness", "_") === "Longer_Sentence_Of_Wordiness");
+//     assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS", "_") === "Longer_Sentence_Of_Wordiness");
+//   });
+// });

From 5ea8df6e8fca06bfc3728784b83bbe78c7d755b0 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 19:16:32 -0700
Subject: [PATCH 12/36] fix: Switch from regular to decadent draft should
 adjust sets

---
 frontend/src/lobby/GameTypes.jsx | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 98fa742b..27214352 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -28,7 +28,14 @@ const GameTypes = () => {
               type={type}
               key={key}
               isChecked={App.state.gamesubtype === type}
-              onChange={() => App.save("gamesubtype", type)}/>
+              onChange={() => {
+                App.save("gamesubtype", type)
+                if (type === "decadent") {
+                  App.state.setsDraft.length = 36;
+                  App._emit("changeSetsNumber", "setsDraft", true)
+                }
+              }
+              }/>
           )}
         </span>
       </p>

From 376aae460b97d358d879e42abea89c8f49a7f49d Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 21:41:54 -0700
Subject: [PATCH 13/36] fix: Swap regular to decadent didn't define sets

---
 frontend/src/events.js             | 20 +++++++++++---------
 frontend/src/lobby/GameOptions.jsx |  4 ++--
 frontend/src/lobby/GameTypes.jsx   |  5 ++++-
 3 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/frontend/src/events.js b/frontend/src/events.js
index 2533965b..d73c5a74 100644
--- a/frontend/src/events.js
+++ b/frontend/src/events.js
@@ -109,6 +109,15 @@ const events = {
 
     let options = {type, seats, title, isPrivate, modernOnly, totalChaos};
 
+    if (gamesubtype === "decadent") {
+      // For decadent draft, all sets are the same.
+      const sets = App.state.setsDraft;
+      const firstSet = sets[0];
+      for (let i = 1; i < sets.length; i++) {
+        sets[i] = firstSet;
+      }
+    }
+
     switch (gamesubtype) {
     case "regular":
     case "decadent": {
@@ -125,10 +134,10 @@ const events = {
     }
     App.send("create", options);
   },
-  changeSetsNumber(type, ensureAllSetsIdentical, event) {
+  changeSetsNumber(type, event) {
     event.preventDefault();
     const packsNumber = event.currentTarget.value;
-    let sets = App.state[type];
+    const sets = App.state[type];
 
     if (sets.length < packsNumber) {
       const toAdd = packsNumber - sets.length;
@@ -138,13 +147,6 @@ const events = {
       sets.splice(packsNumber);
     }
 
-    if (ensureAllSetsIdentical) {
-      const firstSet = sets[0];
-      for (let i = 1; i < sets.length; i++) {
-        sets[i] = firstSet;
-      }
-    }
-
     App.save(type, sets);
   },
   pool(cards) {
diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 80bfe408..f611bd14 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -38,7 +38,7 @@ const Regular = ({ sets, type }) => (
       Number of packs:{" "}
       <Select
         value={sets.length}
-        onChange={App._emit("changeSetsNumber", type, false)}
+        onChange={App._emit("changeSetsNumber", type)}
         opts={_.seq(12, 1)} />
     </div>
     <div className="wrapper">
@@ -63,7 +63,7 @@ const Decadent = ({ sets, type }) => (
       Number of packs:{" "}
       <Select
         value={sets.length}
-        onChange={App._emit("changeSetsNumber", type, true)}
+        onChange={App._emit("changeSetsNumber", type)}
         opts={_.seq(60, 36)} />
     </div>
     <div className="wrapper">
diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 27214352..7fdb1329 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -32,8 +32,11 @@ const GameTypes = () => {
                 App.save("gamesubtype", type)
                 if (type === "decadent") {
                   App.state.setsDraft.length = 36;
-                  App._emit("changeSetsNumber", "setsDraft", true)
                 }
+                else if (type === "regular") {
+                  App.state.setsDraft.length = 3;
+                }
+                App._emit("changeSetsNumber", "setsDraft", true)
               }
               }/>
           )}

From 52dfb34f541cbb3d591268492b814a4954f1a8cd Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 25 Apr 2020 21:43:21 -0700
Subject: [PATCH 14/36] fix: Pass nothing instead of starting new round each
 pass

---
 backend/bot.js    | 15 +++++++++++----
 backend/game.js   | 24 +++++++++---------------
 backend/human.js  | 11 ++++++++---
 backend/player.js |  3 ++-
 4 files changed, 30 insertions(+), 23 deletions(-)

diff --git a/backend/bot.js b/backend/bot.js
index b3b89c23..85aed364 100644
--- a/backend/bot.js
+++ b/backend/bot.js
@@ -1,20 +1,27 @@
-const {sample, pull} = require("lodash");
+const {sample, pull, pullAll} = require("lodash");
 const Player = require("./player");
 
 module.exports = class extends Player {
-  constructor() {
+  constructor(isDecadent) {
     super({
       isBot: true,
       isConnected: true,
       name: "bot",
-      id: ""
+      id: "",
+      isDecadent: isDecadent
     });
   }
 
   getPack(pack) {
     const randomPick = sample(pack);
     this.picks.push(randomPick.name);
-    pull(pack, randomPick);
+    
+    if (!this.isDecadent) {
+      pull(pack, randomPick);
+    } else {
+      pullAll(pack, pack);
+    }
+    
     this.emit("pass", pack);
   }
 };
diff --git a/backend/game.js b/backend/game.js
index c120a4d4..b7fd8306 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -191,7 +191,7 @@ module.exports = class Game extends Room {
     super.join(sock);
     this.logger.debug(`${sock.name} joined the game`);
 
-    const h = new Human(sock);
+    const h = new Human(sock, this.isDecadent);
     if (h.id === this.hostID) {
       h.isHost = true;
       sock.once("start", this.start.bind(this));
@@ -348,20 +348,14 @@ module.exports = class Game extends Room {
   }
 
   pass(p, pack) {
-    if (this.isDecadent) {
-      // Decadent starts new round after each pick.
-      this.startRound();
+    if (!pack.length) {
+      if (!--this.packCount)
+        this.startRound();
+      else
+        this.meta();
       return;
-    } else {
-      if (!pack.length) {
-        if (!--this.packCount)
-          this.startRound();
-        else
-          this.meta();
-        return;
-      }
     }
-
+    
     const index = this.players.indexOf(p) + this.delta;
     const nextPlayer = this.getNextPlayer(index);
     nextPlayer.getPack(pack);
@@ -529,7 +523,7 @@ module.exports = class Game extends Room {
 
       if (addBots) {
         while (this.players.length < this.seats) {
-          this.players.push(new Bot());
+          this.players.push(new Bot(this.isDecadent));
           this.bots++;
         }
       }
@@ -549,7 +543,7 @@ module.exports = class Game extends Room {
       this.logger.info(`Game ${this.id} started.\n${this.toString()}`);
       Game.broadcastGameInfo();
     } catch(err) {
-      this.logger.error(`Game ${this.id}  encountered an error while starting: ${err.stack} GameState: ${this.toString()}`);
+      this.logger.error(`Game ${this.id} encountered an error while starting: ${err.stack} GameState: ${this.toString()}`);
       this.players.forEach(player => {
         if (!player.isBot) {
           player.exit();
diff --git a/backend/human.js b/backend/human.js
index 784359aa..13e778ce 100644
--- a/backend/human.js
+++ b/backend/human.js
@@ -1,16 +1,17 @@
 const Player = require("./player");
 const util = require("./util");
 const hash = require("./hash");
-const {random} = require("lodash");
+const {random, pullAll} = require("lodash");
 const logger = require("./logger");
 
 module.exports = class extends Player {
-  constructor(sock) {
+  constructor(sock, isDecadent) {
     super({
       isBot: false,
       isConnected: true,
       name: sock.name,
-      id: sock.id
+      id: sock.id,
+      isDecadent: isDecadent
     });
     this.attach(sock);
   }
@@ -131,6 +132,10 @@ module.exports = class extends Player {
     else
       this.sendPack(next);
 
+    if (this.isDecadent) {
+      pullAll(pack, pack);
+    }
+
     this.autopick_index = -1;
     this.emit("pass", pack);
   }
diff --git a/backend/player.js b/backend/player.js
index d5494124..b618df2d 100644
--- a/backend/player.js
+++ b/backend/player.js
@@ -5,7 +5,7 @@ const {EventEmitter} = require("events");
  * Abstract class for Human and Bot
  */
 class Player extends EventEmitter {
-  constructor({id, isBot, isConnected, name}) {
+  constructor({id, isBot, isConnected, name, isDecadent}) {
     super();
 
     if (this.constructor === Player) {
@@ -36,6 +36,7 @@ class Player extends EventEmitter {
       self: 0,
       useTimer: false,
       timeLength: "Slow",
+      isDecadent  // Pick a single card and discard the rest. Boujie!
     });
   }
 

From 0b364bd2e692e535e39f39075b4027c7b4b520b1 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 12:30:41 -0700
Subject: [PATCH 15/36] fix: Re-enable frontend tests

---
 frontend/test/utils.spec.js | 74 ++++++++++++++++++-------------------
 webpack.common.js           |  5 ++-
 2 files changed, 41 insertions(+), 38 deletions(-)

diff --git a/frontend/test/utils.spec.js b/frontend/test/utils.spec.js
index 56e76f7c..1e670b67 100644
--- a/frontend/test/utils.spec.js
+++ b/frontend/test/utils.spec.js
@@ -1,44 +1,44 @@
-// const {describe, it} = require("mocha");
-// const assert = require("assert");
-// const {capitalize, toTitleCase} = require("./../src/utils");
+const {describe, it} = require("mocha");
+const assert = require("assert");
+const {capitalize, toTitleCase} = require("./../src/utils");
 
-// describe("Acceptance tests for frontend utils", () => {
-//   it("capitalize() should capitalize a single word", () => {
-//     assert(capitalize("") === "");
-//     assert(capitalize(0) === "");
-//     assert(capitalize({}) === "");
+describe("Acceptance tests for frontend utils", () => {
+  it("capitalize() should capitalize a single word", () => {
+    assert(capitalize("") === "");
+    assert(capitalize(0) === "");
+    assert(capitalize({}) === "");
 
-//     assert(capitalize("draft") === "Draft");
-//     assert(capitalize("Draft") === "Draft");
-//     assert(capitalize("DRAFT") === "Draft");
-//     assert(capitalize("draft") === "Draft");
+    assert(capitalize("draft") === "Draft");
+    assert(capitalize("Draft") === "Draft");
+    assert(capitalize("DRAFT") === "Draft");
+    assert(capitalize("draft") === "Draft");
 
-//     assert(capitalize("cube draft") === "Cube draft");
-//     assert(capitalize("CUBE DRAFT") === "Cube draft");
-//   });
-//   it("toTitleCase() should capitalize each word", () => {
-//     assert(toTitleCase("") === "");
-//     assert(toTitleCase(0) === "");
-//     assert(toTitleCase({}) === "");
+    assert(capitalize("cube draft") === "Cube draft");
+    assert(capitalize("CUBE DRAFT") === "Cube draft");
+  });
+  it("toTitleCase() should capitalize each word", () => {
+    assert(toTitleCase("") === "");
+    assert(toTitleCase(0) === "");
+    assert(toTitleCase({}) === "");
 
-//     assert(toTitleCase("draft") === "Draft");
-//     assert(toTitleCase("Draft") === "Draft");
-//     assert(toTitleCase("DRAFT") === "Draft");
-//     assert(toTitleCase("draft") === "Draft");
+    assert(toTitleCase("draft") === "Draft");
+    assert(toTitleCase("Draft") === "Draft");
+    assert(toTitleCase("DRAFT") === "Draft");
+    assert(toTitleCase("draft") === "Draft");
 
-//     assert(toTitleCase("cube draft") === "Cube Draft");
-//     assert(toTitleCase("CUBE DRAFT") === "Cube Draft");
+    assert(toTitleCase("cube draft") === "Cube Draft");
+    assert(toTitleCase("CUBE DRAFT") === "Cube Draft");
 
-//     assert(toTitleCase("cube_draft") === "Cube_draft");
-//     assert(toTitleCase("CUBE_DRAFT") === "Cube_draft");
-//     assert(toTitleCase("cube_draft", "_") === "Cube_Draft");
-//     assert(toTitleCase("CUBE_DRAFT", "_") === "Cube_Draft");
+    assert(toTitleCase("cube_draft") === "Cube_draft");
+    assert(toTitleCase("CUBE_DRAFT") === "Cube_draft");
+    assert(toTitleCase("cube_draft", "_") === "Cube_Draft");
+    assert(toTitleCase("CUBE_DRAFT", "_") === "Cube_Draft");
 
-//     assert(toTitleCase("longer sentence of wordiness") === "Longer Sentence Of Wordiness");
-//     assert(toTitleCase("LONGER SENTENCE OF WORDINESS") === "Longer Sentence Of Wordiness");
-//     assert(toTitleCase("longer_sentence_of_wordiness") === "Longer_sentence_of_wordiness");
-//     assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS") === "Longer_sentence_of_wordiness");
-//     assert(toTitleCase("longer_sentence_of_wordiness", "_") === "Longer_Sentence_Of_Wordiness");
-//     assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS", "_") === "Longer_Sentence_Of_Wordiness");
-//   });
-// });
+    assert(toTitleCase("longer sentence of wordiness") === "Longer Sentence Of Wordiness");
+    assert(toTitleCase("LONGER SENTENCE OF WORDINESS") === "Longer Sentence Of Wordiness");
+    assert(toTitleCase("longer_sentence_of_wordiness") === "Longer_sentence_of_wordiness");
+    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS") === "Longer_sentence_of_wordiness");
+    assert(toTitleCase("longer_sentence_of_wordiness", "_") === "Longer_Sentence_Of_Wordiness");
+    assert(toTitleCase("LONGER_SENTENCE_OF_WORDINESS", "_") === "Longer_Sentence_Of_Wordiness");
+  });
+});
diff --git a/webpack.common.js b/webpack.common.js
index 683af76e..cb1c20fd 100644
--- a/webpack.common.js
+++ b/webpack.common.js
@@ -43,7 +43,10 @@ module.exports = {
     rules: [
       {
         test: /\.(js|jsx)$/,
-        exclude: /(node_modules|\.spec\.js$)/,
+        exclude: [
+          "./node_modules",
+          "./frontend/test"
+        ],
         use: {
           loader: "babel-loader",
           options: {

From 759e3f1b056607b8498fa337cd3d0e7cae1fedd3 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 12:59:55 -0700
Subject: [PATCH 16/36] fix: Remove frontend/test from webpack copy

---
 webpack.common.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/webpack.common.js b/webpack.common.js
index cb1c20fd..ed00ba7b 100644
--- a/webpack.common.js
+++ b/webpack.common.js
@@ -22,7 +22,7 @@ module.exports = {
       cache: false
     }),
     new CopyWebpackPlugin([
-      { from: "frontend", ignore: ["*.tpl", "src/**/*"] }
+      { from: "frontend", ignore: ["*.tpl", "src/**/*", "test/**/*"] }
     ], {
       copyUnmodified: true
     }),
@@ -44,8 +44,8 @@ module.exports = {
       {
         test: /\.(js|jsx)$/,
         exclude: [
-          "./node_modules",
-          "./frontend/test"
+          /node_modules/,
+          /\.spec\.js$/
         ],
         use: {
           loader: "babel-loader",

From 26668c1435a952ac66ea20fdf22460704404b7fc Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 13:18:43 -0700
Subject: [PATCH 17/36] feedback: Compress set info to e.g. 40x IKO

---
 backend/game.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/backend/game.js b/backend/game.js
index b7fd8306..5fc08b39 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -38,7 +38,9 @@ module.exports = class Game extends Room {
       this.rounds = this.sets.length;
       break;
     case "decadent draft":
-      this.packsInfo = this.sets.join(" / ");
+      // Sets should all be the same and there can be a large number of them.
+      // Compress this info into e.g. "36x IKO" instead of "IKO / IKO / ...".
+      this.packsInfo = `${this.sets.length}x ${this.sets[0]}`;
       this.rounds = this.sets.length;
       this.isDecadent = true;
       break;

From e3a722c970d85b0f94872fd1b25e716c1c4640be Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 13:34:12 -0700
Subject: [PATCH 18/36] fix: Show if game option unavailable

---
 frontend/src/lobby/GameOptions.jsx | 10 +++++++++-
 frontend/style.css                 |  5 +++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index f611bd14..406b4a65 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -28,7 +28,7 @@ const GameOptions = () => {
   case "chaos sealed":
     return <Chaos packsNumber={"chaosSealedPacksNumber"}/>;
   default:
-    return <Regular sets={setsDraft} type={"setsDraft"} />;
+    return <Unavailable/>;
   }
 };
 
@@ -129,4 +129,12 @@ Chaos.propTypes = {
   packsNumber: PropTypes.number
 };
 
+const Unavailable = () => (
+  <Fragment>
+    <div className="unavailable-game-options">
+      This combination of game mode + game type unavailable!
+    </div>
+  </Fragment>
+);
+
 export default GameOptions;
diff --git a/frontend/style.css b/frontend/style.css
index d8a89305..59882603 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -622,3 +622,8 @@ input[type="checkbox"], input[type="radio"] {
   display: flex;
   flex-direction: column;
 }
+
+.unavailable-game-options {
+  color: red;
+  font-weight: bold;
+}
\ No newline at end of file

From bdb576133dd13e0e40610265d7820084623f85eb Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 16:04:07 -0700
Subject: [PATCH 19/36] docs: Add blockquote describing the new mode

---
 frontend/src/lobby/GameOptions.jsx | 4 ++++
 frontend/style.css                 | 8 ++++++++
 2 files changed, 12 insertions(+)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 406b4a65..701ee453 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -59,6 +59,10 @@ const Sets = ({ sets, type }) => (
 
 const Decadent = ({ sets, type }) => (
   <Fragment>
+    <blockquote className="game-mode-description">
+      <p>Also known as "Rich Man's Draft" or "Decadent Sealed".</p>
+      <p>Each round, each player chooses a single card from the pack, discards the rest, then gets a new pack.</p>
+    </blockquote>
     <div>
       Number of packs:{" "}
       <Select
diff --git a/frontend/style.css b/frontend/style.css
index 59882603..adf520f2 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -626,4 +626,12 @@ input[type="checkbox"], input[type="radio"] {
 .unavailable-game-options {
   color: red;
   font-weight: bold;
+}
+
+.game-mode-description {
+  background: #ddd;
+  border-left: 5px solid #bbb;
+  margin: 0.5em 10px;
+  padding: 1px 1px 1px 10px;
+  font-style: italic;
 }
\ No newline at end of file

From 59b681d2dc026301728b1621c86a63c7914eb01f Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 21:34:07 -0700
Subject: [PATCH 20/36] fix: TextArea should use link not 'list'

---
 frontend/src/components/TextArea.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/TextArea.jsx b/frontend/src/components/TextArea.jsx
index dbb29b27..9c7f8e78 100644
--- a/frontend/src/components/TextArea.jsx
+++ b/frontend/src/components/TextArea.jsx
@@ -7,7 +7,7 @@ const TextArea = ({link, ...rest}) => (
   <textarea
     style={ {"overflowY": "scroll", "height": "150px"}}
     onChange=
-      { (e) => { App.save("list", e.currentTarget.value); } }
+      { (e) => { App.save(link, e.currentTarget.value); } }
     value={App.state[link]}
     {...rest} />
 );

From ee6463723ff099a56ad9acf401922fa700cc13ed Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sun, 26 Apr 2020 21:36:29 -0700
Subject: [PATCH 21/36] refactor: Remove now-unused useForAllSets prop

---
 frontend/src/lobby/GameOptions.jsx |  2 +-
 frontend/src/lobby/Set.jsx         | 10 ++--------
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 701ee453..c6b5dbf3 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -71,7 +71,7 @@ const Decadent = ({ sets, type }) => (
         opts={_.seq(60, 36)} />
     </div>
     <div className="wrapper">
-      <Set type={type} selectedSet={sets[0]} index={0} key={0} useForAllSets={true} />
+      <Set type={type} selectedSet={sets[0]} index={0} key={0}/>
     </div>
   </Fragment>
 );
diff --git a/frontend/src/lobby/Set.jsx b/frontend/src/lobby/Set.jsx
index 3eb3ef2d..78161767 100644
--- a/frontend/src/lobby/Set.jsx
+++ b/frontend/src/lobby/Set.jsx
@@ -4,17 +4,11 @@ import PropTypes from "prop-types";
 import App from "../app";
 import { toTitleCase } from "../utils";
 
-const Set = ({ index, selectedSet, type, useForAllSets=false }) => {
+const Set = ({ index, selectedSet, type }) => {
   const onSetChange = (e) => {
     const chosenSet = e.currentTarget.value;
     let sets = App.state[type];
-    if (!useForAllSets) {
-      sets[index] = chosenSet;
-    } else {
-      for (let i = 0; i < sets.length; i++) {
-        sets[i] = chosenSet;
-      }
-    }
+    sets[index] = chosenSet;
     App.save(type, App.state[type]);
   };
   let groups = [];

From 2e29400d94b9e8cf5f5f39191619410cbae3cc48 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Mon, 27 Apr 2020 18:16:48 -0700
Subject: [PATCH 22/36] feedback: Only show supported game mode selections

---
 frontend/src/lobby/GameOptions.jsx | 10 ----------
 frontend/src/lobby/GameTypes.jsx   | 32 +++++++++++++++++++++---------
 frontend/style.css                 |  8 +-------
 3 files changed, 24 insertions(+), 26 deletions(-)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index c6b5dbf3..5655ba20 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -27,8 +27,6 @@ const GameOptions = () => {
     return <Chaos packsNumber={"chaosDraftPacksNumber"} />;
   case "chaos sealed":
     return <Chaos packsNumber={"chaosSealedPacksNumber"}/>;
-  default:
-    return <Unavailable/>;
   }
 };
 
@@ -133,12 +131,4 @@ Chaos.propTypes = {
   packsNumber: PropTypes.number
 };
 
-const Unavailable = () => (
-  <Fragment>
-    <div className="unavailable-game-options">
-      This combination of game mode + game type unavailable!
-    </div>
-  </Fragment>
-);
-
 export default GameOptions;
diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 7fdb1329..0b1bd05e 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -6,24 +6,38 @@ import App from "../app";
 import { toTitleCase } from "../utils";
 
 const GameTypes = () => {
-  const types = ["draft", "sealed"];
-  const subtypes = ["regular", "cube", "chaos", "decadent"];
+  const gameOptions = {
+    draft: ["regular", "cube", "chaos", "decadent"],
+    sealed: ["regular", "cube", "chaos"]
+  }
+
+  const getAvailableTypes = () => Object.keys(gameOptions);
+  const getAvailableSubTypes = (gameType) => gameOptions[gameType];
   return (
     <div>
       <p>Game type:{" "}
         <span className='connected-container'>
-          {types.map((type, key) =>
+          {getAvailableTypes().map((gameType, key) =>
             <GameType name={"type"}
-              type={type}
+              type={gameType}
               key={key}
-              isChecked={App.state.gametype === type}
-              onChange={() => App.save("gametype", type)}/>
+              isChecked={App.state.gametype === gameType}
+              onChange={() => {
+                App.save("gametype", gameType)
+                const availableSubtypes = getAvailableSubTypes(gameType)
+                if (!availableSubtypes.includes(App.state.gamesubtype)) {
+                  // Reset to first available subtype if the currently-selected
+                  // subtype is not available for the newly-selected type.
+                  App.save("gamesubtype", availableSubtypes[0])
+                }
+              }}
+            />
           )}
         </span>
       </p>
       <p>Game mode:{" "}
         <span className='connected-container'>
-          {subtypes.map((type, key) =>
+          {getAvailableSubTypes(App.state.gametype).map((type, key) =>
             <GameType name={"subtype"}
               type={type}
               key={key}
@@ -37,8 +51,8 @@ const GameTypes = () => {
                   App.state.setsDraft.length = 3;
                 }
                 App._emit("changeSetsNumber", "setsDraft", true)
-              }
-              }/>
+              }}
+            />
           )}
         </span>
       </p>
diff --git a/frontend/style.css b/frontend/style.css
index adf520f2..40c44437 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -623,15 +623,9 @@ input[type="checkbox"], input[type="radio"] {
   flex-direction: column;
 }
 
-.unavailable-game-options {
-  color: red;
-  font-weight: bold;
-}
-
 .game-mode-description {
   background: #ddd;
   border-left: 5px solid #bbb;
-  margin: 0.5em 10px;
   padding: 1px 1px 1px 10px;
   font-style: italic;
-}
\ No newline at end of file
+}

From 5850b20df108e249facc6d4d1d19f1ad789e196d Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Mon, 27 Apr 2020 20:59:05 -0700
Subject: [PATCH 23/36] docs: Descriptions for each game mode

---
 frontend/src/lobby/GameOptions.jsx | 92 +++++++++++++++++++++++++-----
 frontend/style.css                 |  1 +
 2 files changed, 78 insertions(+), 15 deletions(-)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 5655ba20..b7187a57 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -14,22 +14,54 @@ const GameOptions = () => {
 
   switch (`${gamesubtype} ${gametype}`) {
   case "regular draft":
-    return <Regular sets={setsDraft} type={"setsDraft"} />;
+    return <RegularDraft sets={setsDraft} type={"setsDraft"} />;
   case "regular sealed":
-    return <Regular sets={setsSealed} type={"setsSealed"} />;
+    return <RegularSealed sets={setsSealed} type={"setsSealed"} />;
   case "decadent draft":
     return <Decadent sets={setsDraft} type={"setsDraft"} />;
   case "cube draft":
-    return <CubeDraft />;
+    return <CubeDraft/>;
   case "cube sealed":
-    return <CubeSealed />;
+    return <CubeSealed/>;
   case "chaos draft":
-    return <Chaos packsNumber={"chaosDraftPacksNumber"} />;
+    return <ChaosDraft/>
   case "chaos sealed":
-    return <Chaos packsNumber={"chaosSealedPacksNumber"}/>;
+    return <ChaosSealed/>;
   }
 };
 
+const RegularDraft = ({sets, type}) => (
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>Each player starts with {sets.length} booster packs.</p>
+      <p>To begin, each player opens one pack, chooses (drafts) one card, and passes the rest to the next player.</p>
+      <p>That player drafts one card from what's left, passes it, and so on until no cards are left.</p>
+      <p>Then everyone opens their next pack, starting a new round. Packs are passed in alternating directions each round.</p>
+      <p>Once all cards have been drafted, each player builds a deck out of the cards they drafted.</p>
+    </blockquote>
+    <Regular sets={sets} type={type} />
+  </Fragment>
+);
+
+RegularDraft.propTypes = {
+  sets: PropTypes.array,
+  type: PropTypes.string
+}
+
+const RegularSealed = ({sets, type}) => (
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>Each player opens {sets.length} booster packs and builds a deck using any of those cards.</p>
+    </blockquote>
+    <Regular sets={sets} type={type} />
+  </Fragment>
+);
+
+RegularSealed.propTypes = {
+  sets: PropTypes.array,
+  type: PropTypes.string
+}
+
 const Regular = ({ sets, type }) => (
   <Fragment>
     <div>
@@ -59,7 +91,7 @@ const Decadent = ({ sets, type }) => (
   <Fragment>
     <blockquote className="game-mode-description">
       <p>Also known as "Rich Man's Draft" or "Decadent Sealed".</p>
-      <p>Each round, each player chooses a single card from the pack, discards the rest, then gets a new pack.</p>
+      <p>Like a regular draft except each player chooses one card from the pack, discards the rest, then opens a new pack.</p>
     </blockquote>
     <div>
       Number of packs:{" "}
@@ -80,17 +112,27 @@ Decadent.propTypes = {
 };
 
 const CubeDraft = () => (
-  <div>
-    <CubeList />
-    <CubeOptions />
-  </div>
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>A draft where each pack is a random set of cards from a large pool of cards called the "cube".</p>
+    </blockquote>
+    <div>
+      <CubeList />
+      <CubeOptions />
+    </div>
+  </Fragment>
 );
 
 const CubeSealed = () => (
-  <div>
-    <CubeList />
-    <CubeSealedOptions />
-  </div>
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>Like sealed except each pack is a random set of cards from a large pool of cards called the "cube".</p>
+    </blockquote>
+    <div>
+      <CubeList />
+      <CubeSealedOptions />
+    </div>
+  </Fragment>
 );
 
 const CubeSealedOptions = () => (
@@ -109,6 +151,26 @@ const CubeOptions = () => (
   </div>
 );
 
+const ChaosDraft = () => (
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>A draft where each booster pack is from a random set.</p>
+      <p>Take it up a notch with total chaos to build booster packs where each card in the pack is from a random set!</p>
+    </blockquote>
+    <Chaos packsNumber={"chaosDraftPacksNumber"} />
+  </Fragment>
+);
+
+const ChaosSealed = () => (
+  <Fragment>
+    <blockquote className="game-mode-description">
+      <p>Like sealed except each booster pack is from a random set.</p>
+      <p>Take it up a notch with total chaos to build booster packs where each card in the pack is from a random set!</p>
+    </blockquote>
+    <Chaos packsNumber={"chaosSealedPacksNumber"}/>
+  </Fragment>
+);
+
 const Chaos = ({ packsNumber }) => (
   <div>
     <div>
diff --git a/frontend/style.css b/frontend/style.css
index 40c44437..4103bc1b 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -626,6 +626,7 @@ input[type="checkbox"], input[type="radio"] {
 .game-mode-description {
   background: #ddd;
   border-left: 5px solid #bbb;
+  margin-left: 0px;
   padding: 1px 1px 1px 10px;
   font-style: italic;
 }

From 8b109b7d4174f1b5aa548ce38f76940f75e97341 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Mon, 27 Apr 2020 21:32:15 -0700
Subject: [PATCH 24/36] feedback: Re-use lodash capitalize()

---
 frontend/src/utils.js       |  8 +-------
 frontend/test/utils.spec.js | 15 +--------------
 2 files changed, 2 insertions(+), 21 deletions(-)

diff --git a/frontend/src/utils.js b/frontend/src/utils.js
index 3c7d9b3d..47146434 100644
--- a/frontend/src/utils.js
+++ b/frontend/src/utils.js
@@ -1,9 +1,4 @@
-const capitalize = (word) => {
-  if (!word || typeof word !== "string") {
-    return "";
-  }
-  return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
-};
+const capitalize = require("lodash/capitalize");
 
 const toTitleCase = (sentence, separator=" ") => {
   if (!sentence || typeof sentence !== "string") {
@@ -16,6 +11,5 @@ const toTitleCase = (sentence, separator=" ") => {
 };
 
 module.exports = {
-  capitalize,
   toTitleCase
 };
diff --git a/frontend/test/utils.spec.js b/frontend/test/utils.spec.js
index 1e670b67..a6dca021 100644
--- a/frontend/test/utils.spec.js
+++ b/frontend/test/utils.spec.js
@@ -1,21 +1,8 @@
 const {describe, it} = require("mocha");
 const assert = require("assert");
-const {capitalize, toTitleCase} = require("./../src/utils");
+const {toTitleCase} = require("./../src/utils");
 
 describe("Acceptance tests for frontend utils", () => {
-  it("capitalize() should capitalize a single word", () => {
-    assert(capitalize("") === "");
-    assert(capitalize(0) === "");
-    assert(capitalize({}) === "");
-
-    assert(capitalize("draft") === "Draft");
-    assert(capitalize("Draft") === "Draft");
-    assert(capitalize("DRAFT") === "Draft");
-    assert(capitalize("draft") === "Draft");
-
-    assert(capitalize("cube draft") === "Cube draft");
-    assert(capitalize("CUBE DRAFT") === "Cube draft");
-  });
   it("toTitleCase() should capitalize each word", () => {
     assert(toTitleCase("") === "");
     assert(toTitleCase(0) === "");

From 61ce34a0d536428f0a84cd78b15dc0b8d4c754c1 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 19:33:57 -0700
Subject: [PATCH 25/36] fix: Chaos component prop type should be string

---
 frontend/src/lobby/GameOptions.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index b7187a57..abf3eeb1 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -190,7 +190,7 @@ const Chaos = ({ packsNumber }) => (
 );
 
 Chaos.propTypes = {
-  packsNumber: PropTypes.number
+  packsNumber: PropTypes.string
 };
 
 export default GameOptions;

From 116c755f8afd269c9ce6e5127b7945cd20d86c9b Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 19:40:00 -0700
Subject: [PATCH 26/36] fix: Remove unused propType on Set

---
 frontend/src/lobby/Set.jsx | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/frontend/src/lobby/Set.jsx b/frontend/src/lobby/Set.jsx
index 78161767..e1da854e 100644
--- a/frontend/src/lobby/Set.jsx
+++ b/frontend/src/lobby/Set.jsx
@@ -34,8 +34,7 @@ const Set = ({ index, selectedSet, type }) => {
 Set.propTypes = {
   index: PropTypes.number,
   selectedSet: PropTypes.string,
-  type: PropTypes.string,
-  useForAllSets: PropTypes.bool
+  type: PropTypes.string
 };
 
 export default Set;

From 930dab5d3ed746b21ef2863ef8dc392e98189253 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 19:53:48 -0700
Subject: [PATCH 27/36] refactor: Extract method for assigning default set

---
 frontend/src/app.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/frontend/src/app.js b/frontend/src/app.js
index 93359acb..98847ecc 100644
--- a/frontend/src/app.js
+++ b/frontend/src/app.js
@@ -170,12 +170,12 @@ let App = {
   },
   set(state) {
     Object.assign(App.state, state);
-    // Set default sets
-    if ( App.state.setsSealed.length === 0 && App.state.latestSet) {
-      App.state.setsSealed = times(6, constant(App.state.latestSet.code));
-    }
-    if ( App.state.setsDraft.length === 0 && App.state.latestSet) {
-      App.state.setsDraft = times(3, constant(App.state.latestSet.code));
+    if (App.state.latestSet) {
+      // Default sets to the latest set.
+      const defaultSetCode = App.state.latestSet.code;
+      const multipleOfDefaultSet = (desiredLength) => times(desiredLength, constant(defaultSetCode));
+      App.state.setsSealed = multipleOfDefaultSet(6);
+      App.state.setsDraft = multipleOfDefaultSet(3);
     }
     App.update();
   },

From e6fd5e583f47f9f5ecdc236fb94d156992346acb Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 20:08:07 -0700
Subject: [PATCH 28/36] feedback: Use setsDecadentDraft instead of setsDraft

---
 frontend/src/app.js                |  8 +++++---
 frontend/src/events.js             | 15 ++++-----------
 frontend/src/lobby/GameOptions.jsx |  6 +++---
 frontend/src/lobby/GameTypes.jsx   | 11 +----------
 frontend/src/lobby/Set.jsx         | 13 ++++++++++---
 5 files changed, 23 insertions(+), 30 deletions(-)

diff --git a/frontend/src/app.js b/frontend/src/app.js
index 98847ecc..5b255a6a 100644
--- a/frontend/src/app.js
+++ b/frontend/src/app.js
@@ -35,6 +35,7 @@ let App = {
     sets: [],
     setsDraft: [],
     setsSealed: [],
+    setsDecadentDraft: [],
     availableSets: {},
     list: "",
     cards: 15,
@@ -173,9 +174,10 @@ let App = {
     if (App.state.latestSet) {
       // Default sets to the latest set.
       const defaultSetCode = App.state.latestSet.code;
-      const multipleOfDefaultSet = (desiredLength) => times(desiredLength, constant(defaultSetCode));
-      App.state.setsSealed = multipleOfDefaultSet(6);
-      App.state.setsDraft = multipleOfDefaultSet(3);
+      const replicateDefaultSet = (desiredLength) => times(desiredLength, constant(defaultSetCode));
+      App.state.setsSealed = replicateDefaultSet(6);
+      App.state.setsDraft = replicateDefaultSet(3);
+      App.state.setsDecadentDraft = replicateDefaultSet(36);
     }
     App.update();
   },
diff --git a/frontend/src/events.js b/frontend/src/events.js
index d73c5a74..766255d5 100644
--- a/frontend/src/events.js
+++ b/frontend/src/events.js
@@ -109,22 +109,15 @@ const events = {
 
     let options = {type, seats, title, isPrivate, modernOnly, totalChaos};
 
-    if (gamesubtype === "decadent") {
-      // For decadent draft, all sets are the same.
-      const sets = App.state.setsDraft;
-      const firstSet = sets[0];
-      for (let i = 1; i < sets.length; i++) {
-        sets[i] = firstSet;
-      }
-    }
-
     switch (gamesubtype) {
-    case "regular":
-    case "decadent": {
+    case "regular": {
       const {setsDraft, setsSealed} = App.state;
       options.sets = gametype === "sealed" ? setsSealed : setsDraft;
       break;
     }
+    case "decadent":
+      options.sets = App.state.setsDecadentDraft;
+      break;
     case "cube":
       options.cube = parseCubeOptions();
       break;
diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index abf3eeb1..8fc69e3f 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -10,7 +10,7 @@ import Set from "./Set";
 import CubeList from "./CubeList";
 
 const GameOptions = () => {
-  const { setsDraft, setsSealed, gametype, gamesubtype } = App.state;
+  const { setsDraft, setsSealed, setsDecadentDraft, gametype, gamesubtype } = App.state;
 
   switch (`${gamesubtype} ${gametype}`) {
   case "regular draft":
@@ -18,7 +18,7 @@ const GameOptions = () => {
   case "regular sealed":
     return <RegularSealed sets={setsSealed} type={"setsSealed"} />;
   case "decadent draft":
-    return <Decadent sets={setsDraft} type={"setsDraft"} />;
+    return <Decadent sets={setsDecadentDraft} type={"setsDecadentDraft"} />;
   case "cube draft":
     return <CubeDraft/>;
   case "cube sealed":
@@ -101,7 +101,7 @@ const Decadent = ({ sets, type }) => (
         opts={_.seq(60, 36)} />
     </div>
     <div className="wrapper">
-      <Set type={type} selectedSet={sets[0]} index={0} key={0}/>
+      <Set type={type} selectedSet={sets[0]} index={0} key={0} useForAllSets={true} />
     </div>
   </Fragment>
 );
diff --git a/frontend/src/lobby/GameTypes.jsx b/frontend/src/lobby/GameTypes.jsx
index 0b1bd05e..6cc789a9 100644
--- a/frontend/src/lobby/GameTypes.jsx
+++ b/frontend/src/lobby/GameTypes.jsx
@@ -42,16 +42,7 @@ const GameTypes = () => {
               type={type}
               key={key}
               isChecked={App.state.gamesubtype === type}
-              onChange={() => {
-                App.save("gamesubtype", type)
-                if (type === "decadent") {
-                  App.state.setsDraft.length = 36;
-                }
-                else if (type === "regular") {
-                  App.state.setsDraft.length = 3;
-                }
-                App._emit("changeSetsNumber", "setsDraft", true)
-              }}
+              onChange={() => App.save("gamesubtype", type)}
             />
           )}
         </span>
diff --git a/frontend/src/lobby/Set.jsx b/frontend/src/lobby/Set.jsx
index e1da854e..3eb3ef2d 100644
--- a/frontend/src/lobby/Set.jsx
+++ b/frontend/src/lobby/Set.jsx
@@ -4,11 +4,17 @@ import PropTypes from "prop-types";
 import App from "../app";
 import { toTitleCase } from "../utils";
 
-const Set = ({ index, selectedSet, type }) => {
+const Set = ({ index, selectedSet, type, useForAllSets=false }) => {
   const onSetChange = (e) => {
     const chosenSet = e.currentTarget.value;
     let sets = App.state[type];
-    sets[index] = chosenSet;
+    if (!useForAllSets) {
+      sets[index] = chosenSet;
+    } else {
+      for (let i = 0; i < sets.length; i++) {
+        sets[i] = chosenSet;
+      }
+    }
     App.save(type, App.state[type]);
   };
   let groups = [];
@@ -34,7 +40,8 @@ const Set = ({ index, selectedSet, type }) => {
 Set.propTypes = {
   index: PropTypes.number,
   selectedSet: PropTypes.string,
-  type: PropTypes.string
+  type: PropTypes.string,
+  useForAllSets: PropTypes.bool
 };
 
 export default Set;

From 181700fba013e03919ec93b9f4a507140015eaa4 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 20:24:22 -0700
Subject: [PATCH 29/36] refactor: Extract SetReplicated component

---
 frontend/src/lobby/GameOptions.jsx   |  3 ++-
 frontend/src/lobby/Set.jsx           | 30 +++++-----------------------
 frontend/src/lobby/SetChoices.jsx    | 24 ++++++++++++++++++++++
 frontend/src/lobby/SetReplicated.jsx | 30 ++++++++++++++++++++++++++++
 4 files changed, 61 insertions(+), 26 deletions(-)
 create mode 100644 frontend/src/lobby/SetChoices.jsx
 create mode 100644 frontend/src/lobby/SetReplicated.jsx

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index 8fc69e3f..da7346bc 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -7,6 +7,7 @@ import Checkbox from "../components/Checkbox";
 import Select from "../components/Select";
 
 import Set from "./Set";
+import SetReplicated from "./SetReplicated";
 import CubeList from "./CubeList";
 
 const GameOptions = () => {
@@ -101,7 +102,7 @@ const Decadent = ({ sets, type }) => (
         opts={_.seq(60, 36)} />
     </div>
     <div className="wrapper">
-      <Set type={type} selectedSet={sets[0]} index={0} key={0} useForAllSets={true} />
+      <SetReplicated type={type} selectedSet={sets[0]} />
     </div>
   </Fragment>
 );
diff --git a/frontend/src/lobby/Set.jsx b/frontend/src/lobby/Set.jsx
index 3eb3ef2d..4e893be5 100644
--- a/frontend/src/lobby/Set.jsx
+++ b/frontend/src/lobby/Set.jsx
@@ -2,37 +2,18 @@ import React from "react";
 import PropTypes from "prop-types";
 
 import App from "../app";
-import { toTitleCase } from "../utils";
+import SetChoices from "./SetChoices";
 
-const Set = ({ index, selectedSet, type, useForAllSets=false }) => {
+const Set = ({ index, selectedSet, type }) => {
   const onSetChange = (e) => {
     const chosenSet = e.currentTarget.value;
     let sets = App.state[type];
-    if (!useForAllSets) {
-      sets[index] = chosenSet;
-    } else {
-      for (let i = 0; i < sets.length; i++) {
-        sets[i] = chosenSet;
-      }
-    }
+    sets[index] = chosenSet;
     App.save(type, App.state[type]);
   };
-  let groups = [];
-  for (let setType in App.state.availableSets) {
-    const allSets = App.state.availableSets[setType];
-    let options = [];
-    allSets.forEach(({ code, name }) => {
-      options.push(
-        <option value={code} key={code}>{name}</option>
-      );
-    });
-    groups.push(
-      <optgroup label={toTitleCase(setType, "_")} key={setType}>{options}</optgroup>
-    );
-  }
   return (
     <select value={selectedSet} onChange={onSetChange} key={index}>
-      {groups}
+      <SetChoices/>
     </select>
   );
 };
@@ -40,8 +21,7 @@ const Set = ({ index, selectedSet, type, useForAllSets=false }) => {
 Set.propTypes = {
   index: PropTypes.number,
   selectedSet: PropTypes.string,
-  type: PropTypes.string,
-  useForAllSets: PropTypes.bool
+  type: PropTypes.string
 };
 
 export default Set;
diff --git a/frontend/src/lobby/SetChoices.jsx b/frontend/src/lobby/SetChoices.jsx
new file mode 100644
index 00000000..90bbc942
--- /dev/null
+++ b/frontend/src/lobby/SetChoices.jsx
@@ -0,0 +1,24 @@
+import React from "react";
+
+import App from "../app";
+import { toTitleCase } from "../utils";
+
+
+const SetChoices = () => {
+  let groups = [];
+  for (let setType in App.state.availableSets) {
+    const allSets = App.state.availableSets[setType];
+    let options = [];
+    allSets.forEach(({ code, name }) => {
+      options.push(
+        <option value={code} key={code}>{name}</option>
+      );
+    });
+    groups.push(
+      <optgroup label={toTitleCase(setType, "_")} key={setType}>{options}</optgroup>
+    );
+  }
+  return groups;
+}
+
+export default SetChoices;
diff --git a/frontend/src/lobby/SetReplicated.jsx b/frontend/src/lobby/SetReplicated.jsx
new file mode 100644
index 00000000..8a2c88b9
--- /dev/null
+++ b/frontend/src/lobby/SetReplicated.jsx
@@ -0,0 +1,30 @@
+import React from "react";
+import PropTypes from "prop-types";
+
+import App from "../app";
+import SetChoices from "./SetChoices";
+
+const SetReplicated = ({ selectedSet, type }) => {
+  // A single dropdown which is used to fill an entire array
+  // of sets with the same selection.
+  const onSetChange = (e) => {
+    const chosenSet = e.currentTarget.value;
+    let sets = App.state[type];
+    for (let i = 0; i < sets.length; i++) {
+      sets[i] = chosenSet;
+    }
+    App.save(type, App.state[type]);
+  };
+  return (
+    <select value={selectedSet} onChange={onSetChange} key={0}>
+      <SetChoices/>
+    </select>
+  );
+};
+
+SetReplicated.propTypes = {
+  selectedSet: PropTypes.string,
+  type: PropTypes.string
+};
+
+export default SetReplicated;

From 1895365ccf2107583826356130fdcbd60eb423e0 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Wed, 29 Apr 2020 20:29:18 -0700
Subject: [PATCH 30/36] refactor: Pull cube list style into css class

---
 frontend/src/components/TextArea.jsx | 7 +++----
 frontend/src/lobby/CubeList.jsx      | 2 +-
 frontend/style.css                   | 5 +++++
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/frontend/src/components/TextArea.jsx b/frontend/src/components/TextArea.jsx
index 9c7f8e78..ed9a2444 100644
--- a/frontend/src/components/TextArea.jsx
+++ b/frontend/src/components/TextArea.jsx
@@ -5,11 +5,10 @@ import App from "../app";
 
 const TextArea = ({link, ...rest}) => (
   <textarea
-    style={ {"overflowY": "scroll", "height": "150px"}}
-    onChange=
-      { (e) => { App.save(link, e.currentTarget.value); } }
+    onChange={(e) => { App.save(link, e.currentTarget.value); }}
     value={App.state[link]}
-    {...rest} />
+    {...rest}
+  />
 );
 
 TextArea.propTypes = {
diff --git a/frontend/src/lobby/CubeList.jsx b/frontend/src/lobby/CubeList.jsx
index c54b4254..af1a0d57 100644
--- a/frontend/src/lobby/CubeList.jsx
+++ b/frontend/src/lobby/CubeList.jsx
@@ -13,7 +13,7 @@ const CubeList = () => {
   return (<div id='cube-list'>
     <div className='column'>
       <div>{`One card per line! (${cubeListLength} cards)`}</div>
-      <TextArea
+      <TextArea className="cube-list"
         placeholder='cube list'
         link='list'
       />
diff --git a/frontend/style.css b/frontend/style.css
index 4103bc1b..af1d25b1 100644
--- a/frontend/style.css
+++ b/frontend/style.css
@@ -630,3 +630,8 @@ input[type="checkbox"], input[type="radio"] {
   padding: 1px 1px 1px 10px;
   font-style: italic;
 }
+
+.cube-list {
+  overflow-y: "scroll";
+  height: 150px;
+}
\ No newline at end of file

From f9c3eb58518d384725070526bffdf320c411811f Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Thu, 30 Apr 2020 16:53:12 -0700
Subject: [PATCH 31/36] feedback: If decadent draft, hide bots and shuffle
 players options

---
 frontend/src/game/StartPanel.jsx | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/frontend/src/game/StartPanel.jsx b/frontend/src/game/StartPanel.jsx
index 051ff0f1..a66d020e 100644
--- a/frontend/src/game/StartPanel.jsx
+++ b/frontend/src/game/StartPanel.jsx
@@ -43,8 +43,14 @@ const Options = () => {
   const timers = ["Fast", "Moderate", "Slow", "Leisurely"];
   return (
     <span>
-      <Checkbox side="left" link="addBots" text="Fill empty seats with Bots"/>
-      <Checkbox side="left" link="shufflePlayers" text="Random seating"/>
+      {showAddBotsCheckbox()
+        ? <Checkbox side="left" link="addBots" text="Fill empty seats with Bots"/>
+        : null
+      }
+      {showShufflePlayersCheckbox()
+        ? <Checkbox side="left" link="shufflePlayers" text="Random seating"/>
+        : null
+      }
       <div>
         <Checkbox side="left" link="useTimer" text="Timer: "/>
         <Select link="timerLength" opts={timers} disabled={!useTimer}/>
@@ -53,4 +59,14 @@ const Options = () => {
   );
 };
 
+const showAddBotsCheckbox = () => {
+  // No need for bots in decadent draft since there's no passing.
+  return !App.state.isDecadentDraft;
+}
+
+const showShufflePlayersCheckbox = () => {
+  // No need to shuffle players in decadent draft because there's no passing.
+  return !App.state.isDecadentDraft;
+}
+
 export default StartPanel;

From f9bef937b1889b7f1446a2e342730fe11ca4ef71 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Thu, 30 Apr 2020 20:49:25 -0700
Subject: [PATCH 32/36] feedback: Use pick delegate

---
 backend/bot.js    | 15 ++++--------
 backend/game.js   | 58 ++++++++++++++++++++++++++++++++++++++++++++++-
 backend/human.js  | 35 ++++------------------------
 backend/player.js |  5 ++--
 4 files changed, 68 insertions(+), 45 deletions(-)

diff --git a/backend/bot.js b/backend/bot.js
index 85aed364..b3b89c23 100644
--- a/backend/bot.js
+++ b/backend/bot.js
@@ -1,27 +1,20 @@
-const {sample, pull, pullAll} = require("lodash");
+const {sample, pull} = require("lodash");
 const Player = require("./player");
 
 module.exports = class extends Player {
-  constructor(isDecadent) {
+  constructor() {
     super({
       isBot: true,
       isConnected: true,
       name: "bot",
-      id: "",
-      isDecadent: isDecadent
+      id: ""
     });
   }
 
   getPack(pack) {
     const randomPick = sample(pack);
     this.picks.push(randomPick.name);
-    
-    if (!this.isDecadent) {
-      pull(pack, randomPick);
-    } else {
-      pullAll(pack, pack);
-    }
-    
+    pull(pack, randomPick);
     this.emit("pass", pack);
   }
 };
diff --git a/backend/game.js b/backend/game.js
index 5fc08b39..ec82c7de 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -193,7 +193,63 @@ module.exports = class Game extends Room {
     super.join(sock);
     this.logger.debug(`${sock.name} joined the game`);
 
-    const h = new Human(sock, this.isDecadent);
+    function regularDraftPickDelegate(index) {
+      const pack = this.packs.shift();
+      const card = pack.splice(index, 1)[0];
+  
+      this.draftLog.pack.push( [`--> ${card.name}`].concat(pack.map(x => `    ${x.name}`)) );
+      this.updateDraftStats(this.draftLog.pack[ this.draftLog.pack.length-1 ], this.pool);
+  
+      let pickcard = card.name;
+      if (card.foil === true)
+        pickcard = "*" + pickcard + "*";
+  
+      this.pool.push(card);
+      this.picks.push(pickcard);
+      this.send("add", card);
+  
+      let [next] = this.packs;
+      if (!next)
+        this.time = 0;
+      else
+        this.sendPack(next);
+  
+      this.autopick_index = -1;
+      this.emit("pass", pack);
+    }
+
+    function decadentDraftPickDelegate(index) {
+      const pack = this.packs.shift();
+      const card = pack.splice(index, 1)[0];
+  
+      this.draftLog.pack.push( [`--> ${card.name}`].concat(pack.map(x => `    ${x.name}`)) );
+      this.updateDraftStats(this.draftLog.pack[ this.draftLog.pack.length-1 ], this.pool);
+  
+      let pickcard = card.name;
+      if (card.foil === true)
+        pickcard = "*" + pickcard + "*";
+  
+      this.pool.push(card);
+      this.picks.push(pickcard);
+      this.send("add", card);
+  
+      let [next] = this.packs;
+      if (!next)
+        this.time = 0;
+      else
+        this.sendPack(next);
+      
+      // Discard the rest of the cards in the pack
+      // after one is chosen.
+      pack.length = 0;
+  
+      this.autopick_index = -1;
+      this.emit("pass", pack);
+    }
+    
+    const draftPickDelegate = this.isDecadent ? decadentDraftPickDelegate : regularDraftPickDelegate;
+
+    const h = new Human(sock, draftPickDelegate);
     if (h.id === this.hostID) {
       h.isHost = true;
       sock.once("start", this.start.bind(this));
diff --git a/backend/human.js b/backend/human.js
index 13e778ce..f1b90cb1 100644
--- a/backend/human.js
+++ b/backend/human.js
@@ -1,18 +1,18 @@
 const Player = require("./player");
 const util = require("./util");
 const hash = require("./hash");
-const {random, pullAll} = require("lodash");
+const {random} = require("lodash");
 const logger = require("./logger");
 
 module.exports = class extends Player {
-  constructor(sock, isDecadent) {
+  constructor(sock, pickDelegate) {
     super({
       isBot: false,
       isConnected: true,
       name: sock.name,
-      id: sock.id,
-      isDecadent: isDecadent
+      id: sock.id
     });
+    this.pickDelegate = pickDelegate.bind(this);
     this.attach(sock);
   }
 
@@ -112,32 +112,7 @@ module.exports = class extends Player {
     this.draftStats.push( { picked, notPicked, pool: namePool } );
   }
   pick(index) {
-    const pack = this.packs.shift();
-    const card = pack.splice(index, 1)[0];
-
-    this.draftLog.pack.push( [`--> ${card.name}`].concat(pack.map(x => `    ${x.name}`)) );
-    this.updateDraftStats(this.draftLog.pack[ this.draftLog.pack.length-1 ], this.pool);
-
-    let pickcard = card.name;
-    if (card.foil === true)
-      pickcard = "*" + pickcard + "*";
-
-    this.pool.push(card);
-    this.picks.push(pickcard);
-    this.send("add", card);
-
-    let [next] = this.packs;
-    if (!next)
-      this.time = 0;
-    else
-      this.sendPack(next);
-
-    if (this.isDecadent) {
-      pullAll(pack, pack);
-    }
-
-    this.autopick_index = -1;
-    this.emit("pass", pack);
+    this.pickDelegate(index);
   }
   pickOnTimeout() {
     let index = this.autopick_index;
diff --git a/backend/player.js b/backend/player.js
index b618df2d..2c7a1979 100644
--- a/backend/player.js
+++ b/backend/player.js
@@ -5,7 +5,7 @@ const {EventEmitter} = require("events");
  * Abstract class for Human and Bot
  */
 class Player extends EventEmitter {
-  constructor({id, isBot, isConnected, name, isDecadent}) {
+  constructor({id, isBot, isConnected, name}) {
     super();
 
     if (this.constructor === Player) {
@@ -35,8 +35,7 @@ class Player extends EventEmitter {
       packSize: 15,
       self: 0,
       useTimer: false,
-      timeLength: "Slow",
-      isDecadent  // Pick a single card and discard the rest. Boujie!
+      timeLength: "Slow"
     });
   }
 

From 7d2086c0c339812c5306a64d509af96bd853943b Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Fri, 1 May 2020 23:25:17 -0700
Subject: [PATCH 33/36] feedback: Missed isDecadent flag on Bot class

---
 backend/game.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/game.js b/backend/game.js
index ec82c7de..7ad6a5f0 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -581,7 +581,7 @@ module.exports = class Game extends Room {
 
       if (addBots) {
         while (this.players.length < this.seats) {
-          this.players.push(new Bot(this.isDecadent));
+          this.players.push(new Bot());
           this.bots++;
         }
       }

From 629e5779e1ab5dccd35ed5e7b9c54236e26b2fd2 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Fri, 1 May 2020 23:27:04 -0700
Subject: [PATCH 34/36] feedback: Remove unnecessary whitespace

---
 backend/game.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/game.js b/backend/game.js
index 7ad6a5f0..6d28bd81 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -413,7 +413,7 @@ module.exports = class Game extends Room {
         this.meta();
       return;
     }
-    
+
     const index = this.players.indexOf(p) + this.delta;
     const nextPlayer = this.getNextPlayer(index);
     nextPlayer.getPack(pack);

From ffc7eecec759d40c91ad87ac4c89423cc4ba96f5 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Sat, 2 May 2020 00:52:11 -0700
Subject: [PATCH 35/36] fix: Don't add bots if decadent draft

---
 backend/game.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/backend/game.js b/backend/game.js
index 6d28bd81..2ce9ef7d 100644
--- a/backend/game.js
+++ b/backend/game.js
@@ -574,12 +574,16 @@ module.exports = class Game extends Room {
     this.startRound();
   }
 
+  shouldAddBots() {
+    return this.addBots && !this.isDecadent;
+  }
+
   start({ addBots, useTimer, timerLength, shufflePlayers }) {
     try {
       Object.assign(this, { addBots, useTimer, timerLength, shufflePlayers });
       this.renew();
 
-      if (addBots) {
+      if (this.shouldAddBots()) {
         while (this.players.length < this.seats) {
           this.players.push(new Bot());
           this.bots++;

From a182b9123ce81c913e038530447d0b5d9d040d10 Mon Sep 17 00:00:00 2001
From: Keeler Russell <keeler.russell@gmail.com>
Date: Mon, 4 May 2020 18:24:43 -0700
Subject: [PATCH 36/36] fix: Attempt to fix nothing returned from render error

---
 frontend/src/lobby/GameOptions.jsx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/frontend/src/lobby/GameOptions.jsx b/frontend/src/lobby/GameOptions.jsx
index da7346bc..9fd3f6ac 100644
--- a/frontend/src/lobby/GameOptions.jsx
+++ b/frontend/src/lobby/GameOptions.jsx
@@ -28,6 +28,8 @@ const GameOptions = () => {
     return <ChaosDraft/>
   case "chaos sealed":
     return <ChaosSealed/>;
+  default:
+    return null;
   }
 };