fix: address feedback

Author: finxol

apps/workflows/src/cron/external-status.ts

@@ -404,6 +404,10 @@ export async function runExternalStatusTick(): Promise<{
   const triplets = buildTriplets(services);
   const tickStartedAt = new Date();
 
+  // The three phases are intentionally independent: each hits a different
+  // upstream endpoint and store, so a failed status fetch must not suppress a
+  // service's components (or vice versa). A tick can therefore persist
+  // component history for a service whose status snapshot failed that tick.
   const [statusOutcomes, incidentOutcomes, componentOutcomes] =
     await Effect.runPromise(
       Effect.all(

packages/services/src/external-service-component/upsert.ts

@@ -49,6 +49,10 @@ export async function upsertExternalComponentsForService(args: {
     updatedAt: now,
   }));
 
+  // No `withTransaction`/`emitAudit`: external services are a global, public,
+  // cron-driven catalogue with no workspace scope or audit log (ADR-0006/0007),
+  // mirroring `upsertExternalIncidentsForService`. `withBusyRetry` wraps the raw
+  // transaction since the audit-aware helper expects a workspace ServiceContext.
   return withBusyRetry(() =>
     db.transaction(async (tx) => {
       const rows = await tx