mirror of
https://github.com/coleam00/Archon
synced 2026-04-21 13:37:41 +00:00
a1a4496132
* feat: Add session state machine with immutable sessions for audit trail This PR implements an explicit session state machine that tracks why sessions are created/deactivated, enabling debugging of agent decision history. Changes: - Add TransitionTrigger type as single source of truth for session transitions - Add parent_session_id and transition_reason columns to sessions table - Add transitionSession(), getSessionHistory(), getSessionChain() functions - Update orchestrator to use transitionSession() for all session creation - Add trigger logging to all command-handler deactivation points - Add comprehensive unit tests for new functionality The session chain can be walked to understand the full history of a conversation, with each session recording why it was created (first-message, plan-to-execute, reset-requested, isolation-changed, etc.) * docs: Update documentation for session state machine * fix: Address PR review feedback - improve type safety and error handling - Add SessionNotFoundError and rowCount validation to updateSession, deactivateSession, and updateSessionMetadata - Strengthen createSession signature to accept TransitionTrigger instead of string for transition_reason - Replace TransitionTrigger arrays with TRIGGER_BEHAVIOR Record for compile-time exhaustiveness checking - Improve isBranchMerged/getLastCommitDate error handling - log unexpected errors while still returning false/null for cleanup safety - Fix ESLint template expression warnings by adding non-null assertions to getTriggerForCommand calls (values guaranteed to exist) - Add tests: SessionNotFoundError cases, getSessionChain with non-existent ID, transitionSession error propagation, null input handling * refactor: Simplify session code per code-simplifier review - Consolidate duplicate error assertion patterns in tests - Simplify shouldDeactivateSession logic: !== 'none' instead of checking both 'creates' and 'deactivates' explicitly
23 lines
1.1 KiB
SQL
23 lines
1.1 KiB
SQL
-- Migration: Add parent linkage and transition tracking for immutable session audit trail
|
|
-- Backward compatible: new columns are nullable
|
|
|
|
-- Link sessions in a chain (child points to parent)
|
|
ALTER TABLE remote_agent_sessions
|
|
ADD COLUMN IF NOT EXISTS parent_session_id UUID REFERENCES remote_agent_sessions(id);
|
|
|
|
-- Record why this session was created
|
|
ALTER TABLE remote_agent_sessions
|
|
ADD COLUMN IF NOT EXISTS transition_reason TEXT;
|
|
|
|
-- Index for walking session chains efficiently
|
|
CREATE INDEX IF NOT EXISTS idx_sessions_parent ON remote_agent_sessions(parent_session_id);
|
|
|
|
-- Index for finding session history by conversation (most recent first)
|
|
CREATE INDEX IF NOT EXISTS idx_sessions_conversation_started
|
|
ON remote_agent_sessions(conversation_id, started_at DESC);
|
|
|
|
-- Comment for documentation
|
|
COMMENT ON COLUMN remote_agent_sessions.parent_session_id IS
|
|
'Links to the previous session in this conversation (for audit trail)';
|
|
COMMENT ON COLUMN remote_agent_sessions.transition_reason IS
|
|
'Why this session was created: plan-to-execute, isolation-changed, reset-requested, etc.';
|