agentjido / jido / 8fd7e87f8151bfb4229faa178f5dd452e1d7d214

66%

Build Type

push

github

Committed by web-flow

Commit Message

Extract Jido Signal into it's own package (#31)

* feat: Implement custom signal types

* feat: reconcile and update signal serialization

* refactor: improve signal dispatch

* feat: Signal middleware

* fix: handle empty signal list in append_signals function

Updated the append_signals function to return an {:ok, state, []} tuple when the input signal list is empty, preventing errors related to UUID generation for empty lists. Additionally, modified the publish function to return an {:error, :empty_signal_list} when an empty list is provided.

Added comprehensive end-to-end tests to ensure proper functionality across multiple buses, including signal partitioning, middleware interactions, and persistent subscriptions.

Refactored existing tests to improve clarity and maintainability.

* refactor: Improve code formatting and structure in signal modules

This commit enhances the readability and maintainability of the signal-related modules by applying consistent formatting and restructuring code for clarity. Key changes include:
- Improved formatting of function calls and case statements in `Jido.Signal.Bus`, `Jido.Signal.Dispatch`, and `Jido.Signal.Router` modules.
- Adjusted comments and documentation for better understanding.
- Updated tests to reflect changes in the router's target handling, allowing any term as a valid target.

These modifications aim to promote cleaner code practices and facilitate easier future development.

* chore: extract signal files to new package

* chore: update jido_signal dependency reference in mix.exs and mark test as flaky

This commit modifies the `mix.exs` file to include a commented-out path reference for the `jido_signal` dependency, allowing for easier local development. Additionally, a test in `server_runtime_test.exs` is tagged as flaky to indicate potential instability in its execution.

* Signal as Deps

* refactor: remove signal system references and documentation

This commit removes the signal system fro... (continued)

Run Details

0 of 2 new or added lines in 2 files covered. (0.0%)

13 existing lines in 3 files now uncovered.

1387 of 1794 relevant lines covered (77.31%)

44696.88 hits per line