fix(apps-cli): drop oclif manifest from build so tests load src, not lib [DX-9770]

Root cause of the 12 apps-cli deploy/update CI failures: apps-cli's tests stub the
SRC command classes (BaseCommand/Update/initMarketplaceSDK). oclif only loads commands
from src (via ts-node in dev) when NO oclif.manifest.json exists; when a manifest is
present it loads the compiled lib, so the src stubs never apply and the command makes a
real developer-hub HTTP call -> 401 'session timed out'.

The build script had been changed to `pnpm compile && oclif manifest && oclif readme`,
which created the manifest during the CI build step; v2-dev's build was `tsc -b` (no
manifest), which is why v2-dev stayed green. Reproduced locally: no manifest -> tests
pass; manifest present -> tests fail.

Fix: build = `pnpm compile` only. The manifest is still generated by prepack for
publishing. Reverts the exploratory hermetic/diagnostic edits to the test files (not
needed once the manifest cause was found). apps-cli: 61 passing.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NbgqgVDTDmh6c9LwDtMEf9

Author: netrajpatel

packages/contentstack-apps-cli/package.json

@@ -80,7 +80,7 @@
     }
   },
   "scripts": {
-    "build": "pnpm compile && oclif manifest && oclif readme",
+    "build": "pnpm compile",
     "lint": "eslint \"src/**/*.ts\"",
     "postpack": "rm -f oclif.manifest.json",
     "prepack": "pnpm compile && oclif manifest && oclif readme",

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

@@ -12,27 +12,11 @@ import { BaseCommand } from "../../../../src/base-command";
 import * as libCommonUtils from "../../../../src/util/common-utils";
 import * as libInquirer from "../../../../src/util/inquirer";
 
-// Hermetic: seed a complete region at module load so the nock URLs below are deterministic
-// regardless of test execution order or ambient CLI config. These tests previously read
-// whatever region happened to be in the shared config, which made them order-dependent
-// (they passed only when another package seeded a region first, e.g. in the old CI order).
-configHandler.set("region", {
-  name: "NA",
-  cma: "https://api.contentstack.io",
-  cda: "https://cdn.contentstack.io",
-  uiHost: "https://app.contentstack.com",
-});
 const region = configHandler.get("region");
 const BaseCommandToStub = BaseCommand;
 const LibDeploy = Deploy;
 const developerHubBaseUrl = getDeveloperHubUrl();
 
-// TEMP diagnostic (removed once CI is green): reveal any request nock did not intercept.
-nock.emitter.on("no match", (req: any) => {
-  // eslint-disable-next-line no-console
-  console.log("NOCK NO MATCH >>", req?.method, req?.options?.host || req?.host, req?.path || req?.options?.path);
-});
-
 describe("app:deploy", () => {
   let sandbox: sinon.SinonSandbox;
 

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

@@ -12,22 +12,8 @@ import { stubAuthentication } from "../../helpers/auth-stub-helper";
 import Update from "../../../../src/commands/app/update";
 import { BaseCommand } from "../../../../src/base-command";
 
-// Hermetic: seed a complete region at module load so the nock URL(s) below are deterministic
-// regardless of test execution order or ambient CLI config.
-configHandler.set("region", {
-  name: "NA",
-  cma: "https://api.contentstack.io",
-  cda: "https://cdn.contentstack.io",
-  uiHost: "https://app.contentstack.com",
-});
 const region = configHandler.get("region");
 
-// TEMP diagnostic (removed once CI is green): reveal any request nock did not intercept.
-nock.emitter.on("no match", (req: any) => {
-  // eslint-disable-next-line no-console
-  console.log("NOCK NO MATCH >>", req?.method, req?.options?.host || req?.host, req?.path || req?.options?.path);
-});
-
 // oclif loads commands from src/ (ts-node is registered), so stub the src classes directly
 const BaseCommandToStub = BaseCommand;
 const LibUpdate = Update;