@objectstack/cli 4.0.4 → 4.0.5

package/test/plugin.test.ts

@@ -1,169 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import PluginList from '../src/commands/plugin/list';
-import PluginInfo from '../src/commands/plugin/info';
-import PluginAdd from '../src/commands/plugin/add';
-import PluginRemove from '../src/commands/plugin/remove';
-import fs from 'fs';
-import path from 'path';
-import os from 'os';
-
-describe('Plugin Commands (oclif)', () => {
-  it('should have plugin list command', () => {
-    expect(PluginList.description).toContain('List');
-  });
-
-  it('should have plugin list alias', () => {
-    expect(PluginList.aliases).toContain('plugin ls');
-  });
-
-  it('should have plugin info command', () => {
-    expect(PluginInfo.description).toContain('information');
-  });
-
-  it('should have plugin add command', () => {
-    expect(PluginAdd.description).toContain('Add');
-  });
-
-  it('should have plugin remove command', () => {
-    expect(PluginRemove.description).toContain('Remove');
-  });
-
-  it('should have plugin remove alias', () => {
-    expect(PluginRemove.aliases).toContain('plugin rm');
-  });
-});
-
-describe('Plugin Config Manipulation', () => {
-  let tmpDir: string;
-  let configPath: string;
-
-  beforeEach(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'os-plugin-test-'));
-    configPath = path.join(tmpDir, 'objectstack.config.ts');
-  });
-
-  afterEach(() => {
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-
-  describe('addPluginToConfig (via file manipulation)', () => {
-    it('should add plugin import and entry to config with existing plugins array', () => {
-      fs.writeFileSync(configPath, `import { defineStack } from '@objectstack/spec';
-
-export default defineStack({
-  manifest: {
-    name: 'test-app',
-    version: '1.0.0',
-  },
-  plugins: [
-  ],
-});
-`);
-
-      // Simulate what the add command does
-      let content = fs.readFileSync(configPath, 'utf-8');
-      const packageName = '@objectstack/plugin-auth';
-      const varName = 'authPlugin';
-      const importLine = `import ${varName} from '${packageName}';\n`;
-      
-      // Add import after last import
-      const importRegex = /^import .+$/gm;
-      let lastImportEnd = 0;
-      let match: RegExpExecArray | null;
-      while ((match = importRegex.exec(content)) !== null) {
-        lastImportEnd = match.index + match[0].length;
-      }
-      content = content.slice(0, lastImportEnd) + '\n' + importLine + content.slice(lastImportEnd);
-      
-      // Add to plugins array
-      content = content.replace(
-        /(plugins\s*:\s*\[)/,
-        `$1\n    ${varName},`
-      );
-
-      fs.writeFileSync(configPath, content);
-      
-      const result = fs.readFileSync(configPath, 'utf-8');
-      expect(result).toContain("import authPlugin from '@objectstack/plugin-auth'");
-      expect(result).toContain('authPlugin,');
-    });
-
-    it('should add plugin to config without existing plugins array', () => {
-      fs.writeFileSync(configPath, `import { defineStack } from '@objectstack/spec';
-
-export default defineStack({
-  manifest: {
-    name: 'test-app',
-    version: '1.0.0',
-  },
-});
-`);
-
-      let content = fs.readFileSync(configPath, 'utf-8');
-      const packageName = '@objectstack/plugin-security';
-      const varName = 'securityPlugin';
-      const importLine = `import ${varName} from '${packageName}';\n`;
-
-      // Add import
-      const importRegex = /^import .+$/gm;
-      let lastImportEnd = 0;
-      let match: RegExpExecArray | null;
-      while ((match = importRegex.exec(content)) !== null) {
-        lastImportEnd = match.index + match[0].length;
-      }
-      content = content.slice(0, lastImportEnd) + '\n' + importLine + content.slice(lastImportEnd);
-
-      // Add plugins array
-      if (!/plugins\s*:\s*\[/.test(content)) {
-        content = content.replace(
-          /(defineStack\(\{[\s\S]*?)(}\s*\))/,
-          `$1  plugins: [\n    ${varName},\n  ],\n$2`
-        );
-      }
-
-      fs.writeFileSync(configPath, content);
-
-      const result = fs.readFileSync(configPath, 'utf-8');
-      expect(result).toContain("import securityPlugin from '@objectstack/plugin-security'");
-      expect(result).toContain('plugins: [');
-      expect(result).toContain('securityPlugin,');
-    });
-  });
-
-  describe('removePluginFromConfig (via file manipulation)', () => {
-    it('should remove plugin import and entry from config', () => {
-      fs.writeFileSync(configPath, `import { defineStack } from '@objectstack/spec';
-import authPlugin from '@objectstack/plugin-auth';
-
-export default defineStack({
-  manifest: {
-    name: 'test-app',
-    version: '1.0.0',
-  },
-  plugins: [
-    authPlugin,
-  ],
-});
-`);
-
-      let content = fs.readFileSync(configPath, 'utf-8');
-      const packageName = '@objectstack/plugin-auth';
-      
-      // Remove import
-      const importRegex = new RegExp(`^import .+['"]${packageName.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}['"].*$\\n?`, 'gm');
-      content = content.replace(importRegex, '');
-
-      // Remove from plugins array
-      content = content.replace(/\s*authPlugin,?\n?/g, '\n');
-
-      fs.writeFileSync(configPath, content);
-
-      const result = fs.readFileSync(configPath, 'utf-8');
-      expect(result).not.toContain('@objectstack/plugin-auth');
-      expect(result).not.toContain('authPlugin');
-      // The rest of the config should remain intact
-      expect(result).toContain("import { defineStack } from '@objectstack/spec'");
-      expect(result).toContain('manifest');
-    });
-  });
-});

package/test/remote-api-commands.test.ts

@@ -1,188 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import AuthLogin from '../src/commands/auth/login';
-import AuthLogout from '../src/commands/auth/logout';
-import AuthWhoami from '../src/commands/auth/whoami';
-import DataQuery from '../src/commands/data/query';
-import DataGet from '../src/commands/data/get';
-import DataCreate from '../src/commands/data/create';
-import DataUpdate from '../src/commands/data/update';
-import DataDelete from '../src/commands/data/delete';
-import MetaList from '../src/commands/meta/list';
-import MetaGet from '../src/commands/meta/get';
-import MetaRegister from '../src/commands/meta/register';
-import MetaDelete from '../src/commands/meta/delete';
-
-describe('Remote API Commands (oclif)', () => {
-  describe('Auth Commands', () => {
-    it('should have auth login command', () => {
-      expect(AuthLogin.description).toContain('Authenticate');
-      expect(AuthLogin.flags).toHaveProperty('url');
-      expect(AuthLogin.flags).toHaveProperty('email');
-      expect(AuthLogin.flags).toHaveProperty('password');
-      expect(AuthLogin.flags).toHaveProperty('json');
-    });
-
-    it('should have auth logout command', () => {
-      expect(AuthLogout.description).toContain('Clear');
-      expect(AuthLogout.flags).toHaveProperty('json');
-    });
-
-    it('should have auth whoami command', () => {
-      expect(AuthWhoami.description).toContain('session');
-      expect(AuthWhoami.flags).toHaveProperty('url');
-      expect(AuthWhoami.flags).toHaveProperty('token');
-      expect(AuthWhoami.flags).toHaveProperty('format');
-    });
-
-    it('auth commands should have examples', () => {
-      expect(AuthLogin.examples).toBeDefined();
-      expect(AuthLogin.examples.length).toBeGreaterThan(0);
-      expect(AuthLogout.examples).toBeDefined();
-      expect(AuthWhoami.examples).toBeDefined();
-    });
-  });
-
-  describe('Data Commands', () => {
-    it('should have data query command', () => {
-      expect(DataQuery.description).toContain('Query');
-      expect(DataQuery.args).toHaveProperty('object');
-      expect(DataQuery.flags).toHaveProperty('filter');
-      expect(DataQuery.flags).toHaveProperty('fields');
-      expect(DataQuery.flags).toHaveProperty('sort');
-      expect(DataQuery.flags).toHaveProperty('limit');
-      expect(DataQuery.flags).toHaveProperty('offset');
-      expect(DataQuery.flags).toHaveProperty('format');
-    });
-
-    it('should have data get command', () => {
-      expect(DataGet.description).toContain('single record');
-      expect(DataGet.args).toHaveProperty('object');
-      expect(DataGet.args).toHaveProperty('id');
-      expect(DataGet.flags).toHaveProperty('format');
-    });
-
-    it('should have data create command', () => {
-      expect(DataCreate.description).toContain('Create');
-      expect(DataCreate.args).toHaveProperty('object');
-      expect(DataCreate.flags).toHaveProperty('data');
-      expect(DataCreate.flags).toHaveProperty('format');
-    });
-
-    it('should have data update command', () => {
-      expect(DataUpdate.description).toContain('Update');
-      expect(DataUpdate.args).toHaveProperty('object');
-      expect(DataUpdate.args).toHaveProperty('id');
-      expect(DataUpdate.flags).toHaveProperty('data');
-      expect(DataUpdate.flags).toHaveProperty('format');
-    });
-
-    it('should have data delete command', () => {
-      expect(DataDelete.description).toContain('Delete');
-      expect(DataDelete.args).toHaveProperty('object');
-      expect(DataDelete.args).toHaveProperty('id');
-      expect(DataDelete.flags).toHaveProperty('format');
-    });
-
-    it('data commands should support common flags', () => {
-      const commands = [DataQuery, DataGet, DataCreate, DataUpdate, DataDelete];
-      commands.forEach(cmd => {
-        expect(cmd.flags).toHaveProperty('url');
-        expect(cmd.flags).toHaveProperty('token');
-      });
-    });
-
-    it('data commands should have examples', () => {
-      expect(DataQuery.examples).toBeDefined();
-      expect(DataQuery.examples.length).toBeGreaterThan(0);
-      expect(DataGet.examples).toBeDefined();
-      expect(DataCreate.examples).toBeDefined();
-      expect(DataUpdate.examples).toBeDefined();
-      expect(DataDelete.examples).toBeDefined();
-    });
-  });
-
-  describe('Metadata Commands', () => {
-    it('should have meta list command', () => {
-      expect(MetaList.description).toContain('List metadata');
-      expect(MetaList.args).toHaveProperty('type');
-      expect(MetaList.flags).toHaveProperty('format');
-    });
-
-    it('should have meta get command', () => {
-      expect(MetaGet.description).toContain('Get');
-      expect(MetaGet.args).toHaveProperty('type');
-      expect(MetaGet.args).toHaveProperty('name');
-      expect(MetaGet.flags).toHaveProperty('format');
-    });
-
-    it('should have meta register command', () => {
-      expect(MetaRegister.description).toContain('Register');
-      expect(MetaRegister.args).toHaveProperty('type');
-      expect(MetaRegister.flags).toHaveProperty('data');
-      expect(MetaRegister.flags).toHaveProperty('format');
-    });
-
-    it('should have meta delete command', () => {
-      expect(MetaDelete.description).toContain('Delete');
-      expect(MetaDelete.args).toHaveProperty('type');
-      expect(MetaDelete.args).toHaveProperty('name');
-      expect(MetaDelete.flags).toHaveProperty('format');
-    });
-
-    it('meta commands should support common flags', () => {
-      const commands = [MetaList, MetaGet, MetaRegister, MetaDelete];
-      commands.forEach(cmd => {
-        expect(cmd.flags).toHaveProperty('url');
-        expect(cmd.flags).toHaveProperty('token');
-      });
-    });
-
-    it('meta commands should have examples', () => {
-      expect(MetaList.examples).toBeDefined();
-      expect(MetaList.examples.length).toBeGreaterThan(0);
-      expect(MetaGet.examples).toBeDefined();
-      expect(MetaRegister.examples).toBeDefined();
-      expect(MetaDelete.examples).toBeDefined();
-    });
-  });
-
-  describe('Command Conventions', () => {
-    it('all remote commands should support --url flag with OBJECTSTACK_URL env var', () => {
-      const commands = [
-        AuthLogin, AuthWhoami,
-        DataQuery, DataGet, DataCreate, DataUpdate, DataDelete,
-        MetaList, MetaGet, MetaRegister, MetaDelete
-      ];
-
-      commands.forEach(cmd => {
-        expect(cmd.flags).toHaveProperty('url');
-        expect(cmd.flags.url).toHaveProperty('env', 'OBJECTSTACK_URL');
-      });
-    });
-
-    it('authenticated commands should support --token flag with OBJECTSTACK_TOKEN env var', () => {
-      const commands = [
-        AuthWhoami,
-        DataQuery, DataGet, DataCreate, DataUpdate, DataDelete,
-        MetaList, MetaGet, MetaRegister, MetaDelete
-      ];
-
-      commands.forEach(cmd => {
-        expect(cmd.flags).toHaveProperty('token');
-        expect(cmd.flags.token).toHaveProperty('env', 'OBJECTSTACK_TOKEN');
-      });
-    });
-
-    it('all commands should support output formatting', () => {
-      const commands = [
-        AuthWhoami,
-        DataQuery, DataGet, DataCreate, DataUpdate, DataDelete,
-        MetaList, MetaGet, MetaRegister, MetaDelete
-      ];
-
-      commands.forEach(cmd => {
-        expect(cmd.flags).toHaveProperty('format');
-      });
-    });
-  });
-});

package/test/remote-api-utils.test.ts

@@ -1,196 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { createApiClient, requireAuth } from '../src/utils/api-client';
-import { readAuthConfig, writeAuthConfig, deleteAuthConfig, getCredentialsPath } from '../src/utils/auth-config';
-import { formatOutput } from '../src/utils/output-formatter';
-import * as fs from 'node:fs/promises';
-
-// Mock fs module
-vi.mock('node:fs/promises');
-
-describe('API Client Utilities', () => {
-  describe('createApiClient', () => {
-    it('should use provided URL and token', async () => {
-      const { client, token } = await createApiClient({
-        url: 'https://test.example.com',
-        token: 'test-token',
-      });
-
-      expect(client).toBeDefined();
-      expect((client as any).baseUrl).toBe('https://test.example.com');
-      expect(token).toBe('test-token');
-    });
-
-    it('should default to localhost when no URL provided', async () => {
-      const { client } = await createApiClient({});
-
-      expect(client).toBeDefined();
-      expect((client as any).baseUrl).toBe('http://localhost:3000');
-    });
-
-    it('should use environment variables if no options provided', async () => {
-      const originalUrl = process.env.OBJECTSTACK_URL;
-      const originalToken = process.env.OBJECTSTACK_TOKEN;
-
-      process.env.OBJECTSTACK_URL = 'https://env.example.com';
-      process.env.OBJECTSTACK_TOKEN = 'env-token';
-
-      const { client, token } = await createApiClient({});
-
-      expect((client as any).baseUrl).toBe('https://env.example.com');
-      expect(token).toBe('env-token');
-
-      // Restore
-      process.env.OBJECTSTACK_URL = originalUrl;
-      process.env.OBJECTSTACK_TOKEN = originalToken;
-    });
-  });
-
-  describe('requireAuth', () => {
-    it('should not throw when token is provided', () => {
-      expect(() => requireAuth('valid-token')).not.toThrow();
-    });
-
-    it('should throw when token is missing', () => {
-      expect(() => requireAuth(undefined)).toThrow(/Authentication required/);
-    });
-
-    it('should throw when token is empty string', () => {
-      expect(() => requireAuth('')).toThrow(/Authentication required/);
-    });
-  });
-});
-
-describe('Auth Config Utilities', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('getCredentialsPath', () => {
-    it('should return path to credentials file', () => {
-      const path = getCredentialsPath();
-      expect(path).toContain('.objectstack');
-      expect(path).toContain('credentials.json');
-    });
-  });
-
-  describe('writeAuthConfig', () => {
-    it('should write credentials to file with correct permissions', async () => {
-      const mockMkdir = vi.mocked(fs.mkdir);
-      const mockWriteFile = vi.mocked(fs.writeFile);
-
-      const config = {
-        url: 'https://test.example.com',
-        token: 'test-token',
-        email: 'user@example.com',
-        createdAt: '2024-01-01T00:00:00.000Z',
-      };
-
-      await writeAuthConfig(config);
-
-      expect(mockMkdir).toHaveBeenCalledWith(
-        expect.stringContaining('.objectstack'),
-        { recursive: true }
-      );
-
-      expect(mockWriteFile).toHaveBeenCalledWith(
-        expect.stringContaining('credentials.json'),
-        expect.stringContaining('test-token'),
-        { mode: 0o600 }
-      );
-    });
-  });
-
-  describe('readAuthConfig', () => {
-    it('should read and parse credentials file', async () => {
-      const mockConfig = {
-        url: 'https://test.example.com',
-        token: 'test-token',
-        email: 'user@example.com',
-        createdAt: '2024-01-01T00:00:00.000Z',
-      };
-
-      const mockReadFile = vi.mocked(fs.readFile);
-      mockReadFile.mockResolvedValue(JSON.stringify(mockConfig));
-
-      const config = await readAuthConfig();
-
-      expect(config).toEqual(mockConfig);
-    });
-
-    it('should throw helpful error when file does not exist', async () => {
-      const mockReadFile = vi.mocked(fs.readFile);
-      mockReadFile.mockRejectedValue({ code: 'ENOENT' });
-
-      await expect(readAuthConfig()).rejects.toThrow(/No stored credentials/);
-    });
-  });
-
-  describe('deleteAuthConfig', () => {
-    it('should delete credentials file', async () => {
-      const mockUnlink = vi.mocked(fs.unlink);
-      mockUnlink.mockResolvedValue(undefined as any);
-
-      await deleteAuthConfig();
-
-      expect(mockUnlink).toHaveBeenCalled();
-      expect(mockUnlink).toHaveBeenCalledWith(
-        expect.stringContaining('credentials.json')
-      );
-    });
-
-    it('should not throw if file does not exist', async () => {
-      const mockUnlink = vi.mocked(fs.unlink);
-      mockUnlink.mockRejectedValue({ code: 'ENOENT' } as any);
-
-      await expect(deleteAuthConfig()).resolves.not.toThrow();
-      expect(mockUnlink).toHaveBeenCalled();
-    });
-  });
-});
-
-describe('Output Formatter Utilities', () => {
-  beforeEach(() => {
-    // Spy on console.log
-    vi.spyOn(console, 'log').mockImplementation(() => {});
-  });
-
-  it('should format JSON output', () => {
-    const data = { name: 'test', value: 123 };
-    formatOutput(data, 'json');
-
-    expect(console.log).toHaveBeenCalledWith(
-      expect.stringContaining('"name": "test"')
-    );
-  });
-
-  it('should format YAML output', () => {
-    const data = { name: 'test', value: 123 };
-    formatOutput(data, 'yaml');
-
-    expect(console.log).toHaveBeenCalled();
-  });
-
-  it('should format table output for arrays', () => {
-    const data = [
-      { name: 'item1', value: 1 },
-      { name: 'item2', value: 2 },
-    ];
-    formatOutput(data, 'table');
-
-    expect(console.log).toHaveBeenCalled();
-  });
-
-  it('should format table output for single object', () => {
-    const data = { name: 'test', value: 123 };
-    formatOutput(data, 'table');
-
-    expect(console.log).toHaveBeenCalled();
-  });
-
-  it('should handle empty arrays', () => {
-    const data: any[] = [];
-    formatOutput(data, 'table');
-
-    expect(console.log).toHaveBeenCalled();
-  });
-});

package/test/serve-host-config.test.ts

@@ -1,77 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { isHostConfig } from '../src/utils/plugin-detection';
-
-/**
- * Tests for the host config detection logic used in serve.ts
- *
- * When the root config is a host/aggregator (i.e. its `plugins` array
- * contains already-instantiated Plugin objects), the CLI must NOT wrap
- * it again with AppPlugin, as that would cause duplicate registration
- * and a `plugin.app.<id> failed to start` error.
- */
-describe('Host config detection', () => {
-
-  it('should detect host config with instantiated plugins', () => {
-    const config = {
-      manifest: { id: 'dev-workspace', name: 'dev_workspace' },
-      plugins: [
-        { name: 'objectql', init: async () => {}, start: async () => {} },
-        { name: 'driver', init: async () => {}, start: async () => {} },
-      ],
-    };
-    expect(isHostConfig(config)).toBe(true);
-  });
-
-  it('should NOT detect pure app bundle config (no plugins)', () => {
-    const config = {
-      manifest: { id: 'my-app', name: 'my_app' },
-      objects: [{ name: 'task', fields: [] }],
-    };
-    expect(isHostConfig(config)).toBe(false);
-  });
-
-  it('should NOT detect config with empty plugins array', () => {
-    const config = {
-      manifest: { id: 'my-app', name: 'my_app' },
-      objects: [{ name: 'task', fields: [] }],
-      plugins: [],
-    };
-    expect(isHostConfig(config)).toBe(false);
-  });
-
-  it('should NOT detect config with string plugin references', () => {
-    const config = {
-      manifest: { id: 'my-app', name: 'my_app' },
-      plugins: ['@objectstack/plugin-auth', '@objectstack/objectql'],
-    };
-    expect(isHostConfig(config)).toBe(false);
-  });
-
-  it('should NOT detect config with plain object plugins (no init method)', () => {
-    const config = {
-      manifest: { id: 'my-app', name: 'my_app' },
-      plugins: [
-        { name: 'some-plugin', version: '1.0.0' },
-      ],
-    };
-    expect(isHostConfig(config)).toBe(false);
-  });
-
-  it('should detect if at least one plugin has init method', () => {
-    const config = {
-      manifest: { id: 'dev-workspace' },
-      plugins: [
-        { name: 'plain-bundle', version: '1.0.0' },
-        { name: 'real-plugin', init: async () => {}, start: async () => {} },
-      ],
-    };
-    expect(isHostConfig(config)).toBe(true);
-  });
-
-  it('should handle config without plugins property', () => {
-    const config = {
-      manifest: { id: 'my-app', name: 'my_app' },
-    };
-    expect(isHostConfig(config)).toBe(false);
-  });
-});

package/tsconfig.build.json

@@ -1,20 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "lib": ["ES2022", "DOM", "DOM.Iterable"],
-    "ignoreDeprecations": "6.0",
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "outDir": "dist",
-    "rootDir": "src",
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true,
-    "types": ["node"]
-  },
-  "include": ["src"]
-}

package/tsconfig.json

@@ -1,25 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "lib": [
-      "ES2022",
-      "DOM",
-      "DOM.Iterable"
-    ],
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "outDir": "dist",
-    "rootDir": "src",
-    "types": [
-      "node"
-    ],
-    "ignoreDeprecations": "6.0"
-  },
-  "include": [
-    "src"
-  ]
-}