Fix naming inconsistency: TargetBotCore → TargetCore - #3
Merged
Merged
Conversation
Merged
…ghout codebase Co-authored-by: mCodex <5920613+mCodex@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update bot core module based on review feedback
Fix naming inconsistency: TargetBotCore → TargetCore
Dec 10, 2025
mCodex
approved these changes
Dec 10, 2025
mCodex
approved these changes
Dec 10, 2025
Contributor
There was a problem hiding this comment.
Pull request overview
This PR corrects a naming inconsistency where the module TargetCore was incorrectly referenced as TargetBotCore in 11 locations across code comments and documentation files.
- Updates all references from
TargetBotCoretoTargetCoreto match the actual module name defined intargetbot/core.lua - Affects only comments and documentation—no functional code changes
- Improves code clarity and consistency across the codebase
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| targetbot/target.lua | Updated 2 comment references in DRY principle comments and distance calculation utility |
| targetbot/creature_attack.lua | Updated 4 comment references in header, wave avoidance system, and utility function documentation |
| docs/TARGETBOT.md | Updated feature list bullet point describing pure function scoring |
| docs/PERFORMANCE.md | Updated section heading and code example showing geometry calculations |
| ROADMAP.md | Updated feature checklist item for TargetCore module |
| README.md | Updated design patterns table entry for pure functions |
Review Summary: All changes are accurate and consistent. The PR successfully corrects all 11 instances of the incorrect TargetBotCore name to the correct TargetCore module name. No functional code was modified—only comments and documentation were updated. No issues found.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The module is named
TargetCorebut was referenced asTargetBotCorein 11 locations across comments and documentation.Changes:
No functional changes—documentation and comment alignment 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.