Skip to content

[codex] ci: add public harness guard - #63

Merged
lefarcen merged 2 commits into
mainfrom
codex/min-public-harness
May 18, 2026
Merged

[codex] ci: add public harness guard#63
lefarcen merged 2 commits into
mainfrom
codex/min-public-harness

Conversation

@PerishCode

Copy link
Copy Markdown
Contributor

What changed

  • Add a public CI workflow for PR and main pushes.
  • Split the repo into a minimal pnpm workspace:
    • @html-anything/next owns the complete Next app under next/.
    • @html-anything/e2e owns Playwright as the sole browser-test source under e2e/.
    • root owns CI, docs, workspace metadata, and scripts/guard.ts.
  • Add scripts/guard.ts to enforce the root / next / e2e project shape.
  • Move the existing UI smoke from tests/ui to e2e/ui.
  • Add a small app Vitest boundary around HTML extraction.
  • Remove next/font/google from the app shell so public CI builds do not depend on external Google font downloads.
  • Update README and AGENTS docs to use pnpm -F @html-anything/next ... and pnpm -F @html-anything/e2e ....

Why

This establishes the minimal public harness boundary before large feature branches such as export-target work rebase onto main. The goal is to make directory shape, app tests, browser smoke, and CI expectations explicit without requiring secrets.

Validation

  • pnpm install --frozen-lockfile
  • pnpm exec tsx scripts/guard.ts
  • pnpm -F @html-anything/next typecheck
  • pnpm -F @html-anything/e2e typecheck
  • pnpm -F @html-anything/next test
  • pnpm -F @html-anything/next build
  • pnpm -F @html-anything/e2e test
  • git diff --check

pnpm -F @html-anything/next build still emits the existing Turbopack NFT tracing warning through next/src/lib/agents/invoke.ts, but the build exits successfully.

@lefarcen
lefarcen requested a review from nettee May 18, 2026 12:08
@lefarcen lefarcen added size/XXL PR size: 1500+ changed lines risk/high High-risk PR: dependencies, infra, security-sensitive, or broad runtime impact type/chore Maintenance/chore change labels May 18, 2026
@PerishCode
PerishCode marked this pull request as ready for review May 18, 2026 12:25
@lefarcen
lefarcen merged commit b799c28 into main May 18, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk/high High-risk PR: dependencies, infra, security-sensitive, or broad runtime impact size/XXL PR size: 1500+ changed lines type/chore Maintenance/chore change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants