From 71ced8fb246cccadc79d91622ab9314b28d0fc7f Mon Sep 17 00:00:00 2001
From: wongtimothy147-lgtm <wongtimothy147-lgtm@users.noreply.github.com>
Date: Tue, 21 Apr 2026 22:32:03 +1000
Subject: [PATCH] fix(#52634): ContextifyContext::PropertyDefinerCallback()
 fails

Automated fix for #52634.
See PR body for full analysis, test results, and AI disclosure.

AI-Assisted: true
Model: holo3-35b-a3b
---
 src/contextify.cc | 54 +++++++++++++++++++++++++++++++++++++++++++++++
 src/contextify.h  | 14 ++++++++++++
 2 files changed, 68 insertions(+)
 create mode 100644 src/contextify.cc
 create mode 100644 src/contextify.h

diff --git a/src/contextify.cc b/src/contextify.cc
new file mode 100644
index 00000000000000..0a2439f4be0808
--- /dev/null
+++ b/src/contextify.cc
@@ -0,0 +1,54 @@
+#include "contextify.h"
+#include <string>
+
+void ContextifyContext::Initialize(v8::Local<v8::Object> target) {
+    v8::Local<v8::FunctionTemplate> t = Nan::New<v8::FunctionTemplate>(Nan::New<v8::FunctionTemplate>([](const v8::FunctionCallbackInfo<v8::Value>& args) {
+        if (args.IsConstructCall()) {
+            ContextifyContext* context = new ContextifyContext();
+            context->Wrap(args.This());
+            Nan::SetPrototypeMethod(args.Callee(), "createContext", createContext);
+            Nan::SetPrototypeMethod(args.Callee(), "definePropertyOnContext", definePropertyOnContext);
+        }
+    }));
+
+    v8::Local<v8::FunctionTemplate> proto = Nan::New<v8::FunctionTemplate>([](const v8::FunctionCallbackInfo<v8::Value>& args) {
+        if (args.IsConstructCall()) {
+            ContextifyContext* context = new ContextifyContext();
+            context->Wrap(args.This());
+            Nan::SetPrototypeMethod(args.Callee(), "createContext", createContext);
+            Nan::SetPrototypeMethod(args.Callee(), "definePropertyOnContext", definePropertyOnContext);
+        }
+    });
+
+    Nan::Set(target, Nan::New("ContextifyContext").ToLocalChecked(), t->GetFunction());
+}
+
+v8::Local<v8::Context> ContextifyContext::CreateContext(v8::Local<v8::Object> sandbox) {
+    v8::Local<v8::Context> context = v8::Context::New(Nan::GetCurrentContext()->GetIsolate());
+    v8::Local<v8::Object> global = context->Global();
+
+    // Copy properties from sandbox to context
+    v8::Local<v8::Array> props = sandbox->GetPropertyNames();
+    for (uint32_t i = 0; i < props->Length(); i++) {
+        v8::Local<v8::Value> prop = props->Get(i);
+        v8::Local<v8::Value> value = sandbox->Get(prop);
+        context->Global()->Set(context, prop, value);
+    }
+
+    return context;
+}
+
+void ContextifyContext::PropertyDefinerCallback(v8::Local<v8::String> property, const v8::PropertyCallbackInfo<v8::Value>& info) {
+    v8::Isolate* isolate = info.GetIsolate();
+    v8::Local<v8::Context> context = isolate->GetCurrentContext();
+    v8::Local<v8::Object> global = context->Global();
+
+    // Check if property exists in global
+    if (global->Has(context, property).FromJust()) {
+        v8::Local<v8::Value> value = global->Get(context, property).ToLocalChecked();
+        info.GetReturnValue().Set(value);
+    } else {
+        // Intercept the request and return undefined
+        info.GetReturnValue().SetUndefined();
+    }
+}
\ No newline at end of file
diff --git a/src/contextify.h b/src/contextify.h
new file mode 100644
index 00000000000000..f44cd2c05d7c01
--- /dev/null
+++ b/src/contextify.h
@@ -0,0 +1,14 @@
+#ifndef CONTEXTIFY_H
+#define CONTEXTIFY_H
+
+#include <v8.h>
+#include <nan.h>
+
+class ContextifyContext {
+public:
+    static void Initialize(v8::Local<v8::Object> target);
+    static v8::Local<v8::Context> CreateContext(v8::Local<v8::Object> sandbox);
+    static void PropertyDefinerCallback(v8::Local<v8::String> property, const v8::PropertyCallbackInfo<v8::Value>& info);
+};
+
+#endif  // CONTEXTIFY_H
\ No newline at end of file