agentjido / jido_action

82%

main: 80%

LAST BUILD ON BRANCH refactor/exec

branch: refactor/exec

CHANGE BRANCH

x

Reset

Sync Branches

• refactor/exec
• add-codebase-guide
• chore/dialyzer-errors
• dependabot/hex/credo-1.7.13
• dependabot/hex/credo-1.7.14
• dependabot/hex/dialyxir-1.4.6
• dependabot/hex/dialyxir-1.4.7
• dependabot/hex/ex_doc-0.38.3
• dependabot/hex/ex_doc-0.38.4
• dependabot/hex/ex_doc-0.39.1
• dependabot/hex/ex_doc-0.39.2
• dependabot/hex/lua-0.4.0
• dependabot/hex/mimic-2.0.2
• dependabot/hex/mimic-2.1.0
• dependabot/hex/mimic-2.1.1
• dependabot/hex/mimic-2.2.0
• dependabot/hex/mix_test_watch-1.4.0
• dependabot/hex/quokka-2.11.0
• dependabot/hex/req-0.5.11
• dependabot/hex/req-0.5.12
• dependabot/hex/req-0.5.16
• dependabot/hex/uniq-0.6.2
• dependabot/hex/zoi-0.11.0
• dependabot/hex/zoi-0.12.0
• feat/jido-plan
• feat/lua-eval-tool
• feature/weather-action
• feature/zoi
• fix-ci-pipeline-issues
• fix/dialyzer-warnings
• main
• refactor/integrate-splode
• refactor/jido-exec
• refactor/jido-exec-2
• refactor/remove-exdbug

Build # ab44ccefc931f99edd409b378e8610b752de0f2b-PR-31

Build Type

Pull #31

github

Committed by mikehostetler

Commit Message

refactor(exec): streamline action execution and error handling

- Simplified the `on_after_run` lifecycle hook to return the result directly.
- Refactored `execute_action_with_timeout` to utilize `Task.Supervisor.async_nolink` for better task management and error handling.
- Enhanced timeout handling with clearer error messages and streamlined cleanup processes.
- Removed the `cleanup_task_group` function as task cleanup is now managed by `Task.Supervisor`.
- Added tests to ensure proper handling of action return shapes and timeout scenarios, including edge cases for atom safety.
- Introduced new tests for `on_after_run` behavior and strict return shape validation.

Pull Request Pull Request #31: refactor(exec): streamline action execution and error handling

Run Details

117 of 160 new or added lines in 12 files covered. (73.13%)

5 existing lines in 3 files now uncovered.

719 of 881 relevant lines covered (81.61%)

107.7 hits per line