From ef1222ff314028883d2ec5162e3b7cf2dd2b0f8a Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Wed, 26 Nov 2025 02:18:57 +0100 Subject: [PATCH] chore: drop refactor note --- docs/refactor/2025-11-26-auto-reply-split.md | 12 ------------ 1 file changed, 12 deletions(-) delete mode 100644 docs/refactor/2025-11-26-auto-reply-split.md diff --git a/docs/refactor/2025-11-26-auto-reply-split.md b/docs/refactor/2025-11-26-auto-reply-split.md deleted file mode 100644 index 9d10af786..000000000 --- a/docs/refactor/2025-11-26-auto-reply-split.md +++ /dev/null @@ -1,12 +0,0 @@ -# Auto-reply refactor notes (2025-11-26) - -- Split `src/auto-reply/reply.ts` into smaller helpers: - - Command handling lives in `src/auto-reply/command-reply.ts`. - - Audio transcription helpers live in `src/auto-reply/transcription.ts`. - - Shared reply types live in `src/auto-reply/types.ts` (re-exported from `reply.ts`). -- `runCommandReply` now returns `{ payload, meta }`, supports injected enqueue runners for tests, logs structured metadata, and respects `mediaMaxMb` for local media paths. -- Timeout messaging now includes `cwd` when provided to speed up debugging slow commands. -- Added focused tests: - - `src/auto-reply/command-reply.test.ts` exercises Claude flag injection, session args, timeout messaging, media token handling, and Claude metadata reporting. - - `src/auto-reply/transcription.test.ts` covers media download + transcription command invocation. -- Existing public surface (`getReplyFromConfig`, `autoReplyIfConfigured`, `ReplyPayload`) remains unchanged; integration tests still pass (`pnpm test`).