@alepot55/chessboardjs 2.2.1 → 2.3.0

package/{chessboard.square.js → src/components/Square.js}

@@ -1,4 +1,4 @@
-import Piece from "./chessboard.piece.js";
+import Piece from "./Piece.js";
 
 class Square {
 
@@ -54,11 +54,51 @@ class Square {
         return this.element.getBoundingClientRect();
     }
 
-    removePiece() {
-        this.element.removeChild(this.piece.element);
-        const piece = this.piece;
+    removePiece(preserve = false) {
+        if (!this.piece) {
+            return null;
+        }
+        // Only destroy the piece if not preserving (i.e., not moving)
+        if (!preserve && typeof this.piece.destroy === 'function') {
+            this.piece.destroy();
+        }
         this.piece = null;
-        return piece;
+        return null;
+    }
+
+    /**
+     * Forcefully removes all pieces from this square
+     */
+    forceRemoveAllPieces() {
+        // Best practice: destroy the piece object if present
+        if (this.piece && typeof this.piece.destroy === 'function') {
+            this.piece.destroy();
+            this.piece = null;
+        }
+        // Remove any orphaned img.piece elements from the DOM
+        const pieceElements = this.element.querySelectorAll('img.piece');
+        pieceElements.forEach(element => {
+            if (element.parentNode === this.element) {
+                this.element.removeChild(element);
+            }
+        });
+    }
+
+    /**
+     * Replaces the current piece with a new one efficiently
+     * @param {Piece} newPiece - The new piece to place
+     */
+    replacePiece(newPiece) {
+        // If there's an existing piece, remove it first
+        if (this.piece) {
+            this.removePiece();
+        }
+
+        // Add the new piece
+        this.putPiece(newPiece);
+
+        // Ensure the piece is properly displayed
+        newPiece.element.style.opacity = '1';
     }
 
     addEventListener(event, callback) {
@@ -66,8 +106,14 @@ class Square {
     }
 
     putPiece(piece) {
+        // If there's already a piece, destroy it to avoid orphaned DOM elements
+        if (this.piece) {
+            this.removePiece(false);
+        }
         this.piece = piece;
-        this.element.appendChild(piece.element);
+        if (piece && piece.element) {
+            this.element.appendChild(piece.element);
+        }
     }
 
     putHint(catchable) {
@@ -150,6 +196,10 @@ class Square {
 
     }
 
+    hasPromotion() {
+        return this.element.querySelector('.choice') !== null;
+    }
+
     removePromotion() {
         let choice = this.element.querySelector('.choice');
         if (choice) {
@@ -159,7 +209,10 @@ class Square {
     }
 
     destroy() {
+        // Remove all piece DOM nodes and clear reference
+        this.forceRemoveAllPieces();
         this.element.remove();
+        this.piece = null;
     }
 
     hasPiece() {
@@ -167,7 +220,7 @@ class Square {
     }
 
     getColor() {
-        return this.piece.getColor();
+        return this.piece ? this.piece.getColor() : null;
     }
 
     check() {
@@ -177,7 +230,7 @@ class Square {
         if (this.col < 1 || this.col > 8) {
             throw new Error("Invalid square: col is out of bounds");
         }
-        
+
     }
 }
 

package/src/constants/index.js

@@ -0,0 +1,15 @@
+/**
+ * Constants module exports
+ * @module constants
+ * @since 2.0.0
+ */
+
+export {
+    STANDARD_POSITIONS,
+    DEFAULT_STARTING_POSITION,
+    PIECE_TYPES,
+    PIECE_COLORS,
+    PROMOTION_PIECES,
+    BOARD_LETTERS,
+    BOARD_SIZE
+} from './positions.js';

package/src/constants/positions.js

@@ -0,0 +1,62 @@
+/**
+ * Standard chess positions and FEN constants
+ * @module constants/positions
+ * @since 2.0.0
+ */
+
+/**
+ * Standard chess positions used throughout the application
+ * @type {Object.<string, string>}
+ * @readonly
+ */
+export const STANDARD_POSITIONS = {
+    'start': 'start',
+    'default': 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1',
+    'empty': '8/8/8/8/8/8/8/8 w - - 0 1'
+};
+
+/**
+ * Default starting position FEN string
+ * @type {string}
+ * @readonly
+ */
+export const DEFAULT_STARTING_POSITION = 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1';
+
+/**
+ * Chess piece types
+ * @type {string[]}
+ * @readonly
+ */
+export const PIECE_TYPES = ['p', 'r', 'n', 'b', 'q', 'k'];
+
+/**
+ * Chess piece colors
+ * @type {string[]}
+ * @readonly
+ */
+export const PIECE_COLORS = ['w', 'b'];
+
+/**
+ * Valid promotion pieces
+ * @type {string[]}
+ * @readonly
+ */
+export const PROMOTION_PIECES = ['q', 'r', 'b', 'n'];
+
+/**
+ * Board coordinates letters
+ * @type {string}
+ * @readonly
+ */
+export const BOARD_LETTERS = 'abcdefgh';
+
+/**
+ * Board size constants
+ * @type {Object}
+ * @readonly
+ */
+export const BOARD_SIZE = {
+    ROWS: 8,
+    COLS: 8,
+    TOTAL_SQUARES: 64
+};