Skip to content

Conversation

@fern-api
Copy link
Contributor

@fern-api fern-api bot commented Nov 13, 2025

This PR regenerates code to match the latest API Definition.

@fern-api fern-api bot force-pushed the fern-bot/2025-11-13T22-30Z branch from 7172e13 to 86d78ee Compare November 13, 2025 22:30
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

Comment on lines 26 to 32
from .resources.sandboxes.client import AsyncSandboxesClient, SandboxesClient
from .resources.test_suite_runs.client import AsyncTestSuiteRunsClient, TestSuiteRunsClient
from .resources.test_suites.client import AsyncTestSuitesClient, TestSuitesClient
from .resources.uploaded_files.client import AsyncUploadedFilesClient, UploadedFilesClient
from .resources.workflow_deployments.client import AsyncWorkflowDeploymentsClient, WorkflowDeploymentsClient
from .resources.workflow_executions.client import AsyncWorkflowExecutionsClient, WorkflowExecutionsClient
from .resources.workflow_sandboxes.client import AsyncWorkflowSandboxesClient, WorkflowSandboxesClient

Choose a reason for hiding this comment

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

P1 Badge Preserve vellum.client.core import path

This change turns vellum.client into a standalone module (see new client.py). The previous package hierarchy vellum.client.core has been deleted, but dozens of modules under src/vellum/workflows/** and the CLI (ee/vellum_cli/*.py) still do from vellum.client.core import ApiError, RequestOptions, etc. Because vellum.client is now a module rather than a package, importing any of those modules raises ModuleNotFoundError: No module named 'vellum.client.core'; 'vellum.client' is not a package before any runtime code executes. We need to keep vellum.client as a package (or add a shim re-exporting core from the new location) or update every caller to import from vellum.core.*; otherwise none of the workflow or CLI helpers can even be imported.

Useful? React with 👍 / 👎.

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.

1 participant