feat: Add conversation variable persistence layer - #17

Open

tomerqodo wants to merge 21 commits into

sentry_only-issues-20260113-augment-codex-sentry_base_feat_add_conversation_variable_persistence_layer__pr167from

sentry_only-issues-20260113-augment-codex-sentry_head_feat_add_conversation_variable_persistence_layer__pr167

Commits on Jan 5, 2026

Added constructor-based DI for VariableAssigner v2 and wired the node factory to pass the ConversationVariableUpdater factory (the only non-VariablePool dependency), plus a unit test to verify the …
laipz8200
committed
335bf83
Updated to avoid the dict path and let the type checker enforce constructor args.
laipz8200
committed
0fd225a
A new command for updating variables. (vibe-kanban 00377ffe)
laipz8200
committed
67007f6
Add a new persistent storage for handling Conversation Variables. (vibe-kanban 0941477f)
laipz8200
committed
43fefe8
- fix(lint): replace `outputs.keys()` iteration in `api/core/app/layers/conversation_variable_persist_layer.py` to satisfy SIM118
laipz8200
committed
89e644f
- feat(graph-engine): enforce variable update value types with SegmentType validation and casting
laipz8200
committed
3edd525
- refactor(variable-assigner-v1): inline updated variable payload and drop common_helpers usage
laipz8200
committed
638b0ef
- refactor(graph-engine): switch VariableUpdate.value to VariableUnion and remove value type validation
laipz8200
committed
5ebc87a
- refactor(variable-assigner): restore common_helpers updated-variable handling for v1/v2 process_data
laipz8200
committed
89d292e
- refactor(variable-assigner): drop updated outputs now that persistence reads from process_data
laipz8200
committed
8505033
- chore(lint): run `make lint` (fails: dotenv-linter module missing after venv changes)
laipz8200
committed
36b4a6e
Revert "- refactor(graph-engine): switch VariableUpdate.value to VariableUnion and remove value type validation"
laipz8200
committed
99509eb
Revert "- feat(graph-engine): enforce variable update value types with SegmentType validation and casting"
laipz8200
committed
2b044dd
Revert "A new command for updating variables. (vibe-kanban 00377ffe)"
laipz8200
committed
2831694
- refactor(services): move ConversationVariableUpdaterImpl and factory out of core.workflow into `api/services/conversation_variable_updater.py`
laipz8200
committed
b083fa2
- chore(lint): run `make lint` (fails: import linter error and dotenv-linter module missing)
laipz8200
committed
deeebb8
- chore(import-linter): remove obsolete ignore for variable_assigner impl import in `api/.importlinter`
laipz8200
committed
1192648
fix(core): guard conversation variable flush when no updates
laipz8200
committed
7c30854

Commits on Jan 6, 2026

Commits on Jan 13, 2026

Apply changes for benchmark PR
tomerqodo
committed
84196e6

Navigation Menu

Uh oh!

feat: Add conversation variable persistence layer - #17

Commits on Jan 5, 2026

Added constructor-based DI for VariableAssigner v2 and wired the node factory to pass the ConversationVariableUpdater factory (the only non-VariablePool dependency), plus a unit test to verify the …

Updated to avoid the dict path and let the type checker enforce constructor args.

A new command for updating variables. (vibe-kanban 00377ffe)

Add a new persistent storage for handling Conversation Variables. (vibe-kanban 0941477f)

- fix(lint): replace `outputs.keys()` iteration in `api/core/app/layers/conversation_variable_persist_layer.py` to satisfy SIM118

- feat(graph-engine): enforce variable update value types with SegmentType validation and casting

- refactor(variable-assigner-v1): inline updated variable payload and drop common_helpers usage

- refactor(graph-engine): switch VariableUpdate.value to VariableUnion and remove value type validation

- refactor(variable-assigner): restore common_helpers updated-variable handling for v1/v2 process_data

- refactor(variable-assigner): drop updated outputs now that persistence reads from process_data

- chore(lint): run `make lint` (fails: dotenv-linter module missing after venv changes)

Revert "- refactor(graph-engine): switch VariableUpdate.value to VariableUnion and remove value type validation"

Revert "- feat(graph-engine): enforce variable update value types with SegmentType validation and casting"

Revert "A new command for updating variables. (vibe-kanban 00377ffe)"

- refactor(services): move ConversationVariableUpdaterImpl and factory out of core.workflow into `api/services/conversation_variable_updater.py`

- chore(lint): run `make lint` (fails: import linter error and dotenv-linter module missing)

- chore(import-linter): remove obsolete ignore for variable_assigner impl import in `api/.importlinter`

fix(core): guard conversation variable flush when no updates

Commits on Jan 6, 2026

fix(core): use system variable conversation_id and add custom updater error

fix(services): use Exception for conversation variable not found

Commits on Jan 13, 2026

Apply changes for benchmark PR