@moxxy/agent 0.2.1 → 0.2.3

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAK/C,MAAM,WAAW,qBAAqB;IACpC;oDACgD;IAChD,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,gFAAgF;IAChF,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,mBAAoB,SAAQ,qBAAqB;IAChE,gFAAgF;IAChF,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,CAAC;CAC3B;AAOD;;;GAGG;AACH,wBAAgB,YAAY,CAAC,IAAI,GAAE,mBAAwB,GAAG,WAAW,CAYxE;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,GAAE,qBAA0B,GAAG,WAAW,CAW7E;AAGD,OAAO,EACL,UAAU,EACV,KAAK,KAAK,EACV,KAAK,WAAW,EAChB,KAAK,iBAAiB,EACtB,KAAK,UAAU,EACf,KAAK,MAAM,EACX,KAAK,OAAO,EACZ,KAAK,kBAAkB,GACxB,MAAM,aAAa,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAK/C,MAAM,WAAW,qBAAqB;IACpC;oDACgD;IAChD,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,gFAAgF;IAChF,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,mBAAoB,SAAQ,qBAAqB;IAChE,gFAAgF;IAChF,QAAQ,CAAC,OAAO,CAAC,EAAE,MAAM,CAAC;CAC3B;AAmBD;;;GAGG;AACH,wBAAgB,YAAY,CAAC,IAAI,GAAE,mBAAwB,GAAG,WAAW,CAYxE;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,GAAE,qBAA0B,GAAG,WAAW,CAW7E;AAGD,OAAO,EACL,UAAU,EACV,KAAK,KAAK,EACV,KAAK,WAAW,EAChB,KAAK,iBAAiB,EACtB,KAAK,UAAU,EACf,KAAK,MAAM,EACX,KAAK,OAAO,EACZ,KAAK,kBAAkB,GACxB,MAAM,aAAa,CAAC"}

package/dist/index.js

@@ -19,9 +19,17 @@
 import defaultModePlugin from '@moxxy/mode-default';
 import openaiPlugin from '@moxxy/plugin-provider-openai';
 import anthropicPlugin from '@moxxy/plugin-provider-anthropic';
-/** Drop `undefined` config fields so they don't override provider defaults. */
+/**
+ * Drop absent config fields so they don't override provider defaults.
+ * Blank strings count as absent: an unset-but-present env var (`OPENAI_API_KEY=`,
+ * or a templated `.env` leaving `OPENAI_API_KEY="   "`) would otherwise freeze
+ * `apiKey: '   '`, defeating the provider's own env fallback (`'…' ?? x`
+ * short-circuits to the blank string) and turning a clear no-key path into an
+ * opaque 401 at the first turn. Trimming here keeps the no-key path live and
+ * mirrors the providers' own `key.trim()` validation.
+ */
 function clean(config) {
-    return Object.fromEntries(Object.entries(config).filter(([, v]) => v !== undefined));
+    return Object.fromEntries(Object.entries(config).filter(([, v]) => v !== undefined && !(typeof v === 'string' && v.trim() === '')));
 }
 /**
  * One-shop OpenAI (and OpenAI-compatible) preset: the default loop + the OpenAI

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAGH,OAAO,iBAAiB,MAAM,qBAAqB,CAAC;AACpD,OAAO,YAAY,MAAM,+BAA+B,CAAC;AACzD,OAAO,eAAe,MAAM,kCAAkC,CAAC;AAe/D,+EAA+E;AAC/E,SAAS,KAAK,CAAC,MAA+B;IAC5C,OAAO,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC;AACvF,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,OAA4B,EAAE;IACzD,OAAO;QACL,OAAO,EAAE,CAAC,iBAAiB,EAAE,YAAY,CAAC;QAC1C,QAAQ,EAAE;YACR,IAAI,EAAE,QAAQ;YACd,MAAM,EAAE,KAAK,CAAC;gBACZ,MAAM,EAAE,IAAI,CAAC,MAAM,IAAI,OAAO,CAAC,GAAG,CAAC,cAAc;gBACjD,KAAK,EAAE,IAAI,CAAC,KAAK;gBACjB,OAAO,EAAE,IAAI,CAAC,OAAO;aACtB,CAAC;SACH;KACF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,OAA8B,EAAE;IAC9D,OAAO;QACL,OAAO,EAAE,CAAC,iBAAiB,EAAE,eAAe,CAAC;QAC7C,QAAQ,EAAE;YACR,IAAI,EAAE,WAAW;YACjB,MAAM,EAAE,KAAK,CAAC;gBACZ,MAAM,EAAE,IAAI,CAAC,MAAM,IAAI,OAAO,CAAC,GAAG,CAAC,iBAAiB;gBACpD,KAAK,EAAE,IAAI,CAAC,KAAK;aAClB,CAAC;SACH;KACF,CAAC;AACJ,CAAC;AAED,4EAA4E;AAC5E,OAAO,EACL,UAAU,GAQX,MAAM,aAAa,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAGH,OAAO,iBAAiB,MAAM,qBAAqB,CAAC;AACpD,OAAO,YAAY,MAAM,+BAA+B,CAAC;AACzD,OAAO,eAAe,MAAM,kCAAkC,CAAC;AAe/D;;;;;;;;GAQG;AACH,SAAS,KAAK,CAAC,MAA+B;IAC5C,OAAO,MAAM,CAAC,WAAW,CACvB,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,MAAM,CAC3B,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,SAAS,IAAI,CAAC,CAAC,OAAO,CAAC,KAAK,QAAQ,IAAI,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAC1E,CACF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,OAA4B,EAAE;IACzD,OAAO;QACL,OAAO,EAAE,CAAC,iBAAiB,EAAE,YAAY,CAAC;QAC1C,QAAQ,EAAE;YACR,IAAI,EAAE,QAAQ;YACd,MAAM,EAAE,KAAK,CAAC;gBACZ,MAAM,EAAE,IAAI,CAAC,MAAM,IAAI,OAAO,CAAC,GAAG,CAAC,cAAc;gBACjD,KAAK,EAAE,IAAI,CAAC,KAAK;gBACjB,OAAO,EAAE,IAAI,CAAC,OAAO;aACtB,CAAC;SACH;KACF,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,OAA8B,EAAE;IAC9D,OAAO;QACL,OAAO,EAAE,CAAC,iBAAiB,EAAE,eAAe,CAAC;QAC7C,QAAQ,EAAE;YACR,IAAI,EAAE,WAAW;YACjB,MAAM,EAAE,KAAK,CAAC;gBACZ,MAAM,EAAE,IAAI,CAAC,MAAM,IAAI,OAAO,CAAC,GAAG,CAAC,iBAAiB;gBACpD,KAAK,EAAE,IAAI,CAAC,KAAK;aAClB,CAAC;SACH;KACF,CAAC;AACJ,CAAC;AAED,4EAA4E;AAC5E,OAAO,EACL,UAAU,GAQX,MAAM,aAAa,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moxxy/agent",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "description": "Batteries-included entry to moxxy: one install + one call to run an agent. Bundles @moxxy/core, the default loop, and the OpenAI + Anthropic providers behind drop-in presets — `setupAgent(openaiPreset({ apiKey }))`.",
   "keywords": [
     "moxxy",
@@ -39,11 +39,11 @@
     "src"
   ],
   "dependencies": {
-    "@moxxy/mode-default": "0.1.0",
-    "@moxxy/core": "0.4.0",
-    "@moxxy/sdk": "0.15.0",
-    "@moxxy/plugin-provider-openai": "0.1.0",
-    "@moxxy/plugin-provider-anthropic": "0.2.0"
+    "@moxxy/core": "0.5.1",
+    "@moxxy/mode-default": "0.1.1",
+    "@moxxy/sdk": "0.15.1",
+    "@moxxy/plugin-provider-openai": "0.1.1",
+    "@moxxy/plugin-provider-anthropic": "0.2.1"
   },
   "devDependencies": {
     "@types/node": "^22.10.0",

package/src/index.test.ts

@@ -3,9 +3,12 @@ import { afterEach, describe, expect, it } from 'vitest';
 import { setupAgent, openaiPreset, anthropicPreset } from './index.js';
 
 const ORIGINAL_OPENAI = process.env.OPENAI_API_KEY;
+const ORIGINAL_ANTHROPIC = process.env.ANTHROPIC_API_KEY;
 afterEach(() => {
   if (ORIGINAL_OPENAI === undefined) delete process.env.OPENAI_API_KEY;
   else process.env.OPENAI_API_KEY = ORIGINAL_OPENAI;
+  if (ORIGINAL_ANTHROPIC === undefined) delete process.env.ANTHROPIC_API_KEY;
+  else process.env.ANTHROPIC_API_KEY = ORIGINAL_ANTHROPIC;
 });
 
 describe('@moxxy/agent presets', () => {
@@ -39,4 +42,59 @@ describe('@moxxy/agent presets', () => {
     expect(openaiPreset().provider?.config?.apiKey).toBe('env-key');
     expect(openaiPreset({ apiKey: 'explicit' }).provider?.config?.apiKey).toBe('explicit');
   });
+
+  it('passes baseURL and model through to provider.config', () => {
+    const config = openaiPreset({ apiKey: 'sk-test', model: 'x', baseURL: 'y' }).provider?.config;
+    expect(config?.model).toBe('x');
+    expect(config?.baseURL).toBe('y');
+  });
+
+  it('omits absent fields entirely (not as undefined) so provider defaults win', () => {
+    delete process.env.OPENAI_API_KEY;
+    const config = openaiPreset().provider?.config ?? {};
+    // Absent keys must not exist at all — a present `undefined`/`''` would
+    // override the provider's own catalog/env fallback.
+    expect('model' in config).toBe(false);
+    expect('baseURL' in config).toBe(false);
+    expect('apiKey' in config).toBe(false);
+  });
+
+  it('treats an empty-string env var as absent so the provider fallback survives', () => {
+    process.env.OPENAI_API_KEY = '';
+    const config = openaiPreset().provider?.config ?? {};
+    // `OPENAI_API_KEY=` must not freeze apiKey:'' (which `'' ?? x` would keep,
+    // defeating the provider's lazy env fallback and yielding an opaque 401).
+    expect('apiKey' in config).toBe(false);
+  });
+
+  it('treats an explicit empty-string apiKey/model/baseURL as absent', () => {
+    delete process.env.OPENAI_API_KEY;
+    const config = openaiPreset({ apiKey: '', model: '', baseURL: '' }).provider?.config ?? {};
+    expect('apiKey' in config).toBe(false);
+    expect('model' in config).toBe(false);
+    expect('baseURL' in config).toBe(false);
+  });
+
+  it('treats a whitespace-only apiKey/env as absent so the provider fallback survives', () => {
+    // A templated/whitespace env var (`OPENAI_API_KEY="   "`) or an explicit blank
+    // key must NOT freeze a blank apiKey — `' ' ?? x` would short-circuit and hand
+    // the SDK a non-credential, yielding an opaque 401 instead of the no-key path.
+    process.env.OPENAI_API_KEY = '   ';
+    expect('apiKey' in (openaiPreset().provider?.config ?? {})).toBe(false);
+    delete process.env.OPENAI_API_KEY;
+    const cfg = openaiPreset({ apiKey: '  \t\n', model: ' ', baseURL: '\t' }).provider?.config ?? {};
+    expect('apiKey' in cfg).toBe(false);
+    expect('model' in cfg).toBe(false);
+    expect('baseURL' in cfg).toBe(false);
+  });
+
+  it('anthropicPreset: env fallback, and blank apiKey/env treated as absent', () => {
+    process.env.ANTHROPIC_API_KEY = 'env-ant';
+    expect(anthropicPreset().provider?.config?.apiKey).toBe('env-ant');
+    expect(anthropicPreset({ apiKey: 'explicit' }).provider?.config?.apiKey).toBe('explicit');
+    process.env.ANTHROPIC_API_KEY = '  ';
+    expect('apiKey' in (anthropicPreset().provider?.config ?? {})).toBe(false);
+    delete process.env.ANTHROPIC_API_KEY;
+    expect('apiKey' in (anthropicPreset({ apiKey: '' }).provider?.config ?? {})).toBe(false);
+  });
 });

package/src/index.ts

@@ -35,9 +35,21 @@ export interface OpenAIPresetOptions extends ProviderPresetOptions {
   readonly baseURL?: string;
 }
 
-/** Drop `undefined` config fields so they don't override provider defaults. */
+/**
+ * Drop absent config fields so they don't override provider defaults.
+ * Blank strings count as absent: an unset-but-present env var (`OPENAI_API_KEY=`,
+ * or a templated `.env` leaving `OPENAI_API_KEY="   "`) would otherwise freeze
+ * `apiKey: '   '`, defeating the provider's own env fallback (`'…' ?? x`
+ * short-circuits to the blank string) and turning a clear no-key path into an
+ * opaque 401 at the first turn. Trimming here keeps the no-key path live and
+ * mirrors the providers' own `key.trim()` validation.
+ */
 function clean(config: Record<string, unknown>): Record<string, unknown> {
-  return Object.fromEntries(Object.entries(config).filter(([, v]) => v !== undefined));
+  return Object.fromEntries(
+    Object.entries(config).filter(
+      ([, v]) => v !== undefined && !(typeof v === 'string' && v.trim() === ''),
+    ),
+  );
 }
 
 /**