Merge pull request #2363 from contentstack/development

staging PR

Author: harshithad0703

.github/workflows/release.yml

@@ -173,6 +173,10 @@ jobs:
         if: ${{env.release_releaseAll == 'true' || env.release_plugins_clone == 'true'}}
         working-directory: ./packages/contentstack-clone
         run: npm install
+      - name: Compiling Clone
+        if: ${{ steps.clone-installation.conclusion == 'success'  }}
+        working-directory: ./packages/contentstack-clone
+        run: npm run prepack
       - name: Publishing clone
         uses: JS-DevTools/npm-publish@v3
         if: ${{ steps.clone-installation.conclusion == 'success'  }}
@@ -226,6 +230,10 @@ jobs:
         if: ${{env.release_releaseAll == 'true' || env.release_plugins_migration == 'true'}}
         working-directory: ./packages/contentstack-migration
         run: npm install
+      - name: Compiling Migration
+        if: ${{ steps.migration-installation.conclusion == 'success'  }}
+        working-directory: ./packages/contentstack-migration
+        run: npm run prepack
       - name: Publishing migration
         uses: JS-DevTools/npm-publish@v3
         if: ${{ steps.migration-installation.conclusion == 'success'  }}

.talismanrc

@@ -1,6 +1,6 @@
 fileignoreconfig:
-  - filename: packages/contentstack-export-to-csv/src/types/index.ts
-    checksum: 28c19efb0c13969d40df964f86fa6de444316e320801f19dee7392df7a36851d
-  - filename: package-lock.json
-    checksum: faa364dac78a89d142916b71b5ceba7fdc80acbf8dc71f15dfb38d622c9dc664
+- filename: package-lock.json
+  checksum: 20001b2b16e2ab8cc8a3ccc30cebb5093c5c63f58e5b5fe64889d260a8d4cc56
+- filename: pnpm-lock.yaml
+  checksum: c2982bbd478bd9aabcc12bc162d2a76e2635d70023674a677dea996a8671e9ab
 version: '1.0'

package-lock.json

@@ -19,7 +19,7 @@
   ],
   "dependencies": {
     "@contentstack/cli-command": "~1.7.2",
-    "@contentstack/cli-utilities": "~1.17.0",
+    "@contentstack/cli-utilities": "~1.17.1",
     "@oclif/core": "^4.3.0",
     "@oclif/plugin-help": "^6.2.28",
     "@oclif/plugin-plugins": "^5.4.54",
@@ -85,4 +85,4 @@
   "keywords": [
     "oclif"
   ]
-}
+}

packages/contentstack-audit/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@contentstack/cli-auth",
   "description": "Contentstack CLI plugin for authentication activities",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "author": "Contentstack",
   "bugs": "https://github.com/contentstack/cli/issues",
   "scripts": {
@@ -23,7 +23,7 @@
   },
   "dependencies": {
     "@contentstack/cli-command": "~1.7.2",
-    "@contentstack/cli-utilities": "~1.17.0",
+    "@contentstack/cli-utilities": "~1.17.1",
     "@oclif/core": "^4.3.0",
     "@oclif/plugin-help": "^6.2.28",
     "otplib": "^12.0.1"
@@ -81,4 +81,4 @@
     }
   },
   "repository": "contentstack/cli"
-}
+}

packages/contentstack-auth/package.json

@@ -182,36 +182,6 @@ class AuthHandler {
       }
     });
   }
-
-  /**
-   * Validate token
-   * @param {string} authtoken
-   * @returns {Promise} Promise object returns response object from Contentstack
-   */
-  async validateAuthtoken(authtoken: string): Promise<object> {
-    log.debug('Starting token validation.', { module: 'auth-handler', hasAuthToken: !!authtoken });
-
-    return new Promise((resolve, reject) => {
-      if (authtoken) {
-        log.debug('Making token validation API call.', { module: 'auth-handler' });
-
-        this._client
-          .getUser()
-          .then((user: object) => {
-            log.debug('Token validation successful.', { module: 'auth-handler', user });
-            resolve(user);
-          })
-          .catch((error: Error) => {
-            log.debug('Token validation failed.', { module: 'auth-handler', error: error.message });
-            cliux.print('CLI_AUTH_TOKEN_VALIDATION_FAILED', { color: 'yellow' });
-            reject(error);
-          });
-      } else {
-        log.debug('Token validation failed: no auth token provided.', { module: 'auth-handler' });
-        reject(new Error(messageHandler.parse('CLI_AUTH_TOKEN_VALIDATION_NO_TOKEN')));
-      }
-    });
-  }
 }
 
 export default new AuthHandler();

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

@@ -1,4 +1,3 @@
 export { default as authHandler } from './auth-handler';
 export { default as mfaHandler } from './mfa-handler';
 export * as interactive from './interactive';
-export * as tokenValidation from './tokens-validation';

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

@@ -104,32 +104,6 @@ class MFAHandler {
     }
   }
 
-  /**
-   * Gets MFA code through manual user input
-   * @returns Promise<string> The MFA code
-   * @throws Error if code format is invalid
-   */
-  async getManualMFACode(): Promise<string> {
-    try {
-      const code = await askOTP();
-      if (!/^\d{6}$/.test(code)) {
-        throw new Error(messageHandler.parse('CLI_AUTH_MFA_INVALID_CODE'));
-      }
-      return code;
-    } catch (error) {
-      log.debug('Failed to get MFA code', { module: 'mfa-handler', error });
-      throw error;
-    }
-  }
-
-  /**
-   * Validates an MFA code format
-   * @param code The MFA code to validate
-   * @returns boolean True if valid, false otherwise
-   */
-  isValidMFACode(code: string): boolean {
-    return /^\d{6}$/.test(code);
-  }
 }
 
 export default new MFAHandler();

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

@@ -2,7 +2,6 @@ import { expect } from 'chai';
 import * as sinon from 'sinon';
 import { configHandler, cliux } from '@contentstack/cli-utilities';
 import TokensAddCommand from '../../../src/commands/auth/tokens/add';
-import { tokenValidation } from '../../../src/utils';
 import { stub, assert } from 'sinon';
 import { config as dotenvConfig } from 'dotenv';
 import nock from 'nock';
@@ -31,8 +30,6 @@ function resetConfig() {
   config.delete(`${configKeyTokens}.newToken`);
 }
 describe('Tokens Add Command', () => {
-  let apiKeyValidationStub: sinon.SinonStub;
-  let environmentTokenValidationStub: sinon.SinonStub;
   let printStub: sinon.SinonStub;
   const validAPIKey = conf.validAPIKey;
   const validDeliveryToken = '***REMOVED***';
@@ -43,28 +40,9 @@ describe('Tokens Add Command', () => {
     resetConfig();
     if ((cliux.print as any).restore) (cliux.print as any).restore();
     printStub = stub(cliux, 'print');
-    apiKeyValidationStub = sinon
-      .stub(tokenValidation, 'validateAPIKey')
-      .callsFake(function (client: any, apiKey: string): Promise<any> {
-        if (apiKey === validAPIKey) {
-          return Promise.resolve({ valid: true, message: 'success' });
-        }
-        return Promise.resolve({ valid: false, message: 'failed' });
-      });
-
-    environmentTokenValidationStub = sinon
-      .stub(tokenValidation, 'validateEnvironment')
-      .callsFake(function (client: any, apiKey: string, environment): Promise<any> {
-        if (environment === validEnvironment) {
-          return Promise.resolve({ valid: true, message: 'success' });
-        }
-        return Promise.resolve({ valid: false, message: 'failed' });
-      });
   });
 
   after(() => {
-    apiKeyValidationStub.restore();
-    environmentTokenValidationStub.restore();
     printStub.restore();
     resetConfig();
   });