@poetora/cli 0.1.5 → 0.1.7

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
 
-> @poetora/cli@0.1.4 build /home/runner/work/poetora/poetora/packages/cli
+> @poetora/cli@0.1.6 build /home/runner/work/poetora/poetora/packages/cli
 > tsc --project tsconfig.build.json
 

package/bin/cli-builder.js

@@ -25,10 +25,9 @@ export class CliBuilder {
         const checkCommand = new CheckCommand(this.logger, openApiCheckService, accessibilityCheckService, this.packageName);
         const linkCommand = new LinkCommand(this.logger, linkService, this.packageName);
         const updateCommand = new UpdateCommand(this.logger, updateService, this.packageName);
-        const cliVersion = versionService.getCliVersion() || '0.0.0';
         return (yargs(hideBin(process.argv))
             .scriptName(this.packageName)
-            .version(cliVersion)
+            .version()
             .middleware(checkNodeVersion)
             .middleware(suppressConsoleWarnings)
             .command('dev', 'initialize a local preview environment', (yargs) => yargs

package/bin/services/version.service.js

@@ -55,15 +55,15 @@ export class VersionService {
         };
     }
     getCliVersion() {
-        if (process.env.CLI_TEST_MODE === 'true') {
-            return 'test-cli';
-        }
         const y = yargs();
         let version;
         y.showVersion((v) => {
             version = v;
             return false;
         });
+        if (process.env.CLI_TEST_MODE === 'true') {
+            return 'test-cli';
+        }
         if (version === 'unknown') {
             version = LOCAL_LINKED_CLI_VERSION;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poetora/cli",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "The CLI for Poetora documentation Engine",
   "engines": {
     "node": ">=18.0.0"
@@ -32,7 +32,7 @@
     "color": "4.2.3",
     "detect-port": "1.5.1",
     "front-matter": "4.0.2",
-    "fs-extra": "11.2.0",
+    "fs-extra": "11.1.0",
     "ink": "6.3.0",
     "inquirer": "12.3.0",
     "js-yaml": "4.1.0",
@@ -43,7 +43,7 @@
     "yargs": "17.7.1",
     "@poetora/link-rot": "0.0.7",
     "@poetora/prebuild": "0.1.6",
-    "@poetora/previewing": "0.1.6",
+    "@poetora/previewing": "0.1.8",
     "@poetora/shared": "0.1.6",
     "@poetora/validation": "0.1.6"
   },

package/src/cli-builder.ts

@@ -58,13 +58,10 @@ export class CliBuilder {
     const linkCommand = new LinkCommand(this.logger, linkService, this.packageName);
     const updateCommand = new UpdateCommand(this.logger, updateService, this.packageName);
 
-    // Get CLI version for yargs
-    const cliVersion = versionService.getCliVersion() || '0.0.0';
-
     return (
       yargs(hideBin(process.argv))
         .scriptName(this.packageName)
-        .version(cliVersion) // Explicitly set version to fix "unknown" issue
+        .version()
         .middleware(checkNodeVersion) // Check Node.js version before any command
         .middleware(suppressConsoleWarnings) // Suppress known console warnings
         // Dev command
@@ -252,7 +249,7 @@ export class CliBuilder {
           () => undefined,
           async () => {
             const versions = versionService.getVersions();
-            // Use chalk for consistent formatting (mintlify uses ink Text, we use chalk)
+            // Use chalk for consistent formatting
             console.log(`${chalk.bold.green('cli version')} ${versions.cli}`);
             console.log(`${chalk.bold.green('client version')} ${versions.client}`);
             process.exit(0);
@@ -261,6 +258,8 @@ export class CliBuilder {
         // Error handling
         .strictCommands()
         .demandCommand(1, 'unknown command. see above for the list of supported commands.')
+
+        // Alias option flags --help = -h, default --version = -v
         .alias('h', 'help')
         .alias('v', 'version')
     );

package/src/cli.ts

@@ -10,7 +10,7 @@ export interface CliOptions {
 }
 
 export const cli = (options: CliOptions): void => {
-  // Initialize ink rendering for logs (like mintlify)
+  // Initialize ink rendering for logs
   render(React.createElement(Logs));
 
   const builder = new CliBuilder(options.packageName);

package/src/commands/__tests__/base.command.test.ts

@@ -104,14 +104,11 @@ describe('BaseCommand', () => {
       (command as unknown as { execute: typeof command.execute }).execute = vi
         .fn()
         .mockRejectedValue('string error');
-      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
       await expect(command.run({ value: 'test' })).rejects.toBe('string error');
 
       expect(mockLogger.error).toHaveBeenCalledWith('An unexpected error occurred');
-      expect(consoleErrorSpy).toHaveBeenCalledWith('string error');
-
-      consoleErrorSpy.mockRestore();
+      expect(mockLogger.logColor).toHaveBeenCalledWith('string error', 'gray');
     });
 
     it('should show stack trace in debug mode', async () => {
@@ -122,13 +119,10 @@ describe('BaseCommand', () => {
       (command as unknown as { execute: typeof command.execute }).execute = vi
         .fn()
         .mockRejectedValue(error);
-      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
       await expect(command.run({ value: 'test' })).rejects.toThrow(error);
 
-      expect(consoleErrorSpy).toHaveBeenCalledWith(error.stack);
-
-      consoleErrorSpy.mockRestore();
+      expect(mockLogger.logColor).toHaveBeenCalledWith(error.stack ?? '', 'gray');
       process.env.DEBUG = originalDebug;
     });
   });

package/src/services/version.service.ts

@@ -83,10 +83,6 @@ export class VersionService {
    * Get CLI version from package.json
    */
   getCliVersion(): string | undefined {
-    if (process.env.CLI_TEST_MODE === 'true') {
-      return 'test-cli';
-    }
-
     const y = yargs();
     let version: string | undefined;
 
@@ -95,7 +91,11 @@ export class VersionService {
       return false;
     });
 
-    // Handle npm link case where version is 'unknown'
+    if (process.env.CLI_TEST_MODE === 'true') {
+      return 'test-cli';
+    }
+
+    // When running npm link or pnpm link, the version is 'unknown'
     if (version === 'unknown') {
       version = LOCAL_LINKED_CLI_VERSION;
     }

package/src/utils/console-logger.ts

@@ -18,7 +18,6 @@ import type { ILogger } from './logger.interface.js';
 /**
  * Logger implementation using ink and addLog from @poetora/previewing
  * Provides consistent output with the preview server's logging system
- * Following mintlify's architecture pattern
  */
 export class ConsoleLogger implements ILogger {
   info(message: string): void {