@fnndsc/chili 3.4.0 → 3.6.0

package/dist/feeds/feedHandler.js

@@ -9,6 +9,7 @@ import { feeds_searchByTerm, feed_deleteById } from "../commands/feeds/delete.js
 import { prompt_confirm } from "../utils/ui.js";
 import { feed_create as feed_create_command } from "../commands/feed/create.js"; // Original name
 import { feedCreate_render } from "../views/feed.js"; // Still needed for feedCreate_render
+import { chiliErrLog, chiliLog } from "../screen/output.js";
 // import { FeedListResult } from "../commands/feeds/list.js"; // No longer needed
 /**
  * Handles commands related to groups of ChRIS feeds.
@@ -25,10 +26,10 @@ export class FeedGroupHandler {
     // async feeds_list(options: CLIoptions): Promise<void> {
     //   try {
     //     const { feeds, selectedFields }: FeedListResult = await feeds_fetchList(options);
-    //     console.log(feedList_render(feeds, selectedFields, { table: options.table, csv: options.csv }));
+    //     chiliLog(feedList_render(feeds, selectedFields, { table: options.table, csv: options.csv }));
     //   } catch (error: unknown) {
     //     const msg = error instanceof Error ? error.message : String(error);
-    //     console.error(chalk.red(`Error listing feeds: ${msg}`));
+    //     chiliErrLog(chalk.red(`Error listing feeds: ${msg}`));
     //   }
     // }
     /**
@@ -41,11 +42,11 @@ export class FeedGroupHandler {
                 table_display(fields, ["fields"]);
             }
             else {
-                console.log(`No resource fields found for ${this.assetName}.`);
+                chiliLog(`No resource fields found for ${this.assetName}.`);
             }
         }
         catch (error) {
-            console.log(errorStack.stack_search(this.assetName)[0]);
+            chiliLog(errorStack.stack_search(this.assetName)[0]);
         }
     }
     /**
@@ -59,18 +60,18 @@ export class FeedGroupHandler {
         for (const searchPart of searchParts) {
             const feedIds = await this.baseGroupHandler.IDs_getFromSearch({ search: searchPart });
             if (!feedIds || feedIds.length === 0) {
-                console.log(`No feeds found matching: ${searchPart}`);
+                chiliLog(`No feeds found matching: ${searchPart}`);
                 continue;
             }
             for (const feedId of feedIds) {
-                console.log(`Sharing feed ID: ${feedId}...`);
+                chiliLog(`Sharing feed ID: ${feedId}...`);
                 const shareOptions = { is_public: options.is_public === true };
                 const success = await feed_shareById(Number(feedId), shareOptions);
                 if (success) {
-                    console.log(`Feed ID ${feedId} shared successfully.`);
+                    chiliLog(`Feed ID ${feedId} shared successfully.`);
                 }
                 else {
-                    console.error(`Failed to share feed ID ${feedId}.`);
+                    chiliErrLog(`Failed to share feed ID ${feedId}.`);
                 }
             }
         }
@@ -85,11 +86,11 @@ export class FeedGroupHandler {
         for (const searchPart of searchParts) {
             const items = await feeds_searchByTerm(searchPart);
             if (items.length === 0) {
-                console.log(`No feeds found matching: ${searchPart}`);
+                chiliLog(`No feeds found matching: ${searchPart}`);
                 continue;
             }
             for (const item of items) {
-                console.log(`Preparing to delete Feed: ID=${item.id}, Name=${item.name}`);
+                chiliLog(`Preparing to delete Feed: ID=${item.id}, Name=${item.name}`);
                 if (!options.force) {
                     const confirmed = await prompt_confirm(`Are you sure you want to delete feed ${item.name} (ID: ${item.id})?`);
                     if (!confirmed)
@@ -98,10 +99,10 @@ export class FeedGroupHandler {
                 const id = typeof item.id === "number" ? item.id : Number(item.id);
                 const success = await feed_deleteById(id);
                 if (success) {
-                    console.log(`Deleted feed ${item.id}`);
+                    chiliLog(`Deleted feed ${item.id}`);
                 }
                 else {
-                    console.error(`Failed to delete feed ${item.id}`);
+                    chiliErrLog(`Failed to delete feed ${item.id}`);
                 }
             }
         }
@@ -160,20 +161,20 @@ export class FeedMemberHandler {
         try {
             const feed = await feed_create_command(options);
             if (feed) {
-                console.log(feedCreate_render(feed));
+                chiliLog(feedCreate_render(feed));
                 return feed;
             }
-            console.error(chalk.red("Feed creation returned null result."));
+            chiliErrLog(chalk.red("Feed creation returned null result."));
             const errors = errorStack.allOfType_get('error'); // Keep error reporting
             if (errors.length > 0) {
-                console.error(chalk.red('Errors:'));
-                errors.forEach(e => console.error(chalk.red(`  - ${e}`)));
+                chiliErrLog(chalk.red('Errors:'));
+                errors.forEach(e => chiliErrLog(chalk.red(`  - ${e}`)));
             }
             return null;
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
-            console.error(chalk.red(`Error: ${message}`));
+            chiliErrLog(chalk.red(`Error: ${message}`));
             return null;
         }
     }
@@ -197,7 +198,7 @@ export class FeedMemberHandler {
             });
         }
         else {
-            console.error(`Failed to find '${this.assetName}' command. The 'new' subcommand was not added.`);
+            chiliErrLog(`Failed to find '${this.assetName}' command. The 'new' subcommand was not added.`);
         }
     }
 }

package/dist/filesystem/fileGroupHandler.d.ts

@@ -13,18 +13,31 @@ import { CLIoptions } from "../utils/cli.js";
  * Handles commands related to groups of ChRIS files, links, or directories.
  */
 export declare class FileGroupHandler {
-    private baseGroupHandler;
     private controller;
     readonly assetName: string;
+    private readonly path?;
     private constructor();
     /**
-     * Factory method to create a new FileGroupHandler instance.
+     * Creates a FileGroupHandler for an asset type. This performs no network work
+     * and resolves no ChRIS context — command registration needs only the asset
+     * name, and the controller is resolved lazily on first use (see
+     * {@link controller_ensure}). Keeping setup cheap means an unrelated command
+     * (or a directory that is not a ChRIS folder) pays no file-context cost.
      *
      * @param assetName - The type of asset to handle ('files', 'links', 'dirs').
-     * @param path - Optional path within ChRIS FS. Defaults to current ChRIS folder context.
-     * @returns A Promise resolving to a new FileGroupHandler instance.
+     * @param path - Optional path within ChRIS FS the controller binds to;
+     *   defaults to the current ChRIS folder context.
+     * @returns A new FileGroupHandler instance.
+     */
+    static handler_create(assetName: string, path?: string): FileGroupHandler;
+    /**
+     * Lazily resolves and memoizes the FileController for this asset against the
+     * bound path (or the current ChRIS context).
+     *
+     * @returns The controller, or null if the context cannot be resolved here
+     *   (e.g. the current directory is not a ChRIS folder).
      */
-    static handler_create(assetName: string, path?: string): Promise<FileGroupHandler>;
+    private controller_ensure;
     /**
      * Lists files, links, or directories using the new command logic.
      */

package/dist/filesystem/fileGroupHandler.js

@@ -12,35 +12,50 @@ import { files_searchByTerm, files_deleteById } from "../commands/files/delete.j
 import { prompt_confirm } from "../utils/ui.js";
 import { files_viewContent } from "../commands/file/view.js";
 import { fileList_render } from "../views/file.js";
+import { chiliErrLog, chiliLog } from "../screen/output.js";
 /**
  * Handles commands related to groups of ChRIS files, links, or directories.
  */
 export class FileGroupHandler {
-    constructor(controller, assetName) {
-        this.controller = controller;
+    constructor(assetName, path) {
+        // Resolved lazily: only `share` needs a controller; list/fields/delete
+        // re-resolve their context through standalone salsa helpers.
+        this.controller = null;
         this.assetName = assetName;
-        // baseGroupHandler will be used for fieldslist, delete, share until those are refactored
-        this.baseGroupHandler = new BaseGroupHandler(this.assetName, this.controller.chrisObject);
+        this.path = path;
     }
     /**
-     * Factory method to create a new FileGroupHandler instance.
+     * Creates a FileGroupHandler for an asset type. This performs no network work
+     * and resolves no ChRIS context — command registration needs only the asset
+     * name, and the controller is resolved lazily on first use (see
+     * {@link controller_ensure}). Keeping setup cheap means an unrelated command
+     * (or a directory that is not a ChRIS folder) pays no file-context cost.
      *
      * @param assetName - The type of asset to handle ('files', 'links', 'dirs').
-     * @param path - Optional path within ChRIS FS. Defaults to current ChRIS folder context.
-     * @returns A Promise resolving to a new FileGroupHandler instance.
+     * @param path - Optional path within ChRIS FS the controller binds to;
+     *   defaults to the current ChRIS folder context.
+     * @returns A new FileGroupHandler instance.
      */
-    static async handler_create(assetName, path) {
-        try {
-            const controller = await FileController.handler_create(assetName, path);
-            if (controller === null) {
-                throw new Error(`Failed to create FileController for asset type: ${assetName}`);
+    static handler_create(assetName, path) {
+        return new FileGroupHandler(assetName, path);
+    }
+    /**
+     * Lazily resolves and memoizes the FileController for this asset against the
+     * bound path (or the current ChRIS context).
+     *
+     * @returns The controller, or null if the context cannot be resolved here
+     *   (e.g. the current directory is not a ChRIS folder).
+     */
+    async controller_ensure() {
+        if (!this.controller) {
+            try {
+                this.controller = await FileController.handler_create(this.assetName, this.path);
+            }
+            catch {
+                this.controller = null;
             }
-            return new FileGroupHandler(controller, assetName);
-        }
-        catch (error) {
-            const errorMessage = error instanceof Error ? error.message : String(error);
-            throw new Error(`Failed to initialize FileGroupHandler for ${assetName}: ${errorMessage}`);
         }
+        return this.controller;
     }
     /**
      * Lists files, links, or directories using the new command logic.
@@ -49,33 +64,33 @@ export class FileGroupHandler {
         try {
             const results = await files_fetchList(options, this.assetName, path);
             if (!results) {
-                console.error(`No ${this.assetName} resources found. Perhaps check your current context?`);
+                chiliErrLog(`No ${this.assetName} resources found. Perhaps check your current context?`);
                 return;
             }
             if (results.tableData.length === 0) {
-                console.log(`No ${this.assetName} found matching the criteria.`);
+                chiliLog(`No ${this.assetName} found matching the criteria.`);
             }
             else {
                 const uniqueResults = resourceColumns_removeDuplicates(results);
                 // cumin returns dynamic table rows (Record<string, unknown>[]);
                 // narrow to the FileResource view model at this boundary.
-                console.log(fileList_render(uniqueResults.tableData, uniqueResults.selectedFields, { table: options.table, csv: options.csv }));
+                chiliLog(fileList_render(uniqueResults.tableData, uniqueResults.selectedFields, { table: options.table, csv: options.csv }));
             }
         }
         catch (error) {
             const errors = errorStack.stack_search(this.assetName);
             if (errors.length > 0) {
-                console.log(errors[0]);
+                chiliLog(errors[0]);
             }
             else {
                 const msg = error instanceof Error ? error.message : String(error);
-                console.error(`Error: ${msg}`);
+                chiliErrLog(`Error: ${msg}`);
                 const errObj = error;
                 if (msg.includes("Internal server error") || (errObj && errObj.response && errObj.response.status === 500)) {
-                    console.error(chalk.yellow("\nHint: This indicates a problem on the ChRIS server (CUBE)."));
-                    console.error(chalk.yellow("      Please contact your system administrator or check the CUBE logs."));
-                    console.error(chalk.yellow("      To see full debug output, run the command again with CHILI_DEBUG=true:"));
-                    console.error(chalk.yellow("      CHILI_DEBUG=true chili ..."));
+                    chiliErrLog(chalk.yellow("\nHint: This indicates a problem on the ChRIS server (CUBE)."));
+                    chiliErrLog(chalk.yellow("      Please contact your system administrator or check the CUBE logs."));
+                    chiliErrLog(chalk.yellow("      To see full debug output, run the command again with CHILI_DEBUG=true:"));
+                    chiliErrLog(chalk.yellow("      CHILI_DEBUG=true chili ..."));
                 }
             }
         }
@@ -90,11 +105,11 @@ export class FileGroupHandler {
                 table_display(fields, ["fields"]);
             }
             else {
-                console.log(`No resource fields found for ${this.assetName}.`);
+                chiliLog(`No resource fields found for ${this.assetName}.`);
             }
         }
         catch (error) {
-            console.log(errorStack.stack_search(this.assetName)[0]);
+            chiliLog(errorStack.stack_search(this.assetName)[0]);
         }
     }
     /**
@@ -105,16 +120,16 @@ export class FileGroupHandler {
         for (const searchPart of searchParts) {
             const items = await files_searchByTerm(searchPart, this.assetName);
             if (items.length === 0) {
-                console.log(`No ${this.assetName} found matching: ${searchPart}`);
+                chiliLog(`No ${this.assetName} found matching: ${searchPart}`);
                 continue;
             }
             for (const item of items) {
                 const displayName = item.fname || item.path || item.id;
                 if (!item.id) {
-                    console.error(`Cannot delete item without ID. Details: ${JSON.stringify(item)}`);
+                    chiliErrLog(`Cannot delete item without ID. Details: ${JSON.stringify(item)}`);
                     continue;
                 }
-                console.log(`Preparing to delete ${this.assetName}: ID=${item.id}, Name=${displayName}`);
+                chiliLog(`Preparing to delete ${this.assetName}: ID=${item.id}, Name=${displayName}`);
                 if (!options.force) {
                     const confirmed = await prompt_confirm(`Are you sure you want to delete ${this.assetName} ${displayName} (ID: ${item.id})?`);
                     if (!confirmed)
@@ -122,10 +137,10 @@ export class FileGroupHandler {
                 }
                 const success = await files_deleteById(item.id, this.assetName);
                 if (success) {
-                    console.log(`Deleted ${this.assetName} ${item.id}`);
+                    chiliLog(`Deleted ${this.assetName} ${item.id}`);
                 }
                 else {
-                    console.error(`Failed to delete ${this.assetName} ${item.id}`);
+                    chiliErrLog(`Failed to delete ${this.assetName} ${item.id}`);
                 }
             }
         }
@@ -136,19 +151,24 @@ export class FileGroupHandler {
      * @param options - CLI options for sharing files.
      */
     async files_share(options) {
+        const controller = await this.controller_ensure();
+        if (!controller) {
+            chiliErrLog(`Cannot share ${this.assetName}: no ChRIS context is available in the current directory.`);
+            return;
+        }
         try {
-            console.log(`Sharing ${this.assetName} from ${this.controller.path_get}...`);
+            chiliLog(`Sharing ${this.assetName} from ${controller.path_get}...`);
             if (options.force) {
-                console.log("Force sharing enabled");
+                chiliLog("Force sharing enabled");
             }
-            await this.controller.files_share(options);
+            await controller.files_share(options);
         }
         catch (error) {
             if (error instanceof Error) {
-                console.error(`Error sharing ${this.assetName}: ${error.message}`);
+                chiliErrLog(`Error sharing ${this.assetName}: ${error.message}`);
             }
             else {
-                console.error(`An unknown error occurred while sharing the ${this.assetName}`);
+                chiliErrLog(`An unknown error occurred while sharing the ${this.assetName}`);
             }
         }
     }
@@ -161,7 +181,9 @@ export class FileGroupHandler {
         const fileGroupCommand = program
             .command(this.assetName)
             .description(`Interact with a group of ChRIS ${this.assetName}`);
-        const listCommand = this.baseGroupHandler.baseListCommand_create(async (options) => {
+        // The list command only needs the asset name to register; a context-less
+        // BaseGroupHandler is enough to build it (its actions re-resolve context).
+        const listCommand = new BaseGroupHandler(this.assetName).baseListCommand_create(async (options) => {
             await this.files_list(options);
         });
         // Add path argument and override action to handle it properly
@@ -233,14 +255,14 @@ export class FileMemberHandler {
             const success = await files_create(fileIdentifier, options);
             if (success) {
                 const resolvedChRISPath = await path_resolveChrisFs(fileIdentifier, options);
-                console.log(`File created successfully at: ${resolvedChRISPath}`);
+                chiliLog(`File created successfully at: ${resolvedChRISPath}`);
             }
             // If success is false, files_create would have thrown an error which is caught below.
         }
         catch (error) {
             // Log the error from files_create
             const message = error instanceof Error ? error.message : String(error);
-            console.error(message);
+            chiliErrLog(message);
         }
     }
     /**
@@ -274,7 +296,7 @@ export class FileMemberHandler {
             .action(async (options) => {
             await this.file_cat(options);
         });
-        console.log("FileMemberHandler commands set up successfully");
+        chiliLog("FileMemberHandler commands set up successfully");
     }
     /**
      * Displays the content of the file.
@@ -284,18 +306,18 @@ export class FileMemberHandler {
     async file_cat(options) {
         try {
             const path = this.controller.path_get;
-            console.log(`Viewing file at ${path}`);
+            chiliLog(`Viewing file at ${path}`);
             const content = await files_viewContent(path);
             if (content !== null) {
-                console.log(content);
+                chiliLog(content);
             }
             else {
-                console.error("Failed to view file content (empty or error).");
+                chiliErrLog("Failed to view file content (empty or error).");
             }
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
-            console.error(`Error viewing file: ${message}`);
+            chiliErrLog(`Error viewing file: ${message}`);
         }
     }
 }

package/dist/filesystem/filesystemHandler.js

@@ -4,6 +4,8 @@
  * @module
  */
 import { ChRISinode_create, listParams_fromOptions, BrowserType, } from "@fnndsc/cumin";
+import { table_render } from "../screen/screen.js";
+import { chiliErrLog, chiliLog } from "../screen/output.js";
 /**
  * Lists resources for a given inode path.
  *
@@ -13,7 +15,7 @@ import { ChRISinode_create, listParams_fromOptions, BrowserType, } from "@fnndsc
 async function inodeResources_list(path, options) {
     const chrisInode = await ChRISinode_create(path);
     if (!chrisInode) {
-        console.error(`Could not find path '${path}' in the CUBE filesystem.`);
+        chiliErrLog(`Could not find path '${path}' in the CUBE filesystem.`);
         return;
     }
     const params = listParams_fromOptions({
@@ -33,28 +35,28 @@ async function inodeResources_list(path, options) {
     for (const browserType of browserTypes) {
         const browser = chrisInode.browser_get(browserType);
         if (!browser) {
-            console.error(`WARNING: ${browserType} browser is not available`);
+            chiliErrLog(`WARNING: ${browserType} browser is not available`);
             continue;
         }
         try {
             const resourceGet = browser.resource_get;
             if (!resourceGet) {
-                console.error(`WARNING: ${browserType} resource is null`);
+                chiliErrLog(`WARNING: ${browserType} resource is null`);
                 continue;
             }
             const resourcesList = await resourceGet.resources_getList(params);
             const resourceFields = await resourceGet.resourceFields_get(resourcesList, fieldOptions[browserType]);
             const results = resourceGet.resources_filterByFields(resourceFields);
             if (results) {
-                console.log(`${browserType} resources:`);
-                console.table(results.tableData, results.selectedFields);
+                chiliLog(`${browserType} resources:`);
+                chiliLog(table_render(results.tableData, results.selectedFields));
             }
             else {
-                console.error(`${browserType} resources: not found or could not be filtered`);
+                chiliErrLog(`${browserType} resources: not found or could not be filtered`);
             }
         }
         catch (error) {
-            console.error(`${browserType} resources: not found`);
+            chiliErrLog(`${browserType} resources: not found`);
         }
     }
 }
@@ -67,12 +69,12 @@ async function inodeResources_list(path, options) {
  */
 function inodeFields_list(inodeType, dataObj) {
     if (!dataObj) {
-        console.log("No " + inodeType + " at this path");
+        chiliLog("No " + inodeType + " at this path");
         return false;
     }
     if (dataObj.fields) {
-        console.log(inodeType);
-        console.table(dataObj.fields);
+        chiliLog(inodeType);
+        chiliLog(table_render(dataObj.fields, ["fields"]));
         return true;
     }
     return false;
@@ -85,7 +87,7 @@ function inodeFields_list(inodeType, dataObj) {
 async function inodeResourceFields_list(path = "") {
     const chrisFiles = await ChRISinode_create(path);
     if (!chrisFiles) {
-        console.error(`Could not create ChRISinode for path: ${path}`);
+        chiliErrLog(`Could not create ChRISinode for path: ${path}`);
         return;
     }
     const fileBrowser = chrisFiles.fileBrowser_get;

package/dist/filesystem/inodeCommand.js

@@ -5,6 +5,7 @@
  */
 import { Command } from "commander";
 import { FileGroupHandler, FileMemberHandler } from "./fileGroupHandler.js";
+import { chiliErrLog } from "../screen/output.js";
 /**
  * Sets up the 'inode' command for interacting with ChRIS filesystem resources.
  *
@@ -18,7 +19,7 @@ export async function inodeCommand_setup(program) {
         const args = command.args.slice(1);
         const subcommand = args[0];
         if (subcommand === "files") {
-            const fileGroupHandler = await FileGroupHandler.handler_create("files", path);
+            const fileGroupHandler = FileGroupHandler.handler_create("files", path);
             const filesProgram = new Command();
             fileGroupHandler.fileGroupCommand_setup(filesProgram);
             await filesProgram.parseAsync(args);
@@ -30,7 +31,7 @@ export async function inodeCommand_setup(program) {
             await fileProgram.parseAsync(args);
         }
         else {
-            console.error("Usage: chili inode <path> <files|file> [options]");
+            chiliErrLog("Usage: chili inode <path> <files|file> [options]");
             command.help();
         }
     });

package/dist/handlers/baseGroupHandler.d.ts

@@ -14,8 +14,14 @@ import { TableOptions } from "../screen/screen.js";
 export declare class BaseGroupHandler {
     assetName: string;
     displayOptions: TableOptions;
-    chrisObject: ChRISPluginGroup | ChRISFeedGroup | ChRISEmbeddedResourceGroup<unknown> | ChRISPACSGroup;
-    constructor(assetName: string, chrisObject: ChRISPluginGroup | ChRISFeedGroup | ChRISEmbeddedResourceGroup<unknown> | ChRISPACSGroup);
+    chrisObject: ChRISPluginGroup | ChRISFeedGroup | ChRISEmbeddedResourceGroup<unknown> | ChRISPACSGroup | null;
+    constructor(assetName: string, chrisObject?: ChRISPluginGroup | ChRISFeedGroup | ChRISEmbeddedResourceGroup<unknown> | ChRISPACSGroup | null);
+    /**
+     * The resolved ChRIS context, for use inside actions. Throws if read before
+     * the context has been set — a lazy handler must resolve its context before
+     * an action dereferences it.
+     */
+    private get chrisObjectResolved();
     /**
      * Lists ChRIS resources based on provided CLI options.
      *