fix: replace NovaWindows automation name with DesktopDriver

Author: y-schwab

.github/workflows/claude-code-review.yml

@@ -1,39 +1,42 @@
 name: Claude Code Review
 
 on:
-  issue_comment:
-    types: [created]
+  pull_request:
+    types: [opened]
+    branches: [main]
 
 jobs:
   claude-review:
-    # Only run when an owner/member comments "/review" on a PR
-    if: |
-      github.event.issue.pull_request != null &&
-      contains(github.event.comment.body, '/review') &&
-      (
-        github.event.comment.author_association == 'OWNER' ||
-        github.event.comment.author_association == 'MEMBER' ||
-        github.event.comment.author_association == 'COLLABORATOR'
-      )
-
     runs-on: ubuntu-latest
     permissions:
       contents: read
       pull-requests: write
       issues: write
       id-token: write
+      actions: read
 
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
         with:
           fetch-depth: 1
 
+      - name: Load review guide
+        id: review-guide
+        run: echo "REVIEW_GUIDE<<EOF" >> $GITHUB_OUTPUT && cat .github/claude-review-guide.md >> $GITHUB_OUTPUT && echo "EOF" >> $GITHUB_OUTPUT
+
       - name: Run Claude Code Review
         id: claude-review
         uses: anthropics/claude-code-action@v1
         with:
           claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
-          plugin_marketplaces: 'https://github.com/anthropics/claude-code.git'
-          plugins: 'code-review@claude-code-plugins'
-          prompt: '/code-review:code-review ${{ github.repository }}/pull/${{ github.event.issue.number }}'
+          additional_permissions: |
+            actions: read
+          prompt: |
+            Please review the changes in pull request #${{ github.event.pull_request.number }} in this repository.
+            Use the `gh` CLI to fetch the PR diff and details.
+            Follow these project-specific review guidelines:
+
+            ${{ steps.review-guide.outputs.REVIEW_GUIDE }}
+
+            Post your review as a PR review with inline comments where applicable using `gh pr review`.

CLAUDE.md

@@ -57,7 +57,7 @@ All driver commands live in `lib/commands/` and are mixed into the driver class
 
 ## Key capabilities
 
-- `platformName`: `"Windows"`, `automationName`: `"NovaWindows"`
+- `platformName`: `"Windows"`, `automationName`: `"DesktopDriver"`
 - Supported locator strategies: `xpath`, `accessibility id`, `id`, `name`, `class name`, `tag name`, `-windows uiautomation`
 - Custom `executeScript()` commands listed in README.md
 - Prerun/postrun PowerShell scripts via session capabilities

README.md

@@ -37,7 +37,7 @@ NovaWindows Driver supports the following capabilities:
 Capability Name | Description
 --- | ---
 platformName | Must be set to `Windows` (case-insensitive).
-automationName | Must be set to `NovaWindows` (case-insensitive).
+automationName | Must be set to `DesktopDriver` (case-insensitive).
 smoothPointerMove | CSS-like easing function (including valid Bezier curve). This controls the smooth movement of the mouse for `delayBeforeClick` ms. Example: `ease-in`, `cubic-bezier(0.42, 0, 0.58, 1)`.
 delayBeforeClick | Time in milliseconds before a click is performed.
 delayAfterClick | Time in milliseconds after a click is performed.

examples/C#/CalculatorTest/CalculatorTest/CalculatorSession.cs

@@ -22,7 +22,7 @@ protected static void Setup()
         var appiumOptions = new AppiumOptions
         {
             App = CalculatorAppId,
-            AutomationName = "NovaWindows",
+            AutomationName = "DesktopDriver",
             PlatformName = "Windows",
         };
 
@@ -41,4 +41,4 @@ protected static void TearDown()
         Session.Quit();
         Session = null;
     }
-}
+}

package.json

@@ -44,7 +44,7 @@
     "platformNames": [
       "Windows"
     ],
-    "mainClass": "DesktopDriver"
+    "mainClass": "NovaWindowsDriver"
   },
   "devDependencies": {
     "@appium/eslint-config-appium-ts": "^2.0.3",

test/e2e/helpers/session.ts

@@ -19,7 +19,7 @@ export async function createCalculatorSession(extraCaps?: Record<string, unknown
         ...APPIUM_SERVER,
         capabilities: {
             platformName: 'Windows',
-            'appium:automationName': 'NovaWindows',
+            'appium:automationName': 'DesktopDriver',
             'appium:app': CALCULATOR_APP_ID,
             ...extraCaps,
         } as Caps,
@@ -33,7 +33,7 @@ export async function createNotepadSession(extraCaps?: Record<string, unknown>):
         ...APPIUM_SERVER,
         capabilities: {
             platformName: 'Windows',
-            'appium:automationName': 'NovaWindows',
+            'appium:automationName': 'DesktopDriver',
             'appium:app': NOTEPAD_APP_PATH,
             ...extraCaps,
         } as Caps,
@@ -47,7 +47,7 @@ export async function createTodoSession(extraCaps?: Record<string, unknown>): Pr
         ...APPIUM_SERVER,
         capabilities: {
             platformName: 'Windows',
-            'appium:automationName': 'NovaWindows',
+            'appium:automationName': 'DesktopDriver',
             'appium:app': TODO_APP_ID,
             ...extraCaps,
         } as Caps,
@@ -61,7 +61,7 @@ export async function createRootSession(extraCaps?: Record<string, unknown>): Pr
         ...APPIUM_SERVER,
         capabilities: {
             platformName: 'Windows',
-            'appium:automationName': 'NovaWindows',
+            'appium:automationName': 'DesktopDriver',
             'appium:app': 'Root',
             ...extraCaps,
         } as Caps,