helloadhavan/github_issues huggingface dataset:
Repository: denoland/deno
Pull request: fix(ext/node): fix missing privateKey.x in curve25519 JWK

Fixes https://github.com/denoland/deno/issues/27972


Changed files:
filename: ext/node/ops/crypto/keys.rs
patch: @@ -1463,19 +1463,28 @@ impl AsymmetricPrivateKey {
       AsymmetricPrivateKey::X25519(static_secret) => {
         let bytes = static_secret.to_bytes();
 
+        let AsymmetricPublicKey::X25519(x) = self.to_public_key() else {
+          unreachable!();
+        };
+
         Ok(deno_core::serde_json::json!({
-            "kty": "OKP",
             "crv": "X25519",
+            "x": bytes_to_b64(x.as_bytes()),
             "d": bytes_to_b64(&bytes),
+            "kty": "OKP",
         }))
       }
       AsymmetricPrivateKey::Ed25519(key) => {
         let bytes = key.to_bytes();
+        let AsymmetricPublicKey::Ed25519(x) = self.to_public_key() else {
+          unreachable!();
+        };
 
         Ok(deno_core::serde_json::json!({
-            "kty": "OKP",
             "crv": "Ed25519",
+            "x": bytes_to_b64(x.as_bytes()),
             "d": bytes_to_b64(&bytes),
+            "kty": "OKP",
         }))
       }
       _ => Err(AsymmetricPrivateKeyJwkError::JwkExportNotImplementedForKeyType),

filename: tests/unit_node/crypto/crypto_key_test.ts
patch: @@ -755,3 +755,16 @@ Deno.test("X509Certificate checkHost", function () {
   assertEquals(cert.checkHost("www.google.com"), undefined);
   assertEquals(cert.checkHost("agent1"), "agent1");
 });
+
+// https://github.com/denoland/deno/issues/27972
+Deno.test("curve25519 generate valid private jwk", function () {
+  const { publicKey, privateKey } = generateKeyPairSync("ed25519", {
+    publicKeyEncoding: { format: "jwk" },
+    privateKeyEncoding: { format: "jwk" },
+  });
+
+  // @ts-ignore @types/node broken
+  assert(!publicKey.d);
+  // @ts-ignore @types/node broken
+  assert(privateKey.d);
+});
Repository: facebook/react
Pull request: Gate DevTools test to fix CI

This test started failing recently in older versions of React because the Scheduler priority inside a microtask is Normal instead of Immediate. This is expected because microtasks are not Scheduler tasks; it's an implementation detail.

I gated the test to only run in v17 because it's a regression test for legacy Suspense behavior, and the implementation details of the snapshot changed in v18.

Test plan
---------

Using latest:

```
yarn test --build --project devtools --release-channel=experimental profilingcache
```

Using v17 (typically runs in a timed CI workflow):

```
/scripts/circleci/download_devtools_regression_build.js 17.0 --replaceBuild
yarn test --build --project devtools --release-channel=experimental --reactVersion 17.0 profilingcache
```

Changed files:
filename: packages/react-devtools-shared/src/__tests__/profilingCache-test.js
patch: @@ -938,8 +938,10 @@ describe('ProfilingCache', () => {
     }
   });
 
-  // @reactVersion >= 18.0
+  // @reactVersion = 17.0
   it('should handle unexpectedly shallow suspense trees', () => {
+    // This test only runs in v17 because it's a regression test for legacy
+    // Suspense behavior, and the implementation details changed in v18.
     const container = document.createElement('div');
 
     utils.act(() => store.profilerStore.startProfiling());
@@ -965,15 +967,7 @@ describe('ProfilingCache', () => {
           "passiveEffectDuration": null,
           "priorityLevel": "Normal",
           "timestamp": 0,
-          "updaters": [
-            {
-              "displayName": "render()",
-              "hocDisplayNames": null,
-              "id": 1,
-              "key": null,
-              "type": 11,
-            },
-          ],
+          "updaters": null,
         },
       ]
     `);

filename: scripts/jest/jest-cli.js
patch: @@ -16,6 +16,8 @@ const devToolsConfig = './scripts/jest/config.build-devtools.js';
 const persistentConfig = './scripts/jest/config.source-persistent.js';
 const buildConfig = './scripts/jest/config.build.js';
 
+const {ReactVersion} = require('../../ReactVersions');
+
 const argv = yargs
   .parserConfiguration({
     // Important: This option tells yargs to move all other options not
@@ -179,9 +181,13 @@ function validateOptions() {
       success = false;
     }
 
-    if (argv.reactVersion && !semver.validRange(argv.reactVersion)) {
-      success = false;
-      logError('please specify a valid version range for --reactVersion');
+    if (argv.reactVersion) {
+      if (!semver.validRange(argv.reactVersion)) {
+        success = false;
+        logError('please specify a valid version range for --reactVersion');
+      }
+    } else {
+      argv.reactVersion = ReactVersion;
     }
   } else {
     if (argv.compactConsole) {
Repository: electron/electron
Pull request: fix: add missing index arg in `navigationHistory.canGoToOffset`

Backport of #44989

See that PR for details.


Notes: Fixed issue with missing index arg for `webContents.canGoToOffset`


Changed files:
filename: lib/browser/api/web-contents.ts
patch: @@ -514,9 +514,9 @@ WebContents.prototype.canGoForward = function () {
 };
 
 const canGoToOffsetDeprecated = deprecate.warnOnce('webContents.canGoToOffset', 'webContents.navigationHistory.canGoToOffset');
-WebContents.prototype.canGoToOffset = function () {
+WebContents.prototype.canGoToOffset = function (index: number) {
   canGoToOffsetDeprecated();
-  return this._canGoToOffset();
+  return this._canGoToOffset(index);
 };
 
 const clearHistoryDeprecated = deprecate.warnOnce('webContents.clearHistory', 'webContents.navigationHistory.clear');

filename: typings/internal-electron.d.ts
patch: @@ -93,7 +93,7 @@ declare namespace Electron {
     _historyLength(): number;
     _canGoBack(): boolean;
     _canGoForward(): boolean;
-    _canGoToOffset(): boolean;
+    _canGoToOffset(index: number): boolean;
     _goBack(): void;
     _goForward(): void;
     _goToOffset(index: number): void;
Repository: denoland/deno
Pull request: fix(ext/node): throw Session methods when database is closed


Changed files:
filename: ext/node/ops/sqlite/database.rs
patch: @@ -376,7 +376,7 @@ impl DatabaseSync {
     Ok(Session {
       inner: raw_session,
       freed: Cell::new(false),
-      _db: self.conn.clone(),
+      db: self.conn.clone(),
     })
   }
 }

filename: ext/node/ops/sqlite/session.rs
patch: @@ -24,7 +24,7 @@ pub struct Session {
   pub(crate) freed: Cell<bool>,
 
   // Hold a strong reference to the database.
-  pub(crate) _db: Rc<RefCell<Option<rusqlite::Connection>>>,
+  pub(crate) db: Rc<RefCell<Option<rusqlite::Connection>>>,
 }
 
 impl GarbageCollected for Session {}
@@ -57,6 +57,10 @@ impl Session {
   // Closes the session.
   #[fast]
   fn close(&self) -> Result<(), SqliteError> {
+    if self.db.borrow().is_none() {
+      return Err(SqliteError::AlreadyClosed);
+    }
+
     self.delete()
   }
 
@@ -66,6 +70,9 @@ impl Session {
   // This method is a wrapper around `sqlite3session_changeset()`.
   #[buffer]
   fn changeset(&self) -> Result<Box<[u8]>, SqliteError> {
+    if self.db.borrow().is_none() {
+      return Err(SqliteError::AlreadyClosed);
+    }
     if self.freed.get() {
       return Err(SqliteError::SessionClosed);
     }
@@ -78,6 +85,9 @@ impl Session {
   // This method is a wrapper around `sqlite3session_patchset()`.
   #[buffer]
   fn patchset(&self) -> Result<Box<[u8]>, SqliteError> {
+    if self.db.borrow().is_none() {
+      return Err(SqliteError::AlreadyClosed);
+    }
     if self.freed.get() {
       return Err(SqliteError::SessionClosed);
     }

filename: tests/unit_node/sqlite_test.ts
patch: @@ -96,6 +96,9 @@ Deno.test("[node/sqlite] createSession and changesets", () => {
   assertThrows(() => session.changeset(), Error, "Session is already closed");
   // Close after close should throw.
   assertThrows(() => session.close(), Error, "Session is already closed");
+
+  db.close();
+  assertThrows(() => session.close(), Error, "Database is already closed");
 });
 
 Deno.test("[node/sqlite] StatementSync integer too large", () => {
Repository: ggml-org/llama.cpp
Pull request: llama : suppress unref var in Windows MSVC

This commit suppresses two warnings that are currently generated for src/llama.cpp when building on Windows MSVC

```console
C:\llama.cpp\src\llama.cpp(14349,45): warning C4101: 'ex': unreferenced local variable [C:\llama.cpp\build\src\llama.vcxproj]
C:\llama.cpp\src\llama.cpp(19285,44): warning C4101: 'e': unreferenced local variable [C:\llama.cpp\build\src\llama.vcxproj]
```



- [x] I have read the [contributing guidelines](https://github.com/ggerganov/llama.cpp/blob/master/CONTRIBUTING.md)
- Self-reported review complexity:
  - [x] Low
  - [ ] Medium
  - [ ] High


Changed files:
filename: src/llama.cpp
patch: @@ -14781,7 +14781,7 @@ struct llm_tokenizer_ugm {
                 size_t prefix_offset = input_offset;
                 unicode_cpt_from_utf8(input, prefix_offset);
                 return { &input[input_offset], prefix_offset - input_offset, prefix_offset - input_offset };
-            } catch(std::invalid_argument & ex) {
+            } catch (std::invalid_argument & /*ex*/) {
                 // if no, consume 1 byte and return U+FFFD - REPLACEMENT CHARACTER
                 return { "\xEF\xBF\xBD", 3, 1 };
             }
@@ -19725,7 +19725,7 @@ static std::string llama_decode_text(const std::string & text) {
         const auto utf8 = unicode_cpt_to_utf8(cpt);
         try {
             decoded_text += unicode_utf8_to_byte(utf8);
-        } catch (const std::out_of_range & e) {
+        } catch (const std::out_of_range & /*e*/) {
             decoded_text += "[UNK_BYTE_0x";
             for (const auto c : utf8) {
                 decoded_text += format("%02x", (uint8_t) c);
Repository: facebook/react
Pull request: Remove unused `initialStatus` parameter from `useHostTransitionStatus`



## Summary

An unlimited `$FlowFixme` was masking an incompatible call of `useHostTransition` without arguments. Looks like `useHostTransition` never uses any arguments so this seems save to remove.

Feel free to close if this is planned, but not implemented yet. 

## How did you test this change?

- `yarn flow dom-node`


Changed files:
filename: packages/react-dom-bindings/src/shared/ReactDOMFormActions.js
patch: @@ -70,7 +70,7 @@ export function useFormStatus(): FormStatus {
     throw new Error('Not implemented.');
   } else {
     const dispatcher = resolveDispatcher();
-    // $FlowFixMe We know this exists because of the feature check above.
+    // $FlowFixMe[not-a-function] We know this exists because of the feature check above.
     return dispatcher.useHostTransitionStatus();
   }
 }

filename: packages/react-reconciler/src/ReactInternalTypes.js
patch: @@ -422,9 +422,7 @@ export type Dispatcher = {
   useId(): string,
   useCacheRefresh?: () => <T>(?() => T, ?T) => void,
   useMemoCache?: (size: number) => Array<any>,
-  useHostTransitionStatus?: (
-    initialStatus: TransitionStatus,
-  ) => TransitionStatus,
+  useHostTransitionStatus?: () => TransitionStatus,
 };
 
 export type CacheDispatcher = {
Repository: vercel/next.js
Pull request: Create Jest project for Turbopack

This allows running a tests for both bundlers via
`pnpm test-dev test/e2e/app --projects jest.config.*`.

Ideally, we'd have a `test-dev-all` shortcut.
However, we can't add the test files after the `--projects` flag since it's now ambiguous whether you specified a test or project config. So the `--projects` flag must come after the specified test making an NPM script using `--projects` impossible.

This is mostly helpful if you want to update snapshots that are different between bundlers.

Changed files:
filename: contributing/core/testing.md
patch: @@ -115,6 +115,12 @@ To run the test suite using Turbopack, you can use the `TURBOPACK=1` environment
 TURBOPACK=1 pnpm test-dev test/e2e/app-dir/app/
 ```
 
+If you want to run a test again both Turbopack and Webpack, use Jest's `--projects` flag:
+
+```sh
+pnpm test-dev test/e2e/app-dir/app/ --projects jest.config.*
+```
+
 ## Integration testing outside the repository with local builds
 
 You can locally generate tarballs for each package in this repository with:

filename: jest.config.js
patch: @@ -5,6 +5,7 @@ const createJestConfig = nextJest()
 // Any custom config you want to pass to Jest
 /** @type {import('jest').Config} */
 const customJestConfig = {
+  displayName: 'default',
   testMatch: ['**/*.test.js', '**/*.test.ts', '**/*.test.jsx', '**/*.test.tsx'],
   setupFilesAfterEnv: ['<rootDir>/jest-setup-after-env.ts'],
   verbose: true,

filename: jest.config.turbopack.js
patch: @@ -0,0 +1,16 @@
+const createJestDefaultConfig = require('./jest.config.js')
+
+module.exports = async function createConfig() {
+  const jestDefaultConfig = await createJestDefaultConfig()
+  /** @type {import('jest').Config} */
+  const customConfig = {
+    ...jestDefaultConfig,
+    displayName: 'Turbopack',
+    setupFiles: [
+      ...(jestDefaultConfig.setupFiles ?? []),
+      '<rootDir>/jest-setup-files.turbopack.js',
+    ],
+  }
+
+  return customConfig
+}

filename: test/jest-setup-files.turbopack.js
patch: @@ -0,0 +1 @@
+process.env.TURBOPACK = '1'
Repository: electron/electron
Pull request: fix: add missing index arg in `navigationHistory.canGoToOffset`

#### Description of Change
This PR:
- [x] Adds missing index arg for the deprecated version of `webContents.canGoToOffset`

<!--
Thank you for your Pull Request. Please provide a description above and review
the requirements below.

Contributors guide: https://github.com/electron/electron/blob/main/CONTRIBUTING.md
-->

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd
- [x] `npm test` passes
- [x] tests are [changed or added](https://github.com/electron/electron/blob/main/docs/development/testing.md)
- [x] relevant documentation, tutorials, templates and examples are changed or added
- [x] [PR release notes](https://github.com/electron/clerk/blob/main/README.md) describe the change in a way relevant to app developers, and are [capitalized, punctuated, and past tense](https://github.com/electron/clerk/blob/main/README.md#examples).

#### Release Notes

Notes: Fixed issue with missing index arg for `webContents.canGoToOffset`


Changed files:
filename: lib/browser/api/web-contents.ts
patch: @@ -514,9 +514,9 @@ WebContents.prototype.canGoForward = function () {
 };
 
 const canGoToOffsetDeprecated = deprecate.warnOnce('webContents.canGoToOffset', 'webContents.navigationHistory.canGoToOffset');
-WebContents.prototype.canGoToOffset = function () {
+WebContents.prototype.canGoToOffset = function (index: number) {
   canGoToOffsetDeprecated();
-  return this._canGoToOffset();
+  return this._canGoToOffset(index);
 };
 
 const clearHistoryDeprecated = deprecate.warnOnce('webContents.clearHistory', 'webContents.navigationHistory.clear');

filename: typings/internal-electron.d.ts
patch: @@ -93,7 +93,7 @@ declare namespace Electron {
     _historyLength(): number;
     _canGoBack(): boolean;
     _canGoForward(): boolean;
-    _canGoToOffset(): boolean;
+    _canGoToOffset(index: number): boolean;
     _goBack(): void;
     _goForward(): void;
     _goToOffset(index: number): void;
Repository: ggml-org/llama.cpp
Pull request: llama.swiftui : fix xcframework dir in README [no ci]

This commit fixes the path to the `xcframework` in the README file which I had forgotten to change after renaming the build directory.


Changed files:
filename: examples/llama.swiftui/README.md
patch: @@ -16,7 +16,7 @@ Open `llama.swiftui.xcodeproj` project in Xcode and you should be able to build
 a simulator or a real device.
 
 To use the framework with a different project, the XCFramework can be added to the project by
-adding `build-ios/llama.xcframework` by dragging and dropping it into the project navigator, or
+adding `build-apple/llama.xcframework` by dragging and dropping it into the project navigator, or
 by manually selecting the framework in the "Frameworks, Libraries, and Embedded Content" section
 of the project settings.
Repository: nodejs/node
Pull request: doc: clarify `unhandledRejection` events behaviors in process doc

Fixes https://github.com/nodejs/node/issues/50075

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: doc/api/process.md
patch: @@ -580,6 +580,10 @@ address such failures, a non-operational
 `resource.loaded`, which would prevent the `'unhandledRejection'` event from
 being emitted.
 
+If an `'unhandledRejection'` event is emitted but not handled it will
+be raised as an uncaught exception. This alongside other behaviors of
+`'unhandledRejection'` events can changed via the [`--unhandled-rejections`][] flag.
+
 ### Event: `'warning'`
 
 <!-- YAML
Repository: denoland/deno
Pull request: fix(ext/napi): napi_is_buffer tests for ArrayBufferView

use correct type check

Fixes: https://github.com/denoland/deno/issues/27951

Changed files:
filename: ext/napi/node_api.rs
patch: @@ -400,18 +400,8 @@ fn napi_is_buffer(
   check_arg!(env, value);
   check_arg!(env, result);
 
-  let buffer_constructor =
-    v8::Local::new(&mut env.scope(), &env.buffer_constructor);
-
-  let Some(is_buffer) = value
-    .unwrap()
-    .instance_of(&mut env.scope(), buffer_constructor.into())
-  else {
-    return napi_set_last_error(env, napi_generic_failure);
-  };
-
   unsafe {
-    *result = is_buffer;
+    *result = value.unwrap().is_array_buffer_view();
   }
 
   napi_clear_last_error(env)

filename: tests/napi/src/typedarray.rs
patch: @@ -144,10 +144,26 @@ extern "C" fn test_external(
   typedarray
 }
 
+extern "C" fn test_is_buffer(
+  env: napi_env,
+  info: napi_callback_info,
+) -> napi_value {
+  let (args, argc, _) = napi_get_callback_info!(env, info, 1);
+  assert_eq!(argc, 1);
+
+  let mut is_buffer: bool = false;
+  assert_napi_ok!(napi_is_buffer(env, args[0], &mut is_buffer));
+
+  let mut result: napi_value = std::ptr::null_mut();
+  assert_napi_ok!(napi_get_boolean(env, is_buffer, &mut result));
+  result
+}
+
 pub fn init(env: napi_env, exports: napi_value) {
   let properties = &[
     napi_new_property!(env, "test_external", test_external),
     napi_new_property!(env, "test_multiply", test_multiply),
+    napi_new_property!(env, "test_is_buffer", test_is_buffer),
   ];
 
   assert_napi_ok!(napi_define_properties(

filename: tests/napi/typedarray_test.js
patch: @@ -1,5 +1,6 @@
 // Copyright 2018-2025 the Deno authors. MIT license.
 
+import { Buffer } from "node:buffer";
 import { assert, assertEquals, loadTestLibrary } from "./common.js";
 
 const typedarray = loadTestLibrary();
@@ -28,6 +29,14 @@ Deno.test("napi typedarray float64", function () {
   assertEquals(Math.round(10 * doubleResult[2]) / 10, -6.6);
 });
 
+Deno.test("napi_is_buffer", () => {
+  assert(!typedarray.test_is_buffer(5));
+  assert(!typedarray.test_is_buffer([]));
+  assert(typedarray.test_is_buffer(new Uint8Array()));
+  assert(typedarray.test_is_buffer(new Uint32Array()));
+  assert(typedarray.test_is_buffer(new Buffer([])));
+});
+
 // TODO(bartlomieju): this test causes segfaults when used with jemalloc.
 // Node documentation provides a hint that this function is not supported by
 // other runtime like electron.
Repository: electron/electron
Pull request: fix: add missing index arg in `navigationHistory.canGoToOffset`

Backport of #44989

See that PR for details.


Notes: Fixed issue with missing index arg for `webContents.canGoToOffset`


Changed files:
filename: lib/browser/api/web-contents.ts
patch: @@ -523,9 +523,9 @@ WebContents.prototype.canGoForward = function () {
 };
 
 const canGoToOffsetDeprecated = deprecate.warnOnce('webContents.canGoToOffset', 'webContents.navigationHistory.canGoToOffset');
-WebContents.prototype.canGoToOffset = function () {
+WebContents.prototype.canGoToOffset = function (index: number) {
   canGoToOffsetDeprecated();
-  return this._canGoToOffset();
+  return this._canGoToOffset(index);
 };
 
 const clearHistoryDeprecated = deprecate.warnOnce('webContents.clearHistory', 'webContents.navigationHistory.clear');

filename: typings/internal-electron.d.ts
patch: @@ -91,7 +91,7 @@ declare namespace Electron {
     _historyLength(): number;
     _canGoBack(): boolean;
     _canGoForward(): boolean;
-    _canGoToOffset(): boolean;
+    _canGoToOffset(index: number): boolean;
     _goBack(): void;
     _goForward(): void;
     _goToOffset(index: number): void;
Repository: denoland/deno
Pull request: fix(ext/sqlite): add `sourceSQL` and `expandedSQL` getters


Changed files:
filename: ext/node/ops/sqlite/mod.rs
patch: @@ -42,6 +42,9 @@ pub enum SqliteError {
   #[class(generic)]
   #[error("Invalid constructor")]
   InvalidConstructor,
+  #[class(generic)]
+  #[error("Expanded SQL text would exceed configured limits")]
+  InvalidExpandedSql,
   #[class(range)]
   #[error("The value of column {0} is too large to be represented as a JavaScript number: {1}")]
   NumberTooLarge(i32, i64),

filename: ext/node/ops/sqlite/statement.rs
patch: @@ -373,4 +373,38 @@ impl StatementSync {
   fn set_read_big_ints(&self, enabled: bool) {
     self.use_big_ints.set(enabled);
   }
+
+  #[getter]
+  #[rename("sourceSQL")]
+  #[string]
+  fn source_sql(&self) -> String {
+    // SAFETY: `self.inner` is a valid pointer to a sqlite3_stmt
+    // as it lives as long as the StatementSync instance.
+    unsafe {
+      let raw = ffi::sqlite3_sql(self.inner);
+      std::ffi::CStr::from_ptr(raw as _)
+        .to_string_lossy()
+        .into_owned()
+    }
+  }
+
+  #[getter]
+  #[rename("expandedSQL")]
+  #[string]
+  fn expanded_sql(&self) -> Result<String, SqliteError> {
+    // SAFETY: `self.inner` is a valid pointer to a sqlite3_stmt
+    // as it lives as long as the StatementSync instance.
+    unsafe {
+      let raw = ffi::sqlite3_expanded_sql(self.inner);
+      if raw.is_null() {
+        return Err(SqliteError::InvalidExpandedSql);
+      }
+      let sql = std::ffi::CStr::from_ptr(raw as _)
+        .to_string_lossy()
+        .into_owned();
+      ffi::sqlite3_free(raw as _);
+
+      Ok(sql)
+    }
+  }
 }

filename: tests/unit_node/sqlite_test.ts
patch: @@ -72,6 +72,9 @@ Deno.test("[node/sqlite] StatementSync read bigints are supported", () => {
 
   stmt.setReadBigInts(true);
   assertEquals(stmt.get(), { key: 1n, __proto__: null });
+
+  assertEquals(stmt.sourceSQL, "SELECT * FROM data");
+  assertEquals(stmt.expandedSQL, "SELECT * FROM data");
 });
 
 Deno.test("[node/sqlite] StatementSync integer too large", () => {