Transaction-Science/open-standards
Watch
1
0
Fork
You've already forked open-standards
 0
Code Issues Pull requests Projects Releases 15 Packages Wiki Activity Actions

jouleclaw: quick-xml 0.41's normalized_value — main's macOS energy backends job is red #455

Closed
dcharlot wants to merge 1 commit from fix/quick-xml-normalized-value into main
pull from: fix/quick-xml-normalized-value
merge into: Transaction-Science:main
Conversation 0 Commits 1 Files changed 2 +6 -6
dcharlot commented 2026-10-24 19:58:47 -04:00
Owner
Copy link

Main has been red since #443 (b1681667) in standards-build's macOS energy backends job, and so is every open PR.

Cause: #443 moved quick-xml to 0.41, which deprecates Attribute::unescape_value. jouleclaw-loader-gguf denies warnings (#414), so it stopped compiling, and every workspace that builds it (jcp-energy, proof-cascade) failed.

Fix: six calls, three in the gguf loader's OpenVINO reader and three in jouleclaw-transform-trx. Each now calls normalized_value(quick_xml::XmlVersion::Implicit1_0). In quick-xml 0.41, unescape_value() is defined as normalized_value_with(XmlVersion::Implicit1_0, 1, resolve_predefined_entity), which is exactly normalized_value(XmlVersion::Implicit1_0), so the output is unchanged byte for byte.

Checks on the build host (macOS, as the job runs):

check result
cargo check --locked -p jouleclaw-loader-gguf --all-targets clean, no warnings
cargo test --locked -p jouleclaw-loader-gguf / -p jouleclaw-transform-trx 8 passed / 10 passed
the job's own steps: jcp-energy and proof-cascade tests pass
cargo metadata --locked --offline (jouleclaw-rs) holds

🤖 Generated with Claude Code

Closed: Superseded by #456, which carries the same quick-xml 0.41 fix (normalized_value), together with #449's answer to the advisory gate's new RUSTSEC-2026-0385/0387.

Main has been red since #443 (`b1681667`) in standards-build's **macOS energy backends** job, and so is every open PR. **Cause:** #443 moved quick-xml to 0.41, which deprecates `Attribute::unescape_value`. `jouleclaw-loader-gguf` denies warnings (#414), so it stopped compiling, and every workspace that builds it (jcp-energy, proof-cascade) failed. **Fix:** six calls, three in the gguf loader's OpenVINO reader and three in `jouleclaw-transform-trx`. Each now calls `normalized_value(quick_xml::XmlVersion::Implicit1_0)`. In quick-xml 0.41, `unescape_value()` is defined as `normalized_value_with(XmlVersion::Implicit1_0, 1, resolve_predefined_entity)`, which is exactly `normalized_value(XmlVersion::Implicit1_0)`, so the output is unchanged byte for byte. **Checks on the build host (macOS, as the job runs):** | check | result | |---|---| | `cargo check --locked -p jouleclaw-loader-gguf --all-targets` | clean, no warnings | | `cargo test --locked -p jouleclaw-loader-gguf` / `-p jouleclaw-transform-trx` | 8 passed / 10 passed | | the job's own steps: `jcp-energy` and `proof-cascade` tests | pass | | `cargo metadata --locked --offline` (jouleclaw-rs) | holds | 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- **Closed:** Superseded by #456, which carries the same quick-xml 0.41 fix (normalized_value), together with #449's answer to the advisory gate's new RUSTSEC-2026-0385/0387.
jouleclaw: quick-xml 0.41's normalized_value, so the gguf loader compiles under its warnings gate
Some checks failed
standards workspaces build + test / which workspaces this change touches (pull_request) Successful in 11s
Details
jouleclaw public surface must stay documented / Public-surface audit (jouleclaw-rs) (pull_request) Successful in 1m8s
Details
standards workspaces build + test / mesh-rs (mesh/mesh-rs, -p mesh-engine-gguf -p mesh-openai --features mesh-engine-gguf/gguf,mesh-openai/server, mesh-rs, linux-arm64) (pull_request) Successful in 1m43s
Details
repository checks / lockfiles, advisories, tests in CI, workflow hygiene, spec and section references, public text (pull_request) Failing after 5m52s
Details
standards workspaces build + test / jouleclaw-rs (-nodebug, jouleclaw/jouleclaw-rs, -p jouleclaw-energy --features rapl,nvml, jouleclaw-rs, linux-arm64, -C target-feature=+fp16) (pull_request) Successful in 6m55s
Details
standards workspaces build + test / jouledesk-rs (jouledesk/jouledesk-rs, jouledesk-rs, macos) (pull_request) Has been cancelled
Details
standards workspaces build + test / macOS energy backends (pull_request) Has been cancelled
Details
c809783859 
#443 moved quick-xml to 0.41, which deprecates Attribute::unescape_value.
jouleclaw-loader-gguf denies warnings (#414), so it no longer compiled. Every
workspace that builds it failed, including standards-build's 'macOS energy
backends' job, red on main since b1681667 and on every open PR.

In 0.41, unescape_value() is defined as
normalized_value_with(XmlVersion::Implicit1_0, 1, resolve_predefined_entity),
which is exactly normalized_value(XmlVersion::Implicit1_0). So the six calls,
three in the gguf loader's OpenVINO reader and three in trx, keep their output
byte for byte.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
dcharlot closed this pull request 2026-10-24 20:08:56 -04:00
Some checks failed
standards workspaces build + test / which workspaces this change touches (pull_request) Successful in 11s
Details
jouleclaw public surface must stay documented / Public-surface audit (jouleclaw-rs) (pull_request) Successful in 1m8s
Details
standards workspaces build + test / mesh-rs (mesh/mesh-rs, -p mesh-engine-gguf -p mesh-openai --features mesh-engine-gguf/gguf,mesh-openai/server, mesh-rs, linux-arm64) (pull_request) Successful in 1m43s
Details
repository checks / lockfiles, advisories, tests in CI, workflow hygiene, spec and section references, public text (pull_request) Failing after 5m52s
Details
standards workspaces build + test / jouleclaw-rs (-nodebug, jouleclaw/jouleclaw-rs, -p jouleclaw-energy --features rapl,nvml, jouleclaw-rs, linux-arm64, -C target-feature=+fp16) (pull_request) Successful in 6m55s
Details
standards workspaces build + test / jouledesk-rs (jouledesk/jouledesk-rs, jouledesk-rs, macos) (pull_request) Has been cancelled
Details
standards workspaces build + test / macOS energy backends (pull_request) Has been cancelled
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Transaction-Science/open-standards!455
No description provided.