docs(policy): document dynamic value mappings by alkalescent · Pull Request #385 · opentdf/docs

alkalescent · 2026-08-12T15:08:12Z

Summary

explain Dynamic Value Mapping configuration, evaluation, resolver behavior, and constraints
add a patient assignment example with matching outcomes
link the page from the Policy overview

Testing

vale docs/components/policy/dynamic-value-mappings.mdx docs/components/policy/index.md
npm run check-vendored-yaml
npm run build

Summary by CodeRabbit

Documentation
- Added comprehensive documentation for Dynamic Value Mappings, including configuration, composition, resolver operators, evaluation behavior, constraints, and CLI usage.
- Added Dynamic Value Mappings to the policy documentation overview and relationship diagram.
- Clarified key-status semantics and the requirements for maintaining alphabetical key-status sorting.
New Features
- Added key-status sorting as a supported policy key sorting option.

coderabbitai · 2026-08-12T15:08:37Z

Warning

Review limit reached

@alkalescent, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 14 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2376e84c-cc17-4552-8908-0873b6c48e4e

📥 Commits

Reviewing files that changed from the base of the PR and between f4f4b78 and e9551cf.

📒 Files selected for processing (1)

docs/components/policy/dynamic-value-mappings.mdx

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4a2e6133-36fb-457f-817b-33f24f6aee9f

📥 Commits

Reviewing files that changed from the base of the PR and between 8da9189 and f4f4b78.

📒 Files selected for processing (1)

docs/components/policy/index.md

🚧 Files skipped from review as they are similar to previous changes (1)

docs/components/policy/index.md

📝 Walkthrough

Walkthrough

The PR adds Dynamic Value Mappings documentation and links it from the Policy documentation. It also updates policy specifications with KeyStatus sorting metadata and mapping requirements.

Changes

Dynamic Value Mappings

Layer / File(s)	Summary
Mapping concepts and configuration `docs/components/policy/dynamic-value-mappings.mdx`	Defines Dynamic Value Mappings, configuration, selectors, supported operators, evaluation behavior, and policy constraints.
Evaluation example and references `docs/components/policy/dynamic-value-mappings.mdx`	Adds a patient-assignment example and links to related Attribute, Subject Mapping, Entity Resolution, Action, CLI, and API documentation.
Policy documentation integration `docs/components/policy/index.md`	Adds Dynamic Value Mappings to the Policy relationship diagram and component list.

KeyStatus sorting specification

Layer / File(s)	Summary
KeyStatus mapping and sorting contract `specs/policy/kasregistry/key_access_server_registry.openapi.yaml`, `specs/policy/objects.openapi.yaml`, `specs/policy/unsafe/unsafe.openapi.yaml`	Documents the required `key_status` mapping and adds `SORT_KAS_KEYS_TYPE_KEY_STATUS` as a supported key-list sorting field.

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: ⚪ Minimal · up to f4f4b

This PR documents dynamic value mappings and updates related policy references and specifications. No actionable merge-blocking risk remains beyond normal checks and review.

Poem

A rabbit mapped each value with care,
Through resolvers hopping everywhere.
Key statuses sorted in rows,
Policy links now clearly show.
“Documentation blooms!” the rabbit sings,
With tidy rules and useful things.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: adding documentation for Dynamic Value Mappings in the policy documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch docs/dynamic-value-mappings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

github-actions · 2026-08-12T15:12:39Z

📄 Preview deployed to https://opentdf-docs-pr-385.surge.sh

pflynn-virtru · 2026-08-13T17:13:00Z

 - Actions (optionally scoped to Namespaces)
 - Subject Mappings (optionally scoped to Namespaces)
  - Subject Condition Sets (optionally scoped to Namespaces)
+- [Dynamic Value Mappings](/components/policy/dynamic-value-mappings) (optionally scoped to Namespaces)


I think we need /docs prepended , or relative

Using relative paths is best practice

That would cause a 404 on the PR preview site.

What we REALLY need is an automated job that will compile and build docs changes and flag on broken links (cc @penadenise ) -- not an immediate blocker obviously

jp-ayyappan · 2026-08-13T17:19:21Z

 - Actions (optionally scoped to Namespaces)
 - Subject Mappings (optionally scoped to Namespaces)
  - Subject Condition Sets (optionally scoped to Namespaces)
+- [Dynamic Value Mappings](/components/policy/dynamic-value-mappings) (optionally scoped to Namespaces)


Using relative paths is best practice

jakedoublev · 2026-08-13T18:57:32Z

+
+```mermaid
+flowchart LR
+  Resource["Resource Attribute Value<br/>.../value/mrn-123"] --> Segment["Value segment<br/>mrn-123"]


It doesn't look like this
is rendering in the GH mermaid renderer 🤔

Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com>

docs(policy): document dynamic value mappings

de7e12b

alkalescent requested review from a team as code owners August 12, 2026 15:08

chore(docs): update vendored OpenAPI specs

8da9189

alkalescent enabled auto-merge (squash) August 13, 2026 02:11

pflynn-virtru reviewed Aug 13, 2026

View reviewed changes

jp-ayyappan requested changes Aug 13, 2026

View reviewed changes

docs(policy): use relative dynamic value mapping link

f4f4b78

alkalescent requested a review from jp-ayyappan August 13, 2026 18:48

jakedoublev previously approved these changes Aug 13, 2026

View reviewed changes

Apply suggestions from code review

e9551cf

Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com>

alkalescent dismissed jakedoublev’s stale review via e9551cf August 13, 2026 19:16

jakedoublev approved these changes Aug 13, 2026

View reviewed changes

Navigation Menu

Uh oh!

docs(policy): document dynamic value mappings - #385

docs(policy): document dynamic value mappings#385
alkalescent wants to merge 4 commits into
mainfrom
docs/dynamic-value-mappings

alkalescent commented Aug 12, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Aug 12, 2026 •

edited

Loading

Review limit reached

Walkthrough

Changes

Uh oh!

github-actions Bot commented Aug 12, 2026

Uh oh!

pflynn-virtru Aug 13, 2026 •

edited

Loading

Uh oh!

jp-ayyappan Aug 13, 2026

Uh oh!

alkalescent Aug 13, 2026

Uh oh!

marythought Aug 13, 2026

Uh oh!

jp-ayyappan Aug 13, 2026

Uh oh!

Uh oh!

Uh oh!

jakedoublev Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

alkalescent commented Aug 12, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Testing

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Aug 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Walkthrough

Changes

Uh oh!

github-actions Bot commented Aug 12, 2026

Uh oh!

pflynn-virtru Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jp-ayyappan Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

alkalescent Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

marythought Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

jp-ayyappan Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

jakedoublev Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

alkalescent commented Aug 12, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Aug 12, 2026 •

edited

Loading

pflynn-virtru Aug 13, 2026 •

edited

Loading