Run workflows through managed hook sessions - #123
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #123 +/- ##
=======================================
Coverage 98.29% 98.30%
=======================================
Files 52 52
Lines 3929 3947 +18
=======================================
+ Hits 3862 3880 +18
Misses 67 67 ☔ View full report in Codecov by Harness. |
There was a problem hiding this comment.
No issues found across 5 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
You’re at about 92% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Auto-approved: Adds Workflow.session and WorkflowSession.run to wrap workflow runs in a managed HookSession, preserves repeated capture values with a new values list, and adds program to WorkflowResult; includes tests and docs.
Re-trigger cubic
Summary
Workflow.session(model)andWorkflowSession.run(data)so imperative capture, replacement, and stopping can wrap a complete workflow runWorkflowPlanwith the sessionHookPrograminWorkflowResultWorkflowcomposition and move HookSession concepts into a dedicated executable tutorial notebookValidation
just checksjust tests(614 passed, 1 skipped)just docs(passed with the existing AutoAPI cyclic-import warning)Closes #114