@postman-cse/onboarding-repo-sync 2.7.1 → 2.7.3

package/dist/action.cjs

@@ -118357,8 +118357,11 @@ var PostmanGatewayAssetsClient = class {
         path: `/jobTemplates/${uid}?_etc=true`
       });
       return true;
-    } catch {
-      return false;
+    } catch (error2) {
+      if (error2 instanceof HttpError && error2.status === 404) {
+        return false;
+      }
+      throw error2;
     }
   }
   async findMonitorByCollection(collectionUid, environmentUid, name) {
@@ -118366,6 +118369,22 @@ var PostmanGatewayAssetsClient = class {
     const environment = String(environmentUid ?? "").trim();
     const monitorName = String(name ?? "").trim();
     const monitors = await this.listMonitors();
+    const sameNameMatches = monitors.filter((monitor) => monitor.name === monitorName);
+    const sameNameEnvironmentMatches = sameNameMatches.filter((monitor) => monitor.environmentUid === environment);
+    try {
+      this.selectExactMatch(
+        "monitor",
+        `workspace ${this.workspaceId}, name "${monitorName}", and environment ${environment || "(none)"}`,
+        sameNameEnvironmentMatches
+      );
+    } catch (error2) {
+      const environments = [...new Set(sameNameMatches.map((monitor) => monitor.environmentUid || "(none)"))];
+      const message = error2 instanceof Error ? error2.message : String(error2);
+      throw new Error(
+        `${message} Same-name monitor(s) also exist on environment(s): ${environments.join(", ")}.`,
+        { cause: error2 }
+      );
+    }
     const matches = monitors.filter(
       (monitor) => monitor.collectionUid === want && monitor.environmentUid === environment && monitor.name === monitorName
     );
@@ -118857,9 +118876,9 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         headBranch = headRef;
         rawRef = clean(env.GITHUB_REF) ?? headRef;
         refKind = "branch";
-        const headRepo = event?.pull_request?.head?.repo?.full_name;
-        const baseRepo = event?.pull_request?.base?.repo?.full_name ?? event?.repository?.full_name;
-        isForkPr = Boolean(headRepo && baseRepo && headRepo !== baseRepo);
+        const headRepo = clean(event?.pull_request?.head?.repo?.full_name)?.toLowerCase();
+        const baseRepo = clean(event?.pull_request?.base?.repo?.full_name)?.toLowerCase();
+        isForkPr = !(headRepo && baseRepo && headRepo === baseRepo);
         headSha = clean(event?.pull_request?.head?.sha) ?? headSha;
       } else {
         rawRef = clean(env.GITHUB_REF) ?? clean(env.GITHUB_REF_NAME);
@@ -118880,7 +118899,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         refKind = "branch";
         const sourceProject = clean(env.CI_MERGE_REQUEST_SOURCE_PROJECT_ID);
         const targetProject = clean(env.CI_MERGE_REQUEST_PROJECT_ID);
-        isForkPr = Boolean(sourceProject && targetProject && sourceProject !== targetProject);
+        isForkPr = !(sourceProject && targetProject && sourceProject === targetProject);
       } else if (clean(env.CI_COMMIT_TAG)) {
         rawRef = clean(env.CI_COMMIT_TAG);
         refKind = "tag";
@@ -118914,7 +118933,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         rawRef = prSource;
         refKind = parsed.kind;
         const forkFlag = clean(env.SYSTEM_PULLREQUEST_ISFORK);
-        isForkPr = forkFlag?.toLowerCase() === "true";
+        isForkPr = forkFlag?.toLowerCase() !== "false";
       } else {
         rawRef = clean(env.BUILD_SOURCEBRANCH);
         const parsed = stripRefPrefix(rawRef);
@@ -118999,6 +119018,15 @@ function resolveBranchDecision(options) {
       reason: `ref kind ${identity.refKind}: never canonical/preview-eligible; no-op with annotation`
     };
   }
+  if (identity.isForkPr) {
+    return {
+      tier: "gated",
+      strategy,
+      identity,
+      canonicalBranch,
+      reason: "fork PR: never write-eligible (canonical/channel/preview require a same-repo head), gated instead"
+    };
+  }
   if (identity.headBranch === canonicalBranch) {
     return {
       tier: "canonical",
@@ -119020,15 +119048,6 @@ function resolveBranchDecision(options) {
     };
   }
   if (strategy === "preview") {
-    if (identity.isForkPr) {
-      return {
-        tier: "gated",
-        strategy,
-        identity,
-        canonicalBranch,
-        reason: "fork PR: preview-ineligible (same-repo gate), gated instead"
-      };
-    }
     return {
       tier: "preview",
       strategy,

package/dist/cli.cjs

@@ -116462,8 +116462,11 @@ var PostmanGatewayAssetsClient = class {
         path: `/jobTemplates/${uid}?_etc=true`
       });
       return true;
-    } catch {
-      return false;
+    } catch (error) {
+      if (error instanceof HttpError && error.status === 404) {
+        return false;
+      }
+      throw error;
     }
   }
   async findMonitorByCollection(collectionUid, environmentUid, name) {
@@ -116471,6 +116474,22 @@ var PostmanGatewayAssetsClient = class {
     const environment = String(environmentUid ?? "").trim();
     const monitorName = String(name ?? "").trim();
     const monitors = await this.listMonitors();
+    const sameNameMatches = monitors.filter((monitor) => monitor.name === monitorName);
+    const sameNameEnvironmentMatches = sameNameMatches.filter((monitor) => monitor.environmentUid === environment);
+    try {
+      this.selectExactMatch(
+        "monitor",
+        `workspace ${this.workspaceId}, name "${monitorName}", and environment ${environment || "(none)"}`,
+        sameNameEnvironmentMatches
+      );
+    } catch (error) {
+      const environments = [...new Set(sameNameMatches.map((monitor) => monitor.environmentUid || "(none)"))];
+      const message = error instanceof Error ? error.message : String(error);
+      throw new Error(
+        `${message} Same-name monitor(s) also exist on environment(s): ${environments.join(", ")}.`,
+        { cause: error }
+      );
+    }
     const matches = monitors.filter(
       (monitor) => monitor.collectionUid === want && monitor.environmentUid === environment && monitor.name === monitorName
     );
@@ -116910,9 +116929,9 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         headBranch = headRef;
         rawRef = clean(env.GITHUB_REF) ?? headRef;
         refKind = "branch";
-        const headRepo = event?.pull_request?.head?.repo?.full_name;
-        const baseRepo = event?.pull_request?.base?.repo?.full_name ?? event?.repository?.full_name;
-        isForkPr = Boolean(headRepo && baseRepo && headRepo !== baseRepo);
+        const headRepo = clean(event?.pull_request?.head?.repo?.full_name)?.toLowerCase();
+        const baseRepo = clean(event?.pull_request?.base?.repo?.full_name)?.toLowerCase();
+        isForkPr = !(headRepo && baseRepo && headRepo === baseRepo);
         headSha = clean(event?.pull_request?.head?.sha) ?? headSha;
       } else {
         rawRef = clean(env.GITHUB_REF) ?? clean(env.GITHUB_REF_NAME);
@@ -116933,7 +116952,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         refKind = "branch";
         const sourceProject = clean(env.CI_MERGE_REQUEST_SOURCE_PROJECT_ID);
         const targetProject = clean(env.CI_MERGE_REQUEST_PROJECT_ID);
-        isForkPr = Boolean(sourceProject && targetProject && sourceProject !== targetProject);
+        isForkPr = !(sourceProject && targetProject && sourceProject === targetProject);
       } else if (clean(env.CI_COMMIT_TAG)) {
         rawRef = clean(env.CI_COMMIT_TAG);
         refKind = "tag";
@@ -116967,7 +116986,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         rawRef = prSource;
         refKind = parsed.kind;
         const forkFlag = clean(env.SYSTEM_PULLREQUEST_ISFORK);
-        isForkPr = forkFlag?.toLowerCase() === "true";
+        isForkPr = forkFlag?.toLowerCase() !== "false";
       } else {
         rawRef = clean(env.BUILD_SOURCEBRANCH);
         const parsed = stripRefPrefix(rawRef);
@@ -117052,6 +117071,15 @@ function resolveBranchDecision(options) {
       reason: `ref kind ${identity.refKind}: never canonical/preview-eligible; no-op with annotation`
     };
   }
+  if (identity.isForkPr) {
+    return {
+      tier: "gated",
+      strategy,
+      identity,
+      canonicalBranch,
+      reason: "fork PR: never write-eligible (canonical/channel/preview require a same-repo head), gated instead"
+    };
+  }
   if (identity.headBranch === canonicalBranch) {
     return {
       tier: "canonical",
@@ -117073,15 +117101,6 @@ function resolveBranchDecision(options) {
     };
   }
   if (strategy === "preview") {
-    if (identity.isForkPr) {
-      return {
-        tier: "gated",
-        strategy,
-        identity,
-        canonicalBranch,
-        reason: "fork PR: preview-ineligible (same-repo gate), gated instead"
-      };
-    }
     return {
       tier: "preview",
       strategy,

package/dist/index.cjs

@@ -118381,8 +118381,11 @@ var PostmanGatewayAssetsClient = class {
         path: `/jobTemplates/${uid}?_etc=true`
       });
       return true;
-    } catch {
-      return false;
+    } catch (error2) {
+      if (error2 instanceof HttpError && error2.status === 404) {
+        return false;
+      }
+      throw error2;
     }
   }
   async findMonitorByCollection(collectionUid, environmentUid, name) {
@@ -118390,6 +118393,22 @@ var PostmanGatewayAssetsClient = class {
     const environment = String(environmentUid ?? "").trim();
     const monitorName = String(name ?? "").trim();
     const monitors = await this.listMonitors();
+    const sameNameMatches = monitors.filter((monitor) => monitor.name === monitorName);
+    const sameNameEnvironmentMatches = sameNameMatches.filter((monitor) => monitor.environmentUid === environment);
+    try {
+      this.selectExactMatch(
+        "monitor",
+        `workspace ${this.workspaceId}, name "${monitorName}", and environment ${environment || "(none)"}`,
+        sameNameEnvironmentMatches
+      );
+    } catch (error2) {
+      const environments = [...new Set(sameNameMatches.map((monitor) => monitor.environmentUid || "(none)"))];
+      const message = error2 instanceof Error ? error2.message : String(error2);
+      throw new Error(
+        `${message} Same-name monitor(s) also exist on environment(s): ${environments.join(", ")}.`,
+        { cause: error2 }
+      );
+    }
     const matches = monitors.filter(
       (monitor) => monitor.collectionUid === want && monitor.environmentUid === environment && monitor.name === monitorName
     );
@@ -118881,9 +118900,9 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         headBranch = headRef;
         rawRef = clean(env.GITHUB_REF) ?? headRef;
         refKind = "branch";
-        const headRepo = event?.pull_request?.head?.repo?.full_name;
-        const baseRepo = event?.pull_request?.base?.repo?.full_name ?? event?.repository?.full_name;
-        isForkPr = Boolean(headRepo && baseRepo && headRepo !== baseRepo);
+        const headRepo = clean(event?.pull_request?.head?.repo?.full_name)?.toLowerCase();
+        const baseRepo = clean(event?.pull_request?.base?.repo?.full_name)?.toLowerCase();
+        isForkPr = !(headRepo && baseRepo && headRepo === baseRepo);
         headSha = clean(event?.pull_request?.head?.sha) ?? headSha;
       } else {
         rawRef = clean(env.GITHUB_REF) ?? clean(env.GITHUB_REF_NAME);
@@ -118904,7 +118923,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         refKind = "branch";
         const sourceProject = clean(env.CI_MERGE_REQUEST_SOURCE_PROJECT_ID);
         const targetProject = clean(env.CI_MERGE_REQUEST_PROJECT_ID);
-        isForkPr = Boolean(sourceProject && targetProject && sourceProject !== targetProject);
+        isForkPr = !(sourceProject && targetProject && sourceProject === targetProject);
       } else if (clean(env.CI_COMMIT_TAG)) {
         rawRef = clean(env.CI_COMMIT_TAG);
         refKind = "tag";
@@ -118938,7 +118957,7 @@ function resolveBranchIdentity(env = process.env, overrides = {}) {
         rawRef = prSource;
         refKind = parsed.kind;
         const forkFlag = clean(env.SYSTEM_PULLREQUEST_ISFORK);
-        isForkPr = forkFlag?.toLowerCase() === "true";
+        isForkPr = forkFlag?.toLowerCase() !== "false";
       } else {
         rawRef = clean(env.BUILD_SOURCEBRANCH);
         const parsed = stripRefPrefix(rawRef);
@@ -119023,6 +119042,15 @@ function resolveBranchDecision(options) {
       reason: `ref kind ${identity.refKind}: never canonical/preview-eligible; no-op with annotation`
     };
   }
+  if (identity.isForkPr) {
+    return {
+      tier: "gated",
+      strategy,
+      identity,
+      canonicalBranch,
+      reason: "fork PR: never write-eligible (canonical/channel/preview require a same-repo head), gated instead"
+    };
+  }
   if (identity.headBranch === canonicalBranch) {
     return {
       tier: "canonical",
@@ -119044,15 +119072,6 @@ function resolveBranchDecision(options) {
     };
   }
   if (strategy === "preview") {
-    if (identity.isForkPr) {
-      return {
-        tier: "gated",
-        strategy,
-        identity,
-        canonicalBranch,
-        reason: "fork PR: preview-ineligible (same-repo gate), gated instead"
-      };
-    }
     return {
       tier: "preview",
       strategy,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@postman-cse/onboarding-repo-sync",
-  "version": "2.7.1",
+  "version": "2.7.3",
   "description": "Postman repo sync GitHub Action.",
   "type": "module",
   "main": "dist/index.cjs",
@@ -26,7 +26,8 @@
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "test": "vitest run --exclude tests/ci-workflow-template.test.ts && vitest run tests/ci-workflow-template.test.ts && node --test .github/scripts/dispatch-e2e-monitor.test.mjs",
-    "typecheck": "tsc --noEmit -p tsconfig.json"
+    "typecheck": "tsc --noEmit -p tsconfig.json",
+    "record:cassettes": "RECORD_FAKE_CASSETTES=1 vitest run tests/contract/record-fake-cassettes.test.ts"
   },
   "keywords": [
     "github-action",
@@ -45,7 +46,7 @@
   "dependencies": {
     "@actions/core": "^3.0.1",
     "@actions/exec": "^3.0.0",
-    "@postman-cse/automation-core": "^1.3.0",
+    "@postman-cse/automation-core": "^1.6.0",
     "@postman/runtime.models": "^1.13.1",
     "@postman/v3.export": "0.2.28",
     "js-yaml": "^5.2.1",