@robinmordasiewicz/f5xc-xcsh 2.0.21-2601090318 → 2.0.21-2601090340

package/completions/xcsh.bash

@@ -47,7 +47,7 @@ _xcsh_completions() {
           return 0
           ;;
         login/profile)
-          COMPREPLY=($(compgen -W "list show create use delete" -- "${cur}"))
+          COMPREPLY=($(compgen -W "list show create use edit delete" -- "${cur}"))
           return 0
           ;;
         login/context)

package/completions/xcsh.fish

@@ -90,6 +90,7 @@ complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcomma
 complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from profile" -a "show" -d 'Show profile details (masked credentials)'
 complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from profile" -a "create" -d 'Create a new connection profile'
 complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from profile" -a "use" -d 'Switch to a different profile'
+complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from profile" -a "edit" -d 'Edit profile in $EDITOR'
 complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from profile" -a "delete" -d 'Delete a saved profile'
 complete -c xcsh -n "__fish_seen_subcommand_from login" -a "context" -d 'Manage default namespace context'
 complete -c xcsh -n "__fish_seen_subcommand_from login; and __fish_seen_subcommand_from context" -a "show" -d 'Show current default namespace'

package/dist/index.js

@@ -28134,7 +28134,7 @@ var require_command = __commonJS({
     var EventEmitter3 = __require("events").EventEmitter;
     var childProcess = __require("child_process");
     var path = __require("path");
-    var fs3 = __require("fs");
+    var fs4 = __require("fs");
     var process13 = __require("process");
     var { Argument: Argument2, humanReadableArgName } = require_argument();
     var { CommanderError: CommanderError2 } = require_error();
@@ -29067,10 +29067,10 @@ Expecting one of '${allowedValues.join("', '")}'`);
         const sourceExt = [".js", ".ts", ".tsx", ".mjs", ".cjs"];
         function findFile(baseDir, baseName) {
           const localBin = path.resolve(baseDir, baseName);
-          if (fs3.existsSync(localBin)) return localBin;
+          if (fs4.existsSync(localBin)) return localBin;
           if (sourceExt.includes(path.extname(baseName))) return void 0;
           const foundExt = sourceExt.find(
-            (ext) => fs3.existsSync(`${localBin}${ext}`)
+            (ext) => fs4.existsSync(`${localBin}${ext}`)
           );
           if (foundExt) return `${localBin}${foundExt}`;
           return void 0;
@@ -29082,7 +29082,7 @@ Expecting one of '${allowedValues.join("', '")}'`);
         if (this._scriptPath) {
           let resolvedScriptPath;
           try {
-            resolvedScriptPath = fs3.realpathSync(this._scriptPath);
+            resolvedScriptPath = fs4.realpathSync(this._scriptPath);
           } catch (err) {
             resolvedScriptPath = this._scriptPath;
           }
@@ -37034,14 +37034,14 @@ var require_parser = __commonJS({
             case "scalar":
             case "single-quoted-scalar":
             case "double-quoted-scalar": {
-              const fs3 = this.flowScalar(this.type);
+              const fs4 = this.flowScalar(this.type);
               if (atNextItem || it.value) {
-                map.items.push({ start, key: fs3, sep: [] });
+                map.items.push({ start, key: fs4, sep: [] });
                 this.onKeyLine = true;
               } else if (it.sep) {
-                this.stack.push(fs3);
+                this.stack.push(fs4);
               } else {
-                Object.assign(it, { key: fs3, sep: [] });
+                Object.assign(it, { key: fs4, sep: [] });
                 this.onKeyLine = true;
               }
               return;
@@ -37169,13 +37169,13 @@ var require_parser = __commonJS({
             case "scalar":
             case "single-quoted-scalar":
             case "double-quoted-scalar": {
-              const fs3 = this.flowScalar(this.type);
+              const fs4 = this.flowScalar(this.type);
               if (!it || it.value)
-                fc.items.push({ start: [], key: fs3, sep: [] });
+                fc.items.push({ start: [], key: fs4, sep: [] });
               else if (it.sep)
-                this.stack.push(fs3);
+                this.stack.push(fs4);
               else
-                Object.assign(it, { key: fs3, sep: [] });
+                Object.assign(it, { key: fs4, sep: [] });
               return;
             }
             case "flow-map-end":
@@ -46994,8 +46994,8 @@ function getLogoModeFromEnv(envPrefix) {
 var CLI_NAME = "xcsh";
 var CLI_FULL_NAME = "F5 Distributed Cloud Shell";
 function getVersion() {
-  if ("v2.0.21-2601090318") {
-    return "v2.0.21-2601090318";
+  if ("v2.0.21-2601090340") {
+    return "v2.0.21-2601090340";
   }
   if (process.env.XCSH_VERSION) {
     return process.env.XCSH_VERSION;
@@ -75423,6 +75423,211 @@ var deleteCommand = {
   }
 };
 
+// src/domains/login/profile/edit.ts
+import { spawn } from "child_process";
+import { promises as fs3 } from "fs";
+import { tmpdir } from "os";
+import { join as join4 } from "path";
+function getEditor() {
+  return process.env.EDITOR || process.env.VISUAL || "vi";
+}
+async function openInEditor(filePath) {
+  const editor = getEditor();
+  return new Promise((resolve, reject) => {
+    const child = spawn(editor, [filePath], {
+      stdio: "inherit",
+      shell: true
+    });
+    child.on("error", (error) => {
+      reject(
+        new Error(
+          `Failed to launch editor '${editor}': ${error.message}`
+        )
+      );
+    });
+    child.on("exit", (code) => {
+      if (code === 0) {
+        resolve();
+      } else {
+        reject(new Error(`Editor exited with code ${code}`));
+      }
+    });
+  });
+}
+async function createTempFile(profile) {
+  const tempDir = tmpdir();
+  const tempFile = join4(
+    tempDir,
+    `xcsh-profile-${profile.name}-${Date.now()}.json`
+  );
+  const content = JSON.stringify(
+    {
+      name: profile.name,
+      apiUrl: profile.apiUrl,
+      apiToken: profile.apiToken || "",
+      defaultNamespace: profile.defaultNamespace || "",
+      // Include optional fields only if they exist
+      ...profile.p12Bundle ? { p12Bundle: profile.p12Bundle } : {},
+      ...profile.cert ? { cert: profile.cert } : {},
+      ...profile.key ? { key: profile.key } : {}
+    },
+    null,
+    2
+  );
+  await fs3.writeFile(tempFile, content, { mode: 384 });
+  return tempFile;
+}
+async function parseEditedProfile(filePath, originalName) {
+  try {
+    const content = await fs3.readFile(filePath, "utf-8");
+    const parsed = JSON.parse(content);
+    if (!parsed.name || typeof parsed.name !== "string") {
+      return { profile: null, error: "Profile name is required" };
+    }
+    if (!parsed.apiUrl || typeof parsed.apiUrl !== "string") {
+      return { profile: null, error: "API URL is required" };
+    }
+    if (parsed.name !== originalName) {
+      return {
+        profile: null,
+        error: `Cannot change profile name from '${originalName}' to '${parsed.name}'. Use 'login profile create' to create a new profile instead.`
+      };
+    }
+    const profile = {
+      name: parsed.name,
+      apiUrl: parsed.apiUrl
+    };
+    if (parsed.apiToken && typeof parsed.apiToken === "string" && parsed.apiToken.trim()) {
+      profile.apiToken = parsed.apiToken.trim();
+    }
+    if (parsed.defaultNamespace && typeof parsed.defaultNamespace === "string" && parsed.defaultNamespace.trim()) {
+      profile.defaultNamespace = parsed.defaultNamespace.trim();
+    }
+    if (parsed.p12Bundle && typeof parsed.p12Bundle === "string" && parsed.p12Bundle.trim()) {
+      profile.p12Bundle = parsed.p12Bundle.trim();
+    }
+    if (parsed.cert && typeof parsed.cert === "string" && parsed.cert.trim()) {
+      profile.cert = parsed.cert.trim();
+    }
+    if (parsed.key && typeof parsed.key === "string" && parsed.key.trim()) {
+      profile.key = parsed.key.trim();
+    }
+    return { profile, error: null };
+  } catch (error) {
+    if (error instanceof SyntaxError) {
+      return { profile: null, error: `Invalid JSON: ${error.message}` };
+    }
+    return {
+      profile: null,
+      error: `Failed to read edited file: ${error instanceof Error ? error.message : "Unknown error"}`
+    };
+  }
+}
+async function cleanupTempFile(filePath) {
+  try {
+    await fs3.unlink(filePath);
+  } catch {
+  }
+}
+var editCommand = {
+  name: "edit",
+  description: "Edit a profile configuration using your preferred text editor ($EDITOR). Opens the profile in JSON format for modification. After saving, the profile is validated and updated. If editing the active profile, the session is automatically refreshed with the new settings.",
+  descriptionShort: "Edit profile in $EDITOR",
+  descriptionMedium: "Open a profile in your text editor for modification, then validate and save changes.",
+  usage: "[name]",
+  aliases: ["modify", "vi"],
+  async execute(args, session) {
+    const manager = getProfileManager();
+    let profileName = args[0];
+    if (!profileName) {
+      const active = await manager.getActive();
+      if (!active) {
+        return errorResult(
+          "No profile specified and no active profile set.\nUsage: login profile edit <name>"
+        );
+      }
+      profileName = active;
+    }
+    const existingProfile = await manager.get(profileName);
+    if (!existingProfile) {
+      return errorResult(
+        `Profile '${profileName}' not found.
+Use 'login profile list' to see available profiles.`
+      );
+    }
+    if (!process.stdin.isTTY) {
+      return errorResult(
+        "Edit command requires an interactive terminal.\nUse 'login profile show <name>' to view profile details."
+      );
+    }
+    let tempFile = null;
+    try {
+      tempFile = await createTempFile(existingProfile);
+      const statsBefore = await fs3.stat(tempFile);
+      const editor = getEditor();
+      const output = [
+        `Opening profile '${profileName}' in ${editor}...`
+      ];
+      await openInEditor(tempFile);
+      const statsAfter = await fs3.stat(tempFile);
+      if (statsBefore.mtimeMs === statsAfter.mtimeMs) {
+        output.push("", "No changes made.");
+        return successResult(output);
+      }
+      const { profile: editedProfile, error } = await parseEditedProfile(
+        tempFile,
+        profileName
+      );
+      if (error || !editedProfile) {
+        return errorResult(`Validation failed: ${error}`);
+      }
+      const saveResult = await manager.save(editedProfile);
+      if (!saveResult.success) {
+        return errorResult(
+          `Failed to save profile: ${saveResult.message}`
+        );
+      }
+      output.push("", `Profile '${profileName}' updated successfully.`);
+      const activeProfileName = await manager.getActive();
+      const isActive = profileName === activeProfileName;
+      if (isActive) {
+        const success = await session.switchProfile(profileName);
+        if (success) {
+          output.push("", "Session refreshed with updated settings.");
+          const profile = session.getActiveProfile();
+          const connectionInfo = buildConnectionInfo(
+            profileName,
+            profile?.apiUrl || "",
+            !!profile?.apiToken,
+            profile?.defaultNamespace || session.getNamespace(),
+            session.isAuthenticated(),
+            session.isTokenValidated(),
+            session.getValidationError() ?? void 0
+          );
+          const tableLines = formatConnectionTable(connectionInfo);
+          output.push("", ...tableLines);
+          return successResult(output, true);
+        }
+      }
+      return successResult(output);
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      return errorResult(`Edit failed: ${message}`);
+    } finally {
+      if (tempFile) {
+        await cleanupTempFile(tempFile);
+      }
+    }
+  },
+  async completion(partial, _args, _session) {
+    const manager = getProfileManager();
+    const profiles = await manager.list();
+    return profiles.map((p) => p.name).filter(
+      (name) => name.toLowerCase().startsWith(partial.toLowerCase())
+    );
+  }
+};
+
 // src/domains/login/profile/active.ts
 var activeCommand = {
   name: "active",
@@ -76235,6 +76440,7 @@ var profileSubcommands = {
     ["show", showCommand],
     ["create", createCommand2],
     ["use", useCommand],
+    ["edit", editCommand],
     ["delete", deleteCommand]
   ]),
   defaultCommand: activeCommand

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robinmordasiewicz/f5xc-xcsh",
-  "version": "2.0.21-2601090318",
+  "version": "2.0.21-2601090340",
   "description": "F5 Distributed Cloud Shell - Interactive CLI for F5 XC",
   "type": "module",
   "bin": {