feature: UI & API improvements — accordion Changes panel, label removal, LLM config cleanup - #29
Merged
Merged
Conversation
- Backend: concurrent git diff --cached/--numstat with per-section error reporting, returning 500 only when both commands fail - Frontend: mutually exclusive accordion in sidebar; toggle triggers re-render so errors follow the expanded section - Docs: updated PRD, ARCHITECTURE, and API docs
- Remove Archived/ArchivedAt fields from ManagedInstance - Remove Archive(), PurgeArchivedInstances() from instance manager - Modify Delete() to work on stopped instances directly - Modify Restart() to auto-delete old instance instead of archiving - Remove /api/instances/archive and /api/instances/purge routes - Remove instance_archive, instance_purge MCP tools - Remove Archive/Purge buttons, Show archived checkbox from UI - Update API docs
Resolved conflicts in docs/API.md: kept staged/unstaged git status split, removed archive/purge (favoring direct delete), kept LLM configuration section from develop.
…ng_split, tighten availability check - Remove openai_compatible protocol; consolidate into openai - Remove auto-fill defaults for model and API address (user must configure explicitly) - Add reasoning_split toggle as a configurable option - Add thinking: false to disable DeepSeek thinking mode by default - IsAvailable() now requires all four fields (protocol, key, address, model) non-empty - Anthropic auth header auto-detects based on URL (x-api-key for api.anthropic.com, Bearer otherwise) - UI: mute AI Generate button when LLM not fully configured, click opens settings - Remove dead DefaultModel/DefaultAddress functions - Sync docs (API.md, PRD.md) with current implementation
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
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.
Summary