Monitor Switch + HA controls + overlay refit; merge-drop fix; full test suite (0.0.48)

.github/workflows/test.yml

@@ -53,6 +53,15 @@ jobs:
     permissions:
       checks: write
 
+  rustfmt_check:
+    name: Formatting for Rust backend
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+      - run: rustup update stable && rustup default stable && rustup component add rustfmt
+      # Source-only formatting check (no compile / no frontend build needed). Run `cargo fmt` to fix.
+      - run: cargo fmt --all --check
+
   client_test:
     name: Tests and linting for React client
     runs-on: ubuntu-latest

AGENTS.md

@@ -201,7 +201,9 @@ the working-directory-aware tools.
 | Build | `cargo build` |
 | Test | `cargo test` |
 | Lint | `cargo clippy` |
+| Format (check / fix) | `cargo fmt --check` / `cargo fmt` (default rustfmt) |
 | Run full app (dev) | `cargo tauri dev` |
+| Run a 2nd dev instance alongside the installed release | `cargo run -- --multi` (or `WIDGETSACK_MULTI=1`) — skips the single-instance lock + isolates config to `<app config>/multi` |
 | Release build → `target/release/widgetsack.exe` | `cargo tauri build` |
 
 > ⚠️ **Build order gotcha:** Tauri embeds `client/build` (`frontendDist`), so the frontend

client/.eslintrc.cjs

@@ -22,5 +22,13 @@ module.exports = {
 	},
 	settings: {
 		react: { version: 'detect' }
-	}
+	},
+	overrides: [
+		{
+			// Test code may use non-null assertions on values it has just constructed / knows are defined
+			// (e.g. asserting on a Patch's `monitor`). Production code keeps the strict rule.
+			files: ['**/*.test.ts', '**/*.test.tsx'],
+			rules: { '@typescript-eslint/no-non-null-assertion': 'off' }
+		}
+	]
 };

client/.gitignore

@@ -18,3 +18,4 @@ build/*
 /blob-report
 /.playwright-mcp
 /playwright/.cache
+coverage/

client/.prettierignore

@@ -20,3 +20,4 @@ build
 /blob-report
 /.playwright-mcp
 /playwright/.cache
+coverage/

client/e2e/helpers.ts

@@ -49,12 +49,24 @@ export async function previewTemplate(page: Page, name: string): Promise<void> {
 	await page.getByRole('button', { name, exact: true }).click();
 }
 
-/** Add a widget from the Inspector palette (Layouts section) and return the now-selected host. */
+/**
+ * Add a widget from the Inspector palette (Layouts section) and return the now-selected host.
+ * `type` is the palette LABEL (e.g. "Gauge", "Analog Clock"). The palette button's accessible name
+ * is "<label> — <description>" (the description was folded into the aria-label), so we anchor on the
+ * label: the accessible name is exactly the label, or the label followed by the " — " separator. The
+ * anchor avoids the substring collisions a bare name match would hit (e.g. "Bar" ⊂ Gauge's
+ * "…linear bar…", "Text" ⊂ Transcribe's "…speech-to-text…").
+ */
 export async function addWidget(page: Page, type: string): Promise<Locator> {
 	await openSection(page, 'layouts');
+	// The Add palette lives in a <details> that auto-collapses once a node is selected, so re-open it
+	// (a no-op on a fresh boot where nothing is selected) before reaching for a palette button.
+	const summary = page.locator('.inspector .add-panel > summary');
+	if (!(await page.locator('.inspector .add-panel[open]').count())) await summary.click();
+	const label = type.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); // escape regex metachars in the label
 	await page
 		.locator('.inspector .palette')
-		.getByRole('button', { name: type, exact: true })
+		.getByRole('button', { name: new RegExp(`^${label}( — |$)`) })
 		.click();
 	const selected = page.locator('.widget.selected');
 	await expect(selected).toBeVisible();