[Copilot]

The V8 bindings were using heavy heap-allocated wrapper objects (such as v8::Boolean, v8::Integer, and v8::Number) when returning primitive values to JavaScript. This pull request replaces these wrapped returns with direct V8 POD (Plain Old Data) Set overloads, improving execution performance and reducing heap allocation overhead.

Refactored Bindings

• WebSocket Bindings (src/WebSocketWrapper.h)

  • Replaced Boolean::New(isolate, success) with direct boolean returns.
  • Replaced Integer::NewFromUnsigned(isolate, port) with direct uint32_t returns for port, buffered amount, and send status.

• HTTP Response Bindings (src/HttpResponseWrapper.h)

  • Replaced Boolean::New with direct boolean returns in write responses.
  • Avoided Integer::NewFromUnsigned and Number::New wrapper objects for port and write offsets.

• App Bindings (src/AppWrapper.h)

  • Streamlined subscriber counts, app descriptor pointers, and publishing success checks to return primitive types directly.

• Addon Utilities (src/addon.cpp)

  • Updated socket port and temporary KV store value returns to use plain integers.

Code Pattern Example

Before:

bool success = ws->subscribe(topic.getString());
args.GetReturnValue().Set(Boolean::New(isolate, success));

After:

bool success = ws->subscribe(topic.getString());
args.GetReturnValue().Set(success);