Skip to content

refactor(workflow): move AgentNode events out of graphon - #39845

Open
laipz8200 wants to merge 1 commit into
mainfrom
laipz8200/refactor-agent-node-events
Open

refactor(workflow): move AgentNode events out of graphon#39845
laipz8200 wants to merge 1 commit into
mainfrom
laipz8200/refactor-agent-node-events

Conversation

@laipz8200

Copy link
Copy Markdown
Member

Summary

  • Move AgentNode-specific node and graph events into core.workflow.nodes.agent.
  • Let AgentNode convert its local log event while delegating generic events to graphon.
  • Update workflow streaming consumers to use the local graph event.

Fixes #39844

Screenshots

Not applicable; backend-only refactor.

@github-actions

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 56.86% 56.86% +0.00%
Strict coverage 56.38% 56.38% +0.00%
Typed symbols 37,167 37,168 +1
Untyped symbols 28,442 28,442 0
Modules 3092 3094 +2

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.18%. Comparing base (c562f78) to head (9828fa7).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #39845   +/-   ##
=======================================
  Coverage   86.18%   86.18%           
=======================================
  Files        5086     5087    +1     
  Lines      287694   287732   +38     
  Branches    57356    57356           
=======================================
+ Hits       247950   247990   +40     
+ Misses      34941    34939    -2     
  Partials     4803     4803           
Flag Coverage Δ
api 86.37% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@laipz8200
laipz8200 marked this pull request as ready for review August 3, 2026 01:29
@laipz8200
laipz8200 requested a review from QuantumGhost as a code owner August 3, 2026 01:29
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. refactor labels Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move AgentNode-specific events out of graphon

1 participant