Remove premature subcomponent build in CompositeComponent geometry

[gpeairs]

Previously calling geometry(cc::CompositeComponent) (e.g. in render! or autofill!) would build! the schematic of cc, replacing its components with CoordinateSystems. There's no reason for this to happen, and it prevents certain operations from working as expected after rendering the schematic (even though rendering shouldn't modify the schematic itself). An example is the use of a custom halo for a subcomponent—this was previously impossible without also defining a custom halo for the composite component that skipped the build! step, and even then the custom subcomponent halo could not be used after rendering. To demonstrate we add a custom halo for the star transmon island.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!