Bug (?): CI build fails when including 'aasvg' image in draft

[EskoDijk]

When including an aasvg image in the draft, a build error appears in CI. Local build remains ok.

Example CI run is at: https://github.com/dnssd-wg/draft-ietf-dnssd-tsr/actions/runs/23638235348/job/68852267205?pr=35
See full log below. The main errors seem related to SVG :

draft-ietf-dnssd-tsr.xml(208): Error: Did not expect element style there, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]
draft-ietf-dnssd-tsr.xml(208): Error: Element svg has extra content: style, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]

The third error is mysterious and mentions RFC 8766; however "8766" doesn't appear in the source document as far as I can tell. This error also goes away when not including the "aasvg" tag for a block of ASCII art. It's this error:

draft-ietf-dnssd-tsr.xml(0): Error: IDREF attribute target references an unknown ID "RFC8766", at None

Full log:

"/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp":"/github/runner_temp" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/draft-ietf-dnssd-tsr/draft-ietf-dnssd-tsr":"/github/workspace" ghcr.io/martinthomson/i-d-template-action:latest  "all"
git clone -q --depth 10 -b main \
    https://github.com/martinthomson/i-d-template lib
VERSIONED="versioned" lib/build-targets.sh draft-ietf-dnssd-tsr >>.targets.mk
make: '.targets.mk' is up to date.
cat draft-ietf-dnssd-tsr.md  | kramdown-rfc --v3 | lib/trace.sh -v -q draft-ietf-dnssd-tsr.xml -s venue python3 lib/add-note.py | lib/trace.sh -v -q draft-ietf-dnssd-tsr.xml -s v2v3 xml2rfc -q --rfc-base-url https://www.rfc-editor.org/rfc/ --id-base-url https://datatracker.ietf.org/doc/html/ --allow-local-file-access --cache=/github/workspace/.refcache --v2v3 /dev/stdin -o /dev/stdout >draft-ietf-dnssd-tsr.xml
lib/trace.sh -v draft-ietf-dnssd-tsr.txt -s xml2rfc-txt xml2rfc -q --rfc-base-url https://www.rfc-editor.org/rfc/ --id-base-url https://datatracker.ietf.org/doc/html/ --allow-local-file-access --cache=/github/workspace/.refcache --text --no-pagination draft-ietf-dnssd-tsr.xml -o draft-ietf-dnssd-tsr.txt
draft-ietf-dnssd-tsr.xml(208): Error: Did not expect element style there, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]
draft-ietf-dnssd-tsr.xml(208): Error: Element svg has extra content: style, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]
draft-ietf-dnssd-tsr.xml(0): Error: IDREF attribute target references an unknown ID "RFC8766", at None
/github/workspace/draft-ietf-dnssd-tsr.xml(10): Error: Invalid document before running preptool.
Unable to complete processing draft-ietf-dnssd-tsr.xml
make[1]: *** [lib/main.mk:145: draft-ietf-dnssd-tsr.txt] Error 1
lib/trace.sh -v draft-ietf-dnssd-tsr.html -s xml2rfc-html xml2rfc -q --rfc-base-url https://www.rfc-editor.org/rfc/ --id-base-url https://datatracker.ietf.org/doc/html/ --allow-local-file-access --cache=/github/workspace/.refcache --html --css=lib/v3.css --metadata-js-url=/dev/null draft-ietf-dnssd-tsr.xml -o draft-ietf-dnssd-tsr.html
draft-ietf-dnssd-tsr.xml(208): Error: Did not expect element style there, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]
draft-ietf-dnssd-tsr.xml(208): Error: Element svg has extra content: style, at /rfc/middle/section[2]/artset/artwork[1]/*/*[1]
draft-ietf-dnssd-tsr.xml(0): Error: IDREF attribute target references an unknown ID "RFC8766", at None
/github/workspace/draft-ietf-dnssd-tsr.xml(10): Error: Invalid document before running preptool.
Unable to complete processing draft-ietf-dnssd-tsr.xml
make[1]: *** [lib/main.mk:137: draft-ietf-dnssd-tsr.html] Error 1
make[1]: Target 'latest' not remade because of errors.
rm draft-ietf-dnssd-tsr.xml
make: *** [lib/main.mk:20: all] Error 2

[martinthomson]

See martinthomson/aasvg#25

The problem is with overzealous checking in xml2rfc. I think that you can avoid this problem by (temporarily) pinning aasvg to version 0.4.3 in your package.json.

[EskoDijk]

Thanks @martinthomson , that works!