Skip to content

feat(ipa): Add IPA-111 validation rules #1460

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wtrocki merged 5 commits into from
Sep 11, 2026
Merged

feat(ipa): Add IPA-111 validation rules #1460

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9fd2085
CLOUDP-439595: add IPA-111 validation rules
mongodb-sage-bot[bot] Aug 27, 2026
308ef01
overrides
yelizhenden-mdb Aug 28, 2026
d10fb37
fix(ipa): complete IPA-111 override list to suppress all 318 spec vio…
wtrocki Sep 3, 2026
51eac3d
refactor(ipa): scope IPA-111 boolean and no-default rules to request …
wtrocki Sep 3, 2026
c8dc50e
chore(ipa): regenerate ruleset README after IPA-111 scoping change
wtrocki Sep 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
146 changes: 146 additions & 0 deletions tools/spectral/ipa/__tests__/IPA111EffectiveFieldsReadOnly.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,146 @@
import testRule from './__helpers__/testRule';
import { DiagnosticSeverity } from '@stoplight/types';

testRule('xgen-IPA-111-effective-fields-read-only', [
{
name: 'valid effective field with readOnly: true',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
effectiveState: { type: 'string', readOnly: true },
},
},
},
},
},
errors: [],
},
{
name: 'valid non-effective field without readOnly',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
state: { type: 'string' },
effective: { type: 'string' },
},
},
},
},
},
errors: [],
},
{
name: 'invalid effective field without readOnly',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
effectiveState: { type: 'string' },
},
},
},
},
},
errors: [
{
code: 'xgen-IPA-111-effective-fields-read-only',
message: 'Effective-value fields must be marked as readOnly: true.',
path: ['components', 'schemas', 'Schema', 'properties', 'effectiveState'],
severity: DiagnosticSeverity.Error,
},
],
},
{
name: 'invalid effective field with readOnly: false',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
effectiveState: { type: 'string', readOnly: false },
},
},
},
},
},
errors: [
{
code: 'xgen-IPA-111-effective-fields-read-only',
message: 'Effective-value fields must be marked as readOnly: true.',
path: ['components', 'schemas', 'Schema', 'properties', 'effectiveState'],
severity: DiagnosticSeverity.Error,
},
],
},
{
name: 'invalid effective field in request schema',
document: {
paths: {
'/resources': {
post: {
requestBody: {
content: {
'application/vnd.atlas.2024-01-01+json': {
schema: {
type: 'object',
properties: {
effectiveState: { type: 'string', readOnly: true },
},
},
},
},
},
},
},
},
},
errors: [
{
code: 'xgen-IPA-111-effective-fields-read-only',
message: 'Effective-value fields represent server-computed state and must not appear in request schemas.',
path: [
'paths',
'/resources',
'post',
'requestBody',
'content',
'application/vnd.atlas.2024-01-01+json',
'schema',
'properties',
'effectiveState',
],
severity: DiagnosticSeverity.Error,
},
],
},
{
name: 'invalid effective field without readOnly - exception',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
effectiveState: {
type: 'string',
'x-xgen-IPA-exception': {
'xgen-IPA-111-effective-fields-read-only': 'Reason',
},
},
},
},
},
},
},
errors: [],
},
]);
162 changes: 162 additions & 0 deletions tools/spectral/ipa/__tests__/IPA111OptionalBooleanFieldsDefaultFalse.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,162 @@
import testRule from './__helpers__/testRule';
import { DiagnosticSeverity } from '@stoplight/types';

testRule('xgen-IPA-111-optional-boolean-fields-default-false', [
{
name: 'valid optional boolean field with default false',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
enabled: { type: 'boolean', default: false },
},
},
},
},
},
errors: [],
},
{
name: 'valid required boolean field without default',
document: {
components: {
schemas: {
Schema: {
type: 'object',
required: ['enabled'],
properties: {
enabled: { type: 'boolean' },
},
},
},
},
},
errors: [],
},
{
name: 'valid non-boolean field without default',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
name: { type: 'string' },
},
},
},
},
},
errors: [],
},
{
name: 'valid optional boolean field without default in component (request-scoped, skipped)',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
enabled: { type: 'boolean' },
},
},
},
},
},
errors: [],
},
{
name: 'valid optional boolean field with default true in component (request-scoped, skipped)',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
enabled: { type: 'boolean', default: true },
},
},
},
},
},
errors: [],
},
{
name: 'invalid optional boolean field in request (paused) but response is skipped',
document: {
paths: {
'/resources': {
post: {
requestBody: {
content: {
'application/vnd.atlas.2024-01-01+json': {
schema: {
type: 'object',
properties: {
paused: { type: 'boolean' },
},
},
},
},
},
responses: {
201: {
content: {
'application/vnd.atlas.2024-01-01+json': {
schema: {
type: 'object',
properties: {
hidden: { type: 'boolean', default: true },
},
},
},
},
},
},
},
},
},
},
errors: [
{
code: 'xgen-IPA-111-optional-boolean-fields-default-false',
message: 'Optional boolean fields must default to false.',
path: [
'paths',
'/resources',
'post',
'requestBody',
'content',
'application/vnd.atlas.2024-01-01+json',
'schema',
'properties',
'paused',
],
severity: DiagnosticSeverity.Error,
},
],
},
{
name: 'invalid optional boolean field without default - exception',
document: {
components: {
schemas: {
Schema: {
type: 'object',
properties: {
enabled: {
type: 'boolean',
'x-xgen-IPA-exception': {
'xgen-IPA-111-optional-boolean-fields-default-false': 'Reason',
},
},
},
},
},
},
},
errors: [],
},
]);
Loading
Loading