Nicholai/jan
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: extracted macos avx2 check code to the utility function

Browse Source
This commit is contained in:
Sherzod Mutalov 2025-06-26 14:16:10 +05:00 committed by Ramon Perez
parent ad9c4854a9
commit 280ea1aa9f
2 changed files with 21 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src-tauri/src/core/mcp.rs
View File

@ -10,7 +10,11 @@ use tokio::{
time::{sleep, timeout},
};
use super::{cmd::get_jan_data_folder_path, state::AppState};
use super::{
cmd::get_jan_data_folder_path,
state::AppState,
utils::can_override_npx,
};
const DEFAULT_MCP_CONFIG: &str = r#"{
"mcpServers": {
@ -512,20 +516,8 @@ async fn schedule_mcp_start_task<R: Runtime>(
.ok_or_else(|| format!("Failed to extract command args from config for {name}"))?;
let mut cmd = Command::new(command.clone());
// we need to check CPU for the AVX2 instruction if we are running under the MacOS
// with Intel CPU. We can replace `npx` command with `bun` only if CPU is
// supporting AVX2, otherwise we need to use default `npx` binary
let mut is_macos_without_avx2 = false;
if cfg!(all(target_os="macos", any(target_arch = "x86", target_arch = "x86_64")))
{
is_macos_without_avx2 = !is_x86_feature_detected!("avx2");
if is_macos_without_avx2 && command == "npx" {
log::warn!("Your CPU doesn't support AVX2 instruction, default npx binary will be used");
}
}
if command == "npx" && !is_macos_without_avx2 {
if command == "npx" && can_override_npx() {
let mut cache_dir = app_path.clone();
cache_dir.push(".npx");
let bun_x_path = format!("{}/bun", bin_path.display());

15
src-tauri/src/core/utils/mod.rs
View File

@ -195,3 +195,18 @@ pub fn is_library_available(library: &str) -> bool {
}
}
}
pub fn can_override_npx() -> bool {
// we need to check the CPU for the AVX2 instruction support if we are running under the MacOS
// with Intel CPU. We can override `npx` command with `bun` only if CPU is
// supporting AVX2, otherwise we need to use default `npx` binary
if cfg!(all(target_os="macos", any(target_arch = "x86", target_arch = "x86_64")))
{
if !is_x86_feature_detected!("avx2") {
log::warn!("Your CPU doesn't support AVX2 instruction, default npx binary will be used");
return false; // we cannot override npx with bun binary
}
}
true // by default, we can override npx with bun binary
}