[Bug]: Excluded routes still appear in the no-config sidebars

[Tyneor]

Version

System:
    OS: Windows 10 10.0.19045
    CPU: (8) x64 Intel(R) Core(TM) i5-10310U CPU @ 1.70GHz
    Memory: 2.99 GB / 15.69 GB
  Browsers:
    Chrome: 121.0.6167.185
    Edge: Chromium (122.0.2365.92)
    Internet Explorer: 11.0.19041.3636
  npmPackages:
    rspress: ^1.0.0 => 1.15.1

Details

I added the following lines to the rspress.config.ts :

route: {
    exclude: ["assets/**/*", "**/4-ignore.*"],
  },

expecting all assets in /docs and the 4-ignore.md not to have routes created for them and not appear in sidebars.

The routes are indeed not created but they still appear in the sidebar:

I don't really know if this is a bug but I would say that no-config sidebars should not display non-existing pages.

Reproduce link

https://codesandbox.io/p/devbox/minimal-route-exclude-in-sidebar-reproduction-gwzr65?file=%2Frspress.config.ts

Reproduce Steps

1. remove _meta.jsonin docs/guide
2. add assets in docs/guide/assets/
3. add some markdown files in /docs/guide including 4-ignore.md
4. in rspress.config.ts add

route: {
    exclude: ["assets/**/*", "**/4-ignore.*"],
  },

5. the assets and the Ignore route still appear in the guide sidebar

[Tyneor]

I found a way to exclude the excluded route from the no-config sidebar.

I can work on a pull request if necessary but since I'm not sure that it is considered a bug or a feature that could be added to pluginAutoNavSidebar I will write my code here:

I replaced the following code:

rspress/packages/plugin-auto-nav-sidebar/src/walk.ts

Line 35 in 37e626d

const subItems = await fs.readdir(workDir);

with one of those:

// can ignore assets/**/* because it doesn't direcly match directories, it only matches files and route.exclude is supposed to target files
const includedFiles = globby.sync(`**`, { cwd: workDir, ignore: [...exclude] });
const subItems = Array.from(new Set(includedFiles.map((file) => file.split("/")[0])));

  // can only ignore assets/** because it only matches direct children
  const subItems = globby.sync(`*`, { cwd: workDir, onlyFiles: false, ignore: [...exclude] });

globby comes from import globby from "globby"; and exclude comes from config.route.exclude

Side note
I also had to add sidebar: {} in the themeConfig to force my plugins: [pluginAutoNavSidebar()] to register the plugin.
Maybe user defined plugins should take precedence over plugins added automatically?

[10Derozan]

Yes, you are right. When fallback to readdir, we need add exclude. Welcome PR

[10Derozan]

Reply to your side note:
config hooks can not determine the order, maybe you need to add sidebar to avoid internal plugin.