Skip to content

fix: skip empty dictation and prepare v1.0.1 - #160

Merged
vriesd merged 3 commits into
mainfrom
fix/skip-empty-dictation
Sep 8, 2026
Merged

fix: skip empty dictation and prepare v1.0.1#160
vriesd merged 3 commits into
mainfrom
fix/skip-empty-dictation

Conversation

@vriesd

@vriesd vriesd commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Empty/silent dictation no longer injects or writes History. Session goes Transcribing → Idle instead of Injecting.
  • v1.0.1 metadata: changelog, metainfo, workspace version.
  • Maintainer cleanup already in this tree: retire host-qualification lab, split rec/ and status/, move whisper tests out of the engine file, drop duplicate PR packaging checks.

Test plan

  • cargo test -p echo-core session::
  • cargo test -p echo --lib rec::tests::
  • cargo test -p echo-desktop --test rec_once_stores
  • clippy -D warnings on echo-core and echo
  • required PR checks check / check and release / release-policy

Silent or blank transcripts no longer inject or appear in History.
Retire the host-qualification lab, split rec and status along owners,
and drop duplicate PR packaging checks.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-08T07:19:51.431564Z 9f29baf PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@vriesd
vriesd merged commit f566191 into main Sep 8, 2026
13 checks passed
@vriesd
vriesd deleted the fix/skip-empty-dictation branch September 8, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants