Skip to content

Conversation

@ytallo
Copy link
Contributor

@ytallo ytallo commented Nov 26, 2025

Summary

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactor
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING.md
  • My code follows the code style of this project
  • I have added tests where applicable
  • I have tested my changes locally
  • I have linked relevant issues
  • I have added screenshots for UI changes (if applicable)

Screenshots (if applicable)

Additional Context


Note

Caches stream instances, normalizes Buffer-like results, and updates RPC processors to support sync handlers with safer request handling.

  • Core Execution (call-step-file.ts):
    • Cache StreamFactory instances via WeakMap and reuse with getStreamInstance.
    • Build streams from Object.entries and pass to runner payload.
    • Normalize result.body when received as Buffer-like JSON (convert to Buffer).
    • Improve stdout handling: parse likely-JSON logs; otherwise log as text.
  • RPC Processing:
    • Allow RpcHandler to return sync or async; handle now consistently returns a Promise.
    • RpcProcessor and RpcStdinProcessor:
      • Guard when closed; respond with error for unknown methods.
      • Handle both sync and Promise-like handler results without throwing.
      • Add exit/close/disconnect guards and line-based stdin parsing reliability.

Written by Cursor Bugbot for commit 15485da. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Nov 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
motia-docs Ready Ready Preview Comment Nov 27, 2025 6:21pm
motia-ui Ready Ready Preview Comment Nov 27, 2025 6:21pm

@github-actions
Copy link

github-actions bot commented Nov 26, 2025

E2E Test Results ✅

All tests have completed. Tests passed.

Matrix: 2 OS (Ubuntu, macOS) × 2 Templates (TypeScript, Python) = 4 jobs

View detailed test results


This comment is automatically posted by the PR E2E Results workflow.

@ytallo ytallo force-pushed the feature/performance-improvement branch from 4589e3d to 8661092 Compare November 27, 2025 13:14
@ytallo ytallo changed the title Feature/performance improvement fix: Performance Improvement Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants