fix(console): stop coercing plain SIP queue targets into skill groups (#227)

reconcileSkillGroupTargets() ran extractSkillGroupId() on every dial
target, but that helper returns its input unchanged when there is no
"skill-group:" prefix. As a result a plain SIP target such as
"sip:1000@localhost" was rewritten to "skill-group:sip:1000@localhost"
and rendered as a skill-group dropdown on every (re)load of the queue
detail form — even on deployments without the CC addon, where
/cc/skill-groups returns 404 and the target can never resolve.

Guard the reconciliation so it only applies to targets that are already
skill groups (skill-group: URI prefix or an explicit skill_group_id),
leaving SIP targets untouched.

Author: v0l

src/addons/queue/templates/queue_detail.html

@@ -837,6 +837,18 @@ <h3 class="text-sm font-semibold text-slate-800" x-text="tt('queue.section_final
                     if (!target || typeof target !== 'object') {
                         return;
                     }
+                    // Only reconcile targets that are ACTUALLY skill groups.
+                    // `extractSkillGroupId` returns its input unchanged when there is no
+                    // `skill-group:` prefix, so without this guard a plain SIP target
+                    // (e.g. `sip:1000@localhost`) would be coerced into
+                    // `skill-group:sip:1000@localhost` and rendered as a skill-group
+                    // dropdown every time the form is (re)loaded.
+                    const isSkillGroup =
+                        (typeof target.uri === 'string' && /^skill(?:-|_)?group:/i.test(target.uri)) ||
+                        !!target.skill_group_id;
+                    if (!isSkillGroup) {
+                        return;
+                    }
                     const groupId = this.extractSkillGroupId(target.skill_group_id || target.uri);
                     if (!groupId) {
                         return;