runwork 0.10.0 → 0.10.1

package/dist/commands/__tests__/info-merge.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/commands/__tests__/info-merge.test.js

@@ -0,0 +1,55 @@
+import { describe, it, expect } from 'vitest';
+import { __testing } from '../info.js';
+const { mergeSources, padName } = __testing;
+describe('mergeSources — bad data resilience', () => {
+    it('drops blueprint entries with missing name', () => {
+        const map = mergeSources([
+            { name: 'good', extra: {} },
+            { name: undefined, extra: {} },
+            { name: '', extra: {} },
+        ], [], (item) => item.name ?? '', 'app-1');
+        expect(Array.from(map.keys())).toEqual(['good']);
+    });
+    it('drops server items whose name resolver returns undefined or empty', () => {
+        const map = mergeSources([], [
+            { name: 'good', appId: 'app-1', appName: 'My App', deploymentMode: 'preview' },
+            { name: undefined, appId: 'app-1', appName: 'My App', deploymentMode: 'preview' },
+            { name: '', appId: 'app-1', appName: 'My App', deploymentMode: 'preview' },
+        ], (item) => item.name ?? '', 'app-1');
+        expect(Array.from(map.keys())).toEqual(['good']);
+    });
+    it('skips server items belonging to other apps', () => {
+        const map = mergeSources([], [
+            { name: 'mine', appId: 'app-1', appName: 'A', deploymentMode: 'preview' },
+            { name: 'theirs', appId: 'app-2', appName: 'B', deploymentMode: 'preview' },
+        ], (item) => item.name ?? '', 'app-1');
+        expect(Array.from(map.keys())).toEqual(['mine']);
+    });
+    it('merges blueprint and server entries by name', () => {
+        const map = mergeSources([{ name: 'agent-x', extra: { type: 'task' } }], [
+            { name: 'agent-x', appId: 'app-1', appName: 'A', deploymentMode: 'production' },
+            { name: 'agent-y', appId: 'app-1', appName: 'A', deploymentMode: 'preview' },
+        ], (item) => item.name ?? '', 'app-1');
+        const x = map.get('agent-x');
+        const y = map.get('agent-y');
+        expect(x?.sources).toEqual(['blueprint', 'production']);
+        expect(x?.blueprintExtra).toEqual({ type: 'task' });
+        expect(y?.sources).toEqual(['preview']);
+    });
+});
+describe('padName — defensive printer', () => {
+    it('pads valid names to the requested width', () => {
+        expect(padName('foo', 8)).toBe('foo     ');
+    });
+    it('returns a placeholder for undefined / null / empty inputs', () => {
+        expect(padName(undefined, 12)).toContain('(unnamed)');
+        expect(padName(null, 12)).toContain('(unnamed)');
+        expect(padName('', 12)).toContain('(unnamed)');
+        // Placeholder is also padded to the requested width.
+        expect(padName(undefined, 12).length).toBe(12);
+    });
+    it('does not throw on non-string inputs', () => {
+        expect(() => padName(42, 8)).not.toThrow();
+        expect(() => padName({}, 8)).not.toThrow();
+    });
+});

package/dist/commands/info.d.ts

@@ -1,3 +1,144 @@
 import { Command } from 'commander';
+import type { WorkspaceAllData } from '../types.js';
+interface BlueprintEntity {
+    entityName: string;
+    schema?: Record<string, unknown>;
+}
+interface BlueprintSchedule {
+    name: string;
+    schedule?: string;
+    description?: string;
+}
+interface BlueprintWorkflow {
+    name: string;
+    description?: string;
+}
+interface BlueprintAgent {
+    name: string;
+    type?: string;
+    description?: string;
+}
+interface BlueprintEndpoint {
+    path: string;
+    method?: string;
+    description?: string;
+}
+interface BlueprintComponent {
+    componentName: string;
+}
+interface Blueprint {
+    entities?: BlueprintEntity[];
+    schedules?: BlueprintSchedule[];
+    scheduledJobs?: BlueprintSchedule[];
+    workflows?: BlueprintWorkflow[];
+    agents?: BlueprintAgent[];
+    endpoints?: BlueprintEndpoint[];
+    publicEndpoints?: BlueprintEndpoint[];
+    components?: BlueprintComponent[];
+    fileStorage?: boolean | {
+        enabled?: boolean;
+    };
+}
+interface MergedEntity {
+    name: string;
+    sources: string[];
+    schema?: Record<string, unknown>;
+}
+interface MergedScheduledJob {
+    name: string;
+    sources: string[];
+    schedule?: string;
+    description?: string;
+}
+interface MergedWorkflow {
+    name: string;
+    sources: string[];
+    description?: string;
+}
+interface MergedAgent {
+    name: string;
+    sources: string[];
+    type?: string;
+    description?: string;
+}
+interface MergedEndpoint {
+    path: string;
+    method?: string;
+    sources: string[];
+    description?: string;
+}
+interface MergedComponent {
+    name: string;
+    sources: string[];
+}
+interface MergedFileStorage {
+    enabled: boolean;
+    sources: string[];
+}
+interface InfoOutput {
+    app: {
+        id: string;
+        name: string;
+        slug: string;
+    } | null;
+    workspace: {
+        id: string;
+        name: string;
+    };
+    preview: {
+        url: string | null;
+        active: boolean;
+    };
+    production: {
+        url: null;
+        deployed: boolean;
+    };
+    integrations: Array<{
+        id: string;
+        status: string;
+    }> | null;
+    registries: {
+        entities: MergedEntity[];
+        workflows: MergedWorkflow[];
+        scheduledJobs: MergedScheduledJob[];
+        agents: MergedAgent[];
+        publicEndpoints: MergedEndpoint[];
+        components: MergedComponent[];
+        fileStorage: MergedFileStorage;
+    } | null;
+    cli: {
+        version: string;
+        commands: Array<{
+            name: string;
+            description: string;
+        }>;
+    };
+}
+interface ServerRegistryItem {
+    appId: string;
+    appName: string;
+    deploymentMode?: 'preview' | 'production';
+}
+declare function mergeSources<T extends ServerRegistryItem>(blueprintItems: Array<{
+    name: string;
+    extra?: Record<string, unknown>;
+}>, serverItems: T[], getServerName: (item: T) => string, appId: string): Map<string, {
+    sources: string[];
+    serverItem?: T;
+    blueprintExtra?: Record<string, unknown>;
+}>;
+declare function buildRegistries(blueprint: Blueprint | null, serverData: WorkspaceAllData | null, appId: string): InfoOutput['registries'];
+/**
+ * Pad a possibly-missing string to a fixed width. Returns a visible
+ * placeholder when the value isn't a usable string so the human-readable
+ * printer can never crash on an unexpectedly-undefined field.
+ */
+declare function padName(value: unknown, width: number): string;
 export declare function formatSources(sources: string[]): string;
+export declare const __testing: {
+    mergeSources: typeof mergeSources;
+    padName: typeof padName;
+    buildRegistries: typeof buildRegistries;
+};
 export declare const infoCommand: Command;
+export {};

package/dist/commands/info.js

@@ -32,13 +32,22 @@ function readBlueprint(cwd) {
 }
 function mergeSources(blueprintItems, serverItems, getServerName, appId) {
     const map = new Map();
+    // Skip nameless entries so we never store an `undefined` key. A bad
+    // blueprint or a partially-populated server-side row would otherwise
+    // produce a registry entry with `name === undefined`, which then crashes
+    // the human-readable printer at `a.name.padEnd(24)` and yields ugly
+    // `"undefined": ...` keys in JSON output.
     for (const bp of blueprintItems) {
+        if (typeof bp.name !== 'string' || bp.name.length === 0)
+            continue;
         map.set(bp.name, { sources: ['blueprint'], blueprintExtra: bp.extra });
     }
     for (const serverItem of serverItems) {
         if (serverItem.appId !== appId)
             continue;
         const name = getServerName(serverItem);
+        if (typeof name !== 'string' || name.length === 0)
+            continue;
         const mode = serverItem.deploymentMode;
         const source = mode === 'production' ? 'production' : 'preview';
         const existing = map.get(name);
@@ -171,6 +180,16 @@ function getCliCommands(command) {
         return { name, description: cmd.description() };
     });
 }
+/**
+ * Pad a possibly-missing string to a fixed width. Returns a visible
+ * placeholder when the value isn't a usable string so the human-readable
+ * printer can never crash on an unexpectedly-undefined field.
+ */
+function padName(value, width) {
+    if (typeof value === 'string' && value.length > 0)
+        return value.padEnd(width);
+    return '(unnamed)'.padEnd(width);
+}
 export function formatSources(sources) {
     return sources
         .map(s => {
@@ -240,7 +259,7 @@ function printHumanOutput(data) {
             if (reg.entities.length > 0) {
                 console.log(`  ${bold('Entities:')}`);
                 for (const e of reg.entities) {
-                    console.log(`    ${e.name.padEnd(24)} ${formatSources(e.sources)}`);
+                    console.log(`    ${padName(e.name, 24)} ${formatSources(e.sources)}`);
                 }
                 console.log('');
             }
@@ -248,7 +267,7 @@ function printHumanOutput(data) {
                 console.log(`  ${bold('Workflows:')}`);
                 for (const w of reg.workflows) {
                     const desc = w.description ? `  ${dim(w.description)}` : '';
-                    console.log(`    ${w.name.padEnd(24)} ${formatSources(w.sources)}${desc}`);
+                    console.log(`    ${padName(w.name, 24)} ${formatSources(w.sources)}${desc}`);
                 }
                 console.log('');
             }
@@ -256,7 +275,7 @@ function printHumanOutput(data) {
                 console.log(`  ${bold('Scheduled Jobs:')}`);
                 for (const s of reg.scheduledJobs) {
                     const schedule = s.schedule ? `  ${gray(s.schedule)}` : '';
-                    console.log(`    ${s.name.padEnd(24)} ${formatSources(s.sources)}${schedule}`);
+                    console.log(`    ${padName(s.name, 24)} ${formatSources(s.sources)}${schedule}`);
                 }
                 console.log('');
             }
@@ -264,22 +283,22 @@ function printHumanOutput(data) {
                 console.log(`  ${bold('Agents:')}`);
                 for (const a of reg.agents) {
                     const type = a.type ? `  ${dim(a.type)}` : '';
-                    console.log(`    ${a.name.padEnd(24)} ${formatSources(a.sources)}${type}`);
+                    console.log(`    ${padName(a.name, 24)} ${formatSources(a.sources)}${type}`);
                 }
                 console.log('');
             }
             if (reg.publicEndpoints.length > 0) {
                 console.log(`  ${bold('Public Endpoints:')}`);
                 for (const e of reg.publicEndpoints) {
-                    const label = e.method ? `${e.method} ${e.path}` : e.path;
-                    console.log(`    ${label.padEnd(24)} ${formatSources(e.sources)}`);
+                    const label = e.method && e.path ? `${e.method} ${e.path}` : (e.path || '');
+                    console.log(`    ${padName(label, 24)} ${formatSources(e.sources)}`);
                 }
                 console.log('');
             }
             if (reg.components.length > 0) {
                 console.log(`  ${bold('Components:')}`);
                 for (const c of reg.components) {
-                    console.log(`    ${c.name.padEnd(24)} ${formatSources(c.sources)}`);
+                    console.log(`    ${padName(c.name, 24)} ${formatSources(c.sources)}`);
                 }
                 console.log('');
             }
@@ -298,6 +317,9 @@ function printHumanOutput(data) {
     console.log(dim(`  CLI version: ${data.cli.version}`));
     console.log('');
 }
+// Internal helpers exposed for unit tests; do not import from anywhere
+// other than `__tests__/`.
+export const __testing = { mergeSources, padName, buildRegistries };
 export const infoCommand = new Command('info')
     .description('Show app context, registries, and CLI reference (use --json for agent discovery)')
     .action(async (_opts, command) => {

package/dist/generated/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.10.0";
+export declare const VERSION = "0.10.1";

package/dist/generated/version.js

@@ -1,2 +1,2 @@
 // Auto-generated by scripts/embed-types.ts -- do not edit
-export const VERSION = "0.10.0";
+export const VERSION = "0.10.1";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "runwork",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "description": "CLI for Runwork: develop, preview, and deploy Runwork apps from your local machine.",
   "license": "UNLICENSED",
   "author": "Runwork <info@runwork.ai> (https://www.runwork.ai)",