Skip to content

Conversation

@sandboxcoder
Copy link
Contributor

@sandboxcoder sandboxcoder commented Nov 19, 2025

Description

Fixing local mac development issue where there was an edge case for a version mismatch between client native node vs obs64. Also update README to demonstrate user is required to use absolute path for CMAKE_INSTALL_PREFIX

Motivation and Context

If developer is using OSN.app bundle (so browser sources work due to Google Chromium restrictions where it needs an app bundle to find the obs64 helper apps to enable the browser), then there was an edge case where desktop/node_modules/obs-studio-client.node file could end up with a version mismatch against OSN.app/distribute/bin/obs64. Updating the module.ts file to look for the native node module in the developer-only OSN.app bundle (instead of the other desktop/node_modules/obs-client.node file) to resolve any possibility of a version mismatch.

How Has This Been Tested?

Built a Desktop app bundle and made sure that still worked. Also verified the local build worked properly when ran directly via yarn start or via visual studio code.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

…n mismatch

* Use obs-client.node included in OBS.app bundle. This way if developer were
to update node dependencies etc it wont cause a version mismatch
* update readme example to indicate developer has to use an abs path. Cant use ~
for CMAKE_INSTALL_PREFIX
@sandboxcoder sandboxcoder marked this pull request as ready for review November 20, 2025 14:48
@sandboxcoder sandboxcoder merged commit e10558c into staging Dec 1, 2025
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants