Skip to content

fix: remove unresolved import for tauriExtensionHost.contribution#1

Merged
Razshy merged 1 commit intoSidenai:mainfrom
Binlogo:fix/missing-tauri-extension-host-contribution
Apr 2, 2026
Merged

fix: remove unresolved import for tauriExtensionHost.contribution#1
Razshy merged 1 commit intoSidenai:mainfrom
Binlogo:fix/missing-tauri-extension-host-contribution

Conversation

@Binlogo
Copy link
Copy Markdown
Contributor

@Binlogo Binlogo commented Apr 2, 2026

Problem

npm run tauri dev fails with a Vite build error:

Failed to resolve import "./contrib/extensions/browser/tauriExtensionHost.contribution.js"
from "src/vs/workbench/workbench.common.main.ts". Does the file exist?

The import at line 302 of workbench.common.main.ts references a file that was never created.

Fix

Remove the dangling import to unblock local development.

If Tauri-specific extension host registration is needed in the future, the corresponding file (tauriExtensionHost.contribution.ts) should be created before re-adding this import.

The file `tauriExtensionHost.contribution.ts` was referenced in
`workbench.common.main.ts` but never created, causing a Vite build
failure at dev startup:

  Failed to resolve import "./contrib/extensions/browser/tauriExtensionHost.contribution.js"

Remove the dangling import to unblock `npm run tauri dev`.
If Tauri-specific extension host registration is needed in the future,
the file should be created before re-adding this import.
@Razshy Razshy merged commit 85bc5cd into Sidenai:main Apr 2, 2026
h8d13 referenced this pull request in h8d13/sidex Apr 2, 2026
…-issues

Fix stale aria label after clearing Settings search results
@Binlogo Binlogo deleted the fix/missing-tauri-extension-host-contribution branch April 3, 2026 11:18
BandiAkarsh added a commit to BandiAkarsh/sidex that referenced this pull request Apr 13, 2026
…nai#1)

## Performance Improvement

Added thread-safe LRU cache for file metadata to avoid repeated stat()
system calls. This significantly improves performance when accessing
the same files repeatedly (e.g., file tree navigation, saving files).

### Changes

- Added `lru` crate to Cargo.toml for O(1) cache operations
- Created `cache.rs` module with FileMetadataCache struct
- Cache stores up to 10,000 file metadata entries
- Automatic eviction of least recently used entries
- Added unit tests for cache operations

### Technical Details

- Uses `dashmap` for thread-safe concurrent access
- Async methods for non-blocking cache operations
- Cache entry includes: size, timestamps, permissions
- Ready to integrate with fs commands in Phase 2

### Testing

- cargo test: 2/2 tests passing
- cargo check: no errors

### Related

- Part of memory optimization plan (PERFORMANCE_OPTIMIZATION.md)
- Follows security-first pattern from validation module

Co-authored-by: Akarsh Bandi <bandiakarsh@gmail.com>
BandiAkarsh added a commit to BandiAkarsh/sidex that referenced this pull request Apr 13, 2026
…nai#1)

## Performance Improvement

Added thread-safe LRU cache for file metadata to avoid repeated stat()
system calls. This significantly improves performance when accessing
the same files repeatedly (e.g., file tree navigation, saving files).

### Changes

- Added `lru` crate to Cargo.toml for O(1) cache operations
- Created `cache.rs` module with FileMetadataCache struct
- Cache stores up to 10,000 file metadata entries
- Automatic eviction of least recently used entries
- Added unit tests for cache operations

### Technical Details

- Uses `dashmap` for thread-safe concurrent access
- Async methods for non-blocking cache operations
- Cache entry includes: size, timestamps, permissions
- Ready to integrate with fs commands in Phase 2

### Testing

- cargo test: 2/2 tests passing
- cargo check: no errors

### Related

- Part of memory optimization plan (PERFORMANCE_OPTIMIZATION.md)
- Follows security-first pattern from validation module

Co-authored-by: Akarsh Bandi <bandiakarsh@gmail.com>
Alex-Keagel added a commit to Alex-Keagel/caduceus-ide that referenced this pull request Apr 13, 2026
…pace'

Root cause: SideX passes workspace folders as file:///path URIs but
the engine tools expect plain /path. All tools failed with 'path escapes
workspace' because the URI was never resolved to a filesystem path.

Fixed in 3 layers (defense in depth):
- Frontend: strip file:// in syncWorkspaceState() and syncActiveFile()
- Backend set_project_root: strip file:// before canonicalize
- Backend resolve_project_root: strip file:// from frontend parameter

This was the Sidenai#1 user-facing bug — agent couldn't read/write any files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Alex-Keagel pushed a commit to Alex-Keagel/caduceus-ide that referenced this pull request Apr 14, 2026
…denai#1)

Remove dangling import for non-existent tauriExtensionHost.contribution.js
Alex-Keagel added a commit to Alex-Keagel/caduceus-ide that referenced this pull request Apr 14, 2026
…pace'

Root cause: SideX passes workspace folders as file:///path URIs but
the engine tools expect plain /path. All tools failed with 'path escapes
workspace' because the URI was never resolved to a filesystem path.

Fixed in 3 layers (defense in depth):
- Frontend: strip file:// in syncWorkspaceState() and syncActiveFile()
- Backend set_project_root: strip file:// before canonicalize
- Backend resolve_project_root: strip file:// from frontend parameter

This was the Sidenai#1 user-facing bug — agent couldn't read/write any files.
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.

2 participants