Skip to content

fix: resolve target GameObjects through ObjectsHelper everywhere; add include_stack_trace to get_console_logs#30

Open
dehuaichendragonplus wants to merge 1 commit into
FunplayAI:mainfrom
dehuaichendragonplus:fix/target-resolution-and-log-stack-trace-v2
Open

fix: resolve target GameObjects through ObjectsHelper everywhere; add include_stack_trace to get_console_logs#30
dehuaichendragonplus wants to merge 1 commit into
FunplayAI:mainfrom
dehuaichendragonplus:fix/target-resolution-and-log-stack-trace-v2

Conversation

@dehuaichendragonplus

Copy link
Copy Markdown
Contributor

What changed

Checklist

  • I tested the package in a clean Unity 2022.3+ project (tested live in a Unity 6000.3.13f1 project via the MCP tools themselves: compiled clean, then exercised the actual runtime behavior -- see below)
  • I verified Funplay > MCP Server opens and starts correctly (verified indirectly: all MCP tool calls below went through a live running server)
  • If I changed setup, update, or config flows, I verified the affected flow end-to-end (N/A, no setup/update/config flow touched)
  • I updated docs for any user-facing behavior changes (CHANGELOG updated; README/tool docs not touched)
  • I did not commit local junk such as .idea/ or .DS_Store
  • I updated CHANGELOG.md when the change affects users

Manual verification performed

  • Compiled with 0 errors after pulling this branch into a live project's embedded package.
  • Created a GameObject, deactivated it, then select_object resolved it by name -- would have failed with the old GameObject.Find.
  • create_button with parent_name set to that inactive GameObject produced the expected hierarchy path (get_game_object_info confirmed Parent/Button).
  • Triggered a real Debug.LogError and confirmed get_console_logs(include_stack_trace=true) returns the indented stack trace under the message.

… include_stack_trace to get_console_logs

Rebased onto v0.5.0 (post FunplayAI#28/FunplayAI#29 merge).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant