Skip to content

chore: add CI — lint SKILL.md frontmatter and smoke-test all scripts #20

Description

@perry-the-pr-reviewer

Problem

The repo has no .github/ directory — zero CI, zero tests. Issues #13 and #14 (wrong throughput sort, wrong modality filter) have been shipping silently because nothing catches regressions before merge.

What CI should do

Frontmatter lint — verify every SKILL.md has required fields:

name: <non-empty string>
description: <non-empty string>

Script smoke test — for every script in */scripts/*.ts, verify:

  1. npm install completes without error
  2. Running the script with no args (or --help) exits cleanly (not with a stack trace)

Optionally: a fixture test for list-models.ts and compare-models.ts that mocks the API response and asserts the sort output is correct — this would have caught #13 immediately.

Why now

The throughput sort bug (#13) is a correctness issue that produces silently wrong agent behavior. With no CI, similar bugs can land undetected. The fix is small — a single GitHub Actions workflow file.

Reviewed by Perry

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions