mirror of
https://github.com/heygen-com/hyperframes.git
synced 2026-09-14 18:01:20 +08:00
d70ee134cc
* feat(cli): add skills version check, update, and freshness manifest
Give the HyperFrames skill bundle a content fingerprint so agents and
users can tell whether installed skills are the latest version, on any
platform that can run the CLI.
- skills-manifest.json (repo root): per-skill sha256 over the whole skill
directory; minimal {source, skills}, no version/timestamp so it is fully
deterministic. Generated by scripts/gen-skills-manifest.ts.
- `hyperframes skills check` [--json]: compares installed skills to the
manifest; exits non-zero when something is outdated (agent/CI gate).
- `hyperframes skills update`: thin wrapper over `npx skills update`.
- Passive nudge on render/lint/validate when skills are stale (24h cache,
same opt-out as the CLI self-update notice).
- "latest" resolved via `git ls-remote` + SHA-pinned raw URL to dodge
GitHub raw-CDN lag, falling back to the main branch URL.
- CI job + lefthook hook keep skills-manifest.json in sync with skills/.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* fix(cli): add execFile to child_process mock in skills test
skills.test.ts mocks node:child_process but only declared execFileSync
and spawn. Loading skills.js transitively loads skillsManifest.ts, which
runs promisify(execFile) at module load, so vitest threw on the missing
execFile named export. Add a bare stub — these tests never invoke it.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* feat(cli): init installs all skills; skills update pulls the full set
Make `hyperframes init` the single place skills are pulled in full, and
make "update" mean "get everything" rather than "refresh what's there".
- init now always installs/refreshes ALL skills (incl. ones not yet
present) instead of prompting "Install AI coding skills?" — opt out
with `init --skip-skills`. Both the interactive and non-interactive
paths pass `--all --yes` so the complete set is fetched.
- `hyperframes skills update` switches from `npx skills update` (which
only refreshes already-installed skills) to `skills add --all`, so it
installs missing skills too — the same install step init runs.
- SKILL.md documents init-installs-all and the new update semantics.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* feat(cli): skills check treats missing skills as needing an update
The full skill set is now the goal (init and `skills update` both pull
all, including ones not installed), so a partial install is no longer
"a choice" — it's something to fix.
- diffSkills: updateAvailable is now true when anything is outdated OR
missing (local-only still doesn't count). So `skills check` exits
non-zero — and renders "Update:" instead of "up to date" — whenever a
skill is missing, not just when one is stale.
- The passive render/lint/validate nudge follows suit: it now counts
missing alongside outdated ("N skills out of date or missing"),
tracked via a new skillsMissingCount cache field.
- SKILL.md documents the stricter check.
Note: platforms that intentionally vendor only a subset of skills (e.g.
a Codex snapshot) will now see check report non-zero.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* fix(cli): install/update skills straight from the GitHub repo
`skills add owner/repo` can resolve through the skills.sh registry, which
lags behind the repo — so `update` could install a stale version while
`check` (which resolves latest directly from GitHub) keeps reporting
"outdated", an endless loop.
Switch the install source to the full GitHub URL
(https://github.com/heygen-com/hyperframes), which makes `skills add`
git-clone the repo directly at latest main, bypassing the registry. This
covers `hyperframes skills`, `hyperframes skills update`, and `init`'s
skill install — all of which go through SOURCES. Now install/update and
check agree on what "latest" means.
The init "install skills" hint now points at `npx hyperframes skills
update` so the manual path uses the same GitHub-direct fetch.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* feat(cli): init checks skills against GitHub, installs only when stale
`hyperframes init` now runs the skills version check first and only
(re)installs when something is outdated or missing — instead of
unconditionally re-pulling every time. Re-running init on an
already-current project is now a no-op ("skills are already up to date").
- New ensureSkillsCurrent() helper, shared by both the interactive and
non-interactive init paths (no duplicated install logic).
- The check resolves "latest" straight from GitHub (same source the
install uses); best-effort — if it can't reach GitHub it installs anyway.
- SKILL.md updated to describe the check-then-install behavior.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* refactor(cli): address skills manifest review feedback
From the PR review (points 1, 2, 4, 5):
1. Remove the `local-only` skill status. checkSkills only ever hashes
manifest-listed skills, so a local-only status could never appear in
the end-to-end output — and making it appear would wrongly flag
unrelated skills (the `.../skills` dir is shared across sources).
diffSkills now reports only on manifest skills; skills on disk that
aren't in the manifest are ignored.
2. Drop the redundant per-directory sort in listFilesSorted — the single
final out.sort() is what guarantees a deterministic hash (verified:
manifest unchanged).
4. resolveLatestManifest local-path detection now uses path.isAbsolute,
so Windows absolute paths (C:\...) are treated as local instead of
falling through to a remote fetch.
5. fetchManifest validates the response shape (asSkillsManifest) instead
of a blind `as` cast, so a CDN error page served as 200 fails with a
clear error rather than a cryptic crash later in diffSkills.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* fix(cli): strict skills update + auto-discover any agent host
Address PR review (Magi blocker + James/Rames robustness):
- Blocker (Magi): `skills update` is the documented recovery path for
`skills check || skills update`, but it delegated to installAllSkills()
which swallowed missing-npx and failed `skills add` as "skipped",
exiting 0 even when nothing changed. Add a strict mode that throws on
failure; update sets a non-zero exit (init stays best-effort). New tests
simulate a non-zero `skills add` (exit 1) and the success path.
- Robustness (James/Rames #2): the upstream `skills` CLI installs into
~72 agent conventions; a hard-coded list (4, or even 11) can't track
that. Replace defaultSkillRoots with discoverSkillRoots — it scans cwd +
$HOME for any `<host>/skills/<manifest-skill>/SKILL.md` (plus the XDG
`.config/<host>/skills`), so detection is structural and future-proof,
no closed list. agentFromDir infers the host from the path.
- Tests (Rames #3): temp-fixture detection tests for every convention ×
{project, global}, scope priority, claude-code preference, the
no-install case, the --dir override, and an unknown/new host (proving
the no-closed-list property).
- Docs (Rames #4/#5): SKILL.md notes init's best-effort GitHub round-trip;
findRepoManifest climbs 16 levels (was 8) for deep monorepos.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* fix(cli): resolve CodeQL file-system race + de-flake Windows npx test
Two CI fixes:
- CodeQL (high, js/file-system-race) at gen-skills-manifest.ts: the
existsSync(outPath) precheck followed by writeFileSync(outPath) is a
check-then-write race. Read the committed manifest directly in a
try/catch instead (missing/unreadable ⇒ "no committed manifest"), so
there's no precheck to race against. Behavior is unchanged.
- Windows Tests: npxCommand.test.ts's real `npx --version` smoke test
cold-starts slower than vitest's 5s default on Windows runners and
timed out. Give the test 60s headroom (and a 30s exec timeout). Kept
as a real execution check — mocking would reduce it to a tautology.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* fix(cli): repair garbled npx smoke-test timeout comment
The explanatory comment for the 60s timeout was scrambled across the
callback/timeout arguments, failing oxfmt --check (and thus preflight,
which in turn skipped preview-parity and failed the regression gate).
Move it above the it() call so it no longer sits between call arguments.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
703 lines
27 KiB
YAML
703 lines
27 KiB
YAML
name: CI
|
|
|
|
permissions:
|
|
contents: read
|
|
pull-requests: read
|
|
|
|
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
|
|
# External users' CI continues to emit telemetry unless they set this themselves.
|
|
env:
|
|
HYPERFRAMES_NO_TELEMETRY: "1"
|
|
|
|
on:
|
|
pull_request:
|
|
# `edited` is required so the workflow re-fires when a PR's base ref is
|
|
# set back to `main` after a Graphite stack restack momentarily flips
|
|
# the base off of `main`. Without it, `pull_request` triggers are not
|
|
# re-evaluated on `base_ref_changed`, leaving required checks skipped
|
|
# for that head SHA forever.
|
|
types: [opened, synchronize, reopened, edited]
|
|
branches: [main]
|
|
push:
|
|
branches: [main]
|
|
|
|
concurrency:
|
|
group: ci-${{ github.ref }}
|
|
cancel-in-progress: true
|
|
|
|
jobs:
|
|
changes:
|
|
name: Detect changes
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 2
|
|
outputs:
|
|
code: ${{ steps.filter.outputs.code }}
|
|
cli: ${{ steps.filter.outputs.cli }}
|
|
skills: ${{ steps.filter.outputs.skills }}
|
|
steps:
|
|
# Force git-based change detection instead of the pull_request REST API.
|
|
# The API path can fail the whole workflow on transient listFiles
|
|
# timeouts before any real CI work starts.
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
fetch-depth: 0
|
|
- uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4
|
|
id: filter
|
|
with:
|
|
token: ""
|
|
filters: |
|
|
code:
|
|
- "packages/**"
|
|
- "scripts/**"
|
|
- "package.json"
|
|
- "bun.lock"
|
|
- "tsconfig*.json"
|
|
- "Dockerfile*"
|
|
- ".github/workflows/**"
|
|
cli:
|
|
- "packages/cli/**"
|
|
- "package.json"
|
|
- "bun.lock"
|
|
- ".github/workflows/ci.yml"
|
|
skills:
|
|
- "skills/**"
|
|
- "skills-manifest.json"
|
|
- "package.json"
|
|
- ".github/workflows/ci.yml"
|
|
|
|
build:
|
|
name: Build
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run build
|
|
|
|
lint:
|
|
name: Lint
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run lint
|
|
|
|
# `fallow audit` runs dead-code + complexity + duplication analysis scoped to
|
|
# the changed files. The default `--gate new-only` means existing legacy
|
|
# findings don't fail the build — only NEW issues introduced by the PR do.
|
|
# This stops bleeding while letting incremental cleanup land separately.
|
|
#
|
|
# On findings, the job posts (or updates) a sticky comment on the PR so
|
|
# reviewers see the full list inline instead of digging through CI logs.
|
|
fallow:
|
|
name: Fallow audit
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true' && github.event_name == 'pull_request'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
# Scope write access to this single job — the rest of `ci.yml` keeps the
|
|
# workflow-level `pull-requests: read` default so build / lint / test
|
|
# tokens can't post or modify PR comments. Job-level permissions override
|
|
# the workflow block.
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
# Full history so `--base origin/main` can diff against the merge
|
|
# base on stacked PRs, not just the shallow tip.
|
|
fetch-depth: 0
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- name: Run fallow audit
|
|
id: audit
|
|
# `bun install` above made `bunx fallow` resolve from node_modules, so
|
|
# we don't re-download fallow each run. The script disables `errexit`
|
|
# so the audit's non-zero exit (on findings) doesn't abort before we
|
|
# write the exit code to the step output. The size check guards
|
|
# against fallow crashing before producing markdown (e.g. transient
|
|
# parse failure) — without it we'd post a blank sticky comment.
|
|
run: |
|
|
set +e
|
|
bunx fallow audit --base origin/main --fail-on-issues \
|
|
--format pr-comment-github \
|
|
> /tmp/fallow-comment.md
|
|
echo "exit_code=$?" >> "$GITHUB_OUTPUT"
|
|
if [ ! -s /tmp/fallow-comment.md ]; then
|
|
echo "fallow produced no output — see the job logs above." > /tmp/fallow-comment.md
|
|
fi
|
|
- name: Post sticky comment (findings)
|
|
if: steps.audit.outputs.exit_code != '0'
|
|
# Fork PRs run with a read-only GITHUB_TOKEN regardless of the
|
|
# workflow's `permissions:` block, so the comment post will fail on
|
|
# forks. Don't fail the whole job — the audit gate below still fires.
|
|
continue-on-error: true
|
|
uses: marocchino/sticky-pull-request-comment@52423e01640425a022ef5fd42c6fb5f633a02728 # v2.9.1
|
|
with:
|
|
# `header` matches fallow's built-in `<!-- fallow-id: fallow-results -->`
|
|
# sentinel so subsequent runs update the same comment.
|
|
header: fallow-results
|
|
path: /tmp/fallow-comment.md
|
|
- name: Remove stale sticky comment (clean run)
|
|
if: steps.audit.outputs.exit_code == '0'
|
|
continue-on-error: true
|
|
uses: marocchino/sticky-pull-request-comment@52423e01640425a022ef5fd42c6fb5f633a02728 # v2.9.1
|
|
with:
|
|
header: fallow-results
|
|
delete: true
|
|
- name: Fail if audit found issues
|
|
if: steps.audit.outputs.exit_code != '0'
|
|
run: |
|
|
echo "::error::Fallow audit found new issues — see the PR comment above for details."
|
|
exit 1
|
|
|
|
format:
|
|
name: Format
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run format:check
|
|
|
|
typecheck:
|
|
name: Typecheck
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run build
|
|
- run: bun run --filter '*' typecheck
|
|
|
|
test:
|
|
name: Test
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run test:scripts
|
|
- run: bun run --cwd packages/core build:hyperframes-runtime
|
|
- run: bun run --filter '!@hyperframes/producer' test
|
|
|
|
# Tests under skills/**/*.test.mjs are bare `node --test` files with only
|
|
# `node:` built-in imports. They aren't part of any workspace package, and
|
|
# the main `Test` job's `code` path filter excludes `skills/**`, so without
|
|
# this dedicated job they'd never run in CI. Examples:
|
|
# * skills/media-use/scripts/resolve.test.mjs
|
|
# * skills/media-use/scripts/lib/manifest.test.mjs
|
|
# Several of these are regression guards (e.g. shell-injection cases), so
|
|
# the whole point is that they fire on PRs that touch skills/.
|
|
test-skills:
|
|
name: "Test: skills"
|
|
needs: changes
|
|
if: needs.changes.outputs.skills == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- name: Discover and run skills tests
|
|
# We expand the test list via bash so the job fails loudly when the
|
|
# matcher comes back empty, rather than silently no-op'ing (which
|
|
# would defeat the whole point of this job).
|
|
run: |
|
|
set -euo pipefail
|
|
mapfile -t SKILLS_TESTS < <(find skills -type f -name "*.test.mjs" | sort)
|
|
if [ "${#SKILLS_TESTS[@]}" -eq 0 ]; then
|
|
echo "::error::No skills/**/*.test.mjs files found. Did the layout change?"
|
|
exit 1
|
|
fi
|
|
printf 'Running %d skills test file(s):\n' "${#SKILLS_TESTS[@]}"
|
|
printf ' * %s\n' "${SKILLS_TESTS[@]}"
|
|
node --test "${SKILLS_TESTS[@]}"
|
|
|
|
# Guards that skills-manifest.json (the published freshness fingerprint read
|
|
# by `hyperframes skills check`) was regenerated when a skill changed. Runs
|
|
# `gen:skills-manifest --check`, which compares per-skill content hashes; the
|
|
# manifest carries no version/timestamp, so it only fails on real content
|
|
# drift. bun runs the TS script directly, no install needed.
|
|
skills-manifest:
|
|
name: "Skills: manifest in sync"
|
|
needs: changes
|
|
if: needs.changes.outputs.skills == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 3
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- name: Verify skills-manifest.json matches skills/
|
|
run: bun packages/cli/scripts/gen-skills-manifest.ts --check
|
|
|
|
cli-npx-shim:
|
|
name: "CLI: npx shim (${{ matrix.os }})"
|
|
needs: changes
|
|
if: needs.changes.outputs.cli == 'true'
|
|
runs-on: ${{ matrix.os }}
|
|
timeout-minutes: 10
|
|
strategy:
|
|
fail-fast: false
|
|
matrix:
|
|
os: [ubuntu-latest, macos-latest, windows-latest]
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- name: Install dependencies
|
|
if: runner.os != 'Windows'
|
|
run: bun install --frozen-lockfile --ignore-scripts
|
|
- name: Install dependencies
|
|
if: runner.os == 'Windows'
|
|
run: bun install --frozen-lockfile --ignore-scripts --linker=hoisted
|
|
- run: bun run --cwd packages/cli test src/utils/npxCommand.test.ts src/commands/skills.test.ts
|
|
|
|
sdk-tests:
|
|
name: "SDK: unit + contract + smoke"
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run --filter @hyperframes/sdk test
|
|
|
|
test-runtime-contract:
|
|
name: "Test: runtime contract"
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run --filter @hyperframes/core test:hyperframe-runtime-ci
|
|
|
|
studio-load-smoke:
|
|
name: "Studio: load smoke"
|
|
needs: [changes]
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run --cwd packages/core build:hyperframes-runtime
|
|
- name: Start studio and check for runtime errors
|
|
run: |
|
|
# Start the studio Vite dev server (fast — no bundle step)
|
|
bun run --filter '@hyperframes/studio' dev -- --port 5199 &
|
|
SERVER_PID=$!
|
|
|
|
# Wait for the server to be ready (up to 20s)
|
|
for i in $(seq 1 40); do
|
|
if curl -sf http://localhost:5199/ >/dev/null 2>&1; then break; fi
|
|
sleep 0.5
|
|
done
|
|
|
|
if ! curl -sf http://localhost:5199/ >/dev/null 2>&1; then
|
|
echo "FAIL: studio dev server did not start"
|
|
kill $SERVER_PID 2>/dev/null || true
|
|
exit 1
|
|
fi
|
|
|
|
# Load the studio in headless Chrome with API mocking to trigger
|
|
# the full splash→main transition (catches hooks-after-early-return bugs)
|
|
cd packages/producer
|
|
node --input-type=module <<'SMOKE_EOF'
|
|
import puppeteer from "puppeteer";
|
|
const browser = await puppeteer.launch({
|
|
headless: "new",
|
|
args: ["--no-sandbox", "--disable-setuid-sandbox"],
|
|
});
|
|
const page = await browser.newPage();
|
|
const errors = [];
|
|
page.on("pageerror", (err) => errors.push(err.message));
|
|
page.on("console", (msg) => {
|
|
if (msg.type() === "error") errors.push(msg.text());
|
|
});
|
|
|
|
// Mock the project API so the studio transitions past the splash screen.
|
|
// Without this, useServerConnection stays in "waiting" and the full React
|
|
// tree (with all hooks) never renders — missing hooks-order violations.
|
|
const COMP_HTML = '<div data-composition-id="root" data-width="1920" data-height="1080" data-duration="1" data-start="0"><div class="clip" data-start="0" data-duration="1">Test</div></div>';
|
|
await page.setRequestInterception(true);
|
|
page.on("request", (req) => {
|
|
const url = req.url();
|
|
if (url.includes("/api/projects") && !url.includes("/files") && !url.includes("/preview") && !url.includes("/gsap")) {
|
|
req.respond({
|
|
status: 200,
|
|
contentType: "application/json",
|
|
body: JSON.stringify({ projects: [{ id: "smoke-test" }] }),
|
|
});
|
|
} else if (url.includes("/api/") && url.includes("/files")) {
|
|
req.respond({
|
|
status: 200,
|
|
contentType: "application/json",
|
|
body: JSON.stringify({ files: [{ path: "index.html", type: "file" }] }),
|
|
});
|
|
} else if (url.includes("/api/") && url.includes("/preview")) {
|
|
req.respond({ status: 200, contentType: "text/html", body: COMP_HTML });
|
|
} else if (url.includes("/api/")) {
|
|
req.respond({ status: 200, contentType: "application/json", body: JSON.stringify({}) });
|
|
} else {
|
|
req.continue();
|
|
}
|
|
});
|
|
|
|
await page.goto("http://localhost:5199/#project=smoke-test", {
|
|
waitUntil: "networkidle0",
|
|
timeout: 30000,
|
|
});
|
|
// Wait for React to render past splash into the full studio UI
|
|
await new Promise((r) => setTimeout(r, 3000));
|
|
|
|
// Check for React error boundary (catches hooks violations, render crashes)
|
|
const errorBoundary = await page.evaluate(() => {
|
|
const text = document.body.innerText;
|
|
if (text.includes("Something went wrong")) return text;
|
|
return null;
|
|
});
|
|
if (errorBoundary) {
|
|
errors.push("React error boundary triggered: " + errorBoundary);
|
|
}
|
|
await browser.close();
|
|
// Filter expected noise from mock endpoints
|
|
const fatal = errors.filter(
|
|
(e) =>
|
|
!e.includes("favicon") &&
|
|
!e.includes("ERR_CONNECTION_REFUSED") &&
|
|
!e.includes("Failed to fetch") &&
|
|
!e.includes("is not iterable") &&
|
|
!e.includes("Cannot read properties of undefined") &&
|
|
!e.includes("Cannot read properties of null"),
|
|
);
|
|
if (fatal.length > 0) {
|
|
console.error("FAIL: studio had runtime errors:");
|
|
for (const e of fatal) console.error(" •", e);
|
|
process.exit(1);
|
|
}
|
|
console.log("PASS: studio loaded and transitioned without runtime errors");
|
|
SMOKE_EOF
|
|
|
|
kill $SERVER_PID 2>/dev/null || true
|
|
|
|
smoke-global-install:
|
|
name: "Smoke: global install"
|
|
needs: [changes, build]
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 10
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- run: bun install --frozen-lockfile
|
|
- run: bun run build
|
|
|
|
# Pack the CLI as a tarball (simulates what `npm publish` produces)
|
|
- name: Pack CLI tarball
|
|
run: cd packages/cli && npm pack
|
|
|
|
# Install globally using --prefix to avoid sudo
|
|
- name: Install globally via npm
|
|
run: npm install -g --prefix /tmp/hf-smoke ./packages/cli/hyperframes-cli-*.tgz
|
|
|
|
# Scaffold a blank project
|
|
- name: Init blank project
|
|
run: |
|
|
export PATH="/tmp/hf-smoke/bin:$PATH"
|
|
mkdir /tmp/hf-project && cd /tmp/hf-project
|
|
hyperframes init test-project --example blank
|
|
|
|
# Start preview, probe the runtime endpoint, assert no esbuild errors
|
|
- name: Smoke-test preview server
|
|
run: |
|
|
export PATH="/tmp/hf-smoke/bin:$PATH"
|
|
cd /tmp/hf-project/test-project
|
|
|
|
# Start the preview server in the background; capture stderr
|
|
CI=true hyperframes preview --port 3099 2>/tmp/hf-stderr.log &
|
|
SERVER_PID=$!
|
|
|
|
# Wait for the server to be ready (up to 15 s)
|
|
for i in $(seq 1 30); do
|
|
if curl -sf http://localhost:3099/ >/dev/null 2>&1; then
|
|
break
|
|
fi
|
|
sleep 0.5
|
|
done
|
|
|
|
# Probe the runtime JS endpoint
|
|
BODY=$(curl -sf http://localhost:3099/api/runtime.js | head -c 200 || true)
|
|
if [ -z "$BODY" ]; then
|
|
echo "FAIL: /api/runtime.js returned empty response"
|
|
kill $SERVER_PID 2>/dev/null || true
|
|
cat /tmp/hf-stderr.log
|
|
exit 1
|
|
fi
|
|
|
|
kill $SERVER_PID 2>/dev/null || true
|
|
wait $SERVER_PID 2>/dev/null || true
|
|
|
|
# Assert stderr does not contain esbuild / runtime load errors
|
|
if grep -qE '✘ \[ERROR\]|Failed to load runtime' /tmp/hf-stderr.log; then
|
|
echo "FAIL: preview emitted runtime errors:"
|
|
cat /tmp/hf-stderr.log
|
|
exit 1
|
|
fi
|
|
|
|
echo "PASS: global install smoke test succeeded"
|
|
|
|
cli-smoke-required:
|
|
name: "CLI smoke (required)"
|
|
needs: changes
|
|
if: needs.changes.outputs.code == 'true'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 25
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
lfs: true
|
|
- uses: oven-sh/setup-bun@0c5077e51419868618aeaa5fe8019c62421857d6 # v2
|
|
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
|
|
with:
|
|
node-version: 22
|
|
- name: Install FFmpeg
|
|
run: |
|
|
sudo apt-get update
|
|
sudo apt-get install -y ffmpeg
|
|
- uses: ./.github/actions/prepare-ffmpeg-bin
|
|
- name: Install dependencies
|
|
run: bun install --frozen-lockfile
|
|
- name: Build monorepo
|
|
run: bun run build
|
|
|
|
- name: Create smoke input video
|
|
run: |
|
|
set -euo pipefail
|
|
ffmpeg -hide_banner -loglevel error \
|
|
-f lavfi -i testsrc2=size=640x360:rate=30 \
|
|
-f lavfi -i sine=frequency=880:sample_rate=48000 \
|
|
-t 3 \
|
|
-c:v libx264 \
|
|
-pix_fmt yuv420p \
|
|
-c:a aac \
|
|
-shortest \
|
|
-y /tmp/hf-cli-input.mp4
|
|
test -s /tmp/hf-cli-input.mp4
|
|
|
|
- name: Smoke-test CLI from monorepo source
|
|
run: |
|
|
set -euo pipefail
|
|
rm -rf /tmp/hf-cli-inside
|
|
|
|
bun run --filter @hyperframes/cli dev -- init /tmp/hf-cli-inside \
|
|
--example warm-grain \
|
|
--video /tmp/hf-cli-input.mp4 \
|
|
--skip-transcribe \
|
|
--non-interactive \
|
|
--skip-skills
|
|
|
|
bun run --filter @hyperframes/cli dev -- lint /tmp/hf-cli-inside
|
|
bun run --filter @hyperframes/cli dev -- validate /tmp/hf-cli-inside --timeout 3000
|
|
bun run --filter @hyperframes/cli dev -- render /tmp/hf-cli-inside \
|
|
--quality standard \
|
|
--workers auto \
|
|
--strict \
|
|
--output /tmp/hf-cli-inside/renders/inside.mp4 2>&1 | tee /tmp/hf-cli-render.log
|
|
|
|
test -s /tmp/hf-cli-inside/renders/inside.mp4
|
|
|
|
- name: Assert page.goto completes under 5s budget
|
|
run: |
|
|
set -euo pipefail
|
|
# Extract the longest page.goto time from render logs.
|
|
# Format: [initSession:MODE] page.goto complete (NNNms)
|
|
MAX_MS=$(grep -oP 'page\.goto complete \(\K[0-9]+' /tmp/hf-cli-render.log | sort -n | tail -1)
|
|
if [ -z "$MAX_MS" ]; then
|
|
echo "::error::No page.goto timing found in render logs — log format may have changed. Update the grep pattern."
|
|
exit 1
|
|
fi
|
|
echo "Slowest page.goto: ${MAX_MS}ms (budget: 5000ms)"
|
|
if [ "$MAX_MS" -gt 5000 ]; then
|
|
echo "::error::page.goto took ${MAX_MS}ms — exceeds 5s budget. Possible Proxy/stub regression."
|
|
exit 1
|
|
fi
|
|
|
|
- name: Pack CLI tarball
|
|
run: |
|
|
set -euo pipefail
|
|
mkdir -p /tmp/hf-cli-pack
|
|
cd packages/cli
|
|
PACKED_TARBALL="$(npm pack --pack-destination /tmp/hf-cli-pack | tail -n 1)"
|
|
test -n "$PACKED_TARBALL"
|
|
test -f "/tmp/hf-cli-pack/$PACKED_TARBALL"
|
|
echo "HF_CLI_TARBALL=/tmp/hf-cli-pack/$PACKED_TARBALL" >> "$GITHUB_ENV"
|
|
|
|
- name: Install packed CLI outside monorepo
|
|
run: |
|
|
set -euo pipefail
|
|
npm install -g --prefix /tmp/hf-cli-global "$HF_CLI_TARBALL"
|
|
|
|
- name: Smoke-test packed CLI outside monorepo
|
|
run: |
|
|
set -euo pipefail
|
|
export PATH="/tmp/hf-cli-global/bin:$PATH"
|
|
rm -rf /tmp/hf-cli-outside
|
|
|
|
hyperframes init /tmp/hf-cli-outside \
|
|
--example warm-grain \
|
|
--video /tmp/hf-cli-input.mp4 \
|
|
--skip-transcribe \
|
|
--non-interactive \
|
|
--skip-skills
|
|
|
|
hyperframes lint /tmp/hf-cli-outside
|
|
hyperframes validate /tmp/hf-cli-outside --timeout 3000
|
|
hyperframes render /tmp/hf-cli-outside \
|
|
--quality standard \
|
|
--workers auto \
|
|
--strict \
|
|
--output /tmp/hf-cli-outside/renders/outside.mp4
|
|
|
|
test -s /tmp/hf-cli-outside/renders/outside.mp4
|
|
|
|
filesize:
|
|
name: File size check
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 1
|
|
steps:
|
|
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
|
|
with:
|
|
fetch-depth: 0
|
|
- name: Check file sizes (max 600 lines)
|
|
# Scoped to files THIS PR changed under packages/studio. Walking the
|
|
# whole tree blamed every unrelated PR for pre-existing offenders.
|
|
# Falls back to a full scan on push events (no base ref available)
|
|
# so the rule still guards main.
|
|
run: |
|
|
set -e
|
|
if [ -n "${{ github.base_ref }}" ]; then
|
|
mapfile -t files < <(
|
|
git diff --name-only --diff-filter=ACMR \
|
|
"origin/${{ github.base_ref }}...HEAD" -- \
|
|
'packages/studio/**/*.ts' 'packages/studio/**/*.tsx' \
|
|
| grep -vE '\.(test|spec)\.(ts|tsx)$|\.generated\.' || true
|
|
)
|
|
else
|
|
mapfile -t files < <(
|
|
find packages/studio -path '*/node_modules' -prune -o \
|
|
\( -name '*.ts' -o -name '*.tsx' \) -print \
|
|
| grep -vE '\.(test|spec)\.(ts|tsx)$|\.generated\.'
|
|
)
|
|
fi
|
|
EXIT=0
|
|
for f in "${files[@]}"; do
|
|
[ -z "$f" ] && continue
|
|
[ -f "$f" ] || continue # skip files deleted in this PR
|
|
lines=$(wc -l < "$f")
|
|
if [ "$lines" -gt 600 ]; then
|
|
echo "::error file=$f::$f has $lines lines (max 600)"
|
|
EXIT=1
|
|
fi
|
|
done
|
|
exit $EXIT
|
|
|
|
semantic-pr-title:
|
|
name: Semantic PR title
|
|
if: github.event_name == 'pull_request'
|
|
runs-on: ubuntu-latest
|
|
timeout-minutes: 5
|
|
steps:
|
|
- uses: amannn/action-semantic-pull-request@e32d7e603df1aa1ba07e981f2a23455dee596825 # v5
|
|
env:
|
|
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
|
with:
|
|
types: |
|
|
feat
|
|
fix
|
|
docs
|
|
style
|
|
refactor
|
|
perf
|
|
test
|
|
build
|
|
ci
|
|
chore
|
|
revert
|