I have a question specifically around giving actions access to the data it needs. Since tasks are their own classes, and my player, enemy, and map data are in their own classes, what is the generally accepted way of letting these tasks in a behavior tree have access to all this game data? With the idea of no globals anywhere (a rule I generally like to follow) how should they be getting access to this game data information?

Are we literally passing all this data to every node starting with the top node which will do the passing of all this data to it's child nodes as it transverses the tree? That doesn't seem extendable as new things need to be created and passed to each node. Are we creating another class that holds all these things and passing that? Seems extendable but sort of cheap to give each node access to things that that specific node doesn't/shouldn't care about. For example if a tasks job is to check if the actor has ammo before going onto another task to shoot the gun, that task doesn't need access to the player information or the map data. It would just need access to the actor that tree is attached to.

That's an interesting question that I haven't seen posed before. In general I would say that as behaviour trees are often aimed at a game designer (not a developer), it is dangerous to filter the available data too much, as the designer may be unable to add the kind of nodes they want further down the tree if the information required by those nodes is not available.

Having said that, global variables or passing huge state objects is not good either. Perhaps some kind of blackboard or factory object which can obtain a node the data that it needs? Or alternately you have node types, which only get a certain type of data, but that limits the capabilities somewhat.

Is there any generally accepted method for this?

I'm very new with Behavior Trees, but I was playing around with the idea of these tasks & actions being functions inside the Gameplay state class which would hold all your game data anyway. Each node would be an event which points to a function you create in your Gameplay state class instead of being their own classes in themselves. When learning about these I find it very odd that actions or tasks are their entirely own class because it seems really all it needs to be is a function. Then all the nodes in your tree just point to the function to run instead of creating an entire object. You'd still need a tree hierarchy setup but they could only store variables of their children, and basically a pointer to the function to run isntead of definding their own Update() methods. I'll try to get this in practice soon, but it just seemed odd that tasks and actions where their entire class. Seems by doing that you have to break/bend some "guidelines" with programming to get them to see the data they need.

I'm a big OOP guy so I can see breaking them into classes helps separate the creation of these easier then if they were all in the same class, but it still just seems odd the tasks/actions are entire classes themselves.