chore: implement CodeRabbit review comments

- Update sqlite-jdbc from 3.47.1.0 to 3.51.0.0 (2025-11-05 release)
- Add error handling for database copy in entrypoint.sh
  - Check cp exit status before printing success message
  - Print clear error with paths on failure and exit non-zero

Author: nanotaboada

.github/copilot-instructions.md

@@ -243,7 +243,7 @@ The Docker container uses a "hold" pattern for the pre-seeded SQLite database:
 
 ### SQLite Configuration Notes
 
-- **Date storage**: Dates are stored as Unix timestamps (INTEGER) for robustness - no parsing issues
+- **Date storage**: LocalDate fields are stored as Unix timestamps (INTEGER) for robustness - no parsing issues
 - **Converter**: `UnixTimestampConverter` handles LocalDate ↔ epoch seconds conversion via JPA `@Convert`
 - **DDL auto**: Use `ddl-auto=none` since the database is pre-seeded (SQLite has limited ALTER TABLE support)
 - **Tests use H2**: The converter works seamlessly with both H2 and SQLite databases

README.md

@@ -205,7 +205,7 @@ open target/site/jacoco/index.html
 **SQLite Configuration Notes:**
 
 - Dates are stored as Unix timestamps (INTEGER) for robustness - no date format parsing issues
-- A JPA `AttributeConverter` handles LocalDate ↔ epoch seconds conversion transparently
+- A JPA `AttributeConverter` handles LocalDate ↔ epoch seconds conversion transparently (UTC-based)
 - Use `ddl-auto=none` since the database is pre-seeded (SQLite has limited ALTER TABLE support)
 - Tests use H2 in-memory database - the converter works seamlessly with both databases
 

pom.xml

@@ -156,7 +156,7 @@
         <dependency>
             <groupId>org.xerial</groupId>
             <artifactId>sqlite-jdbc</artifactId>
-            <version>3.47.1.0</version>
+            <version>3.51.0.0</version>
             <scope>runtime</scope>
         </dependency>
         <!-- Hibernate Community Dialects ==================================== -->

scripts/entrypoint.sh

@@ -12,8 +12,13 @@ if [ ! -f "$VOLUME_STORAGE_PATH" ]; then
   echo "⚠️ No existing database file found in volume."
   if [ -f "$IMAGE_STORAGE_PATH" ]; then
     echo "Copying database file to writable volume..."
-    cp "$IMAGE_STORAGE_PATH" "$VOLUME_STORAGE_PATH"
-    echo "✔ Database initialized at $VOLUME_STORAGE_PATH"
+    if cp "$IMAGE_STORAGE_PATH" "$VOLUME_STORAGE_PATH"; then
+      echo "✔ Database initialized at $VOLUME_STORAGE_PATH"
+    else
+      echo "❌ Failed to copy database from $IMAGE_STORAGE_PATH to $VOLUME_STORAGE_PATH"
+      echo "   Check file permissions and available disk space."
+      exit 1
+    fi
   else
     echo "⚠️ Database file missing at $IMAGE_STORAGE_PATH"
     exit 1

src/main/java/ar/com/nanotaboada/java/samples/spring/boot/models/UnixTimestampConverter.java

@@ -12,8 +12,7 @@
  * (seconds since epoch).
  *
  * This converter stores dates as INTEGER in SQLite, which is more robust than
- * TEXT-based
- * date formats because:
+ * TEXT-based date formats because:
  * - No parsing ambiguity or locale-dependent formatting issues
  * - Works consistently across all SQLite clients and tools
  * - More efficient for date comparisons and indexing