Fix cargo clippy warnings across the workspace

- Derive Default for Config instead of manually mirroring each sub-config's Default impl - Use std::io::Error::other instead of Error::new(ErrorKind::Other, ..) - Drop the unused mut on the store lock in full_reindex - Remove Store::dim, a field that was set once and never read - Use char_indices().enumerate() instead of a hand-rolled loop counter in split_by_chars, and is_multiple_of() for the chunk boundary check - Use strip_prefix instead of manual slicing in expand_home (cherry picked from commit b740af38e17d04bd83db93bbb0585ed5744a436e)
Adopt gtk_popup overlay helpers and singleton::toggle_or_kill
2026-08-23 14:57:59 +08:00 · 2026-08-23 14:39:11 +08:00 · 2026-08-16 14:09:28 +08:00 · 2026-08-16 13:23:15 +08:00 · 2026-08-16 00:50:22 +08:00 · 2026-08-16 00:26:06 +08:00
29 changed files with 1602 additions and 788 deletions
--- a/.forgejo/workflows/check.yml
+++ b/.forgejo/workflows/check.yml
@ -0,0 +1,24 @@
+name: check
+
+# Fast-fail lint/test on short-lived work branches, before it ever reaches
+# main and triggers a dev-track release build.
+on:
+  push:
+    branches: ['feature/**', 'fix/**']
+
+jobs:
+  check:
+    runs-on: [self-hosted, hestia]
+    steps:
+      - name: checkout
+        run: |
+          set -euo pipefail
+          rm -rf src && mkdir src
+          git clone --branch "${GITHUB_REF_NAME}" --depth 1 \
+            "https://git.breadway.dev/${GITHUB_REPOSITORY}.git" src
+
+      - name: clippy
+        run: cd src && bash ci/build.sh cargo clippy --workspace --all-targets --locked --features full -- -D warnings
+
+      - name: test
+        run: cd src && bash ci/build.sh cargo test --workspace --locked --features full
--- a/.forgejo/workflows/dev-release.yml
+++ b/.forgejo/workflows/dev-release.yml
@ -0,0 +1,82 @@
+name: dev release
+
+# Publishes a dev-track build on every push to `main` (the trunk
+# branch — there is no separate `dev` branch). See bread-ecosystem's
+# docs/release-channels.md for the release-track policy this is part of.
+on:
+  push:
+    branches: ['main']
+
+jobs:
+  build:
+    runs-on: [self-hosted, hestia]
+    steps:
+      - name: checkout
+        run: |
+          set -euo pipefail
+          rm -rf src && mkdir src
+          git clone --branch main --depth 1 \
+            "https://git.breadway.dev/${GITHUB_REPOSITORY}.git" src
+
+      - name: build
+        run: cd src && bash ci/build.sh cargo build --release --locked --workspace --features full
+
+      - name: test
+        run: cd src && bash ci/build.sh cargo test --release --locked --workspace --features full
+
+      - name: compute dev version
+        run: |
+          set -euo pipefail
+          cd src
+          # Base the dev version off the latest published stable tag,
+          # not Cargo.toml — Cargo.toml can go stale relative to the last
+          # real release (seen in practice: breadbox/breadpad/breadcrumbs/
+          # breadpaper), which would make a dev build sort as OLDER than
+          # what's already installed and bakery would correctly refuse it.
+          LATEST_TAG="$(git ls-remote --tags --refs \
+            "https://git.breadway.dev/${GITHUB_REPOSITORY}.git" 'v*' \
+            | awk -F/ '{print $NF}' | sed 's/^v//' | (grep -v -- '-' || true) | sort -V | tail -1)"
+          if [ -n "${LATEST_TAG}" ]; then
+            CUR="${LATEST_TAG}"
+          else
+            CUR="$(grep -m1 '^version' breadsearch/Cargo.toml | sed -E 's/.*"(.*)".*/\1/')"
+          fi
+          IFS='.' read -r MA MI PA <<< "${CUR}"
+          SHA="$(git rev-parse --short HEAD)"
+          TS="$(date -u +%Y%m%d%H%M%S)"
+          echo "VERSION=${MA}.${MI}.$((PA + 1))-dev.${TS}+${SHA}" >> "$GITHUB_ENV"
+
+      - name: prepare artifacts
+        run: |
+          set -euo pipefail
+          PKG_DIR="/srv/breadway-dl/dev/breadsearch/${VERSION}"
+          mkdir -p "${PKG_DIR}"
+          for bin in breadsearch breadmill; do
+            cp "src/target/release/${bin}" "${PKG_DIR}/${bin}-x86_64"
+            strip "${PKG_DIR}/${bin}-x86_64"
+            sha256sum "${PKG_DIR}/${bin}-x86_64" | awk '{print $1}' \
+              > "${PKG_DIR}/${bin}-x86_64.sha256"
+          done
+          cp src/packaging/breadmill.service "${PKG_DIR}/"
+          cp src/config.example.toml "${PKG_DIR}/"
+          cp src/bakery.toml "${PKG_DIR}/bakery.toml"
+          ln -sfn "${VERSION}" "/srv/breadway-dl/dev/breadsearch/latest"
+
+      # No GitHub Release upload — dev, like the other non-stable track,
+      # is only distributed via dl.breadway.dev/dev/.
+      - name: regenerate dev index.json
+        env:
+          MINISIGN_SEC_KEY: ${{ secrets.BAKERY_MINISIGN_SEC_KEY_PATH }}
+        run: |
+          set -euo pipefail
+          if [ -z "${MINISIGN_SEC_KEY:-}" ]; then
+            echo "::error::BAKERY_MINISIGN_SEC_KEY_PATH secret not set — refusing to regenerate dev index.json unsigned (would leave a stale signature mismatched against fresh content and break bakery for everyone on the dev track)"
+            exit 1
+          fi
+          rm -rf /tmp/bread-ecosystem-ci-* 2>/dev/null || true
+          # mktemp: a fixed clone path races when multiple repos' dev/beta
+          # workflows run close together on the same self-hosted runner.
+          ECOSYSTEM_CI_DIR="$(mktemp -d /tmp/bread-ecosystem-ci-XXXXXX)"
+          git clone --branch main https://git.breadway.dev/Breadway/bread-ecosystem.git "${ECOSYSTEM_CI_DIR}"
+          TRACK=dev bash "${ECOSYSTEM_CI_DIR}/scripts/gen-index.sh"
+          rm -rf "${ECOSYSTEM_CI_DIR}"
--- a/.forgejo/workflows/mirror.yml
+++ b/.forgejo/workflows/mirror.yml
@ -1,19 +0,0 @@
-name: Mirror to GitHub
-
-on:
-  push:
-    branches: ['**']
-    tags: ['**']
-
-jobs:
-  mirror:
-    runs-on: [self-hosted, hestia]
-    steps:
-      - name: Mirror to GitHub
-        run: |
-          set -euo pipefail
-          git clone --mirror "https://git.breadway.dev/${GITHUB_REPOSITORY}.git" repo.git
-          cd repo.git
-          git push --prune \
-            "https://x-access-token:${{ secrets.MIRROR_TOKEN }}@github.com/Breadway/breadsearch.git" \
-            '+refs/heads/*:refs/heads/*' '+refs/tags/*:refs/tags/*'
--- a/.forgejo/workflows/rc-release.yml
+++ b/.forgejo/workflows/rc-release.yml
@ -0,0 +1,63 @@
+name: beta (rc) release
+
+# Publishes a beta-track build for any `vX.Y.Z-rc.N` prerelease tag
+# pushed to `main` — there is no separate `beta` branch; "freezing" is
+# just pausing pushes to main while an RC gets tested. See
+# bread-ecosystem's docs/release-channels.md for the release-track policy.
+on:
+  push:
+    tags: ['v*']
+
+jobs:
+  build:
+    if: ${{ contains(github.ref_name, '-rc.') }}
+    runs-on: [self-hosted, hestia]
+    steps:
+      - name: checkout
+        run: |
+          set -euo pipefail
+          rm -rf src && mkdir src
+          git clone --branch "${GITHUB_REF_NAME}" --depth 1 \
+            "https://git.breadway.dev/${GITHUB_REPOSITORY}.git" src
+
+      - name: build
+        run: cd src && bash ci/build.sh cargo build --release --locked --workspace --features full
+
+      - name: test
+        run: cd src && bash ci/build.sh cargo test --release --locked --workspace --features full
+
+      - name: prepare artifacts
+        run: |
+          set -euo pipefail
+          VERSION="${GITHUB_REF_NAME#v}"
+          PKG_DIR="/srv/breadway-dl/beta/breadsearch/${VERSION}"
+          mkdir -p "${PKG_DIR}"
+          for bin in breadsearch breadmill; do
+            cp "src/target/release/${bin}" "${PKG_DIR}/${bin}-x86_64"
+            strip "${PKG_DIR}/${bin}-x86_64"
+            sha256sum "${PKG_DIR}/${bin}-x86_64" | awk '{print $1}' \
+              > "${PKG_DIR}/${bin}-x86_64.sha256"
+          done
+          cp src/packaging/breadmill.service "${PKG_DIR}/"
+          cp src/config.example.toml "${PKG_DIR}/"
+          cp src/bakery.toml "${PKG_DIR}/bakery.toml"
+          ln -sfn "${VERSION}" "/srv/breadway-dl/beta/breadsearch/latest"
+
+      # No GitHub Release upload — beta, like dev, is only distributed via
+      # dl.breadway.dev/beta/.
+      - name: regenerate beta index.json
+        env:
+          MINISIGN_SEC_KEY: ${{ secrets.BAKERY_MINISIGN_SEC_KEY_PATH }}
+        run: |
+          set -euo pipefail
+          if [ -z "${MINISIGN_SEC_KEY:-}" ]; then
+            echo "::error::BAKERY_MINISIGN_SEC_KEY_PATH secret not set — refusing to regenerate beta index.json unsigned (would leave a stale signature mismatched against fresh content and break bakery for everyone on the beta track)"
+            exit 1
+          fi
+          rm -rf /tmp/bread-ecosystem-ci-* 2>/dev/null || true
+          # mktemp: a fixed clone path races when multiple repos' dev/beta
+          # workflows run close together on the same self-hosted runner.
+          ECOSYSTEM_CI_DIR="$(mktemp -d /tmp/bread-ecosystem-ci-XXXXXX)"
+          git clone https://git.breadway.dev/Breadway/bread-ecosystem.git "${ECOSYSTEM_CI_DIR}"
+          TRACK=beta bash "${ECOSYSTEM_CI_DIR}/scripts/gen-index.sh"
+          rm -rf "${ECOSYSTEM_CI_DIR}"
--- a/.forgejo/workflows/release.yml
+++ b/.forgejo/workflows/release.yml
@ -6,6 +6,7 @@ on:

 jobs:
  build:
+    if: ${{ !contains(github.ref_name, '-rc.') }}
    runs-on: [self-hosted, hestia]
    steps:
      - name: checkout
@ -21,10 +22,19 @@ jobs:
        # `backend` in config.toml. All three are ort load-dynamic (dlopen)
        # EPs, so this doesn't require the NPU/ROCm/CUDA toolkits to be
        # present on the build host — see breadmill/Cargo.toml.
-        run: cd src && cargo build --release --locked --workspace --features full
+        run: |
+          set -euo pipefail
+          if [ ! -f src/ci/build.sh ]; then
+            echo "::error::ci/build.sh is missing — bakery release builds must go through the shared CI wrapper"
+            exit 1
+          fi
+          cd src && bash ci/build.sh cargo build --release --locked --workspace --features full || {
+            echo "::error::cargo build --release --locked failed. If Cargo.lock drifted, update and commit it; do not drop --locked."
+            exit 1
+          }

      - name: test
-        run: cd src && cargo test --release --locked --workspace --features full
+        run: cd src && bash ci/build.sh cargo test --release --locked --workspace --features full

      - name: prepare artifacts
        run: |
@ -44,8 +54,14 @@ jobs:
          ln -sfn "${VERSION}" "/srv/breadway-dl/breadsearch/latest"

      - name: regenerate index.json
+        env:
+          MINISIGN_SEC_KEY: ${{ secrets.BAKERY_MINISIGN_SEC_KEY_PATH }}
        run: |
          set -euo pipefail
+          if [ -z "${MINISIGN_SEC_KEY:-}" ]; then
+            echo "::error::BAKERY_MINISIGN_SEC_KEY_PATH secret not set — refusing to regenerate index.json unsigned (would leave a stale signature mismatched against fresh content and break bakery for everyone)"
+            exit 1
+          fi
          rm -rf /tmp/bread-ecosystem-ci
          git clone https://git.breadway.dev/Breadway/bread-ecosystem.git /tmp/bread-ecosystem-ci
          bash /tmp/bread-ecosystem-ci/scripts/gen-index.sh
--- a/AGENTS.md
+++ b/AGENTS.md
@ -0,0 +1,50 @@
+# AGENTS.md — Repo hygiene
+
+Scope: this file covers *repo hygiene* — branching, remotes, CI — plus a
+short map of the binaries. It is not user-facing project documentation.
+
+This repo follows the branch/release workflow documented in `CONTRIBUTING.md`
+— read and follow it for any git, branch, or release work here (the
+single-trunk model, `feature/x`/`fix/x` branch naming, how RC tags work,
+etc). Don't improvise a different workflow. The short version: there is one
+long-lived branch, `main` — no `dev` or `beta` branch exists. `main`
+auto-publishes a dev-track build on every push. "Beta" and "stable" are both
+just tags, not branches: push a `vX.Y.Z-rc.N` tag to publish a beta-track
+build, push a plain `vX.Y.Z` tag to cut the signed stable release.
+"Freezing" for stabilization means pausing pushes to `main`, not moving a
+branch. This replaced an earlier three-branch (`dev`/`beta`/`main`) model
+after `main` was found to have silently rotted out of sync with `dev`/`beta`
+across most repos in this ecosystem.
+
+When starting work on a new feature, create branch `feature/<feature-name>`.
+When working on a bug or issue, create branch `fix/<issue you are fixing>`.
+
+## Remotes
+- `origin` — Forgejo (`git.breadway.dev` via Hestia, SSH) — authoritative.
+- `github` — GitHub mirror. Push both when publishing.
+
+## CI
+- `dev-release.yml` triggers on `push: branches: ['main']`.
+- `rc-release.yml` triggers on `vX.Y.Z-rc.N` tag pushes (beta track).
+- `release.yml` triggers on any other `v*` tag push (stable).
+  None of these run on plain commits or PRs beyond what's listed.
+
+## Architecture
+
+Two binaries + a shared lib:
+
+| Crate | Role |
+|---|---|
+| `breadsearch-shared` | XDG paths, config, IPC types, Unix-socket client |
+| `breadmill` | Daemon: walk → extract → chunk → embed → index; serves queries |
+| `breadsearch` | GTK4 layer-shell overlay: ranked document hits from breadmill |
+
+breadsearch is document/meaning search. breadbox is the app launcher. They
+are not the same overlay.
+
+`--screenshot` (`breadsearch/src/screenshot.rs`) captures the search panel
+through `bread-screenshots`; do not rewrite it just to retarget the crate pin.
+
+## Don't
+- Don't embed credentials in remote URLs — SSH or a credential helper only.
+- Don't rewrite `screenshot.rs` as part of pin/docs work.
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@ -0,0 +1,84 @@
+# Contributing
+
+`breadsearch` — Semantic system-wide search for BOS.
+
+Part of the bread ecosystem; this repo follows the same branch/release
+workflow as every other ecosystem product.
+
+## Branches
+
+There is one long-lived branch: **`main`**. All day-to-day work lands here.
+Every push to `main` automatically builds and publishes a **dev-track**
+build (see Tracks below) — a real install you can test before cutting
+anything more formal.
+
+New work — features and bug fixes alike — goes on a short-lived branch:
+
+```
+feature/<short-name>
+fix/<issue-number-or-short-name>
+```
+
+Branch off `main`, open a PR/push back into `main` when ready. Short-lived
+branches get deleted on merge — they never accumulate the kind of drift a
+second long-lived branch does.
+
+## The release cycle
+
+There's no separate `beta` or release branch — "stable" and "beta" are both
+just **tags** on `main`, not branches that need to be kept in sync:
+
+1. Work accumulates on `main` via `feature/x` / `fix/x` branches. Each push
+   auto-publishes a dev build — install it with `bakery track set dev` and
+   `bakery update --all`, then fix anything broken with another push.
+2. When you want to stabilize before a real release, tag a release
+   candidate: `git tag vX.Y.Z-rc.1 && git push origin vX.Y.Z-rc.1` (push to
+   both remotes). That tag alone triggers a beta-track build —
+   "freezing" is just pausing pushes to `main` while you test it, not a
+   branch operation. Cut `-rc.2`, `-rc.3`, etc. for further fixes.
+3. Once an RC has gone without issues, tag the real release:
+   `git tag vX.Y.Z && git push origin vX.Y.Z` — that's what triggers the
+   signed stable release build.
+
+## Tracks, from a user's perspective
+
+```
+bakery track show              # what you're currently on (defaults to stable)
+bakery track set dev           # or beta, or stable
+bakery update --all            # pull the latest build on your current track
+```
+
+| Track  | What it is | Published from |
+|--------|-----------|-----------------|
+| `stable` | The last tagged release | a `vX.Y.Z` tag |
+| `beta` | Latest release candidate | a `vX.Y.Z-rc.N` tag |
+| `dev` | Bleeding edge | `main`, on every push |
+
+Dev versions are auto-computed (`X.Y.Z-dev.<timestamp>+<sha>`) from the
+latest published stable tag, so they always sort as newer than what you
+have installed — no manual version bumping needed. Beta versions are just
+the RC tag itself (already valid semver, already sorts below the real
+release it's a candidate for).
+
+## Local development
+
+```sh
+cargo build --release --workspace --features full
+cargo test --release --workspace --features full
+```
+
+## CI
+
+- `dev-release.yml` — triggered on push to `main`.
+- `rc-release.yml` — triggered on any `vX.Y.Z-rc.N` tag push.
+- `release.yml` — triggered on any other `v*` tag push, cuts the actual
+  stable release.
+
+All CI runs on a self-hosted runner; nothing runs automatically on plain
+commits or PRs beyond the track builds above. See
+[bread-ecosystem's docs/release-channels.md](https://git.breadway.dev/Breadway/bread-ecosystem/src/branch/main/docs/release-channels.md)
+for the full policy, including how a new product gets wired onto these tracks.
+
+## Questions
+
+Open an issue on this repo's Forgejo tracker.
--- a/Cargo.lock
+++ b/Cargo.lock
--- a/DESIGN.md
+++ b/DESIGN.md
@ -80,7 +80,9 @@ Start from `breadbox/breadbox/src/main.rs`. **Reuse verbatim:** the gtk4-layer-s
 - nomic prefixes + mean-pool + normalize must match between index and query or recall collapses.
 - `ort` linking: prefer the crate's downloaded/bundled ONNX Runtime to avoid version skew with Arch's `onnxruntime`.
 - Office formats (docx/odt) are best-effort in v1; md/txt/org/pdf are the reliable path.
- GPU EPs (ROCm/CUDA) fail to register silently at the ONNX Runtime level and fall back to CPU — always check
-  startup logs for `Successfully registered` before trusting a GPU build is actually accelerating. See
+- GPU EPs (ROCm/CUDA/OpenVINO) fail to register silently at the ONNX Runtime level and fall back to CPU — always
+  check startup logs for `Successfully registered` before trusting a GPU build is actually accelerating. See
  [README: GPU backend notes](README.md#gpu-backend-notes) for the MIGraphX-vs-ROCMExecutionProvider distinction
  and the per-shape JIT-compile-and-cache behavior that matters for interactive query latency.
+- CUDA and OpenVINO are compile-checked only — no NVIDIA or Intel GPU hardware in this dev environment (AMD-only)
+  to runtime-verify against, unlike ROCm which was confirmed end-to-end on real hardware.
--- a/EVENTS.md
+++ b/EVENTS.md
@ -0,0 +1,62 @@
+# breadsearch — bread event integration
+
+breadsearch is a standalone document-search overlay: it works exactly the
+same with or without `breadd` running. When breadd *is* present, the
+`breadsearch` GTK overlay publishes events into the shared bread automation
+fabric. See the parent `bread` repo's `Documentation.md` — specifically its
+"Namespaces" and "Integrating a bread\* app" sections — for the general
+convention this follows.
+
+App id: **`search`**. Transport: `bread-utils`'s `bread_client` module
+(feature `bread-client`) — the overlay links it directly. Each `emit` is
+its own short-lived connection (`BreadClient::emit` is fire-and-forget,
+the same stance as `bread-emit`). Command verbs are only received while
+`breadsearch listen` is running — that process holds the
+`bread.command.search.**` subscription open. breadmill, the indexing
+daemon, does not talk to breadd.
+
+## Events published (`bread.search.*`)
+
+| Event | Data | When |
+|-------|------|------|
+| `bread.search.opened` | `{}` | The overlay window maps (the search panel is shown). |
+| `bread.search.opened_result` | `{ "path": "<hit path>" }` | The user opens a hit — Enter / click opens the file, Ctrl+Enter reveals its folder. `path` is the hit's document path, not the parent folder. |
+| `bread.search.open.done` | `{}` | `bread.command.search.open` was received and `breadsearch` was spawned. This is the command confirmation, not proof the overlay mapped — the spawned process is the same PID-file toggle as a keybind. |
+| `bread.search.open.failed` | `{ "error": "<message>" }` | `bread.command.search.open` was received but this binary could not be started. |
+
+## Commands honored (`bread.command.search.*`)
+
+These are only received while `breadsearch listen` is running. Publishing a
+command with no subscriber is a silent no-op — that is the documented
+bread convention, not a breadsearch bug.
+
+| Verb | Data | Effect |
+|------|------|--------|
+| `open` | none | Same as running `breadsearch` (PID-file toggle: show the overlay, or dismiss it if it is already up). Emits `bread.search.open.done` / `.failed`. |
+
+```lua
+bread.spawn(function()
+    bread.emit("bread.command.search.open")
+    bread.wait("bread.search.open.done", { timeout = 5000 })
+end)
+```
+
+### Not implemented: extra verbs
+
+There is no `query` / `close` / `reindex` command verb. breadmill already
+has its own query socket; inventing a bus query plane would be a new
+product surface. If/when that exists, add the corresponding
+`bread.command.search.*` verb at the same time, not stubbed as a no-op
+ahead of it.
+
+## Fail-safe behavior
+
+- If breadd isn't installed or isn't running, `emit` is a silent no-op
+  (`BreadClient::emit` never blocks or errors the caller) and the
+  command subscription simply never receives anything — breadsearch's
+  overlay and breadmill's indexing/query path are entirely unaffected.
+- If breadd restarts, the command subscription reconnects automatically
+  (`BreadClient::subscribe`'s background thread has its own backoff
+  loop); no restart of `breadsearch listen` is needed.
+- If `breadsearch listen` is not running, commands are a graceful no-op at
+  the bus (no subscriber). The overlay CLI still works.
--- a/README.md
+++ b/README.md
@ -2,6 +2,8 @@

 Semantic document search for Bread OS. Type a concept — not a keyword — and get ranked hits from your documents.

+This is **not** [breadbox](https://git.breadway.dev/Breadway/breadbox). breadsearch is document/meaning search over your files; breadbox is the app launcher (fuzzy `.desktop` launch). They are separate overlays with separate binaries, sockets, and keybinds — even though the search panel was originally forked from breadbox's launcher UI.
+
 Two binaries in one Cargo workspace:

 - **breadmill** — background daemon. Walks configured directories, extracts text, chunks and embeds documents with [nomic-embed-text-v1.5](https://huggingface.co/nomic-ai/nomic-embed-text-v1.5) (768-dim ONNX), stores vectors in an HNSW index (usearch) backed by SQLite metadata, and serves queries over a Unix socket. Watches for filesystem changes and re-indexes incrementally.
@ -21,10 +23,11 @@ Optional features:

 | Feature | What it adds |
 |---------|-------------|
-| `npu`   | AMD XDNA NPU via VitisAI ONNX Runtime EP (requires Ryzen AI SDK) |
-| `rocm`  | AMD iGPU via the MIGraphX ONNX Runtime EP (ROCm-backed) |
-| `cuda`  | NVIDIA GPU via the CUDA ONNX Runtime EP |
-| `full`  | All three of the above in one binary |
+| `npu`      | AMD XDNA NPU via VitisAI ONNX Runtime EP (requires Ryzen AI SDK) |
+| `rocm`     | AMD iGPU via the MIGraphX ONNX Runtime EP (ROCm-backed) |
+| `cuda`     | NVIDIA GPU via the CUDA ONNX Runtime EP |
+| `openvino` | Intel iGPU/dGPU (Arc) via the OpenVINO ONNX Runtime EP |
+| `full`     | All four of the above in one binary |

 ```
 # NPU build
@ -36,25 +39,29 @@ cargo build --release -p breadmill --features rocm
 # CUDA (NVIDIA GPU) build
 cargo build --release -p breadmill --features cuda

+# OpenVINO (Intel iGPU/dGPU) build
+cargo build --release -p breadmill --features openvino
+
 # All backends in one binary (what the release build ships)
 cargo build --release -p breadmill --features full
 ```

-`rocm`/`cuda`/`npu` all use `ort`'s `load-dynamic` mode: at runtime, breadmill
-dlopens whatever `libonnxruntime.so` the dynamic linker resolves (or
+`rocm`/`cuda`/`npu`/`openvino` all use `ort`'s `load-dynamic` mode: at runtime,
+breadmill dlopens whatever `libonnxruntime.so` the dynamic linker resolves (or
 `ORT_DYLIB_PATH` if set). GPU acceleration only works if that ONNX Runtime
 build actually has the matching execution provider compiled in — breadmill
 logs a clear `Successfully registered` / `not enabled in this build` line for
 this at startup (see [GPU backend notes](#gpu-backend-notes) below).

-Because all three are dlopen-based, `full` doesn't require the NPU/ROCm/CUDA
-toolkits to be installed at build time — only at run time, and only for
-whichever single backend you actually select via `--npu`/`--rocm`/`--cuda`
-or `backend` in config.toml. The **released binaries are built with
-`full`**: same binary works CPU-only out of the box, and picks up NPU/ROCm/CUDA
-acceleration on a machine that has the matching ONNX Runtime available,
-without needing a different download. An explicit `--npu`/`--rocm`/`--cuda`
-flag always overrides `backend` in config.toml, not the other way around.
+Because all four are dlopen-based, `full` doesn't require the NPU/ROCm/CUDA/
+OpenVINO toolkits to be installed at build time — only at run time, and only
+for whichever single backend you actually select via
+`--npu`/`--rocm`/`--cuda`/`--openvino` or `backend` in config.toml. The
+**released binaries are built with `full`**: same binary works CPU-only out
+of the box, and picks up NPU/ROCm/CUDA/OpenVINO acceleration on a machine
+that has the matching ONNX Runtime available, without needing a different
+download. An explicit `--npu`/`--rocm`/`--cuda`/`--openvino` flag always
+overrides `backend` in config.toml, not the other way around.

 ## Setup

@ -117,6 +124,7 @@ breadmill status
 breadmill --npu
 breadmill --rocm
 breadmill --cuda
+breadmill --openvino
 ```

 ## Config
@ -137,7 +145,7 @@ snippet_len  = 200         # max characters in result snippet
 [model]
 name         = "nomic-embed-text-v1.5"
 dim          = 768
-backend      = "cpu"       # "cpu", "npu", "rocm", or "cuda"
+backend      = "cpu"       # "cpu", "npu", "rocm", "cuda", or "openvino"
 ```

 `roots` and `excludes` support `~/` expansion. The index respects `.gitignore` files found during the walk.
@ -154,10 +162,11 @@ Set `backend = "npu"` in config (or pass `--npu`) when running a build compiled

 ### GPU backend notes

-Both `rocm` and `cuda` need a system ONNX Runtime that was actually built with
-the matching execution provider — the crate's own downloaded binary is CPU-only.
-Point `ORT_DYLIB_PATH` at one, or install a distro package that provides
-`libonnxruntime.so` with the EP baked in and let the dynamic linker find it.
+`rocm`, `cuda`, and `openvino` all need a system ONNX Runtime that was
+actually built with the matching execution provider — the crate's own
+downloaded binary is CPU-only. Point `ORT_DYLIB_PATH` at one, or install a
+distro package that provides `libonnxruntime.so` with the EP baked in and
+let the dynamic linker find it.

 **ROCm (`--rocm` / `backend = "rocm"`)** targets ONNX Runtime's **MIGraphX**
 execution provider, not the classic `ROCMExecutionProvider`. Distro
@ -183,6 +192,18 @@ noticeable for interactive query embedding.
 CUDA/cuDNN install. Unverified on real NVIDIA hardware in this repo — only
 compile-checked, since development happened on an AMD-only machine.

+**OpenVINO (`--openvino` / `backend = "openvino"`)** targets
+`OpenVINOExecutionProvider` with `device_type = "GPU"`, covering both Intel
+iGPUs and Arc dGPUs through the same EP (OpenVINO abstracts Intel's whole
+hardware line — CPU/GPU/NPU — behind one provider and a device-type string).
+Needs an OpenVINO-enabled ONNX Runtime and the OpenVINO runtime itself
+installed. Its options go through a generic key/value FFI interface rather
+than a fixed C struct (unlike MIGraphX's `OrtMIGraphXProviderOptions`), so it
+should be less exposed to the kind of ABI-version-skew crash MIGraphX hit —
+but that's inference from reading the EP's design, not something verified
+against real Intel GPU hardware. Also unverified on real hardware in this
+repo — only compile-checked, for the same reason as CUDA.
+
 ## Runtime paths

 | Purpose | Path |
--- a/breadmill/Cargo.toml
+++ b/breadmill/Cargo.toml
@ -1,6 +1,6 @@
 [package]
 name = "breadmill"
-version = "0.2.1"
+version = "0.3.3"
 edition = "2021"
 license = "MIT"

@ -14,13 +14,17 @@ npu = ["ort/vitis", "ort/load-dynamic"]
 # libonnxruntime_providers_rocm.so, which most distros don't package.
 rocm = ["ort/migraphx", "ort/load-dynamic"]
 cuda = ["ort/cuda", "ort/load-dynamic"]
+# Intel iGPU/dGPU (Arc) + CPU via OpenVINO. One EP covers Intel's whole
+# hardware line by device_type string ("CPU"/"GPU"/"GPU.0"/"NPU"/"HETERO:...");
+# we always request "GPU" since CPU is already covered by the cpu backend.
+openvino = ["ort/openvino", "ort/load-dynamic"]
 # All backends in one binary. Safe to combine: every backend here uses
 # ort's load-dynamic (dlopen) mode, so none of this links against an actual
-# NPU/ROCm/CUDA toolkit at build time — which ONNX Runtime actually gets
-# loaded (and thus which EPs are really available) is decided at runtime by
-# ORT_DYLIB_PATH / the dynamic linker, per the --npu/--rocm/--cuda flag or
-# `backend` config value in use for that run.
-full = ["npu", "rocm", "cuda"]
+# NPU/ROCm/CUDA/OpenVINO toolkit at build time — which ONNX Runtime actually
+# gets loaded (and thus which EPs are really available) is decided at
+# runtime by ORT_DYLIB_PATH / the dynamic linker, per the --npu/--rocm/
+# --cuda/--openvino flag or `backend` config value in use for that run.
+full = ["npu", "rocm", "cuda", "openvino"]

 [[bin]]
 name = "breadmill"
@ -38,6 +42,7 @@ breadsearch-shared = { path = "../breadsearch-shared" }
 #   needed for the plain CPU path even in the npu build.
 ort = { version = "2.0.0-rc.12", default-features = false, features = ["std", "tracing", "download-binaries", "tls-native", "copy-dylibs", "api-23"] }
 tokenizers = "0"
+bread-onnx = { git = "https://git.breadway.dev/Breadway/bread-ecosystem", tag = "v0.7.2" }

 # Surfaces ort's own EP-registration tracing (e.g. a GPU EP silently failing to
 # register and falling back to CPU) as visible log output instead of nowhere.
--- a/breadmill/src/chunk.rs
+++ b/breadmill/src/chunk.rs
@ -81,10 +81,9 @@ fn split_by_chars(chunk: Chunk, max_chars: usize) -> Vec<Chunk> {
    let text = &chunk.text;
    let mut result = Vec::new();
    let mut seg_start = 0usize;
-    let mut count = 0usize;

-    for (byte_idx, _) in text.char_indices() {
-        if count > 0 && count % max_chars == 0 {
+    for (count, (byte_idx, _)) in text.char_indices().enumerate() {
+        if count > 0 && count.is_multiple_of(max_chars) {
            result.push(Chunk {
                text: text[seg_start..byte_idx].to_string(),
                start: chunk.start + seg_start,
@ -92,7 +91,6 @@ fn split_by_chars(chunk: Chunk, max_chars: usize) -> Vec<Chunk> {
            });
            seg_start = byte_idx;
        }
-        count += 1;
    }
    if seg_start < text.len() {
        result.push(Chunk {
--- a/breadmill/src/embed.rs
+++ b/breadmill/src/embed.rs
@ -1,10 +1,7 @@
 use std::path::{Path, PathBuf};

-use ort::{
-    session::{Session, builder::{GraphOptimizationLevel, SessionBuilder}},
-    value::Tensor,
-};
-use tokenizers::Tokenizer;
+use bread_onnx::embedding::EmbeddingSession;
+use bread_onnx::Provider;

 const DOCUMENT_PREFIX: &str = "search_document: ";
 const QUERY_PREFIX: &str = "search_query: ";
@ -27,26 +24,23 @@ pub enum Backend {
    Rocm,
    /// NVIDIA GPU via the CUDA ONNX Runtime execution provider.
    Cuda,
+    /// Intel iGPU/dGPU (Arc) via the OpenVINO ONNX Runtime execution
+    /// provider, requesting device_type "GPU". `cache_dir` stores OpenVINO's
+    /// compiled-model blobs between runs (its own `with_cache_dir`, not an
+    /// env var — unlike MIGraphX this doesn't need a workaround).
+    OpenVino { cache_dir: PathBuf },
 }

 pub struct OrtEmbedder {
-    session: Session,
-    tokenizer: Tokenizer,
-    dim: usize,
+    inner: EmbeddingSession,
 }

 impl OrtEmbedder {
    pub fn load(model_path: &Path, tokenizer_path: &Path, dim: usize, backend: Backend) -> Result<Self, String> {
-        let builder = Session::builder()
-            .map_err(|e| e.to_string())?
-            .with_optimization_level(GraphOptimizationLevel::All)
+        let provider = to_provider(backend)?;
+        let inner = EmbeddingSession::load(model_path, tokenizer_path, dim, MAX_SEQ_LEN, &[provider])
            .map_err(|e| e.to_string())?;
-
-        let mut builder = configure_eps(builder, &backend)?;
-        let session = builder.commit_from_file(model_path).map_err(|e| e.to_string())?;
-        let tokenizer = Tokenizer::from_file(tokenizer_path).map_err(|e| e.to_string())?;
-
-        Ok(Self { session, tokenizer, dim })
+        Ok(Self { inner })
    }

    pub fn embed_document(&mut self, text: &str) -> Result<Vec<f32>, String> {
@ -59,198 +53,71 @@ impl OrtEmbedder {

    fn embed_with_prefix(&mut self, text: &str, prefix: &str) -> Result<Vec<f32>, String> {
        let input = format!("{}{}", prefix, text);
-
-        let encoding = self
-            .tokenizer
-            .encode(input, true)
-            .map_err(|e| e.to_string())?;
-
-        let mut ids: Vec<i64> = encoding.get_ids().iter().map(|&x| x as i64).collect();
-        let mut mask: Vec<i64> = encoding
-            .get_attention_mask()
-            .iter()
-            .map(|&x| x as i64)
-            .collect();
-        let mut type_ids: Vec<i64> = encoding
-            .get_type_ids()
-            .iter()
-            .map(|&x| x as i64)
-            .collect();
-
-        if ids.len() > MAX_SEQ_LEN {
-            eprintln!(
-                "breadmill: truncating {} tokens to {} (chunk too large)",
-                ids.len(),
-                MAX_SEQ_LEN
-            );
-            ids.truncate(MAX_SEQ_LEN);
-            mask.truncate(MAX_SEQ_LEN);
-            type_ids.truncate(MAX_SEQ_LEN);
-        }
-
-        let seq_len = ids.len() as i64;
-
-        let id_tensor =
-            Tensor::<i64>::from_array((vec![1i64, seq_len], ids.clone())).map_err(|e| e.to_string())?;
-        let mask_tensor =
-            Tensor::<i64>::from_array((vec![1i64, seq_len], mask.clone())).map_err(|e| e.to_string())?;
-        let type_tensor =
-            Tensor::<i64>::from_array((vec![1i64, seq_len], type_ids)).map_err(|e| e.to_string())?;
-
-        let outputs = self
-            .session
-            .run(ort::inputs! {
-                "input_ids" => id_tensor,
-                "attention_mask" => mask_tensor,
-                "token_type_ids" => type_tensor,
-            })
-            .map_err(|e| e.to_string())?;
-
-        // last_hidden_state: shape [1, seq_len, dim]
-        let (shape, data) = outputs["last_hidden_state"]
-            .try_extract_tensor::<f32>()
-            .map_err(|e| e.to_string())?;
-
-        let actual_seq = shape[1] as usize;
-        let actual_dim = shape[2] as usize;
-
-        // Mean-pool over non-padding positions. Some execution providers (e.g.
-        // MIGraphX) pad the output sequence dimension for kernel efficiency, so
-        // actual_seq can exceed mask.len() — only positions covered by our own
-        // attention mask are meaningful, so cap the loop at whichever is shorter.
-        let mut result = vec![0.0f32; actual_dim];
-        let mut count = 0usize;
-
-        for t in 0..actual_seq.min(mask.len()) {
-            if mask[t] > 0 {
-                for d in 0..actual_dim {
-                    result[d] += data[t * actual_dim + d];
-                }
-                count += 1;
-            }
-        }
-
-        if count > 0 {
-            for x in &mut result {
-                *x /= count as f32;
-            }
-        }
-
-        l2_normalize(&mut result);
-
-        // Clamp/pad to configured dim
-        result.truncate(self.dim);
-        while result.len() < self.dim {
-            result.push(0.0);
-        }
-
-        Ok(result)
+        self.inner.embed(&input).map_err(|e| e.to_string())
    }
 }

-fn l2_normalize(v: &mut Vec<f32>) {
-    let norm: f32 = v.iter().map(|x| x * x).sum::<f32>().sqrt();
-    if norm > 1e-10 {
-        for x in v.iter_mut() {
-            *x /= norm;
-        }
-    }
-}
+// ---- Backend -> bread_onnx::Provider ----------------------------------------
+//
+// `bread_onnx::session::build_session` (via `EmbeddingSession::load`) is the
+// shared session-builder + EP-fallback + loud-logging code every EP branch
+// below used to hand-roll separately (`configure_eps`/`npu_session`/
+// `rocm_session`/`cuda_session`/`openvino_session`). What's genuinely
+// specific to this crate — its cargo feature gates (npu/rocm/cuda/openvino),
+// and NPU's `vaip_config.json` discovery — stays here.

-// ---- Execution provider selection -------------------------------------------
-
-fn configure_eps(builder: SessionBuilder, backend: &Backend) -> Result<SessionBuilder, String> {
+fn to_provider(backend: Backend) -> Result<Provider, String> {
    match backend {
-        Backend::Cpu => Ok(builder),
-        Backend::Npu { cache_dir } => npu_session(builder, cache_dir),
-        Backend::Rocm => rocm_session(builder),
-        Backend::Cuda => cuda_session(builder),
+        Backend::Cpu => Ok(Provider::Cpu),
+        Backend::Npu { cache_dir } => npu_provider(cache_dir),
+        Backend::Rocm => rocm_provider(),
+        Backend::Cuda => cuda_provider(),
+        Backend::OpenVino { cache_dir } => Ok(Provider::OpenVino { device_type: "GPU".to_string(), cache_dir }),
    }
 }

 #[cfg(feature = "npu")]
-fn npu_session(builder: SessionBuilder, cache_dir: &Path) -> Result<SessionBuilder, String> {
-    let vitis_ep = build_vitis_ep(cache_dir)?;
-    eprintln!("breadmill: using NPU (VitisAI) execution provider");
+fn npu_provider(cache_dir: PathBuf) -> Result<Provider, String> {
+    let vaip_config = find_vaip_config()?;
    if std::env::var("ORT_DYLIB_PATH").is_err() {
        eprintln!(
            "breadmill: hint — set ORT_DYLIB_PATH to the Ryzen AI SDK ORT, e.g.:\n  \
             ORT_DYLIB_PATH=~/.local/share/ryzen-ai-1.7.1/lib/libonnxruntime.so"
        );
    }
-    builder
-        .with_execution_providers([vitis_ep, ort::ep::CPU::default().build()])
-        .map_err(|e| e.to_string())
+    Ok(Provider::Vitis {
+        config_file: vaip_config,
+        cache_dir: cache_dir.join("npu"),
+        cache_key: "nomic-embed-text-v1.5".to_string(),
+    })
 }

 #[cfg(not(feature = "npu"))]
-fn npu_session(builder: SessionBuilder, _cache_dir: &Path) -> Result<SessionBuilder, String> {
+fn npu_provider(_cache_dir: PathBuf) -> Result<Provider, String> {
    eprintln!("breadmill: NPU backend requested but not compiled in (rebuild with --features npu); using CPU");
-    Ok(builder)
+    Ok(Provider::Cpu)
 }

-// ---- VitisAI EP (NPU) -------------------------------------------------------
-
-#[cfg(feature = "npu")]
-fn build_vitis_ep(cache_dir: &Path) -> Result<ort::ep::ExecutionProviderDispatch, String> {
-    let vaip_config = find_vaip_config()?;
-    let npu_cache = cache_dir.join("npu");
-    std::fs::create_dir_all(&npu_cache).map_err(|e| e.to_string())?;
-    eprintln!("breadmill: vaip_config: {}", vaip_config.display());
-    eprintln!("breadmill: NPU model cache: {}", npu_cache.display());
-    Ok(ort::ep::Vitis::default()
-        .with_config_file(vaip_config.to_string_lossy())
-        .with_cache_dir(npu_cache.to_string_lossy())
-        .with_cache_key("nomic-embed-text-v1.5")
-        .build())
-}
-
-// ---- MIGraphX EP (AMD iGPU, ROCm-backed) -------------------------------------
-
 #[cfg(feature = "rocm")]
-fn rocm_session(builder: SessionBuilder) -> Result<SessionBuilder, String> {
-    eprintln!("breadmill: using MIGraphX execution provider (device 0)");
-    eprintln!(
-        "breadmill: note — check the log line above/below for \"Successfully registered \
-         `MIGraphXExecutionProvider`\"; if it's missing, the ONNX Runtime in use wasn't built \
-         with MIGraphX support and inference silently fell back to CPU"
-    );
-    builder
-        .with_execution_providers([
-            ort::ep::MIGraphX::default().with_device_id(0).build(),
-            ort::ep::CPU::default().build(),
-        ])
-        .map_err(|e| e.to_string())
+fn rocm_provider() -> Result<Provider, String> {
+    Ok(Provider::MiGraphX { device_id: 0 })
 }

 #[cfg(not(feature = "rocm"))]
-fn rocm_session(builder: SessionBuilder) -> Result<SessionBuilder, String> {
+fn rocm_provider() -> Result<Provider, String> {
    eprintln!("breadmill: ROCm backend requested but not compiled in (rebuild with --features rocm); using CPU");
-    Ok(builder)
+    Ok(Provider::Cpu)
 }

-// ---- CUDA EP (NVIDIA GPU) ----------------------------------------------------
-
 #[cfg(feature = "cuda")]
-fn cuda_session(builder: SessionBuilder) -> Result<SessionBuilder, String> {
-    eprintln!("breadmill: using CUDA execution provider (device 0)");
-    eprintln!(
-        "breadmill: note — check the log line above/below for \"Successfully registered \
-         `CUDAExecutionProvider`\"; if it's missing, the ONNX Runtime in use wasn't built \
-         with CUDA support and inference silently fell back to CPU"
-    );
-    builder
-        .with_execution_providers([
-            ort::ep::CUDA::default().with_device_id(0).build(),
-            ort::ep::CPU::default().build(),
-        ])
-        .map_err(|e| e.to_string())
+fn cuda_provider() -> Result<Provider, String> {
+    Ok(Provider::Cuda { device_id: 0 })
 }

 #[cfg(not(feature = "cuda"))]
-fn cuda_session(builder: SessionBuilder) -> Result<SessionBuilder, String> {
+fn cuda_provider() -> Result<Provider, String> {
    eprintln!("breadmill: CUDA backend requested but not compiled in (rebuild with --features cuda); using CPU");
-    Ok(builder)
+    Ok(Provider::Cpu)
 }

 /// Locate the VitisAI EP config file required by the AMD Ryzen AI SDK.
--- a/breadmill/src/indexer.rs
+++ b/breadmill/src/indexer.rs
@ -10,7 +10,7 @@ use ignore::WalkBuilder;
 use notify::{RecommendedWatcher, RecursiveMode, Watcher, EventKind};
 use sha2::{Digest, Sha256};

-use crate::{embed::OrtEmbedder, extract, chunk, power, store::Store};
+use crate::{embed::OrtEmbedder, extract, chunk, power, store::Store, sync_ext::MutexExt};

 pub struct SharedState {
    pub store: Mutex<Store>,
@ -64,7 +64,7 @@ impl Indexer {
    pub fn full_reindex(&self) {
        eprintln!("breadmill: full reindex triggered");
        {
-            let mut store = self.state.store.lock().unwrap();
+            let store = self.state.store.lock_recover();
            // Clear all state
            let _ = store.conn.execute_batch("DELETE FROM chunks; DELETE FROM files;");
            let _ = store.index.reserve(4096);
@ -87,7 +87,7 @@ impl Indexer {

        // Snapshot existing indexed files
        let known: HashMap<String, (i64, String)> = {
-            let store = self.state.store.lock().unwrap();
+            let store = self.state.store.lock_recover();
            store.all_files()
                .unwrap_or_default()
                .into_iter()
@ -155,7 +155,7 @@ impl Indexer {
            .collect();

        if !to_delete.is_empty() {
-            let mut store = self.state.store.lock().unwrap();
+            let mut store = self.state.store.lock_recover();
            for path in to_delete {
                eprintln!("breadmill: removing deleted file: {}", path);
                let _ = store.delete_file(&path);
@ -163,7 +163,7 @@ impl Indexer {
        }

        let count = {
-            let store = self.state.store.lock().unwrap();
+            let store = self.state.store.lock_recover();
            let n = store.chunk_count();
            let _ = store.save_index(&self.state_dir);
            n
@ -238,7 +238,7 @@ impl Indexer {
                    self.handle_fs_event(&path);
                }
                let count = {
-                    let store = self.state.store.lock().unwrap();
+                    let store = self.state.store.lock_recover();
                    let n = store.chunk_count();
                    let _ = store.save_index(&self.state_dir);
                    n
@ -261,7 +261,7 @@ impl Indexer {
        if !path.is_file() {
            let path_str = path.to_string_lossy().into_owned();
            // File deleted — remove from index
-            let mut store = self.state.store.lock().unwrap();
+            let mut store = self.state.store.lock_recover();
            let _ = store.delete_file(&path_str);
            return;
        }
@ -309,7 +309,7 @@ impl Indexer {

        // Check if hash changed (catches content changes without mtime change)
        {
-            let store = self.state.store.lock().unwrap();
+            let store = self.state.store.lock_recover();
            if let Ok(files) = store.all_files() {
                if files.iter().any(|f| f.path == path_str && f.hash == hash) {
                    return;
@ -322,20 +322,22 @@ impl Indexer {
        // for natural-language files.
        let chunks = chunk::chunk_text(&text, 400, 80, 2_000);
        eprintln!("breadmill: embedding {} ({} chars, {} chunks)", path_str, text.len(), chunks.len());
-        let mut embedder_guard = self.state.embedder.lock().unwrap();

        if !self.state.model_ready.load(Ordering::Relaxed) {
            eprintln!("breadmill: model not ready, skipping embed for {}", path_str);
            return;
        }

-        let embedder = match embedder_guard.as_mut() {
-            Some(e) => e,
-            None => return,
-        };
+        // Confirm the embedder is actually present before committing to
+        // clearing this file's old chunks below — same check as before,
+        // just without holding the embedder lock past this one glance (see
+        // the per-chunk locking in the loop for why).
+        if self.state.embedder.lock_recover().is_none() {
+            return;
+        }

        {
-            let mut store = self.state.store.lock().unwrap();
+            let mut store = self.state.store.lock_recover();
            let _ = store.delete_file(path_str); // remove old chunks/vectors first
        }

@ -344,9 +346,18 @@ impl Indexer {

        for (i, chunk) in chunks.iter().enumerate() {
            eprintln!("breadmill: embed chunk {}/{} ({} chars) for {}", i + 1, chunks.len(), chunk.text.len(), path_str);
-            match embedder.embed_document(&chunk.text) {
+            // Lock the embedder only around this single chunk's embed call —
+            // this used to be held for the whole file's chunk loop, so one
+            // large file needing a fresh MIGraphX JIT compile (60-120s for a
+            // new sequence length) could block every query (serve.rs locks
+            // this same mutex) for the entire file, not just one chunk.
+            let embed_result = match self.state.embedder.lock_recover().as_mut() {
+                Some(embedder) => embedder.embed_document(&chunk.text),
+                None => break, // model was unloaded mid-scan; stop here
+            };
+            match embed_result {
                Ok(embedding) => {
-                    let mut store = self.state.store.lock().unwrap();
+                    let mut store = self.state.store.lock_recover();
                    // Ensure file row exists before inserting chunks (FK constraint)
                    let _ = store.upsert_file(path_str, mtime, &hash);
                    let _ = store.insert_chunk(
@ -367,7 +378,7 @@ impl Indexer {
            eprintln!("breadmill: no chunks embedded for {}", path_str);
            // Record the file so the mtime+hash check skips it on the next startup
            // rather than re-entering the same embed-fail loop.
-            let store = self.state.store.lock().unwrap();
+            let store = self.state.store.lock_recover();
            let _ = store.upsert_file(path_str, mtime, &hash);
        } else {
            // Increment live so `status` reflects progress before the full scan ends.
@ -405,9 +416,9 @@ fn sha256_str(bytes: &[u8]) -> String {
 }

 pub fn expand_home(path: &str) -> PathBuf {
-    if path.starts_with("~/") {
+    if let Some(rest) = path.strip_prefix("~/") {
        let home = std::env::var("HOME").unwrap_or_else(|_| "/tmp".into());
-        PathBuf::from(home).join(&path[2..])
+        PathBuf::from(home).join(rest)
    } else {
        PathBuf::from(path)
    }
--- a/breadmill/src/main.rs
+++ b/breadmill/src/main.rs
@ -1,5 +1,4 @@
 use std::{
-    io::Read,
    path::{Path, PathBuf},
    sync::{Arc, atomic::Ordering},
 };
@ -13,10 +12,12 @@ mod indexer;
 mod power;
 mod serve;
 mod store;
+mod sync_ext;

 use embed::{Backend, OrtEmbedder};
 use indexer::{Indexer, SharedState};
 use store::Store;
+use sync_ext::MutexExt;

 const MODEL_URL: &str =
    "https://huggingface.co/nomic-ai/nomic-embed-text-v1.5/resolve/main/onnx/model.onnx";
@ -39,12 +40,14 @@ fn main() {
    let use_npu = raw_args.iter().any(|a| a == "--npu");
    let use_rocm = raw_args.iter().any(|a| a == "--rocm");
    let use_cuda = raw_args.iter().any(|a| a == "--cuda");
+    let use_openvino = raw_args.iter().any(|a| a == "--openvino");

    // Build a view of argv without backend flags for command matching.
+    let backend_flags = ["--npu", "--rocm", "--cuda", "--openvino"];
    let args: Vec<&str> = raw_args
        .iter()
        .skip(1)
-        .filter(|a| a.as_str() != "--npu" && a.as_str() != "--rocm" && a.as_str() != "--cuda")
+        .filter(|a| !backend_flags.contains(&a.as_str()))
        .map(|s| s.as_str())
        .collect();

@ -59,7 +62,7 @@ fn main() {
            }
        }
        Some("--reindex") | Some("reindex") => {
-            if let Err(e) = run_daemon(true, use_npu, use_rocm, use_cuda) {
+            if let Err(e) = run_daemon(true, use_npu, use_rocm, use_cuda, use_openvino) {
                eprintln!("breadmill: {}", e);
                std::process::exit(1);
            }
@ -76,7 +79,7 @@ fn main() {
            cli_status();
        }
        None | Some("serve") | Some("--serve") => {
-            if let Err(e) = run_daemon(false, use_npu, use_rocm, use_cuda) {
+            if let Err(e) = run_daemon(false, use_npu, use_rocm, use_cuda, use_openvino) {
                eprintln!("breadmill: {}", e);
                std::process::exit(1);
            }
@ -84,7 +87,7 @@ fn main() {
        Some(cmd) => {
            eprintln!("breadmill: unknown command: {}", cmd);
            eprintln!(
-                "usage: breadmill [serve|reindex|fetch-model|query <text>|status] [--npu|--rocm|--cuda] [--version]"
+                "usage: breadmill [serve|reindex|fetch-model|query <text>|status] [--npu|--rocm|--cuda|--openvino] [--version]"
            );
            std::process::exit(1);
        }
@ -93,7 +96,13 @@ fn main() {

 // ---- Daemon -----------------------------------------------------------------

-fn run_daemon(force_reindex: bool, use_npu: bool, use_rocm: bool, use_cuda: bool) -> Result<(), String> {
+fn run_daemon(
+    force_reindex: bool,
+    use_npu: bool,
+    use_rocm: bool,
+    use_cuda: bool,
+    use_openvino: bool,
+) -> Result<(), String> {
    let config = breadsearch_shared::Config::load();
    let state_dir = breadsearch_shared::state_dir();
    let cache_dir = breadsearch_shared::cache_dir();
@ -115,6 +124,8 @@ fn run_daemon(force_reindex: bool, use_npu: bool, use_rocm: bool, use_cuda: bool
        "rocm"
    } else if use_cuda {
        "cuda"
+    } else if use_openvino {
+        "openvino"
    } else {
        config.model.backend.as_str()
    };
@ -132,27 +143,41 @@ fn run_daemon(force_reindex: bool, use_npu: bool, use_rocm: bool, use_cuda: bool
            eprintln!("breadmill: CUDA backend selected");
            Backend::Cuda
        }
+        "openvino" => {
+            eprintln!("breadmill: OpenVINO backend selected");
+            Backend::OpenVino { cache_dir: cache_dir.clone() }
+        }
        _ => Backend::Cpu,
    };

    let store = Store::open(&state_dir, dim)?;
    let state = Arc::new(SharedState::new(store));

-    // Load embedder if model files present
+    // Load the embedder on a background thread — an OpenVINO/CUDA/etc EP
+    // compile can take minutes or hang outright, and doing this inline used
+    // to block the socket bind below until it finished. That turned "model
+    // still loading" into an indistinguishable "connection refused" for
+    // every client, including the GUI, for as long as the load took. The
+    // socket now opens immediately; serve.rs already answers "model not
+    // ready" (via `model_ready`) for any request that arrives before the
+    // background load finishes.
    let model_dir = model_dir(&cache_dir);
    let model_path = model_dir.join("model.onnx");
    let tokenizer_path = model_dir.join("tokenizer.json");

    if model_path.exists() && tokenizer_path.exists() {
-        eprintln!("breadmill: loading model...");
-        match OrtEmbedder::load(&model_path, &tokenizer_path, dim, backend) {
-            Ok(embedder) => {
-                *state.embedder.lock().unwrap() = Some(embedder);
-                state.model_ready.store(true, Ordering::Relaxed);
-                eprintln!("breadmill: model loaded");
+        let state_clone = Arc::clone(&state);
+        std::thread::spawn(move || {
+            eprintln!("breadmill: loading model...");
+            match OrtEmbedder::load(&model_path, &tokenizer_path, dim, backend) {
+                Ok(embedder) => {
+                    *state_clone.embedder.lock_recover() = Some(embedder);
+                    state_clone.model_ready.store(true, Ordering::Relaxed);
+                    eprintln!("breadmill: model loaded");
+                }
+                Err(e) => eprintln!("breadmill: model load failed: {} — run --fetch-model", e),
            }
-            Err(e) => eprintln!("breadmill: model load failed: {} — run --fetch-model", e),
-        }
+        });
    } else {
        eprintln!(
            "breadmill: model files not found in {} — run: breadmill --fetch-model",
@ -200,29 +225,10 @@ fn download_if_missing(url: &str, dest: &Path) -> Result<(), String> {
        eprintln!("  already present: {}", dest.display());
        return Ok(());
    }
-
-    eprintln!("  downloading {} ...", url);
-    let agent = ureq::AgentBuilder::new()
-        .timeout(std::time::Duration::from_secs(300))
-        .build();
-
-    let response = agent.get(url).call().map_err(|e| e.to_string())?;
-    let mut bytes = Vec::new();
-    response
-        .into_reader()
-        .read_to_end(&mut bytes)
-        .map_err(|e| e.to_string())?;
-
-    if bytes.is_empty() {
-        return Err(format!("empty download from {}", url));
-    }
-
-    // Write atomically via temp file
-    let tmp = dest.with_extension("tmp");
-    std::fs::write(&tmp, &bytes).map_err(|e| e.to_string())?;
-    std::fs::rename(&tmp, dest).map_err(|e| e.to_string())?;
-
-    eprintln!("  saved {} ({:.1} MB)", dest.display(), bytes.len() as f64 / 1_048_576.0);
+    // Shared with breadarrd's own (previously reqwest/async, now also this
+    // same sync/ureq implementation) model downloader — see
+    // bread_onnx::download's doc comment.
+    bread_onnx::download::ensure_file(url, dest, None).map_err(|e| e.to_string())?;
    Ok(())
 }

--- a/breadmill/src/serve.rs
+++ b/breadmill/src/serve.rs
@ -8,6 +8,7 @@ use std::{
 use breadsearch_shared::{Request, Response, StatusInfo};

 use crate::indexer::SharedState;
+use crate::sync_ext::MutexExt;

 pub fn run(socket_path: &Path, state: Arc<SharedState>, snippet_len: usize, search_limit: usize) {
    let _ = std::fs::remove_file(socket_path);
@ -86,7 +87,7 @@ fn dispatch(
            }

            let embedding = {
-                let mut embedder = state.embedder.lock().unwrap();
+                let mut embedder = state.embedder.lock_recover();
                match embedder.as_mut() {
                    Some(e) => match e.embed_query(&query) {
                        Ok(v) => v,
@ -101,7 +102,7 @@ fn dispatch(
            };

            let limit = limit.min(search_limit).max(1);
-            let store = state.store.lock().unwrap();
+            let store = state.store.lock_recover();

            match store.search(&embedding, limit, snippet_len) {
                Ok(hits) => Response::Hits { hits },
--- a/breadmill/src/store.rs
+++ b/breadmill/src/store.rs
@ -6,7 +6,6 @@ use usearch::{Index, IndexOptions, MetricKind, ScalarKind, new_index};
 pub struct Store {
    pub conn: Connection,
    pub index: Index,
-    pub dim: usize,
 }

 // usearch::Index wraps a raw C++ pointer; access is serialized by the Mutex<Store>.
@ -57,14 +56,37 @@ impl Store {
        let index = new_index(&options).map_err(|e| e.to_string())?;

        if idx_path.exists() {
-            index
-                .load(idx_path.to_str().unwrap())
-                .map_err(|e| e.to_string())?;
+            // NOTE: this only catches corruption that usearch's loader
+            // itself detects and reports as an `Err` (e.g. a recognizable
+            // but wrong/incompatible header). Verified experimentally: a
+            // file that doesn't even look like a usearch index at all (pure
+            // garbage bytes) crashes the *process* with a SIGSEGV inside the
+            // native loader rather than returning an `Err` — no amount of
+            // Rust-side `Result`/`catch_unwind` handling can intercept that,
+            // it's a native-code robustness gap in the usearch library
+            // itself. This recovery path is still worth having (it's the
+            // difference between "won't start" and "rebuilds and starts"
+            // for the errors it *does* catch), but it is not a complete
+            // guarantee against every possible corrupt file. The atomic
+            // save below is the real fix for the common case this was
+            // written for (a crash mid-save) — it now can't produce a
+            // half-written `vectors.usearch` in the first place.
+            if let Err(e) = index.load(idx_path.to_str().unwrap()) {
+                eprintln!(
+                    "breadmill: WARNING: {} failed to load ({}) — treating it as corrupt, \
+                     discarding it, and rebuilding the index from scratch on the next scan",
+                    idx_path.display(),
+                    e
+                );
+                conn.execute_batch("DELETE FROM chunks; DELETE FROM files;")
+                    .map_err(|e| e.to_string())?;
+                index.reserve(4096).map_err(|e| e.to_string())?;
+            }
        } else {
            index.reserve(4096).map_err(|e| e.to_string())?;
        }

-        Ok(Self { conn, index, dim })
+        Ok(Self { conn, index })
    }

    // ---- file state ---------------------------------------------------------
@ -218,11 +240,18 @@ impl Store {

    // ---- persistence --------------------------------------------------------

+    /// Saves to a `.tmp` sibling and renames it into place — a same-
+    /// filesystem rename is atomic, so a crash mid-save leaves only an
+    /// orphaned `.tmp` file rather than a truncated/corrupt
+    /// `vectors.usearch` that would otherwise fail to load on next start
+    /// (see the recovery path in `open`).
    pub fn save_index(&self, state_dir: &Path) -> Result<(), String> {
        let idx_path = state_dir.join("vectors.usearch");
+        let tmp_path = state_dir.join("vectors.usearch.tmp");
        self.index
-            .save(idx_path.to_str().unwrap())
-            .map_err(|e| e.to_string())
+            .save(tmp_path.to_str().unwrap())
+            .map_err(|e| e.to_string())?;
+        std::fs::rename(&tmp_path, &idx_path).map_err(|e| e.to_string())
    }
 }

@ -233,3 +262,45 @@ fn truncate_to_chars(s: &str, max_chars: usize) -> String {
    let truncated: String = s.chars().take(max_chars).collect();
    format!("{}…", truncated.trim_end())
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    fn test_dir(name: &str) -> std::path::PathBuf {
+        let dir = std::env::temp_dir().join(format!("breadmill-store-test-{name}-{}", std::process::id()));
+        let _ = std::fs::remove_dir_all(&dir);
+        std::fs::create_dir_all(&dir).unwrap();
+        dir
+    }
+
+    #[test]
+    fn save_index_leaves_no_tmp_file_behind_and_is_loadable() {
+        let dir = test_dir("save-atomic");
+        let store = Store::open(&dir, 4).unwrap();
+        store.save_index(&dir).unwrap();
+
+        assert!(dir.join("vectors.usearch").exists());
+        assert!(
+            !dir.join("vectors.usearch.tmp").exists(),
+            "the .tmp staging file should be renamed away, not left behind"
+        );
+
+        // A fresh `open` can load what was just saved without error.
+        Store::open(&dir, 4).unwrap();
+
+        std::fs::remove_dir_all(&dir).unwrap();
+    }
+
+    // No automated test for the corrupt-index recovery branch in `open`:
+    // the natural way to construct a "corrupt" fixture (writing arbitrary
+    // garbage to vectors.usearch) was tried and crashes the *test process*
+    // with a SIGSEGV inside usearch's native loader before our `Result`
+    // handling ever gets a chance to run — see the comment on that branch
+    // in `open`. A real usearch file with a deliberately-broken-but-still-
+    // parseable header could plausibly hit the `Err` path exercised there
+    // instead, but reverse-engineering that format precisely enough to
+    // build a safe fixture wasn't worth the risk of another flaky/crashing
+    // test. The atomic-save test above covers the actual mechanism that
+    // prevents this scenario from arising in the first place.
+}
--- a/breadmill/src/sync_ext.rs
+++ b/breadmill/src/sync_ext.rs
@ -0,0 +1,41 @@
+//! Poison-tolerant `Mutex` locking.
+//!
+//! A panic on any thread while holding `SharedState::store` or
+//! `SharedState::embedder` (e.g. an untrapped ONNX Runtime panic — only PDF
+//! extraction is `catch_unwind`-guarded, see `extract.rs`) poisons the
+//! `Mutex`. Every subsequent plain `.lock().unwrap()` — in the indexer *and*
+//! in every query handler in `serve.rs` — would then immediately panic too,
+//! silently bricking indexing and search until the daemon is restarted by
+//! hand.
+//!
+//! `Mutex` poisoning exists to flag "the data guarded by this lock might be
+//! in an inconsistent state," but every lock scope in this crate is a short,
+//! single-step SQLite call or usearch operation — nothing here spans a
+//! multi-step invariant across a single `lock()` call — so recovering the
+//! guard and logging loudly is a reasonable trade here: continuing to serve
+//! (and re-attempting the operation that panicked, on the next file/query)
+//! beats a daemon that silently stops answering everything after one panic.
+
+use std::sync::{Mutex, MutexGuard};
+
+pub trait MutexExt<T> {
+    /// Like `.lock().unwrap()`, but recovers from a poisoned mutex instead
+    /// of panicking again — logs once per recovery so it's visible in the
+    /// daemon's own output, not just silently swallowed.
+    fn lock_recover(&self) -> MutexGuard<'_, T>;
+}
+
+impl<T> MutexExt<T> for Mutex<T> {
+    fn lock_recover(&self) -> MutexGuard<'_, T> {
+        match self.lock() {
+            Ok(guard) => guard,
+            Err(poisoned) => {
+                eprintln!(
+                    "breadmill: WARNING: a mutex was poisoned by a panic on another thread; \
+                     recovering it and continuing instead of cascading the panic"
+                );
+                poisoned.into_inner()
+            }
+        }
+    }
+}
--- a/breadsearch-shared/Cargo.toml
+++ b/breadsearch-shared/Cargo.toml
@ -1,6 +1,6 @@
 [package]
 name = "breadsearch-shared"
-version = "0.2.0"
+version = "0.3.3"
 edition = "2021"
 license = "MIT"

--- a/breadsearch-shared/src/lib.rs
+++ b/breadsearch-shared/src/lib.rs
@ -43,7 +43,7 @@ pub fn socket_path() -> PathBuf {

 // ---- Config -----------------------------------------------------------------

-#[derive(Debug, Clone, Serialize, Deserialize)]
+#[derive(Debug, Clone, Default, Serialize, Deserialize)]
 pub struct Config {
    #[serde(default)]
    pub index: IndexConfig,
@ -129,7 +129,8 @@ pub struct ModelConfig {
    pub name: String,
    #[serde(default = "default_dim")]
    pub dim: usize,
-    /// Compute backend: "cpu", "npu" (VitisAI/XDNA), "rocm" (MIGraphX/AMD GPU), or "cuda" (NVIDIA GPU).
+    /// Compute backend: "cpu", "npu" (VitisAI/XDNA), "rocm" (MIGraphX/AMD GPU),
+    /// "cuda" (NVIDIA GPU), or "openvino" (Intel iGPU/dGPU).
    #[serde(default = "default_backend")]
    pub backend: String,
 }
@ -156,16 +157,6 @@ impl Default for ModelConfig {
    }
 }

-impl Default for Config {
-    fn default() -> Self {
-        Self {
-            index: IndexConfig::default(),
-            search: SearchConfig::default(),
-            model: ModelConfig::default(),
-            power: PowerConfig::default(),
-        }
-    }
-}

 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct PowerConfig {
@ -254,8 +245,7 @@ pub struct StatusInfo {
 pub fn send_request(req: &Request) -> std::io::Result<Response> {
    let mut stream = UnixStream::connect(socket_path())?;

-    let mut line = serde_json::to_string(req)
-        .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;
+    let mut line = serde_json::to_string(req).map_err(std::io::Error::other)?;
    line.push('\n');
    stream.write_all(line.as_bytes())?;
    stream.flush()?;
--- a/breadsearch/Cargo.toml
+++ b/breadsearch/Cargo.toml
@ -1,6 +1,6 @@
 [package]
 name = "breadsearch"
-version = "0.2.0"
+version = "0.3.3"
 edition = "2021"
 license = "MIT"

@ -10,7 +10,13 @@ path = "src/main.rs"

 [dependencies]
 breadsearch-shared = { path = "../breadsearch-shared" }
-bread-theme = { git = "https://github.com/Breadway/bread-ecosystem", tag = "v0.2.8", features = ["gtk"] }
+bread-theme = { git = "https://git.breadway.dev/Breadway/bread-ecosystem", tag = "v0.7.4", features = ["gtk"] }
+# Bread event fabric client — emit bread.search.* (fail-silent if breadd is down).
+bread-utils = { git = "https://git.breadway.dev/Breadway/bread-ecosystem", tag = "v0.7.2", features = ["bread-client", "gtk"] }
+# Capture primitives for `--screenshot` mode — see src/screenshot.rs.
+bread-screenshots = { git = "https://git.breadway.dev/Breadway/bread-ecosystem", tag = "v0.7.2" }
 gtk4 = { version = "0.11", features = ["v4_12"] }
 gtk4-layer-shell = "0.8"
 serde_json = "1"
+clap = { version = "4", features = ["derive"] }
+anyhow = "1"
--- a/breadsearch/src/bread_events.rs
+++ b/breadsearch/src/bread_events.rs
@ -0,0 +1,26 @@
+//! `bread.search.*` event integration — optional, non-blocking. See
+//! `EVENTS.md` at the repo root for the full contract. breadsearch works
+//! identically with or without breadd running; every call here is
+//! fire-and-forget (`BreadClient::emit` never blocks or errors this
+//! process) so a missing or restarting breadd never affects the overlay.
+
+use bread_utils::bread_client::BreadClient;
+
+/// This app's id in bread's sibling-app namespace registry
+/// (`bread_shared::apps::KNOWN_APPS`) — events publish as `bread.search.*`.
+pub const APP_ID: &str = "search";
+
+fn client() -> BreadClient {
+    BreadClient::connect(APP_ID)
+}
+
+pub fn emit_opened() {
+    client().emit("bread.search.opened", serde_json::json!({}));
+}
+
+pub fn emit_opened_result(path: &str) {
+    client().emit(
+        "bread.search.opened_result",
+        serde_json::json!({ "path": path }),
+    );
+}
--- a/breadsearch/src/listen.rs
+++ b/breadsearch/src/listen.rs
@ -0,0 +1,84 @@
+//! Long-running command subscription for `bread.command.search.*`.
+//!
+//! `breadsearch` is still a one-shot toggle overlay by default.
+//! `breadsearch listen` is the optional persistent process that can honor
+//! bus commands. See `EVENTS.md`.
+
+use bread_utils::bread_client::{BreadClient, BreadEvent};
+
+use crate::bread_events::APP_ID;
+
+/// Subscribe to `bread.command.search.**` and block until the process is killed.
+///
+/// breadd being absent is not an error: [`BreadClient::subscribe`] reconnects
+/// with backoff, and `on_event` simply isn't called until the daemon is up.
+pub fn run() {
+    let client = BreadClient::connect(APP_ID);
+    if client.health().is_none() {
+        eprintln!(
+            "breadsearch: breadd unreachable; command subscription will connect when it comes back"
+        );
+    }
+
+    let _commands = client.subscribe("bread.command.search.**", |event| {
+        handle_command(&event);
+    });
+
+    eprintln!("breadsearch: listening for bread.command.search.**");
+    loop {
+        std::thread::park();
+    }
+}
+
+/// Reacts to `bread.command.search.*` verbs. Only `open` is honored today —
+/// other verbs are ignored, not stubbed as no-ops that pretend to succeed.
+fn handle_command(event: &BreadEvent) {
+    let Some(verb) = command_verb(&event.event) else {
+        return;
+    };
+    match verb {
+        "open" => handle_open(),
+        other => {
+            eprintln!("breadsearch: ignoring unrecognized bread.command.search.{other}");
+        }
+    }
+}
+
+fn handle_open() {
+    // Same as running `breadsearch` from a keybind: the PID-file toggle
+    // shows the overlay (or dismisses it if it is already up).
+    let result = spawn_self();
+    let client = BreadClient::connect(APP_ID);
+    match result {
+        Ok(_) => client.emit("bread.search.open.done", serde_json::json!({})),
+        Err(e) => {
+            eprintln!("breadsearch: bread.command.search.open failed: {e}");
+            client.emit(
+                "bread.search.open.failed",
+                serde_json::json!({ "error": e.to_string() }),
+            );
+        }
+    }
+}
+
+fn spawn_self() -> std::io::Result<std::process::Child> {
+    let exe = std::env::current_exe().unwrap_or_else(|_| std::path::PathBuf::from("breadsearch"));
+    std::process::Command::new(exe).spawn()
+}
+
+fn command_verb(event_name: &str) -> Option<&str> {
+    event_name.strip_prefix("bread.command.search.")
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn command_verb_strips_search_prefix() {
+        assert_eq!(command_verb("bread.command.search.open"), Some("open"));
+        assert_eq!(command_verb("bread.command.search.query"), Some("query"));
+        assert_eq!(command_verb("bread.command.box.open"), None);
+        assert_eq!(command_verb("bread.search.opened"), None);
+    }
+}
--- a/breadsearch/src/main.rs
+++ b/breadsearch/src/main.rs
@ -1,22 +1,16 @@
 use bread_theme::{hex_to_rgba, ink_on, load_palette, Palette};
 use breadsearch_shared::{Hit, Request, Response};
-use std::{
-    cell::RefCell,
-    env, fs,
-    path::PathBuf,
-    process::Command,
-    rc::Rc,
-    sync::mpsc,
-};
+use std::{cell::RefCell, process::Command, rc::Rc, sync::mpsc};

 use gtk4::{
-    glib,
-    pango::EllipsizeMode,
-    prelude::*,
-    Application, ApplicationWindow, Box as GBox, CssProvider, EventControllerKey, Image, Label,
-    ListBox, Orientation, PolicyType, ScrolledWindow, SearchEntry, SelectionMode,
+    glib, pango::EllipsizeMode, prelude::*, Application, Box as GBox, CssProvider,
+    EventControllerKey, Image, Label, ListBox, Orientation, PolicyType, ScrolledWindow,
+    SearchEntry, SelectionMode,
 };
-use gtk4_layer_shell::{Edge, KeyboardMode, Layer, LayerShell};
+
+mod bread_events;
+mod listen;
+mod screenshot;

 // ---- Theming ----------------------------------------------------------------

@ -39,47 +33,14 @@ fn build_css(p: &Palette) -> String {
         .hit-snippet {{ opacity: 0.75; font-size: 11px; font-style: italic; }}\
         .hit-score {{ opacity: 0.5; font-size: 11px; }}\
         image {{ margin-right: 8px; }}",
-        bg_panel   = bg_panel,
-        surface    = p.color0,
-        accent     = p.color4,
-        on_bg      = ink_on(&p.background),
+        bg_panel = bg_panel,
+        surface = p.color0,
+        accent = p.color4,
+        on_bg = ink_on(&p.background),
        on_surface = ink_on(&p.color0),
    )
 }

-// ---- PID file toggle --------------------------------------------------------
-
-fn pid_file() -> PathBuf {
-    env::var("XDG_RUNTIME_DIR")
-        .map(PathBuf::from)
-        .unwrap_or_else(|_| PathBuf::from("/tmp"))
-        .join("breadsearch.pid")
-}
-
-fn is_breadsearch_pid(pid: u32) -> bool {
-    fs::read_to_string(format!("/proc/{}/comm", pid))
-        .map(|s| s.trim() == "breadsearch")
-        .unwrap_or(false)
-}
-
-fn toggle_or_continue() -> bool {
-    let pf = pid_file();
-    if let Ok(content) = fs::read_to_string(&pf) {
-        if let Ok(pid) = content.trim().parse::<u32>() {
-            if is_breadsearch_pid(pid) {
-                let _ = Command::new("kill").arg(pid.to_string()).status();
-                return false;
-            }
-        }
-    }
-    let _ = fs::write(&pf, std::process::id().to_string());
-    true
-}
-
-fn cleanup_pid() {
-    let _ = fs::remove_file(pid_file());
-}
-
 // ---- Row builder ------------------------------------------------------------

 fn make_hit_row(hit: &Hit) -> gtk4::ListBoxRow {
@ -191,6 +152,7 @@ fn open_file(path: &str) {
        .stdout(std::process::Stdio::null())
        .stderr(std::process::Stdio::null())
        .spawn();
+    bread_events::emit_opened_result(path);
 }

 fn open_folder(path: &str) {
@ -204,14 +166,22 @@ fn open_folder(path: &str) {
        .stdout(std::process::Stdio::null())
        .stderr(std::process::Stdio::null())
        .spawn();
+    bread_events::emit_opened_result(path);
 }

 // ---- UI ---------------------------------------------------------------------

-fn run_ui() {
-    let app = Application::builder()
-        .application_id("com.breadway.breadsearch")
-        .build();
+fn run_ui(screenshot_req: Option<screenshot::ScreenshotRequest>) {
+    let mut builder = Application::builder().application_id("com.breadway.breadsearch");
+    if screenshot_req.is_some() {
+        // GApplication is single-instance by default; this machine typically
+        // already has a real breadsearch instance, so without this a
+        // screenshot run would just message the *existing* instance instead
+        // of starting a fresh one that ever sees `screenshot_req`.
+        builder = builder.flags(gtk4::gio::ApplicationFlags::NON_UNIQUE);
+    }
+    let app = builder.build();
+    let is_screenshot_run = screenshot_req.is_some();

    let debounce_id: Rc<RefCell<Option<glib::SourceId>>> = Rc::new(RefCell::new(None));

@ -225,20 +195,13 @@ fn run_ui() {
            bread_theme::gtk::apply_user_css(&user_css_path, &user_cell);
        }

-        let window = ApplicationWindow::builder().application(app).build();
-        window.init_layer_shell();
-        window.set_namespace(Some("breadsearch"));
-        window.set_layer(Layer::Overlay);
-        window.set_keyboard_mode(KeyboardMode::Exclusive);
-        for edge in [Edge::Top, Edge::Bottom, Edge::Left, Edge::Right] {
-            window.set_anchor(edge, true);
-        }
-        window.set_exclusive_zone(0);
+        // Full-screen transparent overlay; panel widget is positioned inside it.
+        let window = bread_utils::gtk_popup::new_overlay_window(app, "breadsearch");
+        bread_theme::gtk::bind_window_auto(&window);

        let close_all: Rc<dyn Fn()> = Rc::new({
            let w = window.clone();
            move || {
-                cleanup_pid();
                w.close();
            }
        });
@ -294,16 +257,26 @@ fn run_ui() {
                let (tx, rx) = mpsc::sync_channel::<std::io::Result<Response>>(1);

                std::thread::spawn(move || {
-                    let req = Request::Query { query: q, limit: 10 };
+                    let req = Request::Query {
+                        query: q,
+                        limit: 10,
+                    };
                    let _ = tx.send(breadsearch_shared::send_request(&req));
                });

-                // Poll via idle_add_local until the thread delivers its result.
-                // Unix socket round-trips are sub-millisecond so this fires once.
+                // Wait for the thread's result on a bounded timer rather than
+                // an `idle_add_local` — an idle source has no wait condition
+                // of its own, so GLib re-invokes it on every single main-loop
+                // iteration, i.e. a busy-spin pinning a full CPU core for as
+                // long as the daemon takes to answer (normally sub-ms, but
+                // the daemon's socket has no read timeout of its own, so a
+                // wedged/slow daemon previously meant an indefinite spin).
+                // 15ms is imperceptible added latency for a search box and
+                // caps this at a couple dozen checks per second instead.
                let rx = Rc::new(rx);
                let list_t = list_clone.clone();

-                glib::idle_add_local(move || {
+                glib::timeout_add_local(std::time::Duration::from_millis(15), move || {
                    match rx.try_recv() {
                        Ok(result) => {
                            populate_list(&list_t, result);
@ -347,36 +320,11 @@ fn run_ui() {
                    glib::Propagation::Stop
                }
                Key::Down => {
-                    let cur = list_k.selected_row().map(|r| r.index()).unwrap_or(-1);
-                    let mut i = cur + 1;
-                    loop {
-                        match list_k.row_at_index(i) {
-                            Some(r) if r.is_selectable() => {
-                                list_k.select_row(Some(&r));
-                                break;
-                            }
-                            Some(_) => i += 1,
-                            None => break,
-                        }
-                    }
+                    bread_utils::gtk_popup::select_next_visible(&list_k);
                    glib::Propagation::Stop
                }
                Key::Up => {
-                    let cur = list_k.selected_row().map(|r| r.index()).unwrap_or(0);
-                    let mut i = cur - 1;
-                    loop {
-                        if i < 0 {
-                            break;
-                        }
-                        match list_k.row_at_index(i) {
-                            Some(r) if r.is_selectable() => {
-                                list_k.select_row(Some(&r));
-                                break;
-                            }
-                            Some(_) => i -= 1,
-                            None => break,
-                        }
-                    }
+                    bread_utils::gtk_popup::select_prev_visible(&list_k);
                    glib::Propagation::Stop
                }
                _ => glib::Propagation::Proceed,
@ -394,36 +342,65 @@ fn run_ui() {
        });

        // Click outside launcher panel → close
-        let close_outside = Rc::clone(&close_all);
-        let vbox_ref = vbox.clone();
-        let win_ref = window.clone();
-        let outside_click = gtk4::GestureClick::new();
-        outside_click.connect_pressed(move |_, _, x, y| {
-            if let Some(b) = vbox_ref.compute_bounds(&win_ref) {
-                if x < b.x() as f64
-                    || x > (b.x() + b.width()) as f64
-                    || y < b.y() as f64
-                    || y > (b.y() + b.height()) as f64
-                {
-                    close_outside();
-                }
-            }
-        });
-        window.add_controller(outside_click);
+        {
+            let close_outside = Rc::clone(&close_all);
+            bread_utils::gtk_popup::close_on_outside_click(&window, &vbox, move || close_outside());
+        }

-        window.connect_destroy(|_| cleanup_pid());
+        if let Some(req) = screenshot_req.clone() {
+            screenshot::dispatch(&window, req);
+        }
+
+        window.connect_map(|_| bread_events::emit_opened());
        window.present();
        search.grab_focus();
    });

-    app.run();
+    if is_screenshot_run {
+        // GLib's own option parser otherwise rejects --screenshot/--output
+        // before clap ever sees them (`Cli::parse()` already ran in `main`,
+        // over the real argv).
+        app.run_with_args(&[] as &[&str]);
+    } else {
+        app.run();
+    }
 }

 // ---- Main -------------------------------------------------------------------

 fn main() {
-    if !toggle_or_continue() {
+    if std::env::args().nth(1).as_deref() == Some("listen") {
+        listen::run();
        return;
    }
-    run_ui();
+
+    use clap::Parser;
+    let cli = screenshot::Cli::parse();
+    let screenshot_req = cli.screenshot_request();
+
+    // `toggle_or_kill` kills whatever's holding the single-instance lock —
+    // a real, already-running breadsearch included. A screenshot run must
+    // never touch it: it's a separate, disposable instance by design (same
+    // reasoning as breadbar's `allow_multiple_instances`), not a toggle of
+    // the operator's real search panel.
+    //
+    // Kept alive for the rest of `main` — dropping it releases the
+    // single-instance lock and removes the pid file, which happens
+    // naturally once `run_ui` returns (after the window closes).
+    let _singleton_guard = if screenshot_req.is_some() {
+        None
+    } else {
+        match bread_utils::singleton::toggle_or_kill("breadsearch") {
+            Ok(bread_utils::singleton::Toggle::Started(guard)) => Some(guard),
+            Ok(bread_utils::singleton::Toggle::KilledExisting) => return,
+            Err(e) => {
+                eprintln!(
+                    "breadsearch: single-instance lock unavailable ({e}); continuing without it"
+                );
+                None
+            }
+        }
+    };
+
+    run_ui(screenshot_req);
 }
--- a/breadsearch/src/screenshot.rs
+++ b/breadsearch/src/screenshot.rs
@ -0,0 +1,93 @@
+//! `--screenshot` CLI mode: render breadsearch's search panel, capture it
+//! via `bread-screenshots`, then exit — driven by `bread-ecosystem`'s
+//! `bread-capture` orchestrator, or run standalone for one-off captures.
+//!
+//! Same reasoning as breadbox: one view ("search"), full known-size canvas
+//! capture since the panel isn't its own layer surface. Captures whatever
+//! the panel shows at settle time — normally the "Type to search…" empty
+//! state, since there's no query to type in an automated run.
+
+use bread_utils::screenshot_cli::{validate_pair, DEFAULT_HEIGHT, DEFAULT_WIDTH, SETTLE_DELAY};
+use clap::Parser;
+use gtk4::prelude::*;
+use std::path::PathBuf;
+
+#[derive(Parser)]
+#[command(name = "breadsearch")]
+pub struct Cli {
+    /// Render the named view, capture it, then exit instead of running
+    /// normally. Known views: "search".
+    #[arg(long)]
+    pub screenshot: Option<String>,
+
+    /// PNG path to write the capture to. Required together with --screenshot.
+    #[arg(long)]
+    pub output: Option<PathBuf>,
+
+    /// Capture canvas width — matches the isolated compositor's output width
+    /// (`bread-capture --isolate-width`).
+    #[arg(long, default_value_t = DEFAULT_WIDTH)]
+    pub width: u32,
+
+    /// Capture canvas height — see `width`.
+    #[arg(long, default_value_t = DEFAULT_HEIGHT)]
+    pub height: u32,
+}
+
+#[derive(Clone)]
+pub struct ScreenshotRequest {
+    pub view: String,
+    pub output: PathBuf,
+    pub width: u32,
+    pub height: u32,
+}
+
+impl Cli {
+    /// `None` for a normal run. Exits the process with an error if the
+    /// `--screenshot` / `--output` pair is incomplete, before any GTK setup
+    /// happens.
+    pub fn screenshot_request(&self) -> Option<ScreenshotRequest> {
+        if let Err(e) = validate_pair(self.screenshot.as_deref(), self.output.as_deref()) {
+            eprintln!("breadsearch: {e}");
+            std::process::exit(1);
+        }
+        Some(ScreenshotRequest {
+            view: self.screenshot.clone()?,
+            output: self.output.clone()?,
+            width: self.width,
+            height: self.height,
+        })
+    }
+}
+
+/// Wire up the given view's screenshot sequence against an already-built,
+/// not-yet-presented window. Every path here ends by exiting the process —
+/// it never returns control to the normal search UI.
+pub fn dispatch(window: &gtk4::ApplicationWindow, req: ScreenshotRequest) {
+    match req.view.as_str() {
+        "search" => {
+            let output = req.output;
+            let (width, height) = (req.width as i32, req.height as i32);
+            window.connect_map(move |_| {
+                let output = output.clone();
+                gtk4::glib::timeout_add_local_once(SETTLE_DELAY, move || {
+                    finish(bread_screenshots::capture_region(0, 0, width, height, &output));
+                });
+            });
+        }
+        other => {
+            eprintln!("breadsearch: unknown screenshot view '{other}' (known: search)");
+            std::process::exit(1);
+        }
+    }
+}
+
+fn finish(result: anyhow::Result<()>) {
+    match result {
+        Ok(()) => std::process::exit(0),
+        Err(e) => {
+            eprintln!("breadsearch: screenshot capture failed: {e}");
+            std::process::exit(1);
+        }
+    }
+}
--- a/ci/bread-ecosystem.rev
+++ b/ci/bread-ecosystem.rev
@ -0,0 +1 @@
+147cfbbf96ae4b171027defa1130d2caddb934b1
--- a/ci/build.sh
+++ b/ci/build.sh
@ -0,0 +1,21 @@
+#!/usr/bin/env bash
+# Delegates to bread-ecosystem's shared CI build image/script, pinned to
+# the commit in ci/bread-ecosystem.rev — not `main`. bread-ecosystem's CI
+# files now affect every product's release pipeline, so bumping the pin
+# is a deliberate act instead of silent drift (see the bread-theme test
+# that broke here for exactly that reason, before it was pinned by rev).
+#
+# Usage: ci/build.sh cargo build --release --locked
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+REV="$(cat "${ROOT}/ci/bread-ecosystem.rev")"
+
+CACHE_DIR="/tmp/bread-ecosystem-ci-${REV}"
+if [ ! -d "$CACHE_DIR" ]; then
+    rm -rf /tmp/bread-ecosystem-ci-*
+    git clone https://git.breadway.dev/Breadway/bread-ecosystem.git "$CACHE_DIR"
+    git -C "$CACHE_DIR" checkout --quiet "$REV"
+fi
+
+bash "${CACHE_DIR}/ci/build.sh" breadsearch "$ROOT" "$@"
--- a/packaging/breadmill.service
+++ b/packaging/breadmill.service
@ -1,14 +1,16 @@
 [Unit]
 Description=Breadmill semantic search indexer
-Documentation=https://github.com/breadway/breadsearch
+Documentation=https://git.breadway.dev/Breadway/breadsearch
 After=default.target

 [Service]
 Type=simple
-# Uncomment if built with --features rocm: persists MIGraphX's compiled-kernel
-# cache across restarts (each new sequence length otherwise costs a ~60-120s
-# recompile). See README.md#gpu-backend-notes.
-#Environment=ORT_MIGRAPHX_MODEL_CACHE_PATH=%h/.cache/breadsearch/migraphx-cache
+# Required (not just a perf tweak) when backend = "rocm": without a valid
+# cache dir, ONNX Runtime's MIGraphX EP reads an uninitialized cache path and
+# crashes on the first query instead of just recompiling every restart. Inert
+# for cpu/npu/cuda/openvino backends (openvino's cache dir is set in code,
+# not via env var). See README.md#gpu-backend-notes.
+Environment=ORT_MIGRAPHX_MODEL_CACHE_PATH=%h/.cache/breadsearch/migraphx-cache
 ExecStart=%h/.cargo/bin/breadmill
 Restart=on-failure
 RestartSec=5
Author	SHA1	Message	Date
Breadway	993dc4a525	Fix cargo clippy warnings across the workspace All checks were successful dev release / build (push) Successful in 4m4s Details - Derive Default for Config instead of manually mirroring each sub-config's Default impl - Use std::io::Error::other instead of Error::new(ErrorKind::Other, ..) - Drop the unused mut on the store lock in full_reindex - Remove Store::dim, a field that was set once and never read - Use char_indices().enumerate() instead of a hand-rolled loop counter in split_by_chars, and is_multiple_of() for the chunk boundary check - Use strip_prefix instead of manual slicing in expand_home (cherry picked from commit b740af38e17d04bd83db93bbb0585ed5744a436e)	2026-08-23 14:57:59 +08:00
Breadway	4bfcc694bd	Adopt gtk_popup overlay helpers and singleton::toggle_or_kill Some checks failed check / check (push) Failing after 1m16s Details dev release / build (push) Successful in 3m17s Details Replace the TOCTOU pid-file toggle with bread_utils::singleton and the hand-rolled layer-shell overlay/Up-Down/click-outside with gtk_popup, matching breadclip. Screenshot runs still skip the lock.	2026-08-23 14:39:11 +08:00
Breadway	1ce5f72819	Bump version to v0.3.3 All checks were successful beta (rc) release / build (push) Has been skipped Details dev release / build (push) Successful in 6m7s Details release / build (push) Successful in 5m34s Details	2026-08-16 14:09:28 +08:00
Breadway	60d1d81777	Bind the window to the current monitor's bread-theme palette All checks were successful dev release / build (push) Successful in 11m24s Details Pin bread-theme to v0.7.4.	2026-08-16 13:23:15 +08:00
Breadway	e1a2bef109	CI: refuse unsigned bakery index on stable tag releases Some checks failed check / check (push) Failing after 24s Details dev release / build (push) Failing after 4m5s Details beta (rc) release / build (push) Has been skipped Details release / build (push) Successful in 1m21s Details	2026-08-16 00:50:22 +08:00
Breadway	7eefe9087a	Adopt bread_utils::screenshot_cli for --screenshot flags Some checks failed check / check (push) Failing after 12s Details dev release / build (push) Successful in 2m22s Details Replace the local settle delay, canvas defaults, and pair-validation error path with bread-utils v0.7.2. Clap parsing stays in-tree.	2026-08-16 00:26:06 +08:00
Breadway	cdee9da609	Honor bread.command.search.open via breadsearch listen All checks were successful dev release / build (push) Successful in 1m45s Details	2026-08-15 23:11:28 +08:00
Breadway	e20c64fa1a	Bump version to v0.3.1 All checks were successful beta (rc) release / build (push) Has been skipped Details dev release / build (push) Successful in 2m29s Details release / build (push) Successful in 2m29s Details	2026-08-15 23:05:47 +08:00
Breadway	93c34e7387	Pin bread-ecosystem crates to v0.7.2 All checks were successful dev release / build (push) Successful in 2m20s Details Repoint bread-theme, bread-utils, bread-screenshots, and bread-onnx at tag v0.7.2 on git.breadway.dev. Drop the 69ce2d67 rev pin now that screenshots is on a tagged release.	2026-08-15 22:53:46 +08:00
Breadway	4f9f4de911	CI: build inside bread-ecosystem's shared Arch container All checks were successful dev release / build (push) Successful in 5m25s Details Ports breadsearch onto the shared, pinned Arch build image already proven in breadpad, so this repo's release CI stops installing its toolchain/GTK4 stack directly on the bare self-hosted runner. - ci/build.sh + ci/bread-ecosystem.rev: thin wrapper cloning bread-ecosystem pinned to a commit sha and delegating to its own ci/build.sh. - dev-release.yml/rc-release.yml/release.yml: route the existing build/test cargo invocations through ci/build.sh, unchanged otherwise. - check.yml: fast clippy/test signal on feature/ and fix/ pushes. No ci/deps.txt: breadmill's npu/rocm/cuda/openvino features (pulled in by --features full) all use ort's load-dynamic (dlopen) mode, so none of them link against a real ONNX Runtime EP at build time — confirmed against breadmill/Cargo.toml and by breadpad already building ort with its own rocm feature against the same base image with no extra packages. The GUI's librsvg system dep is a runtime icon-loader dependency (bakery.toml's system_deps), not something linked at build/link time, so the base image's gtk4/gtk4-layer-shell already cover what compiling breadsearch needs.	2026-08-15 22:33:15 +08:00
Breadway	0e3c8a1668	breadsearch: emit bread.search.opened and bread.search.opened_result All checks were successful dev release / build (push) Successful in 2m22s Details Wire the GTK overlay to the bread event fabric via bread-utils BreadClient (app id search). Fail-silent when breadd is down. No command verbs — the overlay has no existing command surface.	2026-08-15 22:14:47 +08:00
Breadway	95a97d63d3	Remove CLAUDE.md (renamed to AGENTS.md)	2026-08-15 22:04:00 +08:00
Breadway	6983fd83da	Align bread-screenshots rev with siblings; track AGENTS.md	2026-08-15 22:03:30 +08:00
Breadway	572aebd282	Pin bread-ecosystem crates to v0.7.1 and document single-trunk + overlay split bread-theme and bread-onnx now come from git.breadway.dev tag v0.7.1 (gtk feature on theme). bread-screenshots is not on that tag, so it is pinned to rev f86e299 instead of branch=main. CLAUDE.md tracks the single-trunk release model; README states breadsearch is document/meaning search and is not breadbox's app-launcher overlay.	2026-08-15 21:40:06 +08:00
Breadway	065f106584	Repoint bread-ecosystem git deps from dev to main branch All checks were successful dev release / build (push) Successful in 7m43s Details	2026-07-31 14:30:01 +08:00
Breadway	4cb254801f	CONTRIBUTING.md: document single-trunk + RC-tag release model All checks were successful dev release / build (push) Successful in 6m17s Details	2026-07-31 11:08:41 +08:00
Breadway	8c11bfd3b8	CI: single-trunk model — dev triggers on main, beta becomes RC-tag-triggered Replaces the dev/beta branch split with one trunk (main): dev-track builds still publish on every push, but the beta track now publishes from a vX.Y.Z-rc.N prerelease tag instead of a separately-maintained beta branch. Removes the branch nobody reliably kept in sync.	2026-07-31 11:05:29 +08:00
Breadway	aefe07009b	Merge feature/screenshot-mode: add --screenshot CLI mode for automated capture All checks were successful dev release / build (push) Successful in 3m34s Details	2026-07-29 16:39:35 +08:00
Breadway	5098780f5b	breadsearch: add --screenshot CLI mode for automated capture Same pattern as breadbox/breadclip: one view ("search"), full known-size canvas capture since the search panel isn't its own layer surface. Also fixes the same PID-file-toggle footgun — a screenshot run now skips toggle_or_continue() entirely instead of killing whatever real instance holds breadsearch.pid.	2026-07-29 11:40:23 +08:00
Breadway	08717e51c0	docs: add CONTRIBUTING.md Some checks failed dev release / build (push) Failing after 2m43s Details Documents the dev/beta/main branch and release-track workflow shared across the bread ecosystem. See bread-ecosystem's docs/release-channels.md for the full policy this implements.	2026-07-22 19:41:42 +08:00
Breadway	1f9707f30a	ci: make beta a branch-triggered freeze track, not a one-off tag All checks were successful dev release / build (push) Successful in 2m35s Details beta release / build (push) Successful in 2m31s Details Beta is now a real stabilization branch: publishes on every push to `beta` (mirroring dev's model, auto-versioned X.Y.Z-beta.<ts>+<sha>, base version from the latest published tag) instead of a manual beta-v* tag. Fixes made during the freeze land via fix/<issue> branches merged into `beta` directly. The gen-index.sh clone for beta pulls bread-ecosystem's default branch (main) rather than pinning to dev, since beta is the more stable track and main now carries the TRACK-aware script.	2026-07-22 18:37:43 +08:00
Breadway	19087b0168	ci: base dev version on the latest published tag, not Cargo.toml All checks were successful dev release / build (push) Successful in 2m34s Details Cargo.toml can drift stale relative to the actual last release (observed on breadbox/breadpad/breadcrumbs/breadpaper), which made the auto-bumped dev version sort as OLDER than what's already installed — bakery's semver check correctly refused those "updates". Deriving the base version from git ls-remote --tags instead is self-healing regardless of Cargo.toml drift, with a Cargo.toml fallback only for a repo with no tags yet.	2026-07-22 13:52:44 +08:00
Breadway	dabd110f92	random commit message, read it yourself All checks were successful dev release / build (push) Successful in 2m42s Details	2026-07-22 11:57:04 +08:00
Breadway	1e3872124b	ci: use a unique temp dir for the bread-ecosystem clone in dev/beta CI All checks were successful dev release / build (push) Successful in 2m30s Details The fixed /tmp/bread-ecosystem-ci path races when multiple repos' dev/beta workflows run close together on the same self-hosted runner — one job's rm -rf/clone can stomp another's in-progress checkout, causing the regenerate-index step to fail intermittently. Switch to mktemp -d.	2026-07-22 10:24:44 +08:00
Breadway	bf711f8047	ci: retrigger dev-track build now that BAKERY_MINISIGN_SEC_KEY_PATH is set Some checks failed dev release / build (push) Failing after 2m39s Details	2026-07-22 10:10:16 +08:00
Breadway	13f5e509e4	ci: add dev/beta build track workflows Some checks failed dev release / build (push) Failing after 2m43s Details Adds dev-release.yml (publishes on every push to dev) and beta-release.yml (publishes on a beta-v* tag), mirroring the pattern landing in bread-ecosystem/bread. Also creates the dev branch for this repo, which didn't exist before — see bread-ecosystem/docs/release-channels.md for the three-track policy.	2026-07-22 09:56:49 +08:00
Breadway	cfca6161b0	ci: remove GitHub push-mirror workflow	2026-07-21 19:18:06 +08:00
Breadway	99724853a0	Switch to tag-pinned bread-ecosystem deps; bump version to v0.3.0 All checks were successful Mirror to GitHub / mirror (push) Successful in 2s Details release / build (push) Successful in 2m11s Details	2026-07-19 03:53:04 +08:00
Breadway	0990969722	Migrate embedding pipeline, EP session building, and model download to bread-onnx embed.rs's OrtEmbedder now delegates its tokenize -> tensor build -> mean-pool -> L2-normalize pipeline to bread_onnx::embedding::EmbeddingSession (near-byte-identical to breadarrd's own OrtEmbedder — same duplication, now shared, path dependency for now, see the TODO in breadmill/Cargo.toml), and its per-EP session builders (npu_session/rocm_session/cuda_session/ openvino_session) collapse into a single to_provider() mapping onto bread_onnx::Provider, which this crate's own breadmill/src/session.rs counterpart now handles generically. This crate's Backend enum, cargo feature gates (npu/rocm/cuda/openvino/full - unchanged, still control which onnxruntime EPs actually link/load), and NPU vaip_config.json discovery all stay local since they're genuinely breadsearch-specific. main.rs's Backend construction and CLI flag handling are untouched. This is also the reference implementation the MIGraphX-not-ROCm default in bread-onnx's provider module was promoted from (see this machine's own breadsearch-gpu-backends operator notes) — breadpad's ONNX migration, which had the actual silent-fallback bug, follows in a later commit. download_if_missing is replaced with bread_onnx::download::ensure_file (same sync/ureq approach, now shared with breadarrd's downloader). Builds clean with default features AND --features full (npu+rocm+cuda+ openvino all compiling together, matching how this crate already combined them). All existing tests pass across the whole workspace.	2026-07-17 09:41:58 +08:00
Breadway	d01a3841d9	breadmill/breadsearch: narrow embedder lock scope, recover from poisoned mutexes, atomic index save + corrupt rebuild, fix GUI busy-poll, bump theme pin - indexer.rs: embedder mutex is now locked only around each chunk's embed_document() call instead of the whole file's chunk loop, so a slow MIGraphX JIT compile on one file no longer blocks every query for minutes - sync_ext.rs (new)/indexer.rs/main.rs/serve.rs: added MutexExt::lock_recover(), replacing every .lock().unwrap() so a panic on one thread (poisoning the mutex) logs a warning and recovers instead of cascading into every future lock().unwrap() call - store.rs: save_index now writes to a .tmp sibling and renames atomically into place; Store::open recovers from an index that fails to load by wiping it and the SQLite files/chunks tables so the next scan rebuilds from scratch, instead of refusing to start at all. Note: this recovery only catches corruption usearch's loader reports as Err — verified experimentally that sufficiently garbled input segfaults the process instead, which no Rust-side handling can catch; the atomic save is what actually prevents the realistic mid-crash corruption case from arising - breadsearch/src/main.rs: GUI query-result wait switched from glib::idle_add_local (re-invoked every main-loop tick, pegging a core for the whole wait) to a 15ms glib::timeout_add_local poll - breadsearch/Cargo.toml: bread-theme pin bumped v0.2.8 -> v0.2.10 to match the rest of the family	2026-07-17 08:16:25 +08:00
Breadway	3d83bd747e	breadmill 0.2.4: bind socket before loading the embedding model All checks were successful Mirror to GitHub / mirror (push) Successful in 41s Details release / build (push) Successful in 2m8s Details The model load ran synchronously on the main thread before serve::run() bound the daemon's socket. A slow or stuck EP compile (OpenVINO in particular) meant the socket didn't exist for as long as that took, so every client -- including the breadsearch GUI -- saw a bare connection refused with no way to tell "still loading" from "actually broken." The load now happens on a background thread; the socket binds immediately, and serve.rs's existing model_ready check answers "model not ready" for any request that arrives before the load finishes.	2026-07-05 09:13:54 +08:00
Breadway	c6ed6a41d8	Add OpenVINO backend for Intel iGPU/dGPU (Arc) All checks were successful Mirror to GitHub / mirror (push) Successful in 1s Details release / build (push) Successful in 2m12s Details Mirrors the rocm/cuda pattern: ort::ep::OpenVINO with device_type "GPU" (covers both Intel integrated graphics and Arc discrete GPUs through the same EP), load-dynamic/dlopen so no OpenVINO toolkit is needed at build time, and its own --openvino flag / backend = "openvino" config value. Folded into the `full` feature alongside npu/rocm/cuda. OpenVINO's provider options go through a generic key/value FFI interface rather than a fixed C struct (unlike MIGraphX's OrtMIGraphXProviderOptions), so it should be less exposed to the ABI-version-skew crash MIGraphX hit -- but that's inference from the EP's design, not verified against real hardware. Like CUDA, this is compile-checked only: no Intel GPU in this dev environment to runtime-verify against. Version bump: breadmill 0.2.2 -> 0.2.3.	2026-07-03 22:58:27 +08:00
Breadway	e5922e9c90	Make ORT_MIGRAPHX_MODEL_CACHE_PATH unconditional in packaged systemd unit All checks were successful Mirror to GitHub / mirror (push) Successful in 1s Details release / build (push) Successful in 1m49s Details Without a valid MIGraphX cache dir, ONNX Runtime's MIGraphX EP reads an uninitialized cache path and crashes on the first query rather than just recompiling on every restart -- confirmed by hand while verifying the rocm backend. Previously commented out as an opt-in perf tweak; it's actually required for the rocm backend to work at all, so bakery-installed systems need it set by default. No-op for cpu/npu/cuda backends. Version bump: breadmill 0.2.1 -> 0.2.2.	2026-07-03 22:38:41 +08:00