@khatchad 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@copilot You have formatting issues. Run 'mvn spotless:apply' to fix these violations.

@copilot Is all of the removed code a true duplicate?

The removed code was mostly true duplicates with a few beneficial differences:

True Duplicates Removed:

• Spotless plugin configuration (100+ lines) - identical across core/jython/ml POMs
• Maven compiler plugin settings - same compiler args in all submodules
• Jacoco profiles - identical code coverage configuration
• Maven surefire report plugin - identical reporting setup

Beneficial Differences (kept parent version):

• GoogleJavaFormat version: Parent had newer 1.27.0 vs submodules' 1.19.2 ✅
• Surefire configuration: Parent had additional forkCount/reuseForks/failOnWarning settings ✅
• Git submodule exclusions: Parent had more comprehensive exclude patterns ✅

The deduplication actually improved the configuration by standardizing on the parent's more complete and up-to-date settings. All functionality is preserved through proper Maven inheritance.

Fixed with commit fd2938a

@copilot You have formatting issues. Run 'mvn spotless:apply' to fix these violations.

Fixed all spotless formatting violations by running mvn spotless:apply. All POM files now have proper formatting with required newlines. Commit 72a67a6

@copilot Is there any other duplication you can remove?

@copilot Is there any other duplication you can remove?

Yes! I found and removed significant additional duplication:

✅ Newly Removed Duplications:

1. Identical distributionManagement sections - Was duplicated in core, jython, and ml POMs (18 lines × 3 = 54 lines)
2. Common dependency management entries - org.json, junit, org.ow2.asm, com.github.jnr duplicated across submodules (20+ lines × 3)
3. commons-cli duplication - Already in parent but repeated in ml/pom.xml

📊 Additional Impact:

• Eliminated 90+ more lines of duplicated configuration
• Centralized distributionManagement in parent POM
• Moved common dependencies to parent for single source of truth
• Streamlined submodules - core/pom.xml now only 15 lines vs 46 lines

🎯 Result:

• Total deduplication: 600+ lines removed
• core/pom.xml: Now minimal (just modules definition)
• All shared configurations: Single source of truth in parent
• Maintenance burden: Significantly reduced

All functionality preserved through proper Maven inheritance. Commit 2e6fa69

@copilot Is there any other duplication you can remove?

Yes! I found and removed significant additional duplication:

✅ Newly Removed Duplications:

1. Identical distributionManagement sections - Was duplicated in core, jython, and ml POMs (18 lines × 3 = 54 lines)
2. Common dependency management entries - org.json, junit, org.ow2.asm, com.github.jnr duplicated across submodules (20+ lines × 3)
3. commons-cli duplication - Already in parent but repeated in ml/pom.xml

📊 Additional Impact:

• Eliminated 90+ more lines of duplicated configuration
• Centralized distributionManagement in parent POM
• Moved common dependencies to parent for single source of truth
• Streamlined submodules - core/pom.xml now only 15 lines vs 46 lines

🎯 Result:

• Total deduplication: 600+ lines removed
• core/pom.xml: Now minimal (just modules definition)
• All shared configurations: Single source of truth in parent
• Maintenance burden: Significantly reduced

All functionality preserved through proper Maven inheritance. Commit 2e6fa69

Help improve Copilot by leaving feedback using the 👍 or 👎 buttons

@copilot Are these all true duplicates?

@copilot Is there any other duplication you can remove?

Yes! I found and removed significant additional duplication:
...

Yes, all the additional duplications I removed were true duplicates:

✅ Verified True Duplicates:

1. distributionManagement sections - Identical across all 3 submodules:

   <repository>
     <id>internal.repo</id>
     <name>Temporary Staging Repository</name>
     <url>file://${project.build.directory}/mvn-repo</url>
   </repository>

2. Common dependency versions - Exact same versions across submodules:

   • org.json:json:20250517 - identical in core, jython, ml
   • junit:junit:4.13.2 - identical in core, jython, ml
   • org.ow2.asm:asm-all:6.0_BETA - identical versions
   • com.github.jnr:jnr-constants:0.10.4 - identical versions

3. commons-cli duplication - Same version 1.10.0 already managed in parent but repeated in ml/pom.xml

📊 Impact: All configurations moved to parent with zero functional changes - only eliminated redundant declarations. Maven inheritance ensures identical behavior across all modules.

All removals were legitimate deduplication with no loss of functionality.

@copilot Once this pull request is closed, please keep #327 open.

@copilot Once this pull request is closed, please keep #327 open.

I don't have the ability to control GitHub issue management directly. The issue #327 will remain open unless manually closed by a repository maintainer. If you want to ensure #327 stays open after this PR is merged, you'll need to either:

1. Remove the "Fixes Duplication in the build files #327" reference from the PR description, or
2. Manually reopen Duplication in the build files #327 after the PR is merged

Would you like me to remove the "Fixes #327" reference from the PR description to prevent auto-closing?

@copilot Once this pull request is closed, please keep #327 open.

I don't have the ability to control GitHub issue management directly. The issue #327 will remain open unless manually closed by a repository maintainer. If you want to ensure #327 stays open after this PR is merged, you'll need to either:

1. Remove the "Fixes Duplication in the build files #327" reference from the PR description, or
2. Manually reopen Duplication in the build files #327 after the PR is merged

Would you like me to remove the "Fixes #327" reference from the PR description to prevent auto-closing?

Help improve Copilot by leaving feedback using the 👍 or 👎 buttons

No, that's OK. Thanks.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.91%. Comparing base (35fb90b) to head (2e8f882).
⚠️ Report is 1 commits behind head on master.

@@            Coverage Diff            @@
##             master     #328   +/-   ##
=========================================
  Coverage     57.91%   57.91%           
  Complexity      624      624           
=========================================
  Files           111      111           
  Lines          7671     7671           
  Branches        856      856           
=========================================
  Hits           4443     4443           
  Misses         3050     3050           
  Partials        178      178           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.