PR comment fix

Author: cs-raj

.talismanrc

@@ -1,4 +1,6 @@
 fileignoreconfig:
+- filename: pnpm-lock.yaml
+  checksum: 14f5e6720c8596c182735dffd28e7da2e5175670ec24cd8299809fb05d6735a5
 - filename: package-lock.json
-  checksum: 72a462b8cacd9977871d7e5c6c15ba8be8c217930955667cb543ad93a41c1989
+  checksum: 12cf2fe3303ea849646a97f767e720ff48847ecdcf4439140488dfd3157a0844
 version: "1.0"

package-lock.json

@@ -42,10 +42,7 @@ export default class LoginCommand extends BaseCommand<typeof LoginCommand> {
     }),
     'totp-secret': flags.string({
       description: 'TOTP secret for 2FA authentication.',
-      multiple: false,
-      required: false,
       exclusive: ['oauth'],
-      hidden: false,
     }),
     oauth: flags.boolean({
       description: 'Enables single sign-on (SSO) in Contentstack CLI.',

packages/contentstack-auth/src/commands/auth/login.ts

@@ -17,7 +17,7 @@ export const askOTPChannel = async (): Promise<string> => {
     name: 'otpChannel',
     message: 'CLI_AUTH_LOGIN_ASK_CHANNEL_FOR_OTP',
     choices: [
-      { name: '2FA Authenticator App', value: '2fa_authenticator_app' },
+      { name: 'Authenticator App', value: 'authenticator_app' },
       { name: 'SMS', value: 'sms' },
     ],
   });

packages/contentstack-auth/src/utils/interactive.ts

@@ -41,7 +41,7 @@ class TOTPHandler {
       // Validate and normalize secret
       const normalizedSecret = secret.toUpperCase();
       if (!this.isValidBase32(normalizedSecret)) {
-        throw new Error('Invalid TOTP secret format');
+        throw new Error('Something went wrong with your authentication setup. Please check the secret or try again');
       }
 
       // Generate TOTP code
@@ -50,7 +50,7 @@ class TOTPHandler {
       return code;
     } catch (error) {
       log.debug('Failed to generate TOTP code', { module: 'totp-handler', error });
-      throw new Error('Failed to generate TOTP code from provided secret');
+              throw new Error('Something went wrong with your authentication setup. Please check the secret or try again');
     }
   }
 
@@ -73,7 +73,7 @@ class TOTPHandler {
         source = 'stored configuration';
       } catch (error) {
         log.debug('Failed to decrypt stored TOTP secret', { module: 'totp-handler', error });
-        throw new Error('Failed to decrypt stored TOTP secret');
+        throw new Error('Authentication failed. Please try again.');
       }
     }
 
@@ -84,7 +84,7 @@ class TOTPHandler {
         return code;
       } catch (error) {
         log.debug('Failed to generate TOTP code', { module: 'totp-handler', error, source });
-        const message = `Failed to use TOTP secret from ${source}. Please enter the code manually.`;
+        const message = `We couldn't retrieve your authentication code. Please enter it manually.`;
         cliux.print('Consider reconfiguring TOTP using config:totp:add');
         throw new Error(message);
       }
@@ -103,7 +103,7 @@ class TOTPHandler {
   async getManualTOTPCode(): Promise<string> {
     const code = await askOTP();
     if (!/^\d{6}$/.test(code)) {
-      throw new Error('Invalid TOTP code format. Code must be 6 digits.');
+      throw new Error('Invalid authentication code.');
     }
     return code;
   }

packages/contentstack-auth/src/utils/totp-handler.ts

@@ -16,7 +16,7 @@ describe('TOTP Handler', () => {
     });
 
     it('should throw error for invalid secret', () => {
-      expect(() => totpHandler.generateTOTPFromSecret(invalidSecret)).to.throw('Failed to generate TOTP code from provided secret');
+      expect(() => totpHandler.generateTOTPFromSecret(invalidSecret)).to.throw('Something went wrong with your authentication setup. Please check the secret or try again');
     });
 
     it('should normalize secret to uppercase', () => {
@@ -74,7 +74,7 @@ describe('TOTP Handler', () => {
         await totpHandler.getTOTPCode();
         expect.fail('Should have thrown an error');
       } catch (error) {
-        expect(error).to.be.an('error').with.property('message', 'Failed to decrypt stored TOTP secret');
+        expect(error).to.be.an('error').with.property('message', 'Authentication failed. Please try again.');
       }
     });
   });

packages/contentstack-auth/test/unit/utils/totp-handler.test.ts

@@ -51,6 +51,7 @@
     "inquirer": "8.2.6",
     "node-machine-id": "^1.1.12",
     "open": "^8.4.2",
+    "otplib": "^12.0.1",
     "semver": "^7.7.2",
     "short-uuid": "^4.2.2",
     "uuid": "^9.0.1",
@@ -167,4 +168,4 @@
     }
   },
   "repository": "https://github.com/contentstack/cli"
-}
+}