User references instead of copying

[repugraf]

No description provided.

[Copilot]

Pull Request Overview

This PR updates function signatures in the TypeConversion namespace and related helper functions to pass Napi::Env as a const reference (const Napi::Env &) instead of by value. While this change is technically valid, it introduces a pattern inconsistency with other similar functions in the codebase.

Key Changes

• Modified all TypeConversion namespace function signatures to accept Napi::Env by const reference
• Updated the BusPopWorker::ConvertMessageToJs private method signature accordingly
• Updated forward declaration in async-workers.hpp to match

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/cpp/type-conversion.hpp	Updated function declarations to pass Napi::Env by const reference
src/cpp/type-conversion.cpp	Updated function implementations to match header declarations
src/cpp/async-workers.hpp	Updated forward declaration and private method signature for gst_sample_to_js and ConvertMessageToJs
src/cpp/async-workers.cpp	Updated ConvertMessageToJs implementation to match declaration

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull Request Overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.