Resolved by changing the audit (no child header added)
Decision: these tools are embedded under policyengine.org, which already provides the PolicyEngine site header/nav. Duplicating that header inside the child app is redundant, so we are not adding a child-rendered shell.
Instead, the multizone shell audit in policyengine-app-v2 was updated so this route skips the brand/nav assertion while still being checked for liveness (HTTP status, runtime errors, blank pages):
Note: uc-rebalancing reached the same conclusion earlier (commit "Remove child PolicyEngine header per user preference") and was failing the audit for the same reason — it's covered by the same exemption.
No change is required in this repo. Closing once #1057 merges.
Resolved by changing the audit (no child header added)
Decision: these tools are embedded under policyengine.org, which already provides the PolicyEngine site header/nav. Duplicating that header inside the child app is redundant, so we are not adding a child-rendered shell.
Instead, the multizone shell audit in
policyengine-app-v2was updated so this route skips the brand/nav assertion while still being checked for liveness (HTTP status, runtime errors, blank pages):SHELL_BRAND_EXEMPT_SOURCESin.github/scripts/audit-app-zone-shell.mjsNote:
uc-rebalancingreached the same conclusion earlier (commit "Remove child PolicyEngine header per user preference") and was failing the audit for the same reason — it's covered by the same exemption.No change is required in this repo. Closing once #1057 merges.