Skip to content

fix(app): align cursor context contracts and CI fixtures #1268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
appergb merged 1 commit into from
Oct 25, 2026
Merged

fix(app): align cursor context contracts and CI fixtures #1268

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions docs/audio-startup-latency.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ PCM，而不是等待识别器后再开麦。缓冲按需增长、保留原有
capture，以及默认关闭的预热/预录选项；#1150 已在 2026-10-12 关闭。因此不能将
讨论里的早期流程直接当作当前实现。

仍有可移走的前置步骤：`PipelineDictationEngine::start` 在提交 provisional capture
或调用普通 recorder 之前等待 `transcription.prepare()`。改为：
这项前置等待已经移走：`PipelineDictationEngine::start` 会先提交 provisional
capture 或启动普通 recorder，再并行准备 `transcription.prepare()`。当前流程为：

```text
快捷键 / 正式启动
Expand Down Expand Up @@ -53,7 +53,9 @@ Tauri 录音日志增加 `[audio-startup]` 阶段，只有时间和录音/按键
比较 `first_raw_pcm` 与 `first_output_pcm` 可以区分硬件等待与处理/分帧等待。
Windows 快捷键日志使用 `press` 和 `capture` 关联；预热路线标记为 `warm_hub`。

新增回归测试会故意阻塞识别器准备，验证麦克风已开始、开头 PCM 仍能顺序交付，
并验证准备期间的取消无需等待准备完成即可释放麦克风。另验证准备失败清理。
回归测试 `capture_starts_before_slow_preparation_and_cancellation_releases_it_promptly`
会故意阻塞识别器准备，验证麦克风已开始、开头 PCM 仍能顺序交付，并验证准备期间
的取消无需等待准备完成即可释放麦克风；另有准备失败清理和 Windows WASAPI
资源生命周期测试。完整 `openless-core` 测试与 Windows CI 覆盖这些路径。

本轮未在原生 Windows 麦克风上测量优化前后的 P50/P95，不报告未经测量的毫秒收益。
3 changes: 3 additions & 0 deletions openless-all/app/contract/backend-2.0.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,7 @@
"accountId": null,
"vaultId": null,
"taskId": null,
"serviceOrigin": "https://apic.openless.top:9443",
"status": {
"sequence": "1",
"enabled": false,
Expand Down Expand Up @@ -245,6 +246,7 @@
"accountId": "118526",
"vaultId": "11111111-1111-4111-8111-111111111111",
"taskId": null,
"serviceOrigin": "https://apic.openless.top:9443",
"preview": {
"previewId": "22222222-2222-4222-8222-222222222222",
"unconfirmedOperationId": null,
Expand All @@ -267,6 +269,7 @@
"accountId": "118526",
"vaultId": "11111111-1111-4111-8111-111111111111",
"taskId": null,
"serviceOrigin": "https://apic.openless.top:9443",
"localGeneration": "1",
"uiPreferences": { "locale": "zh-CN", "fontScale": "medium" }
}
Expand Down
4 changes: 4 additions & 0 deletions openless-all/app/crates/openless-core/src/api.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13378,6 +13378,9 @@ mod tests {
)
.unwrap();

let mut preferences = backend.get_preferences();
preferences.cursor_context_enabled = false;
backend.set_preferences(preferences).unwrap();
let context = backend
.capture_host_dictation_context(DictationStartOptions::default())
.await
Expand Down Expand Up @@ -13574,6 +13577,7 @@ mod tests {
.unwrap();
let mut preferences = backend.get_preferences();
preferences.vocabulary_learning_enabled = true;
preferences.cursor_context_enabled = false;
assert!(
!preferences.cursor_context_enabled,
"learning must not require uploading cursor context"
Expand Down
Loading