Conversation
Co-authored-by: chris.skene <chris.skene@gmail.com>
Completes the documentation suite by adding: - API.md: Full reference for Tauri commands, events, and Aurabox REST API - CONFIGURATION.md: Detailed guide for all configuration options and settings These files complement the previously added README.md, ARCHITECTURE.md, and DEVELOPMENT.md to provide complete developer and user documentation.
Write documentation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-TAURIAPPSPLUGINSHELL-9697750 - https://snyk.io/vuln/SNYK-JS-NEXT-8602067
The frontend 'Open Log File' button was constructing the path as
{appName}.log but the backend configures tauri-plugin-log with
file_name 'logs', producing logs.log. Updated the frontend to use
the correct filename and simplified the redundant platform branching.
Also corrected AGENTS.md to document the actual log directories used
by tauri-plugin-log, which differ from the app data directories.
Added a C-FIND Service Class User (SCU) implementation to perform Study Root queries against a PACS. Includes support for configurable query filters and parsing results into structured data. Comprehensive unit tests ensure coverage for command building, identifiers, and result parsing.
…hanced tests Added functionality for PATIENT-level C-FIND queries, including custom identifier construction, result parsing, and comprehensive unit tests. Refactored query handling to differentiate levels and adjusted related test cases accordingly.
When a connected SCU (PACS workstation, HIS) sends a C-FIND to Bounce, the new cfind_handler extracts filter attributes from the DICOM identifier, posts them to Aura's POST /api/bounce/find endpoint, and streams the matching studies back as DICOM C-FIND-RSP Pending PDUs. - cfind_handler.rs: SCP handler with filter extraction, result encoding - FindStudyRequest/FindStudyResult/FindStudiesResponse models - QueryApiClient::find_studies() HTTP client method - Study Root FIND abstract syntax accepted by dicom_server.rs - Command field 0x0020 dispatch in DICOM server loop - 7 find_api tests + 8 cfind_handler unit tests (194 total passing)
The pending response was sending the dataset P-DATA-TF before the command P-DATA-TF, violating DICOM PS3.7/PS3.8 which requires the command to precede the dataset. Conformant SCUs (including Orthanc) read CommandDataSetType from the command to know a dataset follows, so receiving a bare data fragment first caused results to be silently discarded.
Track shared agent instructions, MCP wiring, and project context so Claude, Codex, and OpenCode use the same local automation setup.
Move the Orthanc test compose file under docker/ and drop unused updater manifest files from the repo root.
Wire Tauri log output into the webview so the in-app log screen can filter, clear, and inspect live backend events alongside the log file.
Track pending C-STORE and C-FIND commands until their data PDUs arrive so proxy queries and file receipts stay aligned, and add Aura query diagnostics to make remote PACS troubleshooting clearer.
Expose the repo-managed Automatic skills through Claude's local skills directory so the shared project setup resolves the same skill set across tools.
Stop tracking the local Automatic workspace metadata and snapshots so per-machine state stays out of the repo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I had to fix some security warnings in Bounce, so I did a pass and improved a few things
Added
Integrated shadcn/ui component library for a consistent and modern design system.
Added new UI components: Button, Card, Input, Label, Badge, Alert.
Added Heroicons to the sidebar navigation.
Reintroduced Indigo as the primary brand color across the application (buttons, active states, focus rings).
Changed
UI Overhaul:
Redesigned Sidebar with a dark theme (slate-900) and improved typography.
Updated Dashboard to use card-based layout for status and stats.
Refactored Studies list to use clean, card-based items with badge status indicators.
Improved Settings page layout: API Key and Storage Directory now span full width for better readability.
Modernized Logs and Tools pages with consistent styling.
Theming:
Implemented CSS variables for theme tokens (background, foreground, primary, muted, etc.).
Switched global background to a clean slate tone.
Backend:
Updated Rust version to 1.80.
Fixed various clippy lints and warnings for better code quality.
Fixed
Set default values in the UI to match those that are provided when no values are entered for AT Title etc
Fixed IP address showing as AE title on overview screen
Various security updates for related packages