f

Author: fengmk2

src/utils/sequencify.ts

@@ -18,11 +18,14 @@ function sequence(tasks: Record<string, SequencifyTask>,
   missing: string[], recursive: string[],
   nest: string[], optional: boolean, parent: string) {
   for (const name of names) {
-    if (result.requires[name]) return;
-
+    if (result.requires[name]) {
+      continue;
+    }
     const node = tasks[name];
     if (!node) {
-      if (optional === true) return;
+      if (optional === true) {
+        continue;
+      }
       missing.push(name);
     } else if (nest.includes(name)) {
       nest.push(name);

test/index.test.ts

@@ -1,4 +1,5 @@
 import { strict as assert } from 'node:assert';
+// oxlint-disable-next-line no-namespace
 import * as EggCore from '../src/index.js';
 import type { EggAppConfig } from '../src/index.js';
 
@@ -31,6 +32,7 @@ describe('test/index.test.ts', () => {
       'Router',
       'Singleton',
       'Timing',
+      'sequencify',
       'utils',
     ]);
   });

test/loader/mixin/load_plugin.test.ts

@@ -574,12 +574,11 @@ describe('test/loader/mixin/load_plugin.test.ts', () => {
     assert.equal(loader.allPlugins.zzz.path, getFilepath('load-plugin-config-override/plugins/zzz'));
   });
 
-  it.only('should support optionalDependencies', async () => {
+  it('should support optionalDependencies', async () => {
     app = createApp('plugin-optional-dependencies');
     const loader = app.loader;
     await loader.loadPlugin();
-    console.log(loader.orderPlugins.map(p => p.name));
-    assert.deepEqual(loader.orderPlugins.slice(2).map(p => p.name), [ 'package', 'e', 'b', 'a', 'f' ]);
+    assert.deepEqual(loader.orderPlugins.map(p => p.name), [ 'session', 'zzz', 'package', 'e', 'b', 'a', 'f' ]);
   });
 
   it('should warn when redefine plugin', async () => {