AI and pathfinding nodes by Stranger1992 · Pull Request #1231 · TombEngine/Tomb-Editor

Stranger1992 · 2026-05-18T08:44:32Z

This pull request introduces a comprehensive set of new nodes for controlling and querying various aspects of creature AI in the TombEngine Node system. These additions allow level designers to set and check creature moods, targets, friendliness, poison status, "hurt by player" status, jumping/monkey-swinging state, and goal-reaching status, as well as to set intelligent creature locations. The changes significantly expand the scripting capabilities for creature behavior in custom levels.

New Creature AI Node Functionality:

Added nodes to set and check creature mood, target, friendliness, poison status, hurt-by-player status, jumping, monkey-swinging, and goal-reaching status, as well as to set intelligent creature locations. These nodes enable more granular and dynamic control over enemy and NPC behaviors in levels. (TombLib/TombLib/Catalogs/TEN Node Catalogs/Creature Mood.lua)
Added a helper function to map mood enumeration indices to internal mood types for use in the new mood-related nodes. (TombLib/TombLib/Catalogs/TEN Node Catalogs/_System.lua)

Documentation and Catalog Updates:

Updated the Installer/Changes.txt to document the addition of nodes for checking and setting various aspects of creature AI.

And guarding against moveables not being active.

…oned..." nodes

Accidently deleted on branch.

Copilot

Pull request overview

Adds TEN node catalog support for creature AI/pathfinding controls and queries, expanding scripting options for level designers.

Changes:

Adds new Creature AI nodes for mood, target, friendliness, poison, hurt-by-player, jumping, monkey-swinging, goal, and AI location state.
Adds a _System.lua helper to map mood enumeration indices to TEN mood constants.
Updates installer changelog with the new creature AI node category.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File	Description
`TombLib/TombLib/Catalogs/TEN Node Catalogs/Creature Mood.lua`	Adds new Creature AI node definitions and Lua implementations.
`TombLib/TombLib/Catalogs/TEN Node Catalogs/_System.lua`	Adds helper mapping UI mood enum values to TEN mood constants.
`Installer/Changes.txt`	Documents the new creature AI nodes.

Comments suppressed due to low confidence (2)

TombLib/TombLib/Catalogs/TEN Node Catalogs/Creature Mood.lua:286

This log message says "location status" even though the node is setting the creature's goal-reached status. That makes failures from this node harder to diagnose in scripts that also use the location node.

    if mov:GetStatus() ~= 1 then
        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. Cannot set location status.",TEN.Util.LogLevel.ERROR)

TombLib/TombLib/Catalogs/TEN Node Catalogs/Creature Mood.lua:97

This inactive-object error says "No target set" even though this node is checking the current target. The message should match the check operation so users can distinguish it from failures in the set-target node.

    if mov:GetStatus() ~= 1 then
        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. No target set.",TEN.Util.LogLevel.ERROR)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

MontyTRC89 · 2026-06-18T12:36:29Z

+    local mov = TEN.Objects.GetMoveableByName(moveable)
+
+    if mov:GetStatus() ~= 1 then
+        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. No mood set.",TEN.Util.LogLevel.ERROR)   


MontyTRC89 · 2026-06-18T12:39:36Z

+    local mov = TEN.Objects.GetMoveableByName(moveable)
+
+    if mov:GetStatus() ~= 1 then
+        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. No target set.",TEN.Util.LogLevel.ERROR)


+        local targetMovAI = Objects.Creature(targetMov)
+        targetMovAI:SetTarget(mov)


TrainWrack · 2026-05-21T00:13:10Z

+-- !Arguments "Boolean, 20, {false}, Debug to console."
+LevelFuncs.Engine.Node.SetCreatureLocation = function(objectId, location, debug)
+    local moveables = TEN.Objects.GetMoveableByName(objectId)
+    if moveables:GetStatus() ~= 1 then


Add enum here rather than magic number of 1. For all nodes.

The mood condition node logged "No mood set." (copied from the set node); use "Cannot check mood." to match the other check nodes. Also correct the goal nodes, which said "location status" instead of "goal status".

The target argument is an unrestricted Moveables pick and Retaliate defaults to true, so an active non-creature target would be wrapped via Objects.Creature and crash at runtime. Guard that wrap with pcall, skipping retaliation and logging when the target is not a creature.

Replace the GetStatus() ~= 1 / == 1 active-status checks across all creature AI nodes with TEN.Objects.MoveableStatus.ACTIVE for readability.

MontyTRC89 · 2026-06-18T12:36:16Z

+-- !Arguments "Boolean, 20, {false}, Debug to console."
+LevelFuncs.Engine.Node.SetCreatureLocation = function(objectId, location, debug)
+    local moveables = TEN.Objects.GetMoveableByName(objectId)
+    if moveables:GetStatus() ~= 1 then


MontyTRC89 · 2026-06-18T12:36:29Z

+    local mov = TEN.Objects.GetMoveableByName(moveable)
+
+    if mov:GetStatus() ~= 1 then
+        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. No mood set.",TEN.Util.LogLevel.ERROR)   


MontyTRC89 · 2026-06-18T12:39:36Z

+    local mov = TEN.Objects.GetMoveableByName(moveable)
+
+    if mov:GetStatus() ~= 1 then
+        TEN.Util.PrintLog("moveable [ " .. moveable .. " ] is not active. No target set.",TEN.Util.LogLevel.ERROR)


Lwmte · 2026-06-20T08:54:51Z

Suggestion: merge new nodes into Creatures.lua, do not create separate file Creature Mood.lua

Stranger1992 added 10 commits May 17, 2026 04:07

"Set creature mood" "SetCreatureLocation" node and helper function

107fddc

Re-work "Set intelligent creature location" node

f84fdba

Rename section

6ae14cc

"If creature mood is..." node

da79c9f

"Set creature target" / "If creature target is..."

e20e3be

And guarding against moveables not being active.

"Set creature as friendly" / "If creature is friendly..." nodes

895ae56

Cleanup last commit

673a716

Add early exits / "Set Creature poison status" / "If creature is pois…

ff484d0

…oned..." nodes

More nodes and cleanup

d129203

Restored readme file

b7dd7ae

Accidently deleted on branch.

Stranger1992 added this to the Version 2.0 milestone May 18, 2026

Copilot AI review requested due to automatic review settings May 18, 2026 08:44

Copilot started reviewing on behalf of Stranger1992 May 18, 2026 08:45 View session

Copilot AI reviewed May 18, 2026

View reviewed changes

Stranger1992 and others added 3 commits May 18, 2026 09:52

Fix typo

afac1db

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Add space after commenting out the metadata

869b7eb

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Remove redundant if statement

241c3f9

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Stranger1992 requested a review from Copilot May 18, 2026 08:57

Copilot started reviewing on behalf of Stranger1992 May 18, 2026 08:58 View session

Copilot AI reviewed May 18, 2026

View reviewed changes

TrainWrack requested changes May 21, 2026

View reviewed changes

MontyTRC89 added 4 commits June 18, 2026 11:51

Merge branch 'develop' into AI-and-pathfinding-nodes

125fe60

Creature AI nodes: fix misleading inactive-object log messages

c0200b8

The mood condition node logged "No mood set." (copied from the set node); use "Cannot check mood." to match the other check nodes. Also correct the goal nodes, which said "location status" instead of "goal status".

Creature AI nodes: use MoveableStatus.ACTIVE instead of magic number

336929e

Replace the GetStatus() ~= 1 / == 1 active-status checks across all creature AI nodes with TEN.Objects.MoveableStatus.ACTIVE for readability.

MontyTRC89 approved these changes Jun 18, 2026

View reviewed changes

		local targetMovAI = Objects.Creature(targetMov)
		targetMovAI:SetTarget(mov)

Uh oh!

Conversation

Stranger1992 commented May 18, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

TrainWrack May 21, 2026

Choose a reason for hiding this comment

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

MontyTRC89 Jun 18, 2026

Choose a reason for hiding this comment

Uh oh!

Lwmte commented Jun 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants