Skip to content

Improve Windows setup, Muti-model extension, API auth, sessions, and project recovery - #154

Merged
delibae merged 140 commits into
delibae:mainfrom
Gerard-Devlin:improve-windows-api-sessions
Jul 4, 2026
Merged

Improve Windows setup, Muti-model extension, API auth, sessions, and project recovery#154
delibae merged 140 commits into
delibae:mainfrom
Gerard-Devlin:improve-windows-api-sessions

Conversation

@Gerard-Devlin

Copy link
Copy Markdown
Contributor

Summary

This PR improves the ClaudePrism desktop experience on Windows and fixes several setup, auth, session, and project recovery issues found while running the app locally.

Changes

  • Add a Windows-friendly desktop dev launcher script.
  • Add support for saving an external Claude-compatible API key and optional Base URL.
  • Inject stored ANTHROPIC_API_KEY and ANTHROPIC_BASE_URL into Claude CLI processes.
  • Handle UTF-8 BOM in local auth settings to avoid JSON parse failures.
  • Avoid false “Claude process exited unexpectedly” errors when Claude stream-json reports a successful result.
  • Update the Scientific Skills installer to use K-Dense-AI/scientific-agent-skills, with fallback support for the previous repo layout.
  • Add a delete button for Claude session history entries.
  • Restore existing projects from Documents/ClaudePrism when recent-project localStorage is lost.
  • Allow opening projects even when Claude/API setup is not ready.
  • Fix overflow in the Python Environment dialog for long virtualenv paths.
  • Remove a redundant preview-collapse floating button.

Verification

  • corepack pnpm --filter @claude-prism/desktop build
  • cargo check --no-default-features
  • corepack pnpm exec biome check apps/desktop/src/components/project-picker.tsx
  • corepack pnpm exec biome check apps/desktop/src/components/claude-chat/session-selector.tsx

Notes

  • This is opened as a draft because the PR touches several related desktop setup and recovery flows.

@Gerard-Devlin
Gerard-Devlin marked this pull request as ready for review June 7, 2026 17:04
refactor(proposed-changes-panel): simplify JSX structure for added/removed indicators
refactor(pdf-viewer): format function parameters for better readability
fix(project-name): enhance validation for project names to prevent invalid characters
refactor(build): improve environment flag handling for better clarity
refactor(dev): streamline environment flag appending logic
- Added `import_skill_from_folder` command to import skills from a local directory.
- Implemented `delete_installed_skill` command to remove specific skills.
- Updated the skills installation process to provide detailed download progress.
- Refactored skills loading and management logic for better organization and efficiency.
- Enhanced the UI to support skill importing and deletion with appropriate user feedback.
@Gerard-Devlin

Gerard-Devlin commented Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

Milestone update

This milestone resolves major workflow requests and also includes a set of workspace usability improvements.

Addressed issues

Additional UI improvements

  • Added layout controls for toggling Code, PDF, and Sidebar panes.

- Updated `useClaudeSetupStore` to require Claude Code installation before saving provider credentials.
- Modified `ClaudeSetup` component to support an embedded variant with improved UI for installation and configuration.
- Added detailed status messages for missing Git and installation prompts.
- Refactored project picker to include theme toggle and version display.
- Improved layout and styling for settings and environment status sections.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@delibae

delibae commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Overall a solid PR. I ran the proxy logic against an isolated test crate (no tectonic deps) - all 34 existing tests plus 14 additional verification tests passed.

Key findings:

  • Proxy binds to 127.0.0.1 only - no external exposure ✓
  • serde_yaml 0.9 follows YAML 1.2 spec, so 'yes' stays a string (no bool coercion) ✓
  • Oversized Content-Length + early connection close exits cleanly without hanging ✓
  • Windows titlebar DWM attr 20→19 fallback looks correct ✓

Items to follow up in a future patch:

  • Add a body size cap in read_http_request (no upper bound currently)
  • Strip UTF-8 BOM in the proxy layer before JSON parsing
  • Handle Transfer-Encoding: chunked or return a clearer error

Will merge and address these in the next release. Nice work!

@delibae
delibae merged commit 973005e into delibae:main Jul 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants