feat: make Base.plugin() accept an array instead of

BREAKING CHANGE: `Base.plugin()` now only accepts a single argument which has to be an array of plugin functions

Author: gr2m

index.d.ts

@@ -91,7 +91,7 @@ export declare class Base<TOptions extends Base.Options = Base.Options> {
     Plugins extends [Plugin, ...Plugin[]]
   >(
     this: Class,
-    ...plugins: Plugins
+    plugins: Plugins
   ): Class & {
     plugins: [...Class["plugins"], ...Plugins];
   } & Constructor<UnionToIntersection<ReturnTypeOf<Plugins>>>;

index.js

@@ -6,7 +6,7 @@ export class Base {
     });
   }
 
-  static plugin(...newPlugins) {
+  static plugin(newPlugins) {
     const currentPlugins = this.plugins;
     return class extends this {
       static plugins = currentPlugins.concat(

index.test-d.ts

@@ -33,7 +33,7 @@ new BaseWithEmptyDefaults();
 // @ts-expect-error
 new BaseWithEmptyDefaults({});
 
-const BaseLevelOne = Base.plugin(fooPlugin).defaults({
+const BaseLevelOne = Base.plugin([fooPlugin]).defaults({
   defaultOne: "value",
   required: "1.2.3",
 });
@@ -121,15 +121,15 @@ expectType<string>(baseLevelThree.options.required);
 // @ts-expect-error unknown properties cannot be used, see #31
 baseLevelThree.unknown;
 
-const BaseWithVoidPlugin = Base.plugin(voidPlugin);
+const BaseWithVoidPlugin = Base.plugin([voidPlugin]);
 const baseWithVoidPlugin = new BaseWithVoidPlugin({
   required: "1.2.3",
 });
 
 // @ts-expect-error unknown properties cannot be used, see #31
 baseWithVoidPlugin.unknown;
 
-const BaseWithFooAndBarPlugins = Base.plugin(barPlugin, fooPlugin);
+const BaseWithFooAndBarPlugins = Base.plugin([barPlugin, fooPlugin]);
 const baseWithFooAndBarPlugins = new BaseWithFooAndBarPlugins({
   required: "1.2.3",
 });
@@ -140,11 +140,11 @@ expectType<string>(baseWithFooAndBarPlugins.bar);
 // @ts-expect-error unknown properties cannot be used, see #31
 baseWithFooAndBarPlugins.unknown;
 
-const BaseWithVoidAndNonVoidPlugins = Base.plugin(
+const BaseWithVoidAndNonVoidPlugins = Base.plugin([
   barPlugin,
   voidPlugin,
-  fooPlugin
-);
+  fooPlugin,
+]);
 const baseWithVoidAndNonVoidPlugins = new BaseWithVoidAndNonVoidPlugins({
   required: "1.2.3",
 });
@@ -155,7 +155,7 @@ expectType<string>(baseWithVoidAndNonVoidPlugins.bar);
 // @ts-expect-error unknown properties cannot be used, see #31
 baseWithVoidAndNonVoidPlugins.unknown;
 
-const BaseWithOptionsPlugin = Base.plugin(withOptionsPlugin);
+const BaseWithOptionsPlugin = Base.plugin([withOptionsPlugin]);
 const baseWithOptionsPlugin = new BaseWithOptionsPlugin({
   required: "1.2.3",
 });
@@ -196,7 +196,7 @@ expectType<{
 const BaseWithChainedDefaultsAndPlugins = Base.defaults({
   defaultOne: "value",
 })
-  .plugin(fooPlugin)
+  .plugin([fooPlugin])
   .defaults({
     defaultTwo: 0,
   });
@@ -212,11 +212,11 @@ expectType<string>(baseWithChainedDefaultsAndPlugins.foo);
 const BaseWithManyChainedDefaultsAndPlugins = Base.defaults({
   defaultOne: "value",
 })
-  .plugin(fooPlugin, barPlugin, voidPlugin)
+  .plugin([fooPlugin, barPlugin, voidPlugin])
   .defaults({
     defaultTwo: 0,
   })
-  .plugin(withOptionsPlugin)
+  .plugin([withOptionsPlugin])
   .defaults({
     defaultThree: ["a", "b", "c"],
   });

test.js

@@ -6,25 +6,25 @@ import { fooPlugin } from "./plugins/foo/index.js";
 import { barPlugin } from "./plugins/bar/index.js";
 import { voidPlugin } from "./plugins/void/index.js";
 
-test(".plugin(fooPlugin)", () => {
-  const FooTest = Base.plugin(fooPlugin);
+test(".plugin([fooPlugin])", () => {
+  const FooTest = Base.plugin([fooPlugin]);
   const fooTest = new FooTest();
   assert.equal(fooTest.foo, "foo");
 });
-test(".plugin(fooPlugin, barPlugin)", () => {
-  const FooBarTest = Base.plugin(fooPlugin, barPlugin);
+test(".plugin([fooPlugin, barPlugin])", () => {
+  const FooBarTest = Base.plugin([fooPlugin, barPlugin]);
   const fooBarTest = new FooBarTest();
   assert.equal(fooBarTest.foo, "foo");
   assert.equal(fooBarTest.bar, "bar");
 });
-test(".plugin(fooPlugin, barPlugin, voidPlugin)", () => {
-  const FooBarTest = Base.plugin(fooPlugin, barPlugin, voidPlugin);
+test(".plugin([fooPlugin, barPlugin, voidPlugin])", () => {
+  const FooBarTest = Base.plugin([fooPlugin, barPlugin, voidPlugin]);
   const fooBarTest = new FooBarTest();
   assert.equal(fooBarTest.foo, "foo");
   assert.equal(fooBarTest.bar, "bar");
 });
-test(".plugin(fooPlugin).plugin(barPlugin)", () => {
-  const FooBarTest = Base.plugin(fooPlugin).plugin(barPlugin);
+test(".plugin([fooPlugin]).plugin(barPlugin)", () => {
+  const FooBarTest = Base.plugin([fooPlugin]).plugin([barPlugin]);
   const fooBarTest = new FooBarTest();
   assert.equal(fooBarTest.foo, "foo");
   assert.equal(fooBarTest.bar, "bar");
@@ -61,12 +61,12 @@ test(".defaults({foo: 'bar', baz: 'daz' })", () => {
 });
 
 test(".plugin().defaults()", () => {
-  const BaseWithPluginAndDefaults = Base.plugin(fooPlugin).defaults({
+  const BaseWithPluginAndDefaults = Base.plugin([fooPlugin]).defaults({
     baz: "daz",
   });
   const BaseWithDefaultsAndPlugin = Base.defaults({
     baz: "daz",
-  }).plugin(fooPlugin);
+  }).plugin([fooPlugin]);
 
   const instance1 = new BaseWithPluginAndDefaults();
   const instance2 = new BaseWithDefaultsAndPlugin();