chore(scheduled-tasks): drop the leftovers the removal stranded

An independent audit of the branch turned up dead code and stale docs that the
compiler cannot see — nothing behavioural, but all of it rots silently.

- README still sold the feature: the "Scheduled tasks" tile, the prose listing
  it as a workspace surface, and the now-unreferenced screenshot. The landing
  surface went in c61770a; this tile was missed.
- `resource-content.tsx`: `SCHEDULE_STATUS_LABEL`, `formatScheduleInstant` and
  `ScheduledTaskField` were orphaned when the schedule render branch went.
- `computeNextRunAt`: zero callers, including tests — its only consumer was the
  removed agent-job runner.
- `applyScheduleUpdate`'s `allowCompleted` option: no call site passes it, and
  its comment described self-completion, which no longer exists. The guard stays
  (legacy `sourceType='job'` rows still carry `status='completed'` until the DB
  follow-up); it is simply unconditional now.
- Three TSDoc blocks still described a create-job route and "opening a
  scheduled-task artifact".

Type-check re-run with --force, since a cached turbo replay is not a check.

Author: waleedlatif1

README.md

@@ -47,7 +47,7 @@ Open [http://localhost:3000](http://localhost:3000)
 
 ## One workspace, every surface
 
-<p align="center">Chat and workflows are just the start — tables, files, knowledge, and scheduled tasks all live in the same workspace.</p>
+<p align="center">Chat and workflows are just the start — tables, files, and knowledge all live in the same workspace.</p>
 
 <table>
   <tr>
@@ -65,10 +65,7 @@ Open [http://localhost:3000](http://localhost:3000)
       <img src="apps/sim/public/static/readme-knowledge.png" alt="Knowledge bases in Sim — synced docs your agents can search" width="100%"/>
       <p align="center"><b>Knowledge</b> — your agents' memory</p>
     </td>
-    <td width="50%" valign="top">
-      <img src="apps/sim/public/static/readme-scheduled-tasks.png" alt="Scheduled tasks in Sim — recurring agent runs on a calendar" width="100%"/>
-      <p align="center"><b>Scheduled tasks</b> — runs on your schedule</p>
-    </td>
+    <td width="50%" valign="top"></td>
   </tr>
 </table>
 

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx

@@ -13,7 +13,6 @@ import {
   WorkflowX,
 } from '@sim/emcn/icons'
 import { createLogger } from '@sim/logger'
-import { format } from 'date-fns'
 import { useRouter } from 'next/navigation'
 import { isApiClientError } from '@/lib/api/client/errors'
 import { useSession } from '@/lib/auth/auth-client'
@@ -781,32 +780,6 @@ function EmbeddedFolder({ workspaceId, folderId }: EmbeddedFolderProps) {
   )
 }
 
-const SCHEDULE_STATUS_LABEL: Record<string, string> = {
-  active: 'Active',
-  disabled: 'Paused',
-  completed: 'Completed',
-}
-
-function formatScheduleInstant(iso: string | null): string {
-  if (!iso) return '—'
-  const date = new Date(iso)
-  return Number.isNaN(date.getTime()) ? '—' : format(date, "EEE, MMM d 'at' h:mm a")
-}
-
-interface ScheduledTaskFieldProps {
-  title: string
-  value: string
-}
-
-function ScheduledTaskField({ title, value }: ScheduledTaskFieldProps) {
-  return (
-    <div className='flex flex-col gap-1'>
-      <span className='text-[var(--text-muted)] text-caption'>{title}</span>
-      <span className='text-[var(--text-body)] text-small'>{value}</span>
-    </div>
-  )
-}
-
 interface EmbeddedLogProps {
   workspaceId: string
   logId: string

apps/sim/background/schedule-execution.ts

@@ -136,7 +136,6 @@ async function applyScheduleUpdate(
   context: string,
   options: {
     expectedLastQueuedAt?: Date | null
-    allowCompleted?: boolean
     /**
      * Set at call sites that can transition the row to `disabled`. Presence both
      * opts the site into the auto-disable email and adds a `status <> 'disabled'`
@@ -159,15 +158,10 @@ async function applyScheduleUpdate(
           ? isNull(workflowSchedule.lastQueuedAt)
           : eq(workflowSchedule.lastQueuedAt, options.expectedLastQueuedAt)
 
-    // A run that completes itself mid-execution sets status='completed'. The post-run
-    // bookkeeping that follows would otherwise write status='active' and a
-    // fresh nextRunAt straight back over it — the claim guard does not catch
-    // this, because completing the job does not touch lastQueuedAt. Terminal
-    // means terminal: only callers that explicitly opt in may move a completed
-    // row.
-    const notCompletedGuard = options.allowCompleted
-      ? undefined
-      : ne(workflowSchedule.status, 'completed')
+    // Terminal means terminal: a completed row is never moved back to active
+    // with a fresh nextRunAt. The claim guard does not cover this on its own,
+    // because reaching 'completed' does not touch lastQueuedAt.
+    const notCompletedGuard = ne(workflowSchedule.status, 'completed')
 
     /**
      * `RETURNING` yields the NEW row, so `status === 'disabled'` alone only means

apps/sim/hooks/queries/schedules.ts

@@ -90,10 +90,8 @@ export function useWorkspaceSchedules(workspaceId?: string, options?: { enabled?
 }
 
 /**
- * Fetch a single schedule (job) by id. Used by the mothership resource viewer so
- * opening a scheduled-task artifact does a lightweight by-id read instead of the
- * whole-workspace `useWorkspaceSchedules` fetch (which contended with the chat
- * stream connection and stalled start/resume).
+ * Fetch a single workflow schedule by id — a lightweight by-id read instead of
+ * the whole-workspace `useWorkspaceSchedules` fetch.
  */
 export function useScheduleById(scheduleId?: string) {
   return useQuery({

apps/sim/lib/api/contracts/schedules.ts

@@ -163,9 +163,8 @@ export const listWorkspaceSchedulesContract = defineRouteContract({
 })
 
 /**
- * Single-schedule read by id. Used by the mothership resource viewer so opening
- * a scheduled-task artifact does a lightweight by-id fetch instead of pulling
- * the entire workspace schedule list (which contended with the chat stream).
+ * Single-schedule read by id: a lightweight fetch for one workflow schedule
+ * instead of pulling the whole workspace list.
  */
 export const getScheduleByIdContract = defineRouteContract({
   method: 'GET',
@@ -180,9 +179,8 @@ export const getScheduleByIdContract = defineRouteContract({
 })
 
 /**
- * Newly-created job schedules emit a partial summary with the canonical fields
- * the route synthesizes server-side; everything else is filled in on
- * subsequent reads.
+ * Re-arms a disabled schedule: the route recomputes `nextRunAt` from the stored
+ * cron expression and clears the failure counters.
  */
 export const reactivateScheduleContract = defineRouteContract({
   method: 'PUT',

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

@@ -53,45 +53,6 @@ export function validateCronExpression(
 /** Upper bound on how many excluded occurrences the next-run search skips before giving up. */
 const MAX_OCCURRENCE_SKIP = 1000
 
-/**
- * Computes the next run instant for a recurring schedule, skipping occurrences
- * the user deleted individually and stopping at the recurrence end boundary.
- * Returns `null` when the recurrence has no remaining run (past `endsAt`, or
- * every candidate within the search bound is excluded).
- *
- * Excluded occurrences are matched by exact instant, so callers must record the
- * cron-produced occurrence time (not a rounded value) when excluding.
- */
-export function computeNextRunAt(params: {
-  cronExpression: string
-  timezone?: string
-  from?: Date
-  excludedDates?: string[] | null
-  endsAt?: Date | null
-}): Date | null {
-  const { cronExpression, timezone, from, excludedDates, endsAt } = params
-  let cron: Cron
-  try {
-    cron = new Cron(cronExpression, timezone ? { timezone } : undefined)
-  } catch {
-    return null
-  }
-
-  const excluded = new Set(
-    (excludedDates ?? []).map((iso) => new Date(iso).getTime()).filter((ms) => !Number.isNaN(ms))
-  )
-
-  let cursor = from ?? new Date()
-  for (let i = 0; i < MAX_OCCURRENCE_SKIP; i++) {
-    const next = cron.nextRun(cursor)
-    if (!next) return null
-    if (endsAt && next.getTime() > endsAt.getTime()) return null
-    if (!excluded.has(next.getTime())) return next
-    cursor = next
-  }
-  return null
-}
-
 interface SubBlockValue {
   value: string
 }