Skip to content

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

Merged
alkalescent merged 7 commits into
mainfrom
docs/dynamic-value-mappings
Aug 19, 2026
Merged

docs(policy): document dynamic value mappings#385
alkalescent merged 7 commits into
mainfrom
docs/dynamic-value-mappings

Conversation

@alkalescent

@alkalescent alkalescent commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

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.

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

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 477eb178-68c0-4956-a36b-3016d88c72f6

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

Copy link
Copy Markdown
Contributor

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

@alkalescent
alkalescent enabled auto-merge (squash) August 13, 2026 02:11
Comment thread docs/components/policy/index.md Outdated
Comment thread docs/components/policy/index.md Outdated
jakedoublev
jakedoublev previously approved these changes Aug 13, 2026
Comment thread docs/components/policy/dynamic-value-mappings.mdx Outdated
Comment thread docs/components/policy/dynamic-value-mappings.mdx Outdated
Comment thread docs/components/policy/dynamic-value-mappings.mdx Outdated
Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com>
jakedoublev
jakedoublev previously approved these changes Aug 13, 2026
jp-ayyappan
jp-ayyappan previously approved these changes Aug 17, 2026
Comment thread docs/components/policy/dynamic-value-mappings.mdx
Comment thread docs/components/policy/dynamic-value-mappings.mdx
Comment thread docs/components/policy/dynamic-value-mappings.mdx
@alkalescent
alkalescent dismissed stale reviews from jp-ayyappan and jakedoublev via 0c29517 August 18, 2026 02:12

@jp-ayyappan jp-ayyappan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@alkalescent
alkalescent merged commit 317a6a3 into main Aug 19, 2026
6 checks passed
@alkalescent
alkalescent deleted the docs/dynamic-value-mappings branch August 19, 2026 00:47
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.

6 participants