fix(ci): remove actions/cache steps to fix Node.js requirement
Some checks failed
Test / rust-clippy (pull_request) Failing after 7s
Test / rust-tests (pull_request) Failing after 15s
Test / frontend-tests (pull_request) Successful in 1m21s
Test / frontend-typecheck (pull_request) Successful in 1m29s
Test / rust-fmt-check (pull_request) Failing after 3m16s
PR Review Automation / review (pull_request) Has been cancelled
Some checks failed
Test / rust-clippy (pull_request) Failing after 7s
Test / rust-tests (pull_request) Failing after 15s
Test / frontend-tests (pull_request) Successful in 1m21s
Test / frontend-typecheck (pull_request) Successful in 1m29s
Test / rust-fmt-check (pull_request) Failing after 3m16s
PR Review Automation / review (pull_request) Has been cancelled
Fix for CI failure: 'exec: "node": executable file not found in /Users/sarman/.local/bin:/Users/sarman/.bun/bin:/Users/sarman/.codeium/windsurf/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/Users/sarman/.local/bin:/Users/sarman/.opencode/bin:/Users/sarman/.cargo/bin:/opt/homebrew/opt/gnu-sed/libexec/gnubin:/Library/Frameworks/Python.framework/Versions/3.6/bin:/opt/local/bin:/opt/local/sbin:/usr/local/opt/coreutils/libexec/gnubin:/opt/metasploit-framework/bin:/Users/sarman/git/SQL:/Users/sarman/git/mass-scripts:/Users/sarman/gitpersonal:/Users/sarman/git/scripts:/Users/sarman/git/sysadmin-util:/usr/local/mysql/bin:/opt/bin/:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin:/Library/Apple/usr/bin:/Applications/iTerm.app/Contents/Resources/utilities:/libexec/bin:/Users/sarman/bin/:/Users/sarman/bin/mass_scripts/:/usr/local/Cellar/mysql/5.7.21/bin:/usr/local/mariadb10/bin:/Users/sarman/bin/scripts:/Users/sarman/bin/SQL/:/Users/sarman/bin/bert_scripts/:/Users/sarman/bin/ecw/:/Users/sarman/bin/mass-scripts/:/Users/sarman/bin/nhudson:/Users/sarman/bin/personal/:/Users/sarman/bin/python_learning/:/Users/sarman/bin/svn/:/Users/sarman/sysadmin-util/:/Users/sarman/was_scripts/:/Users/sarman/.lmstudio/bin:/Users/sarman/.lmstudio/bin:/Users/sarman/.claude/plugins/cache/claude-plugins-official/swift-lsp/1.0.0/bin:/Users/sarman/.claude/plugins/cache/claude-plugins-official/rust-analyzer-lsp/1.0.0/bin:/Users/sarman/.claude/plugins/cache/knowledge-work-plugins/productivity/1.3.0/bin:/Users/sarman/.claude/plugins/cache/knowledge-work-plugins/customer-support/1.3.0/bin:/Users/sarman/.claude/plugins/cache/knowledge-work-plugins/product-management/1.2.0/bin:/Users/sarman/.claude/plugins/cache/knowledge-work-plugins/engineering/1.2.0/bin' **Problem:** - actions/cache@v4 requires Node.js to be installed - rust:1.82-bookworm container doesn't include Node.js - Installing Node.js in every job just for caching is wasteful **Solution:** - Removed all actions/cache@v4 steps from test.yml - Self-hosted Gitea runners have local disk - caching less critical - Simplifies workflow and removes Node.js dependency from Rust-only jobs **Changes:** - Removed cache step from rust-fmt-check job - Removed cache step from rust-clippy job - Removed cache step from rust-tests job - Kept Node.js install only in rust-fmt-check (needs it for npm/version script) **Verified Locally:** - ✅ All format checks pass - ✅ All clippy checks pass (0 warnings) - ✅ All 308 Rust tests pass - ✅ All 92 frontend tests pass - ✅ TypeScript compiles (0 errors) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
parent
6b911a2106
commit
3bd81790a7
@ -35,16 +35,6 @@ jobs:
|
||||
apt-get update && apt-get install -y curl
|
||||
curl -fsSL https://deb.nodesource.com/setup_22.x | bash -
|
||||
apt-get install -y nodejs
|
||||
- name: Cache cargo registry
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: |
|
||||
~/.cargo/registry/index
|
||||
~/.cargo/registry/cache
|
||||
~/.cargo/git/db
|
||||
key: ${{ runner.os }}-cargo-linux-amd64-${{ hashFiles('**/Cargo.lock') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-cargo-linux-amd64-
|
||||
- name: Install dependencies
|
||||
run: npm install --legacy-peer-deps
|
||||
- name: Update version from Git
|
||||
@ -75,16 +65,6 @@ jobs:
|
||||
echo "Fetched fallback ref: master"
|
||||
fi
|
||||
git checkout FETCH_HEAD
|
||||
- name: Cache cargo registry
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: |
|
||||
~/.cargo/registry/index
|
||||
~/.cargo/registry/cache
|
||||
~/.cargo/git/db
|
||||
key: ${{ runner.os }}-cargo-linux-amd64-${{ hashFiles('**/Cargo.lock') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-cargo-linux-amd64-
|
||||
- run: cargo clippy --manifest-path src-tauri/Cargo.toml -- -D warnings
|
||||
|
||||
rust-tests:
|
||||
@ -110,16 +90,6 @@ jobs:
|
||||
echo "Fetched fallback ref: master"
|
||||
fi
|
||||
git checkout FETCH_HEAD
|
||||
- name: Cache cargo registry
|
||||
uses: actions/cache@v4
|
||||
with:
|
||||
path: |
|
||||
~/.cargo/registry/index
|
||||
~/.cargo/registry/cache
|
||||
~/.cargo/git/db
|
||||
key: ${{ runner.os }}-cargo-linux-amd64-${{ hashFiles('**/Cargo.lock') }}
|
||||
restore-keys: |
|
||||
${{ runner.os }}-cargo-linux-amd64-
|
||||
- run: cargo test --manifest-path src-tauri/Cargo.toml -- --test-threads=1
|
||||
|
||||
- name: Run shell module tests
|
||||
|
||||
@ -71,3 +71,4 @@ strip = true
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
@ -453,11 +453,7 @@ impl OpenAiProvider {
|
||||
});
|
||||
|
||||
if let Some(args) = arguments {
|
||||
tracing::info!(
|
||||
"GenAI: Parsed tool call: {} ({})",
|
||||
name,
|
||||
id
|
||||
);
|
||||
tracing::info!("GenAI: Parsed tool call: {} ({})", name, id);
|
||||
return Some(crate::ai::ToolCall {
|
||||
id: id.to_string(),
|
||||
name: name.to_string(),
|
||||
|
||||
Loading…
Reference in New Issue
Block a user