Fix/v1 build issues

.github/workflows/unit-test.yml

@@ -66,26 +66,22 @@ jobs:
         working-directory: ./packages/contentstack-query-export
         run: npm run test:unit
 
-      - name: Run tests for Contentstack Apps CLI
-        working-directory: ./packages/contentstack-apps-cli
-        run: npm run test:unit:report:json
-
       - name: Run tests for Contentstack Content Type plugin
         working-directory: ./packages/contentstack-content-type
         run: npm run test:unit
 
       - name: Run tests for Contentstack Regex Validate plugin
         working-directory: ./packages/contentstack-cli-cm-regex-validate
-        run: npm run test:unit
-
-      - name: Run tests for Contentstack Tsgen plugin
-        working-directory: ./packages/contentstack-cli-tsgen
         run: npm run test
 
       - name: Run tests for Contentstack Migrate RTE
         working-directory: ./packages/contentstack-migrate-rte
-        run: npm test
+        run: npm run test
 
       - name: Run tests for Contentstack Bulk Operations
         working-directory: ./packages/contentstack-bulk-operations
-        run: npm test
+        run: npm run test
+
+      - name: Run tests for Contentstack Apps CLI
+        working-directory: ./packages/contentstack-apps-cli
+        run: npm run test:unit:report:json

package.json

@@ -26,5 +26,10 @@
   "packageManager": "pnpm@10.28.0",
   "workspaces": [
     "packages/*"
-  ]
+  ],
+  "pnpm": {
+    "overrides": {
+      "@types/minimatch": "3.0.5"
+    }
+  }
 }

packages/contentstack-apps-cli/package.json

@@ -49,7 +49,6 @@
     "eslint": "^8.57.1",
     "eslint-config-oclif": "^6.0.157",
     "eslint-config-oclif-typescript": "^3.1.14",
-    "fancy-test": "3.0.16",
     "mocha": "^10.8.2",
     "nyc": "^15.1.0",
     "oclif": "^4.23.0",
@@ -80,11 +79,11 @@
     }
   },
   "scripts": {
-    "build": "npm run clean && shx rm -rf lib && tsc -b",
+    "build": "pnpm clean && tsc -b",
     "lint": "eslint . --ext .ts --config .eslintrc",
     "postpack": "shx rm -f oclif.manifest.json",
-    "posttest": "npm run lint",
-    "prepack": "npm run build && oclif manifest && oclif readme",
+    "posttest": "pnpm lint",
+    "prepack": "pnpm build && oclif manifest && oclif readme",
     "test": "mocha --forbid-only \"test/**/*.test.ts\"",
     "version": "oclif readme && git add README.md",
     "clean": "rm -rf ./lib tsconfig.tsbuildinfo oclif.manifest.json",

packages/contentstack-apps-cli/test/helpers/init.js

@@ -4,4 +4,13 @@ process.env.NODE_ENV = 'development'
 
 global.oclif = global.oclif || {}
 global.oclif.columns = 80
-global.commonMock = require(path.join(__dirname, '../unit/mock/common.mock.json'))
+global.commonMock = require(path.join(__dirname, '../unit/mock/common.mock.json'))
+
+const { configHandler } = require('@contentstack/cli-utilities')
+if (!configHandler.get('region')) {
+  configHandler.set('region', {
+    name: 'NA',
+    cma: 'https://api.contentstack.io',
+    cda: 'https://cdn.contentstack.io',
+  })
+}

packages/contentstack-apps-cli/test/unit/commands/app/deploy.test.ts

@@ -4,40 +4,15 @@ import { runCommand } from "@oclif/test";
 import { cliux, configHandler } from "@contentstack/cli-utilities";
 import messages, { $t } from "../../../../src/messages";
 const mock = (global as any).commonMock;
-import { getDeveloperHubUrl } from "../../../../src/util/inquirer";
 import sinon from "sinon";
 import { stubAuthentication } from "../../helpers/auth-stub-helper";
 import Deploy from "../../../../src/commands/app/deploy";
 import { BaseCommand } from "../../../../src/base-command";
-import { join } from "path";
-
-const region = configHandler.get("region");
-// Commands run from lib/ (oclif.commands); stub the same classes/modules the running command uses
-let BaseCommandToStub: typeof BaseCommand;
-let LibDeploy: typeof Deploy;
-let libCommonUtils: any;
-let libInquirer: any;
-try {
-  BaseCommandToStub = require(join(process.cwd(), "lib", "base-command")).BaseCommand;
-} catch {
-  BaseCommandToStub = BaseCommand;
-}
-try {
-  LibDeploy = require(join(process.cwd(), "lib", "commands", "app", "deploy")).default;
-} catch {
-  LibDeploy = Deploy;
-}
-try {
-  libCommonUtils = require(join(process.cwd(), "lib", "util", "common-utils"));
-} catch {
-  libCommonUtils = require("../../../../src/util/common-utils");
-}
-try {
-  libInquirer = require(join(process.cwd(), "lib", "util", "inquirer"));
-} catch {
-  libInquirer = require("../../../../src/util/inquirer");
-}
-const developerHubBaseUrl = getDeveloperHubUrl();
+import * as libCommonUtils from "../../../../src/util/common-utils";
+import * as libInquirer from "../../../../src/util/inquirer";
+
+let region!: { cma: string; name: string; cda: string };
+let developerHubBaseUrl!: string;
 
 describe("app:deploy", () => {
   let sandbox: sinon.SinonSandbox;
@@ -52,6 +27,8 @@ describe("app:deploy", () => {
 
     // Stub authentication using shared helper
     stubAuthentication(sandbox);
+    region = configHandler.get("region");
+    developerHubBaseUrl = libInquirer.getDeveloperHubUrl();
 
     sandbox.stub(cliux, "loader").callsFake(() => {});
     sandbox.stub(cliux, "inquire").callsFake((prompt: any) => {
@@ -64,7 +41,7 @@ describe("app:deploy", () => {
       return Promise.resolve(cases[prompt.name]);
     });
 
-    // Stub utilities used by the running command (lib/util); stub same module it requires
+    // Stub utilities used by the running command (src/util); same module instances oclif loads via tsPath
     sandbox.stub(libCommonUtils, "getProjects").resolves([]);
     sandbox.stub(libCommonUtils, "updateApp").resolves();
     sandbox.stub(libCommonUtils, "disconnectApp").resolves();
@@ -76,14 +53,14 @@ describe("app:deploy", () => {
     sandbox.stub(libInquirer, "getAppUrl").resolves("https://example.com");
     sandbox.stub(libInquirer, "askProjectType").resolves("existing");
     sandbox.stub(libInquirer, "askConfirmation").resolves(false);
-    sandbox.stub(libInquirer, "selectProject").resolves(null);
+    sandbox.stub(libInquirer, "selectProject").resolves(undefined);
     sandbox.stub(libInquirer, "askProjectName").resolves("test-project");
 
     // Stub Launch.run
     sandbox.stub(require("@contentstack/cli-launch").Launch, "run").resolves();
 
-    // Stub getApolloClient on the class that actually runs (lib Deploy) so no real GraphQL runs
-    sandbox.stub(LibDeploy.prototype, "getApolloClient").resolves({
+    // Stub getApolloClient so no real GraphQL runs
+    sandbox.stub(Deploy.prototype, "getApolloClient").resolves({
       query: () =>
         Promise.resolve({
           data: {
@@ -94,7 +71,7 @@ describe("app:deploy", () => {
         }),
     } as any);
 
-    // Stub SDK init so no real HTTP is made (cli-utilities exports use getters so we can't stub those).
+    // Stub SDK init so no real HTTP is made
     const mockManagementSdk = {
       organization: () => ({
         fetchAll: () =>
@@ -116,15 +93,15 @@ describe("app:deploy", () => {
         }),
       }),
     };
-    sandbox.stub(BaseCommandToStub.prototype, "initCmaSDK").callsFake(async function (this: any) {
+    sandbox.stub(BaseCommand.prototype, "initCmaSDK").callsFake(async function (this: any) {
       this.managementSdk = mockManagementSdk;
       this.managementAppSdk = mockManagementSdk;
     });
-    sandbox.stub(BaseCommandToStub.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
+    sandbox.stub(BaseCommand.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
       this.marketplaceAppSdk = mockMarketplaceSdk;
     });
 
-    // Nock CMA and developer hub (SDK may use :443 or different param order)
+    // Nock CMA and developer hub
     const cmaOrigin = region.cma.replace(/\/$/, "");
     const orgReply = {
       organizations: mock.organizations,
@@ -213,7 +190,7 @@ describe("app:deploy", () => {
       sandbox.stub(process, "exit").callsFake(((code?: number) => {
         throw new Error(`process.exit(${code})`);
       }) as any);
-      sandbox.stub(LibDeploy.prototype, "getApolloClient").resolves({
+      sandbox.stub(Deploy.prototype, "getApolloClient").resolves({
         query: () =>
           Promise.resolve({
             data: { projects: { edges: [] } },
@@ -240,11 +217,11 @@ describe("app:deploy", () => {
           }),
         }),
       };
-      sandbox.stub(BaseCommandToStub.prototype, "initCmaSDK").callsFake(async function (this: any) {
+      sandbox.stub(BaseCommand.prototype, "initCmaSDK").callsFake(async function (this: any) {
         this.managementSdk = mockMgmt;
         this.managementAppSdk = mockMgmt;
       });
-      sandbox.stub(BaseCommandToStub.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
+      sandbox.stub(BaseCommand.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
         this.marketplaceAppSdk = mockMkt;
       });
       stubAuthentication(sandbox);
@@ -287,7 +264,7 @@ describe("app:deploy", () => {
       sandbox.stub(process, "exit").callsFake(((code?: number) => {
         throw new Error(`process.exit(${code})`);
       }) as any);
-      sandbox.stub(LibDeploy.prototype, "getApolloClient").resolves({
+      sandbox.stub(Deploy.prototype, "getApolloClient").resolves({
         query: () =>
           Promise.resolve({
             data: {
@@ -330,11 +307,11 @@ describe("app:deploy", () => {
           }),
         }),
       };
-      sandbox.stub(BaseCommandToStub.prototype, "initCmaSDK").callsFake(async function (this: any) {
+      sandbox.stub(BaseCommand.prototype, "initCmaSDK").callsFake(async function (this: any) {
         this.managementSdk = mockMgmt;
         this.managementAppSdk = mockMgmt;
       });
-      sandbox.stub(BaseCommandToStub.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
+      sandbox.stub(BaseCommand.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
         this.marketplaceAppSdk = mockMkt;
       });
       stubAuthentication(sandbox);
@@ -355,14 +332,15 @@ describe("app:deploy", () => {
       sandbox.stub(libInquirer, "askProjectType").resolves("new");
       sandbox.stub(libInquirer, "askProjectName").resolves("new-project");
       sandbox.stub(libInquirer, "askConfirmation").resolves(false);
-      sandbox.stub(libInquirer, "selectProject").resolves(null);
+      sandbox.stub(libInquirer, "selectProject").resolves(undefined);
 
       sandbox.stub(libCommonUtils, "getProjects").resolves([
         {
           name: "new-project",
           uid: "project-2",
           url: "https://new-project.com",
           environmentUid: "env-2",
+          developerHubAppUid: null,
         },
       ]);
       sandbox.stub(libCommonUtils, "setupConfig").returns({

packages/contentstack-apps-cli/test/unit/commands/app/update.test.ts

@@ -14,20 +14,6 @@ import { BaseCommand } from "../../../../src/base-command";
 
 const region = configHandler.get("region");
 
-// Commands run from lib/ (oclif); stub the same class the running command uses
-let BaseCommandToStub: typeof BaseCommand;
-let LibUpdate: typeof Update;
-try {
-  BaseCommandToStub = require(join(process.cwd(), "lib", "base-command")).BaseCommand;
-} catch {
-  BaseCommandToStub = BaseCommand;
-}
-try {
-  LibUpdate = require(join(process.cwd(), "lib", "commands", "app", "update")).default;
-} catch {
-  LibUpdate = Update;
-}
-
 /** Optional override: return a custom marketplace SDK mock for this test. */
 let marketplaceMockOverride: any = null;
 
@@ -55,11 +41,11 @@ function stubUpdateInit(sandbox: sinon.SinonSandbox) {
       }),
     }),
   };
-  sandbox.stub(BaseCommandToStub.prototype, "initCmaSDK").callsFake(async function (this: any) {
+  sandbox.stub(BaseCommand.prototype, "initCmaSDK").callsFake(async function (this: any) {
     this.managementSdk = mockManagementSdk;
     this.managementAppSdk = mockManagementSdk;
   });
-  sandbox.stub(BaseCommandToStub.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
+  sandbox.stub(BaseCommand.prototype, "initMarketplaceSDK").callsFake(async function (this: any) {
     this.marketplaceAppSdk = marketplaceMockOverride ?? defaultMarketplaceSdk;
   });
 }
@@ -95,7 +81,7 @@ describe("app:update", () => {
       sandbox.stub(fs, "writeFileSync").callsFake(() => {});
 
       sandbox
-        .stub(LibUpdate.prototype, "updateAppOnDeveloperHub")
+        .stub(Update.prototype, "updateAppOnDeveloperHub")
         .callsFake(async function (this: any) {
           this.log(this.messages.APP_UPDATE_SUCCESS, "info");
         });
@@ -193,7 +179,7 @@ describe("app:update", () => {
   describe("Update app with wrong app-uid API failure", () => {
     beforeEach(() => {
       sandbox
-        .stub(LibUpdate.prototype, "updateAppOnDeveloperHub")
+        .stub(Update.prototype, "updateAppOnDeveloperHub")
         .callsFake(async function (this: any) {
           this.log(this.messages.INVALID_APP_ID, "error");
           throw { status: 400 };
@@ -214,7 +200,7 @@ describe("app:update", () => {
   describe("Update app API failure", () => {
     beforeEach(() => {
       sandbox
-        .stub(LibUpdate.prototype, "updateAppOnDeveloperHub")
+        .stub(Update.prototype, "updateAppOnDeveloperHub")
         .callsFake(async function (this: any) {
           this.log(this.messages.APP_INVALID_ORG, "error");
           throw { status: 403 };
@@ -234,7 +220,7 @@ describe("app:update", () => {
   describe("Update app with duplicate app name (409 status)", () => {
     beforeEach(() => {
       sandbox
-        .stub(LibUpdate.prototype, "updateAppOnDeveloperHub")
+        .stub(Update.prototype, "updateAppOnDeveloperHub")
         .callsFake(async function (this: any) {
           this.log(
             this.$t(this.messages.DUPLICATE_APP_NAME, {

packages/contentstack-bulk-operations/package.json

@@ -44,7 +44,6 @@
     "eslint-config-oclif-typescript": "^3.1.14",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.5",
-    "fancy-test": "3.0.16",
     "husky": "^9.1.7",
     "lint-staged": "^17.0.2",
     "mocha": "^11.7.5",
@@ -76,23 +75,22 @@
     }
   },
   "scripts": {
-    "build": "npm run clean && shx rm -rf lib && tsc -b",
+    "build": "pnpm clean",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "format": "prettier --write \"src/**/*.ts\" \"test/**/*.ts\"",
     "format:check": "prettier --check \"src/**/*.ts\" \"test/**/*.ts\"",
     "postpack": "shx rm -f oclif.manifest.json",
-    "posttest": "npm run lint",
-    "prepack": "npm run build && oclif manifest && oclif readme && npm run changelog",
+    "posttest": "pnpm lint",
+    "prepack": "pnpm build && oclif manifest && oclif readme && pnpm changelog",
     "changelog": "conventional-changelog -p angular -i CHANGELOG.md -s",
     "test": "mocha --forbid-only \"test/**/*.test.ts\"",
     "test:coverage": "nyc --extension .ts mocha --forbid-only \"test/**/*.test.ts\"",
     "test:coverage:report": "nyc --reporter=lcov --reporter=text --reporter=clover --reporter=json-summary --extension .ts mocha --forbid-only \"test/**/*.test.ts\"",
     "test:json": "mocha --forbid-only \"test/**/*.test.ts\" --reporter json --reporter-options output=report.json",
-    "test:safe": "[ -d test ] && npm test || echo 'No test directory found, skipping tests'",
+    "test:safe": "[ -d test ] && pnpm test || echo 'No test directory found, skipping tests'",
     "version": "oclif readme && git add README.md",
-    "clean": "rm -rf ./lib tsconfig.tsbuildinfo oclif.manifest.json",
-    "prepare": "npx husky && chmod +x .husky/pre-commit"
+    "clean": "rm -rf ./lib tsconfig.tsbuildinfo oclif.manifest.json"
   },
   "engines": {
     "node": ">=20.19.0"

packages/contentstack-cli-cm-regex-validate/jest.config.ts

@@ -3,12 +3,12 @@ module.exports = {
   testMatch: ['**/tests/**/*.+(ts|tsx)', '**/?(*.)+(spec|test).+(ts|tsx)'],
   transform: {
     '^.+\\.(ts|tsx)$': 'ts-jest',
-    'node_modules/uuid/.+\\.js$': [
+    'node_modules/.*uuid[^/]*/.+\\.js$': [
       'babel-jest',
       {presets: [['@babel/preset-env', {modules: 'commonjs'}]]},
     ],
   },
-  transformIgnorePatterns: ['/node_modules/(?!uuid/)'],
+  transformIgnorePatterns: ['node_modules/(?!.*uuid)'],
   verbose: true,
   collectCoverage: true,
 }

packages/contentstack-cli-cm-regex-validate/package.json

@@ -61,8 +61,7 @@
     "posttest": "eslint . --ext .ts --config .eslintrc",
     "prepack": "rm -rf lib && tsc -b && oclif manifest && oclif readme",
     "test": "jest --detectOpenHandles --silent",
-    "version": "oclif-dev readme && git add README.md",
-    "prepare": "npx husky && chmod +x .husky/pre-commit"
+    "version": "oclif-dev readme && git add README.md"
   },
   "dependencies": {
     "@contentstack/cli-command": "^1.8.2",