MetaMask / MetaMask/snaps

Investigate CLI building twice when using `watch`

Open
#2,613 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
853
Forks
662
Avg merge
2d 8h
Merged PRs (30d)
8

Description

When using `mm-snap watch`, the CLI will rebuild the Snap twice after detecting a change in the source code. Once because the source file changed and once again because rebuilding causes `snap.manifest.json` to be updated.

We should look into minimizing this and strictly rebuilding when necessary.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by running `mm-snap watch` and trace how source changes and updates to `snap.manifest.json` trigger rebuilds. Done means a source change causes only the necessary rebuild, without a second rebuild caused by the manifest update.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
build-system, cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.