apcore-js 0.4.0 → 0.6.0

package/tests/test-middleware-manager.test.ts

@@ -1,185 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { Middleware } from '../src/middleware/base.js';
-import { MiddlewareManager, MiddlewareChainError } from '../src/middleware/manager.js';
-import { Context, createIdentity } from '../src/context.js';
-
-function makeContext(): Context {
-  return Context.create(null, createIdentity('test-user'));
-}
-
-class TaggingMiddleware extends Middleware {
-  readonly tag: string;
-
-  constructor(tag: string) {
-    super();
-    this.tag = tag;
-  }
-
-  override before(
-    _moduleId: string,
-    inputs: Record<string, unknown>,
-  ): Record<string, unknown> | null {
-    const trail = ((inputs['trail'] as string) ?? '') + this.tag;
-    return { ...inputs, trail };
-  }
-
-  override after(
-    _moduleId: string,
-    _inputs: Record<string, unknown>,
-    output: Record<string, unknown>,
-  ): Record<string, unknown> | null {
-    const trail = ((output['trail'] as string) ?? '') + this.tag;
-    return { ...output, trail };
-  }
-}
-
-class RecoveringMiddleware extends Middleware {
-  readonly recovery: Record<string, unknown>;
-
-  constructor(recovery: Record<string, unknown>) {
-    super();
-    this.recovery = recovery;
-  }
-
-  override onError(): Record<string, unknown> | null {
-    return this.recovery;
-  }
-}
-
-describe('MiddlewareManager', () => {
-  it('starts empty', () => {
-    const mgr = new MiddlewareManager();
-    expect(mgr.snapshot()).toEqual([]);
-  });
-
-  it('add and snapshot', () => {
-    const mgr = new MiddlewareManager();
-    const mw1 = new Middleware();
-    const mw2 = new Middleware();
-    mgr.add(mw1);
-    mgr.add(mw2);
-    expect(mgr.snapshot()).toHaveLength(2);
-  });
-
-  it('snapshot returns a copy', () => {
-    const mgr = new MiddlewareManager();
-    mgr.add(new Middleware());
-    const snap = mgr.snapshot();
-    snap.pop();
-    expect(mgr.snapshot()).toHaveLength(1);
-  });
-
-  it('remove by identity', () => {
-    const mgr = new MiddlewareManager();
-    const mw1 = new Middleware();
-    const mw2 = new Middleware();
-    mgr.add(mw1);
-    mgr.add(mw2);
-    expect(mgr.remove(mw1)).toBe(true);
-    expect(mgr.snapshot()).toEqual([mw2]);
-  });
-
-  it('remove returns false when not found', () => {
-    const mgr = new MiddlewareManager();
-    expect(mgr.remove(new Middleware())).toBe(false);
-  });
-
-  it('executeBefore runs in forward order', () => {
-    const mgr = new MiddlewareManager();
-    mgr.add(new TaggingMiddleware('A'));
-    mgr.add(new TaggingMiddleware('B'));
-    mgr.add(new TaggingMiddleware('C'));
-    const ctx = makeContext();
-    const [result, executed] = mgr.executeBefore('mod.test', { trail: '' }, ctx);
-    expect(result['trail']).toBe('ABC');
-    expect(executed).toHaveLength(3);
-  });
-
-  it('executeBefore passes original inputs when all return null', () => {
-    const mgr = new MiddlewareManager();
-    mgr.add(new Middleware());
-    mgr.add(new Middleware());
-    const ctx = makeContext();
-    const [result] = mgr.executeBefore('mod.test', { x: 42 }, ctx);
-    expect(result).toEqual({ x: 42 });
-  });
-
-  it('executeAfter runs in reverse order', () => {
-    const mgr = new MiddlewareManager();
-    mgr.add(new TaggingMiddleware('A'));
-    mgr.add(new TaggingMiddleware('B'));
-    mgr.add(new TaggingMiddleware('C'));
-    const ctx = makeContext();
-    const result = mgr.executeAfter('mod.test', {}, { trail: '' }, ctx);
-    expect(result['trail']).toBe('CBA');
-  });
-
-  it('executeAfter passes original output when all return null', () => {
-    const mgr = new MiddlewareManager();
-    mgr.add(new Middleware());
-    const ctx = makeContext();
-    const result = mgr.executeAfter('mod.test', {}, { y: 99 }, ctx);
-    expect(result).toEqual({ y: 99 });
-  });
-
-  it('executeOnError returns first non-null recovery (reverse order)', () => {
-    const mgr = new MiddlewareManager();
-    const mwA = new RecoveringMiddleware({ recovered: 'A' });
-    const mwB = new RecoveringMiddleware({ recovered: 'B' });
-    mgr.add(mwA);
-    mgr.add(mwB);
-    const ctx = makeContext();
-    const result = mgr.executeOnError('mod.test', {}, new Error('oops'), ctx, [mwA, mwB]);
-    expect(result).toEqual({ recovered: 'B' });
-  });
-
-  it('executeOnError returns null when no recovery', () => {
-    const mgr = new MiddlewareManager();
-    const mw = new Middleware();
-    mgr.add(mw);
-    const ctx = makeContext();
-    const result = mgr.executeOnError('mod.test', {}, new Error('oops'), ctx, [mw]);
-    expect(result).toBeNull();
-  });
-
-  it('executeOnError swallows errors in onError handlers', () => {
-    class ThrowingOnError extends Middleware {
-      override onError(): Record<string, unknown> | null {
-        throw new Error('onError also failed');
-      }
-    }
-    const mgr = new MiddlewareManager();
-    const mwRecover = new RecoveringMiddleware({ safe: true });
-    const mwThrow = new ThrowingOnError();
-    mgr.add(mwRecover);
-    mgr.add(mwThrow);
-    const ctx = makeContext();
-    const result = mgr.executeOnError('mod.test', {}, new Error('original'), ctx, [mwRecover, mwThrow]);
-    expect(result).toEqual({ safe: true });
-  });
-
-  it('MiddlewareChainError wraps before() failure', () => {
-    class FailingBefore extends Middleware {
-      override before(): Record<string, unknown> | null {
-        throw new Error('before exploded');
-      }
-    }
-    const mgr = new MiddlewareManager();
-    const ok = new TaggingMiddleware('A');
-    const fail = new FailingBefore();
-    mgr.add(ok);
-    mgr.add(fail);
-    const ctx = makeContext();
-
-    let caught: MiddlewareChainError | undefined;
-    try {
-      mgr.executeBefore('mod.test', { trail: '' }, ctx);
-    } catch (e) {
-      caught = e as MiddlewareChainError;
-    }
-
-    expect(caught).toBeInstanceOf(MiddlewareChainError);
-    expect(caught!.original.message).toBe('before exploded');
-    expect(caught!.executedMiddlewares).toHaveLength(2);
-  });
-});

package/tests/test-middleware.test.ts

@@ -1,86 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { Middleware } from '../src/middleware/base.js';
-import { BeforeMiddleware, AfterMiddleware } from '../src/middleware/adapters.js';
-import { Context, createIdentity } from '../src/context.js';
-
-function makeContext(): Context {
-  return Context.create(null, createIdentity('test-user'));
-}
-
-describe('Middleware base class', () => {
-  it('before() returns null by default', () => {
-    const mw = new Middleware();
-    const ctx = makeContext();
-    expect(mw.before('mod.a', { x: 1 }, ctx)).toBeNull();
-  });
-
-  it('after() returns null by default', () => {
-    const mw = new Middleware();
-    const ctx = makeContext();
-    expect(mw.after('mod.a', { x: 1 }, { y: 2 }, ctx)).toBeNull();
-  });
-
-  it('onError() returns null by default', () => {
-    const mw = new Middleware();
-    const ctx = makeContext();
-    expect(mw.onError('mod.a', { x: 1 }, new Error('boom'), ctx)).toBeNull();
-  });
-});
-
-describe('BeforeMiddleware', () => {
-  it('wraps a callback and delegates to before()', () => {
-    const mw = new BeforeMiddleware((moduleId, inputs) => {
-      return { ...inputs, injected: moduleId };
-    });
-    const ctx = makeContext();
-    const result = mw.before('mod.x', { a: 1 }, ctx);
-    expect(result).toEqual({ a: 1, injected: 'mod.x' });
-  });
-
-  it('after() still returns null', () => {
-    const mw = new BeforeMiddleware(() => ({ replaced: true }));
-    const ctx = makeContext();
-    expect(mw.after('mod.x', {}, {}, ctx)).toBeNull();
-  });
-
-  it('onError() still returns null', () => {
-    const mw = new BeforeMiddleware(() => ({ replaced: true }));
-    const ctx = makeContext();
-    expect(mw.onError('mod.x', {}, new Error('fail'), ctx)).toBeNull();
-  });
-
-  it('can return null from callback', () => {
-    const mw = new BeforeMiddleware(() => null);
-    const ctx = makeContext();
-    expect(mw.before('mod.x', { a: 1 }, ctx)).toBeNull();
-  });
-});
-
-describe('AfterMiddleware', () => {
-  it('wraps a callback and delegates to after()', () => {
-    const mw = new AfterMiddleware((moduleId, _inputs, output) => {
-      return { ...output, processedBy: moduleId };
-    });
-    const ctx = makeContext();
-    const result = mw.after('mod.y', { a: 1 }, { out: 42 }, ctx);
-    expect(result).toEqual({ out: 42, processedBy: 'mod.y' });
-  });
-
-  it('before() still returns null', () => {
-    const mw = new AfterMiddleware(() => ({ replaced: true }));
-    const ctx = makeContext();
-    expect(mw.before('mod.y', {}, ctx)).toBeNull();
-  });
-
-  it('onError() still returns null', () => {
-    const mw = new AfterMiddleware(() => ({ replaced: true }));
-    const ctx = makeContext();
-    expect(mw.onError('mod.y', {}, new Error('fail'), ctx)).toBeNull();
-  });
-
-  it('can return null from callback', () => {
-    const mw = new AfterMiddleware(() => null);
-    const ctx = makeContext();
-    expect(mw.after('mod.y', {}, { out: 1 }, ctx)).toBeNull();
-  });
-});

package/tests/utils/test-pattern.test.ts

@@ -1,109 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { matchPattern } from '../../src/utils/pattern.js';
-
-describe('matchPattern – exact and trivial patterns', () => {
-  it('returns true when pattern is the lone wildcard *', () => {
-    expect(matchPattern('*', 'anything')).toBe(true);
-    expect(matchPattern('*', '')).toBe(true);
-  });
-
-  it('returns true when pattern has no wildcard and equals moduleId', () => {
-    expect(matchPattern('module.a', 'module.a')).toBe(true);
-  });
-
-  it('returns false when pattern has no wildcard and does not equal moduleId', () => {
-    expect(matchPattern('module.a', 'module.b')).toBe(false);
-  });
-});
-
-describe('matchPattern – prefix wildcard (pattern starts with non-*)', () => {
-  it('returns false when moduleId does not start with the prefix segment', () => {
-    // Pattern split: ['core.', ''] – prefix check fails immediately
-    expect(matchPattern('core.*', 'other.module')).toBe(false);
-  });
-
-  it('returns true when moduleId starts with the prefix segment', () => {
-    expect(matchPattern('core.*', 'core.auth')).toBe(true);
-  });
-});
-
-describe('matchPattern – suffix wildcard (pattern does not end with *)', () => {
-  it('returns false when moduleId does not end with the last segment (lines 26-27)', () => {
-    // Pattern 'a*z': segments = ['a', 'z']. moduleId 'a.b.c' starts with 'a',
-    // then the endsWith check for 'z' fails.
-    expect(matchPattern('a*z', 'a.b.c')).toBe(false);
-  });
-
-  it('returns true when moduleId ends with the last segment (lines 25-27 pass)', () => {
-    // Pattern 'a*c': segments = ['a', 'c']. moduleId 'a.b.c' starts with 'a'
-    // and ends with 'c'.
-    expect(matchPattern('a*c', 'a.b.c')).toBe(true);
-  });
-});
-
-describe('matchPattern – multi-segment wildcard (lines 20-23)', () => {
-  it('returns true when all middle segments are found in order (lines 20-23 exercised)', () => {
-    // Pattern 'a*b*c': segments = ['a', 'b', 'c'].
-    // moduleId 'aXXbYYc': startsWith 'a', then indexOf('b') found, then endsWith 'c'.
-    expect(matchPattern('a*b*c', 'aXXbYYc')).toBe(true);
-  });
-
-  it('returns false when a middle segment is not found – idx === -1 (line 21)', () => {
-    // Pattern 'a*MISSING*c': the substring 'MISSING' is absent in 'aXXbYYc'.
-    expect(matchPattern('a*MISSING*c', 'aXXbYYc')).toBe(false);
-  });
-
-  it('returns false when the second middle segment is not found after the first', () => {
-    // Pattern 'a*b*z': 'b' is found in 'aXXbYYc', but 'z' is not found after it.
-    expect(matchPattern('a*b*z', 'aXXbYYc')).toBe(false);
-  });
-
-  it('advances pos correctly so segments must appear in sequence, not out of order', () => {
-    // 'b' appears before 'X' in the string, so 'X' cannot be found *after* 'b'.
-    // Pattern 'a*b*X': 'b' at index 3, then indexOf('X', 4) returns -1.
-    expect(matchPattern('a*b*X', 'aXXbYYc')).toBe(false);
-  });
-});
-
-describe('matchPattern – pattern starting with * (no prefix check)', () => {
-  it('returns true when moduleId ends with the suffix of a leading-* pattern', () => {
-    // Pattern '*suffix': segments = ['', 'suffix']. The startsWith block is skipped.
-    // Loop i=1: 'suffix' is found in 'prefix.suffix'. Pattern ends with 'suffix'
-    // so endsWith check also passes.
-    expect(matchPattern('*suffix', 'prefix.suffix')).toBe(true);
-  });
-
-  it('returns false when moduleId does not end with the suffix', () => {
-    expect(matchPattern('*suffix', 'prefix.other')).toBe(false);
-  });
-
-  it('returns true for *-only-surrounded pattern when middle is found anywhere', () => {
-    // Pattern '*mid*': segments = ['', 'mid', '']. startsWith skipped (starts with *).
-    // Loop i=1: 'mid' found. Loop i=2: empty segment, continue. Pattern ends with *,
-    // so endsWith block is skipped. Returns true.
-    expect(matchPattern('*mid*', 'before.mid.after')).toBe(true);
-  });
-});
-
-describe('matchPattern – consecutive wildcards (empty middle segments, line 19)', () => {
-  it('returns true for a**b when moduleId starts with a and ends with b (empty segment skipped)', () => {
-    // Pattern 'a**b': segments = ['a', '', 'b']. Prefix 'a' matches.
-    // Loop i=1: segment '' → continue (line 19). Loop i=2: 'b' is found.
-    // endsWith 'b' check passes.
-    expect(matchPattern('a**b', 'aXYb')).toBe(true);
-  });
-
-  it('returns false for a**b when moduleId does not start with a', () => {
-    expect(matchPattern('a**b', 'zXYb')).toBe(false);
-  });
-
-  it('returns false for a**b when moduleId does not end with b', () => {
-    expect(matchPattern('a**b', 'aXYz')).toBe(false);
-  });
-
-  it('handles three consecutive wildcards – all middle segments are empty and skipped', () => {
-    // Pattern 'x***y': segments = ['x', '', '', 'y'].
-    expect(matchPattern('x***y', 'xABCy')).toBe(true);
-    expect(matchPattern('x***y', 'xABCz')).toBe(false);
-  });
-});

package/tsconfig.build.json

@@ -1,8 +0,0 @@
-{
-  "extends": "./tsconfig.json",
-  "compilerOptions": {
-    "rootDir": "./src"
-  },
-  "include": ["src/**/*.ts"],
-  "exclude": ["node_modules", "dist", "tests", "**/*.test.ts"]
-}

package/tsconfig.json

@@ -1,20 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "lib": ["ES2022"],
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "outDir": "./dist",
-    "resolveJsonModule": true,
-    "isolatedModules": true
-  },
-  "include": ["src/**/*.ts", "tests/**/*.ts"],
-  "exclude": ["node_modules", "dist"]
-}

package/vitest.config.ts

@@ -1,18 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-export default defineConfig({
-  test: {
-    globals: true,
-    coverage: {
-      provider: 'v8',
-      include: ['src/**/*.ts'],
-      exclude: ['src/**/index.ts'],
-      thresholds: {
-        lines: 90,
-        functions: 90,
-        branches: 90,
-        statements: 90,
-      },
-    },
-  },
-});