@openmixer/plugin-qualify 0.9.0 → 0.11.0

package/tools/benchmark.py

@@ -1086,6 +1086,14 @@ OPTIONS = "http://lv2plug.in/ns/ext/options#"
 BUFSZ = "http://lv2plug.in/ns/ext/buf-size#"
 PARAMS = "http://lv2plug.in/ns/ext/parameters#"
 WORKER = "http://lv2plug.in/ns/ext/worker#"
+# The refusal's code: the LV2 worker spec does not make `work` or `work_response` optional, so a plugin
+# whose interface leaves either NULL is not hostable (end_run may be NULL).
+WORKER_NONCONFORMING = "hosting.worker.nonconforming"
+
+
+class WorkerInterfaceError(Exception):
+    """The plugin's LV2_Worker_Interface is incomplete; the message starts with WORKER_NONCONFORMING."""
+
 
 
 LV2_WORKER_SUCCESS = 0
@@ -1095,7 +1103,7 @@ LV2_WORKER_ERR_NO_SPACE = 3
 class WorkerQueue:
     """The host half of `worker:schedule`: jobs are QUEUED, and run between blocks, off `run()`.
 
-    Spec 2a (`2026-09-25-plugin-qualify.md`): a schedule that accepts a job and
+    A schedule that accepts a job and
     drops it measures an UNBUILT plugin — the Hex Chain amp read 0.9 us/sample that way against 5.4
     built. After every block the host calls `pump(interface)`: the plugin's `work()` for each
     queued job, then `work_response()` with whatever it answered, then `end_run()` once every answer
@@ -1261,6 +1269,10 @@ class _Host:
             handle = ctypes.c_void_p(instance.get_handle())
         except (AttributeError, TypeError, ValueError):
             return None
+        # calling a NULL function pointer through ctypes kills the process: refuse before anything is called
+        for name in ("work", "work_response"):
+            if not getattr(iface, name):
+                raise WorkerInterfaceError(f"{WORKER_NONCONFORMING}: LV2_Worker_Interface.{name} is NULL")
 
         def keep(buf):
             return ctypes.create_string_buffer(bytes(buf), len(buf))

package/tools/private-refs-check.sh

@@ -0,0 +1,90 @@
+#!/usr/bin/env bash
+# SPDX-License-Identifier: GPL-3.0-or-later
+# Copyright (C) 2026 Pau Aliagas <linuxnow@gmail.com>
+#
+# private-refs-check.sh — this repository is public, so its tracked files state facts and never point
+# into a private repository: no private repository name, no private issue or pull request number, no
+# private branch name or file path, no name of a private specification file. Say what the value is and
+# why instead, and cite only public repositories (omx-dsp, omx-contract, omx-plugins, omx-clap-core, plugin-qualify).
+# Product-name mentions of OpenMixer, and values such as the org.openmixer.* CLAP ids, the
+# urn:openmixer:* LV2 URIs and the @openmixer npm scope, are not pointers and are not matched.
+#
+#   tools/private-refs-check.sh              the check, over every tracked file (generated ones included)
+#   tools/private-refs-check.sh --self-test  each sabotage, in every kind of file, must go red
+#
+# ALLOW lists the tracked files exempt from the check, each with the reason; nothing else is.
+set -euo pipefail
+HERE="$(cd "$(dirname "$0")/.." && pwd)"
+
+# <file> — <why it may name them>
+ALLOW=(
+  "tools/private-refs-check.sh"   # it spells the patterns it looks for
+)
+EXTRA_PATHSPEC=()                 # the self-test sets a bad one, to prove a git grep error is red
+
+# Extended regexps, matched case-insensitively. A new kind of pointer is one line here.
+PATTERNS=(
+  'FreeMixer/openmixer([^A-Za-z0-9_-]|$)'             # the private repository
+  'openmixer-ops'                                     # the private operations repository
+  'openmixer ?#[0-9]'                                 # one of its issues or pull requests
+  '[A-Za-z0-9_.]-dev#[0-9]'                           # an issue of a private *-dev repository
+  'integration/[A-Za-z0-9]'                           # a private integration branch
+  'packages/[a-z][a-z0-9-]+'                          # a path into the private repository's packages
+  '\.vue([^A-Za-z0-9_]|$)'                            # its web UI components
+  "openmixer's spec"                                  # its specification files
+  'docs/design/'                                      # its design documents
+  'harness/'                                          # its test harness
+  '20[0-9]{2}-[0-9]{2}-[0-9]{2}-[a-z][a-z0-9-]+\.md'  # a dated specification file name
+  'spec 20[0-9]{2}-[0-9]{2}-[0-9]{2}'                 # a citation of one
+)
+
+check() { # <root>
+  local root="$1" fail=0 args=() p a hits
+  for p in "${PATTERNS[@]}"; do args+=(-e "$p"); done
+  local excl=()
+  for a in "${ALLOW[@]}"; do excl+=(":(exclude)${a%% *}"); done
+  [ -n "$(git -C "$root" ls-files)" ] || { echo "private-refs-check: git lists no file" >&2; return 1; }
+  # git grep exits 1 for no match and 128 or more for an error; an error must not read as "no hits"
+  hits=$(git -C "$root" grep -nIiE "${args[@]}" -- . "${excl[@]}" "${EXTRA_PATHSPEC[@]}") || rc=$?
+  case "${rc:-0}" in 0|1) ;; *) echo "private-refs-check: git grep failed (exit $rc)" >&2; return 1 ;; esac
+  hits=$(printf '%s' "$hits" | cut -c1-200)
+  [ -z "$hits" ] || { printf '%s\n' "$hits" >&2; fail=1; }
+  [ "$fail" = 0 ] || { echo "private-refs-check: the lines above point into private repositories; state the fact instead" >&2; return 1; }
+  echo "private-refs-check: no tracked file points into a private repository"
+}
+
+if [ "${1:-}" = --self-test ]; then
+  scratch="$(mktemp -d)"; trap 'rm -rf "$scratch"' EXIT
+  fail=0
+  # a clean tree with one file of each kind the repository tracks; every sabotage writes one private pointer into one
+  mk() { rm -rf "$scratch/t"; mkdir -p "$scratch/t/src" "$scratch/t/tools" "$scratch/t/docs"
+    git -C "$scratch" init -q t
+    printf 'rdfs:comment "The OpenMixer console x." ;\nlv2:binary <x.so> ; # urn:openmixer:x\n' > "$scratch/t/src/x.ttl"
+    printf '// org.openmixer.x, @openmixer/x\n' > "$scratch/t/tools/gen.mjs"
+    printf '# Docs\n\nOpenMixer plugins.\n' > "$scratch/t/docs/a.md"
+    printf '/* x */\n' > "$scratch/t/src/x.h"; printf '#!/bin/sh\n# x\n' > "$scratch/t/tools/x.sh"; }
+  sab() { # <name> <file> <line>
+    mk; printf '%s\n' "$3" >> "$scratch/t/$2"; git -C "$scratch/t" add -A
+    if check "$scratch/t" >/dev/null 2>&1; then echo "SABOTAGE $1: STAYED GREEN"; fail=1; else echo "SABOTAGE $1: red"; fi; }
+  mk; git -C "$scratch/t" add -A
+  check "$scratch/t" >/dev/null || { echo "self-test: the clean tree is not green" >&2; exit 1; }
+  sab repo-in-header src/x.h                         '/* as FreeMixer/openmixer draws it */'
+  sab branch-in-sh   tools/x.sh                      '# drawn on integration/lean-engine'
+  sab issue-in-md    docs/a.md                       'see openmixer#1580'
+  sab path-in-md     docs/a.md                       'packages/web-ui/Foo.vue'
+  sab default-path   tools/x.sh                      'plugin=${P:-$root/../openmixer/packages/omx-plugins/bin/omx-delay.clap}'
+  sab ops-in-mjs     tools/gen.mjs                   '// see openmixer-ops'
+  sab dev-in-mjs     tools/gen.mjs                   '// omx-dsp-dev#32'
+  sab spec-in-mjs    tools/gen.mjs                   "// openmixer's spec 2026-10-09-plugin-from-contract.md"
+  sab ttl            src/x.ttl       'rdfs:comment "see FreeMixer/openmixer#12" ;'
+  sab header         src/x.h         '/* packages/core/src/x.ts */'
+  sab design-dir     docs/a.md                       'the notes in docs/design/specs'
+  sab harness-dir    tools/gen.mjs                   '// harness/dsp-check.sh'
+  mk; git -C "$scratch/t" add -A
+  if ( EXTRA_PATHSPEC=(":(bogus)x"); check "$scratch/t" >/dev/null 2>&1 ); then echo "SABOTAGE git-grep-error: STAYED GREEN"; fail=1; else echo "SABOTAGE git-grep-error: red"; fi
+  [ "$fail" = 0 ] || exit 1
+  check "$HERE" >/dev/null || { echo "self-test: the repository is not green" >&2; exit 1; }
+  echo "private-refs-check self-test: every sabotage red"
+  exit 0
+fi
+check "$HERE"