Fix bugs when using on clean machine#10
Open
bettercallsaulj wants to merge 8 commits intomainfrom
Open
Conversation
added 8 commits
April 28, 2026 07:26
Switch submodule from gateway-img-with-auth to br_release branch. Includes /ready K8s endpoint, updated LLM models, gopher-mcp v0.1.5.
Auto-detect Homebrew dylib dependencies (OpenSSL, libevent, curl, nghttp2) via otool -L, copy them into native/lib/, and rewrite all paths to @loader_path/ using install_name_tool. npm consumers no longer need Homebrew installed to load the native libraries on macOS.
…#9) Bundle transitive dylib dependencies recursively and re-sign all dylibs after install_name_tool to fix macOS 26 SIGKILL on dlopen.
GitHub Actions workflow to test npm package on clean machines: - macOS ARM64 (Homebrew removed) - macOS x64 (Homebrew removed) - Linux x64 (node:20-slim container) Triggered manually with npm version input. Catches Bug 4 (invalid codesign) and Bug 5 (missing transitive deps).
Added resolve_dep() to search Homebrew paths for @rpath references. Fixes missing libbrotlicommon.1.dylib that caused dlopen failure.
…ion order (#9) - Add export {} to 5 test files to isolate variable scope (TS2451) - Move error class imports outside native availability check in gopher-auth.test.ts (InsufficientScopesError/TokenValidationError) - Move config validation before native lib init in gopher-auth.ts so ConfigurationError is thrown before generic Error
- Suppress ban-types for Express app parameter (Function type) - Suppress unsafe-any rules in middleware (Express req/res are any)
routes.ts and loader.ts intentionally use any for Express request/ response handling and koffi FFI bindings. Added file-level eslint disable for unsafe-any rules.
bettercallsaulj
force-pushed
the
fix_homebrew_native_crash_error
branch
from
45ec3f8 to
b263438
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Detail: https://github.com/GopherSecurity/gopher-issue/issues/1168