Snap & Talk: a retake that hears nothing keeps the earlier words, the toolbar's Cancel asks first, and the notice never says 'Use Off'

オープン
#342 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
18/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
swift
領域
desktop

調査の方向性

Start in ReadbackModel.swift: the stop path around lines 1036-1054 and process() around 1343-1364, plus the cancel path near 1063-1069 and shortcutLabel near 516. The issue depends on the unmerged #330 branch, which supplies the fake recorder for --check-readback, so confirm that branch is merged first. Done means each listed --check-readback case passes and the native microphone checks are confirmed by hand.

索引モデルが issue の本文から書いたものです。

説明

bug

Outcome: a silent retake never hides a section's words. The toolbar never discards a long take without the page's question. The narration notice names a key only when that key works.

1. A retake that hears no words keeps the earlier words
  • Problem:
    • Stop's quiet screen catches only takes under 0.35 s or −55 dB (ReadbackModel.swift:1036-1039).
    • Every other take first moves the section's audio, original text and edited text into history/ (:1044), unlinks both transcripts (:1051) and drops the draft (:1054).
    • When transcription returns nothing, process() throws (:1343) and saves .failed with no text (:1362-1364).
    • The page then shows "Transcription stopped" with only Retry transcription (ReadbackView.swift:462-465). Retry re-runs the same silent audio (ReadbackModel.swift:1118-1124). Re-record is hidden in ⋯ (ReadbackView.swift:419), and the summary says "1 needs a retry" (:240-241).
    • Nothing reads history/, so the notes or narration look gone.
    • Meetings already decided that a recording which heard no speech is never offered for retry (docs/workbench.md:207).
  • Change:
    • Stop: archive only the previous audio and original. Keep narration.txt linked, and keep the draft.
    • process() with words: archive the still-linked edited text, then write the new files.
    • process() with no words:
      • archive the silent WAV and clear audio;
      • set .ready with no failure when text is still linked; otherwise set .needsNarration with "No speech was heard. Record the narration again.";
      • set the notice to "No speech heard. The screenshot and any earlier narration were kept."
    • Real engine errors: keep .failed and Retry, as today.
    • One sentence in docs/snap.md.
  • Check: --check-readback with ControlledTranscriber and #330's fake recorder:
    • imported notes, empty result: .ready, notes intact, silent WAV in history/, no "retry" anywhere;
    • narrated section, empty result: .ready, edited text kept;
    • fresh capture, empty result: .ready;
    • nothing linked, empty result: .needsNarration;
    • engine error: .failed, earlier text still on disk;
    • words: new text, earlier text archived.
    • Removing the no-words branch must fail the first four.
2. The toolbar's Cancel narration asks as the page does
  • Problem: while narrating, the chooser lists Stop narration and Cancel narration side by side (ToolbarActivityActions.swift:60-64).
    • Cancel calls cancelNarration() directly, which deletes the take (ReadbackModel.swift:1063-1069). A take can run 300 s (:1023).
    • The page asks "Discard this narration?" after 10 s (ReadbackView.swift:255-257, :128-131; docs/desktop.md:119).
    • docs/workbench.md:118: every door gives the same result.
  • Change:
    • One rule in ReadbackModel shared by both doors:
      • cancelAsksFirst(elapsed:);
      • a published pendingCancelNarration holding the take's identity;
      • requestCancelNarration();
      • confirmCancelNarration(_:), which re-checks the identity.
    • The question clears whenever recording ends, including the 300 s auto-stop.
    • The page's existing dialog binds to it.
    • The chooser calls requestCancelNarration() and, when it asks, opens Snap & Talk with the page's dialog showing.
    • No NSAlert: it would block the narration timer.
  • Check: --check-readback with a check-only take fixture:
    • at 74 s it asks and keeps the file;
    • Keep recording keeps the take running;
    • Discard with the matching identity removes the file; a stale identity does nothing;
    • at 5 s it discards at once;
    • Stop and the auto-stop clear the question.
    • The existing ToolbarActivityChooserTests pass.
3. The narration notice names a key only when it works
  • Problem: shortcutLabel (ReadbackModel.swift:516) returns "Off" when the shortcut is disabled (VoicePreferences.swift:26-27) and ignores shortcutFailure (:453).
    • The notice reads "Narrating section 2. Use Off again to stop." (:1016).
    • The help (ReadbackView.swift:276) and the options sheet (:580) say "Off captures the display…".
    • The toolbar already filters both cases (FloatingToolbar.swift:371-375), and so does Dictate (AppModel.swift:612). #15 Slice B fixes Home and Dictate.
  • Change:
    • usableShortcutLabel: nil when the shortcut is off or failed to register.
    • static narrationNotice(section:shortcut:): without a usable key it ends "Use Stop narration to finish."
    • The help names the key only when it is usable.
    • The options sheet shows the failure, or "No shortcut is set. Capture & narrate on the page still works."
  • Check: --check-readback:
    • the pure notice, with and without a key;
    • shortcut disabled in a scratch defaults suite: nil;
    • enabled: "⌥C";
    • with a registration failure: nil.
Native checks owed (microphone)
  • Re-record over a section with notes while the input is muted: the notes stay, and no Retry appears.
  • Narrate past 10 s, then choose Cancel narration in the chooser: Snap & Talk opens with "Discard this narration?". Keep recording continues the take.
  • With ⌥C off, choose Capture & narrate: the notice ends "Use Stop narration to finish."
Value before scope
  1. Outcome: no hidden words, no unasked loss of up to five minutes of narration, and no instruction to press "Off". Evidence: the traced paths.
  2. Simpler route: Stop's quiet-take rule, the page's own dialog, and the toolbar's existing shortcut filter.
  3. Owner: ReadbackModel.
  4. Complexity: no new control, label or setting.
  5. Verified by: --check-readback.

Grammar: Quality.

Sequencing: the #330 branch is ReadbackModel's only writer tonight. Build after it merges, reusing its fake recorder: item 1 first, then items 2-3.

Refs #134, Refs #15


Build-ready record from the overnight lead session of 15 Oct 2026 (round two), verified against main edc15df by an independent skeptic per item. Implementation follows on a claude/* branch; this issue closes only after native acceptance.


Build-ready record from the overnight lead session of 15 Oct 2026 (second discovery round), verified against main edc15df by an independent skeptic per item. This issue closes only after native acceptance.

主要言語
Swift
スター
0
フォーク
1
平均マージ
18時間 19分
マージ済み PR(30日)
114

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Ship-Work/workbench のほかの issue

Ship-Work/workbench の issue をすべて見る

似ている issue

Swift の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。