Add Hot-Reload Google Drive sync without restart

[geokoko]

No description provided.

[Copilot]

Pull request overview

This PR extends StudySync’s desktop app to support in-app Google Drive database refresh (hot reload) and also adds new planning features (future-dated study goals and recurring tasks), including the schema/entity/service/UI updates needed to persist and render them.

Changes:

• Add Google Drive “download + reload” flow with dirty-tracking, startup sync-status check, and UI refresh listeners.
• Add recurring task support (DB column + entity/service helpers + task management UI).
• Add future goal planning (DatePicker-based goal creation, calendar/day filtering updates) and reduce repeated delayed-goal processing via a once-per-day guard.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/main/resources/schema.sql	Adds recurring_pattern column + idempotent ALTER migration block
src/main/java/com/studysync/presentation/ui/components/TaskManagementPanel.java	Adds recurring-task UI controls; passes recurring pattern through create/update
src/main/java/com/studysync/presentation/ui/components/StudyPlannerPanel.java	Adds goal date picker + quick-select buttons; saves goals for selected date
src/main/java/com/studysync/presentation/ui/components/ProfileViewPanel.java	Replaces “restart to apply” with download-and-reload; shows local-dirty hinting
src/main/java/com/studysync/presentation/ui/components/DailyViewPanel.java	Adds future-date branch for goal retrieval
src/main/java/com/studysync/presentation/ui/components/CalendarViewPanel.java	Enables future-date navigation, styling, goal planning dialog, and future-date goal filtering
src/main/java/com/studysync/presentation/ui/StudySyncUI.java	Wires DatabaseReloadService, adds sync-status check, registers reload listener to refresh panels
src/main/java/com/studysync/integration/drive/GoogleDriveService.java	Adds dirty tracking, sync-status computation, reload listeners, and downloadAndReload API
src/main/java/com/studysync/integration/drive/GoogleDriveGateway.java	Adds remote modified-time lookup for sync-status checks
src/main/java/com/studysync/domain/service/TaskUpdate.java	Adds recurringPattern field + convenience ctor preserving old call sites
src/main/java/com/studysync/domain/service/TaskService.java	Wires dirty tracking + adds recurring task operations and update/apply logic
src/main/java/com/studysync/domain/service/StudyService.java	Adds future-goal retrieval method, dirty tracking on writes, and once-per-day delayed-goal processing guard
src/main/java/com/studysync/domain/service/ProjectService.java	Adds dirty tracking on project writes
src/main/java/com/studysync/domain/entity/Task.java	Persists/loads recurring_pattern and adds recurrence helpers + recurring queries
src/main/java/com/studysync/config/DatabaseReloadService.java	New service to reload H2/Hikari connections and reapply schema.sql after Drive download
README.md	Documents future goal planning, recurring tasks, and migration approach
ARCHITECTURE.md	Documents recurrence format, future planning, delayed-goal guard, and migration strategy

Comments suppressed due to low confidence (2)

src/main/java/com/studysync/domain/service/StudyService.java:89

• getTodayGoals() now calls ensureDelayedGoalsProcessedToday(), which can perform writes (processAllDelayedGoals). Because calculateDailyProgress() is @transactional(readOnly = true) and calls getTodayGoals() via self-invocation, delayed-goal processing may execute inside a read-only transaction, which can fail or be ignored depending on DB/transaction manager. Consider removing readOnly from calculateDailyProgress(), or moving delayed-goal processing into a separate bean/TransactionTemplate so it runs in a write-capable transaction.

    public List<StudyGoal> getTodayGoals() {
        ensureDelayedGoalsProcessedToday();
        return StudyGoal.findByDate(LocalDate.now());
    }

src/main/java/com/studysync/domain/service/StudyService.java:100

• ensureDelayedGoalsProcessedToday() can trigger processAllDelayedGoals(), which updates/deletes goals. Those DB writes should mark the local DB dirty so Drive sync status remains accurate (similar to other write paths calling markDirty()). Consider capturing the GoalDelayProcessingResult from processAllDelayedGoals() and calling markDirty() when it reports changes.

    private void ensureDelayedGoalsProcessedToday() {
        LocalDate today = LocalDate.now();
        if (!today.equals(lastDelayProcessingDate)) {
            processAllDelayedGoals();
            lastDelayProcessingDate = today;
        }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 10 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.