Add configurable state reference to ChatMessageTrigger#3531

Open

devin-ai-integration[bot] wants to merge 27 commits intomainfrom

devin/1767727073-chat-history-key-config

Commits on Jan 8, 2026

Add configurable chat_history_key to ChatMessageTrigger

devin-ai-integration[bot]
and
vincent0426
committed
Refactor ChatMessageTrigger to use StateValueReference instead of string

devin-ai-integration[bot]
and
vincent0426
committed
Fix codegen: use camelCase stateVariableId and skip inherited message attribute

devin-ai-integration[bot]
and
vincent0426
committed
Fix mypy error: use direct type narrowing for state variable

devin-ai-integration[bot]
and
vincent0426
committed
Fix prettier formatting in chat-message-trigger.ts

devin-ai-integration[bot]
and
vincent0426
committed
Rename _get_chat_history_key to get_state_key

devin-ai-integration[bot]
and
vincent0426
committed
Rename get_state_key to _get_state_key (private method)

devin-ai-integration[bot]
and
vincent0426
committed
Refactor: reuse WorkflowStateVariableWorkflowReference instead of duplicate type

devin-ai-integration[bot]
and
vincent0426
committed
Regenerate entire snapshot file

devin-ai-integration[bot]
and
vincent0426
committed
Update snapshot after rebase

devin-ai-integration[bot]
and
vincent0426
committed
Move custom state tests to e2e test under tests/workflows

devin-ai-integration[bot]
and
vincent0426
committed
Use string-based LazyReference instead of lambda

devin-ai-integration[bot]
and
vincent0426
committed
Move custom state serialization test to workflow_serialization directory

devin-ai-integration[bot]
and
vincent0426
committed
Update test to assert full trigger structure directly

devin-ai-integration[bot]
and
vincent0426
committed
Add assertion to verify state_variable_id matches state_variables

devin-ai-integration[bot]
and
vincent0426
committed
Use StateValueReference.resolve() for proper parent state walking

devin-ai-integration[bot]
and
vincent0426
committed
Update test to use _resolve_state() instead of removed _get_state_key()

devin-ai-integration[bot]
and
vincent0426
committed
Fix: return None when output is None regardless of state

devin-ai-integration[bot]
and
vincent0426
committed
Add validation error when state is not specified and add test

devin-ai-integration[bot]
and
vincent0426
committed
Update test to use full assertion format for triggers

devin-ai-integration[bot]
and
vincent0426
committed
Keep original output validation test, add separate state validation test

devin-ai-integration[bot]
and
vincent0426
committed
Fix test: update expected attribute type from JSON to ARRAY

devin-ai-integration[bot]
and
vincent0426
committed
Fix tests: use list format for message parameter and update assertions

devin-ai-integration[bot]
and
vincent0426
committed
Default to chat_history state when Config.state is not specified

devin-ai-integration[bot]
and
vincent0426
committed
Add validation error when state is None and no chat_history exists

devin-ai-integration[bot]
and
vincent0426
committed
Simplify error message to 'Chat Trigger state must be specified'

devin-ai-integration[bot]
and
vincent0426
committed
Allow state to be None for backward compatibility

devin-ai-integration[bot]
and
vincent0426
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add configurable state reference to ChatMessageTrigger#3531

Add configurable state reference to ChatMessageTrigger#3531
devin-ai-integration[bot] wants to merge 27 commits intomainfrom
devin/1767727073-chat-history-key-config

Commits on Jan 8, 2026

Add configurable chat_history_key to ChatMessageTrigger

Refactor ChatMessageTrigger to use StateValueReference instead of string

Fix codegen: use camelCase stateVariableId and skip inherited message attribute

Fix mypy error: use direct type narrowing for state variable

Fix prettier formatting in chat-message-trigger.ts

Rename _get_chat_history_key to get_state_key

Rename get_state_key to _get_state_key (private method)

Refactor: reuse WorkflowStateVariableWorkflowReference instead of duplicate type

Regenerate entire snapshot file

Update snapshot after rebase

Move custom state tests to e2e test under tests/workflows

Use string-based LazyReference instead of lambda

Move custom state serialization test to workflow_serialization directory

Update test to assert full trigger structure directly

Add assertion to verify state_variable_id matches state_variables

Use StateValueReference.resolve() for proper parent state walking

Update test to use _resolve_state() instead of removed _get_state_key()

Fix: return None when output is None regardless of state

Add validation error when state is not specified and add test

Update test to use full assertion format for triggers

Keep original output validation test, add separate state validation test

Fix test: update expected attribute type from JSON to ARRAY

Fix tests: use list format for message parameter and update assertions

Default to chat_history state when Config.state is not specified

Add validation error when state is None and no chat_history exists

Simplify error message to 'Chat Trigger state must be specified'

Allow state to be None for backward compatibility