chore(scheduled-tasks): convert added inline comments to TSDoc

Author: waleedlatif1

apps/sim/app/api/mothership/execute/route.ts

@@ -154,8 +154,6 @@ export const POST = withRouteHandler(async (req: NextRequest) => {
           workspaceId,
           effectiveChatId
         ).catch((error) => {
-          // Degrade to a context-less run rather than failing the whole execution,
-          // but surface the failure so a broken mention is diagnosable.
           reqLogger.warn('Failed to resolve agent contexts for execution', {
             error: toError(error).message,
           })

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor/use-prompt-editor.ts

@@ -573,8 +573,6 @@ export function usePromptEditor({
       }
 
       const caret = e.target.selectionStart ?? finalValue.length
-      // Keep the ref in lockstep with state so synchronous readers (getValue /
-      // getPlainValue) never see pre-keystroke text before the next render.
       valueRef.current = finalValue
       setValueState(finalValue)
       syncMentionState(e.target, finalValue, caret)

apps/sim/app/workspace/[workspaceId]/scheduled-tasks/components/task-modal/task-modal.tsx

@@ -140,13 +140,15 @@ function TaskModalContent({
 }: Omit<TaskModalProps, 'open'>) {
   const { workspaceId } = useParams<{ workspaceId: string }>()
   const editor = usePromptEditor({ workspaceId, initialValue: edit?.prompt })
-
-  // Re-register the stored mentions once on open so saving an edit preserves
-  // them — the editor seeds from `initialValue` text only, not its contexts.
   const setContexts = editor.setContexts
+
+  /**
+   * Re-registers an edited task's stored `@`-mentions once on open: the editor
+   * seeds from `initialValue` text only, never its contexts. Runs once per open
+   * since the dialog's content remounts each time it opens.
+   */
   useEffect(() => {
     if (edit?.contexts && edit.contexts.length > 0) setContexts(edit.contexts)
-    // Runs once per open; content remounts each time the dialog opens.
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])
   const seed = edit ? { date: edit.launchDate, time: edit.launchTime } : defaultLaunch(slot)

apps/sim/app/workspace/[workspaceId]/scheduled-tasks/utils/recurrence.ts

@@ -170,6 +170,10 @@ export function cronToRecurrence(params: {
  * Materializes a recurring schedule's run instants inside `[rangeStart, rangeEnd]`
  * that are still upcoming (after `from`), skipping individually deleted
  * occurrences and stopping at the recurrence end. Pure given its inputs.
+ *
+ * The lower bound is inclusive: croner's `nextRun(date)` returns the first
+ * occurrence strictly after `date`, so the search starts one millisecond before
+ * the bound to admit an occurrence landing exactly on it.
  */
 export function expandOccurrences(params: {
   cronExpression: string
@@ -195,8 +199,6 @@ export function expandOccurrences(params: {
   const lowerBound = rangeStart.getTime() > from.getTime() ? rangeStart : from
 
   const occurrences: Date[] = []
-  // nextRun(date) returns the first occurrence strictly after `date`; subtract a
-  // millisecond so an occurrence landing exactly on the lower bound is included.
   let cursor = new Date(lowerBound.getTime() - 1)
   for (let i = 0; i < MAX_OCCURRENCES_PER_VIEW; i++) {
     const next = cron.nextRun(cursor)

apps/sim/background/schedule-execution.ts

@@ -1352,8 +1352,6 @@ export async function executeJobInline(payload: JobExecutionPayload) {
         })
       }
 
-      // A recurring job with no remaining occurrence (past its end date or out of
-      // non-excluded runs) is finished, just like a one-time or maxRuns-capped job.
       const isComplete = isOneTime || maxRunsReached || !nextRunAt
 
       await applyScheduleUpdate(
@@ -1397,9 +1395,6 @@ export async function executeJobInline(payload: JobExecutionPayload) {
     }
 
     const maxRunsReached = Boolean(jobRecord.maxRuns && newRunCount >= jobRecord.maxRuns)
-    // A one-time job, a maxRuns-capped job, or a recurring job out of remaining
-    // occurrences (past endsAt / all excluded) is finished even when this run
-    // failed — mirror the success path so it never re-fires past its bounds.
     const isComplete = !jobRecord.cronExpression || maxRunsReached || !nextRunAt
 
     await applyScheduleUpdate(

apps/sim/lib/workflows/schedules/orchestration.ts

@@ -427,7 +427,6 @@ export async function performExcludeOccurrence(
       updatedAt: new Date(),
     }
 
-    // Advancing past the just-removed run only matters when it was the one queued next.
     if (job.nextRunAt && job.nextRunAt.getTime() === occurrence.getTime()) {
       const nextRunAt = computeNextRunAt({
         cronExpression: job.cronExpression,