Incorrect behavior with `mdbook serve` compared to published version.

[andriyDev]

Using mdbook serve locally (following the instructions in the README) is quite different from the version on https://google.github.io/comprehensive-rust.

1. There's no theme or search button.
2. Clicking on a lesson (e.g., Control Flow Basics > break and continue) does not "expand" the appropriate section in the side bar.
3. Links don't "reset" correctly. For example, viewing Control Flow Basics > break and continue, then clicking Tuples and Arrays in the sidebar results in a Document not found page (it is trying to access localhost:3000/control-flow-basics/tuples-and-arrays.html).

This was on Chrome. I also tried it on Firefox (with no extensions) to the same end result. I am serving from a Linux laptop.

[djmitche]

I've seen this as well, and assumed it was something I'd done wrong. This started with #2547 but that was reverted in #2564. I assume it has to do with conflicting versions of various mdbook-* packages installed locally, compared to those expected by the repo contents.

[andriyDev]

How do I know which versions are the right ones? I ran the --locked commands in the README, so I'd assume it should install the same versions as in this repo? (perhaps that's wrong)

[andriyDev]

Ah looks like downgrading mdbook to 0.4.37 (to match the Github Workflow) seems to have worked. All the features now work correctly.

Should we include this in the README instructions? I might need to report this to mdbook-svgbob or something as well...

[djmitche]

I'm not sure how best to fix this. Maybe #2610 will help - WDYT @michael-kerscher?

[michael-kerscher]

Regarding the mismatch: When drafting that pull request I also noticed the mismatch of CI environment and the Cargo.toml.
The mismatch is caused by dependabot #2489 and that's why I did not yet sync this in #2610.

I'm thinking about a good way to have the CI synced with what users are using (with --locked) in the near future.

Regarding you original issue: I don't fully know what might have caused that, but while working on the update I noticed some mdbook .js+.css files (e.g. general.css and others) being created automatically, but some being committed to our repo (e.g. book.js - taking care of themes etc.) I could think of several issues that might contribute to some feature not working correctly because they are not in sync

could easily be index.hbs or some other template that is not in sync.

Before merging my other pull request I will have another look at this

[michael-kerscher]

I could reproduce the issue with a vanilla checkout of main and it turns out that index.hbs indeed needs an update if a recent mdbook is to be used.
So it is even more important to merge #2610 into main - this will also fix this issue

If we want to use automation like dependabot for upgrading mdbook we need to leverage the testing framework I introduced even more and test all the important bits of the page.

[michael-kerscher]

please run install-mdbook.sh to "update" (or actually downgrade) to the versions used in the CI