Skip to content

Preserve FHIR types for Liquid loop variables - #92

Merged
brianpos merged 2 commits into
developfrom
brianpos-fix-liquid-variable-typing
Jul 15, 2026
Merged

brianpos merged 2 commits into
developfrom
brianpos-fix-liquid-variable-typing

Conversation

@brianpos

Copy link
Copy Markdown
Owner

Liquid loop variables lost their FHIR type metadata between expression evaluations, so unprefixed access such as i.value could not resolve polymorphic properties while %i.value could.

This keeps FHIRPath ResourceNode values intact across template evaluations and unwraps them only at boolean and rendering boundaries. As a result, prefixed and unprefixed loop-variable access now resolve Extension.value consistently without exposing internal node wrappers to custom renderers.

Regression coverage verifies string and markdown Extension values, boolean evaluation, and resolved custom-renderer inputs.

brianpos and others added 2 commits July 15, 2026 16:32
Keep internal FHIRPath resource nodes across template evaluations so unprefixed Liquid loop variables retain polymorphic property typing. Resolve nodes only at boolean and rendering boundaries, with regression coverage for Extension.value.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Exclude hidden tabs from the active split-pane set and avoid switching away from an output tab that is already visible after template evaluation.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-06b155100-92.eastasia.4.azurestaticapps.net

1 similar comment
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-06b155100-92.eastasia.4.azurestaticapps.net

@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://yellow-plant-0209dde10-92.centralus.3.azurestaticapps.net

1 similar comment
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://yellow-plant-0209dde10-92.centralus.3.azurestaticapps.net

@brianpos
brianpos merged commit b60545a into develop Jul 15, 2026
4 checks passed
@brianpos
brianpos deleted the brianpos-fix-liquid-variable-typing branch July 15, 2026 06:45
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