molecule-ai/molecule-core
39
0
Fork 2
Code Issues 38 Pull Requests 21 Actions 8 Packages Projects Releases Wiki Activity

fix(delegations): ListDelegations queries delegations table before activity_logs fallback #250

Closed
claude-ceo-assistant wants to merge 1 commits from fix/delegations-list-ledger-fallback into main
pull from: fix/delegations-list-ledger-fallback
merge into: molecule-ai:main
Conversation 11 Commits 1 Files Changed 2 +451 -28

1 Commits

Author SHA1 Message Date
Molecule AI Infra-Runtime-BE
bbc40cb872 fix(delegations): ListDelegations falls back to delegations table before activity_logs
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 13s
Details
sop-tier-check / tier-check (pull_request) Failing after 2s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details 
RFC #2829 PR-1/4: GET /workspaces/:id/delegations previously queried only
activity_logs, returning [] for active/completed delegations while the agent's
check_delegation_status showed them correctly. The new delegations table
(migration 049) now holds durable state for active delegations.

The handler now tries the ledger first (delegations table), falls back to
activity_logs for pre-migration data, and returns [] only when both are empty.
This closes the mismatch where:
  - GET /delegations → []
  - check_delegation_status(task_id) → active/completed

6 new tests:
  TestListDelegations_LedgerRowsReturned
  TestListDelegations_LedgerEmptyFallsBackToActivityLogs
  TestListDelegations_BothEmptyReturnsEmptyArray
  TestListDelegations_LedgerQueryErrorFallsBackToActivityLogs
  TestListDelegations_LedgerCompletedIncludesResultPreview
  TestListDelegations_LedgerFailedIncludesErrorDetail

Updated existing tests TestListDelegations_Empty and
TestListDelegations_WithResults to use the ledger-first flow.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-10 06:52:06 +00:00