From 9f6d8c557fb22a6950658ed13babb94d92a0e259 Mon Sep 17 00:00:00 2001 From: Vincenzo Scamporlino Date: Tue, 22 Apr 2025 11:17:44 +0200 Subject: [PATCH] yarn-plugin: clarify errors Signed-off-by: Vincenzo Scamporlino --- .../src/resolvers/BackstageNpmResolver.ts | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/packages/yarn-plugin/src/resolvers/BackstageNpmResolver.ts b/packages/yarn-plugin/src/resolvers/BackstageNpmResolver.ts index 87bb057055..cd38a8c5c6 100644 --- a/packages/yarn-plugin/src/resolvers/BackstageNpmResolver.ts +++ b/packages/yarn-plugin/src/resolvers/BackstageNpmResolver.ts @@ -74,7 +74,9 @@ export class BackstageNpmResolver implements Resolver { const npmVersion = structUtils.parseRange(descriptor.range).params?.npm; if (!npmVersion) { - throw new Error(`Unreachable`); + throw new Error( + `Missing npm parameter on backstage: range "${descriptor.range}".`, + ); } return { @@ -133,7 +135,9 @@ export class BackstageNpmResolver implements Resolver { * `npm:` protocol. */ shouldPersistResolution = () => { - throw new Error('Unreachable'); + throw new Error( + 'Unreachable: BackstageNpmResolver should never persist resolution as it uses npm: protocol', + ); }; /** @@ -141,6 +145,8 @@ export class BackstageNpmResolver implements Resolver { * `npm:` protocol. */ resolve = async () => { - throw new Error(`Unreachable`); + throw new Error( + 'Unreachable: BackstageNpmResolver should never resolve as it uses npm: protocol', + ); }; }