@qui-cli/env-1password 1.2.1 → 1.2.3

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to this project will be documented in this file. See [commit-and-tag-version](https://github.com/absolute-version/commit-and-tag-version) for commit guidelines.
 
+## [1.2.3](https://github.com/battis/qui-cli/compare/env-1password/1.2.2...env-1password/1.2.3) (2025-12-24)
+
+
+### Bug Fixes
+
+* handle inconsistencies of 1password-sdk sectionIds more gracefully ([bfa2be1](https://github.com/battis/qui-cli/commit/bfa2be1750da265676b31f7162f24e7d75a31502))
+
+## [1.2.2](https://github.com/battis/qui-cli/compare/env-1password/1.2.1...env-1password/1.2.2) (2025-12-24)
+
+
+### Bug Fixes
+
+* handle 1Password secret reference not in sections ([bd57bb6](https://github.com/battis/qui-cli/commit/bd57bb67f5027135425dfdd61fef5b3c1c2301c7))
+
 ## [1.2.1](https://github.com/battis/qui-cli/compare/env-1password/1.2.0...env-1password/1.2.1) (2025-12-23)
 
 

package/README.md

@@ -38,7 +38,7 @@ This package integrates with [@1password/sdk](https://www.npmjs.com/package/@1pa
 3. Update environment variables to be [secret references](https://developer.1password.com/docs/cli/secret-references)
 4. Run!
 
-See [dev-1password-env](https://github.com/battis/qui-cli/tree/main/examples/dev-1password-env#readme) for an example using the 1Password implementation of this package.
+See [dev-env-1password](https://github.com/battis/qui-cli/tree/main/examples/dev-env-1password#readme) for an example using the 1Password implementation of this package.
 
 ## Configuration
 

package/dist/1Password.js

@@ -67,7 +67,7 @@ export function options() {
                 default: config.opAccount
             },
             opItem: {
-                description: `Name or ID of the 1Password API Credential item storing the 1Password service account token; will use environmen variable ${Colors.varName('OP_ITEM')} if present`,
+                description: `Name or ID of the 1Password API Credential item storing the 1Password service account token; will use environment variable ${Colors.varName('OP_ITEM')} if present`,
                 hint: '1Password unique identifier',
                 default: config.opItem
             },
@@ -100,6 +100,8 @@ function isSecretReference(value) {
 function secretReferences(parsed) {
     return Object.fromEntries(Object.entries(parsed).filter((entry) => isSecretReference(entry[1])));
 }
+// FIXME parse is not loading 1Password secrets into process.env
+// Issue URL: https://github.com/battis/qui-cli/issues/81
 export async function parse(file) {
     const parsed = await Env.parse(file);
     if (client) {
@@ -120,9 +122,14 @@ export async function exists({ key, file }) {
     return Env.exists({ key, file });
 }
 function secretFrom(secretReference) {
-    const [vault, item, section, field] = secretReference
-        .replace('op://', '')
+    // eslint-disable-next-line prefer-const
+    let [vault, item, section, field] = secretReference
+        .replace(/^op:\/\//, '')
         .split('/');
+    if (field === undefined) {
+        field = section;
+        section = '';
+    }
     return { vault, item, section, field };
 }
 async function itemFrom(secret) {
@@ -152,14 +159,14 @@ export async function set({ key, value, file, ...rest }) {
                 const updated = {
                     ...item,
                     fields: item.fields.map((field) => {
-                        if (field.title === secret.field &&
-                            ((/^Section_.+/.test(secret.section) &&
-                                field.sectionId == secret.section.replace(/^Section_/, '')) ||
-                                field.sectionId ==
-                                    item.sections
-                                        .filter((section) => section.title === secret.section)
-                                        .shift()?.id)) {
-                            return { ...field, value };
+                        const section = item.sections.find((s) => s.id === field.sectionId);
+                        if (secret.field === field.title || secret.field === field.id) {
+                            if ((!secret.section &&
+                                (field.sectionId === '' || field.sectionId === 'add more')) ||
+                                secret.section === section?.title ||
+                                secret.section === section?.id) {
+                                return { ...field, value };
+                            }
                         }
                         return field;
                     })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qui-cli/env-1password",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "description": "@qui-cli Plugin: Standardized environment configuration",
   "homepage": "https://github.com/battis/qui-cli/tree/main/packages/env-1password#readme",
   "repository": {
@@ -28,9 +28,9 @@
     "del-cli": "^6.0.0",
     "npm-run-all": "^4.1.5",
     "typescript": "^5.9.3",
-    "@qui-cli/colors": "3.1.1",
-    "@qui-cli/log": "4.0.1",
     "@qui-cli/env": "5.0.2",
+    "@qui-cli/log": "4.0.1",
+    "@qui-cli/colors": "3.1.1",
     "@qui-cli/plugin": "4.0.0",
     "@qui-cli/root": "3.0.4",
     "@qui-cli/shell": "3.0.3"