feat: classify path parameter name change as annotation

Author: b41ex

src/openapi/openapi3.classify.ts

@@ -73,7 +73,7 @@ export const parameterAllowReservedClassifyRule: ClassifyRule = [
 export const parameterNameClassifyRule: ClassifyRule = [
   nonBreaking,
   breaking,
-  ({ before }) => (getKeyValue(before.parent, 'in') === 'path' ? nonBreaking : breaking),
+  ({ before }) => (getKeyValue(before.parent, 'in') === 'path' ? annotation : breaking),
 ]
 
 export const parameterRequiredClassifyRule: ClassifyRule = [

test/openapi.diff.test.ts

@@ -193,13 +193,13 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.replace,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}', 'parameters', 0, 'name']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}', 'parameters', 1, 'name']],
-        type: nonBreaking,
+        type: annotation,
       }),
       expect.objectContaining({
         action: DiffAction.replace,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}', 'parameters', 1, 'name']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}', 'parameters', 0, 'name']],
-        type: nonBreaking,
+        type: annotation,
       }),
     ]))
   })
@@ -217,13 +217,13 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.replace,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}', 'get', 'parameters', 0, 'name']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}', 'get', 'parameters', 1, 'name']],
-        type: nonBreaking,
+        type: annotation,
       }),
       expect.objectContaining({
         action: DiffAction.replace,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}', 'get', 'parameters', 1, 'name']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}', 'get', 'parameters', 0, 'name']],
-        type: nonBreaking,
+        type: annotation,
       }),
     ]))
   })
@@ -258,7 +258,7 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.replace,
         beforeDeclarationPaths: [['paths', '/path1/{param1}', 'get', 'parameters', 0, 'name']],
         afterDeclarationPaths: [['paths', '/path1/{param2}', 'get', 'parameters', 0, 'name']],
-        type: nonBreaking,
+        type: annotation,
       }),
       expect.objectContaining({
         action: DiffAction.add,