fix(webapp): suppress sheet spinner on bg reload + uniquify form id

- UpsertScheduleForm: derive a per-schedule `useForm` id
  (`edit-schedule-{friendlyId}` vs `create-schedule`) so both sheets
  can coexist in the DOM without clashing `htmlFor` targets or
  conform's internal error routing.
- ScheduleSheet: drop `detailFetcher.state === "loading"` from
  `isDetailLoading`. The stale-schedule and no-data-yet checks already
  cover the cases where we genuinely lack good data; a background
  reload (e.g. after enable/disable) now keeps the inspector visible
  with its current values until the fresh data arrives, instead of
  flashing a spinner.

Author: samejr

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.tasks.scheduled.$taskParam/route.tsx

@@ -650,10 +650,11 @@ function ScheduleSheet({
   // user clicked a different row) as loading — otherwise we briefly flash
   // the previous schedule's content while the new fetch is in flight.
   const isStaleSchedule = !!schedule && !!openScheduleId && schedule.friendlyId !== openScheduleId;
+  // Only show the loading spinner when we actually lack good data —
+  // background reloads (e.g. after enable/disable) keep the inspector
+  // visible with its current values until the fresh data arrives.
   const isDetailLoading =
-    detailFetcher.state === "loading" ||
-    isStaleSchedule ||
-    (!!openScheduleId && schedule === undefined);
+    isStaleSchedule || (!!openScheduleId && detailFetcher.data === undefined);
   // Distinct from loading: the loader has resolved and the schedule is
   // genuinely gone (returned `null`, e.g. deleted externally).
   const isScheduleMissing =

apps/webapp/app/routes/resources.orgs.$organizationSlug.projects.$projectParam.env.$envParam.schedules.new/route.tsx

@@ -181,7 +181,9 @@ export function UpsertScheduleForm({
 
   const [form, { taskIdentifier, cron, timezone, externalId, environments, deduplicationKey }] =
     useForm({
-      id: "create-schedule",
+      // Disambiguate per-schedule so both sheets (create + edit) can
+      // coexist without duplicate DOM ids breaking `htmlFor` / conform.
+      id: schedule?.friendlyId ? `edit-schedule-${schedule.friendlyId}` : "create-schedule",
       // TODO: type this
       lastSubmission: lastSubmission as any,
       shouldRevalidate: "onSubmit",