Skip to content

[MutabilityWithReadOnly] Optimize to prevent excessive memory usage on large specs #813

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
AkhilaIlla merged 16 commits into from
Jan 30, 2026
Merged

[MutabilityWithReadOnly] Optimize to prevent excessive memory usage on large specs #813

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
98df3f9
Initial plan
Copilot Jan 28, 2026
6891fac
Fix MutabilityWithReadOnly rule to avoid out of memory issue by filte…
Copilot Jan 28, 2026
93c4e14
Remove redundant checks from mutability-with-read-only function
Copilot Jan 28, 2026
af3e536
Add Array.isArray check for x-ms-mutability defensive programming
Copilot Jan 28, 2026
7142114
Move length check to given clause for maximum efficiency
Copilot Jan 28, 2026
b71fdb6
Remove Array.isArray check that changed behavior - keep performance o…
Copilot Jan 29, 2026
aaf7003
Simplify to minimal change: only filter readOnly and x-ms-mutability …
Copilot Jan 29, 2026
ab900ea
Add comprehensive tests for all readOnly and x-ms-mutability combinat…
Copilot Jan 29, 2026
410cf09
Add tests for properties with omitted readOnly or x-ms-mutability fields
Copilot Jan 29, 2026
7d055aa
Refactor tests to reduce redundancy using helper function and consoli…
Copilot Jan 29, 2026
a61fd2f
Change helper function type from any to unknown for better type safety
Copilot Jan 29, 2026
3d9eca9
improve comment
mikeharder Jan 29, 2026
7b0dfe4
Add Rush change file for MutabilityWithReadOnly performance fix
Copilot Jan 29, 2026
7d9c1ff
Update package.json to 2.2.1 and add CHANGELOG entry
Copilot Jan 29, 2026
573ada9
improve changelog
mikeharder Jan 29, 2026
c2ad655
Apply suggestion from @mikeharder
mikeharder Jan 29, 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
6 changes: 2 additions & 4 deletions packages/rulesets/generated/spectral/az-arm.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,9 +144,7 @@ const mutabilityWithReadOnly = (prop, _opts, ctx) => {
if (prop === null || typeof prop !== "object") {
return [];
}
if (prop.readOnly === undefined ||
prop["x-ms-mutability"] === undefined ||
prop["x-ms-mutability"].length === 0) {
if (!Array.isArray(prop["x-ms-mutability"])) {
return [];
}
const path = ctx.path || [];
Expand Down Expand Up @@ -1067,7 +1065,7 @@ const ruleset$1 = {
severity: "error",
resolved: true,
formats: [oas2],
given: ["$[paths,'x-ms-paths']..?(@property === 'readOnly')^"],
given: ["$[paths,'x-ms-paths']..*[?(@ != null && @.readOnly !== undefined && @['x-ms-mutability'] !== undefined && @['x-ms-mutability'].length > 0)]"],
then: {
function: mutabilityWithReadOnly,
},
Expand Down
6 changes: 2 additions & 4 deletions packages/rulesets/generated/spectral/az-common.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,9 +141,7 @@ const mutabilityWithReadOnly = (prop, _opts, ctx) => {
if (prop === null || typeof prop !== "object") {
return [];
}
if (prop.readOnly === undefined ||
prop["x-ms-mutability"] === undefined ||
prop["x-ms-mutability"].length === 0) {
if (!Array.isArray(prop["x-ms-mutability"])) {
return [];
}
const path = ctx.path || [];
Expand Down Expand Up @@ -821,7 +819,7 @@ const ruleset = {
severity: "error",
resolved: true,
formats: [oas2],
given: ["$[paths,'x-ms-paths']..?(@property === 'readOnly')^"],
given: ["$[paths,'x-ms-paths']..*[?(@ != null && @.readOnly !== undefined && @['x-ms-mutability'] !== undefined && @['x-ms-mutability'].length > 0)]"],
then: {
function: mutabilityWithReadOnly,
},
Expand Down
6 changes: 2 additions & 4 deletions packages/rulesets/generated/spectral/az-dataplane.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,9 +141,7 @@ const mutabilityWithReadOnly = (prop, _opts, ctx) => {
if (prop === null || typeof prop !== "object") {
return [];
}
if (prop.readOnly === undefined ||
prop["x-ms-mutability"] === undefined ||
prop["x-ms-mutability"].length === 0) {
if (!Array.isArray(prop["x-ms-mutability"])) {
return [];
}
const path = ctx.path || [];
Expand Down Expand Up @@ -851,7 +849,7 @@ const ruleset$1 = {
severity: "error",
resolved: true,
formats: [oas2],
given: ["$[paths,'x-ms-paths']..?(@property === 'readOnly')^"],
given: ["$[paths,'x-ms-paths']..*[?(@ != null && @.readOnly !== undefined && @['x-ms-mutability'] !== undefined && @['x-ms-mutability'].length > 0)]"],
then: {
function: mutabilityWithReadOnly,
},
Expand Down
2 changes: 1 addition & 1 deletion packages/rulesets/src/spectral/az-common.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,7 @@ const ruleset: any = {
severity: "error",
resolved: true,
formats: [oas2],
given: ["$[paths,'x-ms-paths']..?(@property === 'readOnly')^"],
given: ["$[paths,'x-ms-paths']..*[?(@ != null && @.readOnly !== undefined && @['x-ms-mutability'] !== undefined && @['x-ms-mutability'].length > 0)]"],
Comment thread
mikeharder marked this conversation as resolved.
Outdated
then: {
function: mutabilityWithReadOnly,
},
Expand Down
11 changes: 6 additions & 5 deletions packages/rulesets/src/spectral/functions/Extensions/mutability-with-read-only.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,12 @@ export const mutabilityWithReadOnly = (prop: any, _opts: any, ctx: any) => {
if (prop === null || typeof prop !== "object") {
return [];
}
if (
prop.readOnly === undefined ||
prop["x-ms-mutability"] === undefined ||
prop["x-ms-mutability"].length === 0
) {
// The given clause filters for:
// - readOnly !== undefined
// - x-ms-mutability !== undefined
// - x-ms-mutability.length > 0
// We still check Array.isArray as a defensive measure for type safety
if (!Array.isArray(prop["x-ms-mutability"])) {
Comment thread
mikeharder marked this conversation as resolved.
Outdated
return [];
}
const path = ctx.path || [];
Expand Down
74 changes: 74 additions & 0 deletions packages/rulesets/src/spectral/test/mutability-with-read-only.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,3 +93,77 @@ test("MutabilityWithReadOnly should find no errors", () => {
expect(results.length).toBe(0);
});
});

test("MutabilityWithReadOnly should ignore empty x-ms-mutability arrays", () => {
Comment thread
mikeharder marked this conversation as resolved.
Outdated
const myOpenApiDocument = {
swagger: "2.0",
paths: {
"/api/Paths": {
put: {
operationId: "Path_Create",
responses: {
200: {
description: "Success",
schema: {
$ref: "#/definitions/LroStatusCodeSchema",
},
},
},
},
},
},
definitions: {
LroStatusCodeSchema: {
type: "object",
properties: {
name: {
type: "string",
readOnly: true,
"x-ms-mutability": [],
},
},
},
},
};
return linter.run(myOpenApiDocument).then((results) => {
// Empty x-ms-mutability arrays should be ignored (no errors)
expect(results.length).toBe(0);
});
});

test("MutabilityWithReadOnly should ignore non-array x-ms-mutability values", () => {
Comment thread
mikeharder marked this conversation as resolved.
Outdated
const myOpenApiDocument = {
swagger: "2.0",
paths: {
"/api/Paths": {
put: {
operationId: "Path_Create",
responses: {
200: {
description: "Success",
schema: {
$ref: "#/definitions/LroStatusCodeSchema",
},
},
},
},
},
},
definitions: {
LroStatusCodeSchema: {
type: "object",
properties: {
name: {
type: "string",
readOnly: true,
"x-ms-mutability": "read",
},
},
},
},
};
return linter.run(myOpenApiDocument).then((results) => {
// Non-array x-ms-mutability values should be ignored (no errors)
expect(results.length).toBe(0);
});
});
Loading
⚡