Thanks for this PR! I'll take a look as soon as I can when back at the office the week of the 29th. In the meantime, could you ensure that all of the GH actions pass?

All GH actions are passing now 😄. Looking forward to your comments!

Unless I misunderstood, this PR is meant to offer a replacement solution for the files under assets/_vendor, right? As you can see, with these files removed (via 2951dae), the smoke tests are failing: https://github.com/google/docsy/actions/runs/7740838481/job/21106641865?pr=1797. Can you look into it? Thanks!

Unless I misunderstood, this PR is meant to offer a replacement solution for the files under assets/_vendor, right?

Yes, correct. In an ideal world, we could drop assets/_vendor indeed.

As you can see, with these files removed (via 2951dae), the smoke tests are failing: https://github.com/google/docsy/actions/runs/7740838481/job/21106641865?pr=1797. Can you look into it? Thanks!

But world is not always ideal and (go) software has bugs. The bug that affects us in this case is mentioned here. And the corresponding mount instruction is here:

To make it short, unless we move _rfs.scss to some other place in the repo, we have to keep assets/_vendor. Moving _rfs.scss to another location in the repo seems possible, though, then we have to adjust the mount instruction, too. Afterwards we should be able to delete folder assets/_vendor.

The whole point of the _cp:bs-rfs NPM script is to ensure that the file is automatically kept fresh. If we can't drop the local copy of _rfs.scss, then the mount points added in this PR are of no use: the UG already builds without those extra mount points even if the _cp:bs-rfs script is removed.

Hmm, maybe #1806 offers a win-win solution. PTAL