Skip to content

Restructure to multi-module Gradle project for multi-loader support#27

Draft
Copilot wants to merge 3 commits intomasterfrom
copilot/common-code-for-modloaders
Draft

Restructure to multi-module Gradle project for multi-loader support#27
Copilot wants to merge 3 commits intomasterfrom
copilot/common-code-for-modloaders

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 3, 2026

  • Phase 1: Scaffold multi-module Gradle structure (buildSrc, root build.gradle, settings.gradle, gradle.properties)
  • Phase 2: Create common module (build.gradle + move common-eligible sources and resources)
  • Phase 3: Create platform abstraction layer (NicephoreConfigProvider interface, Services object)
  • Phase 4: Create NeoForge module (build.gradle + move NeoForge-specific sources, NeoForgeConfigProvider, NicephoreNeoForge entry point)
  • Phase 5: Update common source files to use Services.config instead of NicephoreConfig.Client
  • Phase 6: Verify structure integrity (buildSrc compiles, no stale NeoForge imports in common, all references consistent)
  • Phase 7: Run validation (code review + CodeQL passed)

Note: Full Gradle build could not be verified in sandbox due to network restrictions (maven.neoforged.net unreachable). BuildSrc compiles successfully. All source-level references verified clean. Please run ./gradlew :neoforge:build locally to confirm.

Copilot AI and others added 3 commits April 3, 2026 14:27
…ith build files

Agent-Logs-Url: https://github.com/LotuxPunk/Nicephore/sessions/c4e32123-ecc6-47b1-b9e9-664b878580e3

Co-authored-by: LotuxPunk <16562647+LotuxPunk@users.noreply.github.com>
… of NicephoreConfig.Client

Agent-Logs-Url: https://github.com/LotuxPunk/Nicephore/sessions/c4e32123-ecc6-47b1-b9e9-664b878580e3

Co-authored-by: LotuxPunk <16562647+LotuxPunk@users.noreply.github.com>
… buildSrc), add idea plugin to neoforge

Agent-Logs-Url: https://github.com/LotuxPunk/Nicephore/sessions/c4e32123-ecc6-47b1-b9e9-664b878580e3

Co-authored-by: LotuxPunk <16562647+LotuxPunk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants