Qjs exit hook

[xeioex]

No description provided.

[Copilot]

Pull Request Overview

This PR implements the QuickJS exit hook functionality by adding support for njs.on('exit', callback) which allows JavaScript code to register exit hooks that are called when the context is destroyed.

Key changes include:

• Convert core class ID definitions from #define macros to enums for better type safety
• Implement a new njs global object with exit hook functionality
• Add test coverage for the exit hook feature

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
src/qjs.h	Converts class ID macros to enum and adds exit hook function declaration
src/qjs.c	Implements core exit hook functionality including the njs object and event handlers
nginx/t/js_exit.t	Adds new test file for HTTP exit hook functionality
nginx/t/stream_js_exit.t	Removes QuickJS skip condition to enable stream exit hook tests
nginx/ngx_stream_js_module.c	Refactors string handling to use JS_ToCStringLen directly
nginx/ngx_qjs_fetch.c	Updates string conversion calls to use pool-based allocation
nginx/ngx_js_shared_dict.c	Updates string handling for shared dictionary operations
nginx/ngx_js.h	Updates function signature and adds QuickJS conditional compilation
nginx/ngx_js.c	Integrates exit hook calls and updates string conversion implementation
nginx/ngx_http_js_module.c	Updates string conversion calls and fixes cleanup handler type
external/njs_shell.c	Adds exit hook call to engine destruction

Comments suppressed due to low confidence (1)

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[VadimZhestikov]

Looks good

[VadimZhestikov]

Looks good