Skip to content

Fix uri wiring fragment - #315

Merged
stewit merged 2 commits into
developfrom
fix-uri-wiring-fragment
Nov 6, 2026
Merged

stewit merged 2 commits into
developfrom
fix-uri-wiring-fragment

Conversation

@stewit

@stewit stewit commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

stewit added 2 commits November 6, 2026 22:27
The fragment of a uri wiring (#ref_key=...&ref_id_type=..., attrs,
use_default_value) was parsed from the query instead of the fragment.
So it was silently ignored, and a uri with both filters and a fragment
was rejected with "Extra inputs are not permitted". Fragment values are
now single values (a key given twice is an error) and attrs is parsed
from json.

The checks for metadata types (ref_key, ref_id_type) and for ref_id
ran before the uri was expanded into the wiring fields. With an explicit
adapter_id they rejected wirings whose missing fields the uri provides.
They now only run on the expanded wiring. The uri still overrides the
fields it contains.

Assisted-by: Claude Code:claude-opus-5-5
@stewit
stewit merged commit acc41c2 into develop Nov 6, 2026
3 checks passed
@stewit
stewit deleted the fix-uri-wiring-fragment branch November 6, 2026 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant