Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

libs/converter-openapi/src/index.ts (1)

187-199: 💤 Low value

Consider using forEach instead of map for mutation-only side effects.

The outer services.map() creates and discards an array since only the in-place mutations matter. Using forEach would be more semantically appropriate for side-effect-only iteration.

♻️ Suggested refactor

          tap((services) =>
-           services.map((service) => {
+           services.forEach((service) => {
              if (!service.path_prefix) return service;
              service.routes = service.routes!.map((route: ADCSDK.Route) => {
                route.uris = route.uris.map(
                  (uri: string) => `${service.path_prefix}${uri}`,
                );
                return route;
              });
              unset(service, 'path_prefix');
-             return service;
            }),
          ),

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@libs/converter-openapi/src/index.ts` around lines 187 - 199, Replace the
outer services.map() with services.forEach() since the callback only performs
mutations (modifying route uris and unsetting path_prefix) and doesn't need to
return the array. Update the conditional return statement from return service to
just return when path_prefix doesn't exist, and remove the final return service
statement at the end of the forEach callback since forEach doesn't use return
values for iteration.