Add comprehensive review of PulsarWebDriver click methods implementation

[Copilot]

Review of PulsarWebDriver.click(selector, count) and click(selector, modifier) implementations identifies 20+ issues across consistency, robustness, flexibility, and documentation.

Review Document

Added docs-dev/review-pulsar-webdriver-click-methods.md with detailed analysis.

Key Issues Identified

High Priority:

• Missing bringToFront() call (present in hover() but not click()) - can cause clicks to fail or target wrong window

Medium Priority:

• No parameter validation (blank selectors, negative counts, invalid modifiers silently accepted)
• Hardcoded count=1 in modifier variant prevents "Ctrl+double-click" scenarios
• Inconsistent @Throws annotation between the two methods
• Hardcoded position="center" limits use cases (vs. fill() which uses "right")
• Method overloading prevents combining count and modifier parameters
• Invalid modifiers silently ignored instead of failing fast
• 500-1000ms default delay too slow for automation scenarios
• No KDoc documentation on valid values and behavior
• Missing test coverage for both methods

Comparison with existing code:

// hover() - has bringToFront()
override suspend fun hover(selector: String) {
    bringToFront()  // ✓
    driverHelper.invokeOnElement(selector, "hover", scrollIntoView = true) { node ->
        emulator.hover(node, position = "center")
    }
}

// click() - missing bringToFront()
override suspend fun click(selector: String, count: Int) {
    // ✗ no bringToFront()
    driverHelper.invokeOnElement(selector, "click", scrollIntoView = true) { node ->
        val delayMillis = randomDelayMillis("click")
        emulator.click(node, count, position = "center", modifier = null, delayMillis = delayMillis)
    }
}

Each issue includes code examples, impact assessment, and suggested fixes. Document is in Chinese per codebase conventions.

Original prompt

审查 PulsarWebDriver 的以下两个方法的实现细节：

override suspend fun click(selector: String, count: Int)
override suspend fun click(selector: String, modifier: String)

提出尽可能多的问题。

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.