Skip to content

feat!: change frequency type and remove evidence-requirements - #451

Open
jpower432 wants to merge 3 commits into
gemaraproj:mainfrom
jpower432:breaking-assessment-plan-schema
Open

feat!: change frequency type and remove evidence-requirements#451
jpower432 wants to merge 3 commits into
gemaraproj:mainfrom
jpower432:breaking-assessment-plan-schema

Conversation

@jpower432

@jpower432 jpower432 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Description

This PR alters Policy Assessment Plans. Details below.

Schema Changes

Schema Changes Made

  • No schema changes
  • Audit Log schema (auditlog.cue) changes
  • Capability Catalog schema (capabilitycatalog.cue) changes
  • Control Catalog schema (controlcatalog.cue) changes
  • Enforcement Log schema (enforcementlog.cue) changes
  • Evaluation Log schema (evaluationlog.cue) changes
  • Guidance Catalog schema (guidancecatalog.cue) changes
  • Mapping Document schema (mappingdocument.cue) changes
  • Policy Document schema (policy.cue) changes
  • Principle Catalog schema (principlecatalog.cue) changes
  • Risk Catalog schema (riskcatalog.cue) changes
  • Threat Catalog schema (threatcatalog.cue) changes
  • Vector Catalog schema (vectorcatalog.cue) changes
  • Other

Schema Change Details

⚠️ Breaking changes to experimental schema ONLY

frequency field in #AssessmentPlan now int (days) instead of string. evidence-requirements field removed; replaced with evidence-types array of #EvidenceTypeDefinition entries defining valid evidence types and their validity periods.

Consumers must update:

  • Parse frequency as integer days, not string
  • Use evidence-types array instead of evidence-requirements string
  • No automatic migration path; requires manual assessment plan updates

Testing

  • Unit tests added/updated
  • Manual testing performed
  • Test data updated (if applicable)

Related Issues

Related to #417

Reviewer Hints


Self-review checklist

  • This PR has content that was created with AI assistance.
  • I have the experience and knowledge necessary to answer maintainer questions about the content of this PR, without using AI.

frequency field in #AssessmentPlan now int (days) instead of string.
evidence-requirements field removed; replaced with evidence-types array
of #EvidenceTypeDefinition entries defining valid evidence types and
their validity periods.

Consumers must update:
- Parse frequency as integer days, not string
- Use evidence-types array instead of evidence-requirements string
- No automatic migration path; requires manual assessment plan updates

Assisted-by: Claude (Anthropic, Claude Haiku 4.5)
Signed-off-by: Jennifer Power <barnabei.jennifer@gmail.com>
@jpower432
jpower432 marked this pull request as ready for review August 12, 2026 15:47
@jpower432
jpower432 requested a review from a team as a code owner August 12, 2026 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant