zoushiyang/page-agent
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: use const for planASucceeded, clarify LinkedIn comment

Browse Source
This commit is contained in:
voidborne-d
2026-03-20 07:08:45 +00:00
parent 2f92a9cb32
commit 2e18bd862d
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/page-controller/src/actions.ts
View File

@@ -123,12 +123,10 @@ export async function inputTextElement(element: HTMLElement, text: string) {
// to Plan B (execCommand) if the text wasn't actually inserted.
//
// Plan A: Dispatch synthetic events
// Works: React contenteditable, Quill, some LinkedIn versions.
// Works: React contenteditable, Quill, LinkedIn.
// Fails: Slate.js, some contenteditable editors that ignore synthetic events.
// Sequence: beforeinput -> mutation -> input -> change -> blur
let planASucceeded = false
// Dispatch beforeinput + mutation + input for clearing
if (
element.dispatchEvent(
@@ -170,8 +168,7 @@ export async function inputTextElement(element: HTMLElement, text: string) {
}
// Verify Plan A worked by checking if the text was actually inserted
const currentText = element.innerText.trim()
planASucceeded = currentText === text.trim()
const planASucceeded = element.innerText.trim() === text.trim()
if (!planASucceeded) {
// Plan B: execCommand fallback (deprecated but widely supported)