build(deps-dev): bump agentrust-trace-tests from 0.5.0 to 0.5.1 in /integrations/ramen-ai-cmcp - #152
Conversation
Bumps [agentrust-trace-tests](https://github.com/agentrust-io/trace-tests) from 0.5.0 to 0.5.1. - [Release notes](https://github.com/agentrust-io/trace-tests/releases) - [Changelog](https://github.com/agentrust-io/trace-tests/blob/main/CHANGELOG.md) - [Commits](agentrust-io/trace-tests@v0.5.0...v0.5.1) --- updated-dependencies: - dependency-name: agentrust-trace-tests dependency-version: 0.5.1 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
60a0f46 to
992fc33
Compare
agentrust-trace-tests 0.5.1 adds TR-RTE-004, which fires at Level 1 when no verifier nonce is supplied. This harness does not issue one and the fixture carries no runtime.nonce, so a software-only record now fails Level 1 on two runtime rules rather than one. The assertion stays an exact comparison deliberately: a future change to the Level 1 finding set should fail this test rather than pass silently. Coupled to the version bump in the same PR. TR-RTE-004 does not exist in 0.5.0, so this test fails against the previous pin and the bump fails against the previous test. Neither can land alone. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012yyHFd4YA35oNUjhrztJSb
|
Pushed the coupled test fix onto this branch rather than opening a separate PR, because the two changes cannot land apart. What was failing. Why it could not be a separate PR. I checked which codes each version emits: So the updated test fails against the 0.5.0 pin currently on Verified locally, running the way
I kept the assertion as an exact comparison on purpose. A conformance integration test should break when the suite's finding set changes; that is how we find out. The defect here was that nobody updated it, not that it was strict. Note that Merging once CI is green. If dependabot force-pushes this branch the test fix goes with it, in which case I will reapply it. |
imran-siddique
left a comment
There was a problem hiding this comment.
Merging. Three things had to be true together and now are.
The bump itself was never the problem. agentrust-trace-tests 0.5.1 adds TR-RTE-004, which fires at Level 1 when no verifier nonce is supplied, so a software-only record now fails Level 1 on two runtime rules instead of one. The ramen-ai-cmcp conformance test pinned the old finding set by exact equality, and its own name recorded the assumption: "fails only level 1 runtime rule".
The coupling is why the fix rode on this branch. TR-RTE-004 does not exist in 0.5.0, which I confirmed by reading both versions' tr_rte module. So the updated test fails against the pin on main, and the bump fails against the test on main. Either alone leaves the repository red. One commit, both changes.
The validate failure was mine and unrelated to this PR. It came from #148, which I merged while only its hold gate was reporting: an epi-recorder key that tested_against does not permit, and a stale generated README index. Both fixed in #158, and this branch has main merged in, since a stale-base validate does not clear on a re-run.
Fifteen checks, only the maintainer gate red, including all four conformance matrices and the full ramen-ai-cmcp suite at 23 passed.
The assertion stays exact deliberately. A conformance integration test should break when the suite's finding set changes; that is the signal. What went wrong was that nobody acted on the signal, not that the test was strict.
Bumps agentrust-trace-tests from 0.5.0 to 0.5.1.
Release notes
Sourced from agentrust-trace-tests's releases.
Changelog
Sourced from agentrust-trace-tests's changelog.
Commits
3af2b53ci: fail closed when CodeQL analysis fails (#78)d9c0da1fix(tr-sig): stop publishing a finding whose message names a different code (...f7669e6fix(tr-sig): report a malformed record instead of raising on it (#75)71e6bf2docs: align the published error codes and record samples with the modules (#74)10911bddocs: document project sponsorship roles (#77)121c18bci: deploy the docs when the theme or the landing page changes (#73)af4f010fix(seo): social cards said "Home", not the page title (#72)c89a983docs: give the site its own landing page instead of rendering the README (#71)c725bbbfix(tr-sig): canonicalize with RFC 8785, which is the rule this suite scores ...84ecb81fix(schema): resync the packaged copy, and a vector that uses the field whose...