Remove unused getWritable stub function#855
Conversation
🦋 Changeset detectedLatest commit: 009ca28 The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
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 |
🧪 E2E Test Results❌ Some tests failed Summary
❌ Failed Tests🌍 Community Worlds (161 failed)mongodb (40 failed):
redis (40 failed):
starter (41 failed):
turso (40 failed):
Details by Category✅ ▲ Vercel Production
✅ 💻 Local Development
✅ 📦 Local Production
✅ 🐘 Local Postgres
✅ 🪟 Windows
❌ 🌍 Community Worlds
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📊 Benchmark Results
workflow with no steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Nitro | Express | Next.js (Turbopack) workflow with 1 step💻 Local Development
▲ Production (Vercel)
🔍 Observability: Express | Next.js (Turbopack) | Nitro workflow with 10 sequential steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Next.js (Turbopack) | Express | Nitro Promise.all with 10 concurrent steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Nitro | Express | Next.js (Turbopack) Promise.all with 25 concurrent steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Next.js (Turbopack) | Express | Nitro Promise.race with 10 concurrent steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Nitro | Next.js (Turbopack) | Express Promise.race with 25 concurrent steps💻 Local Development
▲ Production (Vercel)
🔍 Observability: Express | Nitro | Next.js (Turbopack) Stream Benchmarks (includes TTFB metrics)workflow with stream💻 Local Development
▲ Production (Vercel)
🔍 Observability: Next.js (Turbopack) | Nitro | Express SummaryFastest Framework by WorldWinner determined by most benchmark wins
Fastest World by FrameworkWinner determined by most benchmark wins
Column Definitions
Worlds:
|
There was a problem hiding this comment.
Pull request overview
This PR removes an unused stub implementation of getWritable that was left over from when there were separate default vs. "step" context export conditions. The cleanup consolidates the codebase by moving the interface definition to a more appropriate location.
Changes:
- Removed the standalone
writable-stream.tsfile containing an unused stub implementation - Moved
WorkflowWritableStreamOptionsinterface tostep/writable-stream.tswhere it's used by the actual implementation - Updated import path in
workflow/writable-stream.tsto reference the interface from its new location
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/core/src/writable-stream.ts | Deleted unused stub file containing the getWritable stub function and interface definition |
| packages/core/src/step/writable-stream.ts | Added WorkflowWritableStreamOptions interface definition, previously imported from deleted file |
| packages/core/src/workflow/writable-stream.ts | Updated import to reference WorkflowWritableStreamOptions from step/writable-stream.ts |
| .changeset/quick-mammals-kneel.md | Added changeset documenting the removal of unused code |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…s-and-commands * origin/main: fix(@workflow/ai): support provider-executed tools (AI SDK v6) (#734) Publish "workflow" and "@workflow/core" package versions in sync (#870) Add SDK version to workflow run executionContext for observability (#868) Allow recreateRun to accept an optional deploymentId parameter (#869) Add support for top-level `using` declarations inside of step / workflow functions (#866) docs: URL in docs was missing the docs/ prefix, 404 errors (#852) Add "classes" object to `manifest.json` file (#864) Fix Nest workbench app build (#865) Ignore Astro on local dev tests for source map e2e tests (#863) Enable custom class serialization transformations for "client" mode (#860) Submit request bodies with CBOR encoding (#844) [world-vercel] Update queue to use VQS v3 API (#799) NestJS framework support (#840) Fix resolve hook theming and token fetching. (#856) docs: rename Control Flow Patterns to Common Patterns and add new content (#846) docs: revamp World documentation pages (#763) Remove unused `getWritable` stub function (#855) # Conflicts: # packages/core/package.json # pnpm-lock.yaml

Removed unused
getWritablestub function and relocated its interface definition. It was left over from a time where there were separate default vs. "step" context export conditions, which no longer exists.What changed?
writable-stream.tsfile which contained an unused stub implementation ofgetWritableWorkflowWritableStreamOptionsinterface from the deleted file tostep/writable-stream.tsworkflow/writable-stream.tsto reference the interface from its new locationHow to test?
Verify that all existing functionality related to writable streams continues to work as expected. No behavior changes should be observed since this is only removing an unused stub function.
Why make this change?
The stub function in
writable-stream.tswas redundant and not being used in the codebase. The actual implementations ofgetWritableexist in the step and workflow modules. This change simplifies the codebase by removing unused code while preserving the interface definition in a more appropriate location.