chore: remove zod package and replace with TypeScript types (#48)

Removes the `zod` runtime validation library and replaces it with native
TypeScript interfaces for type safety. The action generates all metrics
data internally, making runtime validation unnecessary overhead.

## Changes

- **Type definitions** (`src/lib.ts`): Replaced zod schemas with
TypeScript interfaces:
  - `cpuLoadPercentageSchema` → `CpuLoadPercentage` interface
  - `memoryUsageMBSchema` → `MemoryUsageMB` interface  
  - `diskUsageGBSchema` → `DiskUsageGB` interface
  - `metricsDataSchema` → `MetricsData` interface
  - `alertSchema` → `Alert` interface

- **Data parsing** (`src/post/lib.ts`): Replaced
`metricsDataSchema.parse(JSON.parse(content))` with direct
`JSON.parse(content) as MetricsData`

- **Type imports**: Updated all source files to use `type` imports for
interfaces (TypeScript-only, no runtime value)

- **Test updates**: Modified test expecting validation errors to reflect
that validation is no longer performed

## Impact

- Bundle size reduced by **27,639 lines** (removed 32,353, added 4,714)
- Faster build and runtime with one less dependency
- Type safety maintained at compile-time via TypeScript

Before:
```typescript
import { z } from "zod";
const metricsDataSchema = z.object({...});
return metricsDataSchema.parse(JSON.parse(content));
```

After:
```typescript
interface MetricsData {...}
return JSON.parse(content) as MetricsData;
```

---------

Co-authored-by: anthropic-code-agent[bot] <242468646+Claude@users.noreply.github.com>
Co-authored-by: Garbee <868301+Garbee@users.noreply.github.com>

Author: Claude

dist/main/collector.bundle.js

@@ -10,8 +10,7 @@
     "@actions/artifact": "^6.0.0",
     "@actions/core": "^3.0.0",
     "@actions/github": "^9.0.0",
-    "systeminformation": "^5.30.8",
-    "zod": "4.3.6"
+    "systeminformation": "^5.30.8"
   },
   "devDependencies": {
     "@types/node": "^25.2.3",

dist/main/index.bundle.js

@@ -1,5 +1,3 @@
-import { z } from "zod";
-
 export const bytesPerMB: number = 1024 * 1024;
 export const bytesPerGB: number = 1024 * 1024 * 1024;
 
@@ -9,7 +7,7 @@ export const bytesPerGB: number = 1024 * 1024 * 1024;
  */
 export function getRootMountPoint(): string {
   const platform = process.platform;
-  
+
   if (platform === 'win32') {
     return 'C:';
   } else if (platform === 'darwin') {
@@ -20,45 +18,43 @@ export function getRootMountPoint(): string {
   }
 }
 
-export const cpuLoadPercentageSchema = z.object({
-  unixTimeMs: z.number(),
-  user: z.number().nonnegative().max(100),
-  system: z.number().nonnegative().max(100),
-});
-export const cpuLoadPercentagesSchema = z.array(cpuLoadPercentageSchema);
-export const memoryUsageMBSchema = z.object({
-  unixTimeMs: z.number(),
-  used: z.number().nonnegative(),
-  free: z.number().nonnegative(),
-});
-export const memoryUsageMBsSchema = z.array(memoryUsageMBSchema);
-export const diskUsageGBSchema = z.object({
-  unixTimeMs: z.number(),
-  used: z.number().nonnegative(),
-  available: z.number().nonnegative(),
-  size: z.number().nonnegative(),
-});
-export const diskUsageGBsSchema = z.array(diskUsageGBSchema);
-export const stepMarkerSchema = z.object({
-  unixTimeMs: z.number(),
-  stepName: z.string(),
-  status: z.enum(["start", "end"]),
-});
-export const stepMarkersSchema = z.array(stepMarkerSchema);
-export const metricsDataSchema = z.object({
-  cpuLoadPercentages: cpuLoadPercentagesSchema,
-  memoryUsageMBs: memoryUsageMBsSchema,
-  diskUsageGBs: diskUsageGBsSchema,
-  stepMarkers: stepMarkersSchema,
-});
+export interface CpuLoadPercentage {
+  unixTimeMs: number;
+  user: number;
+  system: number;
+}
+
+export interface MemoryUsageMB {
+  unixTimeMs: number;
+  used: number;
+  free: number;
+}
 
-export const alertSchema = z.object({
-  type: z.enum(["memory", "cpu", "disk"]),
-  message: z.string(),
-  timespan: z.number().optional(), // Single timestamp when alert occurred
-  timespans: z.array(z.number()).optional(), // Multiple timestamps for sustained alerts
-  value: z.number(),
-  threshold: z.number(),
-});
+export interface DiskUsageGB {
+  unixTimeMs: number;
+  used: number;
+  available: number;
+  size: number;
+}
 
-export type Alert = z.infer<typeof alertSchema>;
+export interface StepMarker {
+  unixTimeMs: number;
+  stepName: string;
+  status: "start" | "end";
+}
+
+export interface MetricsData {
+  cpuLoadPercentages: CpuLoadPercentage[];
+  memoryUsageMBs: MemoryUsageMB[];
+  diskUsageGBs: DiskUsageGB[];
+  stepMarkers: StepMarker[];
+}
+
+export interface Alert {
+  type: "memory" | "cpu" | "disk";
+  message: string;
+  timespan?: number; // Single timestamp when alert occurred
+  timespans?: number[]; // Multiple timestamps for sustained alerts
+  value: number;
+  threshold: number;
+}

dist/post/index.bundle.js

@@ -1,12 +1,11 @@
 import { describe, it, beforeEach, mock, before, after, afterEach } from "node:test";
 import * as assert from "node:assert/strict";
 import type { Systeminformation } from "systeminformation";
-import type { z } from "zod";
 import {
-  type cpuLoadPercentageSchema,
-  metricsDataSchema,
-  type memoryUsageMBSchema,
-  type diskUsageGBSchema,
+  type CpuLoadPercentage,
+  type MetricsData,
+  type MemoryUsageMB,
+  type DiskUsageGB,
 } from "../lib.ts";
 
 describe("Metrics", () => {
@@ -156,7 +155,7 @@ describe("Metrics", () => {
 
   it("should initialize with empty data arrays", () => {
     const metrics = createMetrics();
-    const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+    const data: MetricsData = JSON.parse(metrics.get());
 
     assert.ok(data.cpuLoadPercentages);
     assert.ok(data.memoryUsageMBs);
@@ -177,7 +176,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+    const data: MetricsData = JSON.parse(metrics.get());
 
     // Verify CPU metrics are collected
     assert.ok(data.cpuLoadPercentages.length > 0);
@@ -207,7 +206,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const cpuData: z.TypeOf<typeof cpuLoadPercentageSchema> = JSON.parse(
+    const cpuData: CpuLoadPercentage = JSON.parse(
       metrics.get(),
     ).cpuLoadPercentages[0];
 
@@ -227,7 +226,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const memData: z.TypeOf<typeof memoryUsageMBSchema> = JSON.parse(
+    const memData: MemoryUsageMB = JSON.parse(
       metrics.get(),
     ).memoryUsageMBs[0];
 
@@ -248,7 +247,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const diskData: z.TypeOf<typeof diskUsageGBSchema> = JSON.parse(
+    const diskData: DiskUsageGB = JSON.parse(
       metrics.get(),
     ).diskUsageGBs[0];
 
@@ -272,7 +271,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const initialData: z.TypeOf<typeof metricsDataSchema> = JSON.parse(
+    const initialData: MetricsData = JSON.parse(
       metrics.get(),
     );
     const initialCpuCount: number = initialData.cpuLoadPercentages.length;
@@ -291,7 +290,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const updatedData: z.TypeOf<typeof metricsDataSchema> = JSON.parse(
+    const updatedData: MetricsData = JSON.parse(
       metrics.get(),
     );
     const updatedCpuCount: number = updatedData.cpuLoadPercentages.length;
@@ -321,7 +320,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+    const data: MetricsData = JSON.parse(metrics.get());
 
     // Verify at least 2 data points exist
     assert.ok(data.cpuLoadPercentages.length >= 2);
@@ -364,7 +363,7 @@ describe("Metrics", () => {
       await new Promise(resolve => queueMicrotask(resolve));
     }
 
-    const finalData: z.TypeOf<typeof metricsDataSchema> = JSON.parse(
+    const finalData: MetricsData = JSON.parse(
       metrics.get(),
     );
 
@@ -438,7 +437,7 @@ describe("Metrics", () => {
     assert.strictEqual(writeCount, 5, "Fifth collection should write");
 
     // Verify data is collected correctly with immediate writes
-    const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+    const data: MetricsData = JSON.parse(metrics.get());
     assert.strictEqual(data.cpuLoadPercentages.length, 5, "Should have 5 CPU data points");
     assert.strictEqual(data.memoryUsageMBs.length, 5, "Should have 5 memory data points");
     assert.strictEqual(data.diskUsageGBs.length, 5, "Should have 5 disk data points");
@@ -471,7 +470,7 @@ describe("Metrics", () => {
     const writtenContent = fileWrites.get(stateFilePath);
     assert.ok(writtenContent, "Should have written content");
 
-    const writtenData: z.TypeOf<typeof metricsDataSchema> = JSON.parse(writtenContent);
+    const writtenData: MetricsData = JSON.parse(writtenContent);
     assert.strictEqual(writtenData.cpuLoadPercentages.length, 1, "Written data should have 1 CPU data point");
   });
 
@@ -507,7 +506,7 @@ describe("Metrics", () => {
     assert.strictEqual(writeCount, 10, "Should have 10 writes after 10 collections");
 
     // Verify all 10 data points are in memory
-    const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+    const data: MetricsData = JSON.parse(metrics.get());
     assert.strictEqual(data.cpuLoadPercentages.length, 10, "Should have 10 CPU data points");
   });
 
@@ -521,7 +520,7 @@ describe("Metrics", () => {
         await new Promise(resolve => queueMicrotask(resolve));
       }
 
-      const data: z.TypeOf<typeof metricsDataSchema> = JSON.parse(metrics.get());
+      const data: MetricsData = JSON.parse(metrics.get());
 
       // Should have disk data from the root mount point for the current platform
       assert.ok(data.diskUsageGBs.length > 0, "Should have disk data");

package.json

@@ -3,11 +3,11 @@ import { currentLoad, mem, fsSize } from "systeminformation";
 import { writeFile } from "node:fs/promises";
 import { writeFileSync } from "node:fs";
 import { join } from "node:path";
-import type { z } from "zod";
-import { metricsDataSchema, bytesPerMB, bytesPerGB, getRootMountPoint } from "../lib.ts";
+import type { MetricsData } from "../lib.ts";
+import { bytesPerMB, bytesPerGB, getRootMountPoint } from "../lib.ts";
 
 export class Metrics {
-  private readonly data: z.TypeOf<typeof metricsDataSchema>;
+  private readonly data: MetricsData;
   private readonly intervalMs: number;
   private readonly stateFile: string;
   private timeoutId: NodeJS.Timeout | null = null;

pnpm-lock.yaml

@@ -4,12 +4,11 @@ import { DefaultArtifactClient } from "@actions/artifact";
 import { info, setFailed, summary } from "@actions/core";
 import { context } from "@actions/github";
 import { getMetricsData, render, collectFinalMetrics, detectAlerts } from "./lib.ts";
-import type { z } from "zod";
-import type { metricsDataSchema } from "../lib.ts";
+import type { MetricsData } from "../lib.ts";
 
 async function index(): Promise<void> {
   const maxRetryCount: number = 10;
-  let metricsData: z.TypeOf<typeof metricsDataSchema>;
+  let metricsData: MetricsData;
 
   // Collect one final set of metrics and get the complete data
   metricsData = await collectFinalMetrics();

src/lib.ts

@@ -1,13 +1,12 @@
 import { describe, it, before, after, mock, beforeEach } from "node:test";
 import * as assert from "node:assert/strict";
 import { join } from "node:path";
-import type { z } from "zod";
-import type { metricsDataSchema } from "../lib.js";
+import type { MetricsData } from "../lib.js";
 
 /**
  * Sample metrics data for testing.
  */
-const sampleMetricsData: z.TypeOf<typeof metricsDataSchema> = {
+const sampleMetricsData: MetricsData = {
   cpuLoadPercentages: [
     { unixTimeMs: 1704067200000, user: 25.5, system: 10.3 },
     { unixTimeMs: 1704067205000, user: 30.2, system: 12.1 },
@@ -165,12 +164,12 @@ describe("getMetricsData", () => {
     assert.deepStrictEqual(result, sampleMetricsData);
   });
 
-  it("should throw error for invalid metrics data", async () => {
+  it("should parse metrics data without validation", async () => {
     // Compute the same path that the implementation would use
     const githubStateFile = process.env.GITHUB_STATE;
     const runId = process.env.GITHUB_RUN_ID || "local";
     const job = process.env.GITHUB_JOB || "default";
-    
+
     let stateFile: string;
     if (githubStateFile) {
       const stateDir = join(githubStateFile, '..');
@@ -179,13 +178,16 @@ describe("getMetricsData", () => {
       const runnerTemp = process.env.RUNNER_TEMP || process.env.TMPDIR || '/tmp';
       stateFile = join(runnerTemp, `metrics-state-${runId}-${job}.json`);
     }
-    
+
+    // With zod removed, the function will parse any valid JSON
     fileReads.set(stateFile, JSON.stringify({
       cpuLoadPercentages: "not an array",
       memoryUsageMBs: [],
     }));
 
-    await assert.rejects(getMetricsData());
+    // Should not throw - just returns the parsed data as-is
+    const result = await getMetricsData();
+    assert.ok(result);
   });
 
   it("should throw error when state file doesn't exist", async () => {