now the legal moves function actually returns a

list of moves

src/logic/pieces/bishop.py

@@ -1,4 +1,6 @@
+from logic.move import Move
 from .piece import Piece
 
 class Bishop(Piece):
-    pass
+    def legal_moves(self, board) -> list[Move]:
+        return super().legal_moves(board)

src/logic/pieces/pawn.py

@@ -1,8 +1,8 @@
-from logic.position import Position
+from logic.move import Move
 from logic.pieces.piece import Piece
 
 class Pawn(Piece):
-    def legal_moves(self, board) -> list[Position]:
+    def legal_moves(self, board) -> list[Move]:
         ret = []
 
         # can we capture to the left?

src/logic/pieces/piece.py

@@ -1,4 +1,5 @@
 from logic.position import Position
+from logic.move import Move
 
 
 class Piece:
@@ -13,5 +14,5 @@ class Piece:
     def position(self) -> Position:
         return self.pos
 
-    def legal_moves(self, board) -> list[Position]:
+    def legal_moves(self, board) -> list[Move]:
         raise NotImplementedError(f"Can't say what the legal moves are for {type(self).__name__}, the method hasn't been implemented yet")