Skip to content

Add Node snapshot smoke test workflow#261

Closed
seveibar wants to merge 1 commit intomainfrom
codex/add-nodejs-smoke-test-workflow
Closed

Add Node snapshot smoke test workflow#261
seveibar wants to merge 1 commit intomainfrom
codex/add-nodejs-smoke-test-workflow

Conversation

@seveibar
Copy link
Contributor

@seveibar seveibar commented Jul 8, 2025

Summary

  • add a workflow running a Node-based smoke test for snapshotting
  • create a small Node script that snapshots a simple board using the installed tsci

Testing

  • bun run build
  • npm install -g .
  • node scripts/node-snapshot-test.js
  • bun run format

https://chatgpt.com/codex/tasks/task_b_686545bff3a0832e8284c5e69117a2df

- name: Build and install tsci
run: |
bun run build
npm install -g .
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use bun?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the point of this test is to make sure that someone who doesn't have bun installed can still run tsci, because in the past we had breakage for people who just did npm install -g tscircuit and tsci dev without ever using bun

@seveibar seveibar closed this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants