release serfver

.eslintignore

@@ -1334,15 +1334,6 @@ packages/lib/uuid.js.map
 packages/lib/versionInfo.d.ts
 packages/lib/versionInfo.js
 packages/lib/versionInfo.js.map
-packages/plugins/ToggleSideBars/api/index.d.ts
-packages/plugins/ToggleSideBars/api/index.js
-packages/plugins/ToggleSideBars/api/index.js.map
-packages/plugins/ToggleSideBars/api/types.d.ts
-packages/plugins/ToggleSideBars/api/types.js
-packages/plugins/ToggleSideBars/api/types.js.map
-packages/plugins/ToggleSideBars/src/index.d.ts
-packages/plugins/ToggleSideBars/src/index.js
-packages/plugins/ToggleSideBars/src/index.js.map
 packages/renderer/HtmlToHtml.d.ts
 packages/renderer/HtmlToHtml.js
 packages/renderer/HtmlToHtml.js.map
@@ -1655,9 +1646,6 @@ packages/server/src/utils/urlUtils.js.map
 packages/server/src/utils/uuidgen.d.ts
 packages/server/src/utils/uuidgen.js
 packages/server/src/utils/uuidgen.js.map
-packages/tools/build-plugin-repository.d.ts
-packages/tools/build-plugin-repository.js
-packages/tools/build-plugin-repository.js.map
 packages/tools/release-server.d.ts
 packages/tools/release-server.js
 packages/tools/release-server.js.map

.gitignore

@@ -1323,15 +1323,6 @@ packages/lib/uuid.js.map
 packages/lib/versionInfo.d.ts
 packages/lib/versionInfo.js
 packages/lib/versionInfo.js.map
-packages/plugins/ToggleSideBars/api/index.d.ts
-packages/plugins/ToggleSideBars/api/index.js
-packages/plugins/ToggleSideBars/api/index.js.map
-packages/plugins/ToggleSideBars/api/types.d.ts
-packages/plugins/ToggleSideBars/api/types.js
-packages/plugins/ToggleSideBars/api/types.js.map
-packages/plugins/ToggleSideBars/src/index.d.ts
-packages/plugins/ToggleSideBars/src/index.js
-packages/plugins/ToggleSideBars/src/index.js.map
 packages/renderer/HtmlToHtml.d.ts
 packages/renderer/HtmlToHtml.js
 packages/renderer/HtmlToHtml.js.map
@@ -1644,9 +1635,6 @@ packages/server/src/utils/urlUtils.js.map
 packages/server/src/utils/uuidgen.d.ts
 packages/server/src/utils/uuidgen.js
 packages/server/src/utils/uuidgen.js.map
-packages/tools/build-plugin-repository.d.ts
-packages/tools/build-plugin-repository.js
-packages/tools/build-plugin-repository.js.map
 packages/tools/release-server.d.ts
 packages/tools/release-server.js
 packages/tools/release-server.js.map

package.json

@@ -7,30 +7,29 @@
   },
   "license": "MIT",
   "scripts": {
-    "bootstrap": "lerna bootstrap --no-ci",
-    "bootstrapIgnoreScripts": "lerna bootstrap --ignore-scripts --no-ci",
-    "build": "lerna run build && npm run tsc",
     "buildApiDoc": "npm start --prefix=packages/app-cli -- apidoc ../../readme/api/references/rest_api.md",
     "buildDoc": "./packages/tools/build-all.sh",
     "buildPluginDoc": "typedoc --name 'Joplin Plugin API Documentation' --mode file -theme './Assets/PluginDocTheme/' --readme './Assets/PluginDocTheme/index.md' --excludeNotExported --excludeExternals --excludePrivate --excludeProtected --out docs/api/references/plugin_api packages/lib/services/plugins/api/",
     "buildTranslations": "npm run tsc && node packages/tools/build-translation.js",
     "buildTranslationsNoTsc": "node packages/tools/build-translation.js",
     "buildWebsite": "npm run buildApiDoc && node ./packages/tools/build-website.js && npm run buildPluginDoc",
-    "circularDependencyCheck": "npx madge --warning --circular --extensions js ./",
     "clean": "lerna clean -y && lerna run clean",
+    "circularDependencyCheck": "npx madge --warning --circular --extensions js ./",
     "generateDatabaseTypes": "node packages/tools/generate-database-types",
     "linkChecker": "linkchecker https://joplinapp.org",
     "linter-ci": "./node_modules/.bin/eslint --resolve-plugins-relative-to . --quiet --ext .js --ext .jsx --ext .ts --ext .tsx",
     "linter-precommit": "./node_modules/.bin/eslint --resolve-plugins-relative-to . --fix --ext .js --ext .jsx --ext .ts --ext .tsx",
     "linter": "./node_modules/.bin/eslint --resolve-plugins-relative-to . --fix --quiet --ext .js --ext .jsx --ext .ts --ext .tsx",
+    "bootstrap": "lerna bootstrap --no-ci",
+    "bootstrapIgnoreScripts": "lerna bootstrap --ignore-scripts --no-ci",
     "postinstall": "npm run bootstrap --no-ci && npm run build",
+    "build": "lerna run build && npm run tsc",
     "publishAll": "git pull && lerna version --yes --no-private --no-git-tag-version && gulp completePublishAll",
     "releaseAndroid": "node packages/tools/release-android.js",
     "releaseCli": "node packages/tools/release-cli.js",
     "releaseClipper": "node packages/tools/release-clipper.js",
     "releaseDesktop": "node packages/tools/release-electron.js",
     "releasePluginGenerator": "node packages/tools/release-plugin-generator.js",
-    "releaseServer": "node packages/tools/release-server.js",
     "setupNewRelease": "node ./packages/tools/setupNewRelease",
     "test-ci": "lerna run test-ci --stream",
     "test": "lerna run test --stream",

packages/app-cli/tests/services/plugins/api/JoplinWorkspace.ts

@@ -1,4 +1,3 @@
-import Setting from '@joplin/lib/models/Setting';
 import PluginService from '@joplin/lib/services/plugins/PluginService';
 
 const { newPluginService, newPluginScript, setupDatabaseAndSynchronizer, switchClient, afterEachCleanUp } = require('../../../test-utils');
@@ -50,26 +49,4 @@ describe('JoplinWorkspace', () => {
 		await service.destroy();
 	});
 
-	test('should return the selected folder', async () => {
-		const service = new newPluginService() as PluginService;
-
-		const pluginScript = newPluginScript(`			
-			joplin.plugins.register({
-				onStart: async function() {
-					const folder = await joplin.workspace.selectedFolder();
-					await joplin.data.put(['folders', folder.id], null, { title: "changedtitle" });
-				},
-			});
-		`);
-
-		const folder = await Folder.save({ title: 'folder' });
-		Setting.setValue('activeFolderId', folder.id);
-
-		const plugin = await service.loadPluginFromJsBundle('', pluginScript);
-		await service.runPlugin(plugin);
-
-		const modFolder = await Folder.load(folder.id);
-		expect(modFolder.title).toBe('changedtitle');
-	});
-
 });

packages/app-cli/tests/support/plugins/codemirror_content_script/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/codemirror_content_script/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/codemirror_content_script/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/codemirror_content_script/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/codemirror_content_script/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/codemirror_content_script/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/content_script/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/content_script/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/content_script/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/content_script/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/content_script/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,8 +18,7 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   },
   "dependencies": {
     "left-pad": "^1.3.0"

packages/app-cli/tests/support/plugins/content_script/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/dialog/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/dialog/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/dialog/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/dialog/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/dialog/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/dialog/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/editor_context_menu/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/editor_context_menu/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/editor_context_menu/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/editor_context_menu/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/editor_context_menu/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/editor_context_menu/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/events/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/events/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/events/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/events/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/events/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/events/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/jpl_test/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/jpl_test/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/jpl_test/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/jpl_test/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/jpl_test/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/jpl_test/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/json_export/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/json_export/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/json_export/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/json_export/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/json_export/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/json_export/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/menu/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/menu/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/menu/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/menu/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/menu/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/menu/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/multi_selection/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/multi_selection/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/multi_selection/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/multi_selection/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/multi_selection/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/multi_selection/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/package.json

@@ -0,0 +1,19 @@
+{
+  "name": "test_plugin",
+  "version": "1.0.0",
+  "description": "",
+  "scripts": {
+    "dist": "webpack"
+  },
+  "keywords": [],
+  "author": "",
+  "license": "MIT",
+  "devDependencies": {
+    "@types/node": "^14.0.14",
+    "copy-webpack-plugin": "^6.0.3",
+    "ts-loader": "^7.0.5",
+    "typescript": "^3.9.3",
+    "webpack": "^4.43.0",
+    "webpack-cli": "^3.3.11"
+  }
+  }

packages/app-cli/tests/support/plugins/register_command/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/register_command/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/register_command/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/register_command/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/register_command/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/register_command/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/selected_text/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/selected_text/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/selected_text/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/selected_text/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/selected_text/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/selected_text/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/settings/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/settings/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/settings/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/settings/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/settings/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,7 +18,6 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   }
 }

packages/app-cli/tests/support/plugins/settings/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/toc/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/toc/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/toc/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/toc/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/toc/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,8 +18,7 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   },
   "dependencies": {
     "uslug": "^1.0.4"

packages/app-cli/tests/support/plugins/toc/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-cli/tests/support/plugins/withExternalModules/.gitignore

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/app-cli/tests/support/plugins/withExternalModules/.npmignore

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/app-cli/tests/support/plugins/withExternalModules/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/app-cli/tests/support/plugins/withExternalModules/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/app-cli/tests/support/plugins/withExternalModules/package.json

@@ -4,9 +4,9 @@
   "description": "",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
-  "keywords": ["joplin-plugin"],
+  "keywords": {},
   "license": "MIT",
   "devDependencies": {
     "@types/node": "^14.0.14",
@@ -18,8 +18,7 @@
     "ts-loader": "^7.0.5",
     "typescript": "^3.9.3",
     "webpack": "^4.43.0",
-    "webpack-cli": "^3.3.11",
-    "chalk": "^4.1.0"
+    "webpack-cli": "^3.3.11"
   },
   "dependencies": {
     "left-pad": "^1.3.0"

packages/app-cli/tests/support/plugins/withExternalModules/webpack.config.js

@@ -1,61 +1,9 @@
 const path = require('path');
-const crypto = require('crypto');
 const fs = require('fs-extra');
-const chalk = require('chalk');
 const CopyPlugin = require('copy-webpack-plugin');
 const WebpackOnBuildPlugin = require('on-build-webpack');
 const tar = require('tar');
 const glob = require('glob');
-const execSync = require('child_process').execSync;
-
-const rootDir = path.resolve(__dirname);
-const distDir = path.resolve(rootDir, 'dist');
-const srcDir = path.resolve(rootDir, 'src');
-const publishDir = path.resolve(rootDir, 'publish');
-
-const manifestPath = `${srcDir}/manifest.json`;
-const packageJsonPath = `${rootDir}/package.json`;
-const manifest = readManifest(manifestPath);
-const pluginArchiveFilePath = path.resolve(publishDir, `${manifest.id}.jpl`);
-const pluginInfoFilePath = path.resolve(publishDir, `${manifest.id}.json`);
-
-fs.removeSync(distDir);
-fs.removeSync(publishDir);
-fs.mkdirpSync(publishDir);
-
-function validatePackageJson() {
-	const content = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
-	if (!content.name || content.name.indexOf('joplin-plugin-') !== 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package name should start with "joplin-plugin-" (found "${content.name}") in ${packageJsonPath}`));
-	}
-
-	if (!content.keywords || content.keywords.indexOf('joplin-plugin') < 0) {
-		console.warn(chalk.yellow(`WARNING: To publish the plugin, the package keywords should include "joplin-plugin" (found "${JSON.stringify(content.keywords)}") in ${packageJsonPath}`));
-	}
-
-	if (content.scripts && content.scripts.postinstall) {
-		console.warn(chalk.yellow(`WARNING: package.json contains a "postinstall" script. It is recommended to use a "prepare" script instead so that it is executed before publish. In ${packageJsonPath}`));
-	}
-}
-
-function fileSha256(filePath) {
-	const content = fs.readFileSync(filePath);
-	return crypto.createHash('sha256').update(content).digest('hex');
-}
-
-function currentGitInfo() {
-	try {
-		let branch = execSync('git rev-parse --abbrev-ref HEAD', { stdio: 'pipe' }).toString().trim();
-		const commit = execSync('git rev-parse HEAD', { stdio: 'pipe' }).toString().trim();
-		if (branch === 'HEAD') branch = 'master';
-		return `${branch}:${commit}`;
-	} catch (error) {
-		const messages = error.message ? error.message.split('\n') : [''];
-		console.info(chalk.cyan('Could not get git commit (not a git repo?):', messages[0].trim()));
-		console.info(chalk.cyan('Git information will not be stored in plugin info file'));
-		return '';
-	}
-}
 
 function readManifest(manifestPath) {
 	const content = fs.readFileSync(manifestPath, 'utf8');
@@ -71,7 +19,7 @@ function createPluginArchive(sourceDir, destPath) {
 	if (!distFiles.length) {
 		// Usually means there's an error, which is going to be printed by
 		// webpack
-		console.warn(chalk.yellow('Plugin archive was not created because the "dist" directory is empty'));
+		console.info('Plugin archive was not created because the "dist" directory is empty');
 		return;
 	}
 
@@ -88,27 +36,21 @@ function createPluginArchive(sourceDir, destPath) {
 		distFiles
 	);
 
-	console.info(chalk.cyan(`Plugin archive has been created in ${destPath}`));
+	console.info(`Plugin archive has been created in ${destPath}`);
 }
 
-function createPluginInfo(manifestPath, destPath, jplFilePath) {
-	const contentText = fs.readFileSync(manifestPath, 'utf8');
-	const content = JSON.parse(contentText);
-	content._publish_hash = `sha256:${fileSha256(jplFilePath)}`;
-	content._publish_commit = currentGitInfo();
-	fs.writeFileSync(destPath, JSON.stringify(content, null, '\t'), 'utf8');
-}
+const rootDir = path.resolve(__dirname);
+const distDir = path.resolve(rootDir, 'dist');
+const srcDir = path.resolve(rootDir, 'src');
+const manifestPath = `${srcDir}/manifest.json`;
+const manifest = readManifest(manifestPath);
+const archiveFilePath = path.resolve(__dirname, `${manifest.id}.jpl`);
 
-function onBuildCompleted() {
-	createPluginArchive(distDir, pluginArchiveFilePath);
-	createPluginInfo(manifestPath, pluginInfoFilePath, pluginArchiveFilePath);
-	validatePackageJson();
-}
+fs.removeSync(distDir);
 
 const baseConfig = {
 	mode: 'production',
 	target: 'node',
-	stats: 'errors-only',
 	module: {
 		rules: [
 			{
@@ -151,7 +93,7 @@ const lastStepConfig = {
 
 							// Currently we don't support JS files for the main
 							// plugin script. We support it for content scripts,
-							// but they should be declared in manifest.json,
+							// but theyr should be declared in manifest.json,
 							// and then they are also compiled and copied to
 							// /dist. So wse also don't need to copy JS files.
 							'**/*.js',
@@ -160,7 +102,9 @@ const lastStepConfig = {
 				},
 			],
 		}),
-		new WebpackOnBuildPlugin(onBuildCompleted),
+		new WebpackOnBuildPlugin(function() {
+			createPluginArchive(distDir, archiveFilePath);
+		}),
 	],
 };
 

packages/app-desktop/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@joplin/app-desktop",
-  "version": "1.6.2",
+  "version": "1.6.1",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

packages/app-desktop/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@joplin/app-desktop",
-  "version": "1.6.2",
+  "version": "1.6.1",
   "description": "Joplin for Desktop",
   "main": "main.js",
   "private": true,

packages/generator-joplin/README.md

@@ -37,19 +37,9 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
-To update the plugin framework, run `npm install -g generator-joplin && yo joplin --update`
+To update the plugin framework, run `yo joplin --update`
 
 Keep in mind that doing so will overwrite all the framework-related files **outside of the "src/" directory** (your source code will not be touched). So if you have modified any of the framework-related files, such as package.json or .gitignore, make sure your code is under version control so that you can check the diff and re-apply your changes.
 

packages/generator-joplin/generators/app/index.js

@@ -3,7 +3,29 @@
 const Generator = require('yeoman-generator');
 const chalk = require('chalk');
 const yosay = require('yosay');
-const { mergePackageKey, mergeIgnoreFile, packageNameFromPluginName } = require('./utils');
+
+function mergePackageKey(parentKey, source, dest) {
+	const output = Object.assign({}, dest);
+
+	for (const k in source) {
+		if (!(k in output)) {
+			// If the key doesn't exist in the destination, add it
+			output[k] = source[k];
+		} else if (parentKey === 'devDependencies') {
+			// If we are dealing with the dependencies, overwrite with the
+			// version from source.
+			output[k] = source[k];
+		} else if (typeof source[k] === 'object' && !Array.isArray(source[k]) && source[k] !== null) {
+			// If it's an object, recursively process it
+			output[k] = mergePackageKey(k, source[k], output[k]);
+		} else {
+			// Otherwise, the default is to preserve the destination key
+			output[k] = dest[k];
+		}
+	}
+
+	return output;
+}
 
 module.exports = class extends Generator {
 
@@ -20,7 +42,7 @@ module.exports = class extends Generator {
 	}
 
 	async prompting() {
-		this.log(yosay(`Welcome to the fine ${chalk.red('Joplin Plugin')} generator!`));
+		this.log(yosay(`Welcome to the fine ${chalk.red('generator-joplin')} generator!`));
 
 		if (this.options.update && !this.options.silent) {
 			const answers = await this.prompt([
@@ -28,10 +50,10 @@ module.exports = class extends Generator {
 					type: 'confirm',
 					name: 'proceed',
 					message: [
-						'Updating will overwrite the config-related files. It will not change the',
-						'  content of /src or README.md. If you have made any changes to some of the',
-						'  config files make sure your code is under version control so that you can',
-						'  inspect the diff and re-apply your changes if needed. Do you want to proceed?',
+						'Updating will overwrite all the generator files **except for the',
+						'src/ directory**. So if you have made any changes outside of src/',
+						'make sure your code is under version control so that you can inspect',
+						'the diff and re-apply your changes if needed. Do you want to proceed?',
 					].join('\n'),
 				},
 			]);
@@ -47,12 +69,12 @@ module.exports = class extends Generator {
 			{
 				type: 'input',
 				name: 'pluginId',
-				message: 'Plugin ID\n  Must be a globally unique ID such as "com.example.MyPlugin" or a UUID:',
+				message: 'Plugin ID [Must be a globally unique ID such as "com.example.MyPlugin" or a UUID]',
 			},
 			{
 				type: 'input',
 				name: 'pluginName',
-				message: 'Plugin name\n  User-friendly string which will be displayed in UI:',
+				message: 'Plugin name [User-friendly string which will be displayed in UI]',
 			},
 			{
 				type: 'input',
@@ -64,11 +86,6 @@ module.exports = class extends Generator {
 				name: 'pluginAuthor',
 				message: 'Author:',
 			},
-			{
-				type: 'input',
-				name: 'pluginRepositoryUrl',
-				message: 'Repository URL:',
-			},
 			{
 				type: 'input',
 				name: 'pluginHomepageUrl',
@@ -81,25 +98,11 @@ module.exports = class extends Generator {
 			for (const prompt of prompts) {
 				props[prompt.name] = '';
 			}
-			props.packageName = '';
-
 			this.props = props;
 		} else {
-			const initialProps = await this.prompt(prompts);
-
-			const defaultPackageName = packageNameFromPluginName(initialProps.pluginName);
-
-			const derivedProps = await this.prompt([
-				{
-					type: 'input',
-					name: 'packageName',
-					message: `The npm package will be named: "${defaultPackageName}"\n  Press ENTER to keep this default, or type a name to change it:`,
-				},
-			]);
-
-			if (!derivedProps.packageName) derivedProps.packageName = defaultPackageName;
-
-			this.props = Object.assign({}, initialProps, derivedProps);
+			return this.prompt(prompts).then(props => {
+				this.props = props;
+			});
 		}
 	}
 
@@ -111,9 +114,9 @@ module.exports = class extends Generator {
 
 		const files = [
 			'.gitignore_TEMPLATE',
-			'.npmignore_TEMPLATE',
-			'GENERATOR_DOC.md',
 			'package_TEMPLATE.json',
+			'README.md',
+			'GENERATOR_DOC.md',
 			'tsconfig.json',
 			'webpack.config.js',
 		];
@@ -121,7 +124,6 @@ module.exports = class extends Generator {
 		const noUpdateFiles = [
 			'src/index.ts',
 			'src/manifest.json',
-			'README.md',
 		];
 
 		const allFiles = files.concat(noUpdateFiles);
@@ -145,17 +147,6 @@ module.exports = class extends Generator {
 						},
 					}
 				);
-			} else if (this.options.update && (destFile === '.gitignore' || destFile === '.npmignore') && this.fs.exists(destFilePath)) {
-				const destContent = this.fs.read(destFilePath);
-
-				this.fs.copy(
-					this.templatePath(file),
-					destFilePath, {
-						process: (sourceBuffer) => {
-							return mergeIgnoreFile(sourceBuffer.toString(), destContent);
-						},
-					}
-				);
 			} else {
 				this.fs.copyTpl(
 					this.templatePath(file),

packages/generator-joplin/generators/app/templates/.gitignore_TEMPLATE

@@ -1,3 +1,3 @@
-dist/
+dist/*
 node_modules/
-publish/
+*.jpl

packages/generator-joplin/generators/app/templates/.npmignore_TEMPLATE

@@ -1,8 +0,0 @@
-*.md
-!README.md
-/*.jpl
-/api
-/src
-/dist
-tsconfig.json
-webpack.config.js

packages/generator-joplin/generators/app/templates/GENERATOR_DOC.md

@@ -37,16 +37,6 @@ To build the plugin, simply run `npm run dist`.
 
 The project is setup to use TypeScript, although you can change the configuration to use plain JavaScript.
 
-## Publishing the plugin
-
-To publish the plugin, add it to npmjs.com by running `npm publish`. Later on, a script will pick up your plugin and add it automatically to the Joplin plugin repository as long as the package satisfies these conditions:
-
-- In `package.json`, the name starts with "joplin-plugin-". For example, "joplin-plugin-toc".
-- In `package.json`, the keywords include "joplin-plugin".
-- In the `publish/` directory, there should be a .jpl and .json file (which are built by `npm run dist`)
-
-In general all this is done automatically by the plugin generator, which will set the name and keywords of package.json, and will put the right files in the "publish" directory. But if something doesn't work and your plugin doesn't appear in the repository, double-check the above conditions.
-
 ## Updating the plugin framework
 
 To update the plugin framework, run `yo joplin --update`

packages/generator-joplin/generators/app/templates/api/JoplinWorkspace.d.ts

@@ -1,4 +1,3 @@
-import { FolderEntity } from '../../database/types';
 import { Disposable } from './types';
 declare enum ItemChangeEventType {
     Create = 1,
@@ -54,13 +53,6 @@ export default class JoplinWorkspace {
      * Gets the currently selected note
      */
     selectedNote(): Promise<any>;
-    /**
-     * Gets the currently selected folder. In some cases, for example during
-     * search or when viewing a tag, no folder is actually selected in the user
-     * interface. In that case, that function would return the last selected
-     * folder.
-     */
-    selectedFolder(): Promise<FolderEntity>;
     /**
      * Gets the IDs of the selected notes (can be zero, one, or many). Use the data API to retrieve information about these notes.
      */

packages/generator-joplin/generators/app/templates/package_TEMPLATE.json

@@ -1,17 +1,13 @@
 {
-  "name": "<%= packageName %>",
+  "name": "joplin_plugin",
   "version": "1.0.0",
   "scripts": {
     "dist": "webpack",
-    "prepare": "npm run dist"
+    "postinstall": "npm run dist"
   },
   "license": "MIT",
-  "keywords": [
-    "joplin-plugin"
-  ],
   "devDependencies": {
     "@types/node": "^14.0.14",
-    "chalk": "^4.1.0",
     "copy-webpack-plugin": "^6.1.0",
     "fs-extra": "^9.0.1",
     "glob": "^7.1.6",