Debug sync-and-simulate cron job: surface curl output and log auth failures

Remove -s (silent) from curl so CI logs show the HTTP response on failure.
Add logger.error in requireCronSecret so 401s from bad/missing secrets appear in Sentry.

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

.forgejo/workflows/sync-and-simulate.yml

@@ -9,6 +9,6 @@ jobs:
     steps:
       - name: Sync standings and run simulations where changed
         run: |
-          curl -sf -X POST https://brackt.com/admin/jobs/sync-and-simulate \
+          curl -f -X POST https://brackt.com/admin/jobs/sync-and-simulate \
             -H "X-Cron-Secret: ${{ secrets.CRON_SECRET }}" \
             -H "Content-Type: application/json"

app/components/scoring/SeasonStandings.tsx

@@ -99,18 +99,14 @@ export function SeasonStandings({
           <Flag className="inline mr-2 h-5 w-5" />
           {title}
         </CardTitle>
-        <CardDescription>
-          {isFinalized ? (
+        {isFinalized && (
+          <CardDescription>
             <span className="text-emerald-400 font-semibold">
               <CheckCircle2 className="inline h-4 w-4 mr-1" />
               Season complete — fantasy points assigned to top 8 finishers
             </span>
-          ) : (
-            <>
-              Current championship standings. Positions calculated from points (highest = 1st).
-            </>
-          )}
-        </CardDescription>
+          </CardDescription>
+        )}
       </CardHeader>
       <CardContent>
         {standings.length === 0 ? (

app/components/scoring/SportSeasonDisplay.tsx

@@ -173,7 +173,7 @@ export function SportSeasonDisplay({
           userParticipantIds={userParticipantIds}
           showOwnership={showOwnership}
           isFinalized={seasonIsFinalized}
-          title={sportSeasonName}
+          title={`${sportSeasonName} Standings`}
         />
       );
 

app/lib/cron-auth.ts

@@ -1,7 +1,10 @@
+import { logger } from "~/lib/logger";
+
 export function requireCronSecret(request: Request): void {
   const secret = process.env.CRON_SECRET;
   if (!secret) throw new Response("CRON_SECRET not configured", { status: 500 });
   if (request.headers.get("x-cron-secret") !== secret) {
+    logger.error("[cron-auth] Invalid or missing X-Cron-Secret header");
     throw new Response("Unauthorized", { status: 401 });
   }
 }

app/routes/leagues/$leagueId.sports-seasons.$sportsSeasonId.server.ts

@@ -333,9 +333,6 @@ export async function loader(args: Route.LoaderArgs) {
       ])
     : [[], []];
 
-  // Derive seasonIsFinalized from status
-  const seasonIsFinalized = sportsSeason.status === "completed";
-
   // Build participantId → expectedValue map for display
   const participantEvs = Object.fromEntries(
     regularSeasonEvs.map((ev) => [ev.participantId, ev.expectedValue])
@@ -357,7 +354,6 @@ export async function loader(args: Route.LoaderArgs) {
     userParticipantIds,
     upcomingEvents,
     recentEvents,
-    seasonIsFinalized,
     regularSeasonStandings,
     participantEvs,
     groupStandings,

app/routes/leagues/$leagueId.sports-seasons.$sportsSeasonId.tsx

@@ -38,7 +38,6 @@ export default function SportSeasonDetail({
     userParticipantIds,
     upcomingEvents,
     recentEvents,
-    seasonIsFinalized,
     regularSeasonStandings,
 
     groupStandings,
@@ -109,7 +108,7 @@ export default function SportSeasonDetail({
       participantPoints={participantPoints}
       partialScoreParticipantIds={partialScoreParticipantIds}
       seasonStandings={seasonStandings}
-      seasonIsFinalized={seasonIsFinalized}
+      seasonIsFinalized={sportsSeason.status === "completed"}
       qpStandings={qpStandings}
       qpIsFinalized={sportsSeason.qualifyingPointsFinalized || false}
       totalMajors={sportsSeason.totalMajors}

app/services/standings-sync/__tests__/f1.test.ts

@@ -42,7 +42,7 @@ const SAMPLE_ESPN_INDYCAR = {
         athlete: { id: "3097", displayName: "Scott Dixon" },
         stats: [
           { name: "rank", value: 1 },
-          { name: "points", value: 437 },
+          { name: "championshipPts", value: 437 },
           { name: "wins", value: 3 },
           { name: "starts", value: 10 },
         ],
@@ -51,7 +51,7 @@ const SAMPLE_ESPN_INDYCAR = {
         athlete: { id: "3098", displayName: "Alex Palou" },
         stats: [
           { name: "rank", value: 2 },
-          { name: "points", value: 398 },
+          { name: "championshipPts", value: 398 },
           { name: "wins", value: 2 },
           { name: "starts", value: 10 },
         ],

app/services/standings-sync/f1.ts

@@ -114,7 +114,11 @@ export class IndyCarStandingsAdapter implements StandingsSyncAdapter {
 
     return entries.map((entry, idx): FetchedStandingsRecord => {
       const sm = statsMap(entry.stats);
-      const points = sm.get("points")?.value ?? sm.get("pts")?.value ?? 0;
+      const points =
+        sm.get("championshipPts")?.value ??
+        sm.get("points")?.value ??
+        sm.get("pts")?.value ??
+        0;
       const wins = sm.get("wins")?.value ?? 0;
       const starts = sm.get("starts")?.value ?? sm.get("racesStarted")?.value ?? 0;
       const rank = sm.get("rank")?.value ?? sm.get("position")?.value ?? idx + 1;