ic-mops 0.29.0-0 → 0.29.0

package/cli.ts

@@ -24,6 +24,7 @@ import {bump} from './commands/bump.js';
 import {sync} from './commands/sync.js';
 import {outdated} from './commands/outdated.js';
 import {update} from './commands/update.js';
+import {transferOwnership} from './commands/transfer-ownership.js';
 // import {docs} from './commands/docs.js';
 
 program.name('mops');
@@ -317,4 +318,12 @@ program
 		await update(pkg);
 	});
 
+// transfer-ownership
+program
+	.command('transfer-ownership [to-principal]')
+	.description('Transfer ownership of the current package to another principal')
+	.action(async (toPrincipal) => {
+		await transferOwnership(toPrincipal);
+	});
+
 program.parse();

package/commands/add.ts

@@ -1,7 +1,7 @@
 import path from 'node:path';
 import chalk from 'chalk';
 import logUpdate from 'log-update';
-import {checkConfigFile, getGithubCommit, getHighestVersion, parseGithubURL, readConfig, writeConfig} from '../mops.js';
+import {checkConfigFile, getHighestVersion, parseGithubURL, readConfig, writeConfig} from '../mops.js';
 import {installFromGithub} from '../vessel.js';
 import {install} from './install.js';
 import {notifyInstalls} from '../notify-installs.js';
@@ -36,20 +36,11 @@ export async function add(name: string, {verbose = false, dev = false} = {}) {
 	}
 	// github package
 	else if (name.startsWith('https://github.com') || name.split('/').length > 1) {
-		let {org, gitName, branch, commitHash} = parseGithubURL(name);
-
-		// fetch latest commit hash of branch if not specified
-		if (!commitHash) {
-			let commit = await getGithubCommit(`${org}/${gitName}`, branch);
-			if (!commit.sha) {
-				throw Error(`Could not find commit hash for ${name}`);
-			}
-			commitHash = commit.sha;
-		}
+		const {org, gitName, branch} = parseGithubURL(name);
 
 		pkgDetails = {
 			name: parseGithubURL(name).gitName,
-			repo: `https://github.com/${org}/${gitName}#${branch}@${commitHash}`,
+			repo: `https://github.com/${org}/${gitName}#${branch}`,
 			version: '',
 		};
 	}

package/commands/init.ts

@@ -174,15 +174,22 @@ async function applyInit({type, config, setupWorkflow, addTest, copyrightOwner}
 	let dfxJsonData;
 	if (existsSync(dfxJson)) {
 		let dfxJsonText = readFileSync(dfxJson).toString();
-		dfxJsonData = JSON.parse(dfxJsonText);
-		console.log('Setting packtool in dfx.json...');
-		dfxJsonData.defaults = dfxJsonData.defaults || {};
-		dfxJsonData.defaults.build = dfxJsonData.defaults.build || {};
-		if (dfxJsonData.defaults.build.packtool !== 'mops sources') {
-			dfxJsonData.defaults.build.packtool = 'mops sources';
-			let indent = dfxJsonText.match(/([ \t]+)"/)?.[1] || '  ';
-			writeFileSync(path.join(process.cwd(), 'dfx.json'), JSON.stringify(dfxJsonData, null, indent));
-			console.log(chalk.green('packtool set to "mops sources"'));
+		try {
+			dfxJsonData = JSON.parse(dfxJsonText);
+		}
+		catch (err) {
+			console.log(chalk.yellow('Failed to parse dfx.json'));
+		}
+		if (dfxJsonData) {
+			console.log('Setting packtool in dfx.json...');
+			dfxJsonData.defaults = dfxJsonData.defaults || {};
+			dfxJsonData.defaults.build = dfxJsonData.defaults.build || {};
+			if (dfxJsonData.defaults.build.packtool !== 'mops sources') {
+				dfxJsonData.defaults.build.packtool = 'mops sources';
+				let indent = dfxJsonText.match(/([ \t]+)"/)?.[1] || '  ';
+				writeFileSync(path.join(process.cwd(), 'dfx.json'), JSON.stringify(dfxJsonData, null, indent));
+				console.log(chalk.green('packtool set to "mops sources"'));
+			}
 		}
 	}
 

package/commands/transfer-ownership.ts

@@ -0,0 +1,42 @@
+import chalk from 'chalk';
+import {checkConfigFile, mainActor, readConfig} from '../mops.js';
+import {Principal} from '@dfinity/principal';
+import prompts from 'prompts';
+
+export async function transferOwnership(toPrincipal: string) {
+	if (!checkConfigFile()) {
+		return;
+	}
+
+	let config = readConfig();
+	let principal = Principal.fromText(toPrincipal);
+
+	let promptsConfig = {
+		onCancel() {
+			console.log('aborted');
+			process.exit(0);
+		}
+	};
+
+	console.log(chalk.red('Warning: ') + 'This action cannot be undone!');
+	let {confirm} = await prompts({
+		type: 'confirm',
+		name: 'confirm',
+		message: `Are you sure you want to transfer ownership of ${chalk.yellow(config.package?.name)} to ${chalk.yellow(toPrincipal)}?`,
+		initial: false,
+	}, promptsConfig);
+
+	if (!confirm) {
+		return;
+	}
+
+	let actor = await mainActor(true);
+
+	let res = await actor.transferOwnership(config.package?.name || '', principal);
+	if ('ok' in res) {
+		console.log(chalk.green('Success!'));
+	}
+	else {
+		console.log(chalk.red('Error: ') + res.err);
+	}
+}

package/commands/update.ts

@@ -1,5 +1,5 @@
 import chalk from 'chalk';
-import {checkConfigFile, getGithubCommit, parseGithubURL, readConfig} from '../mops.js';
+import {checkConfigFile, readConfig} from '../mops.js';
 import {add} from './add.js';
 import {getAvailableUpdates} from './available-updates.js';
 
@@ -9,38 +9,10 @@ export async function update(pkg?: string) {
 	}
 	let config = readConfig();
 
-	if (pkg && !config.dependencies?.[pkg] && !config['dev-dependencies']?.[pkg]) {
-		console.log(chalk.red(`Package "${pkg}" is not installed!`));
-		return;
-	}
-
-	// update github packages
-	let deps = Object.values(config.dependencies || {});
-	let devDeps = Object.values(config['dev-dependencies'] || {});
-	let githubDeps = [...deps, ...devDeps].filter((dep) => dep.repo);
-	if (pkg) {
-		githubDeps = githubDeps.filter((dep) => dep.name === pkg);
-	}
-
-	for (let dep of githubDeps) {
-		let {org, gitName, branch, commitHash} = parseGithubURL(dep.repo || '');
-		let dev = !!config['dev-dependencies']?.[dep.name];
-		let commit = await getGithubCommit(`${org}/${gitName}`, branch);
-		if (commit.sha !== commitHash) {
-			await add(`https://github.com/${org}/${gitName}#${branch}@${commitHash}`, {dev});
-		}
-	}
-
-	// update mops packages
 	let available = await getAvailableUpdates(config, pkg);
 
 	if (available.length === 0) {
-		if (pkg) {
-			console.log(chalk.green(`Package "${pkg}" is up to date!`));
-		}
-		else {
-			console.log(chalk.green('All dependencies are up to date!'));
-		}
+		console.log(chalk.green('All dependencies are up to date!'));
 	}
 	else {
 		for (let dep of available) {

package/declarations/main/main.did

@@ -82,7 +82,7 @@ type Result_6 =
  variant {
    err: Err;
    ok: vec record {
-             PackageName__1;
+             PackageName;
              PackageVersion;
            };
  };
@@ -97,20 +97,20 @@ type Result_4 =
    ok: PackageDetails;
  };
 type Result_3 = 
- variant {
-   err: text;
-   ok;
- };
-type Result_2 = 
  variant {
    err: Err;
    ok: FileId;
  };
-type Result_1 = 
+type Result_2 = 
  variant {
    err: PublishingErr;
    ok: PublishingId;
  };
+type Result_1 = 
+ variant {
+   err: text;
+   ok;
+ };
 type Result = 
  variant {
    err: Err;
@@ -223,7 +223,7 @@ type PackageConfigV2__1 =
    keywords: vec text;
    license: text;
    moc: text;
-   name: PackageName;
+   name: PackageName__1;
    readme: text;
    repository: text;
    scripts: vec Script;
@@ -242,7 +242,7 @@ type PackageConfigV2 =
    keywords: vec text;
    license: text;
    moc: text;
-   name: PackageName;
+   name: PackageName__1;
    readme: text;
    repository: text;
    scripts: vec Script;
@@ -283,7 +283,7 @@ type DownloadsSnapshot =
  };
 type DependencyV2 = 
  record {
-   name: PackageName;
+   name: PackageName__1;
    repo: text;
    version: text;
  };
@@ -304,25 +304,25 @@ service : {
   getBackupCanisterId: () -> (principal) query;
   getDefaultPackages: (text) ->
    (vec record {
-          PackageName__1;
+          PackageName;
           PackageVersion;
         }) query;
   getDownloadTrendByPackageId: (PackageId) ->
    (vec DownloadsSnapshot__1) query;
-  getDownloadTrendByPackageName: (PackageName__1) ->
+  getDownloadTrendByPackageName: (PackageName) ->
    (vec DownloadsSnapshot__1) query;
-  getFileIds: (PackageName__1, PackageVersion) -> (Result_7) query;
+  getFileIds: (PackageName, PackageVersion) -> (Result_7) query;
   getHighestSemverBatch:
    (vec record {
-          PackageName__1;
+          PackageName;
           PackageVersion;
           SemverPart;
         }) -> (Result_6) query;
-  getHighestVersion: (PackageName__1) -> (Result_5) query;
+  getHighestVersion: (PackageName) -> (Result_5) query;
   getMostDownloadedPackages: () -> (vec PackageSummary) query;
   getMostDownloadedPackagesIn7Days: () -> (vec PackageSummary) query;
   getNewPackages: () -> (vec PackageSummary) query;
-  getPackageDetails: (PackageName__1, PackageVersion) -> (Result_4) query;
+  getPackageDetails: (PackageName, PackageVersion) -> (Result_4) query;
   getPackagesByCategory: () -> (vec record {
                                       text;
                                       vec PackageSummary;
@@ -336,17 +336,18 @@ service : {
   getTotalPackages: () -> (nat) query;
   getUser: (principal) -> (opt User__1) query;
   http_request: (Request) -> (Response) query;
-  notifyInstall: (PackageName__1, PackageVersion) -> () oneway;
+  notifyInstall: (PackageName, PackageVersion) -> () oneway;
   notifyInstalls: (vec record {
-                         PackageName__1;
+                         PackageName;
                          PackageVersion;
                        }) -> () oneway;
   restore: (nat, nat) -> ();
   search: (Text, opt nat, opt nat) -> (vec PackageSummary, PageCount) query;
-  setUserProp: (text, text) -> (Result_3);
-  startFileUpload: (PublishingId, Text, nat, blob) -> (Result_2);
-  startPublish: (PackageConfigV2) -> (Result_1);
+  setUserProp: (text, text) -> (Result_1);
+  startFileUpload: (PublishingId, Text, nat, blob) -> (Result_3);
+  startPublish: (PackageConfigV2) -> (Result_2);
   takeAirdropSnapshot: () -> () oneway;
+  transferOwnership: (PackageName, principal) -> (Result_1);
   uploadFileChunk: (PublishingId, FileId, nat, blob) -> (Result);
   uploadNotes: (PublishingId, text) -> (Result);
   uploadTestStats: (PublishingId, TestStats) -> (Result);

package/declarations/main/main.did.d.ts

@@ -7,7 +7,7 @@ export interface DepChange {
   'newVersion' : string,
 }
 export interface DependencyV2 {
-  'name' : PackageName,
+  'name' : PackageName__1,
   'repo' : string,
   'version' : string,
 }
@@ -42,7 +42,7 @@ export interface PackageConfigV2 {
   'scripts' : Array<Script>,
   'baseDir' : string,
   'documentation' : string,
-  'name' : PackageName,
+  'name' : PackageName__1,
   'homepage' : string,
   'description' : string,
   'version' : string,
@@ -60,7 +60,7 @@ export interface PackageConfigV2__1 {
   'scripts' : Array<Script>,
   'baseDir' : string,
   'documentation' : string,
-  'name' : PackageName,
+  'name' : PackageName__1,
   'homepage' : string,
   'description' : string,
   'version' : string,
@@ -159,17 +159,17 @@ export interface Response {
 }
 export type Result = { 'ok' : null } |
   { 'err' : Err };
-export type Result_1 = { 'ok' : PublishingId } |
+export type Result_1 = { 'ok' : null } |
+  { 'err' : string };
+export type Result_2 = { 'ok' : PublishingId } |
   { 'err' : PublishingErr };
-export type Result_2 = { 'ok' : FileId } |
+export type Result_3 = { 'ok' : FileId } |
   { 'err' : Err };
-export type Result_3 = { 'ok' : null } |
-  { 'err' : string };
 export type Result_4 = { 'ok' : PackageDetails } |
   { 'err' : Err };
 export type Result_5 = { 'ok' : PackageVersion } |
   { 'err' : Err };
-export type Result_6 = { 'ok' : Array<[PackageName__1, PackageVersion]> } |
+export type Result_6 = { 'ok' : Array<[PackageName, PackageVersion]> } |
   { 'err' : Err };
 export type Result_7 = { 'ok' : Array<FileId> } |
   { 'err' : Err };
@@ -241,26 +241,26 @@ export interface _SERVICE {
   'getBackupCanisterId' : ActorMethod<[], Principal>,
   'getDefaultPackages' : ActorMethod<
     [string],
-    Array<[PackageName__1, PackageVersion]>
+    Array<[PackageName, PackageVersion]>
   >,
   'getDownloadTrendByPackageId' : ActorMethod<
     [PackageId],
     Array<DownloadsSnapshot__1>
   >,
   'getDownloadTrendByPackageName' : ActorMethod<
-    [PackageName__1],
+    [PackageName],
     Array<DownloadsSnapshot__1>
   >,
-  'getFileIds' : ActorMethod<[PackageName__1, PackageVersion], Result_7>,
+  'getFileIds' : ActorMethod<[PackageName, PackageVersion], Result_7>,
   'getHighestSemverBatch' : ActorMethod<
-    [Array<[PackageName__1, PackageVersion, SemverPart]>],
+    [Array<[PackageName, PackageVersion, SemverPart]>],
     Result_6
   >,
-  'getHighestVersion' : ActorMethod<[PackageName__1], Result_5>,
+  'getHighestVersion' : ActorMethod<[PackageName], Result_5>,
   'getMostDownloadedPackages' : ActorMethod<[], Array<PackageSummary>>,
   'getMostDownloadedPackagesIn7Days' : ActorMethod<[], Array<PackageSummary>>,
   'getNewPackages' : ActorMethod<[], Array<PackageSummary>>,
-  'getPackageDetails' : ActorMethod<[PackageName__1, PackageVersion], Result_4>,
+  'getPackageDetails' : ActorMethod<[PackageName, PackageVersion], Result_4>,
   'getPackagesByCategory' : ActorMethod<
     [],
     Array<[string, Array<PackageSummary>]>
@@ -274,9 +274,9 @@ export interface _SERVICE {
   'getTotalPackages' : ActorMethod<[], bigint>,
   'getUser' : ActorMethod<[Principal], [] | [User__1]>,
   'http_request' : ActorMethod<[Request], Response>,
-  'notifyInstall' : ActorMethod<[PackageName__1, PackageVersion], undefined>,
+  'notifyInstall' : ActorMethod<[PackageName, PackageVersion], undefined>,
   'notifyInstalls' : ActorMethod<
-    [Array<[PackageName__1, PackageVersion]>],
+    [Array<[PackageName, PackageVersion]>],
     undefined
   >,
   'restore' : ActorMethod<[bigint, bigint], undefined>,
@@ -284,13 +284,14 @@ export interface _SERVICE {
     [Text, [] | [bigint], [] | [bigint]],
     [Array<PackageSummary>, PageCount]
   >,
-  'setUserProp' : ActorMethod<[string, string], Result_3>,
+  'setUserProp' : ActorMethod<[string, string], Result_1>,
   'startFileUpload' : ActorMethod<
     [PublishingId, Text, bigint, Uint8Array | number[]],
-    Result_2
+    Result_3
   >,
-  'startPublish' : ActorMethod<[PackageConfigV2], Result_1>,
+  'startPublish' : ActorMethod<[PackageConfigV2], Result_2>,
   'takeAirdropSnapshot' : ActorMethod<[], undefined>,
+  'transferOwnership' : ActorMethod<[PackageName, Principal], Result_1>,
   'uploadFileChunk' : ActorMethod<
     [PublishingId, FileId, bigint, Uint8Array | number[]],
     Result

package/declarations/main/main.did.js

@@ -18,7 +18,7 @@ export const idlFactory = ({ IDL }) => {
   const Err = IDL.Text;
   const Result = IDL.Variant({ 'ok' : IDL.Null, 'err' : Err });
   const Text = IDL.Text;
-  const PackageName__1 = IDL.Text;
+  const PackageName = IDL.Text;
   const PackageVersion = IDL.Text;
   const PackageId = IDL.Text;
   const Time = IDL.Int;
@@ -35,7 +35,7 @@ export const idlFactory = ({ IDL }) => {
     'patch' : IDL.Null,
   });
   const Result_6 = IDL.Variant({
-    'ok' : IDL.Vec(IDL.Tuple(PackageName__1, PackageVersion)),
+    'ok' : IDL.Vec(IDL.Tuple(PackageName, PackageVersion)),
     'err' : Err,
   });
   const Result_5 = IDL.Variant({ 'ok' : PackageVersion, 'err' : Err });
@@ -52,9 +52,9 @@ export const idlFactory = ({ IDL }) => {
     'github' : IDL.Text,
   });
   const Script = IDL.Record({ 'value' : IDL.Text, 'name' : IDL.Text });
-  const PackageName = IDL.Text;
+  const PackageName__1 = IDL.Text;
   const DependencyV2 = IDL.Record({
-    'name' : PackageName,
+    'name' : PackageName__1,
     'repo' : IDL.Text,
     'version' : IDL.Text,
   });
@@ -64,7 +64,7 @@ export const idlFactory = ({ IDL }) => {
     'scripts' : IDL.Vec(Script),
     'baseDir' : IDL.Text,
     'documentation' : IDL.Text,
-    'name' : PackageName,
+    'name' : PackageName__1,
     'homepage' : IDL.Text,
     'description' : IDL.Text,
     'version' : IDL.Text,
@@ -206,15 +206,15 @@ export const idlFactory = ({ IDL }) => {
     'status_code' : IDL.Nat16,
   });
   const PageCount = IDL.Nat;
-  const Result_3 = IDL.Variant({ 'ok' : IDL.Null, 'err' : IDL.Text });
-  const Result_2 = IDL.Variant({ 'ok' : FileId, 'err' : Err });
+  const Result_1 = IDL.Variant({ 'ok' : IDL.Null, 'err' : IDL.Text });
+  const Result_3 = IDL.Variant({ 'ok' : FileId, 'err' : Err });
   const PackageConfigV2 = IDL.Record({
     'dfx' : IDL.Text,
     'moc' : IDL.Text,
     'scripts' : IDL.Vec(Script),
     'baseDir' : IDL.Text,
     'documentation' : IDL.Text,
-    'name' : PackageName,
+    'name' : PackageName__1,
     'homepage' : IDL.Text,
     'description' : IDL.Text,
     'version' : IDL.Text,
@@ -227,7 +227,7 @@ export const idlFactory = ({ IDL }) => {
     'readme' : IDL.Text,
   });
   const PublishingErr = IDL.Text;
-  const Result_1 = IDL.Variant({ 'ok' : PublishingId, 'err' : PublishingErr });
+  const Result_2 = IDL.Variant({ 'ok' : PublishingId, 'err' : PublishingErr });
   const TestStats = IDL.Record({
     'passedNames' : IDL.Vec(IDL.Text),
     'passed' : IDL.Nat,
@@ -243,7 +243,7 @@ export const idlFactory = ({ IDL }) => {
     'getBackupCanisterId' : IDL.Func([], [IDL.Principal], ['query']),
     'getDefaultPackages' : IDL.Func(
         [IDL.Text],
-        [IDL.Vec(IDL.Tuple(PackageName__1, PackageVersion))],
+        [IDL.Vec(IDL.Tuple(PackageName, PackageVersion))],
         ['query'],
       ),
     'getDownloadTrendByPackageId' : IDL.Func(
@@ -252,21 +252,21 @@ export const idlFactory = ({ IDL }) => {
         ['query'],
       ),
     'getDownloadTrendByPackageName' : IDL.Func(
-        [PackageName__1],
+        [PackageName],
         [IDL.Vec(DownloadsSnapshot__1)],
         ['query'],
       ),
     'getFileIds' : IDL.Func(
-        [PackageName__1, PackageVersion],
+        [PackageName, PackageVersion],
         [Result_7],
         ['query'],
       ),
     'getHighestSemverBatch' : IDL.Func(
-        [IDL.Vec(IDL.Tuple(PackageName__1, PackageVersion, SemverPart))],
+        [IDL.Vec(IDL.Tuple(PackageName, PackageVersion, SemverPart))],
         [Result_6],
         ['query'],
       ),
-    'getHighestVersion' : IDL.Func([PackageName__1], [Result_5], ['query']),
+    'getHighestVersion' : IDL.Func([PackageName], [Result_5], ['query']),
     'getMostDownloadedPackages' : IDL.Func(
         [],
         [IDL.Vec(PackageSummary)],
@@ -279,7 +279,7 @@ export const idlFactory = ({ IDL }) => {
       ),
     'getNewPackages' : IDL.Func([], [IDL.Vec(PackageSummary)], ['query']),
     'getPackageDetails' : IDL.Func(
-        [PackageName__1, PackageVersion],
+        [PackageName, PackageVersion],
         [Result_4],
         ['query'],
       ),
@@ -302,13 +302,9 @@ export const idlFactory = ({ IDL }) => {
     'getTotalPackages' : IDL.Func([], [IDL.Nat], ['query']),
     'getUser' : IDL.Func([IDL.Principal], [IDL.Opt(User__1)], ['query']),
     'http_request' : IDL.Func([Request], [Response], ['query']),
-    'notifyInstall' : IDL.Func(
-        [PackageName__1, PackageVersion],
-        [],
-        ['oneway'],
-      ),
+    'notifyInstall' : IDL.Func([PackageName, PackageVersion], [], ['oneway']),
     'notifyInstalls' : IDL.Func(
-        [IDL.Vec(IDL.Tuple(PackageName__1, PackageVersion))],
+        [IDL.Vec(IDL.Tuple(PackageName, PackageVersion))],
         [],
         ['oneway'],
       ),
@@ -318,14 +314,19 @@ export const idlFactory = ({ IDL }) => {
         [IDL.Vec(PackageSummary), PageCount],
         ['query'],
       ),
-    'setUserProp' : IDL.Func([IDL.Text, IDL.Text], [Result_3], []),
+    'setUserProp' : IDL.Func([IDL.Text, IDL.Text], [Result_1], []),
     'startFileUpload' : IDL.Func(
         [PublishingId, Text, IDL.Nat, IDL.Vec(IDL.Nat8)],
-        [Result_2],
+        [Result_3],
         [],
       ),
-    'startPublish' : IDL.Func([PackageConfigV2], [Result_1], []),
+    'startPublish' : IDL.Func([PackageConfigV2], [Result_2], []),
     'takeAirdropSnapshot' : IDL.Func([], [], ['oneway']),
+    'transferOwnership' : IDL.Func(
+        [PackageName, IDL.Principal],
+        [Result_1],
+        [],
+      ),
     'uploadFileChunk' : IDL.Func(
         [PublishingId, FileId, IDL.Nat, IDL.Vec(IDL.Nat8)],
         [Result],

package/dist/cli.js

@@ -22,6 +22,7 @@ import { bump } from './commands/bump.js';
 import { sync } from './commands/sync.js';
 import { outdated } from './commands/outdated.js';
 import { update } from './commands/update.js';
+import { transferOwnership } from './commands/transfer-ownership.js';
 // import {docs} from './commands/docs.js';
 program.name('mops');
 // --version
@@ -288,4 +289,11 @@ program
     .action(async (pkg) => {
     await update(pkg);
 });
+// transfer-ownership
+program
+    .command('transfer-ownership [to-principal]')
+    .description('Transfer ownership of the current package to another principal')
+    .action(async (toPrincipal) => {
+    await transferOwnership(toPrincipal);
+});
 program.parse();

package/dist/commands/add.js

@@ -1,7 +1,7 @@
 import path from 'node:path';
 import chalk from 'chalk';
 import logUpdate from 'log-update';
-import { checkConfigFile, getGithubCommit, getHighestVersion, parseGithubURL, readConfig, writeConfig } from '../mops.js';
+import { checkConfigFile, getHighestVersion, parseGithubURL, readConfig, writeConfig } from '../mops.js';
 import { installFromGithub } from '../vessel.js';
 import { install } from './install.js';
 import { notifyInstalls } from '../notify-installs.js';
@@ -32,18 +32,10 @@ export async function add(name, { verbose = false, dev = false } = {}) {
     }
     // github package
     else if (name.startsWith('https://github.com') || name.split('/').length > 1) {
-        let { org, gitName, branch, commitHash } = parseGithubURL(name);
-        // fetch latest commit hash of branch if not specified
-        if (!commitHash) {
-            let commit = await getGithubCommit(`${org}/${gitName}`, branch);
-            if (!commit.sha) {
-                throw Error(`Could not find commit hash for ${name}`);
-            }
-            commitHash = commit.sha;
-        }
+        const { org, gitName, branch } = parseGithubURL(name);
         pkgDetails = {
             name: parseGithubURL(name).gitName,
-            repo: `https://github.com/${org}/${gitName}#${branch}@${commitHash}`,
+            repo: `https://github.com/${org}/${gitName}#${branch}`,
             version: '',
         };
     }

package/dist/commands/init.js

@@ -146,15 +146,22 @@ async function applyInit({ type, config, setupWorkflow, addTest, copyrightOwner
     let dfxJsonData;
     if (existsSync(dfxJson)) {
         let dfxJsonText = readFileSync(dfxJson).toString();
-        dfxJsonData = JSON.parse(dfxJsonText);
-        console.log('Setting packtool in dfx.json...');
-        dfxJsonData.defaults = dfxJsonData.defaults || {};
-        dfxJsonData.defaults.build = dfxJsonData.defaults.build || {};
-        if (dfxJsonData.defaults.build.packtool !== 'mops sources') {
-            dfxJsonData.defaults.build.packtool = 'mops sources';
-            let indent = dfxJsonText.match(/([ \t]+)"/)?.[1] || '  ';
-            writeFileSync(path.join(process.cwd(), 'dfx.json'), JSON.stringify(dfxJsonData, null, indent));
-            console.log(chalk.green('packtool set to "mops sources"'));
+        try {
+            dfxJsonData = JSON.parse(dfxJsonText);
+        }
+        catch (err) {
+            console.log(chalk.yellow('Failed to parse dfx.json'));
+        }
+        if (dfxJsonData) {
+            console.log('Setting packtool in dfx.json...');
+            dfxJsonData.defaults = dfxJsonData.defaults || {};
+            dfxJsonData.defaults.build = dfxJsonData.defaults.build || {};
+            if (dfxJsonData.defaults.build.packtool !== 'mops sources') {
+                dfxJsonData.defaults.build.packtool = 'mops sources';
+                let indent = dfxJsonText.match(/([ \t]+)"/)?.[1] || '  ';
+                writeFileSync(path.join(process.cwd(), 'dfx.json'), JSON.stringify(dfxJsonData, null, indent));
+                console.log(chalk.green('packtool set to "mops sources"'));
+            }
         }
     }
     // get default packages

package/dist/commands/transfer-ownership.d.ts

@@ -0,0 +1 @@
+export declare function transferOwnership(toPrincipal: string): Promise<void>;