Skip to content

fix(ios): drop trailing slash from WebDriverAgent /source URL - #496

Merged
gmegidish merged 1 commit into
mainfrom
fix/wda-source-trailing-slash
Oct 11, 2026
Merged

gmegidish merged 1 commit into
mainfrom
fix/wda-source-trailing-slash

Conversation

@gmegidish

Copy link
Copy Markdown
Member

Re-signed copy of #488 by @dudnjsckrgo, so it can pass the signed-commits rule on main. The commit is unchanged apart from the signature and a rebase onto current main; authorship stays with @dudnjsckrgo.

Supersedes #488.

Legacy iOS robot requested `/source/?format=json`. Current WebDriverAgent
(appium/WebDriverAgent master, runner build 16.14.x) does not route the
trailing-slash form and answers:

    {"value":{"error":"unknown command","message":"Unhandled endpoint: /source/ ..."}}

The error object has no `children`, so `filterSourceElements` returned an
empty list and `mobile_list_elements_on_screen` reported zero elements on a
physical iPhone (iOS 26.6.2) while `/screenshot` and `/wda/screen` worked.

`/source?format=json` is the route WebDriverAgent registers and is accepted by
both older and current builds.

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

coderabbitai Bot commented Oct 11, 2026

Copy link
Copy Markdown

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: a8e1c82c-8a05-46a1-9479-2cd603300bbc

📥 Commits

Reviewing files that changed from the base of the PR and between 2a6a224 and 78f1b20.


📒 Files selected for processing (1)
  • src/webdriver-agent.ts

 _______________________________
< Yippee-ki-yay, mother-bugger! >
 -------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR


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

@gmegidish
gmegidish merged commit 54cd787 into main Oct 11, 2026
6 of 7 checks passed
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