@aifabrix/builder 2.11.0 → 2.21.0

package/lib/utils/app-register-api.js

@@ -10,7 +10,7 @@
 
 const chalk = require('chalk');
 const logger = require('./logger');
-const { authenticatedApiCall } = require('./api');
+const { registerApplication } = require('../api/applications.api');
 const { formatApiError } = require('./api-error-handler');
 
 /**
@@ -23,14 +23,9 @@ const { formatApiError } = require('./api-error-handler');
  * @returns {Promise<Object>} API response
  */
 async function callRegisterApi(apiUrl, token, environment, registrationData) {
-  const response = await authenticatedApiCall(
-    `${apiUrl}/api/v1/environments/${encodeURIComponent(environment)}/applications/register`,
-    {
-      method: 'POST',
-      body: JSON.stringify(registrationData)
-    },
-    token
-  );
+  // Use centralized API client
+  const authConfig = { type: 'bearer', token: token };
+  const response = await registerApplication(apiUrl, environment, authConfig, registrationData);
 
   if (!response.success) {
     const formattedError = response.formattedError || formatApiError(response);
@@ -48,7 +43,7 @@ async function callRegisterApi(apiUrl, token, environment, registrationData) {
   }
 
   // Handle API response structure:
-  // makeApiCall returns: { success: true, data: <API response> }
+  // registerApplication returns: { success: true, data: <API response> }
   // API response can be:
   // 1. Direct format: { application: {...}, credentials: {...} }
   // 2. Wrapped format: { success: true, data: { application: {...}, credentials: {...} } }

package/lib/utils/deployment-errors.js

@@ -18,8 +18,17 @@ const { parseErrorResponse } = require('./api-error-handler');
  * @returns {Object} Structured error information
  */
 function handleDeploymentError(error) {
+  if (!error) {
+    return {
+      message: 'Unknown error',
+      code: 'UNKNOWN',
+      timeout: false,
+      status: undefined,
+      data: undefined
+    };
+  }
   const safeError = {
-    message: error.message,
+    message: error.message || 'Unknown error',
     code: error.code || 'UNKNOWN',
     timeout: error.code === 'ECONNABORTED',
     status: error.status || error.response?.status,
@@ -42,6 +51,38 @@ function handleDeploymentError(error) {
  * @throws {Error} User-friendly error message
  */
 async function handleDeploymentErrors(error, appName, url, alreadyLogged = false) {
+  // For validation errors (like URL validation), just re-throw them directly
+  // They already have user-friendly messages
+  // Handle both Error objects and strings
+  let errorMessage = '';
+  if (error instanceof Error) {
+    errorMessage = error.message || '';
+  } else if (typeof error === 'string') {
+    errorMessage = error;
+  } else if (error && typeof error === 'object' && error.message) {
+    errorMessage = error.message;
+  }
+
+  if (errorMessage && (
+    errorMessage.includes('Controller URL must use HTTPS') ||
+    errorMessage.includes('Invalid environment key') ||
+    errorMessage.includes('Environment key is required') ||
+    errorMessage.includes('Authentication configuration is required') ||
+    errorMessage.includes('Invalid controller URL format') ||
+    errorMessage.includes('Controller URL is required')
+  )) {
+    // If error is a string, convert to Error object
+    if (typeof error === 'string') {
+      throw new Error(error);
+    }
+    // If it's already an Error object, re-throw it directly
+    if (error instanceof Error) {
+      throw error;
+    }
+    // Otherwise, create a new Error with the message
+    throw new Error(errorMessage);
+  }
+
   // Log to audit log if not already logged
   if (!alreadyLogged) {
     try {
@@ -63,7 +104,17 @@ async function handleDeploymentErrors(error, appName, url, alreadyLogged = false
 
   // Ensure errorData is not undefined before parsing
   // If errorData is undefined, use the error message instead
-  const errorResponse = errorData !== undefined ? errorData : safeError.message;
+  let errorResponse = errorData !== undefined ? errorData : safeError.message;
+
+  // Ensure errorResponse is a string or object, not an Error object
+  if (errorResponse instanceof Error) {
+    errorResponse = errorResponse.message || 'Unknown error occurred';
+  }
+
+  // Ensure errorResponse is not null or undefined
+  if (errorResponse === null || errorResponse === undefined) {
+    errorResponse = safeError.message || 'Unknown error occurred';
+  }
 
   // Determine if this is a network error
   const isNetworkError = safeError.code === 'ECONNREFUSED' ||
@@ -72,13 +123,44 @@ async function handleDeploymentErrors(error, appName, url, alreadyLogged = false
     safeError.timeout;
 
   // Parse error using error handler
-  const parsedError = parseErrorResponse(errorResponse, safeError.status || 0, isNetworkError);
+  let parsedError;
+  try {
+    parsedError = parseErrorResponse(errorResponse, safeError.status || 0, isNetworkError);
+    // Ensure parsedError is a valid object
+    if (!parsedError || typeof parsedError !== 'object') {
+      throw new Error('parseErrorResponse returned invalid result');
+    }
+  } catch (parseErr) {
+    // If parsing fails, use the safe error message
+    parsedError = {
+      message: safeError.message || 'Unknown error occurred',
+      formatted: safeError.message || 'Unknown error occurred',
+      data: undefined
+    };
+  }
+
+  // Ensure parsedError is always a valid object with required properties
+  if (!parsedError || typeof parsedError !== 'object') {
+    parsedError = {
+      message: safeError.message || 'Unknown error occurred',
+      formatted: safeError.message || 'Unknown error occurred',
+      data: undefined
+    };
+  }
+
+  // Ensure we have a message - handle case where parsedError.message might be undefined
+  const finalErrorMessage = (parsedError && parsedError.message) ? parsedError.message : (safeError.message || 'Unknown error occurred');
+
+  // Validate finalErrorMessage is a string
+  if (typeof finalErrorMessage !== 'string') {
+    throw new Error(`Invalid error message type: ${typeof finalErrorMessage}. Error: ${JSON.stringify(error)}`);
+  }
 
   // Throw clean error message (without emoji) - CLI will format it
-  const formattedError = new Error(parsedError.message);
-  formattedError.formatted = parsedError.formatted;
+  const formattedError = new Error(finalErrorMessage);
+  formattedError.formatted = parsedError?.formatted || finalErrorMessage;
   formattedError.status = safeError.status;
-  formattedError.data = parsedError.data;
+  formattedError.data = parsedError?.data;
   formattedError._logged = true; // Mark as logged to prevent double-logging
   throw formattedError;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aifabrix/builder",
-  "version": "2.11.0",
+  "version": "2.21.0",
   "description": "AI Fabrix Local Fabric & Deployment SDK",
   "main": "lib/index.js",
   "bin": {

package/tatus

@@ -1,181 +0,0 @@
-
-                   SSUUMMMMAARRYY OOFF LLEESSSS CCOOMMMMAANNDDSS
-
-      Commands marked with * may be preceded by a number, _N.
-      Notes in parentheses indicate the behavior if _N is given.
-      A key preceded by a caret indicates the Ctrl key; thus ^K is ctrl-K.
-
-  h  H                 Display this help.
-  q  :q  Q  :Q  ZZ     Exit.
- ---------------------------------------------------------------------------
-
-                           MMOOVVIINNGG
-
-  e  ^E  j  ^N  CR  *  Forward  one line   (or _N lines).
-  y  ^Y  k  ^K  ^P  *  Backward one line   (or _N lines).
-  f  ^F  ^V  SPACE  *  Forward  one window (or _N lines).
-  b  ^B  ESC-v      *  Backward one window (or _N lines).
-  z                 *  Forward  one window (and set window to _N).
-  w                 *  Backward one window (and set window to _N).
-  ESC-SPACE         *  Forward  one window, but don't stop at end-of-file.
-  d  ^D             *  Forward  one half-window (and set half-window to _N).
-  u  ^U             *  Backward one half-window (and set half-window to _N).
-  ESC-)  RightArrow *  Right one half screen width (or _N positions).
-  ESC-(  LeftArrow  *  Left  one half screen width (or _N positions).
-  ESC-}  ^RightArrow   Right to last column displayed.
-  ESC-{  ^LeftArrow    Left  to first column.
-  F                    Forward forever; like "tail -f".
-  ESC-F                Like F but stop when search pattern is found.
-  r  ^R  ^L            Repaint screen.
-  R                    Repaint screen, discarding buffered input.
-        ---------------------------------------------------
-        Default "window" is the screen height.
-        Default "half-window" is half of the screen height.
- ---------------------------------------------------------------------------
-
-                          SSEEAARRCCHHIINNGG
-
-  /_p_a_t_t_e_r_n          *  Search forward for (_N-th) matching line.
-  ?_p_a_t_t_e_r_n          *  Search backward for (_N-th) matching line.
-  n                 *  Repeat previous search (for _N-th occurrence).
-  N                 *  Repeat previous search in reverse direction.
-  ESC-n             *  Repeat previous search, spanning files.
-  ESC-N             *  Repeat previous search, reverse dir. & spanning files.
-  ^O^N  ^On         *  Search forward for (_N-th) OSC8 hyperlink.
-  ^O^P  ^Op         *  Search backward for (_N-th) OSC8 hyperlink.
-  ^O^L  ^Ol            Jump to the currently selected OSC8 hyperlink.
-  ESC-u                Undo (toggle) search highlighting.
-  ESC-U                Clear search highlighting.
-  &_p_a_t_t_e_r_n          *  Display only matching lines.
-        ---------------------------------------------------
-        A search pattern may begin with one or more of:
-        ^N or !  Search for NON-matching lines.
-        ^E or *  Search multiple files (pass thru END OF FILE).
-        ^F or @  Start search at FIRST file (for /) or last file (for ?).
-        ^K       Highlight matches, but don't move (KEEP position).
-        ^R       Don't use REGULAR EXPRESSIONS.
-        ^S _n     Search for match in _n-th parenthesized subpattern.
-        ^W       WRAP search if no match found.
-        ^L       Enter next character literally into pattern.
- ---------------------------------------------------------------------------
-
-                           JJUUMMPPIINNGG
-
-  g  <  ESC-<       *  Go to first line in file (or line _N).
-  G  >  ESC->       *  Go to last line in file (or line _N).
-  p  %              *  Go to beginning of file (or _N percent into file).
-  t                 *  Go to the (_N-th) next tag.
-  T                 *  Go to the (_N-th) previous tag.
-  {  (  [           *  Find close bracket } ) ].
-  }  )  ]           *  Find open bracket { ( [.
-  ESC-^F _<_c_1_> _<_c_2_>  *  Find close bracket _<_c_2_>.
-  ESC-^B _<_c_1_> _<_c_2_>  *  Find open bracket _<_c_1_>.
-        ---------------------------------------------------
-        Each "find close bracket" command goes forward to the close bracket 
-          matching the (_N-th) open bracket in the top line.
-        Each "find open bracket" command goes backward to the open bracket 
-          matching the (_N-th) close bracket in the bottom line.
-
-  m_<_l_e_t_t_e_r_>            Mark the current top line with <letter>.
-  M_<_l_e_t_t_e_r_>            Mark the current bottom line with <letter>.
-  '_<_l_e_t_t_e_r_>            Go to a previously marked position.
-  ''                   Go to the previous position.
-  ^X^X                 Same as '.
-  ESC-m_<_l_e_t_t_e_r_>        Clear a mark.
-        ---------------------------------------------------
-        A mark is any upper-case or lower-case letter.
-        Certain marks are predefined:
-             ^  means  beginning of the file
-             $  means  end of the file
- ---------------------------------------------------------------------------
-
-                        CCHHAANNGGIINNGG FFIILLEESS
-
-  :e [_f_i_l_e]            Examine a new file.
-  ^X^V                 Same as :e.
-  :n                *  Examine the (_N-th) next file from the command line.
-  :p                *  Examine the (_N-th) previous file from the command line.
-  :x                *  Examine the first (or _N-th) file from the command line.
-  ^O^O                 Open the currently selected OSC8 hyperlink.
-  :d                   Delete the current file from the command line list.
-  =  ^G  :f            Print current file name.
- ---------------------------------------------------------------------------
-
-                    MMIISSCCEELLLLAANNEEOOUUSS CCOOMMMMAANNDDSS
-
-  -_<_f_l_a_g_>              Toggle a command line option [see OPTIONS below].
-  --_<_n_a_m_e_>             Toggle a command line option, by name.
-  __<_f_l_a_g_>              Display the setting of a command line option.
-  ___<_n_a_m_e_>             Display the setting of an option, by name.
-  +_c_m_d                 Execute the less cmd each time a new file is examined.
-
-  !_c_o_m_m_a_n_d             Execute the shell command with $SHELL.
-  #_c_o_m_m_a_n_d             Execute the shell command, expanded like a prompt.
-  |XX_c_o_m_m_a_n_d            Pipe file between current pos & mark XX to shell command.
-  s _f_i_l_e               Save input to a file.
-  v                    Edit the current file with $VISUAL or $EDITOR.
-  V                    Print version number of "less".
- ---------------------------------------------------------------------------
-
-                           OOPPTTIIOONNSS
-
-        Most options may be changed either on the command line,
-        or from within less by using the - or -- command.
-        Options may be given in one of two forms: either a single
-        character preceded by a -, or a name preceded by --.
-
-  -?  ........  --help
-                  Display help (from command line).
-  -a  ........  --search-skip-screen
-                  Search skips current screen.
-  -A  ........  --SEARCH-SKIP-SCREEN
-                  Search starts just after target line.
-  -b [_N]  ....  --buffers=[_N]
-                  Number of buffers.
-  -B  ........  --auto-buffers
-                  Don't automatically allocate buffers for pipes.
-  -c  ........  --clear-screen
-                  Repaint by clearing rather than scrolling.
-  -d  ........  --dumb
-                  Dumb terminal.
-  -D xx_c_o_l_o_r  .  --color=xx_c_o_l_o_r
-                  Set screen colors.
-  -e  -E  ....  --quit-at-eof  --QUIT-AT-EOF
-                  Quit at end of file.
-  -f  ........  --force
-                  Force open non-regular files.
-  -F  ........  --quit-if-one-screen
-                  Quit if entire file fits on first screen.
-  -g  ........  --hilite-search
-                  Highlight only last match for searches.
-  -G  ........  --HILITE-SEARCH
-                  Don't highlight any matches for searches.
-  -h [_N]  ....  --max-back-scroll=[_N]
-                  Backward scroll limit.
-  -i  ........  --ignore-case
-                  Ignore case in searches that do not contain uppercase.
-  -I  ........  --IGNORE-CASE
-                  Ignore case in all searches.
-  -j [_N]  ....  --jump-target=[_N]
-                  Screen position of target lines.
-  -J  ........  --status-column
-                  Display a status column at left edge of screen.
-  -k _f_i_l_e  ...  --lesskey-file=_f_i_l_e
-                  Use a compiled lesskey file.
-  -K  ........  --quit-on-intr
-                  Exit less in response to ctrl-C.
-  -L  ........  --no-lessopen
-                  Ignore the LESSOPEN environment variable.
-  -m  -M  ....  --long-prompt  --LONG-PROMPT
-                  Set prompt style.
-  -n .........  --line-numbers
-                  Suppress line numbers in prompts and messages.
-  -N .........  --LINE-NUMBERS
-                  Display line number at start of each line.
-  -o [_f_i_l_e] ..  --log-file=[_f_i_l_e]
-                  Copy to log file (standard input only).
-  -O [_f_i_l_e] ..  --LOG-FILE=[_f_i_l_e]
-                  Copy to log file (unconditionally overwrite).
-  -p _p_a_t_t_e_r_n .  --pattern=[_p_a_t_t_e_r_n]
-                  Start at pattern (from command line).
-  -P [_p_r_o_m_p_t]   --prompt=[_p_r_o_m_p_t]