Skip to content

docs: add npm pack method as alternative to npm link for local development#1700

Open
Uday9909 wants to merge 1 commit into
layer5io:masterfrom
Uday9909:feat/readme-npm-pack
Open

docs: add npm pack method as alternative to npm link for local development#1700
Uday9909 wants to merge 1 commit into
layer5io:masterfrom
Uday9909:feat/readme-npm-pack

Conversation

@Uday9909

@Uday9909 Uday9909 commented Jul 9, 2026

Copy link
Copy Markdown

Adds a Method 3: Using npm pack to the README installation section, placed between Method 2 (npm link) and the Note section.

The npm link approach often fails with "Module not found" errors in Meshery UI. The npm pack method works more reliably by creating a tarball and installing it as a regular npm package, avoiding peer dependency issues.

New section covers:

  1. Building the local Sistent fork
  2. Creating a tarball via npm pack
  3. Installing the tarball in the target project
  4. Building the target project
  5. Reverting to the official package

Commit: 169d2a7c
Branch: feat/readme-npm-pack (from fork @Uday9909)

Copilot AI review requested due to automatic review settings July 9, 2026 16:30

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the README.md to replace npm unlink with npm uninstall in the revert instructions, and introduces a new section detailing how to use npm pack to install a local build via a tarball. The feedback recommends replacing the wildcard (*) in the npm install command with a <version> placeholder to prevent potential shell expansion issues on Windows and conflicts when multiple tarball files exist.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread README.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a third local-development installation option to the README by documenting a more reliable npm pack-based workflow between the existing npm link method and the note section.

Changes:

  • Replaces the Method 2 “revert” command snippet to use uninstall/install.
  • Adds Method 3: Using npm pack with step-by-step instructions to pack and install a tarball into a target project.
  • Documents how to revert from the tarball install back to the official @sistent/sistent package.

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
…pment

Add Method 3 (Using npm pack) to the README installation section. The
npm pack approach avoids peer dependency issues that npm link sometimes
causes with Meshery UI.

Signed-off-by: Udaybir Singh <usudaybir@gmail.com>
Signed-off-by: Udaybir Singh <writetoudaybir@gmail.com>
@Uday9909 Uday9909 force-pushed the feat/readme-npm-pack branch from 169d2a7 to 9b4fcc6 Compare July 9, 2026 16:35
@Uday9909

Uday9909 commented Jul 9, 2026

Copy link
Copy Markdown
Author

@leecalcote can you review this please?

@Uday9909

Copy link
Copy Markdown
Author

@KhushamBansal can you review this document fix?

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.

2 participants