fix: revert unintended changes

This reverts commit 6f58225.

Author: b41ex

src/components/compare/compare.operations.ts

@@ -38,7 +38,6 @@ import {
   OperationIdentityMap,
   takeSubstringIf,
   totalChanges,
-  getNormalizedOperationId,
 } from './compare.utils'
 import {
   calculateApiAudienceTransitions,
@@ -350,6 +349,7 @@ const HANDLE_TYPE_FULLY_REMOVED = 'removed'
 type HandleType = typeof HANDLE_TYPE_FULLY_ADDED | typeof HANDLE_TYPE_FULLY_REMOVED
 
 const createPairOperationsMap = (currGroupSlug: string, prevGroupSlug: string, currentOperations: ResolvedOperation[], previousOperations: ResolvedOperation[]): Record<string, { previous?: ResolvedOperation; current: ResolvedOperation }> => {
+
   const operationsMap: Record<string, { previous?: ResolvedOperation; current: ResolvedOperation }> = {}
 
   for (const currentOperation of currentOperations) {

src/components/compare/compare.utils.ts

@@ -29,7 +29,6 @@ import {
 } from '../../types'
 import { BUILD_TYPE, EMPTY_CHANGE_SUMMARY, MESSAGE_SEVERITY } from '../../consts'
 import { convertToSlug, takeIfDefined } from '../../utils'
-import { normalizePath } from '../../utils/builder'
 
 export const totalChanges = (changeSummary?: ChangeSummary): number => {
   return changeSummary
@@ -97,50 +96,31 @@ export function getOperationsHashMapByApiType(
   const resolvedHashMap = { ...operationTypes.find(({ apiType: type }) => type === currentApiType)?.operations || {} }
   const { buildType, currentGroup, previousGroup } = ctx.config
 
-  // Handle prefix group changelog case
-  if (buildType === BUILD_TYPE.PREFIX_GROUPS_CHANGELOG) {
-    if (!currentGroup || !previousGroup) {
-      ctx.notifications.push({
-        severity: MESSAGE_SEVERITY.Warning,
-        message: `Build type is prefix group changelog, but one of the groups is not provided: currentGroup=${currentGroup}, previousGroup=${previousGroup}`,
-      })
-      return resolvedHashMap
-    }
-
-    const groupSlug = convertToSlug(areOperationsFromCurrentVersion ? currentGroup : previousGroup)
-    const newHashMap: ResolvedVersionOperationsHashMap = {}
-
-    // Process each operation
-    for (const [operationId, dataHash] of Object.entries(resolvedHashMap)) {
-      if (operationId.startsWith(groupSlug)) {
-        const changedOperationId = operationId.substring(groupSlug.length)
-        newHashMap[changedOperationId] = dataHash
-        operationIdentityMap[changedOperationId] = operationId
-      }
-    }
+  if (buildType !== BUILD_TYPE.PREFIX_GROUPS_CHANGELOG) {
+    return resolvedHashMap
+  }
 
-    return newHashMap
+  if (!currentGroup || !previousGroup) {
+    ctx.notifications.push({
+      severity: MESSAGE_SEVERITY.Warning,
+      message: `Build type is prefix group changelog, but one of the groups is not provided: currentGroup=${currentGroup}, previousGroup=${previousGroup}`,
+    })
+    return resolvedHashMap
   }
 
-  // Handle path parameter normalization case
-  const newHashMap: ResolvedVersionOperationsHashMap = {}
   for (const [operationId, dataHash] of Object.entries(resolvedHashMap)) {
-    // Get operation metadata to normalize the path
-    const operation = operationTypes.find(({ apiType: type }) => type === currentApiType)
-      ?.operations_metadata?.[operationId]
-    
-    if (operation?.path && operation?.method) {
-      const normalizedPath = normalizePath(operation.path)
-      const normalizedId = `${operation.method.toLowerCase()}-${normalizedPath}`
-      newHashMap[normalizedId] = dataHash
-      operationIdentityMap[normalizedId] = operationId
-    } else {
-      newHashMap[operationId] = dataHash
-      operationIdentityMap[operationId] = operationId
+    Reflect.deleteProperty(resolvedHashMap, operationId)
+
+    const groupSlug = convertToSlug(areOperationsFromCurrentVersion ? currentGroup : previousGroup)
+
+    if (operationId.startsWith(groupSlug)) {
+      const changedOperationId = operationId.substring(groupSlug.length)
+      resolvedHashMap[changedOperationId] = dataHash
+      operationIdentityMap[changedOperationId] = operationId
     }
   }
 
-  return newHashMap
+  return resolvedHashMap
 }
 
 export function getOperationMetadata(operation: ResolvedOperation): OperationChangesMetadata {
@@ -162,26 +142,3 @@ export function takeSubstringIf(condition: boolean, value: string, startIndex: n
 
   return value.substring(startIndex)
 }
-
-/**
- * Normalizes a path by replacing path parameters with a placeholder
- * regardless of the parameter name
- * e.g. /users/{userId}/posts/{postId} -> /users/{param}/posts/{param}
- */
-export function normalizePath(path: string): string {
-  return path.replace(/\{[^}]+\}/g, '{param}')
-}
-
-/**
- * Gets a normalized operation identifier that is resilient to path parameter renaming
- */
-export function getNormalizedOperationId(operationId: string, metadata?: { path?: string, method?: string }): string {
-  // If no path/method metadata, return original ID
-  if (!metadata?.path || !metadata?.method) {
-    return operationId
-  }
-
-  // Create normalized ID from method and normalized path using existing normalizePath utility
-  const normalizedPath = normalizePath(metadata.path)
-  return `${metadata.method.toLowerCase()}-${normalizedPath}`
-}