Skip to content

feat(policy): establish one canonical authored policy representation - #3318

Closed
johnnygreco wants to merge 1 commit into
mainfrom
johnny/policy-semantics-dedup
Closed

johnnygreco wants to merge 1 commit into
mainfrom
johnny/policy-semantics-dedup

Conversation

@johnnygreco

@johnnygreco johnnygreco commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Establish one dependency-light representation of the complete authored OpenShell policy format so runtime loading and policy-analysis consumers share the same schema, parsing, defaults, and intrinsic policy-language behavior.

Related Issue

Closes #3333.

Changes

  • Add openshell-policy-schema as the owner of authored policy types, bounded parsing, serialization, and pure validation.
  • Preserve authoring-time presence, closed-object extensions, and open user-data maps.
  • Keep runtime protobuf messages and prover models as consumer-specific derived representations.
  • Migrate openshell-policy and the existing risk prover to the shared authored representation.
  • Preserve compatibility adapters and document intentional behavior corrections.

Testing

  • Schema round-trip and parser-limit tests
  • Runtime/protobuf conversion and validation tests
  • Cross-consumer parity and existing-prover projection tests
  • mise run pre-commit
  • mise run test
  • mise run e2e:docker
  • mise run ci

Checklist

  • Follows Conventional Commits
  • Commits are signed off
  • Architecture documentation is updated
  • User-facing policy documentation is updated

@copy-pr-bot

copy-pr-bot Bot commented Sep 14, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

Signed-off-by: Johnny Greco <jogreco@nvidia.com>
@johnnygreco johnnygreco changed the title refactor(policy): share access preset semantics feat(policy): establish one canonical authored policy representation Sep 15, 2026
@johnnygreco
johnnygreco changed the base branch from 2109-standalone-policy-prover/jg to main September 15, 2026 02:26
@johnnygreco
johnnygreco deleted the johnny/policy-semantics-dedup branch September 15, 2026 02:27
@johnnygreco
johnnygreco restored the johnny/policy-semantics-dedup branch September 15, 2026 02:28
@johnnygreco johnnygreco reopened this Sep 15, 2026
@johnnygreco
johnnygreco deleted the johnny/policy-semantics-dedup branch September 15, 2026 02:32
@johnnygreco

Copy link
Copy Markdown
Collaborator Author

Superseded by #3334, opened from the clean issue-backed branch 3333-policy-schema/jg against main.

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.

feat(policy): establish one canonical authored policy representation

1 participant