addArchiveStream ignores false values for hasAudio/hasVideo options

[hidday]

opentok-node/lib/archiving.js

Line 427 in 18aa507

hasVideo: archiveOptions.hasVideo ? archiveOptions.hasVideo : true,

Issue Description

🐞 Bug Summary

The addArchiveStream function incorrectly ignores explicit false values passed for hasAudio or hasVideo.
When { hasVideo: false } or { hasAudio: false } is provided, the function treats the value as falsy and defaults to true, resulting in unwanted audio/video being added to the archive.

💡 Expected Behavior

If a caller explicitly passes false for either option, that value should be respected.

⚙️ Current Behavior

hasAudio: archiveOptions.hasAudio ? archiveOptions.hasAudio : true,
hasVideo: archiveOptions.hasVideo ? archiveOptions.hasVideo : true,

Because of the truthy check, any falsy value (including false) triggers the default true.

✅ Proposed Fix

Replace the truthy checks with nullish coalescing (??) to only default when the option is undefined or null:

hasAudio: archiveOptions.hasAudio ?? true,
hasVideo: archiveOptions.hasVideo ?? true,

🔍 Impact

• Developers cannot currently disable audio or video streams when adding to an archive.
• Causes incorrect archive configuration and potential unnecessary media inclusion.

🧩 Affected Module

archive.js → addArchiveStream()

[manchuck]

Fixed with #374