Skip to content

refactor!: remove Translators questionnaire runtime (runner, trivia UI, BP relay) - #11

Merged
JosephOIbrahim merged 1 commit into
masterfrom
decruft-translators-runtime
Jul 2, 2026
Merged

refactor!: remove Translators questionnaire runtime (runner, trivia UI, BP relay)#11
JosephOIbrahim merged 1 commit into
masterfrom
decruft-translators-runtime

Conversation

@JosephOIbrahim

Copy link
Copy Markdown
Owner

What

Removes the legacy Translators cognitive-profiling game from the runtime path:

  • bridge_orchestrator.py — the questionnaire runner (892 lines)
  • Source/UnrealEngineBridge/UI/** — trivia GameMode, HUD, and 6 Slate widgets
  • BridgeComponent.{h,cpp} — thin Blueprint relay; its only consumers were the removed UI, and it called the removed subsystem game-flow methods
  • UEBridgeSubsystem StartGame/StopGame/SubmitAnswer state-machine entry points (file-watch/state protocol internals retained for now)

usd_bridge/ is deliberately untouched — parked, not shipped.

Why

First milestone of the UE 5.8 / Epic-MCP-era refocus: UnrealEngine_Bridge sheds the legacy game product line and concentrates on the differentiated agentic layer (perception, spatial reasoning, lighting, mograph, resilience). Curated from the ANVIL decruft-game task (3/3 features passing).

Verification

  • UE 5.7 editor-target compile: Result: Succeeded (19 actions, all three modules linked)
  • python -m pytest -q: 415 passed
  • Cross-tree symbol check: zero StartGame|StopGame|SubmitAnswer occurrences in UEBridgeRuntime
  • Zero references to removed classes in Config/ or Content/ (scanned before deletion)

🤖 Generated with Claude Code

…I, BP relay)

Removes the legacy cognitive-profiling game from the runtime path:
- bridge_orchestrator.py (the questionnaire runner, 892 lines)
- Source/UnrealEngineBridge/UI/** (trivia GameMode, HUD, 6 Slate widgets)
- BridgeComponent.{h,cpp} (thin BP relay whose only consumers were the
  removed UI; it called the removed subsystem game-flow methods)
- UEBridgeSubsystem StartGame/StopGame/SubmitAnswer state machine entry
  points (file-watch/state protocol internals retained for now)

Zero references to the removed classes in Config/ or Content/ (verified).
usd_bridge/ is deliberately untouched - parked, not shipped.

Curated from the ANVIL decruft-game task (3/3 features passing, suite
green at 415 tests). Cross-tree symbol check: no StartGame/StopGame/
SubmitAnswer remain in UEBridgeRuntime.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@JosephOIbrahim, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 3 seconds

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b636a99e-85c0-4ba2-9fc2-17c48948d397

📥 Commits

Reviewing files that changed from the base of the PR and between 3863183 and 6574998.

📒 Files selected for processing (22)
  • Plugins/UEBridge/Source/UEBridgeRuntime/Private/UEBridgeSubsystem.cpp
  • Plugins/UEBridge/Source/UEBridgeRuntime/Public/BridgeTypes.h
  • Plugins/UEBridge/Source/UEBridgeRuntime/Public/UEBridgeSubsystem.h
  • Source/UnrealEngineBridge/BridgeComponent.cpp
  • Source/UnrealEngineBridge/BridgeComponent.h
  • Source/UnrealEngineBridge/UI/UEBridgeGameMode.cpp
  • Source/UnrealEngineBridge/UI/UEBridgeGameMode.h
  • Source/UnrealEngineBridge/UI/UEBridgeHUD.cpp
  • Source/UnrealEngineBridge/UI/UEBridgeHUD.h
  • Source/UnrealEngineBridge/UI/W_ConnectingScreen.cpp
  • Source/UnrealEngineBridge/UI/W_ConnectingScreen.h
  • Source/UnrealEngineBridge/UI/W_FinaleScreen.cpp
  • Source/UnrealEngineBridge/UI/W_FinaleScreen.h
  • Source/UnrealEngineBridge/UI/W_OptionButton.cpp
  • Source/UnrealEngineBridge/UI/W_OptionButton.h
  • Source/UnrealEngineBridge/UI/W_ProgressIndicator.cpp
  • Source/UnrealEngineBridge/UI/W_ProgressIndicator.h
  • Source/UnrealEngineBridge/UI/W_QuestionDisplay.cpp
  • Source/UnrealEngineBridge/UI/W_QuestionDisplay.h
  • Source/UnrealEngineBridge/UI/W_TitleScreen.cpp
  • Source/UnrealEngineBridge/UI/W_TitleScreen.h
  • bridge_orchestrator.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch decruft-translators-runtime

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@JosephOIbrahim
JosephOIbrahim merged commit f009a71 into master Jul 2, 2026
3 checks passed
@JosephOIbrahim
JosephOIbrahim deleted the decruft-translators-runtime branch July 2, 2026 14:01
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