Skip to content

Move workflow API to agents/workflows subpath#827

Merged
threepointone merged 1 commit into
mainfrom
workflow-exports
Feb 2, 2026
Merged

Move workflow API to agents/workflows subpath#827
threepointone merged 1 commit into
mainfrom
workflow-exports

Conversation

@threepointone

Copy link
Copy Markdown
Contributor

Move workflow-related exports into a dedicated agents/workflows subpath. Renamed packages/agents/src/workflow.ts to workflows.ts, added the ./workflows export entry to package.json, removed workflow re-exports from index.ts, and updated imports across docs, examples, and tests to use "agents/workflows". Added a changeset for the patch release.

Move workflow-related exports into a dedicated agents/workflows subpath. Renamed packages/agents/src/workflow.ts to workflows.ts, added the ./workflows export entry to package.json, removed workflow re-exports from index.ts, and updated imports across docs, examples, and tests to use "agents/workflows". Added a changeset for the patch release.
@changeset-bot

changeset-bot Bot commented Feb 2, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: bb5ab1f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
agents Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude

claude Bot commented Feb 2, 2026

Copy link
Copy Markdown

Claude Code Review

Status: Approved

This PR cleanly refactors workflow exports to a dedicated subpath. The implementation is solid with proper package.json exports configuration and consistent import updates across all files.

Code Quality

  • File rename (workflow.ts → workflows.ts) is appropriate
  • Export configuration in package.json correctly maps to dist files
  • All imports updated consistently (docs, examples, tests)
  • Type re-exports consolidated in workflows.ts
  • Changeset properly documents the breaking change

Minor Note

The workflow-types.ts file comment explains the separation to avoid circular dependencies - this is good architectural documentation.

No issues found.

@pkg-pr-new

pkg-pr-new Bot commented Feb 2, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/cloudflare/agents@827

commit: bb5ab1f

@threepointone
threepointone merged commit e20da53 into main Feb 2, 2026
6 checks passed
@threepointone
threepointone deleted the workflow-exports branch February 2, 2026 11:43
@github-actions github-actions Bot mentioned this pull request Feb 1, 2026
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.

2 participants