@uxland/harmonix-cli 0.0.6 → 0.0.7

package/README.md

@@ -19,7 +19,9 @@ registry=https://your-plugin-store.example.com/upload
 apiKey=your-api-key
 ```
 
-The `--registry` and `--api-key` options override the file.
+The `--registry` and `--api-key` options override the file. A project `.harmonixrc` (the nearest one above the plugin folder) overrides the one in your home folder.
+
+`harmonix publish` exits with code `1` when the upload fails (for example, the plugin does not exist in the store or the version is already published), so scripts can stop on errors.
 
 > `.harmonixrc` contains credentials: never commit it. Projects made with `npm create @uxland/harmonix-plugin@latest` already ignore it.
 

package/index.js

@@ -1,22 +1,27 @@
 #!/usr/bin/env node
 
-const { program } = require('commander');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-const axios = require('axios');
-const FormData = require('form-data');
-
-console.log("reached index");
+const { program } = require("commander");
+const path = require("node:path");
+const fs = require("node:fs");
+const os = require("node:os");
+const axios = require("axios");
+const FormData = require("form-data");
 
 function getConfig() {
   const config = {};
+
+  // User-level config (~/.harmonixrc) first, so the project's .harmonixrc overrides it
+  const homeConfigPath = path.join(os.homedir(), ".harmonixrc");
+  if (fs.existsSync(homeConfigPath)) {
+    Object.assign(config, parseConfigFile(homeConfigPath));
+  }
+
   let dir = process.cwd();
   let steps = 0;
   const maxSteps = 10;
 
   while (steps < maxSteps) {
-    const configPath = path.join(dir, '.harmonixrc');
+    const configPath = path.join(dir, ".harmonixrc");
     if (fs.existsSync(configPath)) {
       Object.assign(config, parseConfigFile(configPath));
       break; // stop at first .harmonixrc found
@@ -28,27 +33,24 @@ function getConfig() {
     steps++;
   }
 
-  // Also merge user-level config (~/.harmonixrc)
-  const homeConfigPath = path.join(os.homedir(), '.harmonixrc');
-  if (fs.existsSync(homeConfigPath)) {
-    Object.assign(config, parseConfigFile(homeConfigPath));
-  }
-
   return config;
 }
 
 function parseConfigFile(filePath) {
   const result = {};
   try {
-    const content = fs.readFileSync(filePath, 'utf8');
-    content.split('\n').forEach(line => {
-      if (line.trim() && !line.startsWith('#')) {
-        const [key, value] = line.split('=');
+    const content = fs.readFileSync(filePath, "utf8");
+    for (const line of content.split("\n")) {
+      if (line.trim() && !line.startsWith("#")) {
+        // Split on the first '=' only: a value may contain '=' (e.g. base64 padding)
+        const separator = line.indexOf("=");
+        const key = separator === -1 ? "" : line.slice(0, separator).trim();
+        const value = separator === -1 ? "" : line.slice(separator + 1).trim();
         if (key && value) {
-          result[key.trim()] = value.trim();
+          result[key] = value;
         }
       }
-    });
+    }
   } catch (err) {
     console.warn(`⚠️ Error reading ${filePath}:`, err.message);
   }
@@ -56,9 +58,9 @@ function parseConfigFile(filePath) {
 }
 
 program
-  .command('publish')
-  .option('--registry <registry>', 'Backend URL (overrides .harmonixrc)')
-  .option('--api-key <apiKey>', 'API Key (overrides .harmonixrc)')
+  .command("publish")
+  .option("--registry <registry>", "Backend URL (overrides .harmonixrc)")
+  .option("--api-key <apiKey>", "API Key (overrides .harmonixrc)")
   .action(async (opts) => {
     try {
       const config = getConfig();
@@ -99,12 +101,19 @@ program
       }
 
       const headers = {
-        'x-api-key': apiKey
+        "x-api-key": apiKey,
       };
       const response = await axios.post(registry, formData, { headers });
-      console.log('✅ Plugin published:', response.data);
+      if (response.data?.success === false) {
+        throw new Error(response.data.error || response.data.message || "Plugin Store rejected the upload");
+      }
+      console.log(`✅ Plugin published: ${pluginId}@${versionNumber}`, response.data);
     } catch (err) {
-      console.error('❌ Failed to publish:', err.message);
+      // Exit non-zero so scripts and CI can tell a failed publish from a successful one
+      const serverError = err.response?.data?.error || err.response?.data?.message;
+      const status = err.response ? ` (HTTP ${err.response.status})` : "";
+      console.error(`❌ Failed to publish${status}:`, serverError || err.message);
+      process.exit(1);
     }
   });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxland/harmonix-cli",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "bin": {
     "harmonix": "index.js"
   },