@icoretech/warden-mcp 0.1.9 → 0.1.10

package/dist/bw/bwCli.js

@@ -16,9 +16,10 @@ export async function runBw(args, opts = {}) {
     const bwBin = process.env.BW_BIN ?? 'bw';
     // Ensure the CLI never blocks waiting for a prompt (e.g. master password).
     // This is critical for running as an MCP server / in test automation.
-    const finalArgs = args.includes('--nointeraction')
-        ? args
-        : ['--nointeraction', ...args];
+    const injectNoInteraction = opts.noInteraction ?? true;
+    const finalArgs = injectNoInteraction && !args.includes('--nointeraction')
+        ? ['--nointeraction', ...args]
+        : args;
     const env = { ...process.env, ...(opts.env ?? {}) };
     const debug = (process.env.KEYCHAIN_DEBUG_BW ?? 'false').toLowerCase() === 'true';
     const startedAt = Date.now();

package/dist/bw/bwSession.js

@@ -169,7 +169,7 @@ export class BwSessionManager {
         });
         const tryUnlock = async () => {
             try {
-                const { stdout } = await runBw(['unlock', '--passwordenv', 'BW_PASSWORD', '--raw'], { env: unlockEnv, timeoutMs: 60_000 });
+                const { stdout } = await runBw(['unlock', '--passwordenv', 'BW_PASSWORD', '--raw'], { env: unlockEnv, timeoutMs: 60_000, noInteraction: false });
                 return stdout.trim();
             }
             catch {
@@ -186,6 +186,7 @@ export class BwSessionManager {
                             BW_HOST: this.env.host,
                         }),
                         timeoutMs: 60_000,
+                        noInteraction: false,
                     });
                     return stdout.trim();
                 }
@@ -195,7 +196,7 @@ export class BwSessionManager {
                     '--passwordenv',
                     'BW_PASSWORD',
                     '--raw',
-                ], { env: unlockEnv, timeoutMs: 60_000 });
+                ], { env: unlockEnv, timeoutMs: 60_000, noInteraction: false });
                 return stdout.trim();
             }
             catch {

package/dist/sdk/usernameGenerator.js

@@ -63,35 +63,34 @@ const CODAS = [
     'ng',
 ];
 function titleCase(s) {
-    if (!s)
-        return s;
-    const first = s.charAt(0);
-    return first.toUpperCase() + s.slice(1);
+    return s.charAt(0).toUpperCase() + s.slice(1);
 }
 function randomWord(opts, deps) {
     // "Word-like" usernames without pulling a large word list dependency.
     // Produces a pronounceable-ish token such as "cravon" or "Plenast7".
-    for (let i = 0; i < 12; i++) {
-        const syllables = 2 + deps.randInt(2); // 2-3
-        let s = '';
-        for (let j = 0; j < syllables; j++) {
-            s += ONSETS[deps.randInt(ONSETS.length)] ?? 'k';
-            s += VOWELS[deps.randInt(VOWELS.length)] ?? 'a';
-            const coda = CODAS[deps.randInt(CODAS.length)] ?? '';
-            // Avoid overly long tokens by preferring empty coda on earlier syllables.
-            s += j === syllables - 1 ? coda : coda.length > 1 ? '' : coda;
+    // With the current arrays, minimum output is 4 chars ("baba") and maximum
+    // is ~14 chars, so the length check always passes on the first iteration.
+    const syllables = 2 + deps.randInt(2); // 2-3
+    let s = '';
+    for (let j = 0; j < syllables; j++) {
+        const onset = ONSETS[deps.randInt(ONSETS.length)];
+        const vowel = VOWELS[deps.randInt(VOWELS.length)];
+        const coda = CODAS[deps.randInt(CODAS.length)];
+        s += onset;
+        s += vowel;
+        // Avoid overly long tokens by preferring empty coda on earlier syllables.
+        if (j === syllables - 1) {
+            s += coda;
+        }
+        else if (coda.length <= 1) {
+            s += coda;
         }
-        if (s.length < 4 || s.length > 18)
-            continue;
-        if (opts.capitalize)
-            s = titleCase(s);
-        if (opts.includeNumber)
-            s += String(deps.randInt(10));
-        return s;
     }
-    // Fallback: still deterministic with deps.randInt in tests.
-    const fallback = `user${deps.randInt(1_000_000)}`;
-    return opts.capitalize ? titleCase(fallback) : fallback;
+    if (opts.capitalize)
+        s = titleCase(s);
+    if (opts.includeNumber)
+        s += String(deps.randInt(10));
+    return s;
 }
 function parseEmail(email) {
     const trimmed = email.trim();
@@ -126,7 +125,7 @@ export function generateUsername(input = {}, deps) {
             throw new Error('email is required for plus_addressed_email');
         }
         const { local, domain } = parseEmail(input.email);
-        const baseLocal = local.split('+')[0] ?? local;
+        const baseLocal = local.split('+')[0];
         return `${baseLocal}+${word}@${domain}`;
     }
     if (type === 'catch_all_email') {
@@ -136,7 +135,6 @@ export function generateUsername(input = {}, deps) {
         const domain = normalizeDomain(input.domain);
         return `${word}@${domain}`;
     }
-    // Exhaustive check.
-    const _never = type;
-    return _never;
+    // Exhaustive check — TypeScript errors here if a new type is added without a handler.
+    throw new Error(`Unsupported username generator type: ${type}`);
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "private": false,
   "name": "@icoretech/warden-mcp",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "type": "module",
   "license": "MIT",
   "description": "Vaultwarden/Bitwarden MCP server backed by Bitwarden CLI (bw).",
@@ -37,6 +37,7 @@
     "build": "tsc -p .",
     "start": "node dist/server.js",
     "test": "npm run build && node --test \"dist/**/*.test.js\"",
+    "test:coverage": "npm run build && node --test --experimental-test-coverage \"dist/**/*.test.js\"",
     "test:integration": "npm run build && node --test --test-timeout=45000 \"dist/integration/**/*.test.js\"",
     "test:session-regression": "node scripts/session-flood-regression.mjs",
     "lint": "biome check --write --assist-enabled=true . && tsc --noEmit"