node-api: `napi_delete_reference` should use `node_api_basic_env`

[KevinEady]

Since napi_delete_reference does not check that a environment is in GC, the function prototype should accept a node_api_basic_env instead of an napi_env.

node/src/js_native_api_v8.cc

Line 2776 in 3b4f9b2

napi_status NAPI_CDECL napi_delete_reference(napi_env env, napi_ref ref) {

Refs:

• node-api: allow napi_delete_reference in basic finalizers #55620

[JeetuSuthar]

@KevinEady Should i raise a PR for this ?

[legendecas]

For context: https://github.com/nodejs/node-addon-api/blob/b65e01b3ad21865fcdc9120730cea4b8a2330ef3/napi-inl.h#L3481-L3482

[legendecas]

@JeetuSuthar you are welcomed to submit a PR.

[JeetuSuthar]

@legendecas I raised the Pr 2 days ago ...can you plz check:-)