ic-mops 2.6.0 → 2.7.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## Next
 
+## 2.7.0
+
+- `mops publish` no longer requires a `repository` field — it is now optional metadata (used by the registry UI for source links)
+- `mops publish` now hard-errors on GitHub `[dependencies]` instead of prompting; the backend has rejected them for some time and the prompt was misleading
+- `mops publish` now fails fast with a clear error when unsupported fields (`dfx`, `moc`, `homepage`, `documentation`, `donation`) are set in `mops.toml`
+- Fix `mops publish` reporting incorrect max length for `license` field (was 30, now matches backend limit of 40)
+
 ## 2.6.0
 
 - Packages can ship lintoko rules for consumers in a `rules/` directory (distinct from `lint/`/`lints/` which check the package itself); `rules/*.toml` files are included automatically when running `mops publish`

package/RELEASE.md

@@ -21,33 +21,36 @@ cd cli
 npm version patch --no-git-tag-version  # or: minor / major
 ```
 
-## 3. Create a release PR
+## 3. Create a release PR and enable auto-merge
 
 ```bash
 git checkout -b <username>/release-X.Y.Z
 git add cli/CHANGELOG.md cli/package.json cli/package-lock.json
 git commit -m "release: CLI vX.Y.Z"
 git push -u origin <username>/release-X.Y.Z
-gh pr create --title "release: CLI vX.Y.Z" --body "..."
+gh pr create \
+  --title "release: CLI vX.Y.Z" \
+  --body "Release CLI vX.Y.Z." \
+  --label release
+gh pr merge --auto --squash
 ```
 
-Wait for CI to pass, then merge.
+The [`release-pr.yml`](../.github/workflows/release-pr.yml) workflow runs on every update and validates:
+- PR title matches `release: CLI vX.Y.Z`
+- `cli/CHANGELOG.md` has an entry for the version
+- `cli/package.json` version matches
 
-## 4. Tag and push
+Once all required checks pass the PR merges automatically. On merge, `release-pr.yml` pushes the `cli-vX.Y.Z` tag, which triggers the [`release.yml`](../.github/workflows/release.yml) workflow — it builds, publishes to npm, creates a GitHub Release, and deploys canisters (`cli.mops.one` and `docs.mops.one`).
 
-```bash
-git checkout main && git pull
-git tag cli-vX.Y.Z
-git push origin cli-vX.Y.Z
-```
-
-This triggers the [`release.yml`](../.github/workflows/release.yml) workflow which builds, publishes to npm, creates a GitHub Release, deploys canisters (`cli.mops.one` and `docs.mops.one`), and opens a PR with on-chain release artifacts.
-
-Monitor at [Actions → Release CLI](https://github.com/caffeinelabs/mops/actions/workflows/release.yml).
+> **Note:** This workflow only deploys the `cli` and `docs` canisters. The `main`, `assets`, `blog`, and `play-frontend` canisters require a manual deploy. If a release includes changes to any of those (e.g. `backend/main/` or `frontend/`), upgrade them manually (staging first, then `ic`):
+>
+> ```bash
+> NODE_ENV=production dfx deploy --no-wallet --identity mops --network <staging|ic> <canister>
+> ```
 
-## 5. Merge artifacts PR
+## 5. Artifacts PR
 
-After the workflow completes, merge the `cli-releases: vX.Y.Z artifacts` PR.
+After the release pipeline completes, it creates and auto-merges a `cli-releases: vX.Y.Z artifacts` PR. No action needed unless it fails — monitor at [Actions → Release CLI](https://github.com/caffeinelabs/mops/actions/workflows/release.yml) and merge the artifacts PR manually if needed.
 
 ## Verify build
 

package/commands/publish.ts

@@ -83,7 +83,7 @@ export async function publish(
   }
 
   // desired fields
-  for (let key of ["description", "repository"]) {
+  for (let key of ["description"]) {
     // @ts-ignore
     if (!config.package[key] && !process.env.CI) {
       let res = await prompts({
@@ -120,6 +120,14 @@ export async function publish(
     }
   }
 
+  // disabled fields
+  for (let key of ["dfx", "moc", "homepage", "documentation", "donation"]) {
+    if ((config.package as any)[key]) {
+      console.log(chalk.red("Error: ") + `package.${key} is not supported yet`);
+      process.exit(1);
+    }
+  }
+
   // check lengths
   let keysMax = {
     name: 50,
@@ -130,7 +138,7 @@ export async function publish(
     documentation: 300,
     homepage: 300,
     readme: 100,
-    license: 30,
+    license: 40,
     files: 20,
     dfx: 10,
     moc: 10,
@@ -166,18 +174,12 @@ export async function publish(
     }
 
     for (let dep of Object.values(config.dependencies)) {
-      if (dep.repo && !process.env.CI) {
-        let res = await prompts({
-          type: "confirm",
-          name: "ok",
-          message:
-            chalk.yellow(
-              "GitHub dependencies make the registry less reliable and limit its capabilities.\nIf you are the owner of the dependency, please consider publishing it to the Mops registry.",
-            ) + "\n\nPublish anyway?",
-        });
-        if (!res.ok) {
-          return;
-        }
+      if (dep.repo) {
+        console.log(
+          chalk.red("Error: ") +
+            "GitHub dependencies are no longer supported.\nIf you are the owner of the dependency, please publish it to the Mops registry.",
+        );
+        process.exit(1);
       }
     }
   }
@@ -345,7 +347,10 @@ export async function publish(
   // parse changelog
   console.log("Parsing CHANGELOG.md...");
   let changelog = parseChangelog(config.package.version);
-  if (!changelog && config.package.repository) {
+  if (
+    !changelog &&
+    config.package.repository?.startsWith("https://github.com/")
+  ) {
     console.log("Fetching release notes from GitHub...");
     changelog = await fetchGitHubReleaseNotes(
       config.package.repository,

package/dist/commands/publish.js

@@ -50,7 +50,7 @@ export async function publish(options = {}) {
         }
     }
     // desired fields
-    for (let key of ["description", "repository"]) {
+    for (let key of ["description"]) {
         // @ts-ignore
         if (!config.package[key] && !process.env.CI) {
             let res = await prompts({
@@ -85,6 +85,13 @@ export async function publish(options = {}) {
             process.exit(1);
         }
     }
+    // disabled fields
+    for (let key of ["dfx", "moc", "homepage", "documentation", "donation"]) {
+        if (config.package[key]) {
+            console.log(chalk.red("Error: ") + `package.${key} is not supported yet`);
+            process.exit(1);
+        }
+    }
     // check lengths
     let keysMax = {
         name: 50,
@@ -95,7 +102,7 @@ export async function publish(options = {}) {
         documentation: 300,
         homepage: 300,
         readme: 100,
-        license: 30,
+        license: 40,
         files: 20,
         dfx: 10,
         moc: 10,
@@ -123,15 +130,10 @@ export async function publish(options = {}) {
             delete dep.path;
         }
         for (let dep of Object.values(config.dependencies)) {
-            if (dep.repo && !process.env.CI) {
-                let res = await prompts({
-                    type: "confirm",
-                    name: "ok",
-                    message: chalk.yellow("GitHub dependencies make the registry less reliable and limit its capabilities.\nIf you are the owner of the dependency, please consider publishing it to the Mops registry.") + "\n\nPublish anyway?",
-                });
-                if (!res.ok) {
-                    return;
-                }
+            if (dep.repo) {
+                console.log(chalk.red("Error: ") +
+                    "GitHub dependencies are no longer supported.\nIf you are the owner of the dependency, please publish it to the Mops registry.");
+                process.exit(1);
             }
         }
     }
@@ -272,7 +274,8 @@ export async function publish(options = {}) {
     // parse changelog
     console.log("Parsing CHANGELOG.md...");
     let changelog = parseChangelog(config.package.version);
-    if (!changelog && config.package.repository) {
+    if (!changelog &&
+        config.package.repository?.startsWith("https://github.com/")) {
         console.log("Fetching release notes from GitHub...");
         changelog = await fetchGitHubReleaseNotes(config.package.repository, config.package.version);
     }

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "type": "module",
   "bin": {
     "mops": "bin/mops.js",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ic-mops",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "type": "module",
   "bin": {
     "mops": "dist/bin/mops.js",