newo 1.5.0 → 1.5.2

package/dist/types.d.ts

@@ -5,9 +5,32 @@ export interface NewoEnvironment {
     NEWO_BASE_URL?: string;
     NEWO_PROJECT_ID?: string;
     NEWO_API_KEY?: string;
+    NEWO_API_KEYS?: string;
     NEWO_ACCESS_TOKEN?: string;
     NEWO_REFRESH_TOKEN?: string;
     NEWO_REFRESH_URL?: string;
+    NEWO_DEFAULT_CUSTOMER?: string;
+    [key: string]: string | undefined;
+}
+export interface ApiKeyConfig {
+    key: string;
+    project_id?: string;
+}
+export interface CustomerConfig {
+    idn: string;
+    apiKey: string;
+    projectId?: string | undefined;
+}
+export interface CustomerProfile {
+    id: string;
+    idn: string;
+    organization_name: string;
+    email: string;
+    [key: string]: any;
+}
+export interface MultiCustomerConfig {
+    customers: Record<string, CustomerConfig>;
+    defaultCustomer?: string | undefined;
 }
 export interface TokenResponse {
     access_token?: string;
@@ -24,63 +47,63 @@ export interface StoredTokens {
     expires_at: number;
 }
 export interface ProjectMeta {
-    id: string;
-    idn: string;
-    title: string;
-    description?: string;
-    created_at?: string;
-    updated_at?: string;
+    readonly id: string;
+    readonly idn: string;
+    readonly title: string;
+    readonly description?: string;
+    readonly created_at?: string;
+    readonly updated_at?: string;
 }
 export interface Agent {
-    id: string;
-    idn: string;
-    title?: string;
-    description?: string;
-    flows?: Flow[];
+    readonly id: string;
+    readonly idn: string;
+    readonly title?: string;
+    readonly description?: string;
+    readonly flows?: readonly Flow[];
 }
 export interface Flow {
-    id: string;
-    idn: string;
-    title: string;
-    description?: string;
-    default_runner_type: RunnerType;
-    default_model: ModelConfig;
+    readonly id: string;
+    readonly idn: string;
+    readonly title: string;
+    readonly description?: string;
+    readonly default_runner_type: RunnerType;
+    readonly default_model: ModelConfig;
 }
 export interface ModelConfig {
-    model_idn: string;
-    provider_idn: string;
+    readonly model_idn: string;
+    readonly provider_idn: string;
 }
 export interface SkillParameter {
-    name: string;
-    default_value?: string;
+    readonly name: string;
+    readonly default_value?: string;
 }
 export interface Skill {
-    id: string;
-    idn: string;
-    title: string;
+    readonly id: string;
+    readonly idn: string;
+    readonly title: string;
     prompt_script?: string;
-    runner_type: RunnerType;
-    model: ModelConfig;
-    parameters: SkillParameter[];
-    path?: string | undefined;
+    readonly runner_type: RunnerType;
+    readonly model: ModelConfig;
+    readonly parameters: readonly SkillParameter[];
+    readonly path?: string | undefined;
 }
 export interface FlowEvent {
-    id: string;
-    idn: string;
-    description: string;
-    skill_selector: SkillSelector;
-    skill_idn?: string;
-    state_idn?: string;
-    integration_idn?: string;
-    connector_idn?: string;
-    interrupt_mode: InterruptMode;
+    readonly id: string;
+    readonly idn: string;
+    readonly description: string;
+    readonly skill_selector: SkillSelector;
+    readonly skill_idn?: string;
+    readonly state_idn?: string;
+    readonly integration_idn?: string;
+    readonly connector_idn?: string;
+    readonly interrupt_mode: InterruptMode;
 }
 export interface FlowState {
-    id: string;
-    idn: string;
-    title: string;
-    default_value?: string;
-    scope: StateFieldScope;
+    readonly id: string;
+    readonly idn: string;
+    readonly title: string;
+    readonly default_value?: string;
+    readonly scope: StateFieldScope;
 }
 export type RunnerType = 'guidance' | 'nsl';
 export type SkillSelector = 'first' | 'last' | 'random' | 'all';
@@ -118,22 +141,22 @@ export interface HashStore {
     [filePath: string]: string;
 }
 export interface ParsedArticle {
-    topic_name: string;
-    persona_id: string | null;
-    topic_summary: string;
-    topic_facts: string[];
-    confidence: number;
-    source: string;
-    labels: string[];
+    readonly topic_name: string;
+    readonly persona_id: string | null;
+    readonly topic_summary: string;
+    readonly topic_facts: readonly string[];
+    readonly confidence: number;
+    readonly source: string;
+    readonly labels: readonly string[];
 }
 export interface AkbImportArticle extends Omit<ParsedArticle, 'persona_id'> {
     persona_id: string;
 }
 export interface CliArgs {
-    _: string[];
-    verbose?: boolean;
-    v?: boolean;
-    [key: string]: unknown;
+    readonly _: readonly string[];
+    readonly verbose?: boolean;
+    readonly v?: boolean;
+    readonly [key: string]: unknown;
 }
 export interface FlowsYamlSkill {
     idn: string;

package/package.json

@@ -1,8 +1,7 @@
 {
   "name": "newo",
-  "version": "1.5.0",
-  "description": "NEWO CLI: sync flows/skills between NEWO and local files, multi-project support, import AKB articles",
-  "type": "module",
+  "version": "1.5.2",
+  "description": "NEWO CLI: sync AI Agent skills between NEWO platform and local files. Multi-customer workspaces, Git-first workflows, comprehensive project management.",
   "bin": {
     "newo": "dist/cli.js"
   },
@@ -46,14 +45,21 @@
     "dotenv": "^16.4.5",
     "fs-extra": "^11.2.0",
     "js-yaml": "^4.1.0",
-    "minimist": "^1.2.8"
+    "minimist": "^1.2.8",
+    "p-limit": "^5.0.0"
   },
   "devDependencies": {
+    "@types/chai": "^4.3.11",
     "@types/fs-extra": "^11.0.4",
     "@types/js-yaml": "^4.0.9",
     "@types/minimist": "^1.2.5",
     "@types/node": "^22.5.4",
+    "@types/sinon": "^17.0.3",
+    "c8": "^9.1.0",
+    "chai": "^5.0.2",
     "mocha": "^10.2.0",
+    "sinon": "^18.0.1",
+    "tsx": "^4.20.5",
     "typescript": "^5.6.2"
   },
   "scripts": {
@@ -64,13 +70,14 @@
     "pull": "npm run build && node ./dist/cli.js pull",
     "push": "npm run build && node ./dist/cli.js push",
     "status": "npm run build && node ./dist/cli.js status",
-    "clean": "rm -rf dist",
+    "clean": "rm -rf dist coverage",
     "typecheck": "tsc --noEmit",
     "lint": "tsc --noEmit --strict",
-    "test": "npm run build && mocha test/*.test.js --timeout 60000",
-    "test:api": "npm run build && mocha test/api.test.js --timeout 30000",
-    "test:sync": "npm run build && mocha test/sync.test.js --timeout 60000",
-    "test:integration": "npm run build && mocha test/integration.test.js --timeout 120000",
+    "test": "npm run build && node --test test/*.test.js",
+    "test:unit": "npm run build && node --test test/{api,sync,auth,hash,fsutil,akb}.test.js",
+    "test:integration": "npm run build && node --test test/integration.test.js",
+    "test:coverage": "npm run build && c8 --reporter=html --reporter=text node --test test/*.test.js",
+    "test:mocha": "npm run build && c8 mocha test/*.test.js --timeout 60000",
     "prepublishOnly": "npm run clean && npm run build"
   }
 }

package/src/akb.ts

@@ -4,8 +4,8 @@ import type { ParsedArticle, AkbImportArticle } from './types.js';
 /**
  * Parse AKB file and extract articles
  */
-export function parseAkbFile(filePath: string): ParsedArticle[] {
-  const content = fs.readFileSync(filePath, 'utf8');
+export async function parseAkbFile(filePath: string): Promise<ParsedArticle[]> {
+  const content = await fs.readFile(filePath, 'utf8');
   const articles: ParsedArticle[] = [];
   
   // Split by article separators (---)
@@ -27,23 +27,28 @@ export function parseAkbFile(filePath: string): ParsedArticle[] {
 /**
  * Parse individual article section
  */
-function parseArticleSection(lines: string[]): ParsedArticle | null {
-  let topicName = '';
-  let category = '';
-  let summary = '';
-  let keywords = '';
-  let topicSummary = '';
+function parseArticleSection(lines: readonly string[]): ParsedArticle | null {
+  const state = {
+    topicName: '',
+    category: '',
+    summary: '',
+    keywords: '',
+    topicSummary: ''
+  };
   
   // Find topic name (# r001)
   const topicLine = lines.find(line => line.match(/^#\s+r\d+/));
-  if (!topicLine) return null;
+  if (!topicLine) {
+    console.warn('No topic line found in section');
+    return null;
+  }
   
-  topicName = topicLine.replace(/^#\s+/, '').trim();
+  state.topicName = topicLine.replace(/^#\s+/, '').trim();
   
   // Extract category/subcategory/description (first ## line)
   const categoryLine = lines.find(line => line.startsWith('## ') && line.includes(' / '));
   if (categoryLine) {
-    category = categoryLine.replace(/^##\s+/, '').trim();
+    state.category = categoryLine.replace(/^##\s+/, '').trim();
   }
   
   // Extract summary (second ## line)
@@ -51,7 +56,7 @@ function parseArticleSection(lines: string[]): ParsedArticle | null {
   if (summaryLineIndex >= 0 && summaryLineIndex + 1 < lines.length) {
     const nextLine = lines[summaryLineIndex + 1];
     if (nextLine && nextLine.startsWith('## ') && !nextLine.includes(' / ')) {
-      summary = nextLine.replace(/^##\s+/, '').trim();
+      state.summary = nextLine.replace(/^##\s+/, '').trim();
     }
   }
   
@@ -62,7 +67,7 @@ function parseArticleSection(lines: string[]): ParsedArticle | null {
   if (keywordsLineIndex >= 0) {
     const keywordsLine = lines[keywordsLineIndex];
     if (keywordsLine) {
-      keywords = keywordsLine.replace(/^##\s+/, '').trim();
+      state.keywords = keywordsLine.replace(/^##\s+/, '').trim();
     }
   }
   
@@ -72,19 +77,19 @@ function parseArticleSection(lines: string[]): ParsedArticle | null {
   
   if (categoryStartIndex >= 0 && categoryEndIndex >= 0) {
     const categoryLines = lines.slice(categoryStartIndex, categoryEndIndex + 1);
-    topicSummary = categoryLines.join('\n');
+    state.topicSummary = categoryLines.join('\n');
   }
   
   // Create topic_facts array
-  const topicFacts = [category, summary, keywords].filter(fact => fact.trim() !== '');
+  const topicFacts = [state.category, state.summary, state.keywords].filter(fact => fact.trim() !== '');
   
   return {
-    topic_name: category, // Use the descriptive title as topic_name
+    topic_name: state.category, // Use the descriptive title as topic_name
     persona_id: null, // Will be set when importing
-    topic_summary: topicSummary,
+    topic_summary: state.topicSummary,
     topic_facts: topicFacts,
     confidence: 100,
-    source: topicName, // Use the ID (r001) as source
+    source: state.topicName, // Use the ID (r001) as source
     labels: ['rag_context']
   };
 }

package/src/api.ts

@@ -1,30 +1,25 @@
 import axios, { type AxiosInstance, type InternalAxiosRequestConfig, type AxiosResponse, type AxiosError } from 'axios';
-import dotenv from 'dotenv';
 import { getValidAccessToken, forceReauth } from './auth.js';
+import { ENV } from './env.js';
 import type { 
-  NewoEnvironment, 
   ProjectMeta, 
   Agent, 
   Skill, 
   FlowEvent, 
   FlowState,
-  AkbImportArticle
+  AkbImportArticle,
+  CustomerProfile
 } from './types.js';
 
-dotenv.config();
+// Per-request retry tracking to avoid shared state issues
+const RETRY_SYMBOL = Symbol('retried');
 
-const { NEWO_BASE_URL } = process.env as NewoEnvironment;
-
-export async function makeClient(verbose: boolean = false): Promise<AxiosInstance> {
-  let accessToken = await getValidAccessToken();
+export async function makeClient(verbose: boolean = false, token?: string): Promise<AxiosInstance> {
+  let accessToken = token || await getValidAccessToken();
   if (verbose) console.log('✓ Access token obtained');
 
-  if (!NEWO_BASE_URL) {
-    throw new Error('NEWO_BASE_URL is not set in environment variables');
-  }
-
   const client = axios.create({
-    baseURL: NEWO_BASE_URL,
+    baseURL: ENV.NEWO_BASE_URL,
     headers: { accept: 'application/json' }
   });
 
@@ -41,7 +36,6 @@ export async function makeClient(verbose: boolean = false): Promise<AxiosInstanc
     return config;
   });
 
-  let retried = false;
   client.interceptors.response.use(
     (response: AxiosResponse) => {
       if (verbose) {
@@ -49,7 +43,8 @@ export async function makeClient(verbose: boolean = false): Promise<AxiosInstanc
         if (response.data && Object.keys(response.data).length < 20) {
           console.log('  Response:', JSON.stringify(response.data, null, 2));
         } else if (response.data) {
-          console.log(`  Response: [${typeof response.data}] ${Array.isArray(response.data) ? response.data.length + ' items' : 'large object'}`);
+          const itemCount = Array.isArray(response.data) ? response.data.length : Object.keys(response.data).length;
+          console.log(`  Response: [${typeof response.data}] ${Array.isArray(response.data) ? itemCount + ' items' : 'large object'}`);
         }
       }
       return response;
@@ -61,15 +56,18 @@ export async function makeClient(verbose: boolean = false): Promise<AxiosInstanc
         if (error.response?.data) console.log('  Error data:', error.response.data);
       }
       
-      if (status === 401 && !retried) {
-        retried = true;
-        if (verbose) console.log('🔄 Retrying with fresh token...');
-        accessToken = await forceReauth();
-        
-        if (error.config) {
-          error.config.headers = error.config.headers || {};
-          error.config.headers.Authorization = `Bearer ${accessToken}`;
-          return client.request(error.config);
+      // Use per-request retry tracking to avoid shared state issues
+      const config = error.config as InternalAxiosRequestConfig & { [RETRY_SYMBOL]?: boolean };
+      
+      if (status === 401 && !config?.[RETRY_SYMBOL]) {
+        if (config) {
+          config[RETRY_SYMBOL] = true;
+          if (verbose) console.log('🔄 Retrying with fresh token...');
+          accessToken = await forceReauth();
+          
+          config.headers = config.headers || {};
+          config.headers.Authorization = `Bearer ${accessToken}`;
+          return client.request(config);
         }
       }
       
@@ -124,4 +122,9 @@ export async function listFlowStates(client: AxiosInstance, flowId: string): Pro
 export async function importAkbArticle(client: AxiosInstance, articleData: AkbImportArticle): Promise<unknown> {
   const response = await client.post('/api/v1/akb/append-manual', articleData);
   return response.data;
+}
+
+export async function getCustomerProfile(client: AxiosInstance): Promise<CustomerProfile> {
+  const response = await client.get<CustomerProfile>('/api/v1/customer/profile');
+  return response.data;
 }