Skip to content

Conversation

@mariano
Copy link
Collaborator

@mariano mariano commented Nov 6, 2025

Summary

  • restructure Flow parsing and bookkeeping to track incoming edges and consolidate Mermaid helpers
  • overhaul flow execution to compute reachable nodes, surface clearer errors, and detect cycles
  • drop __future__ annotations usage in Node and add regression coverage for cyclic flows

Testing

  • poetry run pytest --verbose -s
  • make lint

https://chatgpt.com/codex/tasks/task_e_690bf6be8db48323ae87f9374b6ca49c

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mariano mariano merged commit 6f8879c into main Nov 6, 2025
8 checks passed
@mariano mariano deleted the improve-flow-module branch November 6, 2025 13:49
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