uc-config 0.2.3 → 0.2.5

package/CHANGELOG.md

@@ -9,6 +9,24 @@ npx uc-config init --refresh-docs
 npx uc-config compile && npx uc-config plan   # must show 0 operations
 ```
 
+## 0.2.5
+
+Action required: none.
+
+- `plan` no longer rejects a `select_source` value on entities that publish no
+  `source_list` (e.g. the Onkyo/Pioneer/Integra eISCP driver, which takes free
+  text like `input-selector tv`). Any non-empty string is accepted there; a
+  published list is still enforced.
+
+## 0.2.4
+
+Action required: none. Existing configs keep their keys.
+
+- `import` builds readable keys from display names (`activity.play_ps5`,
+  `activity.play_ps5.button.mute_short_press`, `entity.living_room_lamp`)
+  instead of `activity8`. Duplicate names get `_2`, `_3`. To rename keys in an
+  existing config, use `state move OLD NEW`.
+
 ## 0.2.3
 
 Action required: none. Docs only.

package/dist/adapter.js

@@ -530,7 +530,12 @@ export function validateCommandParameters(command, metadata, entity) {
             const choices = isObject(source)
                 ? source[String(p.items.field)]
                 : undefined;
-            valid = Array.isArray(choices) && choices.includes(value);
+            // Some drivers (e.g. Onkyo eISCP) publish no list at all and accept free
+            // text such as "input-selector tv". With no list there is nothing to check
+            // against, so accept any non-empty string; a published list is enforced.
+            valid = Array.isArray(choices)
+                ? choices.includes(value)
+                : typeof value === "string" && value.trim() !== "";
         }
         else
             throw new Error(`Unsupported parameter metadata for ${key}`);

package/dist/inventory.d.ts

@@ -7,6 +7,8 @@ export interface Inventory {
     unsupported: string[];
 }
 export declare function inventory(client: CoreClient): Promise<Inventory>;
+/** snake_case key segment; falls back to the id when the name has no letters/digits. */
+export declare function slug(name: string | undefined, fallback: string): string;
 export declare function importConfig(client: CoreClient): Promise<{
     config: Config;
     warnings: string[];

package/dist/inventory.js

@@ -58,13 +58,38 @@ export async function inventory(client) {
     }
     return result;
 }
+/** Display name from a string or a localized {en: ...} map. */
+function nameOf(item) {
+    const n = isObject(item) ? item.name : undefined;
+    if (typeof n === "string")
+        return n;
+    if (isObject(n)) {
+        const v = n.en ?? Object.values(n).find((x) => typeof x === "string");
+        if (typeof v === "string")
+            return v;
+    }
+    return undefined;
+}
+/** snake_case key segment; falls back to the id when the name has no letters/digits. */
+export function slug(name, fallback) {
+    const s = (x) => x
+        .normalize("NFKD")
+        .replace(/[\u0300-\u036f]/g, "")
+        .toLowerCase()
+        .replace(/[^a-z0-9]+/g, "_")
+        .replace(/^_+|_+$/g, "");
+    return s(name ?? "") || s(fallback) || "item";
+}
 export async function importConfig(client) {
     const inv = await inventory(client);
     const resources = {};
     const warnings = [...inv.unsupported.map((p) => `Unsupported endpoint ${p}`)];
-    let serial = 0;
-    const add = (r) => {
-        const key = `${r.kind}${++serial}`;
+    // Readable keys from display names ("Play PS5" -> activity.play_ps5);
+    // collisions get _2, _3, ... in import order.
+    const add = (r, base) => {
+        let key = base;
+        for (let n = 2; key in resources; n++)
+            key = `${base}_${n}`;
         resources[key] = r;
         return key;
     };
@@ -93,7 +118,7 @@ export async function importConfig(client) {
                 warnings.push(`${id}: secret-bearing settings require manual secret() references`);
                 continue;
             }
-            const key = add(r);
+            const key = add(r, `${kind}.${slug(nameOf(item), id)}`);
             if (kind === "integration" || kind === "dock")
                 warnings.push(`${key}: existing resource imported; supply create/setup inputs and secret references to reproduce on a new target`);
             if (kind === "remote" && collection !== "externalRemotes")
@@ -112,7 +137,7 @@ export async function importConfig(client) {
                                     parent: { $ref: key },
                                     id: `${b.button}/${press}`,
                                     data: { [press]: b[press] },
-                                });
+                                }, `${key}.button.${slug(String(b.button), "button")}_${press}`);
                     const pages = await client.list(`/${prefix}/${encodeURIComponent(id)}/ui/pages`);
                     for (const p of pages)
                         add({
@@ -120,7 +145,7 @@ export async function importConfig(client) {
                             parent: { $ref: key },
                             id: String(p.page_id),
                             data: writable(`/${prefix}/{entityId}/ui/pages/{pageId}`, "patch", p),
-                        });
+                        }, `${key}.page.${slug(nameOf(p), String(p.page_id))}`);
                 }
                 catch (e) {
                     if (e instanceof ApiError && [404, 405].includes(e.status))
@@ -139,7 +164,7 @@ export async function importConfig(client) {
                             parent: { $ref: key },
                             id: String(c[group ? "group_id" : "page_id"]),
                             data: writable(`/profiles/{profileId}/${child}/{${group ? "groupId" : "pageId"}}`, "patch", c),
-                        });
+                        }, `${key}.${group ? "group" : "page"}.${slug(nameOf(c), String(c[group ? "group_id" : "page_id"]))}`);
                     }
                 }
             if (collection === "irRemotes") {
@@ -158,7 +183,7 @@ export async function importConfig(client) {
                                     format: code.code.format,
                                     value: code.code.value,
                                 },
-                            });
+                            }, `${key}.ir.${slug(code.cmd_id, "code")}`);
             }
         }
     }
@@ -172,7 +197,7 @@ export async function importConfig(client) {
             id: String(item.entity_id),
             parent: item.integration_id,
             data: writable("/entities/{entityId}", "patch", item),
-        });
+        }, `entity.${slug(nameOf(item), String(item.entity_id))}`);
     }
     const usedDrivers = new Set((inv.collections.integrations ?? []).map((i) => i.driver_id));
     for (const item of inv.collections.drivers ?? []) {
@@ -185,7 +210,7 @@ export async function importConfig(client) {
             warnings.push(`${item.driver_id}: driver settings contain secrets; provide secret references manually`);
             continue;
         }
-        add({ kind: "driver", id: String(item.driver_id), data });
+        add({ kind: "driver", id: String(item.driver_id), data }, `driver.${slug(nameOf(item), String(item.driver_id))}`);
     }
     for (const [section, data] of Object.entries(inv.settings)) {
         const picked = writable(`/cfg/${section}`, "patch", data);
@@ -198,7 +223,7 @@ export async function importConfig(client) {
         if (!Object.keys(picked).length)
             continue;
         if (!secretFields(picked).length)
-            add({ kind: "settings", id: section, data: picked });
+            add({ kind: "settings", id: section, data: picked }, `settings.${section}`);
         else
             warnings.push(`${section}: secret-bearing settings omitted; configure secret references`);
     }

package/docs/snippets.md

@@ -138,6 +138,12 @@ sequences: {
 
 `select_source` values are entity-specific. Copy them from an existing sequence
 or the entity's `attributes.source_list` (`npm run uc -- api GET /entities/<id>`).
+When the entity publishes a `source_list`, the planner rejects values outside it.
+Some drivers publish none and accept free text instead; the planner then accepts
+any non-empty string, so take the value from the driver's documentation. For
+example, the Onkyo/Pioneer/Integra eISCP driver takes `input-selector <name>`
+(`input-selector tv`, `input-selector bd`, `input-selector game`; see its
+[text commands](https://github.com/EddyMcNut/uc-intg-onkyo-avr/blob/main/docs/cheats.md#text-commands)).
 
 ## Create a new activity
 
@@ -276,8 +282,9 @@ npm run uc -- compile --config .uc/imported-$(date +%F).config.ts --out .uc/impo
 diff <(jq -S . .uc/build.json) <(jq -S . .uc/imported-build.json) | less
 ```
 
-Imported keys are generic (`activity3`). Match resources by `kind` + `id` and
-copy the changed fields into `remote.config.ts`. Then `plan` should report
+Imported keys come from display names (`activity.watch_tv`), so they usually
+match yours, but a renamed activity gets a different key. Match resources by
+`kind` + `id` and copy the changed fields into `remote.config.ts`. Then `plan` should report
 0 operations.
 
 ## Undo the last apply

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "uc-config",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "type": "module",
   "description": "Configuration-as-code for the Unfolded Circle Remote 3, designed to be driven by coding agents",
   "license": "MIT",