feat: path change due to renaming path parameters classified as annotation

Author: b41ex

src/openapi/openapi3.classify.ts

@@ -14,6 +14,7 @@ import { getKeyValue, isExist, isNotEmptyArray } from '../utils'
 import { emptySecurity, getDefaultStyle, includeSecurity } from './openapi3.utils'
 import type { ClassifyRule, CompareContext } from '../types'
 import { DiffType } from '../types'
+import { hidePathParamNames } from './openapi3.mapping'
 
 export const paramClassifyRule: ClassifyRule = [
   ({ after }) => {
@@ -141,3 +142,17 @@ export const operationSecurityItemClassifyRule: ClassifyRule = [
     after,
   }) => (includeSecurity(after.parent, before.parent) || emptySecurity(after.value) ? nonBreaking : breaking),
 ]
+
+export const pathChangeClassifyRule: ClassifyRule = [
+  nonBreaking,
+  breaking,
+  ({ before, after }) => {
+    const beforePath = before.key as string
+    const afterPath = after.key as string
+    const unifiedBeforePath = hidePathParamNames(beforePath)
+    const unifiedAfterPath = hidePathParamNames(afterPath)
+    
+    // If unified paths are the same, it means only parameter names changed
+    return unifiedBeforePath === unifiedAfterPath ? annotation : breaking
+  }
+]

src/openapi/openapi3.mapping.ts

@@ -124,7 +124,7 @@ export const contentMediaTypeMappingResolver: MappingResolver<string> = (before,
   return result
 }
 
-function hidePathParamNames(path: string): string {
+export function hidePathParamNames(path: string): string {
   return path.replace(PATH_PARAMETER_REGEXP, PATH_PARAM_UNIFIED_PLACEHOLDER)
 }
 

src/openapi/openapi3.rules.ts

@@ -51,6 +51,7 @@ import {
   parameterNameClassifyRule,
   parameterRequiredClassifyRule,
   parameterStyleClassifyRule,
+  pathChangeClassifyRule,
 } from './openapi3.classify'
 import {
   contentMediaTypeMappingResolver,
@@ -341,7 +342,7 @@ export const openApi3Rules = (options: OpenApi3RulesOptions): CompareRules => {
       $: allUnclassified,
       mapping: options.mode === COMPARE_MODE_OPERATION ? singleOperationPathMappingResolver : pathMappingResolver,
       '/*': {
-        $: [nonBreaking, breaking, breaking],
+        $: pathChangeClassifyRule,
         mapping: options.mode === COMPARE_MODE_OPERATION ? singleOperationPathMappingResolver : pathMappingResolver,
         '/summary': { $: allAnnotation },
         '/description': { $: allAnnotation },

test/openapi.diff.test.ts

@@ -7,6 +7,7 @@ import {
   DIFF_META_KEY,
   DiffAction,
   nonBreaking,
+  annotation,
 } from '../src'
 
 import { OpenapiBuilder, TEST_DIFF_FLAG, TEST_ORIGINS_FLAG, TEST_SYNTHETIC_TITLE_FLAG } from './helper'
@@ -186,8 +187,7 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.rename,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}']],
-        // todo fix
-        type: breaking,
+        type: annotation, // Only parameter names changed, unified paths are the same
       }),
       expect.objectContaining({
         action: DiffAction.replace,
@@ -211,8 +211,7 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.rename,
         beforeDeclarationPaths: [['paths', '/path1/{param1}/{anotherParam1}']],
         afterDeclarationPaths: [['paths', '/path1/{param2}/{anotherParam2}']],
-        // todo fix
-        type: breaking,
+        type: annotation, // Only parameter names changed, unified paths are the same
       }),
       expect.objectContaining({
         action: DiffAction.replace,
@@ -253,8 +252,7 @@ describe('Openapi3 operation changes', () => {
         action: DiffAction.rename,
         beforeDeclarationPaths: [['paths', '/path1/{param1}']],
         afterDeclarationPaths: [['paths', '/path1/{param2}']],
-        // todo fix
-        type: breaking,
+        type: annotation, // Only parameter name changed, unified paths are the same
       }),
       expect.objectContaining({
         action: DiffAction.replace,