Skip to content

test: add CI includeBoardFiles edge-case reproduction for site builds#2121

Open
seveibar wants to merge 1 commit intomainfrom
codex/identify-edge-case-for-site-build-failure
Open

test: add CI includeBoardFiles edge-case reproduction for site builds#2121
seveibar wants to merge 1 commit intomainfrom
codex/identify-edge-case-for-site-build-failure

Conversation

@seveibar
Copy link
Contributor

Motivation

  • Reproduce an edge case where includeBoardFiles is configured but matches no files, causing tsci build --ci --concurrency 4 to produce a site shell and transpiled library output but no built circuit.json artifacts (an “empty” site).

Description

  • Add a focused test tests/cli/build/build-ci-site-include-board-edge-case.test.ts that writes a minimal project config with mainEntrypoint, previewComponentPath, siteDefaultComponentPath, includeBoardFiles, alwaysUseLatestTscircuitOnCloud, and prebuildCommand, runs tsci build --ci --concurrency 4, and asserts the site + transpile outputs exist while no circuit.json files are produced and the static file list in index.html is empty.

Testing

  • Ran the new test with bun test tests/cli/build/build-ci-site-include-board-edge-case.test.ts and it passed.
  • Ran typecheck with bunx tsc --noEmit which succeeded.
  • Ran formatter with bun run format which completed successfully.

Codex Task

@github-actions
Copy link
Contributor

This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant