Skip to content

Bump xbrlkit to 0.7.3 - #1366

Merged
jfrench9 merged 1 commit into
mainfrom
chore/bump-xbrlkit-0-7-3
Sep 8, 2026
Merged

Bump xbrlkit to 0.7.3#1366
jfrench9 merged 1 commit into
mainfrom
chore/bump-xbrlkit-0-7-3

Conversation

@jfrench9

@jfrench9 jfrench9 commented Sep 8, 2026

Copy link
Copy Markdown
Member

0.7.3 fixes a term collision in the holon's canonical context. 0.7.0 added a second "label" key, which silently shadowed the "label": "rdfs:label" already there — so every holon xbrlkit wrote in 0.7.0–0.7.2 bound label to rs:label instead. This platform's own holon context binds it to rdfs:label, so the two disagreed about one term across two documents that both call themselves holons.

Why the floor moves, not just the lock

>=0.7.2 would still resolve a version that writes the wrong binding, and the SEC artifact path (adapters/sec/processors/artifacts.py) goes through xbrlkit's holon writer. 0.7.3 is the minimum that produces correct output, not merely the newest — so it belongs in pyproject.toml, not only in the lock.

Nothing changes today

  • RoboLedger's holon comes from the platform's own encoder (operations/serialization/rdf/), untouched by any xbrlkit version
  • its Tavi comes from an emitter byte-identical since 0.6.0
  • the SEC download schedule is DefaultScheduleStatus.STOPPED, so no artifact is being written on either version

This is the prerequisite for regenerating the SEC holons on the CDN, which waits on the serialization waist's phase 3 so the vocabulary is published once rather than twice.

Verification

  • 2,513 tests pass across operations/serialization, adapters/sec, schemas and operations/roboledger, including test_xbrlkit_parity.py
  • just test-code clean
  • the lock moves xbrlkit alone: 0.7.2 -> 0.7.3, no other churn

🤖 Generated with Claude Code

https://claude.ai/code/session_01EiGTpXmxwZjxzaRoVs4VxB

0.7.3 fixes a term collision in the holon's canonical context: 0.7.0 added a
second `"label"` key, which silently shadowed the `"label": "rdfs:label"`
already there, so every holon xbrlkit wrote in 0.7.0-0.7.2 bound `label` to
`rs:label` instead. This platform's own holon context binds it to `rdfs:label`,
so the two disagreed about one term across two documents that both call
themselves holons.

The floor moves with the lock rather than the lock alone. `>=0.7.2` would still
resolve a version that writes the wrong binding, and the SEC artifact path
(`adapters/sec/processors/artifacts.py`) goes through xbrlkit's holon writer —
so 0.7.3 is the minimum that produces correct output, not merely the newest.

Nothing here changes today: RoboLedger's holon comes from the platform's own
encoder, its Tavi from an emitter unchanged since 0.6.0, and the SEC schedule
is stopped, so no artifact is being written on either version. This is the
prerequisite for regenerating the SEC holons on the CDN, which waits on the
serialization waist's phase 3 so the vocabulary is published once.

2,513 tests across serialization, the SEC adapter, the schemas and roboledger
pass; code quality clean. The lock moves xbrlkit alone.
@github-actions github-actions Bot added the change:standard Standard change: test gate only, no secondary review required label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Change classification: standard — only dependency manifests, lockfiles, documentation, release notes or generated output changed.

Under the change standard a standard change is gated by the required test check alone; no secondary review is required. Recorded by the change-classification job.

@jfrench9
jfrench9 merged commit cc4caea into main Sep 8, 2026
8 checks passed
@jfrench9
jfrench9 deleted the chore/bump-xbrlkit-0-7-3 branch September 8, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:standard Standard change: test gate only, no secondary review required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant