feat(dictation): add configurable cleanup and formatting by donjor-agent[bot] · Pull Request #1685 · RSI-Software/t3code-hyprws

donjor-agent · 2026-10-09T10:39:40Z

🤖 codex/cliproxy gpt-6.1-sol (high)

Problem

Dictation needs optional cleanup and formatting while keeping the microphone and transcription settings available.

Cleanup cancellation: raw transcript retained

rec-3a66.e1.mp4

Recorded build: 0622d6b6af9; synthetic speech, held generation, no cuts.

Automatic mode: cleanup, formatting, Undo

Completed transcription triggers cleanup and formatting; Undo restores the draft.

voice-auto-proof.mp4

Recorded build: 58dc668e950b.

Cut	Removed
2–6s	Model waiting
14–23.5s	Idle result

Recording settings restored:

Before	After

Change

Model: independent provider/model
Prompts: editable per project
Context: editable per project
Manual: typed or dictated drafts; Undo
Automatic: after transcription
Threshold: formatting at 50 words

Scope and approval

Maintainer-requested in this T3 thread: optional cleanup/formatting, independent model, editable project prompts/context, optional message history, and automatic processing after transcription only.

Verification

Tests: 68 cases; cancel, history, Undo
Types/builds: web and server
Static: lint, formatting, fork CI
Draft: cleanup, formatting, Undo
Settings: save, generate, cancel
CI/aftercare: green on 3ecf0a2

Fixtures: synthetic transcription with real generation; mocked context and desktop speaker bridge

Not exercised: physical speech service, native mobile, remote/relay

Fork trailers

GPT-6.1-Sol, Codex through T3 Code

Fork-Domain: voice-input
Fork-Tier: core

rsi-software-agent · 2026-10-09T10:39:49Z

🩺 Aftercare: ✅ clear on `3ecf0a2`

A1 bug Automatic cleanup cancellation discards the raw transcript
_{Fixed: in 500d6a9; The active toolbar and Escape now cancel processing rather than dictation. Source-loaded hook/session/controller checks confirmed raw speech is committed and formatting skipped; integrated regression…}
A2 bug History truncation can remove a message's role label
_{Fixed: in 500d6a9; History budgets labeled entries and truncates only message bodies. The previous over-budget reproduction now retains complete role headers within 12,000 characters; regression coverage was added.}
A3 ux Dictation remains excluded from browser Settings search
_{Fixed: in 500d6a9; The desktopOnly restriction is removed and processing keywords are included. Browser search-function checks passed all five queries; browser regression tests were added.}

_{Reviewed 9 Oct 21:03 UTC in 4 min; advisory, never blocks merge.}

Details

A1 Automatic cleanup cancellation discards the raw transcript apps/web/src/voice-input/useVoiceInput.fork.tsx:219
Automatic cleanup runs inside the transcribing phase, hiding and inerting its processing-only Cancel control. The visible Cancel aborts dictation, so the already-produced raw transcript is never committed. Fix: Expose processing status and processing-only cancellation in the active toolbar. Add an integrated test proving Cancel during cleanup commits the raw transcript and skips formatting.

A2 History truncation can remove a message's role label apps/server/src/voice-input/voiceTextPrompts.fork.ts:31
The final 12,000-character slice can begin inside a message body, stripping its role label and boundary. Four 4,000-character messages reproduce unlabeled leading history. Fix: Apply the budget to labeled entries, truncating message bodies without removing role headers or boundaries. Add an over-budget history test.

A3 Dictation remains excluded from browser Settings search apps/web/src/components/settings/voiceInputSearch.fork.ts:7
The restored browser Dictation page still has a desktopOnly search item, so Settings search filters it out. Its keywords also omit cleanup, formatting, and project context. Fix: Remove the stale desktopOnly restriction, add the new processing keywords, and test that browser Settings search returns Dictation.

Head	Result	Time	Check
`3ecf0a2`	✅ review passed: fixed: none; held: none; new: none; withdrawn: none; no item resolved; files (8): apps/server/src/project/ProjectSetupScriptRunner.settlement.fork.test.ts, …and 7 more	4 min	run
`dfaac27`	✅ review passed: fixed: none; held: none; new: none; withdrawn: none; no item resolved; files (30): AGENTS.md, README.md, apps/desktop/src/app/DesktopApp.ts, …and 27 more	4 min	run
`500d6a9`	✅ review passed: fixed: A1, A2, A3; held: none; new: none; withdrawn: none; files (51): AGENTS.md, apps/server/src/provider/Drivers/OpenCodeDriver.ts, …and 49 more	7 min	run
`6e41254`	⚠️ 3 open: fixed: none; held: none; new: A1, A2, A3; withdrawn: none; no item resolved; files (30): apps/server/src/provider/Drivers/CodexManagedProvider.ts, …and 29 more	8 min	run