fixes

Author: aduh95

sources/commands/Base.ts

@@ -44,7 +44,7 @@ export abstract class BaseCommand extends Command<Context> {
     const {data, indent} = nodeUtils.readPackageJson(content);
 
     const previousPackageManager = data.packageManager ?? `unknown`;
-    data.packageManager = `${info.locator.name}@${info.locator.reference}+${info.hash}`;
+    data.packageManager = `${info.locator.name}@${info.locator.reference}${typeof info.locator.reference === `string` ? `+` : `#`}${info.hash}`;
 
     const newContent = nodeUtils.normalizeLineEndings(content, `${JSON.stringify(data, null, indent)}\n`);
     await fs.promises.writeFile(lookup.target, newContent, `utf8`);

sources/commands/Use.ts

@@ -45,8 +45,10 @@ export class UseCommand extends BaseCommand {
       const location = `https://registry.npmjs.com/${this.pattern}/-/${this.pattern}-${version}.tgz`;
       packageManagerInfo = {
         locator: {
-          name: this.pattern as any,
-          reference: location,
+          // @ts-expect-error Needs type update from the other PR
+          name: this.pattern,
+          // @ts-expect-error Needs type update from the other PR
+          reference: new URL(location),
         },
         spec: {
           bin: {},

tests/Use.test.ts

@@ -78,7 +78,7 @@ describe(`UseCommand`, () => {
       });
 
       await expect(xfs.readJsonPromise(ppath.join(cwd, `package.json`))).resolves.toMatchObject({
-        packageManager: `test@https://registry.npmjs.com/test/-/test-3.3.0.tgz+sha1.a2b56c6aa386c5732065793e8d9d92074a9cdd41`,
+        packageManager: `test@https://registry.npmjs.com/test/-/test-3.3.0.tgz#sha1.a2b56c6aa386c5732065793e8d9d92074a9cdd41`,
       });
     });
   });