fix UC missing-component warning for pre-enabled clusters

When you open a sample .zap that already had clusters enabled, the
"Required SLC Component not installed" warning was showing for every
one of them even though Studio had the components installed. The
install-requested gate I added in 9cee621 covers the case where ZAP
itself just clicked Install, but pre-existing clusters never go
through that path, so they fell through to the older id comparison.

That comparison was inconsistent about how it stripped Studio's id
prefixes. getClusterIdsByUcComponents would take everything after
the last `-` for anything containing "zigbee", but it would keep the
full `%extension-...%name` prefix for matter and only strip a
leading `%` in normalizeUcDependencyId. So:

  tree leaf  studiocomproot-...-zigbee_basic       -> "zigbee_basic"
  required   %extension-zigbee%zigbee_basic        -> "extension-zigbee%zigbee_basic"

Different strings, no match, warning shown. The matter path only
worked because both sides happened to keep the same `%extension-...%`
form, so the bad logic was symmetric for that one case.

Pull the id-shortening into a single extractUcClusterCode helper
that strips a leading `%`, takes everything after the last remaining
`%` if there is one, then takes everything after the last `-`. Run
both selected ids and required ids through it. Now both sides reduce
to the same cluster code regardless of which delimiter Studio used:

  studiocomproot-...-zigbee_basic                  -> "zigbee_basic"
  %extension-zigbee%zigbee_basic                   -> "zigbee_basic"
  matter:1.0.0-...-%extension-matter%matter_level_control
                                                   -> "matter_level_control"
  zigbee_basic                                     -> "zigbee_basic"

Matter still matches the way it did before (both sides reduce
identically), zigbee with the %-prefixed required value now matches
too, and the install-requested short-circuit from 9cee621 keeps
covering the freshly-installed flow.

Also adds a JSDoc on the inner applyAdded helper in
applyUcComponentUpdate -- eslint's require-jsdoc was failing the
build with this rule on. Functional code unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: paulr34

src/store/zap/mutations.js

@@ -1026,6 +1026,11 @@ export function applyUcComponentUpdate(state, payload) {
     prevAll.filter((x) => x && x.id != null).map((x) => [String(x.id), x])
   )
 
+  /**
+   * Upsert a component into the selected and catalog maps for this mutation.
+   * @param {*} id Component id to add.
+   * @param {*} node Optional node payload to store; falls back to a stub.
+   */
   function applyAdded(id, node) {
     if (id == null) return
     const key = String(id)

src/util/util.js

@@ -152,48 +152,63 @@ export function getUcComponents(ucComponentTreeResponse) {
   return selectedComponents
 }
 
+/**
+ * Reduce any UC component id (Studio tree leaf, %extension-...%component, raw
+ * short code) to its lowercase short cluster code form, so both sides of the
+ * missing-component comparison line up regardless of which delimiter Studio
+ * happens to use.
+ *
+ * Examples:
+ *   "studiocomproot-Zigbee-Cluster_Library-Common-zigbee_basic"  -> "zigbee_basic"
+ *   "matter:1.0.0-Matter-Clusters-%extension-matter%matter_level_control"
+ *                                                               -> "matter_level_control"
+ *   "%extension-zigbee%zigbee_basic"                             -> "zigbee_basic"
+ *   "zigbee_basic"                                               -> "zigbee_basic"
+ *
+ * @param {*} id
+ * @returns {string}
+ */
+export function extractUcClusterCode(id) {
+  if (id == null) return ''
+  let s = String(id).toLowerCase().replace(/^%+/, '')
+  const lastPct = s.lastIndexOf('%')
+  if (lastPct >= 0 && lastPct < s.length - 1) {
+    s = s.substring(lastPct + 1)
+  }
+  const lastDash = s.lastIndexOf('-')
+  if (lastDash >= 0 && lastDash < s.length - 1) {
+    s = s.substring(lastDash + 1)
+  }
+  return s
+}
+
 /**
  * Extract a list of cluster id from a list of Uc component id
  *
- * return: a list of ids
- *   id value example:
- *     "zigbee_basic" from "studiocomproot-Zigbee-Cluster_Library-Common-zigbee_basic"
-       "%extension-matter%matter_level_control" from "matter:1.0.0-Matter-Clusters-%extension-matter%matter_level_control"
- * @param {*} ucComponentIds - an array of ids
+ * return: a list of ids in their short cluster-code form (see extractUcClusterCode).
+ *
+ * @param {*} ucComponents - an array of UC component objects with `id`
  */
 export function getClusterIdsByUcComponents(ucComponents) {
   if (!Array.isArray(ucComponents)) return []
   return ucComponents
     .map((component) => component?.id)
     .filter((id) => id != null && String(id).length > 0)
-    .map((id) => {
-      id = String(id)
-      const lower = id.toLowerCase()
-      if (lower.includes('zigbee')) {
-        return id.substring(id.lastIndexOf('-') + 1)
-      }
-      const pct = id.lastIndexOf('%extension-')
-      if (pct >= 0) {
-        return id.substring(pct)
-      }
-      const ext = lower.lastIndexOf('extension-')
-      if (ext >= 0) {
-        return id.substring(ext)
-      }
-      return id
-    })
+    .map((id) => extractUcClusterCode(id))
     .filter((id) => id != null && id !== '')
 }
 
 /**
- * Normalize UC dependency id for comparing ZCL extension values to Studio node ids.
+ * Normalize an id from the ZCL extension's `value` so it can be compared
+ * against entries produced by getClusterIdsByUcComponents. Applies the same
+ * delimiter-aware extraction so e.g. "%extension-zigbee%zigbee_basic" reduces
+ * to "zigbee_basic" instead of leaving the prefix intact.
+ *
  * @param {string} id
  * @returns {string}
  */
 export function normalizeUcDependencyId(id) {
-  return String(id || '')
-    .toLowerCase()
-    .replace(/^%+/, '')
+  return extractUcClusterCode(id)
 }
 
 /**