fix: anchor action discovery at Bias_Set

Author: BB-84C

config/parameters.yaml

@@ -4,15 +4,15 @@ defaults:
   ramp_default_interval_s: 0.05
 meta:
   generated_by: scripts/generate_parameters_manifest.py
-  generated_at_utc: '2026-02-24T05:05:10Z'
+  generated_at_utc: '2026-02-24T05:39:57Z'
   source_package: nanonis-spm/1.0.9
-  methods_seen: 684
+  methods_seen: 667
   get_set_commands_imported: 551
-  action_commands_imported: 133
+  action_commands_imported: 116
   parameters_emitted: 327
-  actions_emitted: 133
+  actions_emitted: 116
   with_description_count: 326
-  actions_with_description_count: 117
+  actions_with_description_count: 116
   writable_count: 257
 parameters:
   aprfgen_freq:
@@ -9316,241 +9316,3 @@ actions:
       description: ZSpectr.Stop Stops the current Z Spectroscopy measurement.
     safety:
       mode: guarded
-  close:
-    action_cmd:
-      command: close
-      args: {}
-      arg_types: {}
-      description: ''
-    safety:
-      mode: guarded
-  correctType:
-    action_cmd:
-      command: correctType
-      args:
-        BodyType: 0.0
-        Body: 0.0
-      arg_types:
-        BodyType: float
-        Body: float
-      description: ''
-    safety:
-      mode: guarded
-  decodeArray:
-    action_cmd:
-      command: decodeArray
-      args:
-        response: 0.0
-        index: 0.0
-        numOfElements: 0.0
-        responseType: 0.0
-      arg_types:
-        response: float
-        index: float
-        numOfElements: float
-        responseType: float
-      description: ''
-    safety:
-      mode: guarded
-  decodeArrayPrepended:
-    action_cmd:
-      command: decodeArrayPrepended
-      args:
-        response: 0.0
-        index: 0.0
-        numOfElements: 0.0
-        responseType: 0.0
-      arg_types:
-        response: float
-        index: float
-        numOfElements: float
-        responseType: float
-      description: ''
-    safety:
-      mode: guarded
-  decodeSingularString:
-    action_cmd:
-      command: decodeSingularString
-      args:
-        response: 0.0
-        index: 0.0
-        stringLength: 0.0
-      arg_types:
-        response: float
-        index: float
-        stringLength: float
-      description: ''
-    safety:
-      mode: guarded
-  decodeStringPrepended:
-    action_cmd:
-      command: decodeStringPrepended
-      args:
-        response: 0.0
-        index: 0.0
-        numOfStrings: 0.0
-      arg_types:
-        response: float
-        index: float
-        numOfStrings: float
-      description: ''
-    safety:
-      mode: guarded
-  handle2DArray:
-    action_cmd:
-      command: handle2DArray
-      args:
-        Array: 0.0
-        BodyType: 0.0
-        BodyPart: 0.0
-      arg_types:
-        Array: float
-        BodyType: float
-        BodyPart: float
-      description: ''
-    safety:
-      mode: guarded
-  handleArray:
-    action_cmd:
-      command: handleArray
-      args:
-        Array: 0.0
-        BodyType: 0.0
-        BodyPart: 0.0
-      arg_types:
-        Array: float
-        BodyType: float
-        BodyPart: float
-      description: ''
-    safety:
-      mode: guarded
-  handleArrayPrepend:
-    action_cmd:
-      command: handleArrayPrepend
-      args:
-        Array: 0.0
-        BodyType: 0.0
-        BodyPart: 0.0
-      arg_types:
-        Array: float
-        BodyType: float
-        BodyPart: float
-      description: ''
-    safety:
-      mode: guarded
-  handleArrayString:
-    action_cmd:
-      command: handleArrayString
-      args:
-        Array: 0.0
-        BodyType: 0.0
-        BodyPart: 0.0
-      arg_types:
-        Array: float
-        BodyType: float
-        BodyPart: float
-      description: ''
-    safety:
-      mode: guarded
-  handleString:
-    action_cmd:
-      command: handleString
-      args:
-        BodyElement: 0.0
-        BodyType: 0.0
-        BodyPart: 0.0
-      arg_types:
-        BodyElement: float
-        BodyType: float
-        BodyPart: float
-      description: ''
-    safety:
-      mode: guarded
-  parseError:
-    action_cmd:
-      command: parseError
-      args:
-        response: 0.0
-        index: 0.0
-      arg_types:
-        response: float
-        index: float
-      description: ''
-    safety:
-      mode: guarded
-  parseGeneralResponse:
-    action_cmd:
-      command: parseGeneralResponse
-      args:
-        Response: 0.0
-        ResponseTypes: 0.0
-      arg_types:
-        Response: float
-        ResponseTypes: float
-      description: ''
-    safety:
-      mode: guarded
-  printDebugInfo:
-    action_cmd:
-      command: printDebugInfo
-      args:
-        responseTypes: 0.0
-        bodyParts: 0.0
-      arg_types:
-        responseTypes: float
-        bodyParts: float
-      description: ''
-    safety:
-      mode: guarded
-  quickSend:
-    action_cmd:
-      command: quickSend
-      args:
-        Command: 0.0
-        Body: 0.0
-        BodyType: 0.0
-        ResponseTypes: 0.0
-      arg_types:
-        Command: float
-        Body: float
-        BodyType: float
-        ResponseTypes: float
-      description: 'quickSend(self, Command, Body, BodyType,ResponseTypes) Parameters
-        for quicksend: Command : as written in documentation Body: Body to send as
-        array [] - use [] when no argument should be sent! BodyType: Array of [Type
-        of data] - see also ResponseTypes ResponseTypes: Array of Types to decode
-        response IDENTIFIERS: H : unsigned int16 h : int16 I : unsigned int32 i :
-        int32 f : float32 d : float64 (double) Arrays (1D): Start with * length taken
-        from directly before the array (+ or -) *I : array of unsigned int32 (+ or
-        -) *i : array of int32 (+ or -) *f : array of float32 (+ or -) *d : array
-        of float64 (+ or -) *c : String! (Array of chars - interpreted as string!)
-        NEED TO UPDATE WITHOUT "*"!!!! Arrays (2D): start with 2 width and height
-        taken from the two variables before the array (+ or -) 2f : 2d array of float32
-        UNIQUE FOR RETURN TYPES: "**" Identifier for arrays whose size is defined
-        by the first returned argument'
-    safety:
-      mode: guarded
-  returnDebugInfo:
-    action_cmd:
-      command: returnDebugInfo
-      args:
-        displayInfo: 0.0
-      arg_types:
-        displayInfo: float
-      description: ''
-    safety:
-      mode: guarded
-  send:
-    action_cmd:
-      command: send
-      args:
-        Command: 0.0
-        Body: 0.0
-        BodyType: 0.0
-      arg_types:
-        Command: float
-        Body: float
-        BodyType: float
-      description: ''
-    safety:
-      mode: guarded

docs/extension_workflow.md

@@ -46,4 +46,7 @@ nanonis.close()
 ## Notes
 - Core behavior is generic and spec-driven.
 - `set` and `ramp` operate on any writable parameter with safety settings.
-- The generator imports all discovered `Get`/`Set` methods and keeps per-parameter descriptions from method docstrings.
+- Command discovery is anchored at `Bias_Set`; callable members declared before
+  that method are ignored to avoid importing internal helper methods.
+- Discovered `Get`/`Set` methods are emitted under `parameters`, and non-`Get`/`Set`
+  methods are emitted under `actions` with command-level descriptions.

nanonis_qcodes_controller/qcodes_driver/manifest_generator.py

@@ -73,10 +73,26 @@ def discover_nanonis_commands(*, match_pattern: str = "") -> tuple[CommandInfo,
 
     compiled_pattern = re.compile(match_pattern, re.IGNORECASE) if match_pattern else None
     discovered: list[CommandInfo] = []
-
-    for name, member in inspect.getmembers(nanonis_spm.Nanonis, predicate=callable):
+    ordered_callables: list[tuple[str, Any]] = []
+    for name, member in nanonis_spm.Nanonis.__dict__.items():
         if name.startswith("_"):
             continue
+        callable_member: Any = member
+        if isinstance(member, (staticmethod, classmethod)):
+            callable_member = member.__func__
+        if not callable(callable_member):
+            continue
+        ordered_callables.append((name, callable_member))
+
+    anchor_index: int | None = None
+    for index, (name, _member) in enumerate(ordered_callables):
+        if name == "Bias_Set":
+            anchor_index = index
+            break
+    if anchor_index is None:
+        raise ValueError("Could not find Bias_Set anchor in nanonis_spm.Nanonis callables.")
+
+    for name, member in ordered_callables[anchor_index:]:
         if compiled_pattern is not None and compiled_pattern.search(name) is None:
             continue
 
@@ -164,15 +180,12 @@ def build_unified_manifest(
         generated_actions[info.command] = _build_generated_action_entry(info=info)
 
     merged_actions: dict[str, Any] = {}
-    all_action_names = sorted(set(generated_actions).union(curated_actions))
-    for name in all_action_names:
-        generated_entry = generated_actions.get(name)
+    for name in sorted(generated_actions):
+        generated_entry = generated_actions[name]
         curated_entry = curated_actions.get(name)
-        if generated_entry is not None and curated_entry is not None:
+        if curated_entry is not None:
             merged_actions[name] = _deep_merge(generated_entry, curated_entry)
-        elif curated_entry is not None:
-            merged_actions[name] = curated_entry
-        elif generated_entry is not None:
+        else:
             merged_actions[name] = generated_entry
 
     for _name, parameter in merged_parameters.items():