# π¦ @goodandready/dsh-vision-bridge
---
## β‘ Overview & The Problem
When interacting with text-only chat models (any provider/model that accepts text only) in **DeepSeek Harness**, users cannot natively attach and send images:
1. In **DSH 0.1.2-alpha.2+**, the core session controller performs a strict server-side modality check (`ctx.llm.resolveModelInfo`). If the active conversation model lacks `image` in `inputModalities`, the prompt is immediately rejected with a `session/attachment-invalid` error ("Model does not support image input").
2. Standard text-only adapters throw errors when encountering raw multimodal image blocks in their message payload.
### How `dsh-vision-bridge` Solves This
`dsh-vision-bridge` acts as an intelligent intermediary inside the Cordis runtime:
* **Server-Side Modality Bridge (v0.5.3+)**: Decorates `ctx.llm.resolveModelInfo` and `ctx.llm.listModels` so the session controller accepts image attachments on all models when bridging is active.
* **Automatic Image Rewrite (`agent/pre-step` & `llm/stream`)**: Automatically intercepts image blocks, routes them to a configured vision model (a catalog provider/model, or a local OpenAI-compatible endpoint), receives a descriptive synthesis, and rewrites the image block into text context `[The user attached an image. Description: ...]` before handing it to the text-only chat model.
* **Native Passthrough**: Automatically detects models that natively support vision and allows images to pass directly without unnecessary rewriting.
* **Rich Visual Tool Suite**: Exposes ~40 specialized tools for on-demand OCR (incl. local Tesseract), visual question answering, bounding-box grounding, document/table/formula extraction, QR/barcode reading, UI-flow reconstruction, multi-model consensus and more. `vision_consensus` is opt-in via the `consensusEnabled` setting.
---
## ποΈ Architecture
```mermaid
graph LR
User["User attaches Image in Web UI"] --> Gateway["DSH Session Controller"]
Gateway --> BridgeCheck{"Modality Bridge (v0.5.3)"}
BridgeCheck -->|"Augments inputModalities"| SessionAllowed["Prompt Accepted"]
SessionAllowed --> Hook["agent/pre-step Hook"]
Hook --> CheckNative{"Does chat model support vision natively?"}
CheckNative -->|"Yes (Native Passthrough)"| NativeLLM["Send Raw Image to Chat LLM"]
CheckNative -->|"No (Text-Only)"| VisionRouter["Vision Bridge Channels"]
VisionRouter --> VisionModel["Dedicated Vision Model\n(DSH / OpenAI / Ollama / Webhook)"]
VisionModel --> Description["Generated Text Description + OCR"]
Description --> Rewrite["Substitute Image with Text Marker"]
Rewrite --> ChatModel["Send Enriched Text to Chat LLM"]
ChatModel --> Answer["Assistant Response in Chat"]
```
---
## β¨ Key Features
### 1. Processing Modes
* **`hybrid` (default)**: Automatically describes attached images in chat turns while keeping all ~40 explicit vision tools available for follow-up reasoning.
* **`llm`**: Pure auto-rewrite mode β images are transparently converted to text context; tools remain callable.
* **`tools`**: Auto-rewrite disabled β the chat model is expected to explicitly invoke `describe_image` or OCR tools when required.
### 2. Multi-Channel Endpoint Routing & Fallback
Chain multiple vision backends with automatic failover, parallel racing, and circuit breaker:
* `dsh-catalog`: Auto-detect or select any vision-capable model already registered in DSH.
* `openai-compatible`: Standard OpenAI-compatible vision endpoints (vLLM, SGLang, OpenRouter, etc.).
* `ollama`: Auto-discovery and local inference through any vision-capable model the local Ollama instance exposes.
* `webhook` / `custom`: External HTTP or JSON-RPC vision endpoints.
### 3. High-Performance LRU Description Cache
Caches vision responses by `hash(bytes + prompt + model + mode)` to eliminate redundant vision API calls and save token quota on repeated questions about the same image.
### 4. Comprehensive Visual Tool Inventory (~40 Tools)
| Tool Category | Tools | Description |
|---|---|---|
| **Core** | `describe_image`, `read_image`, `inspect_image` | General image analysis by attachment ID, file path, or URL. |
| **Geometry & Detection** | `vision_ground`, `vision_crop`, `vision_detect`, `vision_compare`, `vision_present` | Bounding box coordinates (0β1000 scale), object inventory, multi-image comparison. |
| **OCR & Text** | `vision_ocr`, `vision_ocr_local`, `vision_long_ocr`, `vision_trace`, `vision_colors`, `vision_extract_foreground` | Transcription, local Tesseract OCR (offline), long screenshot stitching, SVG tracing, color palettes. |
| **Structured & UI** | `vision_describe_structured`, `vision_vqa`, `vision_ui_layout`, `vision_translate_image` | JSON breakdown (`{summary, ocr, layout, entities}`), short VQA, UI section analysis. |
| **Pixel & Diagnostics** | `vision_pixel_diff`, `vision_quality_check` | Semantic visual diff, quality scoring (blur/lighting). |
| **Documents & Intelligence** | `vision_extract_formula`, `vision_extract_table`, `vision_scan_barcode`, `vision_extract_structured`, `vision_audit_accessibility` | Formula (LaTeX), table (Markdown/HTML), QR/barcode scan, JSON-schema extraction, WCAG accessibility audit. |
| **Scenarios, Consensus & Memory** | `vision_ui_flow`, `vision_consensus`, `vision_memory_search` | User-journey graph (Mermaid), multi-model consensus, semantic search over remembered images. |
| **Attachments (v0.5.33)** | `vision_attach_pages`, `vision_attach_frames`, `vision_attach_images` | Publish PDF pages, video frames and local/remote images as conversation attachments so native-vision chat models read the pixels themselves. |
---
## π¦ Installation
```bash
dsh plugin --profile web add @goodandready/dsh-vision-bridge
```
After installation, restart the DSH Web UI. The configuration card is available under **Settings β Plugins β vision-bridge**.
---
## βοΈ Configuration (`settings.yaml`)
```yaml
dsh-vision-bridge:
# Operation mode: 'hybrid' | 'llm' | 'tools'
mode: hybrid
# Auto-detect vision model or specify provider/model explicitly
visionProvider: ""
visionModel: ""
# Allow vision models to receive images natively without rewriting ('prefer' | 'never' | 'always')
nativePassthrough: prefer
hideRedundantTools: true # hide bridge tools when the chat model already sees images
attachMaxItems: 8 # pages/frames/files one attach call may publish
# Enable LRU description cache
cacheEnabled: true
cacheMaxEntries: 200
# Request timeout in milliseconds
timeoutMs: 120000
# Multi-channel routing configuration
channels: []
channelFallback: sequential # 'sequential' | 'parallel-race'
```
### Parameter Reference
| Parameter | Type | Default | Description |
|---|---|---|---|
| `mode` | `string` | `"hybrid"` | Processing mode (`hybrid`, `llm`, `tools`). |
| `visionProvider` | `string` | `""` | ID of the vision provider (empty = auto-detect). |
| `visionModel` | `string` | `""` | ID of the vision model (empty = auto-detect). |
| `nativePassthrough` | `string` | `"prefer"` | Behavior for native vision models (`prefer`, `never`, `always`). |
| `hideRedundantTools` | `boolean` | `true` | When the chat model supports images natively, hide the compensation tools from that agent and keep only the extra instruments. |
| `attachMaxItems` | `number` | `8` | Maximum images one `vision_attach_*` call publishes (PDF pages, video frames, files). Hard ceiling 32. |
| `cacheEnabled` | `boolean` | `true` | Enables LRU caching for descriptions. |
| `cacheMaxEntries` | `number` | `200` | Maximum number of cached items in memory. |
| `timeoutMs` | `number` | `120000` | Execution timeout in milliseconds. |
| `channelFallback` | `string` | `"sequential"` | Channel routing (`sequential`, `parallel-race`); ordering via `channelOrderMode`. |
---
### 15. π‘οΈ Enterprise Security & Settings Governance (`v0.5.27`)
* **Masked Secrets**: `GET /channels` automatically masks sensitive provider API keys (`sk-p...7890` or `********`) and reports `hasApiKey: true` to prevent secret leakage in browser DevTools/XHR. `POST /channels` preserves existing keys when a mask is submitted.
* **CSRF & Origin Guard**: All mutating and cost-incurring endpoints (`/config`, `/channels`, `/upload-pdf`, `/test`, `/bench`, `/batch`, `DELETE /journal`, `DELETE /cache`, and `/doctor?probe=1`) validate `sec-fetch-site !== 'cross-site'` via `isTrustedSettingsRequest`, rejecting cross-origin attacks with `403 Forbidden`. The default `GET /doctor` report is static (no channel probes).
* **SSRF fetch policy**: model-supplied image URLs are fetched only through `safeFetch` β non-http(s) schemes, localhost names and private/loopback/link-local hosts (incl. IPv4-mapped IPv6 and NAT64) are refused on every redirect hop; response bodies are capped. Use `allowedUrlHosts` to explicitly re-allow an internal endpoint.
* **apiKeyRef**: channels may reference a credential-service entry or environment variable by NAME instead of storing a plaintext `apiKey` in settings; keys resolve at call time. `GET /channels` keeps masking values; key preservation on save matches channels by identity, not position.
* **Privacy & honesty**: `maskPII`, `stripEXIF`, `auditLog` and `consensusEnabled` settings are wired end-to-end; the previously decorative face-blur / NSFW / tiling toggles and no-op presets were removed.
* **Native settingsScope Integration**: Settings UI binds directly to `ctx.settingsScope` (`namespace: 'dsh-vision-bridge'`), supporting reactive snapshot listeners and kernel state synchronization.
---
## π Changed in v0.5.30
Security and honesty release. Additive summary of what changed for users:
* **SSRF fetch policy**: model-supplied image URLs (`describe_image` urls, `inspect_image`, and the headless-chrome tools) are now fetched only through a policy layer β non-http(s) schemes, localhost names and private/loopback/link-local hosts (incl. IPv4-mapped IPv6 and NAT64) are refused on every redirect hop; bodies are capped. New setting `allowedUrlHosts` (exact-hostname allowlist) deliberately re-allows an internal endpoint.
* **`apiKeyRef`**: channels can reference a credential-service entry or environment variable BY NAME β plaintext API keys are no longer required in `settings.yaml`. Existing inline `apiKey` values keep working; masked-key preservation on save now matches channels by identity, not position.
* **Route guards**: `POST /bench`, `POST /batch`, `DELETE /batch/:id`, `DELETE /journal`, `DELETE /cache` now require same-origin like the other mutating routes. `GET /doctor` is static by default; channel probes run only with `?probe=1` (same-origin required).
* **Settings honesty**: `maskPII`, `stripEXIF`, `auditLog` and `consensusEnabled` are now wired end-to-end. The previously decorative `blurFaces`, `nsfwFilter`, `tileLargeImages`/`tileThreshold` toggles and the no-op Local/Cloud/LM Studio presets were removed. Changed in v0.5.30: if you relied on them, note they never had an effect.
* **English source language**: all user-facing strings are English; the bundled Russian dictionary was removed β the translation plugin supplies Russian at runtime.
* **Core split**: the pure kernel (config schema + helpers) moved to `lib/vision-core.js`; `lib/index.js` re-exports it β no API changes. `describe_image` lost a v0.5.13 regression that returned an empty description; `vision_annotate` works again; pHash caching no longer mixes up similar images.
---
## π Changed in v0.5.31
Maintenance release β no user-facing behavior changes.
* **Internal structure**: tool registrations moved to `lib/tools/*` domain modules (core / grounding / ocr / document / analysis / media); `lib/index.js` re-exports everything as before. Smaller files, explicit dependencies between the host and the tool domains.
* **Settings card hardening**: locale registration is fault-tolerant, the redundant sidebar fallback was removed, plugin service access goes through a safe `ctx.get` wrapper, and `/config` accepts the expanded field set (`cacheMaxEntries`, `channelFallback`).
---
## π Changed in v0.5.32
Stability and architecture release.
* **Internal structure**: all ~44 tool registrations moved to `lib/tools/*` domain modules (core / grounding / ocr / document / analysis / media) with explicit dependencies; `lib/index.js` keeps the host wiring only.
* **Stability fixes**: finished batch records are released after a 10-minute poll window (memory growth fixed); `/upload-pdf` rejects payloads above the new `maxPdfBytes` setting (20 MiB default) instead of buffering arbitrary bodies; `vision_memory_search` scores each attachment against its own description (previously all attachments matched identically); dead host code removed; journal labels are consistent between the legacy and channels paths.
* **Settings**: new `maxPdfBytes` setting (upload hard cap).
---
## π Changed in v0.5.33
Vision-first release: the bridge now serves chat models that see images natively, not only text-only ones.
* **Attach domain (`vision_attach_pages`, `vision_attach_frames`, `vision_attach_images`)**: PDF pages, sampled video frames and local/directory/URL images are published as **conversation attachments**, so a native-vision chat model looks at the pixels itself instead of paying for a second vision call. Every image is compressed by the existing `imageMaxWidth`/`imageMaxHeight`/`imageQuality` settings and bounded by `maxImageBytes`; URL sources go through the same SSRF policy as the rest, and local paths are restricted by `allowedImageDirs`.
* **Model-aware tool exposure**: on a route whose chat model already accepts images, the compensation tools of the bridge are hidden from that agent (the model does not need them) and only the extra instruments remain; on a text-only route the attach tools are hidden instead, because that model cannot see an attachment. The behaviour is controlled by the new `hideRedundantTools` setting (on by default).
* **New settings in the plugin card**: an **Attachments** group exposes `attachMaxItems` (whole number 1β32, default 8 β how many images one attach call publishes) and `hideRedundantTools`. Both are validated on save; a value outside the range is rejected with a clear message. The image dimension fields are now also written to the live settings snapshot, not only to the route.
* **Batch API**: `DELETE /batch/:id` releases a finished batch immediately, under the same same-origin guard as start/cancel. The batch record's TTL timer no longer keeps a short-lived process alive, which cut the test suite from 10 minutes to ~3.5 seconds.
* **Tools mode**: images attached in chat are now indexed before the sanitisation gate, so tools that take an attachment id (and the `read_image` alias) work in `tools` mode as well; previously the ids were unavailable there.
* **Fixes**: one unreadable source no longer aborts `vision_attach_images` β it is reported as `Skipped N: : ` while the readable sources still attach, and a fetch-policy refusal stays a hard error; the PDF text layer actually appears now (`pdftotext` was never detected, so the layer silently never shipped); a page range or frame count cut by the cap is reported through `truncated` and the note.
* **Internal**: CI installs poppler without `sudo`, runs once per commit, serializes per ref and installs ffmpeg for the frame tests; the repository gained the pull-request template and ignores `.worktrees/`.
---
## π Changed in v0.6.0
Major stability, reliability, and lifecycle release.
* **In-App Auto-Updater**: added dedicated one-click update endpoint (`/api/dsh-vision-bridge/update`) and Settings card controls (`UpdaterBlock`) in `settings.plugin.item`. Features real-time npm version check, progress state, and restart guidance, protected by loopback and CSRF origin verification gates.
* **Error Resilience & Catch Elimination**: audited and replaced all 76 unannotated empty catch blocks across core runtime and tools. Added `bestEffort(label, fn, fallback)` helper for non-fatal side effects, explicit warning propagation in image processing (`smartOptimizeImage` returns `preprocessed: boolean` and `warnings: string[]`), and transparent fallback reporting.
* **Identity & Scope Integrity**: strictly aligned scoped package identity `@goodandready/dsh-vision-bridge` across manifest, Cordis patch, client module loader, and internal runtime metadata.
* **Hardened Settings Security**: upgraded settings mutation route to a strict fail-closed validator (`isTrustedSettingsRequest`) enforcing loopback origin, CSRF header checks, bearer tokens, and rejecting suspicious remote headers.
* **Native Theme Compliance**: client diagnostics and card styles fully migrated to DSH CSS design tokens (`--dsw-alias-*`), eliminating all hardcoded color literals.
* **Sanitized Public Release**: integrated plumbing-based release script (`publish.sh`) and `.gitattributes` export filters ensuring zero private development artifacts in public distribution.
---
## π Changed in v0.6.2
Reliability hardening, error visibility, and degradation warnings release.
* **Real bestEffort Logging**: replaced all remaining dummy `/* bestEffort ... */ void err` catches across core runtime, tools, and routes with active `bestEffort(label, fn)` logging to `console.debug`.
* **Degradation Warnings in Tools**: all tool modules now declare `warnings: { type: 'array', items: { type: 'string' } }` in output schemas and surface `warnings: string[]` on partial JSON parsing errors or engine fallbacks.
* **Image Preprocessing Failure Capture**: preprocessing exceptions (deskew, enhance, stripEXIF, compression) are recorded and surfaced to tool callers instead of failing silently.
* **Expanded Verification**: added unit test suite `test/issue-314-besteffort-warnings.test.js`, bringing total test coverage to 329 passing tests across 104 suites.
---
## π License
MIT Β© [GooDAnDReaDY](https://github.com/GooDAnDReaDY)