feat: add track validation and integrate into action execution

Author: CasperKristiansson

docs/tasks.md

@@ -229,7 +229,7 @@ Use Context7 MCP for up to date documentation.
         Emit `multiple_tracks_detected`, `runners_missing`, `no_matches_found`, `already_latest`, `pr_exists`, `pr_creation_failed`.
         Verify: Tests assert outputs and logs.
 
-37. [ ] **Config validation with zod**
+37. [x] **Config validation with zod**
         Validate `track` as `/^\d+\.\d+$/`.
         Verify: Bad inputs fail fast.
 

src/config.ts

@@ -0,0 +1,14 @@
+import { z } from 'zod';
+
+const trackSchema = z
+  .string()
+  .regex(/^[0-9]+\.[0-9]+$/, 'Track must be in the form X.Y (for example 3.13).');
+
+export function validateTrack(track: string): string {
+  const result = trackSchema.safeParse(track);
+  if (!result.success) {
+    throw new Error(`Input "track" must match X.Y (e.g. 3.13). Received "${track}".`);
+  }
+
+  return result.data;
+}

src/index.ts

@@ -16,6 +16,7 @@ import {
   resolveLatestPatch,
 } from './versioning';
 import { createOrUpdatePullRequest, findExistingPullRequest } from './git';
+import { validateTrack } from './config';
 
 const DEFAULT_TRACK = '3.13';
 const DEFAULT_PATHS = [
@@ -153,6 +154,7 @@ export async function run(): Promise<void> {
   try {
     const trackInput = core.getInput('track').trim();
     const track = trackInput === '' ? DEFAULT_TRACK : trackInput;
+    const validatedTrack = validateTrack(track);
 
     const includePrerelease = getBooleanInput('include_prerelease', false);
     const automerge = getBooleanInput('automerge', false);
@@ -166,7 +168,7 @@ export async function run(): Promise<void> {
 
     core.startGroup('Configuration');
     core.info(`workspace: ${workspace}`);
-    core.info(`track: ${track}`);
+    core.info(`track: ${validatedTrack}`);
     core.info(`include_prerelease: ${includePrerelease}`);
     core.info(`paths (${effectivePaths.length}): ${effectivePaths.join(', ')}`);
     core.info(`automerge: ${automerge}`);
@@ -185,7 +187,7 @@ export async function run(): Promise<void> {
     const result = await executeAction(
       {
         workspace,
-        track,
+        track: validatedTrack,
         includePrerelease,
         paths: effectivePaths,
         dryRun,

tests/config.test.ts

@@ -0,0 +1,16 @@
+import { describe, expect, it } from 'vitest';
+
+import { validateTrack } from '../src/config';
+
+describe('validateTrack', () => {
+  it('returns the track when format is valid', () => {
+    expect(validateTrack('3.13')).toBe('3.13');
+    expect(validateTrack('3.12')).toBe('3.12');
+  });
+
+  it('throws an error when track does not match X.Y', () => {
+    expect(() => validateTrack('3')).toThrow(/must match X\.Y/i);
+    expect(() => validateTrack('3.13.1')).toThrow(/must match X\.Y/i);
+    expect(() => validateTrack('banana')).toThrow(/must match X\.Y/i);
+  });
+});

tests/index.test.ts

@@ -15,10 +15,17 @@ const actionExecutionMocks = vi.hoisted(() => ({
   executeAction: vi.fn(),
 }));
 
+const configMocks = vi.hoisted(() => ({
+  validateTrack: vi.fn((value: string) => value),
+}));
+
 vi.mock('@actions/core', () => coreMocks);
 vi.mock('../src/action-execution', () => ({
   executeAction: actionExecutionMocks.executeAction,
 }));
+vi.mock('../src/config', () => ({
+  validateTrack: configMocks.validateTrack,
+}));
 
 const mockGetInput = coreMocks.getInput;
 const mockGetMultilineInput = coreMocks.getMultilineInput;
@@ -29,6 +36,7 @@ const mockSetOutput = coreMocks.setOutput;
 const mockSetFailed = coreMocks.setFailed;
 const mockWarning = coreMocks.warning;
 const mockExecuteAction = actionExecutionMocks.executeAction as ReturnType<typeof vi.fn>;
+const mockValidateTrack = configMocks.validateTrack as ReturnType<typeof vi.fn>;
 
 import { run } from '../src/index';
 
@@ -46,6 +54,7 @@ describe('run', () => {
       filesChanged: ['Dockerfile'],
       dryRun: true,
     });
+    mockValidateTrack.mockImplementation((value: string) => value);
   });
 
   it('uses default configuration when inputs are empty', async () => {
@@ -134,4 +143,21 @@ describe('run', () => {
 
     expect(mockSetFailed).toHaveBeenCalledWith('run failure');
   });
+
+  it('fails fast when track input is invalid', async () => {
+    mockGetInput.mockImplementation((name: string) => {
+      if (name === 'track') return 'invalid';
+      return '';
+    });
+    mockValidateTrack.mockImplementation(() => {
+      throw new Error('Input "track" must match X.Y (e.g. 3.13). Received "invalid".');
+    });
+
+    await run();
+
+    expect(mockExecuteAction).not.toHaveBeenCalled();
+    expect(mockSetFailed).toHaveBeenCalledWith(
+      'Input "track" must match X.Y (e.g. 3.13). Received "invalid".',
+    );
+  });
 });