fix: resolve merge conflicts and fix threading tests

- Update MessageToolOptions type to include Slack threading options
- Remove duplicate threadTs property in slack/actions.ts
- Fix replyThreadTs parameter name in monitor.ts
- Update test to correctly verify 'first' mode threading behavior:
  - 'off' mode: no threading unless already in a thread
  - 'first' mode: first reply starts a thread
- Add new test case for 'first' mode threading
This commit is contained in:
Austin Mudd
2026-01-09 09:17:47 -08:00
committed by Peter Steinberger
parent b4663ed11c
commit 909c14d443
4 changed files with 56 additions and 4 deletions

View File

@@ -1110,7 +1110,7 @@ export async function monitorSlackProvider(opts: MonitorSlackOpts = {}) {
accountId: account.accountId,
runtime,
textLimit,
threadTs: effectiveThreadTs,
replyThreadTs: effectiveThreadTs,
});
hasRepliedRef.value = true;
},