ci(changesets): version packages#1101
Conversation
Deploying voltagent with
|
| Latest commit: |
08ca546
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fdeefb5d.voltagent.pages.dev |
| Branch Preview URL: | https://changeset-release-main.voltagent.pages.dev |
This comment has been minimized.
This comment has been minimized.
📝 WalkthroughWalkthroughRemoved a changeset file, bumped packages/core to 2.6.0 and added its changelog entries, and updated Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
examples/with-nextjs-resumable-stream/package.json (1)
60-64:⚠️ Potential issue | 🟡 MinorPre-existing
repository.directorymismatch.The
directoryfield points toexamples/with-nextjs-ai-elements, but this package lives underexamples/with-nextjs-resumable-stream. This can mislead tooling that uses the field (e.g., npm provenance, monorepo link generators).🔧 Proposed fix
- "directory": "examples/with-nextjs-ai-elements" + "directory": "examples/with-nextjs-resumable-stream"🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@examples/with-nextjs-resumable-stream/package.json` around lines 60 - 64, The repository.directory field in package.json currently points to "examples/with-nextjs-ai-elements" but this package is under "examples/with-nextjs-resumable-stream"; update the "repository.directory" value to "examples/with-nextjs-resumable-stream" so the repository metadata matches the package location and tooling can resolve provenance correctly (edit the "repository.directory" property in package.json).
🧹 Nitpick comments (1)
examples/with-zapier-mcp/package.json (1)
7-7: Consider aligning the version range operator with the rest of the monorepo.This example uses
~2.6.0(patch-level range) while all other examples use^2.6.0(minor-level range). This is a pre-existing inconsistency, but since the line is already being touched, it's a good opportunity to align it.♻️ Proposed fix
- "@voltagent/core": "~2.6.0", + "@voltagent/core": "^2.6.0",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@examples/with-zapier-mcp/package.json` at line 7, Update the dependency version operator for "@voltagent/core" in the package.json so it matches the monorepo convention: replace the patch-range operator "~2.6.0" with the caret minor-range operator "^2.6.0" (i.e., change the value for the "@voltagent/core" entry).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@examples/with-nextjs-resumable-stream/package.json`:
- Around line 60-64: The repository.directory field in package.json currently
points to "examples/with-nextjs-ai-elements" but this package is under
"examples/with-nextjs-resumable-stream"; update the "repository.directory" value
to "examples/with-nextjs-resumable-stream" so the repository metadata matches
the package location and tooling can resolve provenance correctly (edit the
"repository.directory" property in package.json).
---
Nitpick comments:
In `@examples/with-zapier-mcp/package.json`:
- Line 7: Update the dependency version operator for "@voltagent/core" in the
package.json so it matches the monorepo convention: replace the patch-range
operator "~2.6.0" with the caret minor-range operator "^2.6.0" (i.e., change the
value for the "@voltagent/core" entry).
b34fc6a to
08ca546
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@packages/core/CHANGELOG.md`:
- Around line 78-88: The duplicate "### What's New" heading in the CHANGELOG
entry should be renamed to avoid markdownlint MD024; locate the second heading
that precedes the list of new step context primitives (the block describing
bail(result?), abort(), getStepResult(stepId), getInitData()) and change the
heading text to something unique like "### New Primitives" or "### Execution
Primitives" so the markdown heading is no longer duplicated.
| ### What's New | ||
|
|
||
| Step execution context now includes four new primitives: | ||
| - `bail(result?)`: complete the workflow early with a custom final result | ||
| - `abort()`: cancel the workflow immediately | ||
| - `getStepResult(stepId)`: get a prior step output directly (returns `null` if not available) | ||
| - `getInitData()`: get the original workflow input (stable across resume paths) | ||
|
|
||
| These primitives are available in all step contexts, including nested step flows. | ||
|
|
||
| ### Example: Early Complete with `bail` |
There was a problem hiding this comment.
Avoid duplicate “What’s New” heading to satisfy markdownlint (MD024).
Another “### What’s New” appears earlier in this release entry; markdownlint flags duplicate headings. Consider renaming this one (e.g., “### New Primitives” or “### Execution Primitives”).
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 78-78: Multiple headings with the same content
(MD024, no-duplicate-heading)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@packages/core/CHANGELOG.md` around lines 78 - 88, The duplicate "### What's
New" heading in the CHANGELOG entry should be renamed to avoid markdownlint
MD024; locate the second heading that precedes the list of new step context
primitives (the block describing bail(result?), abort(), getStepResult(stepId),
getInitData()) and change the heading text to something unique like "### New
Primitives" or "### Execution Primitives" so the markdown heading is no longer
duplicated.
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@voltagent/core@2.6.0
Minor Changes
#1100
314ed40Thanks @omeraplak! - feat: add workflow observer/watch APIs for stream resultsWhat's New
WorkflowStreamResult:watch(cb)watchAsync(cb)observeStream()streamLegacy()workflow.stream(...)workflow.timeTravelStream(...).resume(...)observeStream()close semantics.SDK Example
Time Travel Stream Example
#1102
7f923d2Thanks @omeraplak! - feat: add workflow execution primitives (bail,abort,getStepResult,getInitData)What's New
Step execution context now includes four new primitives:
bail(result?): complete the workflow early with a custom final resultabort(): cancel the workflow immediatelygetStepResult(stepId): get a prior step output directly (returnsnullif not available)getInitData(): get the original workflow input (stable across resume paths)These primitives are available in all step contexts, including nested step flows.
Example: Early Complete with
bailExample: Cancel with
abortExample: Use
getStepResult+getInitDataSummary by CodeRabbit
Release Notes
New Features
Chores