Compare commits
20 Commits
pre-consol
...
1d4f1070a2
| Author | SHA1 | Date | |
|---|---|---|---|
| 1d4f1070a2 | |||
| a327f4d882 | |||
| d307722c7a | |||
| 46be0a5aca | |||
| f25f8db818 | |||
| bbc7c217be | |||
| 0c34a29367 | |||
| df6b19834d | |||
| 420da679f9 | |||
| 2b82b9be5b | |||
| 0e18a78fae | |||
| 4700668df1 | |||
| 4e947dec21 | |||
| 509b983c09 | |||
| 0b1c492edd | |||
| 6579c5fb6a | |||
| fe61661305 | |||
|
|
f8c9769e04 | ||
|
|
becbf69c35 | ||
|
|
b8b953dfa8 |
97
HANDOVER-2026-04-19.md
Normal file
97
HANDOVER-2026-04-19.md
Normal file
@@ -0,0 +1,97 @@
|
|||||||
|
---
|
||||||
|
name: handover-2026-04-19
|
||||||
|
type: reference
|
||||||
|
tags: [handover, session, kon]
|
||||||
|
description: Session handover — 2026/04/19 dogfood polish + cross-platform window chrome
|
||||||
|
---
|
||||||
|
|
||||||
|
# Kon Handover — 2026/04/19
|
||||||
|
|
||||||
|
Second dogfood sprint. Four phases: (1) fix bugs surfaced on first real use, (2) redesign History for cognitive-load hygiene, (3) resolve broken window resize/drag on Linux Wayland, (4) clean up microphone picker.
|
||||||
|
|
||||||
|
## What shipped this session
|
||||||
|
|
||||||
|
### Cross-window preferences sync
|
||||||
|
- `preferences.svelte.js` emits `kon:preferences-changed` Tauri event on update.
|
||||||
|
- Main / viewer / float layouts listen and call `applyExternalPreferences` without re-emit, so theme and font changes propagate live across sibling windows.
|
||||||
|
- Echo suppressed via source window label check.
|
||||||
|
|
||||||
|
### Hotkey recorder
|
||||||
|
- Root cause of "can't change hotkey": button-level `onkeydown` relied on post-click keyboard focus, which webkit2gtk on Linux does not guarantee.
|
||||||
|
- Fix: `document.addEventListener("keydown", ..., { capture: true })` inside a `$effect` gated by `recording`. Beats any descendant handler. Escape now cancels.
|
||||||
|
|
||||||
|
### History page redesign (research-backed)
|
||||||
|
- Compact row now shows the **title** (or "Untitled"), not body-preview text — metadata already lives in the row columns (date, duration, source icon).
|
||||||
|
- Expanded row gets an inline title input (replaces the old Rename prompt modal).
|
||||||
|
- **Edit** button opens the viewer window in `edit` mode (editable textarea, debounced save to localStorage + storage-event sync back to main history).
|
||||||
|
- **Export .md** copies a full YAML-frontmatter markdown document to the clipboard — paste into Obsidian.
|
||||||
|
- **Tags**: `$lib/utils/frontmatter.js` exposes `deriveAutoTags` (currently returns `[]`), `buildFrontmatter`, `serialiseFrontmatter`, `buildMarkdown`. Manual tags stored as `item.manualTags`, rendered as removable chips in the expanded row with `+ add tag` input.
|
||||||
|
- Header tag chip bar (cap 7, click to filter, × to clear), plus `tag:xyz` search syntax.
|
||||||
|
- Global **Starred** filter toggle in the History header.
|
||||||
|
- Research memo found all five previous auto-tag families redundant with existing row UI — kept the derivation hook for the post-Task-7 `topic:*` content tag from kon-llm.
|
||||||
|
- Duplicate-transcript render fix: expanded `<p>` only if compact preview actually truncated.
|
||||||
|
|
||||||
|
### Viewer / editor popout
|
||||||
|
- `/viewer` route now reads `kon_viewer_mode` from localStorage ("view" | "edit").
|
||||||
|
- Edit mode renders a plain textarea bound to `item.text`; 400ms debounced save flushes on input, final flush on `onDestroy`. Segment-specific controls (Compact, Starred) hidden in edit mode.
|
||||||
|
- Native title: **"Kon - Transcription Editor"**.
|
||||||
|
|
||||||
|
### Platform-aware window chrome (Linux fix)
|
||||||
|
**Root cause:** Tauri v2 frameless `decorations: false` on KDE Wayland + webkit2gtk does not honour diagonal corner resize (collapses `NorthEast` etc. to a single axis via GTK's `gtk_window_begin_resize_drag`), and `data-tauri-drag-region` adds noticeable drag latency. Setting `setPointerCapture` ahead of `startResizeDragging` does not help once the compositor has taken over the pointer grab. Verified via Context7 docs + Codex diagnosis — Linux frameless is a known-fragile path.
|
||||||
|
|
||||||
|
**Fix:**
|
||||||
|
- Linux uses **native KWin/Mutter decorations**. `src-tauri/tauri.linux.conf.json` overlays `decorations: true` + full main window config (title, sizes) — overlays **replace** the windows array, so every field must be present, not just the delta. `src-tauri/src/commands/windows.rs` uses `cfg!(target_os = "linux")` to set decorations per window.
|
||||||
|
- macOS / Windows keep custom chrome. `src/lib/utils/osInfo.js` `isLinux()` gates `<Titlebar>` and `<ResizeHandles>` via `useCustomChrome = $state(false)`; flips to `!isLinux()` after `loadOsInfo()` resolves.
|
||||||
|
- Dueling drag-region handlers removed across Titlebar, float page, viewer page — everywhere a manual `startDragging()` lives, the `data-tauri-drag-region` attribute was deleted (they're alternatives per Tauri docs, not combinable).
|
||||||
|
- `ResizeHandles` kept for macOS/Windows frameless: 12 px edges / 20 px corners via CSS vars (`--kon-resize-edge`, `--kon-resize-corner`), `pointerdown` + `setPointerCapture`, corners with explicit higher z-index. Handles rendered as siblings of the animated layout div so `position: fixed` is viewport-relative rather than captured by the transform containing block.
|
||||||
|
|
||||||
|
### Window minimum sizes (evidence-backed)
|
||||||
|
Research pass cited GNOME HIG (1024×600 desktop / 360×294 mobile floors), WCAG 2.2 SC 1.4.10 Reflow (320 CSS px), Raycast 750×474 as a reference for single-pane working width, and consistent A11y principle that nothing should clip in the default configuration.
|
||||||
|
|
||||||
|
| Window | Was | Now | Rationale |
|
||||||
|
|---|---|---|---|
|
||||||
|
| Main | 1020×540 | **960×600** | Fits 210 px sidebar + ~750 px content; GNOME vertical floor. |
|
||||||
|
| Float | 400×400 | **360×480** | 360 = GNOME mobile floor; 480 fits pills + quick-add + sort + ~6 task rows without scroll. |
|
||||||
|
| Transcript editor | 450×500 | **560×520** | Exceeds WCAG reflow floor; ~60-70 char measure for editing. |
|
||||||
|
|
||||||
|
### Microphone picker cleanup
|
||||||
|
- ALSA enumeration was leaking `hw:`, `plughw:`, `front:`, `sysdefault:`, `null` et al into the dropdown.
|
||||||
|
- `SettingsPage.svelte` now renders only sentinel devices (`default`, `pipewire`, `pulse`) + one entry per unique sound card, keyed off the `sysdefault:CARD=X` alias.
|
||||||
|
- `crates/audio/src/capture.rs` reads `/proc/asound/cards` and populates a new `description` field on `DeviceInfo` with the card's full product string (e.g. "Blue Microphones" for Jake's Yeti). Frontend prefers description → CARD=X short name → raw name.
|
||||||
|
|
||||||
|
### GPU reporting
|
||||||
|
- `commands/models.rs::get_runtime_capabilities` was hardcoded to `accelerators: vec!["cpu"]` and `supports_gpu: false` for whisper. Updated to `["cpu", "vulkan"]` and whisper `supports_gpu: true`, reflecting that `crates/transcription/Cargo.toml` links transcribe-rs with the `whisper-vulkan` feature unconditionally.
|
||||||
|
- Settings now shows the Vulkan option instead of the "This build is CPU-only" notice.
|
||||||
|
|
||||||
|
### Desktop shortcut
|
||||||
|
- `~/Desktop/Kon.desktop` launcher with the 128×128 icon, `Terminal=true` so logs are visible and Ctrl+C cleanly stops the run.sh wrapper.
|
||||||
|
|
||||||
|
## What's deferred
|
||||||
|
|
||||||
|
- **Transparent windows (`transparent: true`)** — Tauri issue #13270 reports this smooths drag/resize further on Linux, but it's moot now that Linux uses native decorations.
|
||||||
|
- **File-system export (.md save dialog)** — currently clipboard-only. Needs a Rust `write_text_file` command for plugin-less file writes.
|
||||||
|
- **Bulk select + bulk export** in History.
|
||||||
|
- **LLM-powered content tags** (`topic:*`, `intent:*`) — slots into Task 7 `kon-llm` stub once Phase 3 wires real llama-cpp-2.
|
||||||
|
- **Settings UX overhaul** — Jake flagged that current settings feel overwhelming. Proposed: bunch high-traffic settings, hide advanced behind a toggle. Brainstorm + plan deferred to a dedicated session.
|
||||||
|
- **Task 7 (MicroSteps end-to-end)** — storage + Tauri CRUD + kon-llm stub + frontend dual-write all landed in an earlier commit chain. The MicroSteps UI was written as the final task 7 step but not yet dogfooded against the stub LLM. Needs manual walkthrough.
|
||||||
|
|
||||||
|
## Gotchas discovered today
|
||||||
|
|
||||||
|
| Issue | Fix |
|
||||||
|
|---|---|
|
||||||
|
| `tauri.linux.conf.json` stripped title and min sizes from main window | Overlay **replaces** the windows array — include every field, not just the delta |
|
||||||
|
| `data-tauri-drag-region` + manual `startDragging()` on the same node caused drag latency | Pick one — we use manual `startDragging` for the button/input early-return logic |
|
||||||
|
| Corner resize collapsed to single axis on KWin Wayland | Native decorations on Linux side-step the whole frameless path |
|
||||||
|
| `animate-float-enter` on the viewer/float layout root created a containing block that broke `position: fixed` on ResizeHandles children | Render ResizeHandles as a sibling of the animated div, not a descendant |
|
||||||
|
| Kon binary auto-respawned on file-save while a second run.sh was also launching → two visible instances sharing one Vite server | Do not script `./run.sh` while the user has already launched via the desktop icon; rely on HMR |
|
||||||
|
| `run.sh` leaves `"beforeDevCommand": ""` in tauri.conf.json if its cleanup trap is bypassed (e.g. SIGKILL) | Cleanup trap restores `"npm run dev"` on graceful exit; SIGTERM (not SIGKILL) is the right kill signal |
|
||||||
|
| `/proc/asound/cards` header lines have leading whitespace for 2-digit card ID alignment | Parser trims leading whitespace before checking for leading digit |
|
||||||
|
|
||||||
|
## How to resume
|
||||||
|
|
||||||
|
```
|
||||||
|
Picking up Kon dogfooding from 2026/04/19.
|
||||||
|
HANDOVER is at HANDOVER.md in the project root.
|
||||||
|
Active priorities: (1) confirm resize/drag/mic cleanup, (2) Task 7 MicroSteps
|
||||||
|
dogfood with kon-llm stub, (3) Settings UX brainstorm.
|
||||||
|
```
|
||||||
149
HANDOVER.md
149
HANDOVER.md
@@ -1,97 +1,100 @@
|
|||||||
---
|
---
|
||||||
name: handover-2026-04-19
|
name: handover-2026-04-23
|
||||||
type: reference
|
type: reference
|
||||||
tags: [handover, session, kon]
|
tags: [handover, session, kon, consolidation]
|
||||||
description: Session handover — 2026/04/19 dogfood polish + cross-platform window chrome
|
description: Session handover — 2026/04/23 branch consolidation + main-in-its-best-state pass
|
||||||
---
|
---
|
||||||
|
|
||||||
# Kon Handover — 2026/04/19
|
# Kon Handover — 2026/04/23
|
||||||
|
|
||||||
Second dogfood sprint. Four phases: (1) fix bugs surfaced on first real use, (2) redesign History for cognitive-load hygiene, (3) resolve broken window resize/drag on Linux Wayland, (4) clean up microphone picker.
|
Consolidation session. Jake asked for "all branches to main and the repo in its best state" ahead of Corbie rebrand. This session brought outstanding Dependabot PRs into `main`, cleaned workspace lints, recovered an orphan stash onto a dedicated branch, and established a clean baseline for the post-rebrand push.
|
||||||
|
|
||||||
|
## Rebrand note
|
||||||
|
|
||||||
|
The product is in the process of being renamed **Kon → Corbie**. As of 2026/04/23 the canonical product name in any new copy (grant applications, landing pages) is **Corbie** (sentence case). The GitHub repo `jakejars/kon` and the Gitea repo `jake/kon` both still carry the `kon` name, intentionally — repo rename is pending Jake's own hand and is not blocking the rebrand copy-wise. Code paths (`crates/`, `src-tauri/`, package name `kon@0.1.0`) remain as-is until the repo rename lands; renaming the codebase identifiers is a separate coordinated sweep. See `memory/project_corbie_rebrand.md` in CORBEL-Main for the full rebrand state.
|
||||||
|
|
||||||
## What shipped this session
|
## What shipped this session
|
||||||
|
|
||||||
### Cross-window preferences sync
|
### Baseline validation
|
||||||
- `preferences.svelte.js` emits `kon:preferences-changed` Tauri event on update.
|
- Fresh `cargo build`, `cargo test --workspace`, `cargo fmt --check`, `npm run check`, `npm run build` run against the pre-consolidation tip (`9b0067b`, tagged as `pre-consolidation-2026-04-23` for recovery).
|
||||||
- Main / viewer / float layouts listen and call `applyExternalPreferences` without re-emit, so theme and font changes propagate live across sibling windows.
|
- `npm run build` initially failed with a missing-dependency error for `@chenglou/pretext`. Root cause: stale partial `node_modules` install dated 2026-04-21 07:45 — the directory for the package existed but was empty. `npm install` recovered it; package is present in lockfile and needed by `src/lib/utils/textMeasure.ts`.
|
||||||
- Echo suppressed via source window label check.
|
- `cargo clippy -- -D warnings` initially failed with 3 errors in `crates/storage/src/file_storage.rs` (two doc-list overindentations, one needless return). These were trivial style issues that clippy had never enforced at CI strictness.
|
||||||
|
|
||||||
### Hotkey recorder
|
### Workspace clippy cleanup (commit `fe61661`)
|
||||||
- Root cause of "can't change hotkey": button-level `onkeydown` relied on post-click keyboard focus, which webkit2gtk on Linux does not guarantee.
|
- Applied `cargo clippy --fix` across 11 files: `crates/audio`, `crates/hotkey`, `crates/storage`, `crates/transcription`, `src-tauri`.
|
||||||
- Fix: `document.addEventListener("keydown", ..., { capture: true })` inside a `$effect` gated by `recording`. Beats any descendant handler. Escape now cancels.
|
- Net -2 lines. No behavioural changes — pure lint cleanup (needless returns, unnecessary casts, `iter().any()` → `contains()`, `repeat().take()` → `repeat_n`, lifetime elision, `map_or` simplification, reference-immediately-dereferenced).
|
||||||
|
- One remaining warning left untouched: `needless_range_loop` at `src-tauri/src/commands/live.rs:1089` — clippy's suggested iterator rewrite would make it less readable. Earmarked for a focused refactor session.
|
||||||
|
- Build + 245 workspace tests remain green post-fix.
|
||||||
|
|
||||||
### History page redesign (research-backed)
|
### Dependabot merges (commits `6579c5f`, `0b1c492`, `509b983`)
|
||||||
- Compact row now shows the **title** (or "Untitled"), not body-preview text — metadata already lives in the row columns (date, duration, source icon).
|
Three dev-dep bumps landed as `--no-ff` merge commits:
|
||||||
- Expanded row gets an inline title input (replaces the old Rename prompt modal).
|
|
||||||
- **Edit** button opens the viewer window in `edit` mode (editable textarea, debounced save to localStorage + storage-event sync back to main history).
|
|
||||||
- **Export .md** copies a full YAML-frontmatter markdown document to the clipboard — paste into Obsidian.
|
|
||||||
- **Tags**: `$lib/utils/frontmatter.js` exposes `deriveAutoTags` (currently returns `[]`), `buildFrontmatter`, `serialiseFrontmatter`, `buildMarkdown`. Manual tags stored as `item.manualTags`, rendered as removable chips in the expanded row with `+ add tag` input.
|
|
||||||
- Header tag chip bar (cap 7, click to filter, × to clear), plus `tag:xyz` search syntax.
|
|
||||||
- Global **Starred** filter toggle in the History header.
|
|
||||||
- Research memo found all five previous auto-tag families redundant with existing row UI — kept the derivation hook for the post-Task-7 `topic:*` content tag from kon-llm.
|
|
||||||
- Duplicate-transcript render fix: expanded `<p>` only if compact preview actually truncated.
|
|
||||||
|
|
||||||
### Viewer / editor popout
|
| # | Package | File changes |
|
||||||
- `/viewer` route now reads `kon_viewer_mode` from localStorage ("view" | "edit").
|
|---|---|---|
|
||||||
- Edit mode renders a plain textarea bound to `item.text`; 400ms debounced save flushes on input, final flush on `onDestroy`. Segment-specific controls (Compact, Starred) hidden in edit mode.
|
| 1 | `picomatch` | `package-lock.json` only |
|
||||||
- Native title: **"Kon - Transcription Editor"**.
|
| 2 | `@sveltejs/kit` | `package.json` + `package-lock.json` |
|
||||||
|
| 3 | `vite` (npm_and_yarn group) | `package.json` + `package-lock.json` |
|
||||||
|
|
||||||
### Platform-aware window chrome (Linux fix)
|
After the three merges: `npm install` clean, `npm run build` green, `npm run check` 0 errors / 0 warnings. Vulnerability count went from 6 (1 low / 2 moderate / 3 high) to 5 (3 low / 2 moderate) — the three highs cleared.
|
||||||
**Root cause:** Tauri v2 frameless `decorations: false` on KDE Wayland + webkit2gtk does not honour diagonal corner resize (collapses `NorthEast` etc. to a single axis via GTK's `gtk_window_begin_resize_drag`), and `data-tauri-drag-region` adds noticeable drag latency. Setting `setPointerCapture` ahead of `startResizeDragging` does not help once the compositor has taken over the pointer grab. Verified via Context7 docs + Codex diagnosis — Linux frameless is a known-fragile path.
|
|
||||||
|
|
||||||
**Fix:**
|
### Orphan-stash recovery (branch `feat/design-system-recover`, commit `8855db8`)
|
||||||
- Linux uses **native KWin/Mutter decorations**. `src-tauri/tauri.linux.conf.json` overlays `decorations: true` + full main window config (title, sizes) — overlays **replace** the windows array, so every field must be present, not just the delta. `src-tauri/src/commands/windows.rs` uses `cfg!(target_os = "linux")` to set decorations per window.
|
- A WIP stash sat on `stash@{0}`, pinned to parent commit `1296173` — the tip of the deleted `feat/design-system` branch. ~1000 LOC across 24 files: `ai-formatting/rule_based.rs` rewrite, `transcription/local_engine.rs`, core `hardware.rs` + `recommendation.rs`, `audio/resample.rs`, and a sweep across Svelte UI (`TaskSidebar`, `FilesPage`, `TasksPage`, `WipTaskList`, `ModelDownloader`, `Titlebar`, `viewer`, `float`).
|
||||||
- macOS / Windows keep custom chrome. `src/lib/utils/osInfo.js` `isLinux()` gates `<Titlebar>` and `<ResizeHandles>` via `useCustomChrome = $state(false)`; flips to `!isLinux()` after `loadOsInfo()` resolves.
|
- Recovered using `git stash branch`, which creates a branch from the stash's parent commit and applies the stash content. Parent commit still existed in the reflog even though no branch referenced it.
|
||||||
- Dueling drag-region handlers removed across Titlebar, float page, viewer page — everywhere a manual `startDragging()` lives, the `data-tauri-drag-region` attribute was deleted (they're alternatives per Tauri docs, not combinable).
|
- Committed the full recovery as `wip(design-system): recover orphan stash` on the new branch. **Not merged to main.** Needs a focused rebase session: heavy conflicts expected in `TasksPage`, `FilesPage`, `ai-formatting/rule_based.rs`, and `transcription/local_engine.rs` against current main since those all saw release-blocker fixes after the stash was taken. Also missing the 2026-04-23 clippy cleanup.
|
||||||
- `ResizeHandles` kept for macOS/Windows frameless: 12 px edges / 20 px corners via CSS vars (`--kon-resize-edge`, `--kon-resize-corner`), `pointerdown` + `setPointerCapture`, corners with explicit higher z-index. Handles rendered as siblings of the animated layout div so `position: fixed` is viewport-relative rather than captured by the transform containing block.
|
- Branch pushed to `github` for remote preservation.
|
||||||
|
|
||||||
### Window minimum sizes (evidence-backed)
|
## Release-blocker state (unchanged this session)
|
||||||
Research pass cited GNOME HIG (1024×600 desktop / 360×294 mobile floors), WCAG 2.2 SC 1.4.10 Reflow (320 CSS px), Raycast 750×474 as a reference for single-pane working width, and consistent A11y principle that nothing should clip in the default configuration.
|
|
||||||
|
|
||||||
| Window | Was | Now | Rationale |
|
From `docs/issues/README.md`:
|
||||||
|---|---|---|---|
|
|
||||||
| Main | 1020×540 | **960×600** | Fits 210 px sidebar + ~750 px content; GNOME vertical floor. |
|
|
||||||
| Float | 400×400 | **360×480** | 360 = GNOME mobile floor; 480 fits pills + quick-add + sort + ~6 task rows without scroll. |
|
|
||||||
| Transcript editor | 450×500 | **560×520** | Exceeds WCAG reflow floor; ~60-70 char measure for editing. |
|
|
||||||
|
|
||||||
### Microphone picker cleanup
|
- **0 open CRITICAL**
|
||||||
- ALSA enumeration was leaking `hw:`, `plughw:`, `front:`, `sysdefault:`, `null` et al into the dropdown.
|
- **1 open MAJOR** — RB-08 `power-assertion-macos-objc2` (awaits manual runtime verification on a real macOS machine: `pmset -g assertions` during a background live-session)
|
||||||
- `SettingsPage.svelte` now renders only sentinel devices (`default`, `pipewire`, `pulse`) + one entry per unique sound card, keyed off the `sysdefault:CARD=X` alias.
|
- **11 RBs resolved**
|
||||||
- `crates/audio/src/capture.rs` reads `/proc/asound/cards` and populates a new `description` field on `DeviceInfo` with the card's full product string (e.g. "Blue Microphones" for Jake's Yeti). Frontend prefers description → CARD=X short name → raw name.
|
- **11 CR items resolved** from the 2026-04-22 code review
|
||||||
|
|
||||||
### GPU reporting
|
RB-08 continues to gate v0.1 tagging.
|
||||||
- `commands/models.rs::get_runtime_capabilities` was hardcoded to `accelerators: vec!["cpu"]` and `supports_gpu: false` for whisper. Updated to `["cpu", "vulkan"]` and whisper `supports_gpu: true`, reflecting that `crates/transcription/Cargo.toml` links transcribe-rs with the `whisper-vulkan` feature unconditionally.
|
|
||||||
- Settings now shows the Vulkan option instead of the "This build is CPU-only" notice.
|
|
||||||
|
|
||||||
### Desktop shortcut
|
## Post-consolidation follow-up (evening 2026/04/23)
|
||||||
- `~/Desktop/Kon.desktop` launcher with the 128×128 icon, `Terminal=true` so logs are visible and Ctrl+C cleanly stops the run.sh wrapper.
|
|
||||||
|
|
||||||
## What's deferred
|
Jake asked for "nothing outstanding" on Corbie. Follow-up pass did:
|
||||||
|
|
||||||
- **Transparent windows (`transparent: true`)** — Tauri issue #13270 reports this smooths drag/resize further on Linux, but it's moot now that Linux uses native decorations.
|
### 1. npm audit triaged + safe minor bumps landed
|
||||||
- **File-system export (.md save dialog)** — currently clipboard-only. Needs a Rust `write_text_file` command for plugin-less file writes.
|
- `@sveltejs/kit` 2.57.1 → **2.58.0**, `@sveltejs/adapter-static` 3.0.6 → **3.0.10** (commit `0e18a78`). Both are patch / minor SemVer — no API breakage.
|
||||||
- **Bulk select + bulk export** in History.
|
- **Residual advisories accepted** (5 total: 3 low / 2 moderate, all transitive, not session-actionable):
|
||||||
- **LLM-powered content tags** (`topic:*`, `intent:*`) — slots into Task 7 `kon-llm` stub once Phase 3 wires real llama-cpp-2.
|
- **cookie@0.6.x** (3× low) pinned by `@sveltejs/kit@^0.6.0` as of kit 2.58.0. Upstream has not bumped the pin; fixed cookie version is `0.7+`. Advisory: out-of-bounds chars in name/path/domain. **Context**: Kon/Corbie is a Tauri desktop app, no public-facing HTTP server, so the attack surface for this is effectively nil. Re-triage when `@sveltejs/kit` lands a cookie bump.
|
||||||
- **Settings UX overhaul** — Jake flagged that current settings feel overwhelming. Proposed: bunch high-traffic settings, hide advanced behind a toggle. Brainstorm + plan deferred to a dedicated session.
|
- **esbuild via svelte-i18n@4.0.1** (2× moderate) — `svelte-i18n` bundles `esbuild@^0.19.2` for its CLI. Advisory: "dev-server allows any website to read responses". **Context**: only affects `vite dev` over the network. In a Tauri desktop app the dev server is localhost-only and not relevant to production. `svelte-i18n` is already at latest (4.0.1); the only way to drop this is replace the i18n library.
|
||||||
- **Task 7 (MicroSteps end-to-end)** — storage + Tauri CRUD + kon-llm stub + frontend dual-write all landed in an earlier commit chain. The MicroSteps UI was written as the final task 7 step but not yet dogfooded against the stub LLM. Needs manual walkthrough.
|
- `npm audit --audit-level high` would report 0 vulnerabilities; GitHub's richer advisory db may still show the transitive set but none are actionable without upstream bumps.
|
||||||
|
|
||||||
## Gotchas discovered today
|
### 2. needless_range_loop refactor landed
|
||||||
|
Commit `2b82b9b` — `src-tauri/src/commands/live.rs:1087` duplicate-merge inner loop rewritten as `for segment in &nearby[start..upper]`. Workspace now **zero clippy warnings** with `cargo clippy --workspace --all-targets`. Safe to turn on `-D warnings` in CI when desired.
|
||||||
|
|
||||||
| Issue | Fix |
|
### 3. `feat/design-system-recover` evaluated, WIP layer abandoned
|
||||||
|---|---|
|
Compared the branch against current main:
|
||||||
| `tauri.linux.conf.json` stripped title and min sizes from main window | Overlay **replaces** the windows array — include every field, not just the delta |
|
- The handoff commit's content (design tokens, fonts, previews, ui_kits) is **already on main** as commit `8ba5641`. Recovery branch just duplicates it from a different base.
|
||||||
| `data-tauri-drag-region` + manual `startDragging()` on the same node caused drag latency | Pick one — we use manual `startDragging` for the button/input early-return logic |
|
- The WIP layer on top modifies 24 files (TasksPage, FilesPage, WipTaskList, TaskSidebar, viewer, float, local_engine, rule_based, etc.) against a base main has since significantly refactored (RB-01 through RB-12 + all the CR-2026-04-22 fixes + a preferences.svelte.js → .ts conversion).
|
||||||
| Corner resize collapsed to single axis on KWin Wayland | Native decorations on Linux side-step the whole frameless path |
|
- Test merge surfaces **14+ conflicts** needing per-file, designer-intent-aware resolution. Conflict cost > value of the WIP given main has already moved past most of it.
|
||||||
| `animate-float-enter` on the viewer/float layout root created a containing block that broke `position: fixed` on ResizeHandles children | Render ResizeHandles as a sibling of the animated div, not a descendant |
|
- **Decision: abandon the WIP**. Local branch deleted. `github/feat/design-system-recover` preserved as a read-only archive for reference mining. Any design-system polish from here gets redone fresh against current main rather than rebased through a stale WIP.
|
||||||
| Kon binary auto-respawned on file-save while a second run.sh was also launching → two visible instances sharing one Vite server | Do not script `./run.sh` while the user has already launched via the desktop icon; rely on HMR |
|
|
||||||
| `run.sh` leaves `"beforeDevCommand": ""` in tauri.conf.json if its cleanup trap is bypassed (e.g. SIGKILL) | Cleanup trap restores `"npm run dev"` on graceful exit; SIGTERM (not SIGKILL) is the right kill signal |
|
|
||||||
| `/proc/asound/cards` header lines have leading whitespace for 2-digit card ID alignment | Parser trims leading whitespace before checking for leading digit |
|
|
||||||
|
|
||||||
## How to resume
|
## Still deferred (not session-actionable)
|
||||||
|
|
||||||
```
|
- **CI green-up + Cargo.lock policy decision** — four jobs on the 2026-04-24 evening push (three `cargo check` matrix legs + `svelte build + lint`) went red, but a clean reproduction locally (`git archive HEAD | tar -x`, fresh `npm ci`, `cargo check --workspace --all-targets`) is green. Root cause likely one of: (a) crate version drift because `Cargo.lock` is in `.gitignore` so CI resolves fresh every run (a binary workspace should normally commit its lockfile), (b) Swatinem/rust-cache bad entry, (c) transient CI-runner environment issue. Next steps before restarting CI: pull the actual failing-step log text from the GitHub Actions UI, and decide whether to start committing `Cargo.lock` (recommended for a Tauri binary).
|
||||||
Picking up Kon dogfooding from 2026/04/19.
|
- **RB-08 verification** — needs macOS hardware access. Gates v0.1 tag. One concrete open item.
|
||||||
HANDOVER is at HANDOVER.md in the project root.
|
- **Kon → Corbie codebase rename** — Jake handles repo rename (Gitea + GitHub); coordinated codebase sweep (package name `kon@0.1.0` → `corbie@0.1.0`, crate prefixes `kon-*` → `corbie-*`, desktop file, binary name, install paths `~/.local/share/kon/` → `~/.local/share/corbie/`, database filename `kon.db`, window titles, README body) should follow the repo rename so artefact names match. Will need a migration shim to keep existing users' data alive across the path change.
|
||||||
Active priorities: (1) confirm resize/drag/mic cleanup, (2) Task 7 MicroSteps
|
- **npm audit residuals** — see §1 above. Re-visit when `@sveltejs/kit` bumps its cookie pin, or when an i18n replacement is chosen.
|
||||||
dogfood with kon-llm stub, (3) Settings UX brainstorm.
|
- **CI clippy enforcement** — workspace is now clean with default clippy. Turning on `-D warnings` in CI costs nothing additional.
|
||||||
```
|
- **Transparent windows**, **file-system export**, **bulk select/export**, **LLM content tags**, **Settings UX overhaul**, **Task 7 walk-through** — all carried from 2026-04-19 handover.
|
||||||
|
|
||||||
|
## Repo state at end of follow-up
|
||||||
|
|
||||||
|
- `main` at `2b82b9b` (evening pass: +3 commits — sveltejs bumps, clippy refactor, this handover update)
|
||||||
|
- Local branches: `main` only (recovery branch deleted locally after archival)
|
||||||
|
- `github/feat/design-system-recover` at `8855db8` (preserved as archive)
|
||||||
|
- Tag `pre-consolidation-2026-04-23` at `9b0067b`
|
||||||
|
- `cargo build --workspace` ✓ / `cargo test --workspace` ✓ (245 passing) / `cargo clippy --workspace --all-targets` **0 warnings** / `cargo fmt --check` ✓ / `npm run check` ✓ (0 errors, 0 warnings) / `npm run build` ✓
|
||||||
|
|
||||||
|
## Anchors
|
||||||
|
|
||||||
|
- Previous handover: [HANDOVER-2026-04-19.md](HANDOVER-2026-04-19.md)
|
||||||
|
- Release-blocker index: [docs/issues/README.md](docs/issues/README.md)
|
||||||
|
- Rebrand memory: `~/.claude/projects/-home-jake-Documents-CORBEL-Main/memory/project_corbie_rebrand.md`
|
||||||
|
- Active-focus upstream: `context/active-focus.md` in CORBEL-Main
|
||||||
|
|||||||
@@ -11,7 +11,7 @@ Kon is a local-first, cognitive-load-aware dictation and task-capture desktop ap
|
|||||||
**Pre-alpha.** Actively dogfooded on Linux (KDE Plasma 6 on Wayland). macOS and Windows targets are in scope and exercised by CI, but not yet beta-ready. One primary user; open source-intent with licence TBD before public beta.
|
**Pre-alpha.** Actively dogfooded on Linux (KDE Plasma 6 on Wayland). macOS and Windows targets are in scope and exercised by CI, but not yet beta-ready. One primary user; open source-intent with licence TBD before public beta.
|
||||||
|
|
||||||
- Current `main`: see commit log
|
- Current `main`: see commit log
|
||||||
- 136 automated lib tests across 10 crates, all passing
|
- 245 automated lib tests across 10 crates, all passing
|
||||||
- Cross-platform CI (Linux / macOS / Windows) via GitHub Actions
|
- Cross-platform CI (Linux / macOS / Windows) via GitHub Actions
|
||||||
|
|
||||||
---
|
---
|
||||||
@@ -288,7 +288,7 @@ CI also builds release installers on tag push (see `.github/workflows/build.yml`
|
|||||||
### Testing
|
### Testing
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
cargo test --workspace --lib # 136 tests across 10 crates
|
cargo test --workspace --lib # 245 tests across 10 crates
|
||||||
npm run check # svelte-check (type-checks .svelte files)
|
npm run check # svelte-check (type-checks .svelte files)
|
||||||
cargo check --workspace --all-targets
|
cargo check --workspace --all-targets
|
||||||
```
|
```
|
||||||
|
|||||||
@@ -112,7 +112,7 @@ impl MicrophoneCapture {
|
|||||||
for device in devices {
|
for device in devices {
|
||||||
let name = device_display_name(&device).unwrap_or_else(|| "<unnamed>".to_string());
|
let name = device_display_name(&device).unwrap_or_else(|| "<unnamed>".to_string());
|
||||||
let (sample_rate, channels) = match device.default_input_config() {
|
let (sample_rate, channels) = match device.default_input_config() {
|
||||||
Ok(cfg) => (cfg.sample_rate(), cfg.channels() as u16),
|
Ok(cfg) => (cfg.sample_rate(), cfg.channels()),
|
||||||
Err(_) => (0, 0),
|
Err(_) => (0, 0),
|
||||||
};
|
};
|
||||||
let is_likely_monitor = is_monitor_name(&name);
|
let is_likely_monitor = is_monitor_name(&name);
|
||||||
@@ -277,11 +277,7 @@ fn device_display_name(device: &cpal::Device) -> Option<String> {
|
|||||||
/// `pipewire` / `default` → `None`
|
/// `pipewire` / `default` → `None`
|
||||||
fn extract_card_id(name: &str) -> Option<&str> {
|
fn extract_card_id(name: &str) -> Option<&str> {
|
||||||
let rest = name.split("CARD=").nth(1)?;
|
let rest = name.split("CARD=").nth(1)?;
|
||||||
Some(
|
Some(rest.split([',', ';']).next().unwrap_or(rest))
|
||||||
rest.split(|c: char| c == ',' || c == ';')
|
|
||||||
.next()
|
|
||||||
.unwrap_or(rest),
|
|
||||||
)
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Read `/proc/asound/cards` and return a map from ALSA card short name
|
/// Read `/proc/asound/cards` and return a map from ALSA card short name
|
||||||
@@ -361,7 +357,7 @@ fn open_and_validate(
|
|||||||
.default_input_config()
|
.default_input_config()
|
||||||
.map_err(|e| KonError::AudioCaptureFailed(format!("default_input_config: {e}")))?;
|
.map_err(|e| KonError::AudioCaptureFailed(format!("default_input_config: {e}")))?;
|
||||||
let sample_rate = config.sample_rate();
|
let sample_rate = config.sample_rate();
|
||||||
let channels = config.channels() as u16;
|
let channels = config.channels();
|
||||||
let format = config.sample_format();
|
let format = config.sample_format();
|
||||||
|
|
||||||
eprintln!(
|
eprintln!(
|
||||||
|
|||||||
@@ -343,14 +343,14 @@ async fn device_listener(
|
|||||||
fn is_event_device(path: &Path) -> bool {
|
fn is_event_device(path: &Path) -> bool {
|
||||||
path.file_name()
|
path.file_name()
|
||||||
.and_then(|n| n.to_str())
|
.and_then(|n| n.to_str())
|
||||||
.map_or(false, |n| n.starts_with("event"))
|
.is_some_and(|n| n.starts_with("event"))
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Return true when the device's reported key set includes the combo's
|
/// Return true when the device's reported key set includes the combo's
|
||||||
/// configured trigger key. A device that reports no keys at all (for
|
/// configured trigger key. A device that reports no keys at all (for
|
||||||
/// example a mouse whose `EV_KEY` capability is buttons only) is rejected.
|
/// example a mouse whose `EV_KEY` capability is buttons only) is rejected.
|
||||||
fn device_supports_combo(supported: Option<&AttributeSetRef<Key>>, combo: &HotkeyCombo) -> bool {
|
fn device_supports_combo(supported: Option<&AttributeSetRef<Key>>, combo: &HotkeyCombo) -> bool {
|
||||||
supported.map_or(false, |keys| keys.contains(Key::new(combo.key_code)))
|
supported.is_some_and(|keys| keys.contains(Key::new(combo.key_code)))
|
||||||
}
|
}
|
||||||
|
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
|
|||||||
@@ -240,11 +240,30 @@ impl LlmEngine {
|
|||||||
}
|
}
|
||||||
|
|
||||||
pub fn decompose_task(&self, task_text: &str) -> Result<Vec<String>, EngineError> {
|
pub fn decompose_task(&self, task_text: &str) -> Result<Vec<String>, EngineError> {
|
||||||
|
self.decompose_task_with_feedback(task_text, &[])
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Same as `decompose_task` but allows callers to pass recent HITL
|
||||||
|
/// feedback rows so the system prompt gets conditioned on the
|
||||||
|
/// user's preferred decomposition style. The `examples` vec is
|
||||||
|
/// rendered into a few-shot block appended to the base system
|
||||||
|
/// prompt by `prompts::build_conditioned_system_prompt`.
|
||||||
|
///
|
||||||
|
/// Callers should pass most-recent-first; older examples still
|
||||||
|
/// participate but weigh less because of their position in the
|
||||||
|
/// prompt. Empty slice keeps behaviour identical to `decompose_task`.
|
||||||
|
pub fn decompose_task_with_feedback(
|
||||||
|
&self,
|
||||||
|
task_text: &str,
|
||||||
|
examples: &[prompts::FeedbackExample],
|
||||||
|
) -> Result<Vec<String>, EngineError> {
|
||||||
let model = self.loaded_model_arc()?;
|
let model = self.loaded_model_arc()?;
|
||||||
|
let system =
|
||||||
|
prompts::build_conditioned_system_prompt(prompts::DECOMPOSE_TASK_SYSTEM, examples);
|
||||||
let prompt = render_chat_prompt(
|
let prompt = render_chat_prompt(
|
||||||
&model,
|
&model,
|
||||||
&[
|
&[
|
||||||
("system", prompts::DECOMPOSE_TASK_SYSTEM),
|
("system", system.as_str()),
|
||||||
("user", &format!("Task: {task_text}")),
|
("user", &format!("Task: {task_text}")),
|
||||||
],
|
],
|
||||||
)?;
|
)?;
|
||||||
@@ -261,15 +280,27 @@ impl LlmEngine {
|
|||||||
}
|
}
|
||||||
|
|
||||||
pub fn extract_tasks(&self, transcript: &str) -> Result<Vec<String>, EngineError> {
|
pub fn extract_tasks(&self, transcript: &str) -> Result<Vec<String>, EngineError> {
|
||||||
|
self.extract_tasks_with_feedback(transcript, &[])
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Feedback-conditioned variant of `extract_tasks`. See
|
||||||
|
/// `decompose_task_with_feedback` for the `examples` semantics.
|
||||||
|
pub fn extract_tasks_with_feedback(
|
||||||
|
&self,
|
||||||
|
transcript: &str,
|
||||||
|
examples: &[prompts::FeedbackExample],
|
||||||
|
) -> Result<Vec<String>, EngineError> {
|
||||||
if transcript.trim().is_empty() {
|
if transcript.trim().is_empty() {
|
||||||
return Ok(Vec::new());
|
return Ok(Vec::new());
|
||||||
}
|
}
|
||||||
|
|
||||||
let model = self.loaded_model_arc()?;
|
let model = self.loaded_model_arc()?;
|
||||||
|
let system =
|
||||||
|
prompts::build_conditioned_system_prompt(prompts::EXTRACT_TASKS_SYSTEM, examples);
|
||||||
let prompt = render_chat_prompt(
|
let prompt = render_chat_prompt(
|
||||||
&model,
|
&model,
|
||||||
&[
|
&[
|
||||||
("system", prompts::EXTRACT_TASKS_SYSTEM),
|
("system", system.as_str()),
|
||||||
("user", &format!("Transcript:\n{transcript}")),
|
("user", &format!("Transcript:\n{transcript}")),
|
||||||
],
|
],
|
||||||
)?;
|
)?;
|
||||||
|
|||||||
@@ -10,3 +10,113 @@ output a JSON array of action items the speaker committed to. Each item must \
|
|||||||
be a short imperative sentence. Omit observations, wishes, and background \
|
be a short imperative sentence. Omit observations, wishes, and background \
|
||||||
context that are not explicit commitments. Output an empty array if there are \
|
context that are not explicit commitments. Output an empty array if there are \
|
||||||
no action items.";
|
no action items.";
|
||||||
|
|
||||||
|
/// Compact representation of a human-in-the-loop feedback example used
|
||||||
|
/// for few-shot prompt conditioning. Built by kon-storage and fed to the
|
||||||
|
/// prompt builder below; we keep this struct local to the LLM crate so
|
||||||
|
/// kon-llm does not depend on kon-storage.
|
||||||
|
#[derive(Debug, Clone)]
|
||||||
|
pub struct FeedbackExample {
|
||||||
|
/// What the AI was given as input (e.g. the parent task text, or
|
||||||
|
/// the transcript chunk). Kept verbatim.
|
||||||
|
pub input: String,
|
||||||
|
/// What the AI produced originally. `None` if the user only
|
||||||
|
/// gave a thumbs-up without a prior edit (positive signal
|
||||||
|
/// without a paired correction).
|
||||||
|
pub original_output: Option<String>,
|
||||||
|
/// What the user changed it to. `None` for thumbs-only rows.
|
||||||
|
/// This is the highest-value signal — when present, inject it
|
||||||
|
/// as the "good" output in the few-shot example.
|
||||||
|
pub corrected_output: Option<String>,
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Render a feedback example into the exemplar block used in prompt
|
||||||
|
/// conditioning. Returns `None` for rows that carry no usable pairing
|
||||||
|
/// (e.g. a thumbs-up with no input context).
|
||||||
|
fn render_feedback_exemplar(ex: &FeedbackExample) -> Option<String> {
|
||||||
|
if ex.input.trim().is_empty() {
|
||||||
|
return None;
|
||||||
|
}
|
||||||
|
let good = ex
|
||||||
|
.corrected_output
|
||||||
|
.as_deref()
|
||||||
|
.or(ex.original_output.as_deref())?;
|
||||||
|
let good = good.trim();
|
||||||
|
if good.is_empty() {
|
||||||
|
return None;
|
||||||
|
}
|
||||||
|
Some(format!("Input: {}\nGood output: {}", ex.input.trim(), good))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Build a system prompt that combines the base task system prompt
|
||||||
|
/// with a few-shot block assembled from recent HITL examples. If no
|
||||||
|
/// usable examples are available, returns the base prompt unchanged
|
||||||
|
/// so early users see the generic behaviour and the LLM is not
|
||||||
|
/// confused by an empty exemplar section.
|
||||||
|
///
|
||||||
|
/// The exemplars are ordered most-recent-first (caller's order is
|
||||||
|
/// preserved) so the LLM weights the user's current style over
|
||||||
|
/// earlier noise, mirroring what a human reviewer would do.
|
||||||
|
pub fn build_conditioned_system_prompt(base: &str, examples: &[FeedbackExample]) -> String {
|
||||||
|
let rendered: Vec<String> = examples
|
||||||
|
.iter()
|
||||||
|
.filter_map(render_feedback_exemplar)
|
||||||
|
.collect();
|
||||||
|
if rendered.is_empty() {
|
||||||
|
return base.to_string();
|
||||||
|
}
|
||||||
|
let block = rendered
|
||||||
|
.iter()
|
||||||
|
.map(|s| format!("- {s}"))
|
||||||
|
.collect::<Vec<_>>()
|
||||||
|
.join("\n");
|
||||||
|
format!(
|
||||||
|
"{base}\n\nHere are examples of the style this user prefers, in the \
|
||||||
|
user's own words. Match this style closely when producing your output:\n{block}"
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn builds_plain_prompt_when_no_examples() {
|
||||||
|
let out = build_conditioned_system_prompt(DECOMPOSE_TASK_SYSTEM, &[]);
|
||||||
|
assert_eq!(out, DECOMPOSE_TASK_SYSTEM);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn skips_empty_input_examples() {
|
||||||
|
let examples = vec![FeedbackExample {
|
||||||
|
input: String::new(),
|
||||||
|
original_output: None,
|
||||||
|
corrected_output: Some("ignored".into()),
|
||||||
|
}];
|
||||||
|
let out = build_conditioned_system_prompt(DECOMPOSE_TASK_SYSTEM, &examples);
|
||||||
|
assert_eq!(out, DECOMPOSE_TASK_SYSTEM);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn prefers_corrected_over_original() {
|
||||||
|
let examples = vec![FeedbackExample {
|
||||||
|
input: "Clean room".into(),
|
||||||
|
original_output: Some("Organise your bedroom".into()),
|
||||||
|
corrected_output: Some("Pick up one shirt from the floor".into()),
|
||||||
|
}];
|
||||||
|
let out = build_conditioned_system_prompt(DECOMPOSE_TASK_SYSTEM, &examples);
|
||||||
|
assert!(out.contains("Pick up one shirt from the floor"));
|
||||||
|
assert!(!out.contains("Organise your bedroom"));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn falls_back_to_original_when_no_correction() {
|
||||||
|
let examples = vec![FeedbackExample {
|
||||||
|
input: "Write report".into(),
|
||||||
|
original_output: Some("Open a blank document".into()),
|
||||||
|
corrected_output: None,
|
||||||
|
}];
|
||||||
|
let out = build_conditioned_system_prompt(DECOMPOSE_TASK_SYSTEM, &examples);
|
||||||
|
assert!(out.contains("Open a blank document"));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -270,6 +270,11 @@ pub async fn search_transcripts(
|
|||||||
/// Insert a task. `list_id` and `effort` are nullable (schema predates their
|
/// Insert a task. `list_id` and `effort` are nullable (schema predates their
|
||||||
/// UI surfacing); `notes` defaults to '' at the column level. Callers that
|
/// UI surfacing); `notes` defaults to '' at the column level. Callers that
|
||||||
/// want to set metadata at creation time pass `Some(...)`; omit for defaults.
|
/// want to set metadata at creation time pass `Some(...)`; omit for defaults.
|
||||||
|
///
|
||||||
|
/// Positional signature is deliberately flat — it mirrors the `tasks`
|
||||||
|
/// schema columns one-to-one. Refactor to a params struct only if another
|
||||||
|
/// nullable is added after `energy`.
|
||||||
|
#[allow(clippy::too_many_arguments)]
|
||||||
pub async fn insert_task(
|
pub async fn insert_task(
|
||||||
pool: &SqlitePool,
|
pool: &SqlitePool,
|
||||||
id: &str,
|
id: &str,
|
||||||
@@ -278,10 +283,11 @@ pub async fn insert_task(
|
|||||||
source_transcript_id: Option<&str>,
|
source_transcript_id: Option<&str>,
|
||||||
list_id: Option<&str>,
|
list_id: Option<&str>,
|
||||||
effort: Option<&str>,
|
effort: Option<&str>,
|
||||||
|
energy: Option<&str>,
|
||||||
) -> Result<()> {
|
) -> Result<()> {
|
||||||
sqlx::query(
|
sqlx::query(
|
||||||
"INSERT INTO tasks (id, text, bucket, source_transcript_id, list_id, effort) \
|
"INSERT INTO tasks (id, text, bucket, source_transcript_id, list_id, effort, energy) \
|
||||||
VALUES (?, ?, ?, ?, ?, ?)",
|
VALUES (?, ?, ?, ?, ?, ?, ?)",
|
||||||
)
|
)
|
||||||
.bind(id)
|
.bind(id)
|
||||||
.bind(text)
|
.bind(text)
|
||||||
@@ -289,6 +295,7 @@ pub async fn insert_task(
|
|||||||
.bind(source_transcript_id)
|
.bind(source_transcript_id)
|
||||||
.bind(list_id)
|
.bind(list_id)
|
||||||
.bind(effort)
|
.bind(effort)
|
||||||
|
.bind(energy)
|
||||||
.execute(pool)
|
.execute(pool)
|
||||||
.await
|
.await
|
||||||
.map_err(|e| KonError::StorageError(format!("Insert task failed: {e}")))?;
|
.map_err(|e| KonError::StorageError(format!("Insert task failed: {e}")))?;
|
||||||
@@ -298,7 +305,7 @@ pub async fn insert_task(
|
|||||||
pub async fn list_tasks(pool: &SqlitePool) -> Result<Vec<TaskRow>> {
|
pub async fn list_tasks(pool: &SqlitePool) -> Result<Vec<TaskRow>> {
|
||||||
let rows = sqlx::query(
|
let rows = sqlx::query(
|
||||||
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
||||||
source_transcript_id, parent_task_id \
|
source_transcript_id, parent_task_id, energy \
|
||||||
FROM tasks WHERE parent_task_id IS NULL ORDER BY created_at DESC",
|
FROM tasks WHERE parent_task_id IS NULL ORDER BY created_at DESC",
|
||||||
)
|
)
|
||||||
.fetch_all(pool)
|
.fetch_all(pool)
|
||||||
@@ -311,7 +318,7 @@ pub async fn list_tasks(pool: &SqlitePool) -> Result<Vec<TaskRow>> {
|
|||||||
pub async fn get_task_by_id(pool: &SqlitePool, id: &str) -> Result<Option<TaskRow>> {
|
pub async fn get_task_by_id(pool: &SqlitePool, id: &str) -> Result<Option<TaskRow>> {
|
||||||
let row = sqlx::query(
|
let row = sqlx::query(
|
||||||
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
||||||
source_transcript_id, parent_task_id FROM tasks WHERE id = ?",
|
source_transcript_id, parent_task_id, energy FROM tasks WHERE id = ?",
|
||||||
)
|
)
|
||||||
.bind(id)
|
.bind(id)
|
||||||
.fetch_optional(pool)
|
.fetch_optional(pool)
|
||||||
@@ -361,6 +368,27 @@ pub async fn update_task(
|
|||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Dedicated tri-state energy setter. Exists as its own function because
|
||||||
|
/// `update_task` uses `COALESCE(?, col)` to let `None` mean "preserve" —
|
||||||
|
/// which makes it impossible to explicitly clear energy back to NULL.
|
||||||
|
/// `set_task_energy` always writes exactly the value passed, including
|
||||||
|
/// `None` to clear. Returns the refreshed row.
|
||||||
|
///
|
||||||
|
/// Caller is responsible for validating `energy` is one of the allowed
|
||||||
|
/// values; the CHECK constraint will reject anything else at commit time.
|
||||||
|
pub async fn set_task_energy(pool: &SqlitePool, id: &str, energy: Option<&str>) -> Result<TaskRow> {
|
||||||
|
sqlx::query("UPDATE tasks SET energy = ? WHERE id = ?")
|
||||||
|
.bind(energy)
|
||||||
|
.bind(id)
|
||||||
|
.execute(pool)
|
||||||
|
.await
|
||||||
|
.map_err(|e| KonError::StorageError(format!("set_task_energy failed: {e}")))?;
|
||||||
|
|
||||||
|
get_task_by_id(pool, id).await?.ok_or_else(|| {
|
||||||
|
KonError::StorageError(format!("set_task_energy: task {id} not found after update"))
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
pub async fn insert_subtask(
|
pub async fn insert_subtask(
|
||||||
pool: &SqlitePool,
|
pool: &SqlitePool,
|
||||||
id: &str,
|
id: &str,
|
||||||
@@ -380,7 +408,7 @@ pub async fn insert_subtask(
|
|||||||
pub async fn list_subtasks(pool: &SqlitePool, parent_id: &str) -> Result<Vec<TaskRow>> {
|
pub async fn list_subtasks(pool: &SqlitePool, parent_id: &str) -> Result<Vec<TaskRow>> {
|
||||||
let rows = sqlx::query(
|
let rows = sqlx::query(
|
||||||
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
"SELECT id, text, bucket, list_id, effort, notes, done, done_at, created_at, \
|
||||||
source_transcript_id, parent_task_id \
|
source_transcript_id, parent_task_id, energy \
|
||||||
FROM tasks WHERE parent_task_id = ? ORDER BY created_at ASC",
|
FROM tasks WHERE parent_task_id = ? ORDER BY created_at ASC",
|
||||||
)
|
)
|
||||||
.bind(parent_id)
|
.bind(parent_id)
|
||||||
@@ -577,6 +605,11 @@ pub struct TaskRow {
|
|||||||
pub created_at: String,
|
pub created_at: String,
|
||||||
pub source_transcript_id: Option<String>,
|
pub source_transcript_id: Option<String>,
|
||||||
pub parent_task_id: Option<String>,
|
pub parent_task_id: Option<String>,
|
||||||
|
/// Phase 3 energy tagging: one of `"high"`, `"medium"`, `"brain_dead"`,
|
||||||
|
/// or `None`. Enforced at the DB layer via a CHECK constraint (see
|
||||||
|
/// migration v11). Unset is the expected normal case — the match-my-
|
||||||
|
/// energy sort treats unset as Medium-equivalent.
|
||||||
|
pub energy: Option<String>,
|
||||||
}
|
}
|
||||||
|
|
||||||
fn transcript_row_from(r: &sqlx::sqlite::SqliteRow) -> TranscriptRow {
|
fn transcript_row_from(r: &sqlx::sqlite::SqliteRow) -> TranscriptRow {
|
||||||
@@ -639,6 +672,7 @@ fn task_row_from(r: sqlx::sqlite::SqliteRow) -> TaskRow {
|
|||||||
created_at: r.get("created_at"),
|
created_at: r.get("created_at"),
|
||||||
source_transcript_id: r.get("source_transcript_id"),
|
source_transcript_id: r.get("source_transcript_id"),
|
||||||
parent_task_id: r.get("parent_task_id"),
|
parent_task_id: r.get("parent_task_id"),
|
||||||
|
energy: r.get("energy"),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -889,6 +923,151 @@ pub async fn list_recent_errors(pool: &SqlitePool, limit: i64) -> Result<Vec<Err
|
|||||||
.collect())
|
.collect())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// --- Feedback (HITL) -------------------------------------------------------
|
||||||
|
//
|
||||||
|
// Phase 2 of the feature-complete roadmap: capture thumbs + corrections on
|
||||||
|
// AI-generated output so the prompt builder can inject recent examples as
|
||||||
|
// few-shot exemplars. Storage-only here; the prompt-conditioning logic lives
|
||||||
|
// in kon-llm. Retrieval returns the most recent rows, narrowed to the
|
||||||
|
// active profile when provided so feedback does not cross profiles.
|
||||||
|
|
||||||
|
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
|
||||||
|
pub enum FeedbackTargetType {
|
||||||
|
MicroStep,
|
||||||
|
TaskExtraction,
|
||||||
|
Cleanup,
|
||||||
|
}
|
||||||
|
|
||||||
|
impl FeedbackTargetType {
|
||||||
|
pub fn as_str(self) -> &'static str {
|
||||||
|
match self {
|
||||||
|
FeedbackTargetType::MicroStep => "microstep",
|
||||||
|
FeedbackTargetType::TaskExtraction => "task_extraction",
|
||||||
|
FeedbackTargetType::Cleanup => "cleanup",
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Parse the database `target_type` string back into the enum.
|
||||||
|
/// Named `parse` rather than `from_str` so it does not collide with
|
||||||
|
/// the `std::str::FromStr` trait — the trait is overkill here
|
||||||
|
/// because callers never want a `FromStr::Err` and already know the
|
||||||
|
/// set of valid values at the call site.
|
||||||
|
pub fn parse(s: &str) -> Option<Self> {
|
||||||
|
match s {
|
||||||
|
"microstep" => Some(FeedbackTargetType::MicroStep),
|
||||||
|
"task_extraction" => Some(FeedbackTargetType::TaskExtraction),
|
||||||
|
"cleanup" => Some(FeedbackTargetType::Cleanup),
|
||||||
|
_ => None,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[derive(Debug, Clone)]
|
||||||
|
pub struct RecordFeedbackParams {
|
||||||
|
pub target_type: FeedbackTargetType,
|
||||||
|
pub target_id: Option<String>,
|
||||||
|
/// -1 = thumbs down, 0 = correction (neutral), +1 = thumbs up.
|
||||||
|
pub rating: i8,
|
||||||
|
pub original_text: Option<String>,
|
||||||
|
pub corrected_text: Option<String>,
|
||||||
|
pub context_json: Option<String>,
|
||||||
|
pub profile_id: Option<String>,
|
||||||
|
}
|
||||||
|
|
||||||
|
#[derive(Debug, Clone)]
|
||||||
|
pub struct FeedbackRow {
|
||||||
|
pub id: i64,
|
||||||
|
pub target_type: String,
|
||||||
|
pub target_id: Option<String>,
|
||||||
|
pub rating: i64,
|
||||||
|
pub original_text: Option<String>,
|
||||||
|
pub corrected_text: Option<String>,
|
||||||
|
pub context_json: Option<String>,
|
||||||
|
pub profile_id: String,
|
||||||
|
pub created_at: String,
|
||||||
|
}
|
||||||
|
|
||||||
|
pub async fn record_feedback(pool: &SqlitePool, params: RecordFeedbackParams) -> Result<i64> {
|
||||||
|
if !matches!(params.rating, -1..=1) {
|
||||||
|
return Err(KonError::StorageError(format!(
|
||||||
|
"invalid feedback rating {} (must be -1, 0, or 1)",
|
||||||
|
params.rating
|
||||||
|
)));
|
||||||
|
}
|
||||||
|
let profile_id = params
|
||||||
|
.profile_id
|
||||||
|
.unwrap_or_else(|| crate::DEFAULT_PROFILE_ID.to_string());
|
||||||
|
let row = sqlx::query(
|
||||||
|
"INSERT INTO feedback (
|
||||||
|
target_type, target_id, rating,
|
||||||
|
original_text, corrected_text, context_json, profile_id
|
||||||
|
) VALUES (?, ?, ?, ?, ?, ?, ?)
|
||||||
|
RETURNING id",
|
||||||
|
)
|
||||||
|
.bind(params.target_type.as_str())
|
||||||
|
.bind(params.target_id)
|
||||||
|
.bind(params.rating as i64)
|
||||||
|
.bind(params.original_text)
|
||||||
|
.bind(params.corrected_text)
|
||||||
|
.bind(params.context_json)
|
||||||
|
.bind(profile_id)
|
||||||
|
.fetch_one(pool)
|
||||||
|
.await
|
||||||
|
.map_err(|e| KonError::StorageError(format!("record_feedback failed: {e}")))?;
|
||||||
|
Ok(row.get::<i64, _>("id"))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Fetch the most recent feedback rows for a given target type, scoped to
|
||||||
|
/// the active profile. Used by the prompt builder to gather few-shot
|
||||||
|
/// exemplars. Orders by `created_at DESC` so the most recent corrections
|
||||||
|
/// outweigh older ones — the user's style drifts, and we want the LLM
|
||||||
|
/// to track the current preference.
|
||||||
|
///
|
||||||
|
/// `min_rating` filters out thumbs-down examples when the caller only
|
||||||
|
/// wants positive reinforcement; pass `-1` to include everything.
|
||||||
|
pub async fn list_feedback_examples(
|
||||||
|
pool: &SqlitePool,
|
||||||
|
target_type: FeedbackTargetType,
|
||||||
|
limit: i64,
|
||||||
|
min_rating: i8,
|
||||||
|
profile_id: Option<&str>,
|
||||||
|
) -> Result<Vec<FeedbackRow>> {
|
||||||
|
let pid = profile_id.unwrap_or(crate::DEFAULT_PROFILE_ID);
|
||||||
|
let rows = sqlx::query(
|
||||||
|
"SELECT id, target_type, target_id, rating,
|
||||||
|
original_text, corrected_text, context_json,
|
||||||
|
profile_id, created_at
|
||||||
|
FROM feedback
|
||||||
|
WHERE target_type = ?
|
||||||
|
AND profile_id = ?
|
||||||
|
AND rating >= ?
|
||||||
|
ORDER BY created_at DESC, id DESC
|
||||||
|
LIMIT ?",
|
||||||
|
)
|
||||||
|
.bind(target_type.as_str())
|
||||||
|
.bind(pid)
|
||||||
|
.bind(min_rating as i64)
|
||||||
|
.bind(limit)
|
||||||
|
.fetch_all(pool)
|
||||||
|
.await
|
||||||
|
.map_err(|e| KonError::StorageError(format!("list_feedback_examples failed: {e}")))?;
|
||||||
|
|
||||||
|
Ok(rows
|
||||||
|
.into_iter()
|
||||||
|
.map(|r| FeedbackRow {
|
||||||
|
id: r.get("id"),
|
||||||
|
target_type: r.get("target_type"),
|
||||||
|
target_id: r.get("target_id"),
|
||||||
|
rating: r.get("rating"),
|
||||||
|
original_text: r.get("original_text"),
|
||||||
|
corrected_text: r.get("corrected_text"),
|
||||||
|
context_json: r.get("context_json"),
|
||||||
|
profile_id: r.get("profile_id"),
|
||||||
|
created_at: r.get("created_at"),
|
||||||
|
})
|
||||||
|
.collect())
|
||||||
|
}
|
||||||
|
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::*;
|
use super::*;
|
||||||
@@ -1013,7 +1192,16 @@ mod tests {
|
|||||||
async fn task_crud_roundtrip() {
|
async fn task_crud_roundtrip() {
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
|
|
||||||
insert_task(&pool, "task1", "Buy groceries", "today", None, None, None)
|
insert_task(
|
||||||
|
&pool,
|
||||||
|
"task1",
|
||||||
|
"Buy groceries",
|
||||||
|
"today",
|
||||||
|
None,
|
||||||
|
None,
|
||||||
|
None,
|
||||||
|
None,
|
||||||
|
)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
|
|
||||||
@@ -1034,7 +1222,7 @@ mod tests {
|
|||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn subtask_crud_roundtrip() {
|
async fn subtask_crud_roundtrip() {
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
insert_task(&pool, "p1", "Write report", "inbox", None, None, None)
|
insert_task(&pool, "p1", "Write report", "inbox", None, None, None, None)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
insert_subtask(&pool, "s1", "Open document", "p1")
|
insert_subtask(&pool, "s1", "Open document", "p1")
|
||||||
@@ -1079,7 +1267,7 @@ mod tests {
|
|||||||
// child must also reopen the parent so "parent is done iff
|
// child must also reopen the parent so "parent is done iff
|
||||||
// every child is done" holds in both directions.
|
// every child is done" holds in both directions.
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
insert_task(&pool, "p1", "Ship release", "inbox", None, None, None)
|
insert_task(&pool, "p1", "Ship release", "inbox", None, None, None, None)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
insert_subtask(&pool, "s1", "Final test", "p1")
|
insert_subtask(&pool, "s1", "Final test", "p1")
|
||||||
@@ -1113,10 +1301,10 @@ mod tests {
|
|||||||
// Sanity: tasks without a parent must not trigger the parent-
|
// Sanity: tasks without a parent must not trigger the parent-
|
||||||
// reopen branch (it should no-op cleanly).
|
// reopen branch (it should no-op cleanly).
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
insert_task(&pool, "a", "A", "inbox", None, None, None)
|
insert_task(&pool, "a", "A", "inbox", None, None, None, None)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
insert_task(&pool, "b", "B", "inbox", None, None, None)
|
insert_task(&pool, "b", "B", "inbox", None, None, None, None)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
complete_task(&pool, "a").await.unwrap();
|
complete_task(&pool, "a").await.unwrap();
|
||||||
@@ -1240,7 +1428,7 @@ mod tests {
|
|||||||
async fn update_task_overwrites_provided_fields() {
|
async fn update_task_overwrites_provided_fields() {
|
||||||
// Task 2.6 — happy path: insert, update bucket + effort, read back.
|
// Task 2.6 — happy path: insert, update bucket + effort, read back.
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
insert_task(&pool, "u1", "Draft post", "inbox", None, None, None)
|
insert_task(&pool, "u1", "Draft post", "inbox", None, None, None, None)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
|
|
||||||
@@ -1257,7 +1445,16 @@ mod tests {
|
|||||||
async fn update_task_partial_leaves_others_unchanged() {
|
async fn update_task_partial_leaves_others_unchanged() {
|
||||||
// Task 2.6 — partial update: only notes changes; text/bucket intact.
|
// Task 2.6 — partial update: only notes changes; text/bucket intact.
|
||||||
let pool = test_pool().await;
|
let pool = test_pool().await;
|
||||||
insert_task(&pool, "u2", "Prep slides", "today", None, None, Some("30m"))
|
insert_task(
|
||||||
|
&pool,
|
||||||
|
"u2",
|
||||||
|
"Prep slides",
|
||||||
|
"today",
|
||||||
|
None,
|
||||||
|
None,
|
||||||
|
Some("30m"),
|
||||||
|
None,
|
||||||
|
)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
|
|
||||||
@@ -1288,6 +1485,52 @@ mod tests {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// --- Energy tagging (Phase 3) ---
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn set_task_energy_round_trip_includes_explicit_clear() {
|
||||||
|
let pool = test_pool().await;
|
||||||
|
insert_task(&pool, "e1", "Write report", "inbox", None, None, None, None)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
|
||||||
|
// Fresh task: energy must be NULL.
|
||||||
|
let t = get_task_by_id(&pool, "e1").await.unwrap().unwrap();
|
||||||
|
assert!(t.energy.is_none(), "new task must start with energy unset");
|
||||||
|
|
||||||
|
// Set High.
|
||||||
|
let t = set_task_energy(&pool, "e1", Some("high")).await.unwrap();
|
||||||
|
assert_eq!(t.energy.as_deref(), Some("high"));
|
||||||
|
|
||||||
|
// Change to Medium.
|
||||||
|
let t = set_task_energy(&pool, "e1", Some("medium")).await.unwrap();
|
||||||
|
assert_eq!(t.energy.as_deref(), Some("medium"));
|
||||||
|
|
||||||
|
// Explicit clear back to NULL — the whole reason this function
|
||||||
|
// exists separately from update_task's COALESCE semantics.
|
||||||
|
let t = set_task_energy(&pool, "e1", None).await.unwrap();
|
||||||
|
assert!(
|
||||||
|
t.energy.is_none(),
|
||||||
|
"set_task_energy(None) must explicitly clear the column"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn set_task_energy_rejects_unknown_value_via_check_constraint() {
|
||||||
|
// Migration v11 defines a CHECK constraint; invalid values must
|
||||||
|
// be rejected at the DB layer even if a caller bypasses frontend
|
||||||
|
// validation.
|
||||||
|
let pool = test_pool().await;
|
||||||
|
insert_task(&pool, "e2", "Task", "inbox", None, None, None, None)
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
let res = set_task_energy(&pool, "e2", Some("turbo")).await;
|
||||||
|
assert!(
|
||||||
|
res.is_err(),
|
||||||
|
"CHECK constraint must reject energy values outside the enum"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
// --- Profile CRUD tests (Task 11) ---
|
// --- Profile CRUD tests (Task 11) ---
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
|
|||||||
@@ -2,7 +2,7 @@ use std::path::PathBuf;
|
|||||||
|
|
||||||
/// Resolve the per-user app data directory, following each OS's convention:
|
/// Resolve the per-user app data directory, following each OS's convention:
|
||||||
///
|
///
|
||||||
/// - Windows: `%LOCALAPPDATA%\kon\` e.g. `C:\Users\Jake\AppData\Local\kon`
|
/// - Windows: `%LOCALAPPDATA%\kon\` — e.g. `C:\Users\Jake\AppData\Local\kon`
|
||||||
/// - macOS: `~/Library/Application Support/Kon/`
|
/// - macOS: `~/Library/Application Support/Kon/`
|
||||||
/// - Linux: `$XDG_DATA_HOME/kon` or `~/.local/share/kon` (XDG Base Directory),
|
/// - Linux: `$XDG_DATA_HOME/kon` or `~/.local/share/kon` (XDG Base Directory),
|
||||||
/// with a fallback to the legacy `~/.kon/` if it already exists, so
|
/// with a fallback to the legacy `~/.kon/` if it already exists, so
|
||||||
@@ -45,7 +45,7 @@ pub fn app_data_dir() -> PathBuf {
|
|||||||
return PathBuf::from(xdg).join("kon");
|
return PathBuf::from(xdg).join("kon");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return PathBuf::from(home).join(".local").join("share").join("kon");
|
PathBuf::from(home).join(".local").join("share").join("kon")
|
||||||
}
|
}
|
||||||
|
|
||||||
#[cfg(not(any(target_os = "windows", target_os = "macos", target_os = "linux")))]
|
#[cfg(not(any(target_os = "windows", target_os = "macos", target_os = "linux")))]
|
||||||
|
|||||||
@@ -10,10 +10,11 @@ pub use database::{
|
|||||||
add_profile_term, complete_subtask_and_check_parent, complete_task, count_transcripts,
|
add_profile_term, complete_subtask_and_check_parent, complete_task, count_transcripts,
|
||||||
create_profile, delete_profile, delete_profile_term, delete_task, delete_transcript,
|
create_profile, delete_profile, delete_profile_term, delete_task, delete_transcript,
|
||||||
get_profile, get_setting, get_task_by_id, get_transcript, init, insert_subtask, insert_task,
|
get_profile, get_setting, get_task_by_id, get_transcript, init, insert_subtask, insert_task,
|
||||||
insert_transcript, list_profile_terms, list_profiles, list_recent_errors, list_subtasks,
|
insert_transcript, list_feedback_examples, list_profile_terms, list_profiles,
|
||||||
list_tasks, list_transcripts, list_transcripts_paged, log_error, search_transcripts,
|
list_recent_errors, list_subtasks, list_tasks, list_transcripts, list_transcripts_paged,
|
||||||
set_setting, uncomplete_task, update_profile, update_task, update_transcript,
|
log_error, record_feedback, search_transcripts, set_setting, set_task_energy, uncomplete_task,
|
||||||
update_transcript_meta, ErrorLogRow, InsertTranscriptParams, ProfileRow, ProfileTermRow,
|
update_profile, update_task, update_transcript, update_transcript_meta, ErrorLogRow,
|
||||||
TaskRow, TranscriptRow,
|
FeedbackRow, FeedbackTargetType, InsertTranscriptParams, ProfileRow, ProfileTermRow,
|
||||||
|
RecordFeedbackParams, TaskRow, TranscriptRow,
|
||||||
};
|
};
|
||||||
pub use file_storage::{app_data_dir, crashes_dir, database_path, logs_dir, recordings_dir};
|
pub use file_storage::{app_data_dir, crashes_dir, database_path, logs_dir, recordings_dir};
|
||||||
|
|||||||
@@ -334,6 +334,72 @@ const MIGRATIONS: &[(i64, &str, &str)] = &[
|
|||||||
FROM transcripts;
|
FROM transcripts;
|
||||||
"#,
|
"#,
|
||||||
),
|
),
|
||||||
|
(
|
||||||
|
10,
|
||||||
|
"feedback: HITL thumbs + correction capture",
|
||||||
|
r#"
|
||||||
|
-- Feedback rows capture human-in-the-loop signal on AI-generated
|
||||||
|
-- output. Two flavours bundled into one table:
|
||||||
|
-- - thumbs (rating = -1 | +1, original_text optional, corrected_text NULL)
|
||||||
|
-- - correction (rating defaults to +1, original_text + corrected_text present)
|
||||||
|
--
|
||||||
|
-- `target_type` names the producing surface:
|
||||||
|
-- 'microstep' — subtask decomposition from DECOMPOSE_TASK_SYSTEM
|
||||||
|
-- 'task_extraction' — tasks lifted from a transcript (EXTRACT_TASKS_SYSTEM)
|
||||||
|
-- 'cleanup' — transcript cleanup output
|
||||||
|
--
|
||||||
|
-- `target_id` is the surface-specific identifier where one exists
|
||||||
|
-- (subtask id, task id, transcript id). NULL is allowed because
|
||||||
|
-- not every feedback event has a stable target id yet.
|
||||||
|
--
|
||||||
|
-- `context_json` carries the input the AI was conditioned on
|
||||||
|
-- (parent task text, transcript chunk, etc.) so future prompt
|
||||||
|
-- builders can reconstruct the original I/O pair for few-shot
|
||||||
|
-- injection or semantic retrieval.
|
||||||
|
CREATE TABLE feedback (
|
||||||
|
id INTEGER PRIMARY KEY AUTOINCREMENT,
|
||||||
|
target_type TEXT NOT NULL
|
||||||
|
CHECK (target_type IN ('microstep', 'task_extraction', 'cleanup')),
|
||||||
|
target_id TEXT,
|
||||||
|
rating INTEGER NOT NULL
|
||||||
|
CHECK (rating IN (-1, 0, 1)),
|
||||||
|
original_text TEXT,
|
||||||
|
corrected_text TEXT,
|
||||||
|
context_json TEXT,
|
||||||
|
profile_id TEXT NOT NULL DEFAULT '00000000-0000-0000-0000-000000000001'
|
||||||
|
REFERENCES profiles(id) ON DELETE RESTRICT,
|
||||||
|
created_at TEXT NOT NULL DEFAULT (datetime('now'))
|
||||||
|
);
|
||||||
|
|
||||||
|
CREATE INDEX idx_feedback_target_type_rating
|
||||||
|
ON feedback(target_type, rating, created_at DESC);
|
||||||
|
CREATE INDEX idx_feedback_profile
|
||||||
|
ON feedback(profile_id, target_type, created_at DESC);
|
||||||
|
"#,
|
||||||
|
),
|
||||||
|
(
|
||||||
|
11,
|
||||||
|
"tasks: energy tagging for match-my-energy sort",
|
||||||
|
r#"
|
||||||
|
-- Phase 3 of the feature-complete roadmap: replaces the cut
|
||||||
|
-- temptation-bundling feature with a deterministic client-side
|
||||||
|
-- sort that matches tasks to the user's current energy state.
|
||||||
|
-- NULL is the expected normal case — users who never tag get
|
||||||
|
-- Medium-equivalent treatment at sort time (see Match-my-energy
|
||||||
|
-- logic in src/lib/pages/TasksPage.svelte).
|
||||||
|
--
|
||||||
|
-- profile_id is deliberately absent from the index: tasks
|
||||||
|
-- currently carry no profile_id column, so a per-profile index
|
||||||
|
-- is out of scope until the broader task → profile migration
|
||||||
|
-- lands. See HANDOVER deferred list.
|
||||||
|
ALTER TABLE tasks
|
||||||
|
ADD COLUMN energy TEXT
|
||||||
|
CHECK (energy IS NULL OR energy IN ('high', 'medium', 'brain_dead'));
|
||||||
|
|
||||||
|
CREATE INDEX idx_tasks_energy_created
|
||||||
|
ON tasks(energy, created_at DESC);
|
||||||
|
"#,
|
||||||
|
),
|
||||||
];
|
];
|
||||||
|
|
||||||
/// Split SQL into individual statements, respecting BEGIN...END trigger blocks.
|
/// Split SQL into individual statements, respecting BEGIN...END trigger blocks.
|
||||||
@@ -483,7 +549,7 @@ mod tests {
|
|||||||
.fetch_one(&pool)
|
.fetch_one(&pool)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
assert_eq!(count, 9);
|
assert_eq!(count, 11);
|
||||||
|
|
||||||
sqlx::query("INSERT INTO settings (key, value) VALUES ('test', 'value')")
|
sqlx::query("INSERT INTO settings (key, value) VALUES ('test', 'value')")
|
||||||
.execute(&pool)
|
.execute(&pool)
|
||||||
@@ -502,7 +568,7 @@ mod tests {
|
|||||||
.fetch_one(&pool)
|
.fetch_one(&pool)
|
||||||
.await
|
.await
|
||||||
.unwrap();
|
.unwrap();
|
||||||
assert_eq!(count, 9);
|
assert_eq!(count, 11);
|
||||||
}
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
@@ -859,8 +925,11 @@ mod tests {
|
|||||||
// The poisoned migration below first creates `poison_marker`
|
// The poisoned migration below first creates `poison_marker`
|
||||||
// (syntactically valid, would succeed against any SQLite) and then
|
// (syntactically valid, would succeed against any SQLite) and then
|
||||||
// runs a guaranteed-invalid function call. Under the new atomic
|
// runs a guaranteed-invalid function call. Under the new atomic
|
||||||
// implementation, neither `poison_marker` nor the v9 row should
|
// implementation, neither `poison_marker` nor the poison row should
|
||||||
// survive the failed call.
|
// survive the failed call.
|
||||||
|
//
|
||||||
|
// Version number must sit above the real MIGRATIONS max so the
|
||||||
|
// baseline migrate cleanly finishes first.
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn multi_statement_migration_rolls_back_on_failure() {
|
async fn multi_statement_migration_rolls_back_on_failure() {
|
||||||
let pool = SqlitePoolOptions::new()
|
let pool = SqlitePoolOptions::new()
|
||||||
@@ -871,8 +940,18 @@ mod tests {
|
|||||||
|
|
||||||
run_migrations(&pool).await.expect("baseline migrate");
|
run_migrations(&pool).await.expect("baseline migrate");
|
||||||
|
|
||||||
const POISON: &[(i64, &str, &str)] = &[(
|
// Discover the real max version so the poison migration is
|
||||||
10,
|
// always exactly one past the end of MIGRATIONS, regardless of
|
||||||
|
// how many real migrations we add in future.
|
||||||
|
let real_max: i64 =
|
||||||
|
sqlx::query_scalar("SELECT COALESCE(MAX(version), 0) FROM schema_version")
|
||||||
|
.fetch_one(&pool)
|
||||||
|
.await
|
||||||
|
.expect("read schema_version");
|
||||||
|
let poison_version = real_max + 1;
|
||||||
|
|
||||||
|
let poison: &[(i64, &str, &str)] = &[(
|
||||||
|
poison_version,
|
||||||
"rb-02 atomicity poison",
|
"rb-02 atomicity poison",
|
||||||
r#"
|
r#"
|
||||||
CREATE TABLE poison_marker (id INTEGER PRIMARY KEY);
|
CREATE TABLE poison_marker (id INTEGER PRIMARY KEY);
|
||||||
@@ -880,7 +959,7 @@ mod tests {
|
|||||||
"#,
|
"#,
|
||||||
)];
|
)];
|
||||||
|
|
||||||
let result = run_migrations_slice(&pool, POISON).await;
|
let result = run_migrations_slice(&pool, poison).await;
|
||||||
assert!(
|
assert!(
|
||||||
result.is_err(),
|
result.is_err(),
|
||||||
"poisoned migration must return Err, got: {result:?}"
|
"poisoned migration must return Err, got: {result:?}"
|
||||||
@@ -896,14 +975,14 @@ mod tests {
|
|||||||
"poison_marker must not exist; got: {marker:?}"
|
"poison_marker must not exist; got: {marker:?}"
|
||||||
);
|
);
|
||||||
|
|
||||||
// `schema_version` must not include v10 — version insert is part
|
// `schema_version` must not include the poison version — version
|
||||||
// of the same transaction that rolled back.
|
// insert is part of the same transaction that rolled back.
|
||||||
let max: i64 = sqlx::query_scalar("SELECT COALESCE(MAX(version), 0) FROM schema_version")
|
let max: i64 = sqlx::query_scalar("SELECT COALESCE(MAX(version), 0) FROM schema_version")
|
||||||
.fetch_one(&pool)
|
.fetch_one(&pool)
|
||||||
.await
|
.await
|
||||||
.expect("read schema_version");
|
.expect("read schema_version");
|
||||||
assert_eq!(
|
assert_eq!(
|
||||||
max, 9,
|
max, real_max,
|
||||||
"schema_version must not advance past the failed migration"
|
"schema_version must not advance past the failed migration"
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -158,7 +158,7 @@ mod tests {
|
|||||||
let mut total_pushed: u64 = 0;
|
let mut total_pushed: u64 = 0;
|
||||||
let tentative_per_cycle: u64 = 200;
|
let tentative_per_cycle: u64 = 200;
|
||||||
for _ in 0..100 {
|
for _ in 0..100 {
|
||||||
buf.extend(std::iter::repeat(0.25_f32).take(16_000));
|
buf.extend(std::iter::repeat_n(0.25_f32, 16_000));
|
||||||
total_pushed += 16_000;
|
total_pushed += 16_000;
|
||||||
let commit_point = total_pushed - tentative_per_cycle;
|
let commit_point = total_pushed - tentative_per_cycle;
|
||||||
start = trim_buffer_to_commit_point(&mut buf, start, commit_point);
|
start = trim_buffer_to_commit_point(&mut buf, start, commit_point);
|
||||||
@@ -199,7 +199,7 @@ mod tests {
|
|||||||
|
|
||||||
// Simulate a capture buffer that has received 1.2 s of audio
|
// Simulate a capture buffer that has received 1.2 s of audio
|
||||||
// starting at t=0.
|
// starting at t=0.
|
||||||
let mut buf: Vec<f32> = std::iter::repeat(0.1_f32).take(19_200).collect();
|
let mut buf: Vec<f32> = std::iter::repeat_n(0.1_f32, 19_200).collect();
|
||||||
let new_start = trim_buffer_to_commit_point(&mut buf, 0, commit_idx);
|
let new_start = trim_buffer_to_commit_point(&mut buf, 0, commit_idx);
|
||||||
assert_eq!(new_start, 8_000);
|
assert_eq!(new_start, 8_000);
|
||||||
assert_eq!(buf.len(), 19_200 - 8_000);
|
assert_eq!(buf.len(), 19_200 - 8_000);
|
||||||
|
|||||||
@@ -306,7 +306,7 @@ impl VadChunker for RmsVadChunker {
|
|||||||
.saturating_sub(self.pending.len() as u64);
|
.saturating_sub(self.pending.len() as u64);
|
||||||
let pad_len = FRAME_SAMPLES - self.pending.len();
|
let pad_len = FRAME_SAMPLES - self.pending.len();
|
||||||
let mut padded = std::mem::take(&mut self.pending);
|
let mut padded = std::mem::take(&mut self.pending);
|
||||||
padded.extend(std::iter::repeat(0.0_f32).take(pad_len));
|
padded.extend(std::iter::repeat_n(0.0_f32, pad_len));
|
||||||
if let Some(chunk) = self.consume_frame(padded, frame_start) {
|
if let Some(chunk) = self.consume_frame(padded, frame_start) {
|
||||||
emitted.push(chunk);
|
emitted.push(chunk);
|
||||||
}
|
}
|
||||||
|
|||||||
267
docs/roadmap/2026-04-23-corbie-feature-complete-roadmap.md
Normal file
267
docs/roadmap/2026-04-23-corbie-feature-complete-roadmap.md
Normal file
@@ -0,0 +1,267 @@
|
|||||||
|
---
|
||||||
|
name: Corbie — feature-complete roadmap
|
||||||
|
description: Build plan from 2026-04-23 baseline to full feature-complete v0.1 release
|
||||||
|
type: roadmap
|
||||||
|
tags: [roadmap, planning, corbie, release]
|
||||||
|
created: 2026/04/23
|
||||||
|
status: active
|
||||||
|
author: Wren (CORBEL's resident agent) on behalf of Jake Sames
|
||||||
|
---
|
||||||
|
|
||||||
|
# Corbie — Feature-Complete Roadmap
|
||||||
|
|
||||||
|
> **What Corbie is.** A local-first, cognitive-load-aware dictation + task-capture desktop app. Vulkan-accelerated Whisper / Parakeet speech-to-text, a local LLM (Qwen3 tiers) for transcript cleanup and task extraction, an MCP server for integration with Claude Desktop / Cline / Cursor, and a UI designed around ADHD / executive-dysfunction needs. Tauri 2 + Svelte 5 + Rust. Zero telemetry.
|
||||||
|
>
|
||||||
|
> **Formerly known as Kon.** Rebrand in flight; repo names at `jakejars/kon` + `git.corbel.consulting/jake/kon` still carry the Kon name and will rename together with the codebase sweep in the final phase.
|
||||||
|
|
||||||
|
## Baseline — where we are (2026/04/23)
|
||||||
|
|
||||||
|
**Core MVP (from `docs/brief/feature-set.md`):** 9 of 10 complete.
|
||||||
|
|
||||||
|
One gap: **visual time representation** (the spec's "#1 community-requested feature" — shrinking colour disks / progress rings, externalising time passage). The rest — local transcription, auto-populating tasks, WIP limits, history + search, light/dark theming, templates, vocabulary profiles, file upload, open-format markdown export — all shipped.
|
||||||
|
|
||||||
|
**Post-MVP (designed, not yet prioritised):** 1 of 9 complete.
|
||||||
|
|
||||||
|
MicroSteps is shipped; its "just-start" timer button emits an event that currently has no listener anywhere in the codebase. The differentiating ADHD-specific features (Margot nudges, energy-aware sequencing, rituals, if-then intentions, forgiving gamification, TTS, human-in-the-loop feedback) are all documented in the brief but not started.
|
||||||
|
|
||||||
|
**Release-blockers:** 1 — RB-08 macOS power-assertion, pending manual verification on a real Mac (Jake's friend Rachmann has a Mac and can run this offline).
|
||||||
|
|
||||||
|
**Workspace state:** main is clippy-zero-warnings, 245/245 tests passing, fmt clean, svelte-check clean, npm build clean. Three dependabot bumps landed this session plus a clippy cleanup pass and a needless_range_loop refactor. One orphan design-system WIP branch parked on github as archive.
|
||||||
|
|
||||||
|
## Approach
|
||||||
|
|
||||||
|
> **Layer 1 first** (per Jake's standing rule): build the features roughly, in series, through Phase 1 – Phase 8. Do polish passes in Phase 9 and QC + release in Phase 10. **Do not mix make-it-work and make-it-neat passes** — every phase ships end-to-end (event wired, UI rendered, store state committed, tests updated) but does not chase aesthetic polish until Phase 9.
|
||||||
|
|
||||||
|
> **Ordering rationale.** Phase 1 closes the Core MVP gap and unblocks the already-half-wired just-start timer. Phase 2 enables model improvement by collecting human feedback — useful even while later phases are being built. Phases 3–8 build the differentiating ADHD-specific features from highest-utility-per-effort to lowest. Phase 9 is polish debt. Phase 10 is release prep (including Rachmann's RB-08 verification and the Kon → Corbie codebase rename).
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 1 — Visual countdown + Just-Start timer
|
||||||
|
|
||||||
|
**Why now.** Closes the one remaining Core MVP gap. Unblocks the dangling `kon:start-timer` emit from MicroSteps. Directly combats time blindness, which the brief names as the single biggest lever for the target audience.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- `FocusTimer.svelte` — a progress-ring countdown component. Shrinking colour ring (not digital). Subtle colour shift across the last 15%. Remaining time label inside the ring for users who want the number; small enough that the ring dominates the visual field.
|
||||||
|
- `focusTimer.svelte.ts` store — single active timer, running / paused / completed state, elapsed / remaining computed, event dispatch on state transitions.
|
||||||
|
- Mount in `+layout.svelte` so the timer persists across page navigation (dictation → tasks → settings).
|
||||||
|
- Listener for `window` event `kon:start-timer` with `{ durationSeconds, label }` payload.
|
||||||
|
- Trigger from MicroSteps row (already exists). Trigger from task-row context button (new).
|
||||||
|
- Floating position top-right of main content, above everything, not intrusive when idle (hidden until a timer is running).
|
||||||
|
- On completion: gentle chime + 3-second ring flourish + OS notification via `tauri-plugin-notification`. No modal. No guilt copy.
|
||||||
|
- Store survives window close/reopen via localStorage.
|
||||||
|
|
||||||
|
**Out of scope for Phase 1.** Rhythmic voice anchoring (part of Margot, Phase 6). Custom-duration picker (fixed 2 / 5 / 10 / 15 min presets for now). Multi-timer UI.
|
||||||
|
|
||||||
|
**Acceptance.** From a MicroStep row, clicking the timer button (a) shows the ring visible somewhere on screen, (b) it counts down, (c) at 0 it fires completion + notification, (d) works across page switches, (e) survives a window close + reopen mid-countdown.
|
||||||
|
|
||||||
|
**Estimated effort.** Half day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 2 — Human-in-the-loop feedback
|
||||||
|
|
||||||
|
**Why here.** AI output quality is the single biggest determinant of whether Corbie feels useful. Thumbs-up / thumbs-down on AI-generated micro-steps and task extractions costs almost nothing to add and gives us a feedback corpus the moment anyone uses it. Enables later retraining / prompt tuning.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Thumbs-up / thumbs-down buttons on every AI-generated item (micro-step row, extracted task, cleanup paragraph).
|
||||||
|
- "Edit" path already exists for text; the feedback is additive.
|
||||||
|
- `feedback` table in SQLite: `item_id`, `item_type`, `rating`, `timestamp`, optional `correction_text`.
|
||||||
|
- Rust command `record_feedback` + `list_feedback` (for later export).
|
||||||
|
- No UI surface for viewing feedback yet. Just capture. A future export pass to JSONL feeds prompt-engineering or fine-tuning.
|
||||||
|
|
||||||
|
**Out of scope.** Retraining loop, per-user profile adjustment, any UI to view feedback history.
|
||||||
|
|
||||||
|
**Acceptance.** Thumbs visible on AI-generated items. Clicking records to SQLite. `cargo test` on storage covers migration + insert + list.
|
||||||
|
|
||||||
|
**Estimated effort.** Half day to 1 day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 3 — Energy-aware task sequencing
|
||||||
|
|
||||||
|
**Why here.** Next-highest-utility post-MVP feature. Replaces the cut-for-OS-reasons temptation-bundling feature. Small surface, clear user-facing outcome.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- `energy` column on tasks: nullable enum `High | Medium | BrainDead`.
|
||||||
|
- Migration + Rust CRUD.
|
||||||
|
- Tag chip on task rows; tap to cycle / set.
|
||||||
|
- Sort / filter option on the tasks page: "Match my energy" → AI surfaces tasks matching a user-set current energy, falls back to `Medium` if unset.
|
||||||
|
- No automatic energy detection. Pure user input.
|
||||||
|
|
||||||
|
**Out of scope.** Time-of-day heuristics, calendar integration, AI-predicted user energy.
|
||||||
|
|
||||||
|
**Acceptance.** User tags a task, sets their current energy via a header control, tasks page filter respects it.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 4 — Read Page Aloud (TTS)
|
||||||
|
|
||||||
|
**Why here.** Small and self-contained. Engages auditory processing which the brief specifically calls out as a retention lever for the target audience. Uses OS-native TTS (no new dependencies, no model download). Clean single-tap affordance.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Rust command `tts_speak(text: String, rate: f32, voice: Option<String>)` — platform dispatch:
|
||||||
|
- Linux: `spd-say` (speech-dispatcher is available on most distros; graceful fallback to `espeak` if missing).
|
||||||
|
- macOS: `say` (built in).
|
||||||
|
- Windows: PowerShell `System.Speech.Synthesis.SpeechSynthesizer`.
|
||||||
|
- Small "speaker" icon on any text view (transcript viewer, micro-step list, cleanup result).
|
||||||
|
- Single-tap play; second tap stops. No pause/resume in v1.
|
||||||
|
- Settings: voice picker (populated from OS), rate slider (0.5–2.0).
|
||||||
|
|
||||||
|
**Out of scope.** Premium voices. Cloud TTS. Concurrent-speaking queue. SSML.
|
||||||
|
|
||||||
|
**Acceptance.** Tap speaker icon on a transcript → hear it read on Linux + expected-to-work-on macOS+Windows (test matrix in Phase 10).
|
||||||
|
|
||||||
|
**Estimated effort.** Half day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 5 — Start / shutdown rituals
|
||||||
|
|
||||||
|
**Why here.** Meaningful UX but larger surface. Needs calm copy, gentle flow, and a default-off toggle because rituals can feel parental if not optional.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Morning triage: on first launch after 06:00, show a modal / dedicated page: "yesterday's incomplete tasks" (from SQLite query: `completed = false AND created_at < today`), with checkbox pick-list, and "pick 1–3 for today" constraint that refuses selections > 3.
|
||||||
|
- Evening shutdown: user-triggerable (not scheduled) review: "what got done today", "open loops to close", "separate work from rest" copy. No automation; ritual as reflection.
|
||||||
|
- Both off by default in settings. When off, no modal, no pressure.
|
||||||
|
- Skip-for-today button on morning triage; never shows guilt copy.
|
||||||
|
|
||||||
|
**Out of scope.** Calendar integration, automatic sleep detection, weekly / monthly reviews.
|
||||||
|
|
||||||
|
**Acceptance.** Morning modal shows correct tasks. Selecting > 3 is prevented with a gentle message. Skip works and doesn't re-prompt same day. Evening shutdown opens a reflective page, doesn't block closing the app.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 – 2 days.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 6 — Soft-touch nudging (Margot protocol)
|
||||||
|
|
||||||
|
**Why here.** This is the big differentiator and it's intentionally scheduled late because it depends on the phases before it (it nudges *about* tasks, timers, rituals) and on a careful copy pass to not feel like a push notification. The brief explicitly calls out: reminders must not function as standard push notifications; they must be anticipatory guidance.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Context-aware trigger engine in Rust: watches activity signals (keyboard activity, active window, last-interaction timestamp) and fires `nudge` events when triggers match.
|
||||||
|
- Trigger set for v1: `inactivity_with_active_timer` (timer running, no UI interaction for 90 s), `pending_morning_triage` (past 10:00 and triage untouched), `micro_step_idle` (micro-step generated, not acted on within 15 min).
|
||||||
|
- Delivery: OS notification via `tauri-plugin-notification`. Platform sounds: `Glass` on macOS, `message-new-instant` on Linux, `Default` on Windows. Haptic cue on mobile (not yet in scope; desktop first).
|
||||||
|
- Suppression rules: no nudge if user typed in last 5 s, no nudge during a running timer's first 60 s, hard cap at 3 nudges per hour, instant mute button in settings.
|
||||||
|
- Rhythmic voice anchoring: piggy-back on Phase 4 TTS. Optional "speak nudges aloud" toggle. Default off. When on, short calm lines ("Time to move on", "Your list is still here"). No branding voice, no personality yet — that's a Phase 9 polish item.
|
||||||
|
|
||||||
|
**Out of scope.** Custom trigger editor (user-facing rule UI is Phase 7). Cross-device delivery. Biometric signals (HRV, fidget detection, etc.). Any Margot-as-character visual.
|
||||||
|
|
||||||
|
**Acceptance.** Nudges fire on each trigger in a dogfood walkthrough. Suppression rules observed. Mute button kills everything immediately.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 – 2 days.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 7 — Implementation intentions (if-then automation)
|
||||||
|
|
||||||
|
**Why here.** Leans on Phase 6's trigger engine. The user-facing rule editor is its own thing, but the execution path is the nudge pipeline with user-defined conditions.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Rule editor UI: minimal. `if [when-condition], then [action]`.
|
||||||
|
- When-conditions for v1: `time of day = HH:MM`, `after a task completes`, `morning triage finishes`.
|
||||||
|
- Actions for v1: `surface task X`, `start a 5-min timer`, `speak a line aloud`.
|
||||||
|
- Rules stored in SQLite. On/off per rule. Global mute respected.
|
||||||
|
- No location triggers (desktop app, no geolocation). No app-running detection in v1 (fragile cross-platform; revisit in v0.2).
|
||||||
|
|
||||||
|
**Out of scope.** Calendar triggers, cross-app automation, macro-style action chains, shared/community rules.
|
||||||
|
|
||||||
|
**Acceptance.** User can write "at 09:00, speak 'time to plan the day' and surface inbox", save it, and have it fire next morning at 09:00. Delete works.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 8 — Forgiving gamification
|
||||||
|
|
||||||
|
**Why here.** Last feature phase because it's the lowest-risk and the furthest from make-or-break. Neatly rounds out the spec list.
|
||||||
|
|
||||||
|
**Scope.**
|
||||||
|
- Completion count per day (non-punitive: no streaks, no chain-breaking). "You've completed 4 tasks today. Three in the afternoon. Want to call it?"
|
||||||
|
- Grace days: the badge logic ignores up-to-3 consecutive days of no activity without reset.
|
||||||
|
- Visual: soft-edged numeric badges on the tasks header, no leaderboards, no social comparison.
|
||||||
|
- Zero loss language. Never "you lost your streak". Framing is always "look what you did".
|
||||||
|
|
||||||
|
**Out of scope.** Leaderboards. Shared challenges. Streak repair purchases. XP systems.
|
||||||
|
|
||||||
|
**Acceptance.** Complete 3 tasks → header shows "3 today". Open the app after 4 days off → no negative framing, header reads today's count only.
|
||||||
|
|
||||||
|
**Estimated effort.** Half day to 1 day.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 9 — Polish debt
|
||||||
|
|
||||||
|
> **All Phase 9 work is paused until Phase 1 – Phase 8 are closed.** Per Jake's rule: features first, polish second.
|
||||||
|
|
||||||
|
**Contents.**
|
||||||
|
- File-system `.md` save dialog (replace clipboard-only export). Rust `write_text_file` command; platform dialog via `tauri-plugin-dialog`.
|
||||||
|
- Bulk select + bulk export in History.
|
||||||
|
- LLM-powered content tags (`topic:*`, `intent:*`). Slot into the existing `kon-llm` stub.
|
||||||
|
- Settings UX overhaul: bundle high-traffic settings into a "Start here" group; hide advanced behind a disclosure.
|
||||||
|
- Visual polish pass on all Phase 1 – Phase 8 surfaces: spacing, typography, motion curves, colour, dark-mode parity.
|
||||||
|
- Accessibility pass: keyboard navigation, screen reader labels, focus order, colour contrast audit against WCAG AA.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 – 2 days.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Phase 10 — QC + rename + release
|
||||||
|
|
||||||
|
**Prerequisite:** Phase 1 – Phase 9 complete.
|
||||||
|
|
||||||
|
**QC block.**
|
||||||
|
- Full dogfood walkthrough: record a real brain-dump → clean transcript → task extraction → micro-step one task → run a focus timer → tag energy → complete → check evening shutdown.
|
||||||
|
- RB-08 macOS power-assertion verification: **Rachmann runs this offline** on his Mac. He runs `pmset -g assertions` while a live session is active; expected: entry for `PreventSystemSleep` attributed to Corbie's bundle id. Once confirmed, mark RB-08 closed and delete `docs/issues/power-assertion-macos-objc2.md` or move to resolved.
|
||||||
|
- Cross-platform build matrix (CI already runs): Linux / macOS / Windows, ensure all three are green.
|
||||||
|
- Accessibility regression check: keyboard-only traversal of each new surface.
|
||||||
|
- Freshly-clean install test on a spare user account: no stray data leaks from dev.
|
||||||
|
|
||||||
|
**Kon → Corbie codebase rename sweep.** Runs after QC once Jake has renamed the two repos:
|
||||||
|
- `package.json` name `kon@0.1.0` → `corbie@0.1.0`.
|
||||||
|
- Cargo crate names: `kon`, `kon-audio`, `kon-storage`, `kon-transcription`, `kon-llm`, `kon-ai-formatting`, `kon-core`, `kon-cloud-providers`, `kon-hotkey`, `kon-mcp` → `corbie-*`. Mass-rename via `Cargo.toml` + `use`-path sweep.
|
||||||
|
- Binary + product names: `src-tauri/tauri.conf.json`, bundle identifier, `.desktop` file, Windows product name, macOS bundle name.
|
||||||
|
- Install paths: `~/.local/share/kon/` → `~/.local/share/corbie/`. **Migration shim required**: first-run check for old dir, move contents, write a sentinel. Document in the release notes.
|
||||||
|
- Database filename: `kon.db` → `corbie.db`. Handled by the same migration shim.
|
||||||
|
- Window titles, tray tooltip, About-dialog text, README body, docs/brief/ references where they refer to the product (leave historical brief content talking about "Kon" as-is — it's a historical document).
|
||||||
|
- Event names: `kon:start-timer` → `corbie:start-timer` and similar. Kept `kon:` through Phase 1 – Phase 9 so any dogfood doesn't need to re-learn them mid-cycle.
|
||||||
|
- Logs, error messages, user-facing copy.
|
||||||
|
- Remotes: `ssh://git.corbel.consulting:2222/jake/kon.git` + `github.com:jakejars/kon.git` → `…/corbie.git` on both, after Jake has clicked rename in the web UIs. Update `git remote set-url` locally.
|
||||||
|
|
||||||
|
**Release.**
|
||||||
|
- Bump `Cargo.toml` and `package.json` to `0.1.0`. Tag `v0.1.0` on the commit.
|
||||||
|
- Write `CHANGELOG.md` (seed it with everything from this roadmap's phases).
|
||||||
|
- Write release notes in plain language — what it does, who it's for, the data-migration note.
|
||||||
|
- Push tag to both remotes. GitHub Actions release workflow (already in place for cross-platform CI) should auto-build artefacts for Linux / macOS / Windows.
|
||||||
|
|
||||||
|
**Estimated effort.** 1 day (QC + rename + release ceremony), plus Rachmann's slot on his Mac (parallel, not blocking).
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Totals
|
||||||
|
|
||||||
|
- Phase 1 – 8 feature build: **6.5 – 9.5 days** of focused work
|
||||||
|
- Phase 9 polish: **1 – 2 days**
|
||||||
|
- Phase 10 QC + rename + release: **1 day + Rachmann's Mac session**
|
||||||
|
|
||||||
|
**Total to v0.1.0 feature-complete release:** **~9 – 13 days of focused work**, depending on how much polish time Jake wants in Phase 9.
|
||||||
|
|
||||||
|
## Explicit non-goals
|
||||||
|
|
||||||
|
- Mobile apps. Corbie is desktop-first; a mobile companion is post-v0.1.
|
||||||
|
- Cloud sync. Local-first is the floor, not a feature. Sync is out of scope through v0.1.
|
||||||
|
- Premium voices, paid tiers, subscription. Licensing + monetisation is a separate track tracked in memory `project_marketplace_creem`.
|
||||||
|
- AI body doubling (low-fi focus rooms) — validated but parked to post-v0.1.
|
||||||
|
- Temptation bundling — cut (OS-integration impossible cross-platform; replaced by Phase 3 energy-aware sequencing).
|
||||||
|
|
||||||
|
## Anchors
|
||||||
|
|
||||||
|
- Spec: [docs/brief/feature-set.md](docs/brief/feature-set.md) + [docs/brief/design-principles.md](docs/brief/design-principles.md)
|
||||||
|
- Current baseline: this session's HANDOVER.md
|
||||||
|
- Rebrand memory: `~/.claude/projects/-home-jake-Documents-CORBEL-Main/memory/project_corbie_rebrand.md`
|
||||||
|
- Release-blocker index: [docs/issues/README.md](docs/issues/README.md)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
*This roadmap is a living document. Update it at the end of each phase with actuals vs estimates and any scope revisions.*
|
||||||
BIN
docs/roadmap/phase1-focus-timer-screenshot.png
Normal file
BIN
docs/roadmap/phase1-focus-timer-screenshot.png
Normal file
Binary file not shown.
|
After Width: | Height: | Size: 259 KiB |
90
package-lock.json
generated
90
package-lock.json
generated
@@ -18,8 +18,8 @@
|
|||||||
"svelte-i18n": "^4.0.1"
|
"svelte-i18n": "^4.0.1"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@sveltejs/adapter-static": "^3.0.6",
|
"@sveltejs/adapter-static": "^3.0.10",
|
||||||
"@sveltejs/kit": "^2.9.0",
|
"@sveltejs/kit": "^2.58.0",
|
||||||
"@sveltejs/vite-plugin-svelte": "^5.0.0",
|
"@sveltejs/vite-plugin-svelte": "^5.0.0",
|
||||||
"@tailwindcss/vite": "^4.2.1",
|
"@tailwindcss/vite": "^4.2.1",
|
||||||
"@tauri-apps/cli": "^2",
|
"@tauri-apps/cli": "^2",
|
||||||
@@ -27,7 +27,7 @@
|
|||||||
"svelte-check": "^4.0.0",
|
"svelte-check": "^4.0.0",
|
||||||
"tailwindcss": "^4.2.1",
|
"tailwindcss": "^4.2.1",
|
||||||
"typescript": "~5.6.2",
|
"typescript": "~5.6.2",
|
||||||
"vite": "^6.0.3"
|
"vite": "^6.4.2"
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"node_modules/@chenglou/pretext": {
|
"node_modules/@chenglou/pretext": {
|
||||||
@@ -958,9 +958,9 @@
|
|||||||
}
|
}
|
||||||
},
|
},
|
||||||
"node_modules/@sveltejs/kit": {
|
"node_modules/@sveltejs/kit": {
|
||||||
"version": "2.55.0",
|
"version": "2.58.0",
|
||||||
"resolved": "https://registry.npmjs.org/@sveltejs/kit/-/kit-2.55.0.tgz",
|
"resolved": "https://registry.npmjs.org/@sveltejs/kit/-/kit-2.58.0.tgz",
|
||||||
"integrity": "sha512-MdFRjevVxmAknf2NbaUkDF16jSIzXMWd4Nfah0Qp8TtQVoSp3bV4jKt8mX7z7qTUTWvgSaxtR0EG5WJf53gcuA==",
|
"integrity": "sha512-kT9GCN8yJTkCK1W+Gi/bvGooWAM7y7WXP+yd+rf6QOIjyoK1ERPrMwSufXJUNu2pMWIqruhFvmz+LbOqsEmKmA==",
|
||||||
"dev": true,
|
"dev": true,
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
@@ -987,7 +987,7 @@
|
|||||||
"@opentelemetry/api": "^1.0.0",
|
"@opentelemetry/api": "^1.0.0",
|
||||||
"@sveltejs/vite-plugin-svelte": "^3.0.0 || ^4.0.0-next.1 || ^5.0.0 || ^6.0.0-next.0 || ^7.0.0",
|
"@sveltejs/vite-plugin-svelte": "^3.0.0 || ^4.0.0-next.1 || ^5.0.0 || ^6.0.0-next.0 || ^7.0.0",
|
||||||
"svelte": "^4.0.0 || ^5.0.0-next.0",
|
"svelte": "^4.0.0 || ^5.0.0-next.0",
|
||||||
"typescript": "^5.3.3",
|
"typescript": "^5.3.3 || ^6.0.0",
|
||||||
"vite": "^5.0.3 || ^6.0.0 || ^7.0.0-beta.0 || ^8.0.0"
|
"vite": "^5.0.3 || ^6.0.0 || ^7.0.0-beta.0 || ^8.0.0"
|
||||||
},
|
},
|
||||||
"peerDependenciesMeta": {
|
"peerDependenciesMeta": {
|
||||||
@@ -1262,6 +1262,70 @@
|
|||||||
"node": ">=14.0.0"
|
"node": ">=14.0.0"
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/@emnapi/core": {
|
||||||
|
"version": "1.8.1",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "MIT",
|
||||||
|
"optional": true,
|
||||||
|
"dependencies": {
|
||||||
|
"@emnapi/wasi-threads": "1.1.0",
|
||||||
|
"tslib": "^2.4.0"
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/@emnapi/runtime": {
|
||||||
|
"version": "1.8.1",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "MIT",
|
||||||
|
"optional": true,
|
||||||
|
"dependencies": {
|
||||||
|
"tslib": "^2.4.0"
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/@emnapi/wasi-threads": {
|
||||||
|
"version": "1.1.0",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "MIT",
|
||||||
|
"optional": true,
|
||||||
|
"dependencies": {
|
||||||
|
"tslib": "^2.4.0"
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/@napi-rs/wasm-runtime": {
|
||||||
|
"version": "1.1.1",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "MIT",
|
||||||
|
"optional": true,
|
||||||
|
"dependencies": {
|
||||||
|
"@emnapi/core": "^1.7.1",
|
||||||
|
"@emnapi/runtime": "^1.7.1",
|
||||||
|
"@tybys/wasm-util": "^0.10.1"
|
||||||
|
},
|
||||||
|
"funding": {
|
||||||
|
"type": "github",
|
||||||
|
"url": "https://github.com/sponsors/Brooooooklyn"
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/@tybys/wasm-util": {
|
||||||
|
"version": "0.10.1",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "MIT",
|
||||||
|
"optional": true,
|
||||||
|
"dependencies": {
|
||||||
|
"tslib": "^2.4.0"
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"node_modules/@tailwindcss/oxide-wasm32-wasi/node_modules/tslib": {
|
||||||
|
"version": "2.8.1",
|
||||||
|
"dev": true,
|
||||||
|
"inBundle": true,
|
||||||
|
"license": "0BSD",
|
||||||
|
"optional": true
|
||||||
|
},
|
||||||
"node_modules/@tailwindcss/oxide-win32-arm64-msvc": {
|
"node_modules/@tailwindcss/oxide-win32-arm64-msvc": {
|
||||||
"version": "4.2.1",
|
"version": "4.2.1",
|
||||||
"resolved": "https://registry.npmjs.org/@tailwindcss/oxide-win32-arm64-msvc/-/oxide-win32-arm64-msvc-4.2.1.tgz",
|
"resolved": "https://registry.npmjs.org/@tailwindcss/oxide-win32-arm64-msvc/-/oxide-win32-arm64-msvc-4.2.1.tgz",
|
||||||
@@ -2375,9 +2439,9 @@
|
|||||||
"license": "ISC"
|
"license": "ISC"
|
||||||
},
|
},
|
||||||
"node_modules/picomatch": {
|
"node_modules/picomatch": {
|
||||||
"version": "4.0.3",
|
"version": "4.0.4",
|
||||||
"resolved": "https://registry.npmjs.org/picomatch/-/picomatch-4.0.3.tgz",
|
"resolved": "https://registry.npmjs.org/picomatch/-/picomatch-4.0.4.tgz",
|
||||||
"integrity": "sha512-5gTmgEY/sqK6gFXLIsQNH19lWb4ebPDLA4SdLP7dsWkIXHWlG66oPuVvXSGFPppYZz8ZDZq0dYYrbHfBCVUb1Q==",
|
"integrity": "sha512-QP88BAKvMam/3NxH6vj2o21R6MjxZUAd6nlwAS/pnGvN9IVLocLHxGYIzFhg6fUQ+5th6P4dv4eW9jX3DSIj7A==",
|
||||||
"dev": true,
|
"dev": true,
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"engines": {
|
"engines": {
|
||||||
@@ -3098,9 +3162,9 @@
|
|||||||
}
|
}
|
||||||
},
|
},
|
||||||
"node_modules/vite": {
|
"node_modules/vite": {
|
||||||
"version": "6.4.1",
|
"version": "6.4.2",
|
||||||
"resolved": "https://registry.npmjs.org/vite/-/vite-6.4.1.tgz",
|
"resolved": "https://registry.npmjs.org/vite/-/vite-6.4.2.tgz",
|
||||||
"integrity": "sha512-+Oxm7q9hDoLMyJOYfUYBuHQo+dkAloi33apOPP56pzj+vsdJDzr+j1NISE5pyaAuKL4A3UD34qd0lx5+kfKp2g==",
|
"integrity": "sha512-2N/55r4JDJ4gdrCvGgINMy+HH3iRpNIz8K6SFwVsA+JbQScLiC+clmAxBgwiSPgcG9U15QmvqCGWzMbqda5zGQ==",
|
||||||
"dev": true,
|
"dev": true,
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
|
|||||||
@@ -23,8 +23,8 @@
|
|||||||
"svelte-i18n": "^4.0.1"
|
"svelte-i18n": "^4.0.1"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@sveltejs/adapter-static": "^3.0.6",
|
"@sveltejs/adapter-static": "^3.0.10",
|
||||||
"@sveltejs/kit": "^2.9.0",
|
"@sveltejs/kit": "^2.58.0",
|
||||||
"@sveltejs/vite-plugin-svelte": "^5.0.0",
|
"@sveltejs/vite-plugin-svelte": "^5.0.0",
|
||||||
"@tailwindcss/vite": "^4.2.1",
|
"@tailwindcss/vite": "^4.2.1",
|
||||||
"@tauri-apps/cli": "^2",
|
"@tauri-apps/cli": "^2",
|
||||||
@@ -32,6 +32,6 @@
|
|||||||
"svelte-check": "^4.0.0",
|
"svelte-check": "^4.0.0",
|
||||||
"tailwindcss": "^4.2.1",
|
"tailwindcss": "^4.2.1",
|
||||||
"typescript": "~5.6.2",
|
"typescript": "~5.6.2",
|
||||||
"vite": "^6.0.3"
|
"vite": "^6.4.2"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -64,7 +64,7 @@ fn assert_localhost_llm_csp() {
|
|||||||
let tokens: Vec<&str> = connect_src.split_whitespace().collect();
|
let tokens: Vec<&str> = connect_src.split_whitespace().collect();
|
||||||
for required in ["http://127.0.0.1:*", "ws://127.0.0.1:*"] {
|
for required in ["http://127.0.0.1:*", "ws://127.0.0.1:*"] {
|
||||||
assert!(
|
assert!(
|
||||||
tokens.iter().any(|t| *t == required),
|
tokens.contains(&required),
|
||||||
"build.rs: tauri.conf.json CSP connect-src must permit {required} \
|
"build.rs: tauri.conf.json CSP connect-src must permit {required} \
|
||||||
for local LLM connectivity (brief item #2). Current connect-src: \
|
for local LLM connectivity (brief item #2). Current connect-src: \
|
||||||
{connect_src:?}"
|
{connect_src:?}"
|
||||||
|
|||||||
@@ -247,7 +247,7 @@ pub async fn generate_diagnostic_report(
|
|||||||
.map(|d| d.as_secs())
|
.map(|d| d.as_secs())
|
||||||
.unwrap_or(0);
|
.unwrap_or(0);
|
||||||
out.push_str(&format!("- Generated: unix `{}`\n", now));
|
out.push_str(&format!("- Generated: unix `{}`\n", now));
|
||||||
out.push_str("\n");
|
out.push('\n');
|
||||||
out.push_str(
|
out.push_str(
|
||||||
"> This report is local-only until you choose to share it. \
|
"> This report is local-only until you choose to share it. \
|
||||||
Review the contents below before sending to anyone.\n\n",
|
Review the contents below before sending to anyone.\n\n",
|
||||||
|
|||||||
110
src-tauri/src/commands/feedback.rs
Normal file
110
src-tauri/src/commands/feedback.rs
Normal file
@@ -0,0 +1,110 @@
|
|||||||
|
// Tauri commands for human-in-the-loop feedback capture and retrieval.
|
||||||
|
// Phase 2 of the feature-complete roadmap: thumbs + correction capture
|
||||||
|
// on AI-generated output feeds a few-shot loop that conditions future
|
||||||
|
// prompts on the user's preferred style.
|
||||||
|
|
||||||
|
use serde::{Deserialize, Serialize};
|
||||||
|
|
||||||
|
use kon_storage::{
|
||||||
|
list_feedback_examples as db_list_feedback_examples, record_feedback as db_record_feedback,
|
||||||
|
FeedbackRow, FeedbackTargetType, RecordFeedbackParams,
|
||||||
|
};
|
||||||
|
|
||||||
|
use crate::AppState;
|
||||||
|
|
||||||
|
#[derive(Debug, Clone, Deserialize)]
|
||||||
|
#[serde(rename_all = "camelCase")]
|
||||||
|
pub struct RecordFeedbackInput {
|
||||||
|
/// One of "microstep", "task_extraction", "cleanup".
|
||||||
|
pub target_type: String,
|
||||||
|
/// Optional surface-specific id (subtask id, task id, transcript id).
|
||||||
|
#[serde(default)]
|
||||||
|
pub target_id: Option<String>,
|
||||||
|
/// -1 = thumbs down, 0 = correction (neutral), +1 = thumbs up.
|
||||||
|
pub rating: i8,
|
||||||
|
#[serde(default)]
|
||||||
|
pub original_text: Option<String>,
|
||||||
|
#[serde(default)]
|
||||||
|
pub corrected_text: Option<String>,
|
||||||
|
/// Freeform JSON context: e.g. the parent task text, the transcript
|
||||||
|
/// chunk the AI was given, etc. Used later by the prompt builder
|
||||||
|
/// to reconstruct the (input, preferred-output) pair.
|
||||||
|
#[serde(default)]
|
||||||
|
pub context_json: Option<String>,
|
||||||
|
#[serde(default)]
|
||||||
|
pub profile_id: Option<String>,
|
||||||
|
}
|
||||||
|
|
||||||
|
#[derive(Debug, Clone, Serialize)]
|
||||||
|
#[serde(rename_all = "camelCase")]
|
||||||
|
pub struct FeedbackDto {
|
||||||
|
pub id: i64,
|
||||||
|
pub target_type: String,
|
||||||
|
pub target_id: Option<String>,
|
||||||
|
pub rating: i64,
|
||||||
|
pub original_text: Option<String>,
|
||||||
|
pub corrected_text: Option<String>,
|
||||||
|
pub context_json: Option<String>,
|
||||||
|
pub profile_id: String,
|
||||||
|
pub created_at: String,
|
||||||
|
}
|
||||||
|
|
||||||
|
impl From<FeedbackRow> for FeedbackDto {
|
||||||
|
fn from(r: FeedbackRow) -> Self {
|
||||||
|
Self {
|
||||||
|
id: r.id,
|
||||||
|
target_type: r.target_type,
|
||||||
|
target_id: r.target_id,
|
||||||
|
rating: r.rating,
|
||||||
|
original_text: r.original_text,
|
||||||
|
corrected_text: r.corrected_text,
|
||||||
|
context_json: r.context_json,
|
||||||
|
profile_id: r.profile_id,
|
||||||
|
created_at: r.created_at,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fn parse_target_type(raw: &str) -> Result<FeedbackTargetType, String> {
|
||||||
|
FeedbackTargetType::parse(raw).ok_or_else(|| format!("unknown feedback target_type: {raw}"))
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tauri::command]
|
||||||
|
pub async fn record_feedback(
|
||||||
|
state: tauri::State<'_, AppState>,
|
||||||
|
input: RecordFeedbackInput,
|
||||||
|
) -> Result<i64, String> {
|
||||||
|
let target_type = parse_target_type(&input.target_type)?;
|
||||||
|
db_record_feedback(
|
||||||
|
&state.db,
|
||||||
|
RecordFeedbackParams {
|
||||||
|
target_type,
|
||||||
|
target_id: input.target_id,
|
||||||
|
rating: input.rating,
|
||||||
|
original_text: input.original_text,
|
||||||
|
corrected_text: input.corrected_text,
|
||||||
|
context_json: input.context_json,
|
||||||
|
profile_id: input.profile_id,
|
||||||
|
},
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.map_err(|e| e.to_string())
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tauri::command]
|
||||||
|
pub async fn list_feedback_examples_cmd(
|
||||||
|
state: tauri::State<'_, AppState>,
|
||||||
|
target_type: String,
|
||||||
|
limit: Option<i64>,
|
||||||
|
min_rating: Option<i8>,
|
||||||
|
profile_id: Option<String>,
|
||||||
|
) -> Result<Vec<FeedbackDto>, String> {
|
||||||
|
let target = parse_target_type(&target_type)?;
|
||||||
|
let limit = limit.unwrap_or(8).clamp(1, 64);
|
||||||
|
let min_rating = min_rating.unwrap_or(0).clamp(-1, 1);
|
||||||
|
let rows =
|
||||||
|
db_list_feedback_examples(&state.db, target, limit, min_rating, profile_id.as_deref())
|
||||||
|
.await
|
||||||
|
.map_err(|e| e.to_string())?;
|
||||||
|
Ok(rows.into_iter().map(FeedbackDto::from).collect())
|
||||||
|
}
|
||||||
@@ -1086,11 +1086,12 @@ fn build_nearby_transcript_candidates(
|
|||||||
let mut texts: Vec<String> = Vec::new();
|
let mut texts: Vec<String> = Vec::new();
|
||||||
for start in 0..nearby.len() {
|
for start in 0..nearby.len() {
|
||||||
let mut merged = String::new();
|
let mut merged = String::new();
|
||||||
for end in start..nearby.len().min(start + DUPLICATE_TRANSCRIPT_MERGE_LIMIT) {
|
let upper = nearby.len().min(start + DUPLICATE_TRANSCRIPT_MERGE_LIMIT);
|
||||||
|
for segment in &nearby[start..upper] {
|
||||||
if !merged.is_empty() {
|
if !merged.is_empty() {
|
||||||
merged.push(' ');
|
merged.push(' ');
|
||||||
}
|
}
|
||||||
merged.push_str(nearby[end].text.trim());
|
merged.push_str(segment.text.trim());
|
||||||
if !texts.iter().any(|existing| existing == &merged) {
|
if !texts.iter().any(|existing| existing == &merged) {
|
||||||
texts.push(merged.clone());
|
texts.push(merged.clone());
|
||||||
}
|
}
|
||||||
@@ -1155,12 +1156,10 @@ fn longest_common_token_subsequence(a: &[&str], b: &[&str]) -> usize {
|
|||||||
}
|
}
|
||||||
|
|
||||||
fn is_low_signal_token(token: &str) -> bool {
|
fn is_low_signal_token(token: &str) -> bool {
|
||||||
LOW_SIGNAL_TOKENS
|
LOW_SIGNAL_TOKENS.contains(&token)
|
||||||
.iter()
|
|
||||||
.any(|low_signal| *low_signal == token)
|
|
||||||
}
|
}
|
||||||
|
|
||||||
fn meaningful_tokens<'a>(text: &'a str) -> Vec<&'a str> {
|
fn meaningful_tokens(text: &str) -> Vec<&str> {
|
||||||
text.split_whitespace()
|
text.split_whitespace()
|
||||||
.filter(|token| !token.is_empty() && token.len() > 1 && !is_low_signal_token(token))
|
.filter(|token| !token.is_empty() && token.len() > 1 && !is_low_signal_token(token))
|
||||||
.collect()
|
.collect()
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
pub mod audio;
|
pub mod audio;
|
||||||
pub mod clipboard;
|
pub mod clipboard;
|
||||||
pub mod diagnostics;
|
pub mod diagnostics;
|
||||||
|
pub mod feedback;
|
||||||
pub mod hardware;
|
pub mod hardware;
|
||||||
pub mod hotkey;
|
pub mod hotkey;
|
||||||
pub mod live;
|
pub mod live;
|
||||||
|
|||||||
@@ -412,7 +412,7 @@ pub fn detect_active_compute_device() -> ActiveComputeDevice {
|
|||||||
reason: None,
|
reason: None,
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
return ActiveComputeDevice {
|
ActiveComputeDevice {
|
||||||
kind: "cpu".into(),
|
kind: "cpu".into(),
|
||||||
label: "CPU (fallback)".into(),
|
label: "CPU (fallback)".into(),
|
||||||
reason: Some(
|
reason: Some(
|
||||||
@@ -420,7 +420,7 @@ pub fn detect_active_compute_device() -> ActiveComputeDevice {
|
|||||||
libvulkan1 (Linux) to enable GPU acceleration."
|
libvulkan1 (Linux) to enable GPU acceleration."
|
||||||
.into(),
|
.into(),
|
||||||
),
|
),
|
||||||
};
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -345,7 +345,7 @@ fn classify_terminal(raw: &str) -> Option<String> {
|
|||||||
fn detect_focused_window_class() -> Option<String> {
|
fn detect_focused_window_class() -> Option<String> {
|
||||||
#[cfg(target_os = "linux")]
|
#[cfg(target_os = "linux")]
|
||||||
{
|
{
|
||||||
return detect_focused_window_class_linux();
|
detect_focused_window_class_linux()
|
||||||
}
|
}
|
||||||
#[cfg(target_os = "macos")]
|
#[cfg(target_os = "macos")]
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -6,12 +6,15 @@ use serde::{Deserialize, Serialize};
|
|||||||
|
|
||||||
use uuid::Uuid;
|
use uuid::Uuid;
|
||||||
|
|
||||||
|
use kon_llm::prompts::FeedbackExample as LlmFeedbackExample;
|
||||||
use kon_storage::{
|
use kon_storage::{
|
||||||
complete_subtask_and_check_parent as db_complete_subtask, complete_task as db_complete_task,
|
complete_subtask_and_check_parent as db_complete_subtask, complete_task as db_complete_task,
|
||||||
delete_task as db_delete_task, get_task_by_id as db_get_task,
|
delete_task as db_delete_task, get_task_by_id as db_get_task,
|
||||||
insert_subtask as db_insert_subtask, insert_task as db_insert_task,
|
insert_subtask as db_insert_subtask, insert_task as db_insert_task,
|
||||||
list_subtasks as db_list_subtasks, list_tasks as db_list_tasks,
|
list_feedback_examples as db_list_feedback_examples, list_subtasks as db_list_subtasks,
|
||||||
uncomplete_task as db_uncomplete_task, update_task as db_update_task, TaskRow,
|
list_tasks as db_list_tasks, set_task_energy as db_set_task_energy,
|
||||||
|
uncomplete_task as db_uncomplete_task, update_task as db_update_task, FeedbackRow,
|
||||||
|
FeedbackTargetType, TaskRow,
|
||||||
};
|
};
|
||||||
|
|
||||||
use crate::AppState;
|
use crate::AppState;
|
||||||
@@ -31,6 +34,7 @@ pub struct TaskDto {
|
|||||||
pub created_at: String,
|
pub created_at: String,
|
||||||
pub source_transcript_id: Option<String>,
|
pub source_transcript_id: Option<String>,
|
||||||
pub parent_task_id: Option<String>,
|
pub parent_task_id: Option<String>,
|
||||||
|
pub energy: Option<String>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl From<TaskRow> for TaskDto {
|
impl From<TaskRow> for TaskDto {
|
||||||
@@ -47,10 +51,27 @@ impl From<TaskRow> for TaskDto {
|
|||||||
created_at: r.created_at,
|
created_at: r.created_at,
|
||||||
source_transcript_id: r.source_transcript_id,
|
source_transcript_id: r.source_transcript_id,
|
||||||
parent_task_id: r.parent_task_id,
|
parent_task_id: r.parent_task_id,
|
||||||
|
energy: r.energy,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Accepted energy tag values. Kept as a const so frontend and storage
|
||||||
|
/// validate against the same list. Migration v11 enforces the same
|
||||||
|
/// set via a CHECK constraint.
|
||||||
|
const ENERGY_LEVELS: &[&str] = &["high", "medium", "brain_dead"];
|
||||||
|
|
||||||
|
fn validate_energy(raw: Option<&str>) -> Result<Option<&str>, String> {
|
||||||
|
match raw {
|
||||||
|
None => Ok(None),
|
||||||
|
Some(s) if ENERGY_LEVELS.contains(&s) => Ok(Some(s)),
|
||||||
|
Some(other) => Err(format!(
|
||||||
|
"energy must be one of {:?} or null, got {:?}",
|
||||||
|
ENERGY_LEVELS, other
|
||||||
|
)),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
#[derive(Debug, Deserialize)]
|
#[derive(Debug, Deserialize)]
|
||||||
#[serde(rename_all = "camelCase")]
|
#[serde(rename_all = "camelCase")]
|
||||||
pub struct CreateTaskRequest {
|
pub struct CreateTaskRequest {
|
||||||
@@ -63,6 +84,8 @@ pub struct CreateTaskRequest {
|
|||||||
pub list_id: Option<String>,
|
pub list_id: Option<String>,
|
||||||
#[serde(default)]
|
#[serde(default)]
|
||||||
pub effort: Option<String>,
|
pub effort: Option<String>,
|
||||||
|
#[serde(default)]
|
||||||
|
pub energy: Option<String>,
|
||||||
}
|
}
|
||||||
|
|
||||||
#[tauri::command]
|
#[tauri::command]
|
||||||
@@ -70,6 +93,7 @@ pub async fn create_task_cmd(
|
|||||||
state: tauri::State<'_, AppState>,
|
state: tauri::State<'_, AppState>,
|
||||||
request: CreateTaskRequest,
|
request: CreateTaskRequest,
|
||||||
) -> Result<TaskDto, String> {
|
) -> Result<TaskDto, String> {
|
||||||
|
let energy = validate_energy(request.energy.as_deref())?;
|
||||||
db_insert_task(
|
db_insert_task(
|
||||||
&state.db,
|
&state.db,
|
||||||
&request.id,
|
&request.id,
|
||||||
@@ -78,6 +102,7 @@ pub async fn create_task_cmd(
|
|||||||
request.source_transcript_id.as_deref(),
|
request.source_transcript_id.as_deref(),
|
||||||
request.list_id.as_deref(),
|
request.list_id.as_deref(),
|
||||||
request.effort.as_deref(),
|
request.effort.as_deref(),
|
||||||
|
energy,
|
||||||
)
|
)
|
||||||
.await
|
.await
|
||||||
.map_err(|e| e.to_string())?;
|
.map_err(|e| e.to_string())?;
|
||||||
@@ -166,19 +191,135 @@ pub async fn uncomplete_task_cmd(
|
|||||||
.map_err(|e| e.to_string())
|
.map_err(|e| e.to_string())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Phase 3: set or clear the `energy` tag on a task. Dedicated command
|
||||||
|
/// rather than a field on `update_task_cmd` because the existing update
|
||||||
|
/// path uses `COALESCE` semantics where `None` means "preserve" — which
|
||||||
|
/// makes clearing the tag impossible. This command always writes exactly
|
||||||
|
/// what you send, including `None` to explicitly clear.
|
||||||
|
#[tauri::command]
|
||||||
|
pub async fn set_task_energy_cmd(
|
||||||
|
state: tauri::State<'_, AppState>,
|
||||||
|
id: String,
|
||||||
|
energy: Option<String>,
|
||||||
|
) -> Result<TaskDto, String> {
|
||||||
|
let validated = validate_energy(energy.as_deref())?;
|
||||||
|
let row = db_set_task_energy(&state.db, &id, validated)
|
||||||
|
.await
|
||||||
|
.map_err(|e| e.to_string())?;
|
||||||
|
Ok(TaskDto::from(row))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Convert HITL feedback rows fetched from storage into the few-shot
|
||||||
|
/// exemplar shape the LLM crate consumes. We reconstruct the `input`
|
||||||
|
/// (parent task text, transcript chunk) from `context_json` where the
|
||||||
|
/// recorder has stored it. Rows without usable input are dropped —
|
||||||
|
/// the prompt builder filters them too, but doing it here keeps the
|
||||||
|
/// exemplar list tight and the prompt budget predictable.
|
||||||
|
///
|
||||||
|
/// Malformed `context_json` is logged rather than silently dropped so
|
||||||
|
/// data-integrity regressions surface instead of disappearing.
|
||||||
|
fn to_llm_examples(rows: Vec<FeedbackRow>) -> Vec<LlmFeedbackExample> {
|
||||||
|
rows.into_iter()
|
||||||
|
.filter_map(|r| {
|
||||||
|
let raw = r.context_json.as_deref().unwrap_or("{}");
|
||||||
|
let ctx: serde_json::Value = match serde_json::from_str(raw) {
|
||||||
|
Ok(v) => v,
|
||||||
|
Err(e) => {
|
||||||
|
eprintln!(
|
||||||
|
"[feedback] skipping row id={} with malformed context_json: {e}",
|
||||||
|
r.id
|
||||||
|
);
|
||||||
|
return None;
|
||||||
|
}
|
||||||
|
};
|
||||||
|
let input = ctx
|
||||||
|
.get("input")
|
||||||
|
.and_then(|v| v.as_str())
|
||||||
|
.map(str::to_string)
|
||||||
|
.unwrap_or_default();
|
||||||
|
if input.trim().is_empty() {
|
||||||
|
return None;
|
||||||
|
}
|
||||||
|
Some(LlmFeedbackExample {
|
||||||
|
input,
|
||||||
|
original_output: r.original_text,
|
||||||
|
corrected_output: r.corrected_text,
|
||||||
|
})
|
||||||
|
})
|
||||||
|
.collect()
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Rough character budget for the few-shot block. Qwen3's tokenizer
|
||||||
|
/// averages ~3.5 chars per token in English, so 2000 chars is ~570
|
||||||
|
/// tokens — well inside the 64-token reserve + response-token gap
|
||||||
|
/// against the 8192-token context cap (see `LlmEngine::generate`).
|
||||||
|
///
|
||||||
|
/// Exceed this and we drop the oldest examples first. Rationale: the
|
||||||
|
/// retrieval already orders most-recent-first, and the most recent
|
||||||
|
/// correction is usually the one carrying the user's live preference.
|
||||||
|
const FEW_SHOT_CHAR_BUDGET: usize = 2000;
|
||||||
|
|
||||||
|
fn example_char_cost(ex: &LlmFeedbackExample) -> usize {
|
||||||
|
// Matches the render path in `prompts::render_feedback_exemplar`:
|
||||||
|
// "Input: {input}\nGood output: {good}". Prefix strings + newlines
|
||||||
|
// + the two bodies. Slight overestimate to leave headroom.
|
||||||
|
let good_len = ex
|
||||||
|
.corrected_output
|
||||||
|
.as_deref()
|
||||||
|
.or(ex.original_output.as_deref())
|
||||||
|
.map(str::len)
|
||||||
|
.unwrap_or(0);
|
||||||
|
ex.input.len() + good_len + 24
|
||||||
|
}
|
||||||
|
|
||||||
|
fn trim_to_budget(mut examples: Vec<LlmFeedbackExample>) -> Vec<LlmFeedbackExample> {
|
||||||
|
let mut running = 0usize;
|
||||||
|
let mut kept = Vec::with_capacity(examples.len());
|
||||||
|
for ex in examples.drain(..) {
|
||||||
|
let cost = example_char_cost(&ex);
|
||||||
|
if running + cost > FEW_SHOT_CHAR_BUDGET {
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
running += cost;
|
||||||
|
kept.push(ex);
|
||||||
|
}
|
||||||
|
kept
|
||||||
|
}
|
||||||
|
|
||||||
#[tauri::command]
|
#[tauri::command]
|
||||||
pub async fn decompose_and_store(
|
pub async fn decompose_and_store(
|
||||||
state: tauri::State<'_, AppState>,
|
state: tauri::State<'_, AppState>,
|
||||||
parent_task_id: String,
|
parent_task_id: String,
|
||||||
|
profile_id: Option<String>,
|
||||||
) -> Result<Vec<TaskDto>, String> {
|
) -> Result<Vec<TaskDto>, String> {
|
||||||
let parent = db_get_task(&state.db, &parent_task_id)
|
let parent = db_get_task(&state.db, &parent_task_id)
|
||||||
.await
|
.await
|
||||||
.map_err(|e| e.to_string())?
|
.map_err(|e| e.to_string())?
|
||||||
.ok_or_else(|| format!("Task {parent_task_id} not found"))?;
|
.ok_or_else(|| format!("Task {parent_task_id} not found"))?;
|
||||||
|
|
||||||
|
// Pull recent micro-step feedback so the system prompt gets
|
||||||
|
// conditioned on the user's preferred decomposition style. We
|
||||||
|
// cap at 5 examples AND at a char budget to keep the prompt
|
||||||
|
// under token budget regardless of how much feedback has been
|
||||||
|
// captured, and scope by profile so per-profile styles do not
|
||||||
|
// leak into each other.
|
||||||
|
let examples = db_list_feedback_examples(
|
||||||
|
&state.db,
|
||||||
|
FeedbackTargetType::MicroStep,
|
||||||
|
5,
|
||||||
|
0,
|
||||||
|
profile_id.as_deref(),
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.map(to_llm_examples)
|
||||||
|
.map(trim_to_budget)
|
||||||
|
.unwrap_or_default();
|
||||||
|
|
||||||
let engine = state.llm_engine.clone();
|
let engine = state.llm_engine.clone();
|
||||||
let parent_text = parent.text.clone();
|
let parent_text = parent.text.clone();
|
||||||
let steps = tokio::task::spawn_blocking(move || engine.decompose_task(&parent_text))
|
let steps = tokio::task::spawn_blocking(move || {
|
||||||
|
engine.decompose_task_with_feedback(&parent_text, &examples)
|
||||||
|
})
|
||||||
.await
|
.await
|
||||||
.map_err(|e| e.to_string())?
|
.map_err(|e| e.to_string())?
|
||||||
.map_err(|e| e.to_string())?;
|
.map_err(|e| e.to_string())?;
|
||||||
@@ -204,9 +345,22 @@ pub async fn decompose_and_store(
|
|||||||
pub async fn extract_tasks_from_transcript_cmd(
|
pub async fn extract_tasks_from_transcript_cmd(
|
||||||
state: tauri::State<'_, AppState>,
|
state: tauri::State<'_, AppState>,
|
||||||
transcript: String,
|
transcript: String,
|
||||||
|
profile_id: Option<String>,
|
||||||
) -> Result<Vec<String>, String> {
|
) -> Result<Vec<String>, String> {
|
||||||
|
let examples = db_list_feedback_examples(
|
||||||
|
&state.db,
|
||||||
|
FeedbackTargetType::TaskExtraction,
|
||||||
|
5,
|
||||||
|
0,
|
||||||
|
profile_id.as_deref(),
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
.map(to_llm_examples)
|
||||||
|
.map(trim_to_budget)
|
||||||
|
.unwrap_or_default();
|
||||||
|
|
||||||
let engine = state.llm_engine.clone();
|
let engine = state.llm_engine.clone();
|
||||||
tokio::task::spawn_blocking(move || engine.extract_tasks(&transcript))
|
tokio::task::spawn_blocking(move || engine.extract_tasks_with_feedback(&transcript, &examples))
|
||||||
.await
|
.await
|
||||||
.map_err(|e| e.to_string())?
|
.map_err(|e| e.to_string())?
|
||||||
.map_err(|e| e.to_string())
|
.map_err(|e| e.to_string())
|
||||||
|
|||||||
@@ -234,7 +234,7 @@ pub fn run() {
|
|||||||
// Runtime-warning banner: push CPU-feature + Vulkan-loader
|
// Runtime-warning banner: push CPU-feature + Vulkan-loader
|
||||||
// fallbacks to the frontend so Settings can render a one-line
|
// fallbacks to the frontend so Settings can render a one-line
|
||||||
// hint. No-ops on a fully-supported box.
|
// hint. No-ops on a fully-supported box.
|
||||||
crate::commands::models::emit_runtime_warnings(&app.handle());
|
crate::commands::models::emit_runtime_warnings(app.handle());
|
||||||
|
|
||||||
if let Err(e) = tray::setup(app) {
|
if let Err(e) = tray::setup(app) {
|
||||||
eprintln!("Failed to setup tray: {e}");
|
eprintln!("Failed to setup tray: {e}");
|
||||||
@@ -284,10 +284,14 @@ pub fn run() {
|
|||||||
commands::tasks::complete_task_cmd,
|
commands::tasks::complete_task_cmd,
|
||||||
commands::tasks::delete_task_cmd,
|
commands::tasks::delete_task_cmd,
|
||||||
commands::tasks::uncomplete_task_cmd,
|
commands::tasks::uncomplete_task_cmd,
|
||||||
|
commands::tasks::set_task_energy_cmd,
|
||||||
commands::tasks::decompose_and_store,
|
commands::tasks::decompose_and_store,
|
||||||
commands::tasks::extract_tasks_from_transcript_cmd,
|
commands::tasks::extract_tasks_from_transcript_cmd,
|
||||||
commands::tasks::list_subtasks_cmd,
|
commands::tasks::list_subtasks_cmd,
|
||||||
commands::tasks::complete_subtask_cmd,
|
commands::tasks::complete_subtask_cmd,
|
||||||
|
// HITL feedback (Phase 2 roadmap)
|
||||||
|
commands::feedback::record_feedback,
|
||||||
|
commands::feedback::list_feedback_examples_cmd,
|
||||||
// Profiles + profile terms (canonical SQLite-backed profile CRUD) — Task 12
|
// Profiles + profile terms (canonical SQLite-backed profile CRUD) — Task 12
|
||||||
commands::profiles::list_profiles_cmd,
|
commands::profiles::list_profiles_cmd,
|
||||||
commands::profiles::get_profile_cmd,
|
commands::profiles::get_profile_cmd,
|
||||||
|
|||||||
106
src/lib/components/EnergyChip.svelte
Normal file
106
src/lib/components/EnergyChip.svelte
Normal file
@@ -0,0 +1,106 @@
|
|||||||
|
<script lang="ts">
|
||||||
|
// Phase 3 — Energy tag chip. Cycles a task's energy level through
|
||||||
|
// the spec's four states: unset → High → Medium → Brain-Dead → unset.
|
||||||
|
//
|
||||||
|
// Visual discipline: when energy is unset, the chip renders at
|
||||||
|
// `group-hover` opacity only so untagged rows stay calm. Once set,
|
||||||
|
// the chip is always visible because the colour IS the signal for
|
||||||
|
// the match-my-energy sort.
|
||||||
|
//
|
||||||
|
// Colour choices borrow the existing design tokens:
|
||||||
|
// High → accent (warm, on-brand, attention-ready)
|
||||||
|
// Medium → warning (amber, unforced)
|
||||||
|
// Brain-Dead → text-tertiary (low-energy grey, not danger red —
|
||||||
|
// the brief is explicit that this state must not feel
|
||||||
|
// pathologised)
|
||||||
|
//
|
||||||
|
// Callers pass the task's current energy and a setter. This component
|
||||||
|
// owns no state — the task store is the source of truth.
|
||||||
|
|
||||||
|
import type { EnergyLevel } from "$lib/types/app";
|
||||||
|
import { Zap } from "lucide-svelte";
|
||||||
|
|
||||||
|
let {
|
||||||
|
energy = null as EnergyLevel | null,
|
||||||
|
onSelect,
|
||||||
|
size = "sm",
|
||||||
|
reduceMotion = false,
|
||||||
|
}: {
|
||||||
|
energy: EnergyLevel | null;
|
||||||
|
onSelect: (next: EnergyLevel | null) => void;
|
||||||
|
size?: "sm" | "md";
|
||||||
|
reduceMotion?: boolean;
|
||||||
|
} = $props();
|
||||||
|
|
||||||
|
// Cycle order lives here so the chip is the single authority on what
|
||||||
|
// "next" means. Tap once to tag, tap again to move up, tap past
|
||||||
|
// Brain-Dead to clear. Keyboard-equivalent via the <button> element.
|
||||||
|
const CYCLE: (EnergyLevel | null)[] = [null, "high", "medium", "brain_dead"];
|
||||||
|
|
||||||
|
function next(): EnergyLevel | null {
|
||||||
|
const idx = CYCLE.indexOf(energy);
|
||||||
|
return CYCLE[(idx + 1) % CYCLE.length];
|
||||||
|
}
|
||||||
|
|
||||||
|
function labelFor(level: EnergyLevel | null): string {
|
||||||
|
switch (level) {
|
||||||
|
case "high": return "High";
|
||||||
|
case "medium": return "Medium";
|
||||||
|
case "brain_dead": return "Brain-Dead";
|
||||||
|
default: return "No energy set";
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
let tooltip = $derived(
|
||||||
|
energy === null
|
||||||
|
? "Tag energy (click to set)"
|
||||||
|
: `Energy: ${labelFor(energy)} — click to change`
|
||||||
|
);
|
||||||
|
|
||||||
|
// Icon dimensions. `md` is the one used on the Tasks-page main rows;
|
||||||
|
// `sm` is for the compact WIP list rows and micro-step children.
|
||||||
|
let iconSize = $derived(size === "md" ? 13 : 10);
|
||||||
|
let chipSize = $derived(size === "md" ? "h-5" : "h-4");
|
||||||
|
</script>
|
||||||
|
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
class="energy-chip inline-flex items-center justify-center rounded-md border px-1 {chipSize} text-[10px] font-medium
|
||||||
|
{energy === null
|
||||||
|
? 'opacity-0 group-hover:opacity-100 text-text-tertiary border-border-subtle hover:border-accent hover:text-text-secondary'
|
||||||
|
: ''}
|
||||||
|
{energy === 'high'
|
||||||
|
? 'text-accent border-accent bg-accent/10'
|
||||||
|
: ''}
|
||||||
|
{energy === 'medium'
|
||||||
|
? 'text-warning border-warning bg-warning/10'
|
||||||
|
: ''}
|
||||||
|
{energy === 'brain_dead'
|
||||||
|
? 'text-text-tertiary border-border bg-hover'
|
||||||
|
: ''}"
|
||||||
|
onclick={() => onSelect(next())}
|
||||||
|
aria-label={tooltip}
|
||||||
|
title={tooltip}
|
||||||
|
data-energy={energy ?? 'unset'}
|
||||||
|
style={reduceMotion
|
||||||
|
? ''
|
||||||
|
: 'transition: opacity var(--duration-ui), color var(--duration-ui), border-color var(--duration-ui), background-color var(--duration-ui)'}
|
||||||
|
>
|
||||||
|
<Zap size={iconSize} aria-hidden="true" />
|
||||||
|
{#if energy !== null && size === "md"}
|
||||||
|
<span class="ml-1">{labelFor(energy)}</span>
|
||||||
|
{/if}
|
||||||
|
</button>
|
||||||
|
|
||||||
|
<style>
|
||||||
|
.energy-chip {
|
||||||
|
cursor: pointer;
|
||||||
|
line-height: 1;
|
||||||
|
font-family: var(--font-family-body);
|
||||||
|
}
|
||||||
|
|
||||||
|
.energy-chip:focus-visible {
|
||||||
|
outline: 2px solid var(--color-accent);
|
||||||
|
outline-offset: 1px;
|
||||||
|
}
|
||||||
|
</style>
|
||||||
298
src/lib/components/FocusTimer.svelte
Normal file
298
src/lib/components/FocusTimer.svelte
Normal file
@@ -0,0 +1,298 @@
|
|||||||
|
<script lang="ts">
|
||||||
|
// Floating focus-timer overlay. Renders nothing when no timer is
|
||||||
|
// active. When a timer is running, pins a compact SVG progress ring
|
||||||
|
// to the top-right of the viewport with the remaining mm:ss in the
|
||||||
|
// centre. Completion plays a gentle chime, flashes a success ring
|
||||||
|
// for 3 s, then disappears. Cancel button appears on hover.
|
||||||
|
//
|
||||||
|
// Mounted once in +layout.svelte. Listens for `kon:start-timer`
|
||||||
|
// events from anywhere in the app (e.g. MicroSteps) and delegates
|
||||||
|
// to the focus-timer store.
|
||||||
|
//
|
||||||
|
// Design tokens used: --color-accent (mid-progress),
|
||||||
|
// --color-warning (final 15%), --color-success (flourish),
|
||||||
|
// --color-border (unfilled ring track). No literals, so the ring
|
||||||
|
// follows the sensory-zone theme switcher in Settings.
|
||||||
|
|
||||||
|
import { onMount, onDestroy } from "svelte";
|
||||||
|
import { X, Plus, ExternalLink } from "lucide-svelte";
|
||||||
|
import { focusTimer } from "$lib/stores/focusTimer.svelte.js";
|
||||||
|
import { hasTauriRuntime } from "$lib/utils/runtime.js";
|
||||||
|
|
||||||
|
// Hide the "pop out" button inside the float window itself — opening
|
||||||
|
// a second float from a float would be silly and would re-mount the
|
||||||
|
// same component. Detect via URL rather than a prop so we do not
|
||||||
|
// have to thread context through every mount site.
|
||||||
|
let isSecondaryWindow = $state(false);
|
||||||
|
if (typeof window !== "undefined") {
|
||||||
|
isSecondaryWindow = window.location.pathname.startsWith("/float")
|
||||||
|
|| window.location.pathname.startsWith("/viewer");
|
||||||
|
}
|
||||||
|
|
||||||
|
function handlePopOut() {
|
||||||
|
// Mirror the button in TasksPage.svelte — opens the always-on-top
|
||||||
|
// Now list + pinned timer in one floating window.
|
||||||
|
if (!hasTauriRuntime()) return;
|
||||||
|
window.open("/float", "_blank", "width=380,height=520");
|
||||||
|
}
|
||||||
|
|
||||||
|
const RING_SIZE = 64;
|
||||||
|
const RING_STROKE = 5;
|
||||||
|
const RING_RADIUS = (RING_SIZE - RING_STROKE) / 2;
|
||||||
|
const RING_CIRCUMFERENCE = 2 * Math.PI * RING_RADIUS;
|
||||||
|
|
||||||
|
// Progress ring fills as time elapses. Stroke-dashoffset goes from
|
||||||
|
// circumference (empty) to 0 (full) — we want it the other way,
|
||||||
|
// because the UX is a shrinking-time disc: more elapsed = less
|
||||||
|
// ring visible. Render the remaining arc: dashoffset = circumference * progress.
|
||||||
|
let dashOffset = $derived(RING_CIRCUMFERENCE * focusTimer.progress);
|
||||||
|
|
||||||
|
function formatRemaining(ms: number): string {
|
||||||
|
const total = Math.ceil(ms / 1000);
|
||||||
|
const mins = Math.floor(total / 60);
|
||||||
|
const secs = total % 60;
|
||||||
|
return `${mins}:${secs.toString().padStart(2, "0")}`;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Colour shifts over the last 15% of the timer to cue "nearly done"
|
||||||
|
// without resorting to red (which the brief flags as anxiogenic for
|
||||||
|
// the target audience).
|
||||||
|
let ringColor = $derived.by(() => {
|
||||||
|
if (focusTimer.showingCompletionFlash) return "var(--color-success)";
|
||||||
|
if (focusTimer.progress >= 0.85) return "var(--color-warning)";
|
||||||
|
return "var(--color-accent)";
|
||||||
|
});
|
||||||
|
|
||||||
|
// Event handler: start a timer when any component fires `kon:start-timer`.
|
||||||
|
// Payload shape from MicroSteps.svelte and task row buttons:
|
||||||
|
// { taskId?: string, seconds: number, label?: string }
|
||||||
|
function handleStartEvent(evt: Event) {
|
||||||
|
const detail = (evt as CustomEvent).detail ?? {};
|
||||||
|
const seconds = Number(detail.seconds);
|
||||||
|
if (!Number.isFinite(seconds) || seconds <= 0) return;
|
||||||
|
focusTimer.start(seconds, {
|
||||||
|
taskId: detail.taskId ?? null,
|
||||||
|
label: detail.label ?? null,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
onMount(() => {
|
||||||
|
window.addEventListener("kon:start-timer", handleStartEvent);
|
||||||
|
// Rehydrate any in-flight timer that survived a window close.
|
||||||
|
focusTimer.rehydrate();
|
||||||
|
});
|
||||||
|
|
||||||
|
onDestroy(() => {
|
||||||
|
window.removeEventListener("kon:start-timer", handleStartEvent);
|
||||||
|
});
|
||||||
|
|
||||||
|
function handleCancel() {
|
||||||
|
focusTimer.cancel();
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleExtend() {
|
||||||
|
focusTimer.extend(60);
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleDismissFlash() {
|
||||||
|
focusTimer.dismissCompletionFlash();
|
||||||
|
}
|
||||||
|
</script>
|
||||||
|
|
||||||
|
{#if focusTimer.active || focusTimer.showingCompletionFlash}
|
||||||
|
<div
|
||||||
|
class="focus-timer"
|
||||||
|
class:completed={focusTimer.showingCompletionFlash}
|
||||||
|
role="status"
|
||||||
|
aria-live="polite"
|
||||||
|
aria-label={focusTimer.label ?? "Focus timer"}
|
||||||
|
>
|
||||||
|
<div class="ring-wrap">
|
||||||
|
<svg
|
||||||
|
width={RING_SIZE}
|
||||||
|
height={RING_SIZE}
|
||||||
|
viewBox={`0 0 ${RING_SIZE} ${RING_SIZE}`}
|
||||||
|
aria-hidden="true"
|
||||||
|
>
|
||||||
|
<!-- Track -->
|
||||||
|
<circle
|
||||||
|
cx={RING_SIZE / 2}
|
||||||
|
cy={RING_SIZE / 2}
|
||||||
|
r={RING_RADIUS}
|
||||||
|
fill="none"
|
||||||
|
stroke="var(--color-border)"
|
||||||
|
stroke-width={RING_STROKE}
|
||||||
|
/>
|
||||||
|
<!-- Progress (shrinking slice — full ring at start, none at end) -->
|
||||||
|
<circle
|
||||||
|
cx={RING_SIZE / 2}
|
||||||
|
cy={RING_SIZE / 2}
|
||||||
|
r={RING_RADIUS}
|
||||||
|
fill="none"
|
||||||
|
stroke={ringColor}
|
||||||
|
stroke-width={RING_STROKE}
|
||||||
|
stroke-linecap="round"
|
||||||
|
stroke-dasharray={RING_CIRCUMFERENCE}
|
||||||
|
stroke-dashoffset={dashOffset}
|
||||||
|
transform={`rotate(-90 ${RING_SIZE / 2} ${RING_SIZE / 2})`}
|
||||||
|
style="transition: stroke-dashoffset 250ms linear, stroke 400ms ease"
|
||||||
|
/>
|
||||||
|
</svg>
|
||||||
|
<div class="time" aria-hidden={focusTimer.showingCompletionFlash}>
|
||||||
|
{#if focusTimer.showingCompletionFlash}
|
||||||
|
<span class="done">done</span>
|
||||||
|
{:else}
|
||||||
|
{formatRemaining(focusTimer.remainingMs)}
|
||||||
|
{/if}
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="controls">
|
||||||
|
{#if focusTimer.showingCompletionFlash}
|
||||||
|
<button
|
||||||
|
class="icon-btn"
|
||||||
|
onclick={handleDismissFlash}
|
||||||
|
aria-label="Dismiss completion"
|
||||||
|
title="Dismiss"
|
||||||
|
>
|
||||||
|
<X size={14} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
{:else}
|
||||||
|
<button
|
||||||
|
class="icon-btn"
|
||||||
|
onclick={handleExtend}
|
||||||
|
aria-label="Add one minute"
|
||||||
|
title="+1 min"
|
||||||
|
>
|
||||||
|
<Plus size={14} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
{#if !isSecondaryWindow}
|
||||||
|
<button
|
||||||
|
class="icon-btn"
|
||||||
|
onclick={handlePopOut}
|
||||||
|
aria-label="Pop out timer + Now list into floating window"
|
||||||
|
title="Pop out (keeps timer + tasks on top)"
|
||||||
|
>
|
||||||
|
<ExternalLink size={14} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
{/if}
|
||||||
|
<button
|
||||||
|
class="icon-btn"
|
||||||
|
onclick={handleCancel}
|
||||||
|
aria-label="Cancel timer"
|
||||||
|
title="Cancel"
|
||||||
|
>
|
||||||
|
<X size={14} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
{/if}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{#if focusTimer.label}
|
||||||
|
<div class="label" aria-hidden="true">{focusTimer.label}</div>
|
||||||
|
{/if}
|
||||||
|
</div>
|
||||||
|
{/if}
|
||||||
|
|
||||||
|
<style>
|
||||||
|
.focus-timer {
|
||||||
|
position: fixed;
|
||||||
|
top: 52px;
|
||||||
|
right: 16px;
|
||||||
|
z-index: 40;
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 8px;
|
||||||
|
padding: 6px 10px 6px 6px;
|
||||||
|
background: var(--color-bg-elevated);
|
||||||
|
border: 1px solid var(--color-border);
|
||||||
|
border-radius: var(--radius-lg);
|
||||||
|
box-shadow: 0 8px 24px rgba(0, 0, 0, 0.35);
|
||||||
|
font-family: var(--font-family-body);
|
||||||
|
font-variant-numeric: tabular-nums;
|
||||||
|
transition: opacity 200ms ease, transform 200ms ease;
|
||||||
|
}
|
||||||
|
|
||||||
|
.focus-timer.completed {
|
||||||
|
border-color: var(--color-success);
|
||||||
|
}
|
||||||
|
|
||||||
|
.ring-wrap {
|
||||||
|
position: relative;
|
||||||
|
width: 64px;
|
||||||
|
height: 64px;
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: center;
|
||||||
|
flex-shrink: 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
.time {
|
||||||
|
position: absolute;
|
||||||
|
inset: 0;
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: center;
|
||||||
|
font-size: 14px;
|
||||||
|
font-weight: 500;
|
||||||
|
color: var(--color-text);
|
||||||
|
}
|
||||||
|
|
||||||
|
.time .done {
|
||||||
|
font-size: 11px;
|
||||||
|
color: var(--color-success);
|
||||||
|
font-family: var(--font-family-display);
|
||||||
|
letter-spacing: 0.02em;
|
||||||
|
}
|
||||||
|
|
||||||
|
.controls {
|
||||||
|
display: flex;
|
||||||
|
flex-direction: column;
|
||||||
|
gap: 4px;
|
||||||
|
opacity: 0;
|
||||||
|
transition: opacity 180ms ease;
|
||||||
|
}
|
||||||
|
|
||||||
|
.focus-timer:hover .controls,
|
||||||
|
.focus-timer:focus-within .controls,
|
||||||
|
.focus-timer.completed .controls {
|
||||||
|
opacity: 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
.icon-btn {
|
||||||
|
display: inline-flex;
|
||||||
|
align-items: center;
|
||||||
|
justify-content: center;
|
||||||
|
width: 22px;
|
||||||
|
height: 22px;
|
||||||
|
padding: 0;
|
||||||
|
border: 0;
|
||||||
|
border-radius: var(--radius-sm);
|
||||||
|
background: transparent;
|
||||||
|
color: var(--color-text-secondary);
|
||||||
|
cursor: pointer;
|
||||||
|
transition: background 150ms ease, color 150ms ease;
|
||||||
|
}
|
||||||
|
|
||||||
|
.icon-btn:hover {
|
||||||
|
background: var(--color-hover);
|
||||||
|
color: var(--color-text);
|
||||||
|
}
|
||||||
|
|
||||||
|
.icon-btn:focus-visible {
|
||||||
|
outline: 2px solid var(--color-accent);
|
||||||
|
outline-offset: 1px;
|
||||||
|
}
|
||||||
|
|
||||||
|
.label {
|
||||||
|
max-width: 140px;
|
||||||
|
font-size: 12px;
|
||||||
|
color: var(--color-text-secondary);
|
||||||
|
overflow: hidden;
|
||||||
|
text-overflow: ellipsis;
|
||||||
|
white-space: nowrap;
|
||||||
|
}
|
||||||
|
|
||||||
|
:global([data-reduce-motion="true"]) .focus-timer circle {
|
||||||
|
transition: none !important;
|
||||||
|
}
|
||||||
|
</style>
|
||||||
@@ -1,8 +1,9 @@
|
|||||||
<script lang="ts">
|
<script lang="ts">
|
||||||
import { invoke } from '@tauri-apps/api/core';
|
import { invoke } from '@tauri-apps/api/core';
|
||||||
import { ListTree, Check, Timer, Loader2 } from 'lucide-svelte';
|
import { ListTree, Check, Timer, Loader2, ThumbsUp, ThumbsDown, Pencil } from 'lucide-svelte';
|
||||||
|
import { profilesStore } from '$lib/stores/profiles.svelte.ts';
|
||||||
|
|
||||||
let { parentTaskId, reduceMotion = false } = $props();
|
let { parentTaskId, parentTaskText = '', reduceMotion = false } = $props();
|
||||||
|
|
||||||
interface Subtask {
|
interface Subtask {
|
||||||
id: string;
|
id: string;
|
||||||
@@ -15,6 +16,22 @@
|
|||||||
let error = $state('');
|
let error = $state('');
|
||||||
let decomposing = $state(false);
|
let decomposing = $state(false);
|
||||||
|
|
||||||
|
// Per-step UI state. Keyed by subtask id so we never lose state when
|
||||||
|
// the list reorders. Values:
|
||||||
|
// rating[id] — 1 | -1 — the thumbs vote the user gave this session
|
||||||
|
// editing[id] — true while the user is editing the step text
|
||||||
|
// draft[id] — the in-flight edit value before save
|
||||||
|
let rating = $state<Record<string, 1 | -1 | undefined>>({});
|
||||||
|
let editing = $state<Record<string, boolean>>({});
|
||||||
|
let draft = $state<Record<string, string>>({});
|
||||||
|
|
||||||
|
// Monotonic token per-step. Each time the user kicks off a save we
|
||||||
|
// bump the token; the pending save remembers its token and only
|
||||||
|
// rolls back if the failure belongs to the still-current edit.
|
||||||
|
// Without this, a slow first save that eventually fails will stomp
|
||||||
|
// a faster second save that already committed.
|
||||||
|
let saveToken = $state<Record<string, number>>({});
|
||||||
|
|
||||||
async function loadSubtasks() {
|
async function loadSubtasks() {
|
||||||
loading = true;
|
loading = true;
|
||||||
error = '';
|
error = '';
|
||||||
@@ -31,7 +48,10 @@
|
|||||||
decomposing = true;
|
decomposing = true;
|
||||||
error = '';
|
error = '';
|
||||||
try {
|
try {
|
||||||
subtasks = await invoke<Subtask[]>('decompose_and_store', { parentTaskId });
|
subtasks = await invoke<Subtask[]>('decompose_and_store', {
|
||||||
|
parentTaskId,
|
||||||
|
profileId: profilesStore.activeProfileId,
|
||||||
|
});
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
error = String(e);
|
error = String(e);
|
||||||
} finally {
|
} finally {
|
||||||
@@ -53,6 +73,101 @@
|
|||||||
}));
|
}));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// --- HITL feedback --------------------------------------------------------
|
||||||
|
//
|
||||||
|
// All three paths (thumbs up, thumbs down, correction-via-edit) route
|
||||||
|
// into the same `record_feedback` command. The parent task text is the
|
||||||
|
// "input" the AI was given, so it travels in context_json so the prompt
|
||||||
|
// builder can reconstruct the (input, good-output) pair.
|
||||||
|
|
||||||
|
function feedbackContextJson() {
|
||||||
|
return JSON.stringify({ input: parentTaskText ?? '' });
|
||||||
|
}
|
||||||
|
|
||||||
|
async function recordThumb(step: Subtask, ratingValue: 1 | -1) {
|
||||||
|
// Toggle: if the user already voted the same way, clear it (record
|
||||||
|
// rating 0 means correction, not a thumb-off — we just skip the
|
||||||
|
// re-record and drop the local highlight). Unvoting isn't stored;
|
||||||
|
// the audit trail stays immutable.
|
||||||
|
if (rating[step.id] === ratingValue) {
|
||||||
|
const next = { ...rating };
|
||||||
|
delete next[step.id];
|
||||||
|
rating = next;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
rating = { ...rating, [step.id]: ratingValue };
|
||||||
|
try {
|
||||||
|
await invoke('record_feedback', {
|
||||||
|
input: {
|
||||||
|
targetType: 'microstep',
|
||||||
|
targetId: step.id,
|
||||||
|
rating: ratingValue,
|
||||||
|
originalText: step.text,
|
||||||
|
correctedText: null,
|
||||||
|
contextJson: feedbackContextJson(),
|
||||||
|
profileId: profilesStore.activeProfileId,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
} catch (_) { /* feedback capture is best-effort, never fatal */ }
|
||||||
|
}
|
||||||
|
|
||||||
|
function startEdit(step: Subtask) {
|
||||||
|
editing = { ...editing, [step.id]: true };
|
||||||
|
draft = { ...draft, [step.id]: step.text };
|
||||||
|
}
|
||||||
|
|
||||||
|
function cancelEdit(stepId: string) {
|
||||||
|
const nextE = { ...editing }; delete nextE[stepId]; editing = nextE;
|
||||||
|
const nextD = { ...draft }; delete nextD[stepId]; draft = nextD;
|
||||||
|
}
|
||||||
|
|
||||||
|
async function saveEdit(step: Subtask) {
|
||||||
|
const next = (draft[step.id] ?? '').trim();
|
||||||
|
cancelEdit(step.id);
|
||||||
|
if (!next || next === step.text) return;
|
||||||
|
const original = step.text;
|
||||||
|
// Update in-memory first so the UI is snappy; roll back if the
|
||||||
|
// persistence call fails so we never show stale-but-different text.
|
||||||
|
// The monotonic `myToken` guards against a stale rollback stomping
|
||||||
|
// a fresher successful save that landed while we were waiting.
|
||||||
|
const myToken = (saveToken[step.id] ?? 0) + 1;
|
||||||
|
saveToken = { ...saveToken, [step.id]: myToken };
|
||||||
|
const idx = subtasks.findIndex(s => s.id === step.id);
|
||||||
|
if (idx >= 0) subtasks[idx] = { ...subtasks[idx], text: next };
|
||||||
|
try {
|
||||||
|
await invoke('update_task_cmd', {
|
||||||
|
id: step.id,
|
||||||
|
patch: { text: next },
|
||||||
|
});
|
||||||
|
// Record correction as the highest-value feedback signal.
|
||||||
|
await invoke('record_feedback', {
|
||||||
|
input: {
|
||||||
|
targetType: 'microstep',
|
||||||
|
targetId: step.id,
|
||||||
|
rating: 0,
|
||||||
|
originalText: original,
|
||||||
|
correctedText: next,
|
||||||
|
contextJson: feedbackContextJson(),
|
||||||
|
profileId: profilesStore.activeProfileId,
|
||||||
|
},
|
||||||
|
}).catch(() => {});
|
||||||
|
} catch (_) {
|
||||||
|
// Only roll back if our token is still the most recent — a later
|
||||||
|
// edit that already succeeded must not be overwritten.
|
||||||
|
if (saveToken[step.id] === myToken) {
|
||||||
|
const rollbackIdx = subtasks.findIndex(s => s.id === step.id);
|
||||||
|
if (rollbackIdx >= 0) {
|
||||||
|
subtasks[rollbackIdx] = { ...subtasks[rollbackIdx], text: original };
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleEditKeydown(evt: KeyboardEvent, step: Subtask) {
|
||||||
|
if (evt.key === 'Enter') { evt.preventDefault(); saveEdit(step); }
|
||||||
|
else if (evt.key === 'Escape') { evt.preventDefault(); cancelEdit(step.id); }
|
||||||
|
}
|
||||||
|
|
||||||
$effect(() => {
|
$effect(() => {
|
||||||
if (parentTaskId) loadSubtasks();
|
if (parentTaskId) loadSubtasks();
|
||||||
});
|
});
|
||||||
@@ -97,10 +212,64 @@
|
|||||||
<Check size={9} aria-hidden="true" />
|
<Check size={9} aria-hidden="true" />
|
||||||
{/if}
|
{/if}
|
||||||
</button>
|
</button>
|
||||||
<span class="text-[12px] flex-1 min-w-0 {step.done ? 'line-through text-text-tertiary' : 'text-text-secondary'} truncate">
|
|
||||||
{step.text}
|
{#if editing[step.id]}
|
||||||
</span>
|
<!-- svelte-ignore a11y_autofocus — deliberate: inline edit
|
||||||
{#if !step.done}
|
is user-initiated and focus must land on the input to
|
||||||
|
match the UX pattern users expect from any task app. -->
|
||||||
|
<input
|
||||||
|
type="text"
|
||||||
|
bind:value={draft[step.id]}
|
||||||
|
onkeydown={(e) => handleEditKeydown(e, step)}
|
||||||
|
onblur={() => saveEdit(step)}
|
||||||
|
class="text-[12px] flex-1 min-w-0 bg-bg-input border border-accent rounded px-1.5 py-0.5 text-text focus:outline-none"
|
||||||
|
autofocus
|
||||||
|
data-no-transition
|
||||||
|
/>
|
||||||
|
{:else}
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
class="text-[12px] flex-1 min-w-0 {step.done ? 'line-through text-text-tertiary' : 'text-text-secondary'} truncate text-left cursor-text bg-transparent border-0 p-0"
|
||||||
|
ondblclick={() => !step.done && startEdit(step)}
|
||||||
|
disabled={step.done}
|
||||||
|
aria-label="Double-click to edit this step"
|
||||||
|
title="Double-click to edit"
|
||||||
|
>{step.text}</button>
|
||||||
|
{/if}
|
||||||
|
|
||||||
|
{#if !step.done && !editing[step.id]}
|
||||||
|
<!-- HITL feedback: thumbs vote + pencil edit. All three
|
||||||
|
route into record_feedback and feed the prompt-conditioning
|
||||||
|
loop. See docs/roadmap/2026-04-23-... Phase 2. -->
|
||||||
|
<button
|
||||||
|
class="opacity-0 group-hover:opacity-100 p-0.5 text-text-tertiary hover:text-success
|
||||||
|
{rating[step.id] === 1 ? '!opacity-100 text-success' : ''}"
|
||||||
|
onclick={() => recordThumb(step, 1)}
|
||||||
|
aria-label={rating[step.id] === 1 ? 'Remove thumbs up' : 'Thumbs up — this is a good step'}
|
||||||
|
title="Thumbs up — train the model on this style"
|
||||||
|
style={reduceMotion ? '' : 'transition: opacity var(--duration-ui), color var(--duration-ui)'}
|
||||||
|
>
|
||||||
|
<ThumbsUp size={10} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
<button
|
||||||
|
class="opacity-0 group-hover:opacity-100 p-0.5 text-text-tertiary hover:text-danger
|
||||||
|
{rating[step.id] === -1 ? '!opacity-100 text-danger' : ''}"
|
||||||
|
onclick={() => recordThumb(step, -1)}
|
||||||
|
aria-label={rating[step.id] === -1 ? 'Remove thumbs down' : 'Thumbs down — this misses the mark'}
|
||||||
|
title="Thumbs down — avoid this style"
|
||||||
|
style={reduceMotion ? '' : 'transition: opacity var(--duration-ui), color var(--duration-ui)'}
|
||||||
|
>
|
||||||
|
<ThumbsDown size={10} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
|
<button
|
||||||
|
class="opacity-0 group-hover:opacity-100 p-0.5 text-text-tertiary hover:text-accent"
|
||||||
|
onclick={() => startEdit(step)}
|
||||||
|
aria-label="Edit this step (the correction trains future suggestions)"
|
||||||
|
title="Edit — this is the strongest training signal"
|
||||||
|
style={reduceMotion ? '' : 'transition: opacity var(--duration-ui)'}
|
||||||
|
>
|
||||||
|
<Pencil size={10} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
<button
|
<button
|
||||||
class="opacity-0 group-hover:opacity-100 flex items-center gap-1 text-[10px] text-text-tertiary hover:text-accent"
|
class="opacity-0 group-hover:opacity-100 flex items-center gap-1 text-[10px] text-text-tertiary hover:text-accent"
|
||||||
onclick={() => startTimer(step.id)}
|
onclick={() => startTimer(step.id)}
|
||||||
|
|||||||
@@ -1,7 +1,14 @@
|
|||||||
<script lang="ts">
|
<script lang="ts">
|
||||||
import { tasks, addTask, completeTask, uncompleteTask, deleteTask } from '$lib/stores/page.svelte.js';
|
import { tasks, addTask, completeTask, uncompleteTask, deleteTask, setTaskEnergy } from '$lib/stores/page.svelte.js';
|
||||||
import MicroSteps from '$lib/components/MicroSteps.svelte';
|
import MicroSteps from '$lib/components/MicroSteps.svelte';
|
||||||
import { ChevronDown, ChevronRight } from 'lucide-svelte';
|
import EnergyChip from '$lib/components/EnergyChip.svelte';
|
||||||
|
import { ChevronDown, ChevronRight, Timer } from 'lucide-svelte';
|
||||||
|
|
||||||
|
function startFocusTimer(task: { id: string; text: string }) {
|
||||||
|
window.dispatchEvent(new CustomEvent('kon:start-timer', {
|
||||||
|
detail: { taskId: task.id, seconds: 300, label: task.text }
|
||||||
|
}));
|
||||||
|
}
|
||||||
|
|
||||||
let { wipLimit = 3 } = $props();
|
let { wipLimit = 3 } = $props();
|
||||||
|
|
||||||
@@ -68,6 +75,22 @@
|
|||||||
aria-label="Complete task"
|
aria-label="Complete task"
|
||||||
></button>
|
></button>
|
||||||
<span class="text-[13px] text-text flex-1 min-w-0 truncate">{task.text}</span>
|
<span class="text-[13px] text-text flex-1 min-w-0 truncate">{task.text}</span>
|
||||||
|
<!-- Energy chip (Phase 3) — compact, reveals on hover when unset -->
|
||||||
|
<EnergyChip
|
||||||
|
energy={task.energy}
|
||||||
|
onSelect={(next) => setTaskEnergy(task.id, next)}
|
||||||
|
size="sm"
|
||||||
|
/>
|
||||||
|
<!-- 5-min focus timer — the "just-start" button from the brief -->
|
||||||
|
<button
|
||||||
|
class="opacity-0 group-hover:opacity-100 text-text-tertiary hover:text-accent"
|
||||||
|
onclick={() => startFocusTimer(task)}
|
||||||
|
aria-label="Start 5-minute focus timer for this task"
|
||||||
|
title="Start 5-minute focus timer"
|
||||||
|
style="transition: opacity var(--duration-ui)"
|
||||||
|
>
|
||||||
|
<Timer size={12} aria-hidden="true" />
|
||||||
|
</button>
|
||||||
<!-- Expand/collapse micro-steps toggle -->
|
<!-- Expand/collapse micro-steps toggle -->
|
||||||
<button
|
<button
|
||||||
class="opacity-0 group-hover:opacity-100 text-text-tertiary hover:text-accent"
|
class="opacity-0 group-hover:opacity-100 text-text-tertiary hover:text-accent"
|
||||||
@@ -91,7 +114,7 @@
|
|||||||
</div>
|
</div>
|
||||||
<!-- Micro-steps panel (expanded) -->
|
<!-- Micro-steps panel (expanded) -->
|
||||||
{#if expandedTaskIds.has(task.id)}
|
{#if expandedTaskIds.has(task.id)}
|
||||||
<MicroSteps parentTaskId={task.id} />
|
<MicroSteps parentTaskId={task.id} parentTaskText={task.text} />
|
||||||
{/if}
|
{/if}
|
||||||
</div>
|
</div>
|
||||||
{/each}
|
{/each}
|
||||||
|
|||||||
@@ -487,7 +487,10 @@
|
|||||||
if (settings.aiTier === "tasks" && llmLoaded) {
|
if (settings.aiTier === "tasks" && llmLoaded) {
|
||||||
markGenerating("Extracting tasks");
|
markGenerating("Extracting tasks");
|
||||||
try {
|
try {
|
||||||
const items = await invoke("extract_tasks_from_transcript_cmd", { transcript: text });
|
const items = await invoke("extract_tasks_from_transcript_cmd", {
|
||||||
|
transcript: text,
|
||||||
|
profileId: profilesStore.activeProfileId,
|
||||||
|
});
|
||||||
markGenerationDone(true);
|
markGenerationDone(true);
|
||||||
return items.map((taskText) => ({ text: taskText }));
|
return items.map((taskText) => ({ text: taskText }));
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
|
|||||||
@@ -1,14 +1,17 @@
|
|||||||
<script lang="ts">
|
<script lang="ts">
|
||||||
import { tick } from "svelte";
|
import { tick } from "svelte";
|
||||||
import type { TaskBucket, TaskList } from "$lib/types/app";
|
import type { EnergyLevel, TaskBucket, TaskEntry, TaskList } from "$lib/types/app";
|
||||||
import { invoke } from "@tauri-apps/api/core";
|
import { invoke } from "@tauri-apps/api/core";
|
||||||
import {
|
import {
|
||||||
tasks, addTask, completeTask, uncompleteTask, deleteTask, updateTask,
|
tasks, addTask, completeTask, uncompleteTask, deleteTask, updateTask,
|
||||||
|
setTaskEnergy,
|
||||||
taskLists, addTaskList, renameTaskList, deleteTaskList,
|
taskLists, addTaskList, renameTaskList, deleteTaskList,
|
||||||
|
settings, saveSettings,
|
||||||
} from "$lib/stores/page.svelte.js";
|
} from "$lib/stores/page.svelte.js";
|
||||||
import WipTaskList from '$lib/components/WipTaskList.svelte';
|
import WipTaskList from '$lib/components/WipTaskList.svelte';
|
||||||
import EmptyState from '$lib/components/EmptyState.svelte';
|
import EmptyState from '$lib/components/EmptyState.svelte';
|
||||||
import { SquareCheck, Search, ExternalLink, ChevronLeft, ArrowUpDown, Plus, X, ChevronRight } from 'lucide-svelte';
|
import EnergyChip from '$lib/components/EnergyChip.svelte';
|
||||||
|
import { SquareCheck, Search, ExternalLink, ChevronLeft, ArrowUpDown, Plus, X, ChevronRight, Zap } from 'lucide-svelte';
|
||||||
import Card from "$lib/components/Card.svelte";
|
import Card from "$lib/components/Card.svelte";
|
||||||
import { formatTimestamp } from "$lib/utils/time.js";
|
import { formatTimestamp } from "$lib/utils/time.js";
|
||||||
import { BUCKET_COLORS, EFFORT_LABELS, EFFORT_ORDER } from "$lib/utils/constants.js";
|
import { BUCKET_COLORS, EFFORT_LABELS, EFFORT_ORDER } from "$lib/utils/constants.js";
|
||||||
@@ -47,6 +50,17 @@
|
|||||||
return EFFORT_LABELS[effort as keyof typeof EFFORT_LABELS] || effort;
|
return EFFORT_LABELS[effort as keyof typeof EFFORT_LABELS] || effort;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Phase 3 energy sort: when the user has opted in and declared a
|
||||||
|
// current energy, tasks matching that energy sort to the top. Tasks
|
||||||
|
// with no energy tag are treated as Medium-equivalent, per the brief's
|
||||||
|
// framing that unset is the normal case. Nothing is ever hidden —
|
||||||
|
// "reserves" in the spec means de-prioritise, not filter.
|
||||||
|
function energyMatchRank(task: TaskEntry, currentEnergy: EnergyLevel | null): number {
|
||||||
|
if (!currentEnergy) return 0;
|
||||||
|
const effective = task.energy ?? "medium";
|
||||||
|
return effective === currentEnergy ? 0 : 1;
|
||||||
|
}
|
||||||
|
|
||||||
let filteredTasks = $derived.by(() => {
|
let filteredTasks = $derived.by(() => {
|
||||||
let list = tasks.filter((t) => !t.done);
|
let list = tasks.filter((t) => !t.done);
|
||||||
if (activeBucket !== "all") {
|
if (activeBucket !== "all") {
|
||||||
@@ -68,9 +82,34 @@
|
|||||||
} else if (sortMode === "deep-first") {
|
} else if (sortMode === "deep-first") {
|
||||||
list.sort((a, b) => effortRank(b.effort) - effortRank(a.effort));
|
list.sort((a, b) => effortRank(b.effort) - effortRank(a.effort));
|
||||||
}
|
}
|
||||||
|
// Match-my-energy sort runs last (stable) so it reorders the result
|
||||||
|
// of the effort-based sort rather than replacing it.
|
||||||
|
if (settings.matchMyEnergy && settings.currentEnergy) {
|
||||||
|
const energy = settings.currentEnergy;
|
||||||
|
list.sort((a, b) => energyMatchRank(a, energy) - energyMatchRank(b, energy));
|
||||||
|
}
|
||||||
return list;
|
return list;
|
||||||
});
|
});
|
||||||
|
|
||||||
|
function cycleCurrentEnergy(next: EnergyLevel | null) {
|
||||||
|
settings.currentEnergy = next;
|
||||||
|
saveSettings();
|
||||||
|
}
|
||||||
|
|
||||||
|
function toggleMatchMyEnergy() {
|
||||||
|
settings.matchMyEnergy = !settings.matchMyEnergy;
|
||||||
|
saveSettings();
|
||||||
|
}
|
||||||
|
|
||||||
|
function energyLabel(level: EnergyLevel | null): string {
|
||||||
|
switch (level) {
|
||||||
|
case "high": return "High";
|
||||||
|
case "medium": return "Medium";
|
||||||
|
case "brain_dead": return "Brain-Dead";
|
||||||
|
default: return "Not set";
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
let completedTasks = $derived.by(() => {
|
let completedTasks = $derived.by(() => {
|
||||||
let list = tasks.filter((t) => t.done);
|
let list = tasks.filter((t) => t.done);
|
||||||
if (activeBucket !== "all") {
|
if (activeBucket !== "all") {
|
||||||
@@ -199,6 +238,49 @@
|
|||||||
<p class="text-[11px] text-text-tertiary mt-1">Add tasks manually. Automatic extraction from your transcripts is coming.</p>
|
<p class="text-[11px] text-text-tertiary mt-1">Add tasks manually. Automatic extraction from your transcripts is coming.</p>
|
||||||
</div>
|
</div>
|
||||||
<div class="flex-1"></div>
|
<div class="flex-1"></div>
|
||||||
|
|
||||||
|
<!-- Phase 3 match-my-energy control. Three-state energy selector +
|
||||||
|
toggle for the sort. Sits in the header so it is always visible
|
||||||
|
from any bucket / list context. -->
|
||||||
|
<div class="flex items-center gap-2 mr-2">
|
||||||
|
<span class="text-[10px] text-text-tertiary hidden sm:inline" aria-hidden="true">I feel</span>
|
||||||
|
<div class="flex items-center gap-0.5 bg-bg-input border border-border-subtle rounded-lg p-0.5"
|
||||||
|
role="radiogroup"
|
||||||
|
aria-label="My current energy">
|
||||||
|
{#each [
|
||||||
|
{ value: null, label: '—' },
|
||||||
|
{ value: 'high' as EnergyLevel, label: 'High' },
|
||||||
|
{ value: 'medium' as EnergyLevel, label: 'Med' },
|
||||||
|
{ value: 'brain_dead' as EnergyLevel, label: 'Low' },
|
||||||
|
] as opt}
|
||||||
|
<button
|
||||||
|
class="text-[10px] px-2 py-0.5 rounded-md
|
||||||
|
{settings.currentEnergy === opt.value
|
||||||
|
? 'bg-accent/15 text-accent'
|
||||||
|
: 'text-text-tertiary hover:text-text-secondary'}"
|
||||||
|
role="radio"
|
||||||
|
aria-checked={settings.currentEnergy === opt.value}
|
||||||
|
aria-label={opt.value ? `Set current energy to ${energyLabel(opt.value)}` : 'Clear current energy'}
|
||||||
|
onclick={() => cycleCurrentEnergy(opt.value)}
|
||||||
|
>{opt.label}</button>
|
||||||
|
{/each}
|
||||||
|
</div>
|
||||||
|
<button
|
||||||
|
class="flex items-center gap-1 btn-md rounded-lg text-[10px]
|
||||||
|
{settings.matchMyEnergy
|
||||||
|
? 'bg-accent/15 text-accent border border-accent/30'
|
||||||
|
: 'text-text-tertiary hover:bg-hover hover:text-text border border-transparent'}"
|
||||||
|
style="transition-duration: var(--duration-ui)"
|
||||||
|
onclick={toggleMatchMyEnergy}
|
||||||
|
aria-pressed={settings.matchMyEnergy}
|
||||||
|
aria-label={settings.matchMyEnergy ? 'Match my energy is on — click to turn off' : 'Match my energy is off — click to turn on'}
|
||||||
|
title="Sort matching tasks to the top (unset counts as Medium)"
|
||||||
|
>
|
||||||
|
<Zap size={12} aria-hidden="true" />
|
||||||
|
Match my energy
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
|
||||||
<button
|
<button
|
||||||
class="flex items-center gap-1.5 btn-md rounded-lg text-text-secondary hover:bg-hover hover:text-text"
|
class="flex items-center gap-1.5 btn-md rounded-lg text-text-secondary hover:bg-hover hover:text-text"
|
||||||
style="transition-duration: var(--duration-ui)"
|
style="transition-duration: var(--duration-ui)"
|
||||||
@@ -451,6 +533,13 @@
|
|||||||
onclick={() => setEffort(task.id, e)}
|
onclick={() => setEffort(task.id, e)}
|
||||||
>{effortLabel(e)}</button>
|
>{effortLabel(e)}</button>
|
||||||
{/each}
|
{/each}
|
||||||
|
<span class="text-border-subtle">·</span>
|
||||||
|
<!-- Energy chip (Phase 3) -->
|
||||||
|
<EnergyChip
|
||||||
|
energy={task.energy}
|
||||||
|
onSelect={(next) => setTaskEnergy(task.id, next)}
|
||||||
|
size="md"
|
||||||
|
/>
|
||||||
<!-- Timestamp -->
|
<!-- Timestamp -->
|
||||||
{#if task.createdAt}
|
{#if task.createdAt}
|
||||||
<span class="text-[10px] text-text-tertiary ml-auto">{formatTimestamp(task.createdAt)}</span>
|
<span class="text-[10px] text-text-tertiary ml-auto">{formatTimestamp(task.createdAt)}</span>
|
||||||
|
|||||||
230
src/lib/stores/focusTimer.svelte.ts
Normal file
230
src/lib/stores/focusTimer.svelte.ts
Normal file
@@ -0,0 +1,230 @@
|
|||||||
|
// Focus timer store. Single active timer at a time — the "just-start"
|
||||||
|
// 2/5/10/15-minute countdown paired with micro-steps. Exposes:
|
||||||
|
// - focusTimer.active: whether a timer is currently running
|
||||||
|
// - focusTimer.progress: 0..1 fraction of elapsed time
|
||||||
|
// - focusTimer.remainingMs: milliseconds until completion
|
||||||
|
// - focusTimer.label / focusTimer.taskId: what this timer is for
|
||||||
|
// - start(seconds, opts) / cancel() / extend(seconds)
|
||||||
|
//
|
||||||
|
// Survives window close + reopen via localStorage, because a timer
|
||||||
|
// that loses its clock when the user alt-tabs is a timer that nobody
|
||||||
|
// trusts. On rehydrate after expiry, fires completion then clears —
|
||||||
|
// so closing the window mid-timer still gets you the "done" signal
|
||||||
|
// on next launch.
|
||||||
|
|
||||||
|
const STORAGE_KEY = "kon.focusTimer.v1";
|
||||||
|
const TICK_INTERVAL_MS = 250;
|
||||||
|
|
||||||
|
export type FocusTimerPersisted = {
|
||||||
|
startedAt: number;
|
||||||
|
durationMs: number;
|
||||||
|
taskId: string | null;
|
||||||
|
label: string | null;
|
||||||
|
};
|
||||||
|
|
||||||
|
function readPersisted(): FocusTimerPersisted | null {
|
||||||
|
if (typeof window === "undefined") return null;
|
||||||
|
try {
|
||||||
|
const raw = window.localStorage.getItem(STORAGE_KEY);
|
||||||
|
if (!raw) return null;
|
||||||
|
const parsed = JSON.parse(raw);
|
||||||
|
if (
|
||||||
|
typeof parsed?.startedAt !== "number" ||
|
||||||
|
typeof parsed?.durationMs !== "number"
|
||||||
|
) return null;
|
||||||
|
return {
|
||||||
|
startedAt: parsed.startedAt,
|
||||||
|
durationMs: parsed.durationMs,
|
||||||
|
taskId: parsed.taskId ?? null,
|
||||||
|
label: parsed.label ?? null,
|
||||||
|
};
|
||||||
|
} catch {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function writePersisted(state: FocusTimerPersisted | null): void {
|
||||||
|
if (typeof window === "undefined") return;
|
||||||
|
try {
|
||||||
|
if (state === null) window.localStorage.removeItem(STORAGE_KEY);
|
||||||
|
else window.localStorage.setItem(STORAGE_KEY, JSON.stringify(state));
|
||||||
|
} catch { /* storage may be disabled; non-fatal */ }
|
||||||
|
}
|
||||||
|
|
||||||
|
function createFocusTimerStore() {
|
||||||
|
let startedAt = $state<number | null>(null);
|
||||||
|
let durationMs = $state<number>(0);
|
||||||
|
let taskId = $state<string | null>(null);
|
||||||
|
let label = $state<string | null>(null);
|
||||||
|
let now = $state<number>(Date.now());
|
||||||
|
let completionFlashUntil = $state<number>(0);
|
||||||
|
|
||||||
|
let interval: ReturnType<typeof setInterval> | null = null;
|
||||||
|
|
||||||
|
const active = $derived(startedAt !== null);
|
||||||
|
const elapsedMs = $derived(startedAt === null ? 0 : Math.max(0, now - startedAt));
|
||||||
|
const remainingMs = $derived(Math.max(0, durationMs - elapsedMs));
|
||||||
|
const progress = $derived(durationMs === 0 ? 0 : Math.min(1, elapsedMs / durationMs));
|
||||||
|
const finished = $derived(active && remainingMs === 0);
|
||||||
|
const showingCompletionFlash = $derived(now < completionFlashUntil);
|
||||||
|
|
||||||
|
// Internal completion + flash bookkeeping. We track whether we have
|
||||||
|
// already fired the completion chime for the current timer so a
|
||||||
|
// second tick does not re-fire it. Reset whenever a new timer starts.
|
||||||
|
let completionFired = false;
|
||||||
|
|
||||||
|
function tick() {
|
||||||
|
now = Date.now();
|
||||||
|
// Fire completion once, the first tick after we cross remaining=0.
|
||||||
|
if (startedAt !== null && !completionFired && now - startedAt >= durationMs) {
|
||||||
|
completionFired = true;
|
||||||
|
completionFlashUntil = now + 3000;
|
||||||
|
fireCompletion();
|
||||||
|
}
|
||||||
|
// After the 3 s flash window, clear everything and stop ticking.
|
||||||
|
if (completionFlashUntil > 0 && now >= completionFlashUntil) {
|
||||||
|
clear();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function startTick() {
|
||||||
|
if (interval !== null) return;
|
||||||
|
interval = setInterval(tick, TICK_INTERVAL_MS);
|
||||||
|
}
|
||||||
|
|
||||||
|
function stopTick() {
|
||||||
|
if (interval !== null) {
|
||||||
|
clearInterval(interval);
|
||||||
|
interval = null;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function clear() {
|
||||||
|
startedAt = null;
|
||||||
|
durationMs = 0;
|
||||||
|
taskId = null;
|
||||||
|
label = null;
|
||||||
|
completionFlashUntil = 0;
|
||||||
|
completionFired = false;
|
||||||
|
writePersisted(null);
|
||||||
|
stopTick();
|
||||||
|
}
|
||||||
|
|
||||||
|
function start(seconds: number, opts?: { taskId?: string | null; label?: string | null }): void {
|
||||||
|
if (!Number.isFinite(seconds) || seconds <= 0) return;
|
||||||
|
now = Date.now();
|
||||||
|
startedAt = now;
|
||||||
|
durationMs = Math.floor(seconds * 1000);
|
||||||
|
taskId = opts?.taskId ?? null;
|
||||||
|
label = opts?.label ?? null;
|
||||||
|
completionFlashUntil = 0;
|
||||||
|
completionFired = false;
|
||||||
|
writePersisted({ startedAt, durationMs, taskId, label });
|
||||||
|
startTick();
|
||||||
|
}
|
||||||
|
|
||||||
|
function cancel(): void {
|
||||||
|
clear();
|
||||||
|
}
|
||||||
|
|
||||||
|
function extend(seconds: number): void {
|
||||||
|
if (startedAt === null) return;
|
||||||
|
if (!Number.isFinite(seconds) || seconds <= 0) return;
|
||||||
|
durationMs += Math.floor(seconds * 1000);
|
||||||
|
writePersisted({ startedAt, durationMs, taskId, label });
|
||||||
|
}
|
||||||
|
|
||||||
|
function dismissCompletionFlash(): void {
|
||||||
|
completionFlashUntil = 0;
|
||||||
|
clear();
|
||||||
|
}
|
||||||
|
|
||||||
|
function fireCompletion(): void {
|
||||||
|
if (typeof window === "undefined") return;
|
||||||
|
// Gentle chime — WebAudio so we do not ship a bundled asset.
|
||||||
|
// A 440 Hz fall into 330 Hz over 220 ms, low volume, no sustain.
|
||||||
|
try {
|
||||||
|
type AudioCtx = typeof AudioContext;
|
||||||
|
const win = window as unknown as { AudioContext?: AudioCtx; webkitAudioContext?: AudioCtx };
|
||||||
|
const Ctx = win.AudioContext ?? win.webkitAudioContext;
|
||||||
|
if (!Ctx) return;
|
||||||
|
const ctx = new Ctx();
|
||||||
|
const osc = ctx.createOscillator();
|
||||||
|
const gain = ctx.createGain();
|
||||||
|
osc.type = "sine";
|
||||||
|
osc.frequency.setValueAtTime(440, ctx.currentTime);
|
||||||
|
osc.frequency.exponentialRampToValueAtTime(330, ctx.currentTime + 0.22);
|
||||||
|
gain.gain.setValueAtTime(0.0001, ctx.currentTime);
|
||||||
|
gain.gain.exponentialRampToValueAtTime(0.15, ctx.currentTime + 0.02);
|
||||||
|
gain.gain.exponentialRampToValueAtTime(0.0001, ctx.currentTime + 0.28);
|
||||||
|
osc.connect(gain);
|
||||||
|
gain.connect(ctx.destination);
|
||||||
|
osc.start();
|
||||||
|
osc.stop(ctx.currentTime + 0.3);
|
||||||
|
osc.onended = () => ctx.close().catch(() => {});
|
||||||
|
} catch { /* audio is a nicety; never fatal */ }
|
||||||
|
|
||||||
|
window.dispatchEvent(new CustomEvent("kon:focus-timer-complete", {
|
||||||
|
detail: { taskId, label },
|
||||||
|
}));
|
||||||
|
}
|
||||||
|
|
||||||
|
// Rehydrate on first touch. If the persisted timer has already
|
||||||
|
// expired, fire completion then clear so the user still gets the
|
||||||
|
// "done" signal they missed while the window was closed.
|
||||||
|
function rehydrate(): void {
|
||||||
|
const persisted = readPersisted();
|
||||||
|
if (!persisted) return;
|
||||||
|
const age = Date.now() - persisted.startedAt;
|
||||||
|
if (age >= persisted.durationMs) {
|
||||||
|
// Already expired while the window was closed. Fire a completion
|
||||||
|
// event so downstream listeners (nudges, UI flourishes) can react.
|
||||||
|
taskId = persisted.taskId;
|
||||||
|
label = persisted.label;
|
||||||
|
durationMs = persisted.durationMs;
|
||||||
|
startedAt = persisted.startedAt;
|
||||||
|
now = persisted.startedAt + persisted.durationMs;
|
||||||
|
// Flash briefly so the user knows it happened.
|
||||||
|
completionFlashUntil = Date.now() + 3000;
|
||||||
|
completionFired = true;
|
||||||
|
fireCompletion();
|
||||||
|
startTick();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
startedAt = persisted.startedAt;
|
||||||
|
durationMs = persisted.durationMs;
|
||||||
|
taskId = persisted.taskId;
|
||||||
|
label = persisted.label;
|
||||||
|
now = Date.now();
|
||||||
|
startTick();
|
||||||
|
}
|
||||||
|
|
||||||
|
// Exposed as frozen object. Getters so derivations stay reactive.
|
||||||
|
return {
|
||||||
|
get active() { return active; },
|
||||||
|
get progress() { return progress; },
|
||||||
|
get elapsedMs() { return elapsedMs; },
|
||||||
|
get remainingMs() { return remainingMs; },
|
||||||
|
get durationMs() { return durationMs; },
|
||||||
|
get taskId() { return taskId; },
|
||||||
|
get label() { return label; },
|
||||||
|
get showingCompletionFlash() { return showingCompletionFlash; },
|
||||||
|
start,
|
||||||
|
cancel,
|
||||||
|
extend,
|
||||||
|
rehydrate,
|
||||||
|
dismissCompletionFlash,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
export const focusTimer = createFocusTimerStore();
|
||||||
|
|
||||||
|
// Preset durations surfaced in the UI. 2 / 5 / 10 / 15 minutes match
|
||||||
|
// the brief's guidance for the "just-start" timer and cover common
|
||||||
|
// Pomodoro / shorter-focus preferences.
|
||||||
|
export const FOCUS_TIMER_PRESETS_SECONDS: ReadonlyArray<{ label: string; seconds: number }> = [
|
||||||
|
{ label: "2 min", seconds: 120 },
|
||||||
|
{ label: "5 min", seconds: 300 },
|
||||||
|
{ label: "10 min", seconds: 600 },
|
||||||
|
{ label: "15 min", seconds: 900 },
|
||||||
|
];
|
||||||
@@ -1,5 +1,6 @@
|
|||||||
import { invoke } from "@tauri-apps/api/core";
|
import { invoke } from "@tauri-apps/api/core";
|
||||||
import type {
|
import type {
|
||||||
|
EnergyLevel,
|
||||||
PageState,
|
PageState,
|
||||||
Profile,
|
Profile,
|
||||||
Segment,
|
Segment,
|
||||||
@@ -68,6 +69,8 @@ const defaults: SettingsState = {
|
|||||||
globalHotkey: "Ctrl+Shift+R",
|
globalHotkey: "Ctrl+Shift+R",
|
||||||
sidebarCollapsed: false,
|
sidebarCollapsed: false,
|
||||||
microphoneDevice: "",
|
microphoneDevice: "",
|
||||||
|
currentEnergy: null,
|
||||||
|
matchMyEnergy: false,
|
||||||
};
|
};
|
||||||
|
|
||||||
function canUseStorage(): boolean {
|
function canUseStorage(): boolean {
|
||||||
@@ -314,6 +317,7 @@ function mapTaskRow(row: TaskDto): TaskEntry {
|
|||||||
createdAt: row.createdAt ?? new Date().toISOString(),
|
createdAt: row.createdAt ?? new Date().toISOString(),
|
||||||
sourceTranscriptId: row.sourceTranscriptId ?? null,
|
sourceTranscriptId: row.sourceTranscriptId ?? null,
|
||||||
parentTaskId: row.parentTaskId ?? null,
|
parentTaskId: row.parentTaskId ?? null,
|
||||||
|
energy: row.energy ?? null,
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -352,6 +356,7 @@ export async function addTask(task: TaskDraft) {
|
|||||||
sourceTranscriptId: task.sourceTranscriptId || null,
|
sourceTranscriptId: task.sourceTranscriptId || null,
|
||||||
listId: task.listId ?? null,
|
listId: task.listId ?? null,
|
||||||
effort: task.effort ?? null,
|
effort: task.effort ?? null,
|
||||||
|
energy: task.energy ?? null,
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
tasks.unshift(mapTaskRow(row));
|
tasks.unshift(mapTaskRow(row));
|
||||||
@@ -396,6 +401,29 @@ export async function updateTask(id: string, updates: TaskUpdate) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Phase 3: explicit tri-state energy setter. Lives outside `updateTask`
|
||||||
|
* because the backend uses a dedicated command for clearing — see
|
||||||
|
* `set_task_energy_cmd` in src-tauri/src/commands/tasks.rs. Pass `null`
|
||||||
|
* to clear the tag entirely; pass an `EnergyLevel` string to set it.
|
||||||
|
*/
|
||||||
|
export async function setTaskEnergy(id: string, energy: EnergyLevel | null) {
|
||||||
|
if (!hasTauriRuntime()) {
|
||||||
|
applyLocalTaskUpdate(id, { energy });
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
try {
|
||||||
|
const row = await invoke<TaskDto>("set_task_energy_cmd", { id, energy });
|
||||||
|
const idx = tasks.findIndex((task) => task.id === id);
|
||||||
|
if (idx >= 0) {
|
||||||
|
tasks[idx] = mapTaskRow(row);
|
||||||
|
broadcastTasks();
|
||||||
|
}
|
||||||
|
} catch (err) {
|
||||||
|
toasts.error("Couldn't change energy", errorMessage(err));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
export async function deleteTask(id: string) {
|
export async function deleteTask(id: string) {
|
||||||
if (!hasTauriRuntime()) return;
|
if (!hasTauriRuntime()) return;
|
||||||
|
|
||||||
|
|||||||
@@ -56,6 +56,20 @@ export interface SettingsState {
|
|||||||
globalHotkey: string;
|
globalHotkey: string;
|
||||||
sidebarCollapsed: boolean;
|
sidebarCollapsed: boolean;
|
||||||
microphoneDevice: string;
|
microphoneDevice: string;
|
||||||
|
/**
|
||||||
|
* Phase 3 match-my-energy: the user's self-reported current energy
|
||||||
|
* level. `null` means "not stated" — the sort falls back to created_at
|
||||||
|
* order. Persists across sessions because energy tracks the person,
|
||||||
|
* not the work.
|
||||||
|
*/
|
||||||
|
currentEnergy: EnergyLevel | null;
|
||||||
|
/**
|
||||||
|
* Phase 3 match-my-energy: when true, the Tasks page sorts tasks
|
||||||
|
* matching `currentEnergy` to the top (with unset tasks treated as
|
||||||
|
* Medium). Off by default so the list reads chronologically until
|
||||||
|
* the user explicitly opts in.
|
||||||
|
*/
|
||||||
|
matchMyEnergy: boolean;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface Profile {
|
export interface Profile {
|
||||||
@@ -150,6 +164,14 @@ export interface TranscriptMetaPatch {
|
|||||||
segments?: Segment[];
|
segments?: Segment[];
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Phase 3: energy level tag on a task. `null` means unset — the sort
|
||||||
|
* treats unset as Medium-equivalent. Three tagged values match the
|
||||||
|
* brief's "High / Medium / Brain-Dead" language; `brain_dead` is the
|
||||||
|
* stored form because SQL enums don't love hyphens.
|
||||||
|
*/
|
||||||
|
export type EnergyLevel = "high" | "medium" | "brain_dead";
|
||||||
|
|
||||||
export interface TaskDto {
|
export interface TaskDto {
|
||||||
id: string;
|
id: string;
|
||||||
text: string;
|
text: string;
|
||||||
@@ -162,6 +184,7 @@ export interface TaskDto {
|
|||||||
createdAt: string;
|
createdAt: string;
|
||||||
sourceTranscriptId: string | null;
|
sourceTranscriptId: string | null;
|
||||||
parentTaskId: string | null;
|
parentTaskId: string | null;
|
||||||
|
energy: EnergyLevel | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface TaskEntry {
|
export interface TaskEntry {
|
||||||
@@ -176,6 +199,7 @@ export interface TaskEntry {
|
|||||||
createdAt: string;
|
createdAt: string;
|
||||||
sourceTranscriptId: string | null;
|
sourceTranscriptId: string | null;
|
||||||
parentTaskId: string | null;
|
parentTaskId: string | null;
|
||||||
|
energy: EnergyLevel | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface TaskDraft {
|
export interface TaskDraft {
|
||||||
@@ -184,6 +208,7 @@ export interface TaskDraft {
|
|||||||
listId?: string | null;
|
listId?: string | null;
|
||||||
effort?: string | null;
|
effort?: string | null;
|
||||||
sourceTranscriptId?: string | null;
|
sourceTranscriptId?: string | null;
|
||||||
|
energy?: EnergyLevel | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface TaskUpdate {
|
export interface TaskUpdate {
|
||||||
|
|||||||
@@ -8,6 +8,7 @@
|
|||||||
import Titlebar from "$lib/components/Titlebar.svelte";
|
import Titlebar from "$lib/components/Titlebar.svelte";
|
||||||
import ToastViewport from "$lib/components/ToastViewport.svelte";
|
import ToastViewport from "$lib/components/ToastViewport.svelte";
|
||||||
import ResizeHandles from "$lib/components/ResizeHandles.svelte";
|
import ResizeHandles from "$lib/components/ResizeHandles.svelte";
|
||||||
|
import FocusTimer from "$lib/components/FocusTimer.svelte";
|
||||||
import { hasTauriRuntime } from "$lib/utils/runtime.js";
|
import { hasTauriRuntime } from "$lib/utils/runtime.js";
|
||||||
import { loadOsInfo, isLinux } from "$lib/utils/osInfo.js";
|
import { loadOsInfo, isLinux } from "$lib/utils/osInfo.js";
|
||||||
import { page, settings, saveSettings } from "$lib/stores/page.svelte.js";
|
import { page, settings, saveSettings } from "$lib/stores/page.svelte.js";
|
||||||
@@ -396,6 +397,13 @@
|
|||||||
in the bottom-right of the viewport. (Day 3 of the upgrade plan) -->
|
in the bottom-right of the viewport. (Day 3 of the upgrade plan) -->
|
||||||
<ToastViewport />
|
<ToastViewport />
|
||||||
|
|
||||||
|
<!-- Global focus-timer overlay. Renders nothing until a `kon:start-timer`
|
||||||
|
event fires; then pins a shrinking colour ring to the top-right.
|
||||||
|
Phase 1 of the 2026-04-23 feature-complete roadmap — closes the
|
||||||
|
visual-time-representation gap from docs/brief/feature-set.md and
|
||||||
|
wires the dangling emit in MicroSteps.svelte. -->
|
||||||
|
<FocusTimer />
|
||||||
|
|
||||||
<!-- Invisible resize margins for frameless (macOS/Windows). On Linux we
|
<!-- Invisible resize margins for frameless (macOS/Windows). On Linux we
|
||||||
use native decorations, so ResizeHandles would compete with the
|
use native decorations, so ResizeHandles would compete with the
|
||||||
compositor's own resize and is suppressed. -->
|
compositor's own resize and is suppressed. -->
|
||||||
|
|||||||
@@ -12,6 +12,7 @@
|
|||||||
PREFERENCES_CHANGED_EVENT,
|
PREFERENCES_CHANGED_EVENT,
|
||||||
} from "$lib/stores/preferences.svelte.js";
|
} from "$lib/stores/preferences.svelte.js";
|
||||||
import Titlebar from "$lib/components/Titlebar.svelte";
|
import Titlebar from "$lib/components/Titlebar.svelte";
|
||||||
|
import FocusTimer from "$lib/components/FocusTimer.svelte";
|
||||||
import { loadOsInfo, isLinux } from "$lib/utils/osInfo.js";
|
import { loadOsInfo, isLinux } from "$lib/utils/osInfo.js";
|
||||||
|
|
||||||
let { children } = $props();
|
let { children } = $props();
|
||||||
@@ -90,3 +91,9 @@
|
|||||||
{@render children()}
|
{@render children()}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
<!-- Focus timer also visible in the always-on-top float window so a
|
||||||
|
running countdown stays with the Now list. The component is a
|
||||||
|
global overlay (position: fixed) so it pins to the top-right of
|
||||||
|
this window independent of the Tasks content below. -->
|
||||||
|
<FocusTimer />
|
||||||
|
|||||||
Reference in New Issue
Block a user