From bcae9972a41de352a33b132a1815619fcf1093f4 Mon Sep 17 00:00:00 2001
From: Kyungrae Kang <148605113+kyungrae2002@users.noreply.github.com>
Date: Mon, 7 Sep 2026 19:25:14 +0900
Subject: [PATCH] sqlite: use cached strings for property keys

Signed-off-by: Kyungrae Kang <kyungrae2002@gmail.com>
Assisted-by: Codex
---
 src/node_sqlite.cc | 24 ++++++------------------
 1 file changed, 6 insertions(+), 18 deletions(-)

diff --git a/src/node_sqlite.cc b/src/node_sqlite.cc
index 3234d9881b51..546ac2ccc234 100644
--- a/src/node_sqlite.cc
+++ b/src/node_sqlite.cc
@@ -2171,9 +2171,7 @@ void Database::Prepare(const FunctionCallbackInfo<Value>& args) {
     Local<Object> options = args[1].As<Object>();
 
     Local<Value> return_arrays_v;
-    if (!options
-             ->Get(env->context(),
-                   FIXED_ONE_BYTE_STRING(env->isolate(), "returnArrays"))
+    if (!options->Get(env->context(), env->return_arrays_string())
              .ToLocal(&return_arrays_v)) {
       return;
     }
@@ -2188,9 +2186,7 @@ void Database::Prepare(const FunctionCallbackInfo<Value>& args) {
     }
 
     Local<Value> read_big_ints_v;
-    if (!options
-             ->Get(env->context(),
-                   FIXED_ONE_BYTE_STRING(env->isolate(), "readBigInts"))
+    if (!options->Get(env->context(), env->read_bigints_string())
              .ToLocal(&read_big_ints_v)) {
       return;
     }
@@ -2205,10 +2201,7 @@ void Database::Prepare(const FunctionCallbackInfo<Value>& args) {
     }
 
     Local<Value> allow_bare_named_params_v;
-    if (!options
-             ->Get(env->context(),
-                   FIXED_ONE_BYTE_STRING(env->isolate(),
-                                         "allowBareNamedParameters"))
+    if (!options->Get(env->context(), env->allow_bare_named_params_string())
              .ToLocal(&allow_bare_named_params_v)) {
       return;
     }
@@ -2224,10 +2217,7 @@ void Database::Prepare(const FunctionCallbackInfo<Value>& args) {
     }
 
     Local<Value> allow_unknown_named_params_v;
-    if (!options
-             ->Get(env->context(),
-                   FIXED_ONE_BYTE_STRING(env->isolate(),
-                                         "allowUnknownNamedParameters"))
+    if (!options->Get(env->context(), env->allow_unknown_named_params_string())
              .ToLocal(&allow_unknown_named_params_v)) {
       return;
     }
@@ -5338,10 +5328,8 @@ static void Initialize(Local<Object> target,
                           db_tmpl,
                           FIXED_ONE_BYTE_STRING(isolate, "isTransaction"),
                           Database::IsTransactionGetter);
-  SetSideEffectFreeGetter(isolate,
-                          db_tmpl,
-                          FIXED_ONE_BYTE_STRING(isolate, "limits"),
-                          Database::LimitsGetter);
+  SetSideEffectFreeGetter(
+      isolate, db_tmpl, env->limits_string(), Database::LimitsGetter);
   Local<String> sqlite_type_key = FIXED_ONE_BYTE_STRING(isolate, "sqlite-type");
   Local<v8::Symbol> sqlite_type_symbol =
       v8::Symbol::For(isolate, sqlite_type_key);