Skip to content

[ci-maintainer] Import reference architectures fails at validate:architectures — SVG→PNG conversion does not rewrite asset reference (run 35175044427) #218

Description

@hivecommons-hive

CI Issue

The daily Import reference architectures workflow now fails one step later than before. The librsvg2-bin fix landed (the apt install step runs and import:architectures succeeds), but npm run validate:architectures fails:

[error] swisscom-cloud-native-telco: missing asset /img/architectures/swisscom-cloud-native-telco/swisscom-cloud-native-telco-automation-architecture-overview.svg
[error] zeiss: missing asset /img/architectures/zeiss/solutionArchitecture.svg

Root cause: scripts/import-architectures.mjs converts raster-embedded SVGs to PNG:

Converted raster-embedded SVG to PNG: img/architectures/swisscom-cloud-native-telco/swisscom-cloud-native-telco-automation-architecture-overview.png
Converted raster-embedded SVG to PNG: img/architectures/zeiss/solutionArchitecture.png

but does not rewrite the asset reference in the architecture MDX/frontmatter, which still points at the .svg path. Validation then looks for an .svg that the import deliberately never wrote.

Evidence

Recommendation

In scripts/import-architectures.mjs, after rsvg-convert converts a raster-embedded SVG to PNG, rewrite the asset reference in the generated architecture MDX (and any mirrored asset index) from <name>.svg to <name>.png so validate:architectures checks the file that actually exists.

Coordination note: scripts/import-architectures.mjs is currently touched by hold-gated PR #195 (sec-check, SVG active-content rejection) and its tests by PR #216. This fix should land after/rebased on those to avoid conflicts.

Completion criterion

  • Import reference architectures scheduled run passes validate:architectures with the two raster-embedded architectures present as PNG assets referenced from their MDX.

Filed by ci-maintainer agent (ACMM L4/L5 — hold-gated mode)

🐝 Hive Agent: ci-maintainer | Instance: hosted-available-lke648397-260827-5n31 | SHA: unknown

— hive: agent=ci-maintainer backend=copilot model=kimi-k3

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent/ci-maintainerApproved by a Hive merger/owner for auto-merge on green CIciApproved by a Hive merger/owner for auto-merge on green CIhive/hosted-available-lke648397-260827-5n31Approved by a Hive merger/owner for auto-merge on green CI

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions