[aaltat]

Summary

A JavaScript extension keyword has two routes into Node: the generated keyword, and call_js_keyword for a Python library built on Browser. Only the generated keyword filled the injected argument defaultTimeout, although the jsextension docstring said the library fills it. Follow-up to #5318, on top of #5352.

 generated keyword (Browser._jskeyword_call)
-  copy of the gRPC call, streaming and decoding
-  defaultTimeout = int(self.timeout)
+  call_js_keyword(name, **its arguments)

 call_js_keyword(keyword_name, /, **args)
+  args.setdefault("defaultTimeout", int(self.timeout))
   injected names -> "RESERVED"
   join bodyPart chunks, or None
-  json branch (Node never sets json on an extension response)

• A value the caller passes wins, so a library with a timeout of its own (the Electron library today, others that adopt a context they create) keeps it. Called as a keyword, the function always gets the library timeout; Robot users change it with Set Browser Timeout.
• Every call now sends defaultTimeout, also to functions that do not declare it. Node passes a function only the names it declares, so nothing changes for them.
• keyword_name is positional-only, so a JavaScript argument with that name no longer clashes with it.
• call_js_keyword gets a docstring; the jsextension docstring says who fills defaultTimeout. Browser/CONTEXT.md names JavaScript extension keyword and injected argument.

Evidence

• Before: call_js_keyword("openOwnContext", url="https://a.b") sends [["url", "https://a.b"]]; the function gets defaultTimeout === undefined.
  After: it sends [["url", "https://a.b"], ["defaultTimeout", 7000]]; with defaultTimeout=5000 it sends 5000.
• utest/test_js_extension_call.py: 12 passed. New: filled when left out, the caller's value kept, an injected name the caller passes sent as "RESERVED", an argument named keyword_name. The test: characterize the generated JavaScript extension keyword #5352 tests guard the refactor unchanged, except that two payloads now end with defaultTimeout.
• inv atest --suite "05 JS Tests": 59 passed; --suite "09 Plugins": 19 passed.

Merge Danger

Door: two-way

Blast Radius: small

Libraries that call call_js_keyword: a function that declares defaultTimeout now gets the library timeout instead of undefined when the caller leaves it out, and passing keyword_name= by name now fails. A subclass that overrides call_js_keyword now also sees generated keyword calls.

🤖 Generated with Claude Code