feat: foundational SmartBrainLib architecture for HumanoidNPC

[Copilot]

Implements the four-component SBL brain architecture for a state-driven autonomous NPC that uses custom memory, sensors, and contextual behaviors rather than reactive goal AI.

New Components

• ModMemoryTypes — DeferredRegister<MemoryModuleType<?>> with three slots: BASE_LOCATION (GlobalPos), MACRO_OBJECTIVE (String), NEEDED_MATERIALS (List<String>)
• InventoryStateSensor (ExtendedSensor<HumanoidNPC>) — polls main-hand, offhand, and carried inventory every 10 ticks; flags empty slots and tools below 25% durability; writes to NEEDED_MATERIALS
• ContextualToolSwapBehavior (ExtendedBehaviour<HumanoidNPC>) — requires MACRO_OBJECTIVE; maps objective keywords → tool class (wood→Axe, stone→Pickaxe, combat→Sword, farm→Hoe, dig→Shovel); swaps first matching tool from carried inventory into main hand; no-ops when already optimal
• HumanoidNPC (PathfinderMob + SmartBrainOwner<HumanoidNPC>) — declares all three memory types in brainProvider(); CORE (placeholder), IDLE (placeholder), and WORK activities with ContextualToolSwapBehavior; sensors polled before brain tick in customServerAiStep()

// Brain wiring: sensors run before brain tick to keep memories fresh
@Override
protected void customServerAiStep() {
    super.customServerAiStep();
    if (level() instanceof ServerLevel sl) {
        for (ExtendedSensor<? super HumanoidNPC> sensor : sensorCache) {
            sensor.tick(sl, this);   // writes NEEDED_MATERIALS
        }
        getBrain().tick(sl, this);   // ContextualToolSwapBehavior reads MACRO_OBJECTIVE
    }
}

Integration

• MillEntities — registers HUMANOID_NPC entity type
• Millenaire2 — wires ModMemoryTypes.MEMORY_MODULE_TYPES register on the mod event bus; registers HumanoidNPC attributes in EntityAttributeCreationEvent

Tests

HumanoidNpcTests — 12 @GameTest methods covering: brain configuration on spawn, all three memory types accessible, default WORK activity, sensor writes (empty hands, damaged tool detection), tool-swap correctness per objective keyword, no-op when already optimal, no-op when no matching tool, and base-location memory helpers.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://launchermeta.mojang.com/mc/game/version_manifest_v2.json
  • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java -Dstderr.encoding=UTF-8 -Dstdout.encoding=UTF-8 -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -jar /home/REDACTED/.gradle/caches/modules-2/files-2.1/net.neoforged/neoform-runtime/2.0.18/af5a11e64f209ac5909434ac2617b6af19a02a8d/neoform-runtime-2.0.18-all.jar --home-dir /home/REDACTED/.gradle/caches/neoformruntime --work-dir /home/REDACTED/work/Millenaire_port/Millenaire_port/build/tmp/neoformruntime run --java-executable /usr/lib/jvm/temurin-21-jdk-amd64/bin/java --parchment-data /home/REDACTED/.gradle/caches/modules-2/files-2.1/org.parchmentmc.data/parchment-1.21.3/2024.12.07/b53d6f313ce9f126e3e9ec9419e383aa3a365f46/parchment-1.21.3-2024.12.07.zip --parchment-conflict-prefix p_ (http block)
• launchermeta.mojang.com
  • Triggering command: /usr/lib/jvm/temurin-21-jdk-amd64/bin/java /usr/lib/jvm/temurin-21-jdk-amd64/bin/java -Dstderr.encoding=UTF-8 -Dstdout.encoding=UTF-8 -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -jar /home/REDACTED/.gradle/caches/modules-2/files-2.1/net.neoforged/neoform-runtime/2.0.18/af5a11e64f209ac5909434ac2617b6af19a02a8d/neoform-runtime-2.0.18-all.jar --home-dir /home/REDACTED/.gradle/caches/neoformruntime --work-dir /home/REDACTED/work/Millenaire_port/Millenaire_port/build/tmp/neoformruntime run --java-executable /usr/lib/jvm/temurin-21-jdk-amd64/bin/java --parchment-data /home/REDACTED/.gradle/caches/modules-2/files-2.1/org.parchmentmc.data/parchment-1.21.3/2024.12.07/b53d6f313ce9f126e3e9ec9419e383aa3a365f46/parchment-1.21.3-2024.12.07.zip --parchment-conflict-prefix p_ (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

Adds the initial SmartBrainLib-style (stubbed) brain architecture for a new HumanoidNPC entity, including custom memory modules, a sensor that writes inventory-derived needs into memory, and a contextual tool-swap behaviour wired into the entity’s WORK activity.

Changes:

• Introduces HumanoidNPC with manual sensor ticking + Brain ticking, plus a small carried-inventory abstraction and base-location memory helpers.
• Adds ModMemoryTypes, InventoryStateSensor, and ContextualToolSwapBehavior to support memory-driven tool equipping.
• Registers the new entity + memory-module DeferredRegister and adds a comprehensive GameTest suite for the new brain components.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/main/java/org/dizzymii/millenaire2/gametest/HumanoidNpcTests.java	Adds GameTests validating the new HumanoidNPC brain wiring, sensor output, tool-swap behavior, and base-location memory helper.
src/main/java/org/dizzymii/millenaire2/entity/brain/sensor/InventoryStateSensor.java	New sensor that inspects hands + carried inventory and writes NEEDED_MATERIALS memory.
src/main/java/org/dizzymii/millenaire2/entity/brain/behaviour/ContextualToolSwapBehavior.java	New behaviour that selects a tool based on MACRO_OBJECTIVE and equips it in main hand.
src/main/java/org/dizzymii/millenaire2/entity/brain/ModMemoryTypes.java	Registers custom Brain MemoryModuleTypes used by HumanoidNPC.
src/main/java/org/dizzymii/millenaire2/entity/MillEntities.java	Registers the HUMANOID_NPC entity type.
src/main/java/org/dizzymii/millenaire2/entity/HumanoidNPC.java	New entity implementing the SBL-like owner interface, manual sensor polling, and activity wiring.
src/main/java/org/dizzymii/millenaire2/Millenaire2.java	Wires memory-module registration on the mod event bus and registers HumanoidNPC attributes.

[Copilot]

The Javadoc and tests describe the damaged-tool threshold as “≤ 25% durability remaining”, but the implementation uses a strict < DAMAGE_THRESHOLD comparison. At exactly 25% remaining, the tool will not be flagged as damaged. Either change the comparison to <= or update the documentation/tests to match the intended strictness.

[DizzyMii]

@copilot apply changes based on this feedback

[Copilot]

start() copies the chosen tool into the main hand (bestTool.copy()) but never removes it from carriedInventory or returns the previous main-hand stack back into inventory. This effectively duplicates items and isn't a true swap. Consider performing an actual move/swap (remove the selected stack from carried inventory, place it in main hand, and optionally store the previous main-hand stack back into carried inventory). Updating/adding a GameTest to assert the carried inventory changes would prevent regressions.

Suggested change

	entity.setItemInHand(InteractionHand.MAIN_HAND, bestTool.copy());
	int index = inventory.indexOf(bestTool);
	if (index >= 0) {
	ItemStack previousMainHand = entity.getItemInHand(InteractionHand.MAIN_HAND);
	// Move the selected tool into the main hand
	entity.setItemInHand(InteractionHand.MAIN_HAND, bestTool);
	// Put the previous main-hand stack back into the carried inventory slot,
	// or clear the slot if the main hand was empty.
	if (previousMainHand.isEmpty()) {
	inventory.set(index, ItemStack.EMPTY);
	} else {
	inventory.set(index, previousMainHand);
	}
	}