@enerlence/suntropy-cli 0.1.1 → 0.1.3

package/README.md

@@ -140,14 +140,17 @@ suntropy inventory panels delete <id>
 #   kits, charger-kits, heatpump-kits, manufacturers
 ```
 
-**Kit assembly:**
+**Kit assembly (create kit components first, then assemble):**
 
 ```bash
-suntropy inventory kits panels create --data '{"name":"Panel Kit","peakPower":450}'
-suntropy inventory kits inverters create --data '{"name":"Inversor Kit","nominalPower":5000}'
+# 1. Create kit-specific panel and inverter
+suntropy inventory kits panels create --data '{"name":"Panel Kit 450W","peakPower":450}'
+suntropy inventory kits inverters create --data '{"name":"Inversor Kit 5kW","nominalPower":5000}'
+
+# 2. Assemble kit referencing component IDs
 suntropy inventory kits assemble \
   --name "Kit Solar 5kW" \
-  --panel <panelId> --inverter <inverterId> \
+  --panel <kitPanelId> --inverter <kitInverterId> \
   --panels-count 12 --peak-power 5.4 --price 3500
 ```
 

package/dist/bin/suntropy.js

@@ -632,6 +632,8 @@ function registerKitsCommands(inventory) {
     batchDeletePath: "delete-batch-solar-kits",
     getViaFilter: true
   });
+  const kitsCreateIdx = kits.commands.findIndex((c) => c.name() === "create");
+  if (kitsCreateIdx !== -1) kits.commands.splice(kitsCreateIdx, 1);
   kits.command("archive <kitId>").description("Archive a solar kit (soft-disable)").action(async (kitId) => {
     try {
       const global = getGlobalOpts3(kits);
@@ -788,10 +790,71 @@ function registerKitsCommands(inventory) {
   });
   kits.command("assemble").description(
     'Assemble a solar kit from existing components by ID.\nReferences kit panels, inverters, batteries, and custom assets by their IDs.\n\nCustom assets format: --custom-asset <assetId>:<units> (repeatable)\n\nExamples:\n  suntropy inventory kits assemble --name "Kit 5kW" --panel 123 --inverter 456 --panels-count 12 --price 6500\n  suntropy inventory kits assemble --name "Kit Premium" --panel 123 --inverter 456 --battery 789 \\\n    --panels-count 12 --inverters-count 1 --batteries-count 1 --peak-power 5.4 --price 8500 \\\n    --custom-asset 100:12 --custom-asset 200:1 --phase single_phase'
-  ).requiredOption("--name <identifier>", "Kit name/identifier").option("--panel <kitPanelId>", "Kit panel ID (idKitSolarPanel)").option("--inverter <kitInverterId>", "Kit inverter ID (idKitInverter)").option("--battery <batteryId>", "Battery ID from inventory (batteryId)").option("--panels-count <n>", "Number of panels", "12").option("--inverters-count <n>", "Number of inverters", "1").option("--batteries-count <n>", "Number of batteries", "0").option("--peak-power <kW>", "Total peak power in kW").option("--price <eur>", "Kit price in EUR").option("--phase <type>", "Phase: single_phase or three_phase", "single_phase").option("--coplanar", "Coplanar mounting").option("--taxes <pct>", "Default tax percentage", "21").option("--custom-asset <id:units>", "Custom asset as id:units (repeatable)", collectCustomAssets, []).action(async (opts) => {
+  ).requiredOption("--name <identifier>", "Kit name/identifier").option("--panel <kitPanelId>", "Kit panel ID (idKitSolarPanel)").option("--inverter <kitInverterId>", "Kit inverter ID (idKitInverter)").option("--battery <batteryId>", "Battery ID from inventory (batteryId)").option("--panels-count <n>", "Number of panels", "12").option("--inverters-count <n>", "Number of inverters", "1").option("--batteries-count <n>", "Number of batteries", "0").option("--peak-power <kW>", "Total peak power in kW").requiredOption("--price <eur>", "Kit price in EUR (required)").option("--phase <type>", "Phase: single_phase or three_phase", "single_phase").option("--coplanar", "Coplanar mounting").option("--taxes <pct>", "Default tax percentage", "21").option("--custom-asset <id:units>", "Custom asset as id:units (repeatable)", collectCustomAssets, []).action(async (opts) => {
     try {
       const global = getGlobalOpts3(kits);
       const client = createServiceClient("solar", global);
+      const validationErrors = [];
+      if (opts.panel) {
+        try {
+          const res2 = await client.post("/solar-kits/solar-panels/filter", { idKitSolarPanel: parseInt(opts.panel) });
+          const data = Array.isArray(res2.data) ? res2.data : res2.data?.data || [];
+          if (data.length === 0) {
+            validationErrors.push(
+              `Panel ID ${opts.panel} is not a KitSolarPanel. Use "suntropy inventory kits panels list" to see available kit panels, or "suntropy inventory kits panels create --data '{...}'" to create one first.`
+            );
+          }
+        } catch {
+          validationErrors.push(
+            `Could not validate panel ID ${opts.panel}. Make sure it is a KitSolarPanel ID (not a regular inventory panel). Use "suntropy inventory kits panels list" to see available kit panels.`
+          );
+        }
+      }
+      if (opts.inverter) {
+        try {
+          const res2 = await client.get("/solar-kits/inverters", { params: { limit: 200 } });
+          const inverters = Array.isArray(res2.data) ? res2.data : res2.data?.data || [];
+          const found = inverters.find(
+            (inv) => String(inv.idKitInverter) === String(opts.inverter)
+          );
+          if (!found) {
+            validationErrors.push(
+              `Inverter ID ${opts.inverter} is not a KitInverter. Use "suntropy inventory kits inverters list" to see available kit inverters, or "suntropy inventory kits inverters create --data '{...}'" to create one first.`
+            );
+          }
+        } catch {
+          validationErrors.push(
+            `Could not validate inverter ID ${opts.inverter}. Make sure it is a KitInverter ID (not a regular inventory inverter). Use "suntropy inventory kits inverters list" to see available kit inverters.`
+          );
+        }
+      }
+      if (opts.battery) {
+        try {
+          const res2 = await client.get("/solar-kits/batteries", { params: { limit: 200 } });
+          const batteries = Array.isArray(res2.data) ? res2.data : res2.data?.data || [];
+          const found = batteries.find(
+            (bat) => String(bat.idKitBattery) === String(opts.battery)
+          );
+          if (!found) {
+            validationErrors.push(
+              `Battery ID ${opts.battery} is not a KitBattery. Use "suntropy inventory kits batteries list" to see available kit batteries, or "suntropy inventory kits batteries create --data '{...}'" to create one first.`
+            );
+          }
+        } catch {
+          validationErrors.push(
+            `Could not validate battery ID ${opts.battery}. Make sure it is a KitBattery ID (not a regular inventory battery). Use "suntropy inventory kits batteries list" to see available kit batteries.`
+          );
+        }
+      }
+      if (validationErrors.length > 0) {
+        outputError({
+          error: true,
+          message: "Kit assembly validation failed",
+          issues: validationErrors,
+          hint: "The assemble command requires KitSolarPanel, KitInverter, and KitBattery IDs \u2014 these are kit-specific entities, not regular inventory items."
+        });
+        return;
+      }
       const body = {
         identifier: opts.name,
         panelNumber: parseInt(opts.panelsCount),
@@ -805,7 +868,7 @@ function registerKitsCommands(inventory) {
       if (opts.inverter) body.kitInverter = { idKitInverter: parseInt(opts.inverter) };
       if (opts.battery) body.battery = { batteryId: parseInt(opts.battery) };
       if (opts.peakPower) body.peakPower = parseFloat(opts.peakPower);
-      if (opts.price) body.price = parseFloat(opts.price);
+      body.price = parseFloat(opts.price);
       if (opts.coplanar) body.coplanar = true;
       if (opts.customAsset && opts.customAsset.length > 0) {
         body.solarKitCustomAssets = opts.customAsset.map((ca) => ({