newo 1.6.1 → 1.7.1

package/CHANGELOG.md

@@ -5,6 +5,51 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.7.1] - 2025-09-15
+
+### Enhanced
+- **Multi-Customer Commands**: Improved user experience for multi-customer operations
+  - `newo status` now automatically checks all customers when no default is specified
+  - `newo push` provides interactive customer selection dialog when multiple customers exist
+  - No more error messages for commands that support multi-customer operations
+  - Better user guidance with clear options for customer selection
+
+### Fixed
+- **Command Flow**: Moved customer selection logic into command-specific handlers
+  - Prevents early exit errors for multi-customer operations
+  - Each command now handles customer selection appropriately
+  - Maintains backward compatibility with single-customer setups
+
+## [1.7.0] - 2025-09-15
+
+### Added
+- **Customer Attributes Synchronization**: Complete pull/push functionality for customer attributes
+  - `GET /api/v1/bff/customer/attributes?include_hidden=true` - Fetches all 233 customer attributes including hidden system attributes
+  - `PUT /api/v1/customer/attributes/{attributeId}` - Updates individual customer attributes
+  - Saves to `newo_customers/{customerIdn}/attributes.yaml` in customer root directory
+  - YAML format matches reference specification exactly with literal blocks, enum types, and proper multi-line formatting
+  - Separate ID mapping stored in `.newo/{customerIdn}/attributes-map.json` for push operations
+  - Integrated into existing `pull` and `push` commands seamlessly
+  - Full TypeScript type safety with `CustomerAttribute` and `CustomerAttributesResponse` interfaces
+
+### Enhanced
+- **YAML Format Compliance**: Perfect format matching with reference files
+  - Literal block scalars (`|-`) for multi-line strings
+  - Proper enum format (`!enum "AttributeValueTypes.string"`)
+  - Complex JSON string formatting with proper line breaks
+  - No escaped quotes in output for better readability
+- **Metadata Generation**: Removed legacy JSON metadata files, YAML-only approach
+  - Eliminates redundant `metadata.json` files
+  - Cleaner file structure with single source of truth
+  - Improved performance with fewer file operations
+
+### Technical
+- **API Layer**: Added `updateCustomerAttribute()` and enhanced `getCustomerAttributes()` with `includeHidden` parameter
+- **Sync Engine**: Integrated attributes handling into `pullAll()` and `pushChanged()` functions
+- **File System**: Added `customerAttributesPath()` and `customerAttributesMapPath()` utilities
+- **Type Safety**: Extended type definitions with proper customer attribute interfaces
+- **Error Handling**: Comprehensive error handling for attributes operations with graceful fallbacks
+
 ## [1.6.1] - 2025-09-13
 
 ### Fixed

package/dist/api.d.ts

@@ -1,5 +1,5 @@
 import { type AxiosInstance } from 'axios';
-import type { ProjectMeta, Agent, Skill, FlowEvent, FlowState, AkbImportArticle, CustomerProfile } from './types.js';
+import type { ProjectMeta, Agent, Skill, FlowEvent, FlowState, AkbImportArticle, CustomerProfile, CustomerAttribute, CustomerAttributesResponse } from './types.js';
 export declare function makeClient(verbose?: boolean, token?: string): Promise<AxiosInstance>;
 export declare function listProjects(client: AxiosInstance): Promise<ProjectMeta[]>;
 export declare function listAgents(client: AxiosInstance, projectId: string): Promise<Agent[]>;
@@ -11,4 +11,6 @@ export declare function listFlowEvents(client: AxiosInstance, flowId: string): P
 export declare function listFlowStates(client: AxiosInstance, flowId: string): Promise<FlowState[]>;
 export declare function importAkbArticle(client: AxiosInstance, articleData: AkbImportArticle): Promise<unknown>;
 export declare function getCustomerProfile(client: AxiosInstance): Promise<CustomerProfile>;
+export declare function getCustomerAttributes(client: AxiosInstance, includeHidden?: boolean): Promise<CustomerAttributesResponse>;
+export declare function updateCustomerAttribute(client: AxiosInstance, attribute: CustomerAttribute): Promise<void>;
 //# sourceMappingURL=api.d.ts.map

package/dist/api.js

@@ -100,4 +100,25 @@ export async function getCustomerProfile(client) {
     const response = await client.get('/api/v1/customer/profile');
     return response.data;
 }
+export async function getCustomerAttributes(client, includeHidden = true) {
+    const response = await client.get('/api/v1/bff/customer/attributes', {
+        params: { include_hidden: includeHidden }
+    });
+    return response.data;
+}
+export async function updateCustomerAttribute(client, attribute) {
+    if (!attribute.id) {
+        throw new Error(`Attribute ${attribute.idn} is missing ID - cannot update`);
+    }
+    await client.put(`/api/v1/customer/attributes/${attribute.id}`, {
+        idn: attribute.idn,
+        value: attribute.value,
+        title: attribute.title,
+        description: attribute.description,
+        group: attribute.group,
+        is_hidden: attribute.is_hidden,
+        possible_values: attribute.possible_values,
+        value_type: attribute.value_type
+    });
+}
 //# sourceMappingURL=api.js.map

package/dist/cli.js

@@ -2,7 +2,7 @@
 import minimist from 'minimist';
 import dotenv from 'dotenv';
 import { makeClient, getProjectMeta, importAkbArticle } from './api.js';
-import { pullAll, pushChanged, status } from './sync.js';
+import { pullAll, pushChanged, status, saveCustomerAttributes } from './sync.js';
 import { parseAkbFile, prepareArticlesForImport } from './akb.js';
 import { initializeEnvironment, ENV, EnvValidationError } from './env.js';
 import { parseCustomerConfigAsync, listCustomers, getCustomer, getDefaultCustomer, tryGetDefaultCustomer, getAllCustomers, validateCustomerConfig } from './customerAsync.js';
@@ -151,45 +151,9 @@ async function main() {
         }
         return;
     }
-    if (args.customer) {
-        const customer = getCustomer(customerConfig, args.customer);
-        if (!customer) {
-            console.error(`Unknown customer: ${args.customer}`);
-            console.error(`Available customers: ${listCustomers(customerConfig).join(', ')}`);
-            process.exit(1);
-        }
-        selectedCustomer = customer;
-    }
-    else {
-        // For pull command, try to get default but fall back to all customers if multiple exist
-        if (cmd === 'pull') {
-            try {
-                selectedCustomer = tryGetDefaultCustomer(customerConfig);
-                if (!selectedCustomer) {
-                    // Multiple customers exist with no default, pull from all
-                    allCustomers = getAllCustomers(customerConfig);
-                    if (verbose)
-                        console.log(`📥 No default customer specified, pulling from all ${allCustomers.length} customers`);
-                }
-            }
-            catch (error) {
-                const message = error instanceof Error ? error.message : String(error);
-                console.error(message);
-                process.exit(1);
-            }
-        }
-        else {
-            // For other commands, require explicit customer selection
-            try {
-                selectedCustomer = getDefaultCustomer(customerConfig);
-            }
-            catch (error) {
-                const message = error instanceof Error ? error.message : String(error);
-                console.error(message);
-                process.exit(1);
-            }
-        }
-    }
+    // Customer selection logic moved inside command processing to avoid early failures
+    if (verbose)
+        console.log(`🔍 Command parsed: "${cmd}"`);
     if (!cmd || ['help', '-h', '--help'].includes(cmd)) {
         console.log(`NEWO CLI - Multi-Customer Support
 Usage:
@@ -201,7 +165,7 @@ Usage:
   newo import-akb <file> <persona_id> [--customer <idn>]  # import AKB articles from file
 
 Flags:
-  --customer <idn>             # specify customer (if not set, uses default or all for pull)
+  --customer <idn>             # specify customer (if not set, uses default or interactive selection)
   --verbose, -v                # enable detailed logging
 
 Environment Variables:
@@ -219,8 +183,9 @@ Multi-Customer Examples:
   # Commands:
   newo pull                                    # Pull from all customers (if no default set)
   newo pull --customer acme                    # Pull projects for Acme only
-  newo push --customer globex                  # Push changes for Globex
-  newo status                                  # Status for default customer
+  newo status                                  # Status for all customers (if no default set)
+  newo push                                    # Interactive selection for multiple customers
+  newo push --customer globex                  # Push changes for Globex only
 
 File Structure:
   newo_customers/
@@ -233,7 +198,28 @@ File Structure:
 `);
         return;
     }
+    if (verbose)
+        console.log(`🔍 Starting command processing for: ${cmd}`);
     if (cmd === 'pull') {
+        // Handle customer selection for pull command
+        if (args.customer) {
+            const customer = getCustomer(customerConfig, args.customer);
+            if (!customer) {
+                console.error(`Unknown customer: ${args.customer}`);
+                console.error(`Available customers: ${listCustomers(customerConfig).join(', ')}`);
+                process.exit(1);
+            }
+            selectedCustomer = customer;
+        }
+        else {
+            // Try to get default, fall back to all customers
+            selectedCustomer = tryGetDefaultCustomer(customerConfig);
+            if (!selectedCustomer) {
+                allCustomers = getAllCustomers(customerConfig);
+                if (verbose)
+                    console.log(`📥 No default customer specified, pulling from all ${allCustomers.length} customers`);
+            }
+        }
         if (selectedCustomer) {
             // Single customer pull
             const accessToken = await getValidAccessToken(selectedCustomer);
@@ -255,7 +241,128 @@ File Structure:
         }
         return;
     }
+    if (cmd === 'status') {
+        // Handle customer selection for status command
+        if (args.customer) {
+            const customer = getCustomer(customerConfig, args.customer);
+            if (!customer) {
+                console.error(`Unknown customer: ${args.customer}`);
+                console.error(`Available customers: ${listCustomers(customerConfig).join(', ')}`);
+                process.exit(1);
+            }
+            selectedCustomer = customer;
+        }
+        else {
+            // Try to get default, fall back to all customers
+            selectedCustomer = tryGetDefaultCustomer(customerConfig);
+            if (!selectedCustomer) {
+                allCustomers = getAllCustomers(customerConfig);
+                console.log(`🔄 Checking status for ${allCustomers.length} customers...`);
+            }
+        }
+        if (selectedCustomer) {
+            // Single customer status
+            await status(selectedCustomer, verbose);
+        }
+        else if (allCustomers.length > 0) {
+            // Multi-customer status
+            for (const customer of allCustomers) {
+                console.log(`\n📋 Status for customer: ${customer.idn}`);
+                await status(customer, verbose);
+            }
+            console.log(`\n✅ Status check completed for all ${allCustomers.length} customers`);
+        }
+        return;
+    }
+    if (cmd === 'push') {
+        // Handle customer selection for push command
+        if (args.customer) {
+            const customer = getCustomer(customerConfig, args.customer);
+            if (!customer) {
+                console.error(`Unknown customer: ${args.customer}`);
+                console.error(`Available customers: ${listCustomers(customerConfig).join(', ')}`);
+                process.exit(1);
+            }
+            selectedCustomer = customer;
+        }
+        else {
+            // Try to get default, provide interactive selection if multiple exist
+            selectedCustomer = tryGetDefaultCustomer(customerConfig);
+            if (!selectedCustomer) {
+                // Multiple customers exist with no default, ask user
+                allCustomers = getAllCustomers(customerConfig);
+                console.log(`\n📤 Multiple customers available for push:`);
+                allCustomers.forEach((customer, index) => {
+                    console.log(`  ${index + 1}. ${customer.idn}`);
+                });
+                console.log(`  ${allCustomers.length + 1}. All customers`);
+                const readline = await import('readline');
+                const rl = readline.createInterface({
+                    input: process.stdin,
+                    output: process.stdout
+                });
+                const choice = await new Promise((resolve) => {
+                    rl.question(`\nSelect customer to push (1-${allCustomers.length + 1}): `, resolve);
+                });
+                rl.close();
+                const choiceNum = parseInt(choice.trim());
+                if (choiceNum === allCustomers.length + 1) {
+                    // User selected "All customers"
+                    console.log(`🔄 Pushing to all ${allCustomers.length} customers...`);
+                }
+                else if (choiceNum >= 1 && choiceNum <= allCustomers.length) {
+                    // User selected specific customer
+                    selectedCustomer = allCustomers[choiceNum - 1] || null;
+                    allCustomers = []; // Clear to indicate single customer mode
+                    if (selectedCustomer) {
+                        console.log(`🔄 Pushing to customer: ${selectedCustomer.idn}`);
+                    }
+                }
+                else {
+                    console.error('Invalid choice. Exiting.');
+                    process.exit(1);
+                }
+            }
+        }
+        if (selectedCustomer) {
+            // Single customer push
+            const accessToken = await getValidAccessToken(selectedCustomer);
+            const client = await makeClient(verbose, accessToken);
+            await pushChanged(client, selectedCustomer, verbose);
+        }
+        else if (allCustomers.length > 0) {
+            // Multi-customer push (user selected "All customers")
+            console.log(`🔄 Pushing to ${allCustomers.length} customers...`);
+            for (const customer of allCustomers) {
+                console.log(`\n📤 Pushing for customer: ${customer.idn}`);
+                const accessToken = await getValidAccessToken(customer);
+                const client = await makeClient(verbose, accessToken);
+                await pushChanged(client, customer, verbose);
+            }
+            console.log(`\n✅ Push completed for all ${allCustomers.length} customers`);
+        }
+        return;
+    }
     // For all other commands, require a single selected customer
+    if (args.customer) {
+        const customer = getCustomer(customerConfig, args.customer);
+        if (!customer) {
+            console.error(`Unknown customer: ${args.customer}`);
+            console.error(`Available customers: ${listCustomers(customerConfig).join(', ')}`);
+            process.exit(1);
+        }
+        selectedCustomer = customer;
+    }
+    else {
+        try {
+            selectedCustomer = getDefaultCustomer(customerConfig);
+        }
+        catch (error) {
+            const message = error instanceof Error ? error.message : String(error);
+            console.error(message);
+            process.exit(1);
+        }
+    }
     if (!selectedCustomer) {
         console.error('Customer selection required for this command');
         process.exit(1);
@@ -263,13 +370,7 @@ File Structure:
     // Get access token for the selected customer
     const accessToken = await getValidAccessToken(selectedCustomer);
     const client = await makeClient(verbose, accessToken);
-    if (cmd === 'push') {
-        await pushChanged(client, selectedCustomer, verbose);
-    }
-    else if (cmd === 'status') {
-        await status(selectedCustomer, verbose);
-    }
-    else if (cmd === 'meta') {
+    if (cmd === 'meta') {
         if (!selectedCustomer.projectId) {
             console.error(`No project ID configured for customer ${selectedCustomer.idn}`);
             console.error(`Set NEWO_CUSTOMER_${selectedCustomer.idn.toUpperCase()}_PROJECT_ID in your .env file`);
@@ -278,6 +379,11 @@ File Structure:
         const meta = await getProjectMeta(client, selectedCustomer.projectId);
         console.log(JSON.stringify(meta, null, 2));
     }
+    else if (cmd === 'pull-attributes') {
+        console.log(`🔍 Fetching customer attributes for ${selectedCustomer.idn}...`);
+        await saveCustomerAttributes(client, selectedCustomer, verbose);
+        console.log(`✅ Customer attributes saved to newo_customers/${selectedCustomer.idn}/attributes.yaml`);
+    }
     else if (cmd === 'import-akb') {
         const akbFile = args._[1];
         const personaId = args._[2];

package/dist/fsutil.d.ts

@@ -9,7 +9,15 @@ export declare function hashesPath(customerIdn: string): string;
 export declare function ensureState(customerIdn: string): Promise<void>;
 export declare function projectDir(customerIdn: string, projectIdn: string): string;
 export declare function flowsYamlPath(customerIdn: string): string;
+export declare function customerAttributesPath(customerIdn: string): string;
+export declare function customerAttributesMapPath(customerIdn: string): string;
 export declare function skillPath(customerIdn: string, projectIdn: string, agentIdn: string, flowIdn: string, skillIdn: string, runnerType?: RunnerType): string;
+export declare function skillFolderPath(customerIdn: string, projectIdn: string, agentIdn: string, flowIdn: string, skillIdn: string): string;
+export declare function skillScriptPath(customerIdn: string, projectIdn: string, agentIdn: string, flowIdn: string, skillIdn: string, runnerType?: RunnerType): string;
+export declare function projectMetadataPath(customerIdn: string, projectIdn: string): string;
+export declare function agentMetadataPath(customerIdn: string, projectIdn: string, agentIdn: string): string;
+export declare function flowMetadataPath(customerIdn: string, projectIdn: string, agentIdn: string, flowIdn: string): string;
+export declare function skillMetadataPath(customerIdn: string, projectIdn: string, agentIdn: string, flowIdn: string, skillIdn: string): string;
 export declare function metadataPath(customerIdn: string, projectIdn: string): string;
 export declare const ROOT_DIR: string;
 export declare const MAP_PATH: string;

package/dist/fsutil.js

@@ -28,10 +28,39 @@ export function projectDir(customerIdn, projectIdn) {
 export function flowsYamlPath(customerIdn) {
     return path.posix.join(customerProjectsDir(customerIdn), 'flows.yaml');
 }
+export function customerAttributesPath(customerIdn) {
+    return path.posix.join(customerDir(customerIdn), 'attributes.yaml');
+}
+export function customerAttributesMapPath(customerIdn) {
+    return path.join(customerStateDir(customerIdn), 'attributes-map.json');
+}
+// Legacy skill path - direct file
 export function skillPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn, runnerType = 'guidance') {
     const extension = runnerType === 'nsl' ? '.jinja' : '.guidance';
     return path.posix.join(customerProjectsDir(customerIdn), projectIdn, agentIdn, flowIdn, `${skillIdn}${extension}`);
 }
+// New hierarchical structure paths
+export function skillFolderPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn) {
+    return path.posix.join(customerProjectsDir(customerIdn), projectIdn, agentIdn, flowIdn, skillIdn);
+}
+export function skillScriptPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn, runnerType = 'guidance') {
+    const extension = runnerType === 'nsl' ? '.jinja' : '.guidance';
+    return path.posix.join(skillFolderPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn), `skill${extension}`);
+}
+// Metadata paths for hierarchical structure
+export function projectMetadataPath(customerIdn, projectIdn) {
+    return path.posix.join(customerProjectsDir(customerIdn), projectIdn, 'metadata.yaml');
+}
+export function agentMetadataPath(customerIdn, projectIdn, agentIdn) {
+    return path.posix.join(customerProjectsDir(customerIdn), projectIdn, agentIdn, 'metadata.yaml');
+}
+export function flowMetadataPath(customerIdn, projectIdn, agentIdn, flowIdn) {
+    return path.posix.join(customerProjectsDir(customerIdn), projectIdn, agentIdn, flowIdn, 'metadata.yaml');
+}
+export function skillMetadataPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn) {
+    return path.posix.join(skillFolderPath(customerIdn, projectIdn, agentIdn, flowIdn, skillIdn), 'metadata.yaml');
+}
+// Legacy metadata path - keep for backwards compatibility
 export function metadataPath(customerIdn, projectIdn) {
     return path.posix.join(customerProjectsDir(customerIdn), projectIdn, 'metadata.json');
 }

package/dist/sync.d.ts

@@ -1,5 +1,6 @@
 import type { AxiosInstance } from 'axios';
 import type { ProjectData, CustomerConfig } from './types.js';
+export declare function saveCustomerAttributes(client: AxiosInstance, customer: CustomerConfig, verbose?: boolean): Promise<void>;
 export declare function pullSingleProject(client: AxiosInstance, customer: CustomerConfig, projectId: string, projectIdn: string, verbose?: boolean): Promise<ProjectData>;
 export declare function pullAll(client: AxiosInstance, customer: CustomerConfig, projectId?: string | null, verbose?: boolean): Promise<void>;
 export declare function pushChanged(client: AxiosInstance, customer: CustomerConfig, verbose?: boolean): Promise<void>;