fix(triggers): skip READY TO PROCESS label for cards in non-eligible lists

[zbigniewsobiecki]

Summary

• Bug: When the READY TO PROCESS label was added to a card in an unrecognised list (IN PROGRESS, IN REVIEW, DONE, etc.) the handler fell through to a hardcoded 'splitting' default, firing an unintended splitting run on a card that was already in progress.
• Fix: Replace the else fallback with an early return null so the trigger is silently skipped for any list that isn't splitting / planning / todo.
• Log level: Downgraded from warn (misleading — it was advertising the wrong behaviour) to info.

Root cause

src/triggers/trello/label-added.ts lines 64–68 (before fix):

} else {
    // Default to splitting if list not recognized
    logger.warn('Card in unrecognized list, defaulting to splitting', { currentListId, lists });
    agentType = 'splitting';
}

Test plan

• Updated unit test returns null when card is in an unrecognized list (e.g. IN PROGRESS) — was previously asserting agentType === 'splitting', now asserts result === null
• All existing handle-path tests still pass (splitting / planning / todo / missing card ID)
• npm test — 3 259 tests, 189 files, all green
• npm run lint — no issues
• npm run typecheck — no issues

🤖 Generated with Claude Code