Skip to content

docs(cli): clarify that memtrace install requires sudo when npm prefix is system-owned - #83

Open
ege-arhan wants to merge 1 commit into
syncable-dev:mainfrom
ege-arhan:fix/issue-72-install-sudo-doc
Open

ege-arhan wants to merge 1 commit into
syncable-dev:mainfrom
ege-arhan:fix/issue-72-install-sudo-doc

Conversation

@ege-arhan

@ege-arhan ege-arhan commented Sep 13, 2026

Copy link
Copy Markdown

Problem

When users run memtrace install on systems where the global npm directory is root-owned (such as default Linux installs under /usr/lib/node_modules), the command fails with EACCES: permission denied.

Solution

Document in docs/cli-reference.md that memtrace install requires sudo if the npm prefix is system-owned, pointing users toward the right invocation without ambiguity.

Impact

Prevents confusion and unhandled permission errors during self-updates.

Evidence

Verified docs table formatting and accurate rendering against existing command options.

Fixes #72


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@blocksorg

blocksorg Bot commented Sep 13, 2026

Copy link
Copy Markdown

Mention Blocks like a regular teammate with your question or request:

@blocks review this pull request
@blocks make the following changes ...
@blocks create an issue from what was mentioned in the following comment ...
@blocks explain the following code ...
@blocks are there any security or performance concerns?

Run @blocks /help for more information.

Workspace settings | Disable this message

@ege-arhan
ege-arhan force-pushed the fix/issue-72-install-sudo-doc branch from c01eb76 to 0bf28c0 Compare September 14, 2026 14:59
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.

memtrace install needs sudo, but CLI hint does not mention it

1 participant