Merge branch 'main' into fix/bundler-sibling-refs

Author: vadyvas

.changeset/curly-tools-tap.md

@@ -0,0 +1,5 @@
+---
+"@redocly/openapi-core": patch
+---
+
+Fixed an issue where `no-required-schema-properties-undefined` rule did not evaluate properties in `any` and `oneOf`.

.changeset/proud-phones-sniff.md

@@ -0,0 +1,5 @@
+---
+"@redocly/cli": patch
+---
+
+Applied proxy settings during Respect execution.

.changeset/tender-pigs-brake.md

@@ -40,8 +40,8 @@ Can't resolve $ref: ENOENT: no such file or directory './__tests__/bundle/bundle
 26 |   test-wrong-ref:
 27 |     summary: Example shouldn't resolved to value
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-28 |     externalValue: './external-value-bad-path.json'
-   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+28 |     externalValue: ./external-value-bad-path.json
+   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 29 |
 
 Error was generated by the bundler rule.

__tests__/bundle/bundle-external-value/snapshot.txt

@@ -25,9 +25,9 @@ paths:
             examples:
               test-resolved:
                 summary: Example should resolved to value
-                externalValue: './external-value.json'
+                externalValue: ./external-value.json
               test-no-resolved:
                 summary: Example shouldn't be resolved to value
                 value:
                   type: object
-                externalValue: './external-value.json'
+                externalValue: ./external-value.json

__tests__/bundle/bundle-external-value/test-success.yaml

@@ -25,4 +25,4 @@ paths:
             examples:
               test-wrong-ref:
                 summary: Example shouldn't resolved to value
-                externalValue: './external-value-bad-path.json'
+                externalValue: ./external-value-bad-path.json

__tests__/bundle/bundle-external-value/test-wrong-examples.yaml

@@ -1,7 +1,7 @@
 import { readFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { spawnSync } from 'node:child_process';
-import { parseYaml } from '../packages/core/src/utils.js'; // not able to import from @redocly/openapi-core
+import { parseYaml } from '../packages/core/src/utils/yaml-fs-helper.js'; // not able to import from @redocly/openapi-core
 
 export function getParams(indexEntryPoint: string, args: string[] = []): string[] {
   return [indexEntryPoint, ...args];

__tests__/helpers.ts

@@ -26,7 +26,49 @@ Property `wrong` is not expected here.
 Error was generated by the struct rule.
 
 
-[3] openapi.yaml:51:7 at #/paths/~1products/query/wrong
+[3] openapi.yaml:16:13 at #/tags/0/parent
+
+Tag parent 'products' is not defined in the API description.
+
+14 | summary: Books & Literature
+15 | description: Book catalog and recommendations
+16 | parent: products
+   |         ^^^^^^^^
+17 | kind: nav
+18 | wrong: Should catch wrong fields in tags!
+
+Error was generated by the spec-no-invalid-tag-parents rule.
+
+
+[4] openapi.yaml:23:13 at #/tags/1/parent
+
+Tag parent 'products' is not defined in the API description.
+
+21 | summary: Music CDs
+22 | description: Music CD catalog and reviews
+23 | parent: products
+   |         ^^^^^^^^
+24 | kind: nav
+25 |
+
+Error was generated by the spec-no-invalid-tag-parents rule.
+
+
+[5] openapi.yaml:29:13 at #/tags/2/parent
+
+Tag parent 'products' is not defined in the API description.
+
+27 | summary: Gift Cards
+28 | description: Digital and physical gift cards
+29 | parent: products
+   |         ^^^^^^^^
+30 | kind: nav
+31 |
+
+Error was generated by the spec-no-invalid-tag-parents rule.
+
+
+[6] openapi.yaml:51:7 at #/paths/~1products/query/wrong
 
 Property `wrong` is not expected here.
 
@@ -40,7 +82,7 @@ Property `wrong` is not expected here.
 Error was generated by the struct rule.
 
 
-[4] openapi.yaml:48:9 at #/paths/~1products/additionalOperations/test/wrong
+[7] openapi.yaml:48:9 at #/paths/~1products/additionalOperations/test/wrong
 
 Property `wrong` is not expected here.
 
@@ -54,7 +96,7 @@ Property `wrong` is not expected here.
 Error was generated by the struct rule.
 
 
-[5] openapi.yaml:156:9 at #/components/schemas/MyResponseType/discriminator/wrong
+[8] openapi.yaml:156:9 at #/components/schemas/MyResponseType/discriminator/wrong
 
 Property `wrong` is not expected here.
 
@@ -68,7 +110,7 @@ Property `wrong` is not expected here.
 Error was generated by the struct rule.
 
 
-[6] openapi.yaml:132:11 at #/components/securitySchemes/petstore_auth/flows/implicit/wrong
+[9] openapi.yaml:132:11 at #/components/securitySchemes/petstore_auth/flows/implicit/wrong
 
 Property `wrong` is not expected here.
 
@@ -82,7 +124,7 @@ Property `wrong` is not expected here.
 Error was generated by the struct rule.
 
 
-[7] openapi.yaml:144:9 at #/components/securitySchemes/device_auth/flows/deviceAuthorization
+[10] openapi.yaml:144:9 at #/components/securitySchemes/device_auth/flows/deviceAuthorization
 
 The field `scopes` must be present on this level.
 
@@ -96,7 +138,7 @@ The field `scopes` must be present on this level.
 Error was generated by the struct rule.
 
 
-[8] openapi.yaml:2:1 at #/info
+[11] openapi.yaml:2:1 at #/info
 
 Info object should contain `license` field.
 
@@ -109,7 +151,7 @@ Info object should contain `license` field.
 Warning was generated by the info-license rule.
 
 
-[9] openapi.yaml:9:10 at #/servers/0/url
+[12] openapi.yaml:9:10 at #/servers/0/url
 
 Server `url` should not point to example.com or localhost.
 
@@ -123,7 +165,7 @@ Server `url` should not point to example.com or localhost.
 Warning was generated by the no-server-example.com rule.
 
 
-[10] openapi.yaml:39:5 at #/paths/~1giftcards/get/operationId
+[13] openapi.yaml:39:5 at #/paths/~1giftcards/get/operationId
 
 Operation object should contain `operationId` field.
 
@@ -137,7 +179,7 @@ Operation object should contain `operationId` field.
 Warning was generated by the operation-operationId rule.
 
 
-[11] openapi.yaml:49:5 at #/paths/~1products/query/operationId
+[14] openapi.yaml:49:5 at #/paths/~1products/query/operationId
 
 Operation object should contain `operationId` field.
 
@@ -151,7 +193,7 @@ Operation object should contain `operationId` field.
 Warning was generated by the operation-operationId rule.
 
 
-[12] openapi.yaml:65:7 at #/paths/~1products/query/responses
+[15] openapi.yaml:65:7 at #/paths/~1products/query/responses
 
 Operation must have at least one `4XX` response.
 
@@ -165,7 +207,7 @@ Operation must have at least one `4XX` response.
 Warning was generated by the operation-4xx-response rule.
 
 
-[13] openapi.yaml:46:7 at #/paths/~1products/additionalOperations/test/operationId
+[16] openapi.yaml:46:7 at #/paths/~1products/additionalOperations/test/operationId
 
 Operation object should contain `operationId` field.
 
@@ -179,7 +221,7 @@ Operation object should contain `operationId` field.
 Warning was generated by the operation-operationId rule.
 
 
-[14] openapi.yaml:70:5 at #/paths/~1stream/get/operationId
+[17] openapi.yaml:70:5 at #/paths/~1stream/get/operationId
 
 Operation object should contain `operationId` field.
 
@@ -193,7 +235,7 @@ Operation object should contain `operationId` field.
 Warning was generated by the operation-operationId rule.
 
 
-[15] openapi.yaml:72:7 at #/paths/~1stream/get/responses
+[18] openapi.yaml:72:7 at #/paths/~1stream/get/responses
 
 Operation must have at least one `4XX` response.
 
@@ -207,7 +249,7 @@ Operation must have at least one `4XX` response.
 Warning was generated by the operation-4xx-response rule.
 
 
-[16] openapi.yaml:83:5 at #/paths/~1thing/get/operationId
+[19] openapi.yaml:83:5 at #/paths/~1thing/get/operationId
 
 Operation object should contain `operationId` field.
 
@@ -221,7 +263,7 @@ Operation object should contain `operationId` field.
 Warning was generated by the operation-operationId rule.
 
 
-[17] openapi.yaml:85:7 at #/paths/~1thing/get/responses
+[20] openapi.yaml:85:7 at #/paths/~1thing/get/responses
 
 Operation must have at least one `4XX` response.
 
@@ -235,7 +277,7 @@ Operation must have at least one `4XX` response.
 Warning was generated by the operation-4xx-response rule.
 
 
-[18] openapi.yaml:149:5 at #/components/schemas/MyResponseType
+[21] openapi.yaml:149:5 at #/components/schemas/MyResponseType
 
 Component: "MyResponseType" is never used.
 
@@ -253,6 +295,6 @@ Warning was generated by the no-unused-components rule.
 validating openapi.yaml using lint rules for api 'main'...
 openapi.yaml: validated in <test>ms
 
-❌ Validation failed with 7 errors and 11 warnings.
+❌ Validation failed with 10 errors and 11 warnings.
 run `redocly lint --generate-ignore-file` to add all problems to the ignore file.
 

__tests__/lint/oas3.2/snapshot.txt

@@ -20,8 +20,19 @@ describe('local-json-server', () => {
   });
 
   afterAll(() => {
-    // Kill the process group to ensure child processes are cleaned up
-    process.kill(-serverProcess.pid);
+    try {
+      if (process.platform === 'win32') {
+        spawn('taskkill', ['/pid', serverProcess.pid.toString(), '/f', '/t']);
+      } else {
+        spawn('kill', ['-TERM', `-${serverProcess.pid}`]);
+      }
+    } catch (error) {
+      try {
+        serverProcess.kill('SIGTERM');
+      } catch (e) {
+        // Process may have already exited
+      }
+    }
 
     // Restore original state
     if (originalData) {

__tests__/respect/local-json-server/local-json-server.test.ts

@@ -15,14 +15,14 @@ Enforces the inclusion of `style` and `explode` fields for parameters with array
 ```mermaid
 flowchart TD
 
-root ==> Paths --> PathItem --> Operation --> Parameter --enforces style and explode fields for array types--> Schema
+Root ==> Paths --> PathItem --> Operation --> Parameter --enforces style and explode fields for array types--> Schema
 PathItem --> Parameter
-NamedParameter --> Parameter
+NamedParameters --> Parameter
 
-root ==> components
+Root ==> components
 
 subgraph components
-NamedParameter
+NamedParameters
 end
 
 style Parameter fill:#codaf9,stroke:#0044d4,stroke-width:5px