Skip to content

fix: resolve build and typecheck failures across packages#9

Merged
jrusso1020 merged 1 commit intomainfrom
03-23-fix_resolve_build_and_typecheck_failures_across_packages
Mar 23, 2026
Merged

fix: resolve build and typecheck failures across packages#9
jrusso1020 merged 1 commit intomainfrom
03-23-fix_resolve_build_and_typecheck_failures_across_packages

Conversation

@jrusso1020
Copy link
Copy Markdown
Collaborator

What

Fix pre-existing build and typecheck failures that prevent pnpm build and pnpm -r typecheck from succeeding.

Why

CI pipeline (next PR in stack) needs clean build + typecheck. These issues block that.

How

  • engine: Exclude *.test.ts from tsconfig exclude (tsc was failing on vitest imports)
  • engine: Add missing vitest devDependency and test script
  • producer: Generate .d.ts declarations via tsc --emitDeclarationOnly after esbuild bundle (CLI couldn't resolve producer types)
  • cli: Replace stale renderComposition() calls with createRenderJob() + executeRenderJob() (API was renamed during migration)

Test plan

  • pnpm build passes
  • pnpm -r typecheck passes
  • pnpm --filter @hyperframes/core test — 330 tests pass
  • pnpm --filter @hyperframes/engine test — 18 tests pass
  • pnpm --filter @hyperframes/core test:hyperframe-runtime-ci — all contract tests pass

@jrusso1020 jrusso1020 merged commit 3e4692d into main Mar 23, 2026
1 check passed
vanceingalls added a commit that referenced this pull request Apr 16, 2026
Blockers:
- #2: late_init_set false positive on fractional opacity (0.5 matched as 0)
  Fixed: /opacity\s*:\s*0(?![.\d])/ negative lookahead
- #3: scene-1 prefix skip matches scene 10+ (s1- matches s10-)
  Fixed: extract full number and compare exactly

High severity:
- #4: autoAlpha not covered by late_init_set
  Fixed: checks both opacity and autoAlpha
- #5: al() crashes on non-hex colors (#fff shorthand, rgb(), null)
  Fixed: guard + shorthand expansion + NaN fallback
- #6: "Full palette" with null bg crashes isDark
  Fixed: null guard defaults to dark
- #7: template literals missed by tl_from_in_multiscene
  Fixed: regex includes backtick quotes

Medium:
- #9: no retry limit on eval failures → infinite loop
  Fixed: max 2 retries, then escalate to user
- #10: vague ID convention
  Fixed: explicit s{N}- prefix rule in multi-scene.md
- #11: visual-style.md backward compat
  Fixed: Step 0b checks both filenames
- #13: preview_html script injection
  Fixed: documented prohibition in design-picker.md

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants