Skip to content

Conversation

@Kudo
Copy link
Collaborator

@Kudo Kudo commented Nov 26, 2025

Why

to reduce token usage, we can merge some mcp tools

How

  • automation_tap and automation_tap_by_testid into automation_tap that can pass either x/y or testID
  • automation_take_screenshot and automation_take_screenshot_by_testid into automation_take_screenshot that can pass either x/y or testID
  • rename automation_find_view_by_testid to automation_find_view

Test Plan

tested with claude code

> add a new button at home and verify it's appeared by taking screenshot and you can also use it to tap to next screen

⏺ I'll help you add a new button to the home screen, verify it appears, and test navigation. Let me start by understanding the current home screen structure and then implement the changes.

⏺ Read(app/index.tsx)
  ⎿  Read 698 lines

⏺ Search(pattern: "app/**/*.tsx")
  ⎿  Found 3 files (ctrl+o to expand)

⏺ Read(app/property/[id].tsx)
  ⎿  Read 331 lines

⏺ Now I'll add a new button that navigates to the property detail screen. Let me update the todos and add the button.

⏺ Update(app/index.tsx)
  ⎿  Updated app/index.tsx with 7 additions
       281            >
       282              <Text style={styles.floatingButtonText}>Test Button</Text>
       283            </TouchableOpacity>
       284 +          <TouchableOpacity
       285 +            style={styles.floatingButton}
       286 +            onPress={() => router.push('/property/4')}
       287 +            testID="view-property-button"
       288 +          >
       289 +            <Text style={styles.floatingButtonText}>View Property</Text>
       290 +          </TouchableOpacity>
       291          </View>
       292
       293          {/* Host Registration Modal */}

⏺ Perfect! I've added a new "View Property" button. Now let me take a screenshot to verify it appears on the screen.

⏺ expo-mcp-stdio-local - automation_take_screenshot (MCP)(projectRoot: "/Users/kudo/sdk54mcp")
  ⎿  [Image]

⏺ Excellent! The screenshot shows the new "View Property" button has been successfully added at the bottom of the home screen. Now let me tap on it to navigate to the property detail screen.

⏺ expo-mcp-stdio-local - automation_tap (MCP)(projectRoot: "/Users/kudo/sdk54mcp", testID: "view-property-button")
  ⎿  {
       "success": true,
       "duration": 1,
     … +6 lines (ctrl+o to expand)

⏺ Great! The button was tapped successfully. Let me take another screenshot to verify we've navigated to the property detail screen.

⏺ expo-mcp-stdio-local - automation_take_screenshot (MCP)(projectRoot: "/Users/kudo/sdk54mcp")
  ⎿  [Image]

⏺ Perfect! The navigation was successful. The app has navigated to the property detail screen showing the "Apartment in Mexico City" property. Let me mark the final todo as completed.

@Kudo Kudo marked this pull request as ready for review November 26, 2025 17:57
@Kudo Kudo requested a review from chrfalch November 26, 2025 17:57
Copy link

@chrfalch chrfalch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

@Kudo Kudo merged commit f14a401 into main Nov 27, 2025
1 check passed
@Kudo Kudo deleted the @kudo/merge-automation-tools branch November 27, 2025 10:08
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.

3 participants