Network: Add host discovery

- Rust-side changes based on host-discovery.md, implementing Bonjour discovery.
- Svelte-side changes based on host-discovery.md. Added NetworkBrowser.svelte
- Update docs: host-discovery.md based on new decisions / nuances
- It is super fast and super sleek!

Author: vdavid

docs/features/network-smb/host-discovery.md

@@ -5,7 +5,7 @@ How Rusty Commander discovers SMB hosts on the local network.
 ## Overview
 
 Network hosts are discovered automatically using **Bonjour** (Apple's implementation of mDNS/DNS-SD). When an
-SMB-capable device advertises itself on the local network, it appears in the volume selector under "Network".
+SMB-capable device advertises itself on the local network, it appears in the Network browser view.
 
 ## How Bonjour works
 
@@ -43,90 +43,103 @@ Delegate receives callbacks as services appear/disappear
 NSNetService objects (one per discovered host)
     │
     ▼
-resolve() each to get IP address + port
+hostname/IP derived from service name (lazy resolution)
 ```
 
 ### Challenges
 
-| Challenge              | Solution                                                                                                  |
-| ---------------------- | --------------------------------------------------------------------------------------------------------- |
-| **Delegate pattern**   | Create a Rust struct that implements `NSNetServiceBrowserDelegate` using `objc2`'s `declare_class!` macro |
-| **Async/streaming**    | Services appear over time; use Rust channels or async streams to propagate updates                        |
-| **Service resolution** | Each `NSNetService` needs `resolve()` call for IP/hostname (another async operation)                      |
-| **Run loop**           | Bonjour callbacks require a run loop; use Tauri's main thread or dedicated run loop                       |
+| Challenge              | Solution                                                                                                 |
+| ---------------------- | -------------------------------------------------------------------------------------------------------- |
+| **Delegate pattern**   | Create a Rust struct that implements `NSNetServiceBrowserDelegate` using `objc2`'s `define_class!` macro |
+| **Async/streaming**    | Services appear over time; use Tauri events to propagate updates                                         |
+| **Service resolution** | Hostname derived from service name, IP resolved via DNS on hover/demand                                  |
+| **Run loop**           | Bonjour callbacks require a run loop; use Tauri's main thread                                            |
 
 ### Lifecycle
 
 1. **App startup**: Start `NSNetServiceBrowser` listening for `_smb._tcp.local`
 2. **Continuous**: Receive callbacks as hosts appear/disappear on network
-3. **Cache results**: Keep discovered hosts in memory for instant volume selector display
-4. **Volume selector opened**: Show cached hosts immediately; continue updating as new hosts found
+3. **Cache results**: Keep discovered hosts in memory for instant display
+4. **Network view opened**: Show cached hosts immediately; continue updating as new hosts found
 
 ### Lazy resolution
 
-Resolution (getting IP/hostname from `NSNetService`) adds ~50–200 ms latency per host. To keep discovery snappy:
+Resolution (getting IP/hostname) is deferred until needed to keep discovery fast:
 
 - **On discovery**: Store host name only (from mDNS announcement)
-- **On hover (500 ms debounce)**: Resolve that specific host
+- **On hover**: Resolve that specific host's hostname and IP
 - **On navigate**: Resolve if not already cached
 - **Cache**: Keep resolved addresses in memory
 
-This way the volume selector populates instantly with host names, and resolution happens just-in-time.
+This way the network browser populates instantly with host names, and resolution happens just-in-time.
 
 ### Host disappearance
 
 Bonjour proactively notifies when hosts stop advertising (via `netServiceBrowser:didRemoveService:`).
 
 **UI behavior when a host disappears:**
 
-- **In volume selector**: Remove host from list, keep cursor at same index (or last item if was last)
-- **In file pane browsing network hosts**: Remove disappeared host, handle gracefully like any deleted item
-
-### Prefetching for known hosts
-
-After discovery settles (~3 seconds), prefetch share information for hosts in `knownNetworkShares`:
-
-```
-Discovery complete
-    → For each known host that was discovered:
-        → Queue share enumeration (parallel, cap at 10)
-        → Cache results for instant display when user navigates
-```
-
-This provides instant response for servers the user has connected to before, without probing unknown hosts.
+- **In Network browser**: Remove host from list, adjust selection
+- **During connection**: Show appropriate error if mid-operation
 
 ## UX behavior
 
-### In the volume selector
+### Volume selector
+
+The volume selector shows a single "Network" item under the Network section:
 
 ```
 📁 Favorites
    └── Documents
    └── Downloads
 📁 Macintosh HD
 📁 External Drive
-📶 Network ← Click to expand or show inline
-   └── David's M1 MBP
-   └── Naspolya
-   └── PI
-   └── (Searching...) ← While initial discovery in progress
+🌐 Network ← Click to open Network browser
+```
+
+### Network browser view
+
+When user clicks "Network" in the volume selector, the pane switches to the Network browser view:
+
+```
+┌────────────────────────────────────────────────────────────┐
+│ Name              │ IP address     │ Hostname      │ Status │
+├────────────────────────────────────────────────────────────┤
+│ 🖥️ David's MacBook │ 192.168.1.10  │ macbook.local │   —    │
+│ 🖥️ NAS-Server      │ 192.168.1.50  │ nas.local     │   —    │
+│ 🖥️ Office-PC       │ —             │ —             │   —    │
+│                                                            │
+│ Searching... (if discovery in progress)                    │
+└────────────────────────────────────────────────────────────┘
 ```
 
-### Progress indication
+**Columns:**
+
+- **Name**: Host's advertised name from Bonjour
+- **IP address**: Resolved IP (on hover/demand), or "—" if not yet resolved
+- **Hostname**: Derived `.local` hostname
+- **Shares**: Share count (future, currently "—")
+- **Status**: Connection status (future, currently "—")
+
+**Keyboard navigation:**
+
+- Arrow Up/Down: Navigate between hosts
+- Enter: Select host (future: show shares)
+- Backspace: No-op (can't go "up" from network root)
+
+**Back/Forward navigation:**
 
-- **While searching**: Subtle spinner or "Searching..." text
-- **Hosts appear incrementally**: Each host shows up as discovered (streaming UX)
-- **After ~3–5 seconds**: Consider discovery "complete" but keep listening for changes
+- Works as expected: going Back returns to previous file view
+- Coming Forward again returns to Network browser
 
 ## Network change detection
 
-Bonjour automatically handles network changes, but we enhance this:
+Bonjour automatically handles network changes:
 
-| Event                    | Detection               | Action                 |
-| ------------------------ | ----------------------- | ---------------------- |
-| Host starts advertising  | Bonjour callback        | Add to list            |
-| Host stops advertising   | Bonjour callback        | Remove from list       |
-| Network interface change | `SCNetworkReachability` | Pause/resume discovery |
+| Event                   | Detection        | Action           |
+| ----------------------- | ---------------- | ---------------- |
+| Host starts advertising | Bonjour callback | Add to list      |
+| Host stops advertising  | Bonjour callback | Remove from list |
 
 ## What Bonjour doesn't cover
 
@@ -156,16 +169,13 @@ Discovery is lightweight enough to start immediately at app launch.
 
 ## Testing
 
-### Unit tests
+### Backend unit tests
 
-- Mock `NSNetServiceBrowser` to simulate host discovery
-- Test callback handling for service found/lost events
-- Test caching and deduplication logic
-- Test lazy resolution caching
+- `test_service_name_to_hostname`: Tests hostname derivation from service names
+- `test_service_name_to_id`: Tests ID generation from service names
+- `test_network_host_serialization`: Tests JSON serialization of NetworkHost
 
-### Integration tests
+### Frontend tests
 
-- Verify hosts appear in volume list when discovered
-- Test host disappearance when service goes offline
-- Test behavior when no hosts found (empty "Network" section)
-- Test prefetching triggers for known hosts
+- `network-hosts.test.ts`: Tests network host type interfaces and event handling logic
+- Integration tests verify NetworkBrowser renders correctly and handles events

docs/features/network-smb/task-list.md

@@ -2,6 +2,8 @@
 
 All implementation tasks for network SMB support, organized by area.
 
+See [index.md](./index.md) for an overview of this whole feature. It's a helpful starting point.
+
 ## Legend
 
 - ⬜ Not started
@@ -17,22 +19,22 @@ See [host-discovery.md](./host-discovery.md) for details.
 
 ### Backend (Rust)
 
-- ⬜ **1.1** Create `network` module in `src-tauri/src/`
-- ⬜ **1.2** Implement `NSNetServiceBrowserDelegate` using `objc2` `declare_class!`
-- ⬜ **1.3** Create `BonjourDiscovery` struct that manages the browser lifecycle
-- ⬜ **1.4** Implement service resolution (get IP/hostname from `NSNetService`)
-- ⬜ **1.5** Create Tauri commands: `start_network_discovery`, `get_discovered_hosts`
-- ⬜ **1.6** Add event emission for real-time host updates to frontend
-- ⬜ **1.7** Start discovery at app initialization (in `lib.rs`)
-- ⬜ **1.8** Add unit tests with mocked Bonjour responses
+- ✅ **1.1** Create `network` module in `src-tauri/src/`
+- ✅ **1.2** Implement `NSNetServiceBrowserDelegate` using `objc2` `define_class!`
+- ✅ **1.3** Create `BonjourDiscovery` struct that manages the browser lifecycle
+- ✅ **1.4** Implement service resolution (lazy hostname generation + DNS lookup)
+- ✅ **1.5** Create Tauri commands: `list_network_hosts`, `get_network_discovery_state`, `resolve_host`
+- ✅ **1.6** Add event emission for real-time host updates to frontend
+- ✅ **1.7** Start discovery at app initialization (in `lib.rs`)
+- ✅ **1.8** Add unit tests for hostname conversion and serialization
 
 ### Frontend (Svelte)
 
-- ⬜ **1.9** Add "Network" section to volume selector
-- ⬜ **1.10** Subscribe to host discovery events from backend
-- ⬜ **1.11** Display discovered hosts with appropriate icons
-- ⬜ **1.12** Show "Searching..." indicator during initial discovery
-- ⬜ **1.13** Add frontend tests for network host display
+- ✅ **1.9** Add "Network" section to volume selector
+- ✅ **1.10** Subscribe to host discovery events from backend
+- ✅ **1.11** Display discovered hosts with appropriate icons
+- ✅ **1.12** Show "Searching..." indicator during initial discovery
+- ✅ **1.13** Add frontend tests for network host display
 
 ---
 

eslint.config.js

@@ -29,11 +29,11 @@ export default tseslint.config(
     prettierConfig,
     ...tseslint.configs.strictTypeChecked.map((config) => ({
         ...config,
-        files: ['**/*.{ts,tsx,svelte}'],
+        files: ['**/*.{ts,tsx,svelte.ts,svelte}'],
     })),
     ...svelte.configs['flat/recommended'],
     {
-        files: ['**/*.{ts,tsx}'],
+        files: ['**/*.{ts,tsx,svelte.ts}'],
         plugins: {
             '@typescript-eslint': tseslint.plugin,
             prettier,
@@ -110,6 +110,47 @@ export default tseslint.config(
             'prettier/prettier': 'error',
         },
     },
+    {
+        // Svelte 5 runes files (.svelte.ts) - TypeScript with Svelte runes support
+        files: ['**/*.svelte.ts'],
+        plugins: {
+            '@typescript-eslint': tseslint.plugin,
+            prettier,
+        },
+        languageOptions: {
+            parser: tseslint.parser,
+            ecmaVersion: 'latest',
+            sourceType: 'module',
+            globals: {
+                ...globals.browser,
+                ...globals.node,
+                ...globals.es2021,
+            },
+            parserOptions: {
+                projectService: true,
+                tsconfigRootDir: import.meta.dirname,
+            },
+        },
+        rules: {
+            'prettier/prettier': 'error',
+            // Type safety rules - same as main TypeScript files
+            '@typescript-eslint/no-unused-vars': 'error',
+            '@typescript-eslint/no-unsafe-assignment': 'error',
+            '@typescript-eslint/no-unsafe-call': 'error',
+            '@typescript-eslint/no-unsafe-member-access': 'error',
+            '@typescript-eslint/no-unsafe-return': 'error',
+            // Async/Promise safety
+            '@typescript-eslint/no-floating-promises': 'error',
+            '@typescript-eslint/await-thenable': 'error',
+            '@typescript-eslint/no-misused-promises': 'error',
+            '@typescript-eslint/require-await': 'error',
+            // No any
+            '@typescript-eslint/no-explicit-any': 'error',
+            'no-console': 'warn',
+            // Complexity limit
+            complexity: ['error', { max: 15 }],
+        },
+    },
     {
         files: ['**/*.svelte'],
         plugins: {

knip.json

@@ -1,6 +1,6 @@
 {
     "$schema": "https://unpkg.com/knip@5/schema.json",
-    "ignore": ["src/lib/icon-cache.ts", "src/routes/+layout.ts", "src/lib/benchmark.ts"],
+    "ignore": ["src/lib/icon-cache.ts", "src/routes/+layout.ts", "src/lib/benchmark.ts", "src/lib/tauri-commands.ts"],
     "ignoreDependencies": ["@tauri-apps/cli", "@testing-library/svelte", "@sveltejs/adapter-static"],
     "ignoreExportsUsedInFile": true,
     "project": ["src/**/*.{ts,svelte}"],

src-tauri/Cargo.lock

@@ -39,6 +39,7 @@ bincode2 = "2"
 tauri-plugin-drag = "2.1.0"
 tauri-plugin-fs = "2.4.4"
 alphanumeric-sort = "1.5"
+tokio = { version = "1.49.0", features = ["rt"] }
 [target.'cfg(not(any(target_os = "android", target_os = "ios")))'.dependencies]
 tauri-plugin-window-state = "2"
 
@@ -48,10 +49,10 @@ core-services = "1.0.0"
 icns = "0.3.1"
 plist = "1.8.0"
 urlencoding = "2.1.3"
-objc2 = "0.6"
+objc2 = { version = "0.6", features = ["std"] }
 objc2-foundation = { version = "0.3", features = [
     "NSURL", "NSString", "NSDictionary", "NSDate", "NSArray", "NSValue", "NSError",
-    "NSFileManager"
+    "NSFileManager", "NSNetServices", "NSRunLoop"
 ] }
 
 [dev-dependencies]

src-tauri/Cargo.toml

@@ -4,6 +4,8 @@ pub mod file_system;
 pub mod font_metrics;
 pub mod icons;
 #[cfg(target_os = "macos")]
+pub mod network;
+#[cfg(target_os = "macos")]
 pub mod sync_status;
 pub mod ui;
 #[cfg(target_os = "macos")]