Merge pull request 'fix(m9): revert sidebar container a11y id; measure presetSelect frame (#130)' (#131) from feat/130-sidebar-a11y-revert into milestone/m9-monterey

On SDK 13.1 the 'sidebar' identifier on the root VStack propagates to
every child, erasing openSettingsBtn/openAboutBtn/btnToggleAllHelp
(a11y dump, run 32201). Revert it; the test now compares presetSelect
frame before/after toggling help overlays.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

Sources/ICCery/SidebarView.swift

@@ -129,7 +129,6 @@ struct SidebarView: View {
         }
         .frame(width: Theme.Metrics.sidebarWidth)
         .background(Theme.panel)
-        .accessibilityIdentifier("sidebar")
     }
 }
 

Tests/ICCeryUITests/AboutHelpUITests.swift

@@ -69,15 +69,18 @@ final class AboutHelpUITests: XCTestCase {
         let toggle = app.buttons["btnToggleAllHelp"]
         XCTAssertTrue(toggle.waitForExistence(timeout: 10))
 
-        let sidebar = app.descendants(matching: .any)["sidebar"]
-        XCTAssertTrue(sidebar.waitForExistence(timeout: 10))
-        let before = sidebar.frame
+        // SDK 13.1 emits no AXGroup for the sidebar root, and an
+        // identifier on the container clobbers child identifiers
+        // (#130) — measure a stable sidebar child instead.
+        let sidebarChild = app.descendants(matching: .any)["presetSelect"]
+        XCTAssertTrue(sidebarChild.waitForExistence(timeout: 10))
+        let before = sidebarChild.frame
 
         toggle.click()
-        let after = sidebar.frame
+        let after = sidebarChild.frame
 
-        XCTAssertEqual(before.size.height, after.size.height,
-                       "Toggling global help must not reflow the sidebar height.")
+        XCTAssertEqual(before, after,
+                       "Toggling global help must not reflow the sidebar.")
         XCTAssertTrue(app.descendants(matching: .any)["openSettingsBtn"].exists)
     }
 }