helloadhavan/github_issues huggingface dataset:
Repository: ollama/ollama
Pull request: x: disable web search tool registration

Temporarily disable web search from being registered in the default tool registry while keeping the implementation intact for future use.

Changed files:
filename: x/tools/registry.go
patch: @@ -94,9 +94,10 @@ func (r *Registry) Count() int {
 // - OLLAMA_AGENT_DISABLE_BASH=1 disables bash
 func DefaultRegistry() *Registry {
 	r := NewRegistry()
-	if os.Getenv("OLLAMA_AGENT_DISABLE_WEBSEARCH") == "" {
-		r.Register(&WebSearchTool{})
-	}
+	// TODO(parthsareen): re-enable web search once it's ready for release
+	// if os.Getenv("OLLAMA_AGENT_DISABLE_WEBSEARCH") == "" {
+	// 	r.Register(&WebSearchTool{})
+	// }
 	if os.Getenv("OLLAMA_AGENT_DISABLE_BASH") == "" {
 		r.Register(&BashTool{})
 	}

filename: x/tools/registry_test.go
patch: @@ -93,19 +93,14 @@ func TestRegistry_Execute(t *testing.T) {
 func TestDefaultRegistry(t *testing.T) {
 	r := DefaultRegistry()
 
-	if r.Count() != 2 {
-		t.Errorf("expected 2 tools in default registry, got %d", r.Count())
+	if r.Count() != 1 {
+		t.Errorf("expected 1 tool in default registry, got %d", r.Count())
 	}
 
 	_, ok := r.Get("bash")
 	if !ok {
 		t.Error("expected bash tool in default registry")
 	}
-
-	_, ok = r.Get("web_search")
-	if !ok {
-		t.Error("expected web_search tool in default registry")
-	}
 }
 
 func TestDefaultRegistry_DisableWebsearch(t *testing.T) {
@@ -133,18 +128,8 @@ func TestDefaultRegistry_DisableBash(t *testing.T) {
 
 	r := DefaultRegistry()
 
-	if r.Count() != 1 {
-		t.Errorf("expected 1 tool with bash disabled, got %d", r.Count())
-	}
-
-	_, ok := r.Get("web_search")
-	if !ok {
-		t.Error("expected web_search tool in registry")
-	}
-
-	_, ok = r.Get("bash")
-	if ok {
-		t.Error("expected bash to be disabled")
+	if r.Count() != 0 {
+		t.Errorf("expected 0 tools with bash disabled, got %d", r.Count())
 	}
 }
Repository: vercel/next.js
Pull request: remove worktrees accidentally committed in #90166

🤦‍♂️ 

Changed files:
filename: .claude/worktrees/keen-bassi
patch: @@ -1 +0,0 @@
-Subproject commit 0d1968d350efcb3890dda9f429053027013de536

filename: .claude/worktrees/modest-stonebraker
patch: @@ -1 +0,0 @@
-Subproject commit b9757fc8d466ee39d6b4437ea6d75c598f7f0cc3
Repository: ollama/ollama
Pull request: olmo3: fix flaky test

I introduced this in <https://github.com/ollama/ollama/pull/13525>

Changed files:
filename: model/renderers/olmo3_test.go
patch: @@ -227,9 +227,9 @@ func TestOlmo3Renderer(t *testing.T) {
 							ID: "call_1",
 							Function: api.ToolCallFunction{
 								Name: "book_flight",
-								Arguments: testArgs(map[string]any{
-									"from": "SFO",
-									"to":   "NYC",
+								Arguments: testArgsOrdered([]orderedArg{
+									{"from", "SFO"},
+									{"to", "NYC"},
 								}),
 							},
 						},
@@ -243,9 +243,9 @@ func TestOlmo3Renderer(t *testing.T) {
 						Name: "book_flight",
 						Parameters: api.ToolFunctionParameters{
 							Type: "object",
-							Properties: testPropsMap(map[string]api.ToolProperty{
-								"from": {Type: api.PropertyType{"string"}},
-								"to":   {Type: api.PropertyType{"string"}},
+							Properties: testPropsOrdered([]orderedProp{
+								{"from", api.ToolProperty{Type: api.PropertyType{"string"}}},
+								{"to", api.ToolProperty{Type: api.PropertyType{"string"}}},
 							}),
 						},
 					},

filename: model/renderers/testhelpers_test.go
patch: @@ -34,3 +34,18 @@ func testArgsOrdered(pairs []orderedArg) api.ToolCallFunctionArguments {
 	}
 	return args
 }
+
+// orderedProp represents a key-value pair for ordered property creation
+type orderedProp struct {
+	Key   string
+	Value api.ToolProperty
+}
+
+// testPropsOrdered creates a ToolPropertiesMap with a specific key order
+func testPropsOrdered(pairs []orderedProp) *api.ToolPropertiesMap {
+	props := api.NewToolPropertiesMap()
+	for _, p := range pairs {
+		props.Set(p.Key, p.Value)
+	}
+	return props
+}
Repository: electron/electron
Pull request: fix: correct utility process exit code on Windows

Backport of #50256

See that PR for details.


Notes: Fixed utilityProcess exit event reporting incorrect exit codes on Windows when the exit code has the high bit


Changed files:
filename: shell/browser/api/electron_api_utility_process.cc
patch: @@ -259,7 +259,7 @@ void UtilityProcessWrapper::OnServiceProcessLaunch(
   EmitWithoutEvent("spawn");
 }
 
-void UtilityProcessWrapper::HandleTermination(uint64_t exit_code) {
+void UtilityProcessWrapper::HandleTermination(uint32_t exit_code) {
   // HandleTermination is called from multiple callsites,
   // we need to ensure we only process it for the first callsite.
   if (terminated_)
@@ -327,7 +327,7 @@ void UtilityProcessWrapper::CloseConnectorPort() {
   }
 }
 
-void UtilityProcessWrapper::Shutdown(uint64_t exit_code) {
+void UtilityProcessWrapper::Shutdown(uint32_t exit_code) {
   node_service_remote_.reset();
   HandleTermination(exit_code);
 }

filename: shell/browser/api/electron_api_utility_process.h
patch: @@ -57,7 +57,7 @@ class UtilityProcessWrapper final
   static gin_helper::Handle<UtilityProcessWrapper> Create(gin::Arguments* args);
   static raw_ptr<UtilityProcessWrapper> FromProcessId(base::ProcessId pid);
 
-  void Shutdown(uint64_t exit_code);
+  void Shutdown(uint32_t exit_code);
 
   // gin_helper::Wrappable
   static gin::DeprecatedWrapperInfo kWrapperInfo;
@@ -77,7 +77,7 @@ class UtilityProcessWrapper final
   void OnServiceProcessLaunch(const base::Process& process);
   void CloseConnectorPort();
 
-  void HandleTermination(uint64_t exit_code);
+  void HandleTermination(uint32_t exit_code);
 
   void PostMessage(gin::Arguments* args);
   bool Kill();

filename: spec/api-utility-process-spec.ts
patch: @@ -129,6 +129,22 @@ describe('utilityProcess module', () => {
       expect(code).to.equal(exitCode);
     });
 
+    ifit(process.platform === 'win32')('emits correct exit code when high bit is set on Windows', async () => {
+      // NTSTATUS code with high bit set should not be mangled by sign extension.
+      const exitCode = 0xC0000005;
+      const child = utilityProcess.fork(path.join(fixturesPath, 'custom-exit.js'), [`--exitCode=${exitCode}`]);
+      const [code] = await once(child, 'exit');
+      expect(code).to.equal(exitCode);
+    });
+
+    ifit(process.platform !== 'win32')('emits correct exit code when child process crashes on posix', async () => {
+      // Crash exit codes should not be sign-extended to large 64-bit values.
+      const child = utilityProcess.fork(path.join(fixturesPath, 'crash.js'));
+      const [code] = await once(child, 'exit');
+      expect(code).to.not.equal(0);
+      expect(code).to.be.lessThanOrEqual(0xFFFFFFFF);
+    });
+
     it('does not run JS after process.exit is called', async () => {
       const file = path.join(os.tmpdir(), `no-js-after-exit-log-${Math.random()}`);
       const child = utilityProcess.fork(path.join(fixturesPath, 'no-js-after-exit.js'), [`--testPath=${file}`]);
Repository: huggingface/transformers
Pull request: pipelines do not have modelcard

# What does this PR do?

Fixes #43170 for `modelcard` removal

Quick reproduce:

```
from transformers import pipeline
fmp=pipeline("fill-mask","google-bert/bert-base-cased")
fmp.save_pretrained("tmpdir")
```

## Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [x] Did you read the [contributor guideline](https://github.com/huggingface/transformers/blob/main/CONTRIBUTING.md#create-a-pull-request),
      Pull Request section?
- [ ] Was this discussed/approved via a Github issue or the [forum](https://discuss.huggingface.co/)? Please add a link
      to it if that's the case.
- [ ] Did you make sure to update the documentation with your changes? Here are the
      [documentation guidelines](https://github.com/huggingface/transformers/tree/main/docs), and
      [here are tips on formatting docstrings](https://github.com/huggingface/transformers/tree/main/docs#writing-source-documentation).
- [ ] Did you write any new necessary tests?


## Who can review?

Library:

- pipelines: @Rocketknight1


Changed files:
filename: src/transformers/pipelines/base.py
patch: @@ -985,9 +985,6 @@ def save_pretrained(self, save_directory: str | os.PathLike, **kwargs: Any):
         if self.image_processor is not None:
             self.image_processor.save_pretrained(save_directory, **kwargs)
 
-        if self.modelcard is not None:
-            self.modelcard.save_pretrained(save_directory)
-
     def transform(self, X):
         """
         Scikit / Keras interface to transformers' pipelines. This method will forward to __call__().
Repository: rust-lang/rust
Pull request: triagebot: add reminder for bumping CI LLVM stamp

I'm not sure what else can be done automatically to help us not forget this, but at least this gives a chance for the PR author/reviewer to be reminded (e.g. myself).

Changed files:
filename: triagebot.toml
patch: @@ -1412,6 +1412,15 @@ cc = ["@m-ou-se"]
 [mentions."compiler/rustc_ast_lowering/src/format.rs"]
 cc = ["@m-ou-se"]
 
+[mentions."src/ci/github-actions/jobs.yml"]
+message = """
+> [!WARNING]
+>
+> If you are changing how CI LLVM is built or linked, make sure to bump
+> `src/bootstrap/download-ci-llvm-stamp`.
+"""
+cc = ["@jieyouxu"]
+
 # Content-based mentions
 
 [mentions."#[miri::intrinsic_fallback_is_spec]"]
Repository: electron/electron
Pull request: fix: correct utility process exit code on Windows

Backport of #50256

See that PR for details.


Notes: Fixed utilityProcess exit event reporting incorrect exit codes on Windows when the exit code has the high bit


Changed files:
filename: shell/browser/api/electron_api_utility_process.cc
patch: @@ -246,7 +246,7 @@ void UtilityProcessWrapper::OnServiceProcessLaunch(
   EmitWithoutEvent("spawn");
 }
 
-void UtilityProcessWrapper::HandleTermination(uint64_t exit_code) {
+void UtilityProcessWrapper::HandleTermination(uint32_t exit_code) {
   // HandleTermination is called from multiple callsites,
   // we need to ensure we only process it for the first callsite.
   if (terminated_)
@@ -314,7 +314,7 @@ void UtilityProcessWrapper::CloseConnectorPort() {
   }
 }
 
-void UtilityProcessWrapper::Shutdown(uint64_t exit_code) {
+void UtilityProcessWrapper::Shutdown(uint32_t exit_code) {
   node_service_remote_.reset();
   HandleTermination(exit_code);
 }

filename: shell/browser/api/electron_api_utility_process.h
patch: @@ -58,7 +58,7 @@ class UtilityProcessWrapper final
   static gin_helper::Handle<UtilityProcessWrapper> Create(gin::Arguments* args);
   static raw_ptr<UtilityProcessWrapper> FromProcessId(base::ProcessId pid);
 
-  void Shutdown(uint64_t exit_code);
+  void Shutdown(uint32_t exit_code);
 
   // gin_helper::Wrappable
   static gin::DeprecatedWrapperInfo kWrapperInfo;
@@ -78,7 +78,7 @@ class UtilityProcessWrapper final
   void OnServiceProcessLaunch(const base::Process& process);
   void CloseConnectorPort();
 
-  void HandleTermination(uint64_t exit_code);
+  void HandleTermination(uint32_t exit_code);
 
   void PostMessage(gin::Arguments* args);
   bool Kill();

filename: spec/api-utility-process-spec.ts
patch: @@ -129,6 +129,22 @@ describe('utilityProcess module', () => {
       expect(code).to.equal(exitCode);
     });
 
+    ifit(process.platform === 'win32')('emits correct exit code when high bit is set on Windows', async () => {
+      // NTSTATUS code with high bit set should not be mangled by sign extension.
+      const exitCode = 0xC0000005;
+      const child = utilityProcess.fork(path.join(fixturesPath, 'custom-exit.js'), [`--exitCode=${exitCode}`]);
+      const [code] = await once(child, 'exit');
+      expect(code).to.equal(exitCode);
+    });
+
+    ifit(process.platform !== 'win32')('emits correct exit code when child process crashes on posix', async () => {
+      // Crash exit codes should not be sign-extended to large 64-bit values.
+      const child = utilityProcess.fork(path.join(fixturesPath, 'crash.js'));
+      const [code] = await once(child, 'exit');
+      expect(code).to.not.equal(0);
+      expect(code).to.be.lessThanOrEqual(0xFFFFFFFF);
+    });
+
     it('does not run JS after process.exit is called', async () => {
       const file = path.join(os.tmpdir(), `no-js-after-exit-log-${Math.random()}`);
       const child = utilityProcess.fork(path.join(fixturesPath, 'no-js-after-exit.js'), [`--testPath=${file}`]);
Repository: nodejs/node
Pull request: deps: update merve to 1.2.2

This is an automated update of merve to 1.2.2.

Changed files:
filename: deps/merve/merve.cpp
patch: @@ -1,4 +1,4 @@
-/* auto-generated on 2026-03-06 11:46:19 -0500. Do not edit! */
+/* auto-generated on 2026-03-11 12:53:21 -0400. Do not edit! */
 #include "merve.h"
 
 /* begin file src/parser.cpp */
@@ -1478,11 +1478,13 @@ class CJSLexer {
   }
 
   void tryBacktrackAddStarExportBinding(const char* bPos) {
-    while (*bPos == ' ' && bPos > source)
+    if (bPos < source) return;
+    while (bPos > source && *bPos == ' ')
       bPos--;
     if (*bPos == '=') {
+      if (bPos <= source) return;
       bPos--;
-      while (*bPos == ' ' && bPos > source)
+      while (bPos > source && *bPos == ' ')
         bPos--;
       const char* id_end = bPos;
       bool identifierStart = false;
@@ -1497,7 +1499,7 @@ class CJSLexer {
         if (starExportStack == STAR_EXPORT_STACK_END)
           return;
         starExportStack->id = std::string_view(bPos + 1, static_cast<size_t>(id_end - bPos));
-        while (*bPos == ' ' && bPos > source)
+        while (bPos > source && *bPos == ' ')
           bPos--;
         switch (*bPos) {
           case 'r':

filename: deps/merve/merve.h
patch: @@ -1,4 +1,4 @@
-/* auto-generated on 2026-03-06 11:46:19 -0500. Do not edit! */
+/* auto-generated on 2026-03-11 12:53:21 -0400. Do not edit! */
 /* begin file include/merve.h */
 #ifndef MERVE_H
 #define MERVE_H
@@ -15,14 +15,14 @@
 #ifndef MERVE_VERSION_H
 #define MERVE_VERSION_H
 
-#define MERVE_VERSION "1.2.0" // x-release-please-version
+#define MERVE_VERSION "1.2.2" // x-release-please-version
 
 namespace lexer {
 
 enum {
   MERVE_VERSION_MAJOR = 1,     // x-release-please-major
   MERVE_VERSION_MINOR = 2,     // x-release-please-minor
-  MERVE_VERSION_REVISION = 0,  // x-release-please-patch
+  MERVE_VERSION_REVISION = 2,  // x-release-please-patch
 };
 
 }  // namespace lexer
Repository: facebook/react
Pull request: [compiler] Remove useState argument constraint. no-derived-computations-in-effects


Summary:
I missed this conditional messing things up for undefined useState() calls. We should be tracking them.

I also missed a test that expect an error was not throwing.

Test Plan:
Update broken test

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/react/pull/35174).
* __->__ #35174
* #35173

Changed files:
filename: compiler/packages/babel-plugin-react-compiler/src/Validation/ValidateNoDerivedComputationsInEffects_exp.ts
patch: @@ -388,7 +388,7 @@ function recordInstructionDerivations(
           dependencies: deps,
         });
       }
-    } else if (isUseStateType(lvalue.identifier) && value.args.length > 0) {
+    } else if (isUseStateType(lvalue.identifier)) {
       typeOfValue = 'fromState';
       context.derivationCache.addDerivationEntry(
         lvalue,

filename: compiler/packages/babel-plugin-react-compiler/src/__tests__/fixtures/compiler/effect-derived-computations/usestate-derived-from-prop-no-show-in-data-flow-tree.expect.md
patch: @@ -64,6 +64,7 @@ function Component(t0) {
 ## Logs
 
 ```
+{"kind":"CompileError","detail":{"options":{"description":"Using an effect triggers an additional render which can hurt performance and user experience, potentially briefly showing stale values to the user\n\nThis setState call is setting a derived value that depends on the following reactive sources:\n\nState: [second]\n\nData Flow Tree:\n└── second (State)\n\nSee: https://react.dev/learn/you-might-not-need-an-effect#updating-state-based-on-props-or-state","category":"EffectDerivationsOfState","reason":"You might not need an effect. Derive values in render, not effects.","details":[{"kind":"error","loc":{"start":{"line":14,"column":4,"index":443},"end":{"line":14,"column":8,"index":447},"filename":"usestate-derived-from-prop-no-show-in-data-flow-tree.ts","identifierName":"setS"},"message":"This should be computed during render, not in an effect"}]}},"fnLoc":null}
 {"kind":"CompileSuccess","fnLoc":{"start":{"line":3,"column":0,"index":64},"end":{"line":18,"column":1,"index":500},"filename":"usestate-derived-from-prop-no-show-in-data-flow-tree.ts"},"fnName":"Component","memoSlots":5,"memoBlocks":2,"memoValues":3,"prunedMemoBlocks":0,"prunedMemoValues":0}
 ```
Repository: nodejs/node
Pull request: doc: remove outdated Chrome 66 and ndb references from debugger

Remove outdated references from `doc/api/debugger.md`:

- Chrome 66.0.3345.0 version check from 2018 (irrelevant for any currently supported Chrome)
- "yet" language about worker thread debugging support in Chrome DevTools (supported since 2020)
- Recommendation to use [ndb](https://github.com/GoogleChromeLabs/ndb/), which has been archived

Fixes: https://github.com/nodejs/node/issues/54078

Changed files:
filename: doc/api/debugger.md
patch: @@ -260,13 +260,5 @@ For help, see: https://nodejs.org/en/docs/inspector
 at the end of the URL is generated on the fly, it varies in different
 debugging sessions.)
 
-If the Chrome browser is older than 66.0.3345.0,
-use `inspector.html` instead of `js_app.html` in the above URL.
-
-Chrome DevTools doesn't support debugging [worker threads][] yet.
-[ndb][] can be used to debug them.
-
 [Chrome DevTools Protocol]: https://chromedevtools.github.io/devtools-protocol/
 [`debugger`]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/debugger
-[ndb]: https://github.com/GoogleChromeLabs/ndb/
-[worker threads]: worker_threads.md
Repository: ollama/ollama
Pull request: app/ui: add swift syntax highlighting support

Fixes #13476

Adds Swift as a supported language in the syntax highlighter for code blocks in the chat UI.

Changed files:
filename: app/ui/app/src/lib/highlighter.ts
patch: @@ -147,6 +147,7 @@ export const highlighterPromise = createHighlighter({
     "c",
     "cpp",
     "sql",
+    "swift",
     "yaml",
     "markdown",
   ],
Repository: electron/electron
Pull request: ci: output build cache hit rate as GHA annotation

Backport of #50361

See that PR for details.


Notes: none <!-- Please add a one-line description for app developers to read in the release notes, or 'none' if no notes relevant to app developers. Examples and help on special cases: https://github.com/electron/clerk/blob/main/README.md#examples -->


Changed files:
filename: script/build-stats.mjs
patch: @@ -32,7 +32,8 @@ async function main () {
   }));
   const hitRate = stats.CacheHit / (stats.Remote + stats.CacheHit + stats.LocalFallback);
 
-  console.log(`Effective cache hit rate: ${(hitRate * 100).toFixed(2)}%`);
+  const messagePrefix = process.env.GITHUB_ACTIONS ? '::notice title=Build Stats::' : '';
+  console.log(`${messagePrefix}Effective cache hit rate: ${(hitRate * 100).toFixed(2)}%`);
 
   if (uploadStats) {
     if (!process.env.DD_API_KEY) {
Repository: nodejs/node
Pull request: doc: add throwIfNoEntry version history to fs.stat

Add missing YAML version history entries for the `throwIfNoEntry` option on `fs.stat` and `fsPromises.stat`, added in v25.7.0 via #61178.

The synchronous variants (`fs.statSync`, `fsPromises.statSync`) already had entries from #33716. This brings the async/callback and promise-based APIs in line.

Verified: `git tag --contains 4cecbe1f53f` → v25.7.0.

Fixes: https://github.com/nodejs/node/issues/62185

Changed files:
filename: doc/api/fs.md
patch: @@ -1709,6 +1709,10 @@ Removes files and directories (modeled on the standard POSIX `rm` utility).
 <!-- YAML
 added: v10.0.0
 changes:
+  - version: v25.7.0
+    pr-url: https://github.com/nodejs/node/pull/61178
+    description: Accepts a `throwIfNoEntry` option to specify whether
+                 an exception should be thrown if the entry does not exist.
   - version: v10.5.0
     pr-url: https://github.com/nodejs/node/pull/20220
     description: Accepts an additional `options` object to specify whether
@@ -4412,6 +4416,10 @@ completion callback.
 <!-- YAML
 added: v0.0.2
 changes:
+  - version: v25.7.0
+    pr-url: https://github.com/nodejs/node/pull/61178
+    description: Accepts a `throwIfNoEntry` option to specify whether
+                 an exception should be thrown if the entry does not exist.
   - version: v18.0.0
     pr-url: https://github.com/nodejs/node/pull/41678
     description: Passing an invalid callback to the `callback` argument
Repository: rust-lang/rust
Pull request: Notify stdarch maintainers on changes in std_detect

cc @Amanieu @folkertdev @Kobzol

It would be nice to be notified when std_detect changes, as it is spiritually a part of stdarch.

Also assign @rust-lang/libs to std_detect

Changed files:
filename: triagebot.toml
patch: @@ -1073,6 +1073,10 @@ instead.
 """
 cc = ["@Amanieu", "@folkertdev", "@sayantn"]
 
+[mentions."library/std_detect"]
+message = "Some changes occurred in `std_detect`"
+cc = ["@Amanieu", "@folkertdev", "@sayantn"]
+
 [mentions."library/core/src/intrinsics/simd.rs"]
 message = """
 Some changes occurred to the platform-builtins intrinsics. Make sure the
@@ -1601,6 +1605,7 @@ dep-bumps = [
 "/library/std" =                                         ["libs", "@ChrisDenton"]
 "/library/std/src/sys/pal/windows" =                     ["@ChrisDenton"]
 "/library/stdarch" =                                     ["libs"]
+"/library/std_detect" =                                  ["libs"]
 "/library/test" =                                        ["libs"]
 "/src/bootstrap" =                                       ["bootstrap"]
 "/src/ci" =                                              ["infra-ci"]
Repository: vercel/next.js
Pull request: [test] Deflake instant-navs-devtools

Apparently `timeout` is racing Playwright's IPC communication. I've never seen `*instant` selectors timeout though so I'm just increasing the `*instant` timeout for this specific test. Chances are the problem is some (css) transition that didn't finish in time.

Changed files:
filename: test/development/app-dir/instant-navs-devtools/instant-navs-devtools.test.ts
patch: @@ -38,7 +38,10 @@ describe('instant-nav-panel', () => {
   }
 
   async function clickStartClientNav(browser: Playwright) {
-    await browser.elementByCssInstant('[data-instant-nav-client]').click()
+    await browser
+      // TODO: Monitor if we need to increase timeouts for all *instant calls
+      .elementByCss('[data-instant-nav-client]', { timeout: 50 })
+      .click()
     await waitForInstantModeCookie(browser)
   }