fix: filter completedEpics to exclude on-demand epics (progress >100% fix)

Addresses Copilot review feedback: completedEpics was counting ALL epics
(including on-demand) while totalEpics excluded on-demand. This could cause
progress to be inaccurate or exceed 100% when an on-demand epic completes.

Fix: use Object.entries() + filter by !EPIC_CONFIG[num]?.onDemand in both
_calculateProgressFromState() and getProgressPercentage().

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: nikolasdehor

.aios-core/core/orchestration/master-orchestrator.js

@@ -1357,8 +1357,8 @@ class MasterOrchestrator extends EventEmitter {
     const totalEpics = Object.keys(EPIC_CONFIG).filter((num) => !EPIC_CONFIG[num].onDemand).length;
     if (totalEpics === 0) return 0;
 
-    const completedEpics = Object.values(state.epics).filter(
-      (epic) => epic.status === EpicStatus.COMPLETED,
+    const completedEpics = Object.entries(state.epics).filter(
+      ([num, epic]) => epic.status === EpicStatus.COMPLETED && !EPIC_CONFIG[num]?.onDemand,
     ).length;
 
     return Math.round((completedEpics / totalEpics) * 100);
@@ -1414,8 +1414,8 @@ class MasterOrchestrator extends EventEmitter {
     const totalEpics = Object.keys(EPIC_CONFIG).filter((num) => !EPIC_CONFIG[num].onDemand).length;
     if (totalEpics === 0) return 0;
 
-    const completedEpics = Object.values(this.executionState.epics).filter(
-      (epic) => epic.status === EpicStatus.COMPLETED,
+    const completedEpics = Object.entries(this.executionState.epics).filter(
+      ([num, epic]) => epic.status === EpicStatus.COMPLETED && !EPIC_CONFIG[num]?.onDemand,
     ).length;
 
     return Math.round((completedEpics / totalEpics) * 100);