Skip to content

fix(core): recurse variable substitution into nested dicts and lists - #415

Open
Ankit-cs wants to merge 1 commit into
StackGuardian:mainfrom
Ankit-cs:fix/parameterization-nested-vars
Open

Ankit-cs wants to merge 1 commit into
StackGuardian:mainfrom
Ankit-cs:fix/parameterization-nested-vars

Conversation

@Ankit-cs

Copy link
Copy Markdown

Related to #296

Pull Request Template

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • Code has been linted.
  • Documentation has been updated (if needed).
  • Tests have been added and verified (100% passing).

@Ankit-cs

Copy link
Copy Markdown
Author

looking for feedback @refeed

@codecov

codecov Bot commented Oct 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/tirith/core/policy_parameterization.py 86.36% 2 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
src/tirith/core/policy_parameterization.py 94.33% <86.36%> (-5.67%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

No deployments
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.

fix(core): variable substitution does not recurse — nested {{var.x}} silently produces wrong verdicts

1 participant