From d9576a042243dc7e9f599f2df4f500522ae9e762 Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Wed, 29 Oct 2025 09:50:49 +0900 Subject: [PATCH 1/2] chore: remove .skip in plugin tests --- test/plugin/plugin.test.js | 48 ++++++++++++++++---------------------- 1 file changed, 20 insertions(+), 28 deletions(-) diff --git a/test/plugin/plugin.test.js b/test/plugin/plugin.test.js index d6128749f..c26929136 100644 --- a/test/plugin/plugin.test.js +++ b/test/plugin/plugin.test.js @@ -22,42 +22,34 @@ describe('loading plugins from packages', function () { spawnSync('npm', ['install'], { cwd: testPackagePath, timeout: 5000 }); }); - it.skip( - 'should load plugins that are the default export (module.exports = pluginObj)', - async function () { - const loader = new PluginLoader([join(testPackagePath, 'default-export.js')]); - await loader.load(); - expect(loader.pushPlugins.length).to.equal(1); - expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true; - expect(loader.pushPlugins[0]).to.be.an.instanceOf(PushActionPlugin); - }, - ).timeout(10000); + it('should load plugins that are the default export (module.exports = pluginObj)', async function () { + const loader = new PluginLoader([join(testPackagePath, 'default-export.js')]); + await loader.load(); + expect(loader.pushPlugins.length).to.equal(1); + expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true; + expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPushActionPlugin'))).to + .be.true; + }).timeout(10000); - it.skip( - 'should load multiple plugins from a module that match the plugin class (module.exports = { pluginFoo, pluginBar })', - async function () { - const loader = new PluginLoader([join(testPackagePath, 'multiple-export.js')]); - await loader.load(); - expect(loader.pushPlugins.length).to.equal(1); - expect(loader.pullPlugins.length).to.equal(1); - expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true; - expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPushActionPlugin'))) - .to.be.true; - expect(loader.pullPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPullActionPlugin'))) - .to.be.true; - expect(loader.pushPlugins[0]).to.be.instanceOf(PushActionPlugin); - expect(loader.pullPlugins[0]).to.be.instanceOf(PullActionPlugin); - }, - ).timeout(10000); + it('should load multiple plugins from a module that match the plugin class (module.exports = { pluginFoo, pluginBar })', async function () { + const loader = new PluginLoader([join(testPackagePath, 'multiple-export.js')]); + await loader.load(); + expect(loader.pushPlugins.length).to.equal(1); + expect(loader.pullPlugins.length).to.equal(1); + expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true; + expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPushActionPlugin'))).to + .be.true; + expect(loader.pullPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPullActionPlugin'))).to + .be.true; + }).timeout(10000); - it.skip('should load plugins that are subclassed from plugin classes', async function () { + it('should load plugins that are subclassed from plugin classes', async function () { const loader = new PluginLoader([join(testPackagePath, 'subclass.js')]); await loader.load(); expect(loader.pushPlugins.length).to.equal(1); expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true; expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p, 'isGitProxyPushActionPlugin'))).to .be.true; - expect(loader.pushPlugins[0]).to.be.instanceOf(PushActionPlugin); }).timeout(10000); it('should not load plugins that are not valid modules', async function () { From a1a256d30ad01ae6eeddacb76abbf5697181b986 Mon Sep 17 00:00:00 2001 From: Juan Escalada Date: Wed, 29 Oct 2025 09:51:24 +0900 Subject: [PATCH 2/2] fix: add ./plugin to package.json exports entry --- package-lock.json | 2 +- package.json | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/package-lock.json b/package-lock.json index 4e2549c3c..bbb0085e4 100644 --- a/package-lock.json +++ b/package-lock.json @@ -55,7 +55,7 @@ "yargs": "^17.7.2" }, "bin": { - "git-proxy": "index.js", + "git-proxy": "dist/index.js", "git-proxy-all": "concurrently 'npm run server' 'npm run client'" }, "devDependencies": { diff --git a/package.json b/package.json index c0f2bb297..56c5679dd 100644 --- a/package.json +++ b/package.json @@ -29,6 +29,11 @@ "import": "./dist/src/types/models.js", "require": "./dist/src/types/models.js", "types": "./dist/src/types/models.d.ts" + }, + "./plugin": { + "import": "./dist/src/plugin.js", + "require": "./dist/src/plugin.js", + "types": "./dist/src/plugin.d.ts" } }, "scripts": {