@tolgamorf/env2op-cli 0.2.0 → 0.2.2

package/dist/index.d.ts

@@ -113,7 +113,7 @@ interface TemplateOptions {
 * @returns ParseResult containing variables and any errors
 * @throws Env2OpError if file not found
 */
-declare function parseEnvFile(filePath: string): ParseResult;
+declare function parseEnvFile(filePath: string): Promise<ParseResult>;
 /**
 * Validate that the parsed result has variables
 *
@@ -198,6 +198,7 @@ declare const ErrorCodes: {
 	readonly VAULT_CREATE_FAILED: "VAULT_CREATE_FAILED";
 	readonly ITEM_EXISTS: "ITEM_EXISTS";
 	readonly ITEM_CREATE_FAILED: "ITEM_CREATE_FAILED";
+	readonly ITEM_EDIT_FAILED: "ITEM_EDIT_FAILED";
 	readonly PARSE_ERROR: "PARSE_ERROR";
 	readonly TEMPLATE_NOT_FOUND: "TEMPLATE_NOT_FOUND";
 	readonly INJECT_FAILED: "INJECT_FAILED";
@@ -215,6 +216,7 @@ declare const errors: {
 	vaultCreateFailed: (message: string) => Env2OpError;
 	itemExists: (title: string, vault: string) => Env2OpError;
 	itemCreateFailed: (message: string) => Env2OpError;
+	itemEditFailed: (message: string) => Env2OpError;
 	parseError: (line: number, message: string) => Env2OpError;
 };
 export { writeTemplate, vaultExists, validateParseResult, signIn, parseEnvFile, itemExists2 as itemExists, generateUsageInstructions, generateTemplateContent, errors, editSecureNote, createVault, createSecureNote, checkSignedIn, checkOpCli, TemplateOptions, ParseResult, ErrorCodes, EnvVariable, EnvLine, Env2OpError, CreateItemResult, CreateItemOptions, ConvertOptions };

package/dist/index.js

@@ -1,5 +1,5 @@
 // src/core/env-parser.ts
-import { existsSync, readFileSync } from "node:fs";
+import { readFile } from "node:fs/promises";
 
 // src/utils/errors.ts
 class Env2OpError extends Error {
@@ -22,6 +22,7 @@ var ErrorCodes = {
   VAULT_CREATE_FAILED: "VAULT_CREATE_FAILED",
   ITEM_EXISTS: "ITEM_EXISTS",
   ITEM_CREATE_FAILED: "ITEM_CREATE_FAILED",
+  ITEM_EDIT_FAILED: "ITEM_EDIT_FAILED",
   PARSE_ERROR: "PARSE_ERROR",
   TEMPLATE_NOT_FOUND: "TEMPLATE_NOT_FOUND",
   INJECT_FAILED: "INJECT_FAILED"
@@ -35,6 +36,7 @@ var errors = {
   vaultCreateFailed: (message) => new Env2OpError(`Failed to create vault: ${message}`, ErrorCodes.VAULT_CREATE_FAILED),
   itemExists: (title, vault) => new Env2OpError(`Item "${title}" already exists in vault "${vault}"`, ErrorCodes.ITEM_EXISTS, "Use default behavior (overwrites) or choose a different item name"),
   itemCreateFailed: (message) => new Env2OpError(`Failed to create 1Password item: ${message}`, ErrorCodes.ITEM_CREATE_FAILED),
+  itemEditFailed: (message) => new Env2OpError(`Failed to edit 1Password item: ${message}`, ErrorCodes.ITEM_EDIT_FAILED),
   parseError: (line, message) => new Env2OpError(`Parse error at line ${line}: ${message}`, ErrorCodes.PARSE_ERROR)
 };
 
@@ -82,12 +84,20 @@ function parseValue(raw) {
   const parts = trimmed.split(/\s+#/);
   return (parts[0] ?? trimmed).trim();
 }
-function parseEnvFile(filePath) {
-  if (!existsSync(filePath)) {
+function stripBom(content) {
+  if (content.charCodeAt(0) === 65279) {
+    return content.slice(1);
+  }
+  return content;
+}
+async function parseEnvFile(filePath) {
+  let rawContent;
+  try {
+    rawContent = await readFile(filePath, "utf-8");
+  } catch {
     throw errors.envFileNotFound(filePath);
   }
-  const rawContent = readFileSync(filePath, "utf-8");
-  const content = stripHeaders(rawContent);
+  const content = stripHeaders(stripBom(rawContent));
   const rawLines = content.split(`
 `);
   const variables = [];
@@ -151,33 +161,34 @@ async function exec(command, args = [], options = {}) {
     const proc = spawn(command, args, {
       stdio: ["ignore", "pipe", "pipe"]
     });
-    let stdout = "";
-    let stderr = "";
+    const stdoutChunks = [];
+    const stderrChunks = [];
     proc.stdout?.on("data", (data) => {
-      const text = data.toString();
-      stdout += text;
+      const text = Buffer.isBuffer(data) ? data.toString() : String(data);
+      stdoutChunks.push(text);
       if (verbose) {
         process.stdout.write(text);
       }
     });
     proc.stderr?.on("data", (data) => {
-      const text = data.toString();
-      stderr += text;
+      const text = Buffer.isBuffer(data) ? data.toString() : String(data);
+      stderrChunks.push(text);
       if (verbose) {
         process.stderr.write(text);
       }
     });
     proc.on("close", (code) => {
       resolve({
-        stdout,
-        stderr,
-        exitCode: code ?? 0
+        stdout: stdoutChunks.join(""),
+        stderr: stderrChunks.join(""),
+        exitCode: code ?? 1
       });
     });
-    proc.on("error", () => {
+    proc.on("error", (err) => {
+      stderrChunks.push(err.message);
       resolve({
-        stdout,
-        stderr,
+        stdout: stdoutChunks.join(""),
+        stderr: stderrChunks.join(""),
         exitCode: 1
       });
     });
@@ -193,27 +204,36 @@ async function execWithStdin(command, args = [], options) {
     const proc = spawn(command, args, {
       stdio: ["pipe", "pipe", "pipe"]
     });
-    let stdout = "";
-    let stderr = "";
+    const stdoutChunks = [];
+    const stderrChunks = [];
     proc.stdin?.write(stdinContent);
     proc.stdin?.end();
     proc.stdout?.on("data", (data) => {
-      const text = data.toString();
-      stdout += text;
+      const text = Buffer.isBuffer(data) ? data.toString() : String(data);
+      stdoutChunks.push(text);
       if (verbose)
         process.stdout.write(text);
     });
     proc.stderr?.on("data", (data) => {
-      const text = data.toString();
-      stderr += text;
+      const text = Buffer.isBuffer(data) ? data.toString() : String(data);
+      stderrChunks.push(text);
       if (verbose)
         process.stderr.write(text);
     });
     proc.on("close", (code) => {
-      resolve({ stdout, stderr, exitCode: code ?? 0 });
+      resolve({
+        stdout: stdoutChunks.join(""),
+        stderr: stderrChunks.join(""),
+        exitCode: code ?? 1
+      });
     });
-    proc.on("error", () => {
-      resolve({ stdout, stderr, exitCode: 1 });
+    proc.on("error", (err) => {
+      stderrChunks.push(err.message);
+      resolve({
+        stdout: stdoutChunks.join(""),
+        stderr: stderrChunks.join(""),
+        exitCode: 1
+      });
     });
   });
 }
@@ -333,7 +353,7 @@ async function editSecureNote(options) {
     };
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
-    throw errors.itemCreateFailed(message);
+    throw errors.itemEditFailed(message);
   }
 }
 // src/core/template-generator.ts
@@ -341,7 +361,7 @@ import { writeFileSync } from "node:fs";
 // package.json
 var package_default = {
   name: "@tolgamorf/env2op-cli",
-  version: "0.2.0",
+  version: "0.2.2",
   description: "Convert .env files to 1Password Secure Notes and generate templates for op inject/run",
   type: "module",
   main: "dist/index.js",