CedarCopilot/cedar-mail

SKILL.mdfix(review): address 6 findings from thermo-review of this session's commits - .cursor/skills/account-config/SKILL.md: fix dangling references to the deleted resources/ directory (a third skill mirror the consolidation commit missed) -- repoint to the surviving .claude/skills files. - load-skill: extract buildLoadSkillDescription() so the regression test measures the actual wire payload (fixed prefix + terse catalog), not just the inner catalog -- the real headroom before the 2KB limit was tighter than the previous test tracked. - conversationTool.ts: wrap selectedAopNames/visibleColumns/extraFields in coerceJsonString too, for consistency with the other 5 fields already wrapped by the same fix. - content-shortcuts.ts: extractLineRange now rejects a reversed range (end < start) with a clear message instead of silently returning an empty slice; added a test. - chat-message-persistence.ts: recordExternalChatMessage now verifies an existing thread's userId matches the caller before attaching a message -- a reused/colliding host sessionId could otherwise mix one user's turns into another user's thread once this dormant code is wired up. - readDocumentTool.ts: the #lines shortcut's wordCount no longer counts each line-number prefix as an extra word. Co-Authored-By: Claude Sonnet 5 <<email>>Jul 10, 2026, 10:42 PM