Merge pull request #7 from DojoCodingLabs/daniel/doj-2430-date-handling

feat: cross-platform date handling library (DOJ-2430)

Author: bitfalt

scripts/lib/date-compat.sh

@@ -0,0 +1,49 @@
+#!/usr/bin/env bash
+# Cross-platform date utilities for CodeSensei
+# Supports: GNU date (Linux), BSD date (macOS), python3 fallback
+
+# Get today's date in YYYY-MM-DD format (UTC)
+date_today() {
+  date -u '+%Y-%m-%d' 2>/dev/null || python3 -c "from datetime import datetime; print(datetime.utcnow().strftime('%Y-%m-%d'))"
+}
+
+# Get yesterday's date in YYYY-MM-DD format (UTC)
+date_yesterday() {
+  # Try GNU date first
+  date -u -d 'yesterday' '+%Y-%m-%d' 2>/dev/null && return
+  # Try BSD date (macOS)
+  date -u -v-1d '+%Y-%m-%d' 2>/dev/null && return
+  # Python fallback
+  python3 -c "from datetime import datetime, timedelta; print((datetime.utcnow() - timedelta(days=1)).strftime('%Y-%m-%d'))" 2>/dev/null && return
+  # Last resort: empty string
+  echo ""
+}
+
+# Convert date string (YYYY-MM-DD) to epoch seconds
+date_to_epoch() {
+  local date_str="$1"
+  # Try GNU date
+  date -u -d "$date_str" '+%s' 2>/dev/null && return
+  # Try BSD date (macOS)
+  date -u -j -f '%Y-%m-%d %H:%M:%S' "${date_str} 00:00:00" '+%s' 2>/dev/null && return
+  # Python fallback
+  python3 -c "from datetime import datetime, timezone; print(int(datetime.strptime('$date_str', '%Y-%m-%d').replace(tzinfo=timezone.utc).timestamp()))" 2>/dev/null && return
+  echo "0"
+}
+
+# Get date N days ago in YYYY-MM-DD format (UTC)
+date_days_ago() {
+  local days="$1"
+  # Try GNU date
+  date -u -d "${days} days ago" '+%Y-%m-%d' 2>/dev/null && return
+  # Try BSD date (macOS)
+  date -u -v-${days}d '+%Y-%m-%d' 2>/dev/null && return
+  # Python fallback
+  python3 -c "from datetime import datetime, timedelta; print((datetime.utcnow() - timedelta(days=${days})).strftime('%Y-%m-%d'))" 2>/dev/null && return
+  echo ""
+}
+
+# Get current UTC timestamp in ISO format
+date_now_iso() {
+  date -u '+%Y-%m-%dT%H:%M:%SZ' 2>/dev/null || python3 -c "from datetime import datetime; print(datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%SZ'))"
+}

scripts/quiz-selector.sh

@@ -38,6 +38,9 @@ else
   check_jq() { command -v jq &>/dev/null; }
 fi
 
+# shellcheck source=scripts/lib/date-compat.sh
+source "$PLUGIN_ROOT/scripts/lib/date-compat.sh"
+
 # Default output if we can't determine anything
 DEFAULT_OUTPUT='{"mode":"dynamic","concept":null,"reason":"No profile data available","static_question":null,"belt":"white","quiz_format":"multiple_choice"}'
 
@@ -89,8 +92,8 @@ if [ $? -ne 0 ]; then
   CORRECT_QUIZZES=0
 fi
 
-TODAY=$(date -u +%Y-%m-%d)
-NOW_EPOCH=$(date +%s)
+TODAY=$(date_today)
+NOW_EPOCH=$(date_to_epoch "$TODAY")
 
 # Determine quiz format based on belt level
 # Orange Belt+ gets a mix of formats; lower belts get multiple choice
@@ -149,10 +152,10 @@ if [ "$QUIZ_HISTORY" != "[]" ]; then
       continue
     fi
 
-    # Calculate days since last wrong answer (handles both GNU and BSD date)
+    # Calculate days since last wrong answer using cross-platform helpers
     LAST_WRONG_DATE=$(printf '%s' "$LAST_WRONG" | cut -d'T' -f1)
-    LAST_EPOCH=$(date -j -f "%Y-%m-%d" "$LAST_WRONG_DATE" +%s 2>/dev/null || date -d "$LAST_WRONG_DATE" +%s 2>/dev/null)
-    if [ -n "$LAST_EPOCH" ]; then
+    LAST_EPOCH=$(date_to_epoch "$LAST_WRONG_DATE")
+    if [ -n "$LAST_EPOCH" ] && [ "$LAST_EPOCH" != "0" ]; then
       DAYS_SINCE=$(( (NOW_EPOCH - LAST_EPOCH) / 86400 ))
     else
       log_error "$SCRIPT_NAME" "Could not parse date '$LAST_WRONG_DATE' for spaced repetition; defaulting days_since=999"

scripts/session-start.sh

@@ -7,19 +7,21 @@ SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 
 # shellcheck source=lib/profile-io.sh
 source "${SCRIPT_DIR}/lib/profile-io.sh"
+# shellcheck source=lib/date-compat.sh
+source "${SCRIPT_DIR}/lib/date-compat.sh"
 
 LIB_DIR="${SCRIPT_DIR}/lib"
 if [ -f "${LIB_DIR}/error-handling.sh" ]; then
   # shellcheck source=lib/error-handling.sh
   source "${LIB_DIR}/error-handling.sh"
 else
   LOG_FILE="${PROFILE_DIR}/error.log"
-  log_error() { printf '[%s] [%s] %s\n' "$(date -u '+%Y-%m-%dT%H:%M:%SZ' 2>/dev/null || date '+%Y-%m-%d')" "${1:-unknown}" "$2" >> "$LOG_FILE" 2>/dev/null; }
+  log_error() { printf '[%s] [%s] %s\n' "$(date_now_iso 2>/dev/null || date -u '+%Y-%m-%dT%H:%M:%SZ' 2>/dev/null || date '+%Y-%m-%d')" "${1:-unknown}" "$2" >> "$LOG_FILE" 2>/dev/null; }
   check_jq() { command -v jq &>/dev/null; }
 fi
 
 SESSION_LOG="${PROFILE_DIR}/sessions.log"
-TODAY=$(date -u +%Y-%m-%d)
+TODAY=$(date_today)
 
 ensure_profile_dir
 if [ ! -d "$PROFILE_DIR" ]; then
@@ -28,7 +30,7 @@ if [ ! -d "$PROFILE_DIR" ]; then
 fi
 
 if [ ! -f "$PROFILE_FILE" ]; then
-  CREATED_AT=$(date -u +%Y-%m-%dT%H:%M:%SZ)
+  CREATED_AT=$(date_now_iso)
   if ! cat <<PROFILE | write_profile
 {
   "version": "1.0.0",
@@ -113,7 +115,7 @@ fi
 if [ "$LAST_SESSION" = "$TODAY" ]; then
   NEW_STREAK=$CURRENT_STREAK
 elif [ -n "$LAST_SESSION" ]; then
-  YESTERDAY=$(date -u -d "yesterday" +%Y-%m-%d 2>/dev/null || date -u -v-1d +%Y-%m-%d 2>/dev/null)
+  YESTERDAY=$(date_yesterday)
   if [ -z "$YESTERDAY" ]; then
     log_error "$SCRIPT_NAME" "Failed to compute yesterday's date; resetting streak to 1"
     YESTERDAY=""