.github/workflows/build_addon.yml

@@ -1,35 +1,40 @@
-name: Build Addon
+name: build addon
 
 on:
   push:
-    tags:
-      - 'v*.*.*'
+    tags: ["*"]
+    # To build on main/master branch, uncomment the following line:
+    # branches: [ main , master ]
+
+  pull_request:
+    branches: [ main, master ]
 
   workflow_dispatch:
 
 jobs:
   build:
+
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ['3.11']
+        os: [ubuntu-latest]
 
     steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
+      - uses: actions/checkout@v4
       - name: Set up PDM
         uses: pdm-project/setup-pdm@v4
         with:
-          python-version: '3.11'
-
+          python-version: ${{ matrix.python-version }}
+  
       - name: Install dependencies
         run: |
           pdm sync -d -G build
 
-      - name: Build addon
+      - name: building addon
         run: pdm run scons
 
-      - name: Upload build artifacts
-        if: success()
-        uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v3
         with:
           name: packaged_addon
           path: ./*.nvda-addon
@@ -37,20 +42,17 @@ jobs:
   upload_release:
     runs-on: ubuntu-latest
     if: ${{ startsWith(github.ref, 'refs/tags/') }}
-    needs: build
-
+    needs: ["build"]
     steps:
-      - name: Download releases files
-        uses: actions/download-artifact@v4.1.7
-        with:
-          name: packaged_addon
+    - uses: actions/checkout@v3
+    - name: download releases files
+      uses: actions/download-artifact@v4.1.7
+    - name: Display structure of downloaded files
+      run: ls -R
 
-      - name: Display structure of downloaded files
-        run: tree
-
-      - name: Release
-        uses: softprops/action-gh-release@v1
-        with:
-          files: ./*.nvda-addon
-          fail_on_unmatched_files: true
-          prerelease: ${{ contains(github.ref, '-') }}
+    - name: Release
+      uses: softprops/action-gh-release@v1
+      with:
+        files: packaged_addon/*.nvda-addon
+        fail_on_unmatched_files: true
+        prerelease: ${{ contains(github.ref, '-') }}

.pre-commit-config.yaml

@@ -2,11 +2,6 @@ repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.3.0
     hooks:
+      - id: check-ast
+      - id: check-case-conflict
       - id: check-yaml
-      - id: end-of-file-fixer
-      - id: trailing-whitespace
-
-  - repo: https://github.com/pdm-project/pdm
-    rev: 2.26.6
-    hooks:
-      - id: pdm-lock-check

README.md

@@ -1,4 +1,4 @@
-[![pdm-managed](https://img.shields.io/endpoint?url=https%3A%2F%2Fcdn.jsdelivr.net%2Fgh%2Fpdm-project%2F.github%2Fbadge.json)](https://pdm-project.org)
+[![pdm-managed](https://img.shields.io/badge/pdm-managed-blueviolet)](https://pdm.fming.dev)
 [![Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](https://github.com/astral-sh/ruff)
 
 # NVDA Addon Voicemeeter
@@ -15,27 +15,27 @@ This addon can be installed through the Add-on store, `Install from external sou
 
 ## Default Keybinds
 
-### Channel Controllers
+### Controllers
 
-- `NVDA+alt+s`: Enable strip controller
-- `NVDA+alt+b`: Enable bus controller.
+- `NVDA+alt+s`: Enable strip mode
+- `NVDA+alt+b`: Enable bus mode.
 
-- `NVDA+alt+1`: Enable controller index 1 (strip|bus)
-- `NVDA+alt+2`: Enable controller index 2 (strip|bus)
-- `NVDA+alt+3`: Enable controller index 3 (strip|bus)
-- `NVDA+alt+4`: Enable controller index 4 (strip|bus)
-- `NVDA+alt+5`: Enable controller index 5 (strip|bus)
-- `NVDA+alt+6`: Enable controller index 6 (strip|bus)
-- `NVDA+alt+7`: Enable controller index 7 (strip|bus)
-- `NVDA+alt+8`: Enable controller index 8 (strip|bus)
+- `NVDA+alt+1`: Enable controller for channel 1 (strip|bus)
+- `NVDA+alt+2`: Enable controller for channel 2 (strip|bus)
+- `NVDA+alt+3`: Enable controller for channel 3 (strip|bus)
+- `NVDA+alt+4`: Enable controller for channel 4 (strip|bus)
+- `NVDA+alt+5`: Enable controller for channel 5 (strip|bus)
+- `NVDA+alt+6`: Enable controller for channel 6 (strip|bus)
+- `NVDA+alt+7`: Enable controller for channel 7 (strip|bus)
+- `NVDA+alt+8`: Enable controller for channel 8 (strip|bus)
 
-### Slider Controllers
+### Slider Modes
 
-- `NVDA+alt+g`: Enable gain slider controller.
-- `NVDA+alt+c`: Enable comp slider controller.
-- `NVDA+alt+t`: Enable gate slider controller.
-- `NVDA+alt+d`: Enable denoiser slider controller.
-- `NVDA+alt+a`: Enable audibility slider controller.
+- `NVDA+alt+g`: Enable gain slider mode.
+- `NVDA+alt+c`: Enable comp slider mode.
+- `NVDA+alt+t`: Enable gate slider mode.
+- `NVDA+alt+d`: Enable denoiser slider mode.
+- `NVDA+alt+a`: Enable audibility slider mode.
 
 ### Sliders
 
@@ -53,24 +53,22 @@ This addon can be installed through the Add-on store, `Install from external sou
 - `NVDA+shift+m`: Mute
 - `NVDA+shift+c`: MC
 - `NVDA+shift+k`: Karaoke
-- `NVDA+shift+n`: Next Bus Mode
-- `NVDA+shift+p`: Previous Bus Mode
 
 ### Bus Assignments (A1-A5|B1-B3)
 
-- `NVDA+shift+1`: Toggle BUS assignment 1
-- `NVDA+shift+2`: Toggle BUS assignment 2
-- `NVDA+shift+3`: Toggle BUS assignment 3
-- `NVDA+shift+4`: Toggle BUS assignment 4
-- `NVDA+shift+5`: Toggle BUS assignment 5
-- `NVDA+shift+6`: Toggle BUS assignment 6
-- `NVDA+shift+7`: Toggle BUS assignment 7
-- `NVDA+shift+8`: Toggle BUS assignment 8
+- `NVDA+shift+1`: Toggle BUS assignment 1 for a strip
+- `NVDA+shift+2`: Toggle BUS assignment 2 for a strip
+- `NVDA+shift+3`: Toggle BUS assignment 3 for a strip
+- `NVDA+shift+4`: Toggle BUS assignment 4 for a strip
+- `NVDA+shift+5`: Toggle BUS assignment 5 for a strip
+- `NVDA+shift+6`: Toggle BUS assignment 6 for a strip
+- `NVDA+shift+7`: Toggle BUS assignment 7 for a strip
+- `NVDA+shift+8`: Toggle BUS assignment 8 for a strip
 
 ### Announcements
 
 - `NVDA+shift+q`: Announce current controller.
-- `NVDA+shift+a`: Announce Voicemeeter kind and version.
+- `NVDA+shift+a`: Announce Voicemeeter kind.
 
 ## Configuration
 
@@ -98,13 +96,10 @@ example:
     "NVDA+alt+a": "audibility_mode",
     "NVDA+shift+q": "announce_controller",
     "NVDA+shift+z": "announce_voicemeeter_version",
-    "NVDA+shift+o": "rotate_mono",
     "NVDA+shift+s": "toggle_solo",
     "NVDA+shift+m": "toggle_mute",
     "NVDA+shift+c": "toggle_mc",
-    "NVDA+shift+k": "rotate_karaoke",
-    "NVDA+shift+n": "rotate_bus_mode_next",
-    "NVDA+shift+p": "rotate_bus_mode_previous",
+    "NVDA+shift+k": "karaoke",
     "NVDA+shift+upArrow": "slider_increase_by_point_one",
     "NVDA+shift+downArrow": "slider_decrease_by_point_one",
     "NVDA+shift+alt+upArrow": "slider_increase_by_one",

Taskfile.yml

@@ -1,41 +0,0 @@
-version: '3'
-
-vars:
-  SHELL: pwsh
-
-tasks:
-  default:
-    desc: Build the addon
-    cmds:
-      - task: build
-
-  copy:
-    desc: Copy sources files to scratchpad directory
-    platforms: [windows]
-    vars:
-      SOURCE: addon\globalPlugins\voicemeeter
-      DEST: ${env:appdata}\nvda\scratchpad\globalPlugins\voicemeeter\
-    cmds:
-      - "{{.SHELL}} -Command 'robocopy {{.SOURCE}} {{.DEST}} /NDL'"
-    ignore_error: true
-
-  build:
-    desc: Build the addon
-    platforms: [windows]
-    cmds:
-      - '{{.SHELL}} -Command "pdm run scons"'
-
-  bump:
-    desc: Bump the version number
-    preconditions:
-      - sh: '{{.SHELL}} -c "if (Get-Command bump) { exit 0 } else { exit 1 }"'
-        msg: "The 'bump' command is not available. Run `go install github.com/onyx-and-iris/bump/cmd/bump@latest`."
-    cmds:
-      - |
-        {{if eq .CLI_ARGS "show"}}
-          {{.SHELL}} -c "bump show -f buildVars.py -p \"'addon_version': '(\d+\.\d+\.\d+)'\""
-          {{.SHELL}} -c "bump show -f pyproject.toml -p \"version = .(\d+\.\d+\.\d+).\""
-        {{else}}
-          {{.SHELL}} -c "bump {{.CLI_ARGS}} -w -f buildVars.py -p \"'addon_version': '(\d+\.\d+\.\d+)'\""
-          {{.SHELL}} -c "bump {{.CLI_ARGS}} -w -f pyproject.toml -p \"version = .(\d+\.\d+\.\d+).\""
-        {{end}}

addon/globalPlugins/voicemeeter/binds.py

@@ -2,7 +2,7 @@ import ctypes as ct
 from ctypes.wintypes import CHAR, FLOAT, LONG
 
 from .cdll import libc
-from .error import VMAddonCAPIError
+from .error import VMCAPIError
 
 
 class Binds:
@@ -38,32 +38,8 @@ class Binds:
     bind_set_parameter_float.restype = LONG
     bind_set_parameter_float.argtypes = [ct.POINTER(CHAR), FLOAT]
 
-    def _call(self, fn, *args, ok=(0,)):
+    def call(self, fn, *args, ok=(0,)):
         retval = fn(*args)
         if retval not in ok:
-            raise VMAddonCAPIError(fn.__name__, retval)
+            raise VMCAPIError(fn.__name__, retval)
         return retval
-
-    def login(self):
-        return self._call(self.bind_login, ok=(0, 1))
-
-    def logout(self):
-        return self._call(self.bind_logout)
-
-    def run_voicemeeter(self, kind_val):
-        return self._call(self.bind_run_voicemeeter, kind_val)
-
-    def get_voicemeeter_type(self, c_type):
-        return self._call(self.bind_get_voicemeeter_type, ct.byref(c_type))
-
-    def get_voicemeeter_version(self, ver):
-        return self._call(self.bind_get_voicemeeter_version, ct.byref(ver))
-
-    def is_parameters_dirty(self):
-        return self._call(self.bind_is_parameters_dirty, ok=(0, 1))
-
-    def get_parameter_float(self, param, buf):
-        return self._call(self.bind_get_parameter_float, param, ct.byref(buf))
-
-    def set_parameter_float(self, param, val):
-        return self._call(self.bind_set_parameter_float, param, val)

addon/globalPlugins/voicemeeter/cdll.py

@@ -1,22 +1,16 @@
 import ctypes as ct
 import platform
+import winreg
 from pathlib import Path
 
-from .error import VMAddonError
-
-try:
-    import winreg
-except ImportError as e:
-    ERR_MSG = 'winreg module not found, only Windows OS supported'
-    raise VMAddonError(ERR_MSG) from e
-
-# Defense against edge cases where winreg imports but we're not on Windows
-if platform.system() != 'Windows':
-    ERR_MSG = f'Unsupported OS: {platform.system()}, only Windows OS supported'
-    raise VMAddonError(ERR_MSG)
+from .error import VMError
 
 BITS = 64 if ct.sizeof(ct.c_voidp) == 8 else 32
 
+if platform.system() != 'Windows':
+    raise VMError('Only Windows OS supported')
+
+
 VM_KEY = 'VB:Voicemeeter {17359A74-1236-5467}'
 REG_KEY = '\\'.join(
     filter(
@@ -41,14 +35,12 @@ def get_vmpath():
 try:
     vm_parent = Path(get_vmpath()).parent
 except FileNotFoundError as e:
-    ERR_MSG = 'Voicemeeter installation not found in registry'
-    raise VMAddonError(ERR_MSG) from e
+    raise VMError('Unable to fetch DLL path from the registry') from e
 
 DLL_NAME = f'VoicemeeterRemote{"64" if BITS == 64 else ""}.dll'
 
 dll_path = vm_parent.joinpath(DLL_NAME)
 if not dll_path.is_file():
-    ERR_MSG = f'Could not find {dll_path}'
-    raise VMAddonError(ERR_MSG)
+    raise VMError(f'Could not find {dll_path}')
 
 libc = ct.WinDLL(str(dll_path))

addon/globalPlugins/voicemeeter/commands.py

@@ -1,7 +1,7 @@
 import ui
 from logHandler import log
 
-from . import context, util
+from . import context
 
 
 class CommandsMixin:
@@ -56,93 +56,37 @@ class CommandsMixin:
     def script_audibility_mode(self, _):
         self.__set_slider_mode('audibility')
 
-    ### STRIP|BUS PARAMETERS ###
+    ### BOOLEAN PARAMETERS ###
 
-    def script_rotate_mono(self, _):
-        if isinstance(self.controller.ctx.strategy, context.StripStrategy):
-            val = not self.controller.ctx.get_bool('mono')
-            self.controller.ctx.set_bool('mono', val)
-            ui.message('on' if val else 'off')
-        else:
-            opts = ['off', 'on', 'stereo reverse']
-            val = self.controller.ctx.get_int('mono')
-            new_val = (val + 1) % len(opts)
-            self.controller.ctx.set_int('mono', new_val)
-            ui.message(opts[new_val])
+    def script_toggle_mono(self, _):
+        val = not self.controller.ctx.get_bool('mono')
+        self.controller.ctx.set_bool('mono', val)
+        ui.message('on' if val else 'off')
+
+    def script_toggle_solo(self, _):
+        val = not self.controller.ctx.get_bool('solo')
+        self.controller.ctx.set_bool('solo', val)
+        ui.message('on' if val else 'off')
 
     def script_toggle_mute(self, _):
         val = not self.controller.ctx.get_bool('mute')
         self.controller.ctx.set_bool('mute', val)
         ui.message('on' if val else 'off')
 
-    ### STRIP PARAMETERS ###
-
-    def script_toggle_solo(self, _):
-        if not isinstance(self.controller.ctx.strategy, context.StripStrategy):
-            ui.message('Solo only available for strips')
-            return
-
-        val = not self.controller.ctx.get_bool('solo')
-        self.controller.ctx.set_bool('solo', val)
-        ui.message('on' if val else 'off')
-
     def script_toggle_mc(self, _):
-        if not isinstance(self.controller.ctx.strategy, context.StripStrategy):
-            ui.message('MC only available for strips')
-            return
-
-        valid_indices_zero_based = [self.kind.phys_in]
-        match self.kind.name:
-            case 'potato':
-                valid_indices_zero_based.append(self.kind.phys_in + self.kind.virt_in - 1)
-
-        if self.controller.ctx.index not in valid_indices_zero_based:
-            valid_indices_display = [i + 1 for i in valid_indices_zero_based]
-            if len(valid_indices_display) == 1:
-                ui.message(f'MC only available for strip {valid_indices_display[0]} for Voicemeeter {self.kind}')
-            else:
-                ui.message(
-                    f'MC only available for strips {valid_indices_display[0]} and {valid_indices_display[1]} for Voicemeeter {self.kind}'
-                )
-            return
-
         val = not self.controller.ctx.get_bool('mc')
         self.controller.ctx.set_bool('mc', val)
         ui.message('on' if val else 'off')
 
-    def script_rotate_karaoke(self, _):
-        if not isinstance(self.controller.ctx.strategy, context.StripStrategy):
-            ui.message('Karaoke mode only available for strips')
-            return
-
-        if self.kind.name not in ['banana', 'potato']:
-            ui.message(f'Karaoke mode not available for Voicemeeter {self.kind}')
-            return
-
-        valid_index_zero_based = None
-        match self.kind.name:
-            case 'banana':
-                valid_index_zero_based = self.kind.phys_in + self.kind.virt_in - 1
-            case 'potato':
-                valid_index_zero_based = self.kind.phys_in + self.kind.virt_in - 2
-
-        if self.controller.ctx.index != valid_index_zero_based:
-            ui.message(
-                f'Karaoke mode only available for strip {valid_index_zero_based + 1} for Voicemeeter {self.kind}'
-            )
-            return
-
+    def script_karaoke(self, _):
         opts = ['off', 'k m', 'k 1', 'k 2', 'k v']
-        val = self.controller.ctx.get_int('karaoke')
-        new_val = (val + 1) % len(opts)
-        self.controller.ctx.set_int('karaoke', new_val)
-        ui.message(opts[new_val])
+        val = self.controller.ctx.get_int('karaoke') + 1
+        if val == len(opts):
+            val = 0
+        self.controller.ctx.set_int('karaoke', val)
+        ui.message(opts[val])
 
     def script_bus_assignment(self, gesture):
-        if not isinstance(self.controller.ctx.strategy, context.StripStrategy):
-            ui.message('Bus assignment only available for strips')
-            return
-
         proposed = int(gesture.displayName[-1])
         if proposed - 1 < self.kind.phys_out:
             output = f'A{proposed}'
@@ -152,76 +96,34 @@ class CommandsMixin:
         self.controller.ctx.set_bool(output, val)
         ui.message('on' if val else 'off')
 
-    ### BUS PARAMETERS ###
-
-    def script_rotate_bus_mode_next(self, _):
-        if not isinstance(self.controller.ctx.strategy, context.BusStrategy):
-            ui.message('Bus mode only available for buses')
-            return
-
-        opts = util._get_bus_mode_opts(self.kind.name)
-        for mode in opts:
-            if self.controller.ctx.get_bool(f'mode.{mode}'):
-                current_mode = mode
-                log.info(f'INFO - bus {self.controller.ctx.index} current mode {current_mode}')
-                break
-        else:
-            log.warning(f'WARNING - no bus mode found for bus {self.controller.ctx.index}')
-            return
-
-        new_val = (opts.index(current_mode) + 1) % len(opts)
-        log.info(f'INFO - bus {self.controller.ctx.index} mode {opts[new_val]}')
-        self.controller.ctx.set_bool(f'mode.{opts[new_val]}', True)
-        ui.message(util._get_bus_mode_readable_name(opts[new_val]))
-
-    def script_rotate_bus_mode_previous(self, _):
-        if not isinstance(self.controller.ctx.strategy, context.BusStrategy):
-            ui.message('Bus mode only available for buses')
-            return
-
-        opts = util._get_bus_mode_opts(self.kind.name)
-        for mode in opts:
-            if self.controller.ctx.get_bool(f'mode.{mode}'):
-                current_mode = mode
-                log.info(f'INFO - bus {self.controller.ctx.index} current mode {current_mode}')
-                break
-        else:
-            log.warning(f'WARNING - no bus mode found for bus {self.controller.ctx.index}')
-            return
-
-        new_val = (opts.index(current_mode) - 1) % len(opts)
-        log.info(f'INFO - bus {self.controller.ctx.index} mode {opts[new_val]}')
-        self.controller.ctx.set_bool(f'mode.{opts[new_val]}', True)
-        ui.message(util._get_bus_mode_readable_name(opts[new_val]))
-
     ### CONTROL SLIDERS ###
 
-    def script_slider_increase_by_point_one(self, _):
+    def script_slider_increase_by_point_one(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) + 0.1
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))
 
-    def script_slider_decrease_by_point_one(self, _):
+    def script_slider_decrease_by_point_one(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) - 0.1
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))
 
-    def script_slider_increase_by_one(self, _):
+    def script_slider_increase_by_one(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) + 1
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))
 
-    def script_slider_decrease_by_one(self, _):
+    def script_slider_decrease_by_one(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) - 1
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))
 
-    def script_slider_increase_by_three(self, _):
+    def script_slider_increase_by_three(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) + 3
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))
 
-    def script_slider_decrease_by_three(self, _):
+    def script_slider_decrease_by_three(self, gesture):
         val = self.controller.ctx.get_float(self.controller.ctx.slider_mode) - 3
         self.controller.ctx.set_float(self.controller.ctx.slider_mode, val)
         ui.message(str(round(val, 1)))

addon/globalPlugins/voicemeeter/context.py

@@ -28,22 +28,22 @@ class Strategy(ABC):
         self._slider_mode = val
 
     def get_bool(self, param: str) -> bool:
-        return self._controller.get(f'{self.identifier}.{param}') == 1
+        return self._controller._get(f'{self.identifier}.{param}') == 1
 
     def set_bool(self, param: str, val: bool):
-        self._controller.set(f'{self.identifier}.{param}', 1 if val else 0)
+        self._controller._set(f'{self.identifier}.{param}', 1 if val else 0)
 
     def get_float(self, param: str) -> float:
-        return round(self._controller.get(f'{self.identifier}.{param}'), 1)
+        return round(self._controller._get(f'{self.identifier}.{param}'), 1)
 
     def set_float(self, param: str, val: float):
-        self._controller.set(f'{self.identifier}.{param}', val)
+        self._controller._set(f'{self.identifier}.{param}', val)
 
     def get_int(self, param: str) -> int:
-        return int(self._controller.get(f'{self.identifier}.{param}'))
+        return int(self._controller._get(f'{self.identifier}.{param}'))
 
     def set_int(self, param: str, val: int):
-        self._controller.set(f'{self.identifier}.{param}', val)
+        self._controller._set(f'{self.identifier}.{param}', val)
 
 
 class StripStrategy(Strategy):
@@ -92,6 +92,20 @@ class Context:
     def slider_mode(self, val):
         self._strategy._slider_mode = val
 
-    def __getattr__(self, name):
-        """Delegate method calls to the strategy object."""
-        return getattr(self._strategy, name)
+    def get_bool(self, *args) -> bool:
+        return self._strategy.get_bool(*args)
+
+    def set_bool(self, *args):
+        self._strategy.set_bool(*args)
+
+    def get_float(self, *args) -> float:
+        return self._strategy.get_float(*args)
+
+    def set_float(self, *args):
+        self._strategy.set_float(*args)
+
+    def get_int(self, *args) -> int:
+        return self._strategy.get_int(*args)
+
+    def set_int(self, *args):
+        self._strategy.set_int(*args)

addon/globalPlugins/voicemeeter/controller.py

@@ -9,31 +9,30 @@ from .context import Context, StripStrategy
 from .kinds import KindId
 
 
-class Controller:
+class Controller(Binds):
     def __init__(self):
-        self._binds = Binds()
         self.ctx = Context(StripStrategy(self, 0))
         self.bits = config.get('bits', BITS)
 
     def login(self):
-        retval = self._binds.login()
+        retval = self.call(self.bind_login, ok=(0, 1))
         log.info('INFO - logged into Voicemeeter Remote API')
         return retval
 
     def logout(self):
-        self._binds.logout()
+        self.call(self.bind_logout)
         log.info('NFO - logged out of Voicemeeter Remote API')
 
     @property
     def kind_id(self):
         c_type = ct.c_long()
-        self._binds.get_voicemeeter_type(c_type)
+        self.call(self.bind_get_voicemeeter_type, ct.byref(c_type))
         return KindId(c_type.value).name.lower()
 
     @property
     def version(self):
         ver = ct.c_long()
-        self._binds.get_voicemeeter_version(ver)
+        self.call(self.bind_get_voicemeeter_version, ct.byref(ver))
         return '{}.{}.{}.{}'.format(
             (ver.value & 0xFF000000) >> 24,
             (ver.value & 0x00FF0000) >> 16,
@@ -45,17 +44,17 @@ class Controller:
         val = kind_id.value
         if self.bits == 64:
             val += 3
-        self._binds.run_voicemeeter(val)
+        self.call(self.bind_run_voicemeeter, val)
 
     def __clear(self):
-        while self._binds.is_parameters_dirty() == 1:
+        while self.call(self.bind_is_parameters_dirty, ok=(0, 1)) == 1:
             pass
 
-    def get(self, param):
+    def _get(self, param):
         self.__clear()
         buf = ct.c_float()
-        self._binds.get_parameter_float(param.encode(), buf)
+        self.call(self.bind_get_parameter_float, param.encode(), ct.byref(buf))
         return buf.value
 
-    def set(self, param, val):
-        self._binds.set_parameter_float(param.encode(), ct.c_float(float(val)))
+    def _set(self, param, val):
+        self.call(self.bind_set_parameter_float, param.encode(), ct.c_float(float(val)))

addon/globalPlugins/voicemeeter/error.py

@@ -1,9 +1,9 @@
-class VMAddonError(Exception):
-    """Base voicemeeter add-on exception class"""
+class VMError(Exception):
+    """Base voicemeeterlib exception class"""
 
 
-class VMAddonCAPIError(VMAddonError):
-    """Exception raised when the Voicemeeter C-API returns an error code"""
+class VMCAPIError(VMError):
+    """Exception raised when the C-API returns an error code"""
 
     def __init__(self, fn_name, code):
         self.fn_name = fn_name

addon/globalPlugins/voicemeeter/kinds.py

@@ -1,7 +1,7 @@
 from dataclasses import dataclass
 from enum import Enum, unique
 
-from .error import VMAddonError
+from .error import VMError
 
 
 @unique
@@ -78,8 +78,7 @@ def kind_factory(kind_id):
     elif kind_id == 'potato':
         _kind_map = PotatoMap
     else:
-        ERR_MSG = f'Unknown Voicemeeter kind {kind_id}'
-        raise ValueError(ERR_MSG)
+        raise ValueError(f'Unknown Voicemeeter kind {kind_id}')
     return _kind_map(name=kind_id)
 
 
@@ -88,5 +87,5 @@ def request_kind_map(kind_id):
     try:
         KIND_obj = kind_factory(kind_id)
     except ValueError as e:
-        raise VMAddonError(str(e)) from e
+        raise VMError(str(e)) from e
     return KIND_obj

addon/globalPlugins/voicemeeter/util.py

@@ -26,13 +26,11 @@ def _make_gestures(kind_id):
         'kb:NVDA+alt+a': 'audibility_mode',
         'kb:NVDA+shift+q': 'announce_controller',
         'kb:NVDA+shift+v': 'announce_voicemeeter_version',
-        'kb:NVDA+shift+o': 'rotate_mono',
+        'kb:NVDA+shift+o': 'toggle_mono',
         'kb:NVDA+shift+s': 'toggle_solo',
         'kb:NVDA+shift+m': 'toggle_mute',
         'kb:NVDA+shift+c': 'toggle_mc',
-        'kb:NVDA+shift+k': 'rotate_karaoke',
-        'kb:NVDA+shift+n': 'rotate_bus_mode_next',
-        'kb:NVDA+shift+p': 'rotate_bus_mode_previous',
+        'kb:NVDA+shift+k': 'karaoke',
         'kb:NVDA+shift+upArrow': 'slider_increase_by_point_one',
         'kb:NVDA+shift+downArrow': 'slider_decrease_by_point_one',
         'kb:NVDA+shift+alt+upArrow': 'slider_increase_by_one',
@@ -51,47 +49,3 @@ def _make_gestures(kind_id):
         defaults = {k: v for k, v in defaults.items() if v not in matching_values}
         return {**defaults, **overrides}
     return defaults
-
-
-_bus_mode_map = {
-    'normal': 'Normal',
-    'amix': 'Mix Down A',
-    'bmix': 'Mix Down B',
-    'repeat': 'Stereo Repeat',
-    'composite': 'Composite',
-    'tvmix': 'Up Mix TV',
-    'upmix21': 'Up Mix 2.1',
-    'upmix41': 'Up Mix 4.1',
-    'upmix61': 'Up Mix 6.1',
-    'centeronly': 'Center Only',
-    'lfeonly': 'Low Frequency Effect Only',
-    'rearonly': 'Rear Only',
-}
-
-
-def _get_bus_mode_opts(kind_id):
-    if kind_id == 'basic':
-        return [
-            'normal',
-            'amix',
-            'repeat',
-            'composite',
-        ]
-    return [
-        'normal',
-        'amix',
-        'bmix',
-        'repeat',
-        'composite',
-        'tvmix',
-        'upmix21',
-        'upmix41',
-        'upmix61',
-        'centeronly',
-        'lfeonly',
-        'rearonly',
-    ]
-
-
-def _get_bus_mode_readable_name(mode):
-    return _bus_mode_map.get(mode, mode)

build.ps1

@@ -0,0 +1,25 @@
+param (
+    [switch]$build
+)
+
+function Copy-FilesToScratchpad {
+    $source = Join-Path $PSScriptRoot "addon" "globalPlugins" "voicemeeter"
+    $target = Join-Path $env:appdata "nvda" "scratchpad" "globalPlugins" "voicemeeter"
+    Robocopy $source $target | Out-Null
+}
+
+function Build-Addon {
+    "Building add-on" | Write-Host
+    scons
+}
+
+function Main {
+    "Copying updated files to Scratchpad" | Write-Host
+    Copy-FilesToScratchpad
+
+    if ($build) {
+        Build-Addon
+    }
+}
+
+if ($MyInvocation.InvocationName -ne '.') { Main }

buildVars.py

@@ -28,7 +28,7 @@ addon_info = {
 The add-on requires Voicemeeter to be installed."""
     ),
     # version
-    'addon_version': '1.3.0',
+    'addon_version': '1.1.0',
     # Author(s)
     'addon_author': 'onyx-and-iris <code@onyxandiris.online>',
     # URL for the add-on documentation support

pyproject.toml

@@ -1,22 +1,27 @@
 [project]
 name = "nvda-addon-voicemeeter"
-version = "1.3.0"
+version = "1.1.0"
 description = "A GUI-less NVDA Addon for Voicemeeter using the Remote API"
-authors = [{ name = "Onyx and Iris", email = "code@onyxandiris.online" }]
+authors = [
+    {name = "Onyx and Iris", email = "code@onyxandiris.online"},
+]
 dependencies = []
 requires-python = "==3.11.*"
 readme = "README.md"
-license = { text = "MIT" }
+license = {text = "MIT"}
 
 [dependency-groups]
-build = ["scons>=4.8.1", "markdown>=3.7"]
+build = [
+    "scons>=4.8.1",
+    "markdown>=3.7",
+]
 
 [tool.pdm]
 distribution = false
 
 [tool.pdm.scripts]
-copy = "task copy"
-release = "task build"
+copy = "pwsh build.ps1"
+build = "pwsh build.ps1 -build"
 
 [tool.ruff]
 exclude = [
@@ -51,11 +56,9 @@ target-version = "py311"
 
 [tool.ruff.lint]
 # Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
-# Enable flake8-errmsg (EM) warnings.
-# Enable flake8-bugbear (B) warnings.
 # Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
 # McCabe complexity (`C901`) by default.
-select = ["E4", "E7", "E9", "EM", "F", "B"]
+select = ["E4", "E7", "E9", "F"]
 ignore = []
 
 # Allow fix for all enabled rules (when `--fix`) is provided.
@@ -97,4 +100,7 @@ docstring-code-line-length = "dynamic"
 max-complexity = 10
 
 [tool.ruff.lint.per-file-ignores]
-"__init__.py" = ["E402", "F401"]
+"__init__.py" = [
+    "E402",
+    "F401",
+]