Improve package naming and add interface for engines to implement

engines/engine.go

@@ -0,0 +1,9 @@
+package engines
+
+import (
+	"github.com/notnil/chess"
+)
+
+type Engine interface {
+	SuggestedMove(pos *chess.Position, maxPlayer bool) *chess.Move
+}

engine/minimax.go → engines/minimax.go

@@ -1,4 +1,4 @@
-package engine
+package engines
 
 import (
 	"fmt"
@@ -10,9 +10,11 @@ import (
 	"github.com/notnil/chess"
 )
 
+type MinimaxEngine struct {}
+
 // SuggestedMove calculates the most advantageous move for the player. It currently works
 // synchronously, but can probably be made concurrent with goroutines.
-func SuggestedMove(pos *chess.Position, maxPlayer bool) *chess.Move {
+func (e *MinimaxEngine) SuggestedMove(pos *chess.Position, maxPlayer bool) *chess.Move {
 	startingDepth := 3
 	return minimax(pos, startingDepth, maxPlayer).move
 }

engine/minimax_test.go → engines/minimax_test.go

@@ -1,4 +1,4 @@
-package engine
+package engines
 
 import (
 	"fmt"

main.go

@@ -6,11 +6,12 @@ import (
 
 	"github.com/notnil/chess"
 
-	"github.com/jameshochadel/chess-bot/engine"
+	"github.com/jameshochadel/chess-bot/engines"
 )
 
 func main() {
 	game := chess.NewGame()
+	engine := engines.MinimaxEngine{}
 
 	for game.Outcome() == chess.NoOutcome {
 		// how to do context with timeout? basically want to go as far as possible until timeout.