fix: in get_selected_device_parameters only send available parameters

Author: fabb

src/main/java/io/github/fabb/wigai/bitwig/BitwigApiFacade.java

@@ -60,11 +60,11 @@ public BitwigApiFacade(ControllerHost host, Logger logger) {
         // Initialize device control - use CursorTrack.createCursorDevice() instead of deprecated host.createCursorDevice()
         this.cursorTrack = host.createCursorTrack(0, 0);
         this.cursorDevice = cursorTrack.createCursorDevice();
-        this.deviceParameterBank = cursorDevice.createCursorRemoteControlsPage(128);
+        this.deviceParameterBank = cursorDevice.createCursorRemoteControlsPage(8);
 
         // Initialize project parameter access via MasterTrack (project parameters)
         MasterTrack masterTrack = host.createMasterTrack(0);
-        this.projectParameterBank = masterTrack.createCursorRemoteControlsPage(128);
+        this.projectParameterBank = masterTrack.createCursorRemoteControlsPage(8);
 
         // Initialize track bank for clip launching (support up to 128 tracks and 128 scenes for full functionality)
         this.trackBank = host.createTrackBank(128, 0, 128);
@@ -86,6 +86,7 @@ public BitwigApiFacade(ControllerHost host, Logger logger) {
         // Mark interest in all device parameter properties to enable value access
         for (int i = 0; i < deviceParameterBank.getParameterCount(); i++) {
             RemoteControl parameter = deviceParameterBank.getParameter(i);
+            parameter.exists().markInterested();
             parameter.name().markInterested();
             parameter.value().markInterested();
             parameter.displayedValue().markInterested();
@@ -270,16 +271,20 @@ public List<ParameterInfo> getSelectedDeviceParameters() {
 
         for (int i = 0; i < deviceParameterBank.getParameterCount(); i++) {
             RemoteControl parameter = deviceParameterBank.getParameter(i);
-            String name = parameter.name().get();
-            double value = parameter.value().get();
-            String displayValue = parameter.displayedValue().get();
+            boolean exists = parameter.exists().get();
 
-            // Handle null or empty names
-            if (name != null && name.trim().isEmpty()) {
-                name = null;
-            }
+            if (exists) {
+                String name = parameter.name().get();
+                double value = parameter.value().get();
+                String displayValue = parameter.displayedValue().get();
 
-            parameters.add(new ParameterInfo(i, name, value, displayValue));
+                // Handle null or empty names
+                if (name != null && name.trim().isEmpty()) {
+                    name = null;
+                }
+
+                parameters.add(new ParameterInfo(i, name, value, displayValue));
+            }
         }
 
         logger.info("BitwigApiFacade: Retrieved " + parameters.size() + " parameters");

src/main/java/io/github/fabb/wigai/mcp/tool/StatusTool.java

@@ -53,7 +53,6 @@ public Object execute() throws Exception {
                             for (io.github.fabb.wigai.common.data.ParameterInfo param : projectParams) {
                                 Map<String, Object> paramMap = new LinkedHashMap<>();
                                 paramMap.put("index", param.index());
-                                paramMap.put("exists", true);
                                 paramMap.put("name", param.name());
                                 paramMap.put("value", param.value());
                                 paramMap.put("display_value", param.display_value());

src/test/java/io/github/fabb/wigai/bitwig/BitwigApiFacadeTest.java

@@ -85,11 +85,11 @@ void setUp() {
         when(mockHost.createApplication()).thenReturn(mockApplication);
         when(mockHost.createCursorTrack(0, 0)).thenReturn(mockCursorTrack);
         when(mockCursorTrack.createCursorDevice()).thenReturn(mockCursorDevice);
-        when(mockCursorDevice.createCursorRemoteControlsPage(128)).thenReturn(mockParameterBank);
+        when(mockCursorDevice.createCursorRemoteControlsPage(8)).thenReturn(mockParameterBank);
 
         // Setup new mocks for story 5.2
         when(mockHost.createMasterTrack(0)).thenReturn(mockMasterTrack);
-        when(mockMasterTrack.createCursorRemoteControlsPage(128)).thenReturn(mockProjectParameterBank);
+        when(mockMasterTrack.createCursorRemoteControlsPage(8)).thenReturn(mockProjectParameterBank);
 
         // Setup TrackBank mocks (new for clip launching) - use smaller sizes for testing
         when(mockHost.createTrackBank(128, 0, 128)).thenReturn(mockTrackBank);
@@ -112,6 +112,7 @@ void setUp() {
         lenient().when(mockCursorDevice.exists()).thenReturn(mock(com.bitwig.extension.controller.api.BooleanValue.class));
         lenient().when(mockCursorDevice.name()).thenReturn(mock(com.bitwig.extension.controller.api.SettableStringValue.class));
         lenient().when(mockCursorDevice.isEnabled()).thenReturn(mock(com.bitwig.extension.controller.api.SettableBooleanValue.class));
+        lenient().when(mockRemoteControl.exists()).thenReturn(mock(com.bitwig.extension.controller.api.BooleanValue.class));
         lenient().when(mockRemoteControl.name()).thenReturn(mock(com.bitwig.extension.controller.api.SettableStringValue.class));
         lenient().when(mockRemoteControl.value()).thenReturn(mock(com.bitwig.extension.controller.api.SettableRangedValue.class));
         lenient().when(mockRemoteControl.displayedValue()).thenReturn(mock(com.bitwig.extension.controller.api.SettableStringValue.class));