fix: missing napi_delete_reference on ObjectWrap ref

[legendecas]

According to https://nodejs.org/api/n-api.html#napi_wrap, the napi_reference returned from napi_wrap should be deleted with napi_delete_reference. In node-addon-api, for an ObjectWrap, this is done with ~Reference destructor when the _ref is not nullptr.

To prevent ~ObjectWrap from creating a handle during finalization, add a new flag to guard on it, instead of nullify _ref.

Submitted nodejs/node#55620 to allow napi_delete_reference been immediately invoked in an node_api_basic_finalizer.

Fixes #1602

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.35%. Comparing base (19002d5) to head (4e05944).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1607      +/-   ##
==========================================
- Coverage   66.40%   66.35%   -0.05%     
==========================================
  Files           3        3              
  Lines        2143     2143              
  Branches      702      703       +1     
==========================================
- Hits         1423     1422       -1     
  Misses        150      150              
- Partials      570      571       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mhdawson]

LGTM