zotero-plugin-scaffold 0.0.6 → 0.0.8

package/dist/config.js

@@ -55,7 +55,7 @@ export async function loadConfig(file) {
             ghOwner: owner,
             ghRepo: repo,
             addonRef: pkg.config?.addonRef || _.kebabCase(addonName),
-            addonInstance: pkg.config?.addonInstence || _.camelCase(addonName),
+            addonInstance: pkg.config?.addonInstance || _.camelCase(addonName),
             prefsPrefix: `extensions.zotero.${addonRef}`,
             xpiName: xpiName,
             releasePage: releasePage,
@@ -134,19 +134,19 @@ export async function loadConfig(file) {
         extraBuilder: () => { },
         addonLint: {},
         release: {
-            releaseIt: {
-                preReleaseId: "beta",
-                git: {
-                    tagName: "v${version}",
-                    requireCleanWorkingDir: false,
-                },
-                npm: {
-                    publish: false,
-                },
-                github: {
-                    assets: [`${userConfig.dist}/*.xpi`],
-                },
-            },
+            // releaseIt: {
+            //   preReleaseId: "beta",
+            //   git: {
+            //     tagName: "v${version}",
+            //     requireCleanWorkingDir: false,
+            //   },
+            //   npm: {
+            //     publish: false,
+            //   },
+            //   github: {
+            //     assets: [`${userConfig.dist}/*.xpi`],
+            //   },
+            // },
             bumpp: {
                 release: "prompt",
                 preid: "beta",

package/dist/lib/release.d.ts

@@ -22,7 +22,7 @@ export default class Release extends Base {
     /**
      * Create new release and upload XPI to asset
      */
-    uploadXPI(): void;
+    uploadXPI(): Promise<void>;
     getReleaseByTag(tag: string): Promise<{
         url: string;
         html_url: string;
@@ -249,6 +249,7 @@ export default class Release extends Base {
         } | null;
     }>;
     uploadUpdateJSON(): Promise<void>;
+    getChangelog(): Promise<string>;
     getClient(): Octokit;
     get owner(): string;
     get repo(): string;

package/dist/lib/release.js

@@ -1,12 +1,12 @@
 import { Base } from "./base.js";
 import { Octokit } from "@octokit/rest";
+import versionBump from "bumpp";
 import ci from "ci-info";
+import conventionalChangelog from "conventional-changelog";
 import { default as glob } from "fast-glob";
 import fs from "fs-extra";
-import _ from "lodash";
 import mime from "mime";
 import path from "path";
-import releaseIt from "release-it";
 export default class Release extends Base {
     isCI;
     client;
@@ -40,30 +40,38 @@ export default class Release extends Base {
      * release: bump version, run build, git add, git commit, git tag, git push
      */
     bump() {
-        // versionBump(this.config.release.bumpp);
-        const releaseItConfig = {
-            "only-version": true,
-        };
-        releaseIt(_.defaultsDeep(releaseItConfig, this.config.release.releaseIt));
+        versionBump(this.config.release.bumpp);
+        // const releaseItConfig: ReleaseItConfig = {
+        //   "only-version": true,
+        // };
+        // releaseIt(_.defaultsDeep(releaseItConfig, this.config.release.releaseIt));
     }
     /**
      * Create new release and upload XPI to asset
      */
-    uploadXPI() {
-        const releaseItConfig = {
-            increment: false,
-            git: {
-                commit: false,
-                tag: false,
-                push: false,
-            },
-            github: {
-                release: true,
-            },
-            verbose: 2,
-            ci: true,
-        };
-        releaseIt(_.defaultsDeep(releaseItConfig, this.config.release.releaseIt));
+    async uploadXPI() {
+        // const releaseItConfig: ReleaseItConfig = {
+        //   increment: false,
+        //   git: { commit: false, tag: false, push: false },
+        //   github: {
+        //     release: true,
+        //   },
+        //   verbose: 2,
+        //   ci: true,
+        // };
+        // releaseIt(_.defaultsDeep(releaseItConfig, this.config.release.releaseIt));
+        const release = await this.creatRelease({
+            owner: this.owner,
+            repo: this.repo,
+            tag_name: `v${this.version}`,
+            name: `Release v${this.version}`,
+            body: await this.getChangelog(),
+            prerelease: this.version.includes("-"),
+            make_latest: "true",
+        });
+        if (!release)
+            throw new Error("Creat release failed!");
+        this.uploadAsset(release.id, path.join(this.config.dist, `${this.xpiName}.xpi`));
     }
     async getReleaseByTag(tag) {
         return await this.client.repos
@@ -119,12 +127,17 @@ export default class Release extends Base {
                 owner: this.owner,
                 repo: this.repo,
                 tag_name: "release",
-                name: "Release Manifest",
-                body: `This release is used to host \`update.json\`, please do not delete or modify it! \n Updated in UTC ${new Date().toISOString()} for version ${this.version}`,
-                make_latest: "false",
             }));
         if (!release)
             throw new Error("Get or create 'release' failed.");
+        await this.client.repos.updateRelease({
+            owner: this.owner,
+            repo: this.repo,
+            release_id: release.id,
+            name: "Release Manifest",
+            body: `This release is used to host \`update.json\`, please do not delete or modify it! \n Updated in UTC ${new Date().toISOString()} for version ${this.version}`,
+            make_latest: "false",
+        });
         const existAssets = await this.client.repos
             .listReleaseAssets({
             owner: this.owner,
@@ -147,6 +160,22 @@ export default class Release extends Base {
             await this.uploadAsset(release.id, path.join(this.config.dist, asset));
         }
     }
+    getChangelog() {
+        return new Promise((resolve, reject) => {
+            let changelog = "";
+            conventionalChangelog({ releaseCount: 2 }, { version: this.version })
+                .on("data", (chunk) => {
+                changelog += chunk.toString();
+            })
+                .on("end", () => {
+                this.logger.debug("changelog:", changelog);
+                resolve(changelog);
+            })
+                .on("error", (err) => {
+                reject(err);
+            });
+        });
+    }
     getClient() {
         if (!process.env.GITHUB_TOKEN)
             throw new Error("No GITHUB_TOKEN.");

package/dist/lib/serve.d.ts

@@ -1,3 +1,4 @@
+/// <reference types="node" />
 import { Config } from "../types.js";
 import { Base } from "./base.js";
 export default class Serve extends Base {
@@ -8,7 +9,7 @@ export default class Serve extends Base {
      * watch source dir and build when file changed
      */
     watch(): Promise<void>;
-    startZotero(): void;
+    startZotero(): Promise<import("child_process").ChildProcessWithoutNullStreams>;
     /**
      * start zotero with plugin installed and reload when dist changed
      */

package/dist/lib/serve.js

@@ -16,10 +16,10 @@ export default class Serve extends Base {
     }
     async run() {
         // build
-        this.builder.run();
+        await this.builder.run();
         // start Zotero
-        // this.startZotero();
-        this.startZoteroWebExt();
+        this.startZotero();
+        // this.startZoteroWebExt();
         // watch
         await this.config.extraServer(this.config);
         await this.watch();
@@ -63,19 +63,33 @@ export default class Serve extends Base {
             this.logger.error("Server start failed!", err);
         });
     }
-    startZotero() {
+    async startZotero() {
+        let isZoteroReady = false;
         if (!fs.existsSync(this.zoteroBinPath)) {
             throw new Error("Zotero binary does not exist.");
         }
         if (!fs.existsSync(this.profilePath)) {
             throw new Error("The given Zotero profile does not exist.");
         }
+        this.prepareDevEnv();
         const zoteroProcess = spawn(this.zoteroBinPath, [
             "--debugger",
             "--purgecaches",
             "-profile",
             this.profilePath,
         ]);
+        zoteroProcess.stdout?.on("data", (data) => {
+            if (!isZoteroReady &&
+                data
+                    .toString()
+                    .includes(`Calling bootstrap method 'startup' for plugin ${this.addonID}`)) {
+                console.log(data.toString());
+                isZoteroReady = true;
+                setTimeout(() => {
+                    this.openDevTool();
+                }, 1000);
+            }
+        });
         zoteroProcess.on("close", (code) => {
             this.logger.info(`Zotero terminated with code ${code}.`);
             exit(0);
@@ -85,6 +99,7 @@ export default class Serve extends Base {
             zoteroProcess.kill();
             exit();
         });
+        return zoteroProcess;
     }
     /**
      * start zotero with plugin installed and reload when dist changed
@@ -221,11 +236,11 @@ export default class Serve extends Base {
         execSync(command);
     }
     get zoteroBinPath() {
-        this.logger.debug("zoteroBinPath", process.env.zoteroBinPath);
+        // this.logger.debug("zoteroBinPath", process.env.zoteroBinPath);
         return process.env.zoteroBinPath ?? "";
     }
     get profilePath() {
-        this.logger.debug("profilePath", process.env.profilePath);
+        // this.logger.debug("profilePath", process.env.profilePath);
         return process.env.profilePath ?? "";
     }
     get dataDir() {

package/dist/types.d.ts

@@ -271,7 +271,6 @@ export interface ConfigBase {
      * 发布相关配置
      */
     release: {
-        releaseIt: Partial<ReleaseItConfig>;
         bumpp: VersionBumpOptions;
     };
     /**

package/dist/utils/log.js

@@ -42,8 +42,9 @@ export default class Log {
             this.log(chalk.green("INFO"), ...args);
     }
     debug(...args) {
+        // if (this.logLevel <= 1) this.log(chalk.grey("DEBUG"), ...args);
         if (this.logLevel <= 1)
-            this.log(chalk.grey("DEBUG"), ...args);
+            this.log(...args);
     }
     trace(...args) {
         if (this.logLevel <= 0)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zotero-plugin-scaffold",
-  "version": "0.0.6",
+  "version": "0.0.8",
   "description": "A scaffold for Zotero plugin development.",
   "main": "dist/index.js",
   "bin": {
@@ -52,9 +52,12 @@
     "ci-info": "^4.0.0",
     "commander": "^11.1.0",
     "compressing": "^1.10.0",
+    "conventional-changelog": "^5.1.0",
+    "conventional-commits-detector": "^1.0.3",
     "cosmiconfig": "^9.0.0",
     "dotenv": "^16.4.1",
     "esbuild": "^0.19.12",
+    "execa": "^8.0.1",
     "fast-glob": "^3.3.2",
     "fs-extra": "^11.2.0",
     "lodash": "^4.17.21",
@@ -67,12 +70,13 @@
   },
   "devDependencies": {
     "@trivago/prettier-plugin-sort-imports": "^4.3.0",
+    "@types/conventional-changelog": "^3.1.5",
     "@types/fs-extra": "^11.0.4",
     "@types/lodash": "^4.14.202",
-    "@types/node": "^20.11.9",
+    "@types/node": "^20.11.16",
     "@types/update-notifier": "^6.0.8",
-    "@typescript-eslint/eslint-plugin": "^6.19.1",
-    "@typescript-eslint/parser": "^6.19.1",
+    "@typescript-eslint/eslint-plugin": "^6.20.0",
+    "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.56.0",
     "eslint-config-prettier": "^9.1.0",
     "prettier": "^3.2.4",