[PR PREVIEW - DO NOT MERGE] 🌿 Fern Regeneration -- January 23, 2026 #3639
Conversation
vellum-automation
bot
changed the title
vellum-automation
bot
added
the
preview
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0a16a115c2
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| - name: Install dependencies | ||
| run: poetry install | ||
|
|
||
| # Needed for codegen integration tests (test_zero_diff_transforms) | ||
| - name: Set up Node.js | ||
| uses: actions/setup-node@v3 | ||
| with: | ||
| node-version: "18.18.0" | ||
| - name: Install codegen dependencies | ||
| run: npm install | ||
| working-directory: ee/codegen | ||
|
|
||
| - name: Test | ||
| run: make test-ci |
There was a problem hiding this comment.
The test job now runs make test-ci immediately after poetry install, but make test-ci runs pytest . (see Makefile lines 33-41), which includes ee/codegen_integration/test_zero_diff_transforms.py. That test shells out to npx tsx src/cli.ts in ee/codegen (lines 173-187), which requires Node plus the codegen package’s node_modules. With the Node setup and npm install step removed, CI will fail at runtime with missing tsx/module dependencies whenever those integration tests are collected. Consider restoring the Node setup + npm install step or explicitly skipping those tests in CI.
Useful? React with 👍 / 👎.
This PR regenerates code to match the latest API Definition.