Fix empty AIC cells in daily guardrail summary when no prior runs exist in 24h window

[Copilot]

When the daily AIC guardrail finds no completed runs within the 24-hour window, all stats are zero. formatAIC(0) intentionally returns "", leaving the summary table with blank cells for total, average, std dev, and min/max AIC — visually indistinguishable from missing data.

Changes

• renderDailyAICSummary: detect stats.count === 0 and render explicit fallbacks instead of relying on formatAICCredits to produce something meaningful for zero values:
  • 24h total AIC → "0" (correct: no credits consumed)
  • Avg AIC / run, Std dev AIC, Min / Max AIC → "—" (undefined without data points)

const noRunData = stats.count === 0;
const totalAICFormatted = formatAICCredits(stats.total) || "0";
const avgAICFormatted = noRunData ? "—" : formatAICCredits(stats.average) || "0";
const stddevAICFormatted = noRunData ? "—" : formatAICCredits(stats.stddev) || "0";
const minMaxAICFormatted = noRunData ? "— / —" : `${formatAICCredits(stats.min)} / ${formatAICCredits(stats.max)}`;

• check_daily_aic_workflow_guardrail.test.cjs: adds a dedicated test asserting the zero-runs rendering produces 0 and — values rather than empty strings.

[Copilot]

Pull request overview

This PR improves the daily AI Credits (AIC) guardrail summary rendering when there are no completed runs in the 24-hour inspection window, avoiding misleading empty table cells caused by formatAICCredits(0) returning an empty string.

Changes:

• Updates renderDailyAICSummary to render explicit fallback values for the “zero prior runs” case (e.g., 24h total AIC → 0, and stats that are undefined without data → —).
• Adds a unit test covering the zero-runs rendering to ensure the markdown includes 0/— rather than blank cells.

Show a summary per file

File	Description
actions/setup/js/check_daily_aic_workflow_guardrail.cjs	Adds stats.count === 0 handling and formatted fallbacks for AIC summary stats to avoid empty markdown cells.
actions/setup/js/check_daily_aic_workflow_guardrail.test.cjs	Adds a test validating correct markdown output when no prior runs exist in the 24h window.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 2/2 changed files
• Comments generated: 1

[mnkiefer]

@copilot Fix tests and lint errors

[Copilot]

@copilot Fix tests and lint errors

Fixed in the latest commit. The previous commit accidentally merged two template literal lines together (missing a comma between them), causing a TS2796 tagged template expression error that broke the typecheck and tests. The comma is now restored.