@feralfile/cli 2.1.0 → 2.2.1

package/README.md

@@ -119,7 +119,6 @@ npm run dev -- setup
 npm run dev -- find https://objkt.com/tokens/hicetnunc/111068 --play
 npm run dev -- play "https://example.com/video.mp4" --skip-verify
 ```
-
 ## Documentation
 
 - Getting started and usage: `./docs/README.md`

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@feralfile/cli",
-    "version": "2.1.0",
+    "version": "2.2.1",
     "description": "CLI for building, signing, and playing DP-1 playlists of digital art on Feral File devices",
     "repository": {
         "type": "git",
@@ -59,7 +59,7 @@
     "license": "MIT",
     "dependencies": {
         "@napi-rs/keyring": "1.3.0",
-        "@feralfile/source-resolver": "git+https://github.com/feral-file/ff-source-resolver.git#54be943014c8cf7860114510257cdd713bbf4e79",
+        "@feralfile/source-resolver": "^1.0.0",
         "axios": "^1.6.2",
         "bonjour-service": "^1.3.0",
         "chalk": "^4.1.2",

package/dist/src/commands/device.js

@@ -11,6 +11,7 @@ const device_lookup_1 = require("../utilities/device-lookup");
 const device_normalize_1 = require("../utilities/device-normalize");
 const device_upsert_1 = require("../utilities/device-upsert");
 const device_default_1 = require("../utilities/device-default");
+const device_rename_1 = require("../utilities/device-rename");
 const config_files_1 = require("./helpers/config-files");
 const device_discovery_1 = require("./helpers/device-discovery");
 const prompt_1 = require("./helpers/prompt");
@@ -206,8 +207,10 @@ deviceCommand
         // existingIndex === -1 (no confirmed match, e.g. manual IP → .local migration),
         // a same-name entry is the upsertDevice case-3 migration path — blocking it would
         // prevent the user from retaining their existing device name during host migration.
+        // Case-insensitive: lookups everywhere else ignore case, so "Kitchen"
+        // alongside "kitchen" would make one of them unreachable by name.
         const nameConflict = existingIndex !== -1
-            ? existingDevices.find((d, i) => d.name === deviceName && i !== existingIndex)
+            ? existingDevices.find((d, i) => d.name?.toLowerCase() === deviceName.toLowerCase() && i !== existingIndex)
             : undefined;
         if (nameConflict) {
             if (options.name) {
@@ -220,7 +223,7 @@ deviceCommand
             const retryAnswer = await ask('Device name: ');
             deviceName = retryAnswer || 'ff1';
             const retryConflict = existingIndex !== -1
-                ? existingDevices.find((d, i) => d.name === deviceName && i !== existingIndex)
+                ? existingDevices.find((d, i) => d.name?.toLowerCase() === deviceName.toLowerCase() && i !== existingIndex)
                 : undefined;
             if (retryConflict) {
                 console.error(chalk_1.default.red(`\nName "${deviceName}" is also taken. No changes made.`));
@@ -254,21 +257,7 @@ deviceCommand
         }
         const config = await (0, config_files_1.readConfigFile)(configPath);
         const existingDevices = config.ff1Devices?.devices || [];
-        // Match by name (case-insensitive) or by host URL so unnamed legacy/manual
-        // entries (stored without a name field) can still be targeted and removed.
-        const normalizedArg = name.toLowerCase();
-        let normalizedArgHost = '';
-        try {
-            normalizedArgHost = (0, device_normalize_1.normalizeDeviceHost)(name).toLowerCase();
-        }
-        catch {
-            // not a valid URL — host matching will not apply
-        }
-        const deviceIndex = existingDevices.findIndex((d) => (d.name && d.name.toLowerCase() === normalizedArg) ||
-            (d.host && d.host.toLowerCase() === normalizedArg) ||
-            (normalizedArgHost &&
-                d.host &&
-                (0, device_normalize_1.normalizeDeviceHost)(d.host).toLowerCase() === normalizedArgHost));
+        const deviceIndex = (0, device_lookup_1.findConfiguredDeviceIndex)(existingDevices, name);
         if (deviceIndex === -1) {
             console.error(chalk_1.default.red(`\nDevice "${name}" not found`));
             if (existingDevices.length > 0) {
@@ -291,6 +280,50 @@ deviceCommand
         process.exit(1);
     }
 });
+deviceCommand
+    .command('rename')
+    .description('Rename a configured FF1 device (host, API key, and default status are untouched)')
+    .argument('<name>', 'Current device name or host')
+    .argument('<new-name>', 'New device name')
+    .action(async (name, newName) => {
+    try {
+        const configPath = await (0, config_files_1.resolveExistingConfigPath)();
+        if (!configPath) {
+            console.log(chalk_1.default.red('config.json not found'));
+            console.log(chalk_1.default.dim('Run: ff-cli setup'));
+            process.exit(1);
+        }
+        const config = await (0, config_files_1.readConfigFile)(configPath);
+        const existingDevices = config.ff1Devices?.devices || [];
+        if (existingDevices.length === 0) {
+            console.log(chalk_1.default.yellow('\nNo devices configured'));
+            console.log(chalk_1.default.dim('Run: ff-cli device add\n'));
+            process.exit(1);
+        }
+        let result;
+        try {
+            result = (0, device_rename_1.renameDevice)(existingDevices, name, newName);
+        }
+        catch (error) {
+            console.error(chalk_1.default.red(`\n${error.message}`));
+            const names = existingDevices.map((d) => d.name || d.host).join(', ');
+            console.log(chalk_1.default.dim(`Available devices: ${names}\n`));
+            process.exit(1);
+        }
+        if (result.unchanged) {
+            console.log(chalk_1.default.dim(`\n"${result.renamed.name}" is already named that.\n`));
+            return;
+        }
+        config.ff1Devices = { devices: result.devices };
+        await fs_1.promises.writeFile(configPath, `${JSON.stringify(config, null, 2)}\n`, 'utf-8');
+        const from = result.previousName || result.renamed.host;
+        console.log(chalk_1.default.green(`\nRenamed device: ${from} → ${result.renamed.name}\n`));
+    }
+    catch (error) {
+        console.error(chalk_1.default.red('\nError:'), error.message);
+        process.exit(1);
+    }
+});
 deviceCommand
     .command('default')
     .description('Set the default FF1 device (reorders so this device is used when -d is omitted)')

package/dist/src/utilities/device-default.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.promoteDeviceToDefault = promoteDeviceToDefault;
-const device_normalize_1 = require("./device-normalize");
+const device_lookup_1 = require("./device-lookup");
 /**
  * Move the named device to index 0 so it becomes the implicit default.
  *
@@ -11,19 +11,7 @@ const device_normalize_1 = require("./device-normalize");
  * @throws {Error} When no device matches the identifier
  */
 function promoteDeviceToDefault(devices, identifier) {
-    const normalizedArg = identifier.toLowerCase();
-    let normalizedArgHost = '';
-    try {
-        normalizedArgHost = (0, device_normalize_1.normalizeDeviceHost)(identifier).toLowerCase();
-    }
-    catch {
-        // not a valid URL — host matching will not apply
-    }
-    const index = devices.findIndex((d) => (d.name && d.name.toLowerCase() === normalizedArg) ||
-        (d.host && d.host.toLowerCase() === normalizedArg) ||
-        (normalizedArgHost &&
-            d.host &&
-            (0, device_normalize_1.normalizeDeviceHost)(d.host).toLowerCase() === normalizedArgHost));
+    const index = (0, device_lookup_1.findConfiguredDeviceIndex)(devices, identifier);
     if (index === -1) {
         throw new Error(`Device "${identifier}" not found`);
     }

package/dist/src/utilities/device-lookup.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.findExistingDeviceEntry = findExistingDeviceEntry;
+exports.findConfiguredDeviceIndex = findConfiguredDeviceIndex;
+const device_normalize_1 = require("./device-normalize");
 /**
  * Find an existing configured device that corresponds to a newly discovered host.
  *
@@ -106,3 +108,29 @@ function findExistingDeviceEntry(existingDevices, newHost, discoveredName, disco
     }
     return undefined;
 }
+/**
+ * Find the index of a configured device by a user-supplied identifier.
+ *
+ * Matches by friendly name (case-insensitive) or by host URL — raw or
+ * normalized — so unnamed legacy/manual entries (stored without a name field)
+ * can still be targeted. Shared by `device remove`, `device default`,
+ * `device rename`, and runtime `-d` resolution so every surface answers to
+ * the same identifier the same way.
+ *
+ * Returns -1 when nothing matches.
+ */
+function findConfiguredDeviceIndex(devices, identifier) {
+    const normalizedArg = identifier.toLowerCase();
+    let normalizedArgHost = '';
+    try {
+        normalizedArgHost = (0, device_normalize_1.normalizeDeviceHost)(identifier).toLowerCase();
+    }
+    catch {
+        // not a valid URL — host matching will not apply
+    }
+    return devices.findIndex((d) => (d.name && d.name.toLowerCase() === normalizedArg) ||
+        (d.host && d.host.toLowerCase() === normalizedArg) ||
+        (normalizedArgHost &&
+            d.host &&
+            (0, device_normalize_1.normalizeDeviceHost)(d.host).toLowerCase() === normalizedArgHost));
+}

package/dist/src/utilities/device-rename.js

@@ -0,0 +1,38 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.renameDevice = renameDevice;
+const device_lookup_1 = require("./device-lookup");
+/**
+ * Rename a configured device without touching host, apiKey, topicID, or list
+ * position — a pure config edit that needs no discovery or network access
+ * (unlike `device add`, whose update path requires the host or mDNS).
+ *
+ * Matches by name (case-insensitive) or by host URL, mirroring `device remove`
+ * so unnamed legacy entries can still be targeted (renaming gives them a name
+ * for the first time).
+ *
+ * @throws {Error} When no device matches the identifier, the new name is
+ *   empty, or the new name is already used by a different device
+ *   (case-insensitive, so lookups elsewhere stay unambiguous).
+ */
+function renameDevice(devices, identifier, newName) {
+    const trimmedName = newName.trim();
+    if (!trimmedName) {
+        throw new Error('New device name must not be empty');
+    }
+    const index = (0, device_lookup_1.findConfiguredDeviceIndex)(devices, identifier);
+    if (index === -1) {
+        throw new Error(`Device "${identifier}" not found`);
+    }
+    const conflict = devices.find((d, i) => i !== index && d.name && d.name.toLowerCase() === trimmedName.toLowerCase());
+    if (conflict) {
+        throw new Error(`Device name "${trimmedName}" is already used by another device (${conflict.host})`);
+    }
+    const target = devices[index];
+    if (target.name === trimmedName) {
+        return { devices: [...devices], renamed: target, previousName: target.name, unchanged: true };
+    }
+    const updated = [...devices];
+    updated[index] = { ...target, name: trimmedName };
+    return { devices: updated, renamed: updated[index], previousName: target.name, unchanged: false };
+}

package/dist/src/utilities/ff1-compatibility.js

@@ -39,6 +39,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolveConfiguredDevice = resolveConfiguredDevice;
 exports.assertFF1CommandCompatibility = assertFF1CommandCompatibility;
 const config_1 = require("../config");
+const device_lookup_1 = require("./device-lookup");
 const logger = __importStar(require("../logger"));
 const FF1_COMMAND_POLICIES = {
     displayPlaylist: {
@@ -51,7 +52,8 @@ const FF1_COMMAND_POLICIES = {
 /**
  * Load and validate the configured FF1 device selected by name.
  *
- * @param {string} [deviceName] - Optional device name, exact match required
+ * @param {string} [deviceName] - Optional device name (case-insensitive) or host URL,
+ *   resolved with the same matching rules as `device remove`/`default`/`rename`
  * @param {Object} [options] - Optional dependency overrides
  * @param {Function} [options.getFF1DeviceConfigFn] - Optional config loader override
  * @returns {FF1DeviceSelectionResult} Selected device or reason for failure
@@ -70,7 +72,8 @@ function resolveConfiguredDevice(deviceName, options = {}) {
     }
     let device = deviceConfig.devices[0];
     if (deviceName) {
-        device = deviceConfig.devices.find((item) => item.name === deviceName);
+        const index = (0, device_lookup_1.findConfiguredDeviceIndex)(deviceConfig.devices, deviceName);
+        device = index === -1 ? undefined : deviceConfig.devices[index];
         if (!device) {
             const availableNames = deviceConfig.devices
                 .map((item) => item.name)

package/docs/CONFIGURATION.md

@@ -139,6 +139,7 @@ You can also manage devices independently with:
   non-interactively with `--host`, `--name`, and the physical `--id` shown in
   Feral File mobile. Manual devices need the ID before relayer pairing.
 - `ff-cli device list` – Show all configured devices.
+- `ff-cli device rename <name> <new-name>` – Rename a device without touching its host, API key, or default status. Works offline; also accepts the host URL as the identifier, so unnamed legacy entries can be given a name.
 - `ff-cli device remove <name>` – Remove a device by name.
 - `ff-cli device default <name>` – Promote a device to the top of the list so it is used when `-d` is omitted.
 - `ff-cli device pair [name]` – Display a one-time handoff code and key-bound
@@ -150,7 +151,7 @@ Setup and `device add` both preserve existing devices. Adding a device with the
 Selection rules when sending:
 
 - If you omit `-d`, the first configured device is used.
-- If you pass `-d <name>`, the CLI matches the device by `name` (exact match). If not found, you’ll see an error listing available devices.
+- If you pass `-d <name>`, the CLI matches the device by `name` (case-insensitive) or by host URL — the same rules as `device remove`/`default`/`rename`, so unnamed entries can be targeted by host. If not found, you’ll see an error listing available devices.
 
 Device cast contract:
 

package/docs/EXAMPLES.md

@@ -385,6 +385,9 @@ npm run dev -- device add
 # Add a device non-interactively
 npm run dev -- device add --host 192.168.1.100 --name kitchen
 
+# Rename a device (host, API key, and default status are untouched)
+npm run dev -- device rename kitchen gallery
+
 # Remove a device by name
 npm run dev -- device remove kitchen
 

package/docs/README.md

@@ -120,6 +120,7 @@ Both paths run the same deterministic pipeline: fetch metadata, assemble a DP-1
 - `device add` – Add a device with mDNS or manual `--host`, `--name`, and `--id`
   - Options: `--host <host>`, `--name <name>`, `--id <id>`
 - `device pair [name]` – Securely receive relayer access from Feral File mobile
+- `device rename <name> <new-name>` – Rename a configured FF1 device (host, API key, and default status are untouched)
 - `device remove <name>` – Remove a configured FF1 device
 - `device default <name>` – Set the default FF1 device (used when `-d` is omitted)
 - `config <init|show|validate>` – Manage configuration
@@ -274,6 +275,9 @@ ff-cli device add
 # Add a device non-interactively
 ff-cli device add --host 192.168.1.100 --name kitchen --id FF1-XXXXXXXX
 
+# Rename a device (host, API key, and default status are untouched)
+ff-cli device rename kitchen gallery
+
 # Remove a device by name
 ff-cli device remove kitchen
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@feralfile/cli",
-  "version": "2.1.0",
+  "version": "2.2.1",
   "description": "CLI for building, signing, and playing DP-1 playlists of digital art on Feral File devices",
   "repository": {
     "type": "git",
@@ -59,7 +59,7 @@
   "license": "MIT",
   "dependencies": {
     "@napi-rs/keyring": "1.3.0",
-    "@feralfile/source-resolver": "git+https://github.com/feral-file/ff-source-resolver.git#54be943014c8cf7860114510257cdd713bbf4e79",
+    "@feralfile/source-resolver": "^1.0.0",
     "axios": "^1.6.2",
     "bonjour-service": "^1.3.0",
     "chalk": "^4.1.2",