@docupace/pipeline 1.1.6 → 1.1.8

package/.changeset/validate-changeset-bin-fallback.md

@@ -0,0 +1,5 @@
+---
+'@docupace/pipeline': patch
+---
+
+Fix changeset validation failing with "No changeset found" on pull-request builds. Validation ran `./node_modules/.bin/changeset status` (since 1.1.6), but the validate step has no install, so the binary did not exist (exit 127). It now reads the release plan in-process with `@changesets/get-release-plan`, which needs neither the repo's `node_modules` nor a `changeset-output.json` file.

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @docupace/pipeline
 
+## 1.1.7
+
+### Patch Changes
+
+- 82e71f0: Run the changeset binary installed by the CLI directly instead of via `pnpm changeset`, so pnpm cannot re-sync node_modules to the repo lockfile and swap the changelog hook to an older pinned @docupace/pipeline.
+
+  Log the @docupace/pipeline version (to stderr) before every command of every dp-\*-cli, and log the CLI and changelog hook versions during changeset versioning.
+
 ## 1.1.4
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@docupace/pipeline",
-  "version": "1.1.6",
+  "version": "1.1.8",
   "description": "",
   "main": "index.js",
   "type": "module",
@@ -25,6 +25,7 @@
   "dependencies": {
     "@aws-sdk/client-secrets-manager": "^3.600.0",
     "@changesets/cli": "2.27.12",
+    "@changesets/get-release-plan": "^4.0.16",
     "@docupace/core-client": "^1.0.1",
     "@jest/globals": "^30.1.2",
     "auth0-deploy-cli": "^8.8.0",

package/src/changeset/validate.js

@@ -5,7 +5,7 @@ import { $ } from 'zx';
 
 import { setGitInfo } from '../next/util.js';
 import { getPackageNames } from './package-util.js';
-import { versionStatus } from './version.js';
+import getReleasePlan from '@changesets/get-release-plan';
 
 const PACKAGE_ROOT = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
@@ -16,6 +16,17 @@ const PACKAGE_VERSION = JSON.parse(
   fs.readFileSync(path.join(PACKAGE_ROOT, 'package.json'), 'utf8'),
 ).version;
 
+// Reads the release plan in-process with the changesets library instead of
+// running `changeset status`, so validation needs neither the repo's
+// node_modules (no install step runs before validate) nor an output file.
+export async function getValidationReleases(sinceBranch) {
+  const releasePlan = await getReleasePlan(process.cwd(), sinceBranch);
+  if (releasePlan.changesets.length === 0) {
+    throw new Error(`No changesets found since ${sinceBranch}`);
+  }
+  return releasePlan.releases;
+}
+
 export async function validateChangesets() {
   console.log(
     `validateChangesets: @docupace/pipeline version = ${PACKAGE_VERSION}`,
@@ -32,7 +43,7 @@ export async function validateChangesets() {
 
   try {
     console.log('🔍 Validating changesets...');
-    releases = await versionStatus(destBranch);
+    releases = await getValidationReleases(destBranch);
   } catch (e) {
     console.error(
       '❌ No changeset found. Please add a changeset to this PR. Please see more information at https://docupacellc.atlassian.net/wiki/spaces/PM/pages/1350893571/Creating+changesets+for+Docupace+Next',

package/src/changeset/validate.test.js

@@ -0,0 +1,79 @@
+import { afterEach, beforeEach, describe, expect, test } from '@jest/globals';
+import { execFileSync } from 'child_process';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { getValidationReleases } from './validate.js';
+
+const git = (cwd, ...args) =>
+  execFileSync('git', args, { cwd, stdio: 'pipe' }).toString();
+
+// A real repo with no node_modules, like the pipeline's validate step.
+function makeRepo() {
+  const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'validate-changeset-'));
+  git(dir, 'init', '-q', '-b', 'master');
+  git(dir, 'config', 'user.email', 't@example.com');
+  git(dir, 'config', 'user.name', 'test');
+  fs.writeFileSync(
+    path.join(dir, 'package.json'),
+    JSON.stringify({ name: 'sample', version: '1.0.0' }),
+  );
+  fs.mkdirSync(path.join(dir, '.changeset'));
+  fs.writeFileSync(
+    path.join(dir, '.changeset', 'config.json'),
+    JSON.stringify({
+      changelog: false,
+      baseBranch: 'master',
+      access: 'restricted',
+    }),
+  );
+  git(dir, 'add', '-A');
+  git(dir, 'commit', '-q', '-m', 'base');
+  git(dir, 'checkout', '-q', '-b', 'feature');
+  fs.writeFileSync(path.join(dir, 'file.txt'), 'change');
+  return dir;
+}
+
+describe('getValidationReleases', () => {
+  let dir;
+  let originalCwd;
+
+  beforeEach(() => {
+    originalCwd = process.cwd();
+    dir = makeRepo();
+    process.chdir(dir);
+  });
+
+  afterEach(() => {
+    process.chdir(originalCwd);
+    fs.rmSync(dir, { recursive: true, force: true });
+  });
+
+  test('returns the releases when the branch adds a changeset', async () => {
+    fs.writeFileSync(
+      path.join(dir, '.changeset', 'DC-1.md'),
+      "---\n'sample': patch\n---\n\nA change\n",
+    );
+    git(dir, 'add', '-A');
+    git(dir, 'commit', '-q', '-m', 'feature');
+
+    const releases = await getValidationReleases('master');
+
+    expect(releases).toEqual([
+      expect.objectContaining({
+        name: 'sample',
+        type: 'patch',
+        newVersion: '1.0.1',
+      }),
+    ]);
+  });
+
+  test('throws when the branch has no changeset', async () => {
+    git(dir, 'add', '-A');
+    git(dir, 'commit', '-q', '-m', 'feature');
+
+    await expect(getValidationReleases('master')).rejects.toThrow(
+      'No changesets found since master',
+    );
+  });
+});

package/.changeset/run-changeset-bin-directly.md

@@ -1,7 +0,0 @@
----
-'@docupace/pipeline': patch
----
-
-Run the changeset binary installed by the CLI directly instead of via `pnpm changeset`, so pnpm cannot re-sync node_modules to the repo lockfile and swap the changelog hook to an older pinned @docupace/pipeline.
-
-Log the @docupace/pipeline version (to stderr) before every command of every dp-*-cli, and log the CLI and changelog hook versions during changeset versioning.