* @param errorPoms the set of POMs that had errors
 */
public record UpgradeResult(Set<Path> processedPoms, Set<Path> modifiedPoms, Set<Path> errorPoms) {

    public UpgradeResult {
        // Defensive copying to ensure immutability
        processedPoms = Set.copyOf(processedPoms);
        modifiedPoms = Set.copyOf(modifiedPoms);
        errorPoms = Set.copyOf(errorPoms);
    }

    /**

            assertFalse(result.success(), "Orchestrator should fail when any strategy fails");
            assertEquals(1, result.errorPoms().size(), "Should have one error POM");
            assertTrue(result.errorPoms().contains(Paths.get("pom.xml")), "Should contain the failed POM");
        }

        @Test
        @DisplayName("should handle strategy exceptions gracefully")

        if (!result.errorPoms().isEmpty()) {
            context.failure("Strategy completed with errors");
            context.indent();
            context.info("Processed: " + result.processedPoms().size() + " POMs");
            context.info("Modified: " + result.modifiedPoms().size() + " POMs");
            context.failure("Errors: " + result.errorPoms().size() + " POMs");
            context.unindent();

                    errorPoms.add(pomPath);
                }
            } catch (Exception e) {
                context.failure("Model upgrade failed: " + e.getMessage());
                errorPoms.add(pomPath);
            } finally {
                context.unindent();
            }
        }

        return new UpgradeResult(processedPoms, modifiedPoms, errorPoms);
    }

    /**

            assertEquals(1, merged.modifiedPoms().size(), "Should only include successfully modified POMs");
            assertEquals(1, merged.errorPoms().size(), "Should include error POMs");
            assertTrue(merged.errorPoms().contains(pom2), "Should contain failed POM");
        }

        @Test
        @DisplayName("should handle merging with different POM sets")

            context.failure("Failed to create temp project structure: " + e.getMessage());
            // Mark all POMs as errors
            errorPoms.addAll(pomMap.keySet());
        }

        return new UpgradeResult(processedPoms, modifiedPoms, errorPoms);
    }

    /**
     * Upgrades plugins in the document.
     * Checks both build/plugins and build/pluginManagement/plugins sections.

                context.failure("Failed to fix Maven 4 compatibility issues" + ": " + e.getMessage());
                errorPoms.add(pomPath);
            } finally {
                context.unindent();
            }
        }

        return new UpgradeResult(processedPoms, modifiedPoms, errorPoms);
    }

    /**
     * Fixes unsupported combine.children attribute values.

                context.failure("Failed to apply inference optimizations" + ": " + e.getMessage());
                errorPoms.add(pomPath);
            } finally {
                context.unindent();
            }
        }

        return new UpgradeResult(processedPoms, modifiedPoms, errorPoms);
    }

    /**
     * Applies limited parent-related inference optimizations for Maven 4.0.0+ models.

            // This is needed for both 4.0.0 and 4.1.0 to help Maven find the project root
            createMvnDirectoryIfNeeded(context);

            return result.errorPoms().isEmpty() ? 0 : 1;
        } catch (Exception e) {
            context.failure("Strategy execution failed: " + e.getMessage());
            return 1;
        }
    }

    /**