@build-astron-co/nimbus 0.4.1 → 0.4.3

package/dist/src/watcher/index.js

@@ -39,8 +39,9 @@ export class FileWatcher extends EventEmitter {
     }
     /** Return all stored change events in chronological order. */
     getOrderedChanges() {
-        if (this.changesSize === 0)
+        if (this.changesSize === 0) {
             return [];
+        }
         const start = this.changesSize < this.maxChanges
             ? 0
             : this.changesHead; // oldest slot when buffer is full
@@ -169,12 +170,15 @@ export class FileWatcher extends EventEmitter {
             // Categorize for DevOps context
             let category = '';
             const lower = rel.toLowerCase();
-            if (lower.endsWith('.tf') || lower.endsWith('.tfvars'))
+            if (lower.endsWith('.tf') || lower.endsWith('.tfvars')) {
                 category = ' [terraform]';
-            else if (lower.endsWith('.yaml') || lower.endsWith('.yml'))
+            }
+            else if (lower.endsWith('.yaml') || lower.endsWith('.yml')) {
                 category = ' [yaml/k8s]';
-            else if (lower.includes('dockerfile'))
+            }
+            else if (lower.includes('dockerfile')) {
                 category = ' [docker]';
+            }
             return `  - ${rel}${category}`;
         });
         return `Files changed externally:\n${lines.join('\n')}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@build-astron-co/nimbus",
-  "version": "0.4.1",
+  "version": "0.4.3",
   "description": "AI-Powered DevOps Engineering Agent",
   "main": "dist/src/nimbus.js",
   "bin": {
@@ -8,13 +8,11 @@
   },
   "files": [
     "bin/",
-    "src/",
-    "dist/",
+    "dist/src/",
     "completions/",
     "README.md",
     "CHANGELOG.md",
-    "package.json",
-    "tsconfig.json"
+    "package.json"
   ],
   "publishConfig": {
     "access": "public"

package/src/__tests__/alias.test.ts

@@ -1,133 +0,0 @@
-/**
- * Alias Command Tests — L2
- *
- * Tests resolveAlias expansion and alias file management.
- */
-
-import { describe, test, expect, beforeEach, afterEach, vi } from 'vitest';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
-import * as os from 'node:os';
-
-// Patch homedir so aliases are stored in a temp dir during tests
-let tmpDir: string;
-
-vi.mock('node:os', async () => {
-  const actual = await vi.importActual<typeof os>('node:os');
-  return {
-    ...actual,
-    homedir: () => tmpDir ?? actual.homedir(),
-  };
-});
-
-// Re-import after mock is set up (dynamic to pick up the homedir mock)
-async function getAliasModule() {
-  // Force re-import so homedir mock is active
-  return await import('../commands/alias');
-}
-
-describe('resolveAlias (L2)', () => {
-  beforeEach(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'nimbus-alias-test-'));
-    // Clear module cache so homedir mock takes effect
-    vi.resetModules();
-  });
-
-  afterEach(() => {
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-    vi.resetModules();
-  });
-
-  test('returns args unchanged when no alias exists', async () => {
-    const { resolveAlias } = await getAliasModule();
-    expect(resolveAlias(['run', '--help'])).toEqual(['run', '--help']);
-  });
-
-  test('returns empty array for empty input', async () => {
-    const { resolveAlias } = await getAliasModule();
-    expect(resolveAlias([])).toEqual([]);
-  });
-
-  test('expands a defined alias', async () => {
-    // Write alias file manually
-    const nimbusDir = path.join(tmpDir, '.nimbus');
-    fs.mkdirSync(nimbusDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(nimbusDir, 'aliases.json'),
-      JSON.stringify({ deploy: 'run --auto-approve "deploy staging"' }),
-      'utf-8'
-    );
-
-    const { resolveAlias } = await getAliasModule();
-    const result = resolveAlias(['deploy']);
-    expect(result[0]).toBe('run');
-    expect(result).toContain('--auto-approve');
-  });
-
-  test('appends remaining args after expanding alias', async () => {
-    const nimbusDir = path.join(tmpDir, '.nimbus');
-    fs.mkdirSync(nimbusDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(nimbusDir, 'aliases.json'),
-      JSON.stringify({ tf: 'run "terraform plan"' }),
-      'utf-8'
-    );
-
-    const { resolveAlias } = await getAliasModule();
-    const result = resolveAlias(['tf', '--verbose']);
-    expect(result[result.length - 1]).toBe('--verbose');
-  });
-});
-
-describe('aliasCommand list/add/remove (L2)', () => {
-  beforeEach(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'nimbus-alias-cmd-test-'));
-    vi.resetModules();
-  });
-
-  afterEach(() => {
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-    vi.resetModules();
-  });
-
-  test('list shows "no aliases" when file is absent', async () => {
-    const { aliasCommand } = await getAliasModule();
-    const logs: string[] = [];
-    vi.spyOn(console, 'log').mockImplementation((...args) => logs.push(args.join(' ')));
-    // Should not throw even if file doesn't exist
-    await expect(aliasCommand('list', [])).resolves.not.toThrow();
-    vi.restoreAllMocks();
-  });
-
-  test('set writes alias to file', async () => {
-    const { aliasCommand } = await getAliasModule();
-    vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await aliasCommand('myalias=run "do something"', []);
-
-    const aliasFile = path.join(tmpDir, '.nimbus', 'aliases.json');
-    const data = JSON.parse(fs.readFileSync(aliasFile, 'utf-8')) as Record<string, string>;
-    expect(data['myalias']).toBe('run "do something"');
-    vi.restoreAllMocks();
-  });
-
-  test('remove deletes alias from file', async () => {
-    const nimbusDir = path.join(tmpDir, '.nimbus');
-    fs.mkdirSync(nimbusDir, { recursive: true });
-    fs.writeFileSync(
-      path.join(nimbusDir, 'aliases.json'),
-      JSON.stringify({ myalias: 'run stuff' }),
-      'utf-8'
-    );
-
-    const { aliasCommand } = await getAliasModule();
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await aliasCommand('remove', ['myalias']);
-
-    const data = JSON.parse(fs.readFileSync(path.join(nimbusDir, 'aliases.json'), 'utf-8')) as Record<string, string>;
-    expect(data['myalias']).toBeUndefined();
-    vi.restoreAllMocks();
-  });
-});

package/src/__tests__/app.test.ts

@@ -1,76 +0,0 @@
-/**
- * Tests for src/app.ts
- *
- * Verifies initApp() / shutdownApp() lifecycle semantics:
- * - initApp() returns an AppContext with db, router, and nimbusDir
- * - Calling initApp() twice returns the exact same context object (lazy singleton)
- * - shutdownApp() resets the cached context so getAppContext() returns null
- * - shutdownApp() is idempotent (safe to call when already shut down)
- */
-
-import { describe, it, expect, afterEach } from 'vitest';
-import { initApp, shutdownApp, getAppContext } from '../app';
-
-describe('app lifecycle', () => {
-  afterEach(async () => {
-    // Always clean up so that each test starts from a fresh state.
-    await shutdownApp();
-  });
-
-  it('initApp() returns an AppContext object', async () => {
-    const ctx = await initApp();
-    expect(ctx).toBeDefined();
-    expect(typeof ctx).toBe('object');
-  });
-
-  it('AppContext has a db property', async () => {
-    const ctx = await initApp();
-    expect(ctx.db).toBeDefined();
-  });
-
-  it('AppContext has a router property', async () => {
-    const ctx = await initApp();
-    expect(ctx.router).toBeDefined();
-  });
-
-  it('AppContext has a nimbusDir string property', async () => {
-    const ctx = await initApp();
-    expect(typeof ctx.nimbusDir).toBe('string');
-    expect(ctx.nimbusDir.length).toBeGreaterThan(0);
-  });
-
-  it('calling initApp() twice returns the same context instance', async () => {
-    const ctx1 = await initApp();
-    const ctx2 = await initApp();
-    expect(ctx1).toBe(ctx2);
-  });
-
-  it('getAppContext() returns null before initApp() is called', () => {
-    // After afterEach has run shutdownApp(), the context is null.
-    const ctx = getAppContext();
-    expect(ctx).toBeNull();
-  });
-
-  it('getAppContext() returns the context after initApp()', async () => {
-    await initApp();
-    const ctx = getAppContext();
-    expect(ctx).not.toBeNull();
-  });
-
-  it('shutdownApp() clears the context so getAppContext() returns null', async () => {
-    await initApp();
-    await shutdownApp();
-    expect(getAppContext()).toBeNull();
-  });
-
-  it('shutdownApp() is idempotent when called without a prior initApp()', async () => {
-    // Should not throw even though context is already null.
-    await expect(shutdownApp()).resolves.toBeUndefined();
-  });
-
-  it('shutdownApp() called twice does not throw', async () => {
-    await initApp();
-    await shutdownApp();
-    await expect(shutdownApp()).resolves.toBeUndefined();
-  });
-});