In the fall me and Jonathan worked on an update to the Engine specifically with SnailMode in mind, and refactored SnailMode to use this new `PreUpdate` method

This allowed us to not have to use off-board hazards as state!

However we want to use Snail Mode for our first Community Tournament! And the web engine doesn't support PreUpdate yet, so we are reverting Snail Mode to its old Pre-PreUpdate version :lol: check-in: c07601343c user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.2.2

* DEV-1761: Clean up Ruleset interface (#115)

* remove legacy ruleset types and simplify ruleset interface

* remove unnecessary settings argument from Ruleset interface

* decouple rules.Settings from client API and store settings as strings

* DEV 1761: Add new BoardState and Point fields (#117)

* add Point.TTL, Point.Value, GameState and PointState to BoardState

* allow maps to access BoardState.GameState,PointState

* add PreUpdateBoard and refactor snail_mode with it

* fix bug where an extra turn was printed to the console

* fix formatting

* fix lint errors

Co-authored-by: JonathanArns <jonathan.arns@googlemail.com> check-in: ed19c4e81c user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.2.0

* track latency and status code from CLI games

* actually track errors from snake responses check-in: f1c499c1c3 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.21

* move PlaceFoodFixed and PlaceSnakesInQuadrants to maps package

* don't spawn food on hazards in islands/rivers and bridges maps check-in: 9ab548177e user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.20

* ensure /end request is always called, and refactor win/draw logic

* clean up logging and error handling during initialization

* automatically generate friendly snake names

* title-case snake names

* print out list of alive snake names instead of count

* log snake names, IDs, and URLs at startup

* print out state for turn zero check-in: dd47bfbab3 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.18

* Adding two new variations with hazards along the edges

* Implementing code review feedback check-in: 6c076d0573 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.17

* Add snail-mode map

* snail-mode: cap max hazards to 7

- Ensure that no more than 7 hazards are added to a square. This fixes a bug where some squares were getting way too many hazards applied to them. There must be some other bug at work here as well. - Change author names to be github usernames instead of first names

* snail-mode: fix bug with eliminated snakes

- Ensure that hazard snail-trail is not added for eliminated snakes

* Update from Stream July 31

Added comments to most functions and important bits of code

Also changed the map so that instead of a fixed number of 7 hazards, we add hazards equal to the length of the snake.

* snail-mode: add TAG_EXPERIMENTAL and TAG_HAZARD_PLACEMENT

* snail-mode: use Point as map key

Co-authored-by: Corey Alexander <coreyja@gmail.com> check-in: b9306747dc user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.16

* add 'map' cli command

- provides the following map information functions: - list all available maps in the global registry - display map metadata

- update docs with map command examples

* add list and info subcommands to map cli command

* rename map command list and info factory functions

* add --all flag to map info subcommand

* handle cmd.Help error check-in: 3f28759287 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.15

* Adding a basic map that adds a few hazards based on fixed coordinates to the map.

* Used wrong board size definitions check-in: 1dad93a0ed user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.14

* change map support for large #'s of snakes

* test square board fn

* format comment

* add whitespace

* better support large #'s of snakes on small boards

* include an intermediate xlarge size check-in: f0c1f1517f user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.13

* Commit from Stream

We got the maze working decently well!

Started off with exploring the repo a bit and getting used to the Map stuff

The Maze is working pretty well except the mazes that are generated aren't actually complete-able yet. They end up creating spaces that are blocked off from the rest of the map

The maze generation algo is based off 'Recursive division method' mostly following this Wiki

This looked to always create a 'valid' maze, while mine doesn't so need to figure out what my bug is, cause I'm pretty sure there is one!

* I think we got maze generation working!

Did a lot, should have commited more lol

Got map generation working where I can see it go frame by frame, and think we squashed the last few bugs with wall generation and hole cutting

Now need it to actually make a new maze when you finish this first one

* We can run on a bigger board now, and we place the food randomly on the board after you grab it the first time

* Make sure my health is always 100 so don't have to worry about that lol

* Fix placing of food on maze to NOT be on an existing hazard and after 3 foods make a new maze

* Maze generation works better now and a bit of cleanup

* Committing old code

* Got state saving as Hazards Bits Working well, I think whats up next is making the map grow each level

* Draw maze in the center of the board, the maze grows as you complete levels until it fills the entire board

* Ran go fmt

* Remove my little debugging script that I added

* Revert changes I made initially to the map interface

* Move my copy-pasted printMap function to my map specifically

* Go fmt

* Write some doc comments and fix some weird code artifacts

* A few more comments and one last go fmt

* Cut less holes, this makes the maze more exciting

* Add version to Metadata and go fmt again

* Can collapse this down

* Random food spawns are more fun for sure

* add minimal support for serving a game to the board UI

* We did 3 main things here:

- Fixed the bug where hazards were getting multi-stacked - Fixed the bug where the food could spawn on top of the snake head - Fixed the lint errors

* Games aren't infinite now, they end by not spawning a food when you have reached a certain number of turns at the max board size

* Extract some functions to make deciding when the game is over easier

* Get evil mode implemented to force the game to end

* Fix small bugs about where we spawned the food in evil mode

* Have the snake keep growing by adding tail segments to match the current level. We start at two length because 1 lenth snakes look weird on the board since they don't have a neck to orient themselves. Here we also fix a bug where evil mode was trying to place food off the map

* Run go fmt

* Add the missing meta-data pieces

* Support a smaller board without the change of infinite loops and change the name of the maze

* Rename the actual struct and fix the tests to not make extra snakes when the map doesn't support it

* Revert "add minimal support for serving a game to the board UI"

This reverts commit 0ac3592c7669ab1bccd7bd3322adffbef5e911ce.

* add minimal support for serving a game to the board UI

* Cleanup and extract and function to place food so we don't have basically the same body twice. This also will prevent the initial food spawn from being right next to the snake

* Revert "add minimal support for serving a game to the board UI"

This reverts commit c1a3b134213dabf44df56e3ca1e6d30ff8aaa516.

* Fix lint

* 2 length snakes are silly, so lets start with 3 each time

* We start out with 3 length snakes, and a fixed set of map sizes

Co-authored-by: Rob O'Dwyer <odwyerrob@gmail.com> check-in: 62e2203191 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.12

* add minimal support for serving a game to the board UI

* refactor into new board package

* support reporting errors and author names to board

* support passing an alternate board URL

* avoid using IPv6 for local URL

* use rules.Point instead of a custom Point type for board package

* use zero for generic communication error code in cli

* rename createGameEvent to buildFrameEvent

* tests for conversion from boardState/snakeState to game frame check-in: e51464d9ce user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.10

* add support for automatic solo through RulesetBuilder

* allow solo games with all modes in CLI check-in: 6c87dca4f6 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.7

* add "namcap" map

* adjust hazards and starting food positions

* add food randomly, not on top of hazards

* add exits on the top and bottom

* rename to arcade_maze

* add maps README

* test for ArcadeMazeMap

* adjustments to hazards in arcade_maze check-in: d0af9e1571 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.5

* remove squad support and switch to using pipelines only in RulesBuilder

* remove spawn_food.standard from legacy ruleset definitions

* bugfix: Royale map generates Standard food

* add maps support to CLI

* add automated tests for all registered GameMap implementations

* update README check-in: 6d94a0db18 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.4

* move random generator into Settings

* add empty and royale maps

* place snakes on either cardinal or corner positions first check-in: 1a22e38342 user: noreply@github.com tags: DEV-1247-map-generator-bugfix, fix-end-request-on-draw-tests, trunk, v1.1.2

* add a new "pipeline" concept

- added new Pipeline type which is a series of stages - added a global registry to facilitate plugin architecture - 100% test coverage

* Refactor rulesets to provide and use Pipeline

* fix copypasta comments

* fix lint for unused method

* include game over stages in ruleset pipelines

* clean up unused private standard methods

* remove unused private methods in squad ruleset

* remove unused private methods in royale ruleset

* refactor: pipeline clone + return next board state

* YAGNI: remove unused Append

* refactor: improve stage names

* add no-op behavior to stages for initial state

* refactor: no-op decision within stage functions

* remove misleading comment that isn't true

* dont bother checking for init in gameover stages

* remove redundant test

* refactor: provide a combined ruleset/pipeline type

* fix: movement no-op for GameOver check

IsGameOver needs to run pipeline, move snakes needs to no-op for that

* add test coverage

* refactor: improve stage names and use constants

* add Error method

Support error checking before calling Execute()

* update naming to be American style

* panic when overwriting stages in global registry

* rename "Error" method and improve docs

* use testify lib for panic assertion

* remove redundant food stage

* use ruleset-specific logic for game over checks

* re-work Pipeline errors

* rework errors again

* add defensive check for zero length snake

* use old logic which checks current state, not next

* add warning about how PipelineRuleset checks for game over check-in: 6c4f0d03d2 user: noreply@github.com tags: fix-end-request-on-draw-tests, trunk, v1.0.33

- Update usage section in the README with the latest flags. - Update default sample output (BoardState fields have changed, OutOfBounds has been removed). - Add section describing options for getting JSON output with the `--output` and `--debug-requests` flag. check-in: c4e1a1189c user: noreply@github.com tags: fix-end-request-on-draw-tests, trunk, v1.0.32

Refactor rulesets into smaller composable operations

In order to mix up the functionality from different rulesets like Solo, Royale, etc. the code in these classes needs to be broken up into small functions that can be composed in a pipeline to make a custom game mode. Leaf check-in: 1b36b776bb user: noreply@github.com tags: trunk, v1.0.28

* add HazardMap and HazardMapAuthor to RulesetSettings

* test for format of hazard map properties check-in: 44b3226eba user: noreply@github.com tags: trunk, v1.0.27

* move snake API structs into a new client package

* add customizations to snake objects

* refactor and add support for passing snake customizations in games check-in: 800290c5b7 user: noreply@github.com tags: trunk, v1.0.23

* wrapped ruleset

* resurrect like brad suggested

* introduce mode to binary

* go fmt

* address edge crossing collision and eating bugs

* Update board_state.go

* removed binary

* rename variable check-in: f8825815e0 user: noreply@github.com tags: trunk, v1.0.21

* move hazard damage into StandardRuleset

* OutOfBounds -> Hazards

* remove "out of bounds" in comment

* add cases for eating food to hazard damage test check-in: 551a1b9721 user: noreply@github.com tags: trunk, v1.0.20

Co-authored-by: Brad Van Vugt <1531419+bvanvugt@users.noreply.github.com> check-in: 3ebbb1b7e0 user: noreply@github.com tags: trunk, v1.0.19

Co-authored-by: Jiri Novotny <hello@jiricodes.com> check-in: 462194d6c8 user: noreply@github.com tags: trunk, v1.0.18

Randomly placed snakes will now always spawn on "even" squares. check-in: c74e7e79c2 user: noreply@github.com tags: trunk, v1.0.6