fix(dry-run): stop searching for "warn" to avoid ANSI color complications

travi · travi · commit bee5db6550e4 · 2025-07-27T22:34:52.000-05:00
diff --git a/lib/verify-auth.js b/lib/verify-auth.js
@@ -29,7 +29,7 @@ export default async function (npmrc, pkg, context) {
     publishDryRunResult.stderr.pipe(stderr, { end: false });
     const {stderr: execaStderr} = await publishDryRunResult;
     execaStderr.forEach((line) => {
-      if (line.includes("warn This command requires you to be logged in to https://registry.npmjs.org/")) {
+      if (line.includes("This command requires you to be logged in to https://registry.npmjs.org/")) {
         throw new AggregateError([new Error('no auth context')]);
       }
     })

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ export default async function (npmrc, pkg, context) {`
`29`	`29`	`publishDryRunResult.stderr.pipe(stderr, { end: false });`
`30`	`30`	`const {stderr: execaStderr} = await publishDryRunResult;`
`31`	`31`	`execaStderr.forEach((line) => {`
`32`		`- if (line.includes("warn This command requires you to be logged in to https://registry.npmjs.org/")) {`
	`32`	`+ if (line.includes("This command requires you to be logged in to https://registry.npmjs.org/")) {`
`33`	`33`	`throw new AggregateError([new Error('no auth context')]);`
`34`	`34`	`}`
`35`	`35`	`})`