Skip to content

Move HTN Preconditions and Operators to IDependencyCollection instead of IoCManager#43694

Open
jessicamaybe wants to merge 8 commits intospace-wizards:masterfrom
jessicamaybe:2026-04-22-HTN-iDependency
Open

Move HTN Preconditions and Operators to IDependencyCollection instead of IoCManager#43694
jessicamaybe wants to merge 8 commits intospace-wizards:masterfrom
jessicamaybe:2026-04-22-HTN-iDependency

Conversation

@jessicamaybe
Copy link
Copy Markdown
Contributor

About the PR

Moved HTN Preconditions and Operators to IDependencyCollection instead of IoCManager

Why / Balance

its 2026 I don't want to use IEntityManager.System<T> for everything

Technical details

  • things resolves via dependencies now instead of having to do it manually
  • all of the operators/preconditions use mostly shared code now (npcs in shared one day 🙏)
  • slight changese in medibot system (princess cheeseballs made me do it)

Media

Requirements

Breaking changes

HTNOperator's and HTNPrecondition's now resolve their dependencies via IDependencyCollection instead of IoCManager

HTN Operators and Preconditions will need to be refactored to to use [Dependency] instead of _entMan.System<T>

Changelog

@PJBot PJBot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/M Denotes a PR that changes 100-999 lines. labels Apr 23, 2026
@Pok27 Pok27 added P3: Standard Priority: Default priority for repository items. T: Refactor Type: Refactor of notable amount of codebase D2: Medium Difficulty: A good amount of codebase knowledge required. T: Cleanup Type: Code clean-up, without being a full refactor or feature D3: Low Difficulty: Some codebase knowledge required. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. T: Refactor Type: Refactor of notable amount of codebase D2: Medium Difficulty: A good amount of codebase knowledge required. labels Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D3: Low Difficulty: Some codebase knowledge required. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/M Denotes a PR that changes 100-999 lines. T: Cleanup Type: Code clean-up, without being a full refactor or feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants