Skip to content

[codex] integrate upstream shortest-path catalog and headless transport tests - #1825

Draft
chsami wants to merge 2 commits into
developmentfrom
feature/upstream-shortest-path-integration
Draft

[codex] integrate upstream shortest-path catalog and headless transport tests#1825
chsami wants to merge 2 commits into
developmentfrom
feature/upstream-shortest-path-integration

Conversation

@chsami

@chsami chsami commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • pin the Skretzo shortest-path collision map, league-region data, and transport catalogs with upstream provenance
  • parse upstream transport requirements into Microbot's pathfinder and route supported transports through explicit walker execution intents
  • expose transport-aware route steps and integrate the selected transport metadata with walker and banking planning
  • add a deterministic headless harness that tests collision-map routing, transport eligibility, and execution intent without starting or logging in a RuneLite client
  • document the integration decision, movement caveats, and offline testing workflow

Why

Microbot's web walker was missing transport and collision coverage available in the upstream shortest-path project. Transport behavior also depended heavily on logged-in runtime testing. This change imports a pinned, reviewable catalog and adds side-effect-free coverage for route selection and transport requirements.

Impact

The web walker can consider the expanded transport catalog while preserving Microbot-specific execution handlers and configuration. Developers can validate membership, skill, quest, varbit, item, equipment, currency, teleport, and transport-selection behavior offline. Live smoke tests are still required for scene objects, NPCs, widgets, dialogue, animation timing, and other dynamic game contracts.

Validation

  • ./gradlew :client:compileJava
  • shortest-path unit suite, including collision snapshot, catalog integration, route-step, execution-registry, and headless harness tests
  • shortest-path benchmark
  • Checkstyle
  • focused offline harness: ./gradlew :client:runUnitTests --tests 'net.runelite.client.plugins.microbot.shortestpath.HeadlessTransportHarnessTest'

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f2a740f3-a5c8-4aa4-a09d-0963739f12d6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

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.

2 participants