Compare commits
2 Commits
74e85fb8a2
...
windows
| Author | SHA1 | Date | |
|---|---|---|---|
| 50cce7a22c | |||
| 027999a9e5 |
@@ -47,11 +47,18 @@ pub fn find_claude_binary(app_handle: &tauri::AppHandle) -> Result<String, Strin
|
||||
|row| row.get::<_, String>(0),
|
||||
) {
|
||||
info!("Found stored claude path in database: {}", stored_path);
|
||||
|
||||
|
||||
// Check if the path still exists and works
|
||||
#[cfg(not(target_os = "windows"))]
|
||||
let final_path = stored_path.clone();
|
||||
#[cfg(not(target_os = "windows"))]
|
||||
let path_buf = PathBuf::from(&stored_path);
|
||||
|
||||
#[cfg(target_os = "windows")]
|
||||
let mut final_path = stored_path.clone();
|
||||
#[cfg(target_os = "windows")]
|
||||
let mut path_buf = PathBuf::from(&stored_path);
|
||||
|
||||
|
||||
// On Windows, if stored path exists but is not executable (shell script), try .cmd version
|
||||
#[cfg(target_os = "windows")]
|
||||
if path_buf.exists() && !stored_path.ends_with(".cmd") && !stored_path.ends_with(".exe") {
|
||||
@@ -62,28 +69,28 @@ pub fn find_claude_binary(app_handle: &tauri::AppHandle) -> Result<String, Strin
|
||||
let cmd_path_buf = PathBuf::from(&cmd_path);
|
||||
if cmd_path_buf.exists() {
|
||||
if let Ok(_) = get_claude_version(&cmd_path) {
|
||||
final_path = cmd_path.clone();
|
||||
final_path = cmd_path;
|
||||
path_buf = cmd_path_buf;
|
||||
info!("Using .cmd version instead of shell script: {}", cmd_path);
|
||||
info!("Using .cmd version instead of shell script: {}", final_path);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
if path_buf.exists() && path_buf.is_file() {
|
||||
return Ok(final_path);
|
||||
} else {
|
||||
warn!("Stored claude path no longer exists: {}", stored_path);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// Check user preference
|
||||
let preference = conn.query_row(
|
||||
"SELECT value FROM app_settings WHERE key = 'claude_installation_preference'",
|
||||
[],
|
||||
|row| row.get::<_, String>(0),
|
||||
).unwrap_or_else(|_| "system".to_string());
|
||||
|
||||
|
||||
info!("User preference for Claude installation: {}", preference);
|
||||
}
|
||||
}
|
||||
@@ -206,7 +213,7 @@ fn find_which_installations() -> Vec<ClaudeInstallation> {
|
||||
// Process each line (Windows 'where' can return multiple paths)
|
||||
for line in output_str.lines() {
|
||||
let mut path = line.trim().to_string();
|
||||
|
||||
|
||||
if path.is_empty() {
|
||||
continue;
|
||||
}
|
||||
@@ -445,10 +452,10 @@ fn get_claude_version(path: &str) -> Result<Option<String>, String> {
|
||||
/// Extract version string from command output
|
||||
fn extract_version_from_output(stdout: &[u8]) -> Option<String> {
|
||||
let output_str = String::from_utf8_lossy(stdout);
|
||||
|
||||
|
||||
// Debug log the raw output
|
||||
debug!("Raw version output: {:?}", output_str);
|
||||
|
||||
|
||||
// Use regex to directly extract version pattern (e.g., "1.0.41")
|
||||
// This pattern matches:
|
||||
// - One or more digits, followed by
|
||||
@@ -458,7 +465,7 @@ fn extract_version_from_output(stdout: &[u8]) -> Option<String> {
|
||||
// - One or more digits
|
||||
// - Optionally followed by pre-release/build metadata
|
||||
let version_regex = regex::Regex::new(r"(\d+\.\d+\.\d+(?:-[a-zA-Z0-9.-]+)?(?:\+[a-zA-Z0-9.-]+)?)").ok()?;
|
||||
|
||||
|
||||
if let Some(captures) = version_regex.captures(&output_str) {
|
||||
if let Some(version_match) = captures.get(1) {
|
||||
let version = version_match.as_str().to_string();
|
||||
@@ -466,7 +473,7 @@ fn extract_version_from_output(stdout: &[u8]) -> Option<String> {
|
||||
return Some(version);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
debug!("No version found in output");
|
||||
None
|
||||
}
|
||||
@@ -546,7 +553,7 @@ fn compare_versions(a: &str, b: &str) -> Ordering {
|
||||
/// This ensures commands like Claude can find Node.js and other dependencies
|
||||
pub fn create_command_with_env(program: &str) -> Command {
|
||||
let mut cmd = Command::new(program);
|
||||
|
||||
|
||||
info!("Creating command for: {}", program);
|
||||
|
||||
// Inherit essential environment variables from parent process
|
||||
@@ -574,7 +581,7 @@ pub fn create_command_with_env(program: &str) -> Command {
|
||||
cmd.env(&key, &value);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
// Log proxy-related environment variables for debugging
|
||||
info!("Command will use proxy settings:");
|
||||
if let Ok(http_proxy) = std::env::var("HTTP_PROXY") {
|
||||
|
||||
@@ -5,7 +5,7 @@ use tauri::{AppHandle, Emitter, State};
|
||||
use tokio::sync::Mutex;
|
||||
use uuid::Uuid;
|
||||
use anyhow::Result;
|
||||
use portable_pty::{native_pty_system, CommandBuilder, PtySize};
|
||||
use portable_pty::{native_pty_system, CommandBuilder, PtySize, Child, MasterPty};
|
||||
use std::io::{Read, Write};
|
||||
|
||||
#[derive(Debug, Clone, Serialize, Deserialize)]
|
||||
@@ -19,6 +19,8 @@ pub struct TerminalSession {
|
||||
/// Terminal child process wrapper
|
||||
pub struct TerminalChild {
|
||||
writer: Arc<Mutex<Box<dyn Write + Send>>>,
|
||||
_master: Box<dyn MasterPty + Send>, // Keep master PTY alive
|
||||
_child: Box<dyn Child + Send + Sync>, // Keep child process alive
|
||||
}
|
||||
|
||||
/// State for managing terminal sessions
|
||||
@@ -60,36 +62,69 @@ pub async fn create_terminal_session(
|
||||
|
||||
// Get shell command
|
||||
let shell = get_default_shell();
|
||||
log::info!("Using shell: {}", shell);
|
||||
let mut cmd = CommandBuilder::new(&shell);
|
||||
|
||||
// Set as login interactive shell
|
||||
if shell.contains("bash") || shell.contains("zsh") {
|
||||
cmd.arg("-il"); // Interactive login shell
|
||||
} else if shell.contains("fish") {
|
||||
cmd.arg("-il");
|
||||
// Set shell-specific arguments
|
||||
if cfg!(target_os = "windows") {
|
||||
if shell.contains("pwsh") {
|
||||
// PowerShell Core - stay interactive
|
||||
cmd.arg("-NoLogo");
|
||||
cmd.arg("-NoExit");
|
||||
} else if shell.contains("powershell") {
|
||||
// Windows PowerShell - stay interactive
|
||||
cmd.arg("-NoLogo");
|
||||
cmd.arg("-NoExit");
|
||||
} else {
|
||||
// cmd.exe - use /K to keep session open
|
||||
cmd.arg("/K");
|
||||
}
|
||||
} else {
|
||||
// Unix shells: Set as login interactive shell
|
||||
if shell.contains("bash") || shell.contains("zsh") {
|
||||
cmd.arg("-il"); // Interactive login shell
|
||||
} else if shell.contains("fish") {
|
||||
cmd.arg("-il");
|
||||
}
|
||||
}
|
||||
|
||||
// Set working directory
|
||||
cmd.cwd(working_directory.clone());
|
||||
|
||||
// Set environment variables
|
||||
cmd.env("TERM", "xterm-256color");
|
||||
cmd.env("COLORTERM", "truecolor");
|
||||
cmd.env("LANG", std::env::var("LANG").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
cmd.env("LC_ALL", std::env::var("LC_ALL").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
cmd.env("LC_CTYPE", std::env::var("LC_CTYPE").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
|
||||
// 继承其他环境变量
|
||||
for (key, value) in std::env::vars() {
|
||||
if !key.starts_with("TERM") && !key.starts_with("COLORTERM") && !key.starts_with("LC_") && !key.starts_with("LANG") {
|
||||
cmd.env(&key, &value);
|
||||
// Set environment variables based on platform
|
||||
if cfg!(target_os = "windows") {
|
||||
// Windows-specific environment
|
||||
cmd.env("TERM", "xterm-256color");
|
||||
// Keep PATH and other essential Windows environment variables
|
||||
for (key, value) in std::env::vars() {
|
||||
if !key.starts_with("TAURI_") && !key.starts_with("VITE_") {
|
||||
cmd.env(&key, &value);
|
||||
}
|
||||
}
|
||||
} else {
|
||||
// Unix-specific environment
|
||||
cmd.env("TERM", "xterm-256color");
|
||||
cmd.env("COLORTERM", "truecolor");
|
||||
cmd.env("LANG", std::env::var("LANG").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
cmd.env("LC_ALL", std::env::var("LC_ALL").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
cmd.env("LC_CTYPE", std::env::var("LC_CTYPE").unwrap_or_else(|_| "en_US.UTF-8".to_string()));
|
||||
|
||||
// Inherit other Unix environment variables
|
||||
for (key, value) in std::env::vars() {
|
||||
if !key.starts_with("TERM") && !key.starts_with("COLORTERM") &&
|
||||
!key.starts_with("LC_") && !key.starts_with("LANG") &&
|
||||
!key.starts_with("TAURI_") && !key.starts_with("VITE_") {
|
||||
cmd.env(&key, &value);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// Spawn the shell process
|
||||
let _child = pty_pair.slave.spawn_command(cmd)
|
||||
let child = pty_pair.slave.spawn_command(cmd)
|
||||
.map_err(|e| format!("Failed to spawn shell: {}", e))?;
|
||||
|
||||
log::info!("Shell process spawned successfully for session: {}", session_id);
|
||||
|
||||
// Get writer for stdin
|
||||
let writer = pty_pair.master.take_writer()
|
||||
.map_err(|e| format!("Failed to get PTY writer: {}", e))?;
|
||||
@@ -103,15 +138,20 @@ pub async fn create_terminal_session(
|
||||
// Spawn reader thread
|
||||
std::thread::spawn(move || {
|
||||
let mut buffer = [0u8; 4096];
|
||||
log::info!("PTY reader thread started for session: {}", session_id_clone);
|
||||
loop {
|
||||
match reader.read(&mut buffer) {
|
||||
Ok(0) => break, // EOF
|
||||
Ok(0) => {
|
||||
log::warn!("PTY reader got EOF for session: {}", session_id_clone);
|
||||
break; // EOF
|
||||
}
|
||||
Ok(n) => {
|
||||
let data = String::from_utf8_lossy(&buffer[..n]).to_string();
|
||||
log::debug!("PTY reader got {} bytes for session {}: {:?}", n, session_id_clone, data);
|
||||
let _ = app_handle_clone.emit(&format!("terminal-output:{}", session_id_clone), &data);
|
||||
}
|
||||
Err(e) => {
|
||||
log::error!("Error reading PTY output: {}", e);
|
||||
log::error!("Error reading PTY output for session {}: {}", session_id_clone, e);
|
||||
break;
|
||||
}
|
||||
}
|
||||
@@ -119,9 +159,11 @@ pub async fn create_terminal_session(
|
||||
log::debug!("PTY reader thread finished for session: {}", session_id_clone);
|
||||
});
|
||||
|
||||
// Store the session with PTY writer
|
||||
// Store the session with PTY writer, master PTY and child process
|
||||
let terminal_child = TerminalChild {
|
||||
writer: Arc::new(Mutex::new(writer)),
|
||||
_master: pty_pair.master,
|
||||
_child: child,
|
||||
};
|
||||
|
||||
{
|
||||
@@ -245,13 +287,13 @@ pub async fn cleanup_terminal_sessions(
|
||||
/// Get the default shell for the current platform
|
||||
fn get_default_shell() -> String {
|
||||
if cfg!(target_os = "windows") {
|
||||
// Try PowerShell first, fallback to cmd
|
||||
// Try PowerShell Core (pwsh) first, then Windows PowerShell, fallback to cmd
|
||||
if std::process::Command::new("pwsh").arg("--version").output().is_ok() {
|
||||
"pwsh".to_string()
|
||||
} else if std::process::Command::new("powershell").arg("-Version").output().is_ok() {
|
||||
"powershell".to_string()
|
||||
} else {
|
||||
"cmd".to_string()
|
||||
"cmd.exe".to_string()
|
||||
}
|
||||
} else {
|
||||
// Unix-like systems: try zsh, bash, then sh
|
||||
|
||||
@@ -18,7 +18,7 @@
|
||||
}
|
||||
],
|
||||
"security": {
|
||||
"csp": "default-src 'self'; img-src 'self' asset: https://asset.localhost blob: data:; style-src 'self' 'unsafe-inline' blob: data: asset: https://asset.localhost; style-src-elem 'self' 'unsafe-inline' blob: data: asset: https://asset.localhost; style-src-attr 'self' 'unsafe-inline'; script-src 'self' 'unsafe-eval' https://app.posthog.com https://*.posthog.com https://*.i.posthog.com https://*.assets.i.posthog.com; worker-src 'self' blob: asset: https://asset.localhost; font-src 'self' data: blob: asset: https://asset.localhost; connect-src 'self' ipc: https://ipc.localhost https://app.posthog.com https://*.posthog.com https://*.i.posthog.com",
|
||||
"csp": "default-src 'self'; img-src 'self' asset: https://asset.localhost blob: data:; style-src 'self' 'unsafe-inline' blob: data: asset: https://asset.localhost; style-src-elem 'self' 'unsafe-inline' blob: data: asset: https://asset.localhost; style-src-attr 'self' 'unsafe-inline'; script-src 'self' 'unsafe-eval' https://app.posthog.com https://*.posthog.com https://*.i.posthog.com https://*.assets.i.posthog.com; worker-src 'self' blob: asset: https://asset.localhost; font-src 'self' data: blob: asset: https://asset.localhost; connect-src 'self' ipc: http://ipc.localhost https://ipc.localhost https://app.posthog.com https://*.posthog.com https://*.i.posthog.com",
|
||||
"assetProtocol": {
|
||||
"enable": true,
|
||||
"scope": [
|
||||
@@ -34,9 +34,7 @@
|
||||
},
|
||||
"bundle": {
|
||||
"active": true,
|
||||
"targets": [
|
||||
"msi"
|
||||
],
|
||||
"targets": "all",
|
||||
"icon": [
|
||||
"icons/32x32.png",
|
||||
"icons/128x128.png",
|
||||
|
||||
@@ -45,7 +45,8 @@ import { SplitPane } from "@/components/ui/split-pane";
|
||||
import { WebviewPreview } from "./WebviewPreview";
|
||||
import { FileExplorerPanelEnhanced } from "./FileExplorerPanelEnhanced";
|
||||
import { GitPanelEnhanced } from "./GitPanelEnhanced";
|
||||
import { FileEditorEnhanced } from "./FileEditorEnhanced";
|
||||
// 动态导入 FileEditorEnhanced 以减少初始包大小
|
||||
const FileEditorEnhanced = React.lazy(() => import("./FileEditorEnhanced"));
|
||||
import { SlashCommandsManager } from "./SlashCommandsManager";
|
||||
import type { ClaudeStreamMessage } from "./AgentExecution";
|
||||
import { useVirtualizer } from "@tanstack/react-virtual";
|
||||
@@ -1904,11 +1905,17 @@ export const ClaudeCodeSession: React.FC<ClaudeCodeSessionProps> = ({
|
||||
<div className={cn("h-full w-full", layout.activeView === 'terminal' ? 'hidden' : 'block')}>
|
||||
{layout.activeView === 'editor' && layout.editingFile ? (
|
||||
// 文件编辑器视图
|
||||
<FileEditorEnhanced
|
||||
filePath={layout.editingFile}
|
||||
onClose={closeFileEditor}
|
||||
className="h-full"
|
||||
/>
|
||||
<React.Suspense fallback={
|
||||
<div className="flex items-center justify-center h-full">
|
||||
<div className="animate-spin rounded-full h-8 w-8 border-b-2 border-primary"></div>
|
||||
</div>
|
||||
}>
|
||||
<FileEditorEnhanced
|
||||
filePath={layout.editingFile}
|
||||
onClose={closeFileEditor}
|
||||
className="h-full"
|
||||
/>
|
||||
</React.Suspense>
|
||||
) : layout.activeView === 'preview' && layout.previewUrl ? (
|
||||
// 预览视图
|
||||
<SplitPane
|
||||
|
||||
@@ -51,10 +51,17 @@ export default defineConfig(async () => ({
|
||||
'react-vendor': ['react', 'react-dom'],
|
||||
'ui-vendor': ['@radix-ui/react-dialog', '@radix-ui/react-dropdown-menu', '@radix-ui/react-select', '@radix-ui/react-tabs', '@radix-ui/react-tooltip', '@radix-ui/react-switch', '@radix-ui/react-popover'],
|
||||
'editor-vendor': ['@uiw/react-md-editor'],
|
||||
'monaco-editor': ['monaco-editor', '@monaco-editor/react'],
|
||||
'syntax-vendor': ['react-syntax-highlighter'],
|
||||
// Animation and motion
|
||||
'framer-motion': ['framer-motion'],
|
||||
// Tauri and other utilities
|
||||
'tauri': ['@tauri-apps/api', '@tauri-apps/plugin-dialog', '@tauri-apps/plugin-shell'],
|
||||
'tauri': ['@tauri-apps/api', '@tauri-apps/plugin-dialog', '@tauri-apps/plugin-shell', '@tauri-apps/plugin-fs', '@tauri-apps/plugin-clipboard-manager'],
|
||||
'utils': ['date-fns', 'clsx', 'tailwind-merge'],
|
||||
// Charts and visualization
|
||||
'recharts': ['recharts'],
|
||||
// Virtual scrolling
|
||||
'virtual': ['@tanstack/react-virtual'],
|
||||
},
|
||||
},
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user