Skip to content

Conversation

@louis-jan
Copy link
Contributor

Describe Your Changes

There’s an issue with running MCP on Mac that limits memory allocation to 128 bytes, which is broken. Since MCP runs using npx (with the built-in bun), it shouldn’t run with JIT, so no issues are introduced.

Fixes Issues

Failed to start MCP server Jan Browser MCP: Resolving dependencies
Resolved, downloaded and extracted [2]
Saved lockfile
Ran out of executable memory while allocating 128 bytes.

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Copilot AI review requested due to automatic review settings November 28, 2025 06:42
Copilot finished reviewing on behalf of louis-jan November 28, 2025 06:42
@github-actions
Copy link
Contributor

Barecheck - Code coverage report

Total: 30.18%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

@Jarred-Sumner
Copy link

There's ~no way this is correct. Is the binary codesigned properly? That's the more likely issue.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@louis-jan
Copy link
Contributor Author

louis-jan commented Nov 28, 2025

There's ~no way this is correct. Is the binary codesigned properly? That's the more likely issue.

Yeah, that’s weird. Initially, I've thought about codesigning, but haven’t found the clue yet. Do you have any ideas about why disabling JIT works?

UPDATE:
Found this: https://developer.apple.com/documentation/bundleresources/entitlements/com.apple.security.cs.allow-jit

@louis-jan
Copy link
Contributor Author

Closed, this PR provides the proper fix for the same issue

@louis-jan louis-jan closed this Nov 30, 2025
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.

6 participants