MetaMask / MetaMask/detect-provider

Build process is outdated

Open
#106 0 comments 0 reactions 0 assignees View on GitHub
team-wallet-framework
Dominant language
JavaScript
Stars
238
Forks
63
PR merge metrics
No merged PRs in 30d

Description

In order to achieve standardization with our other libraries, we should:

- Remove `browserify` as a dependency
- Add `tsup` as a dependency (copy version from module template)
- Remove `build.sh`
- Add the `build:types` package script (copy from module template)
- Update the `build` package script to use `tsup` to compile the TypeScript code to JavaScript and `tsc` to create TypeScript declaration files.
- Rename the `typings` field in `package.json` to `types`
- Add `exports` field to `package.json` (copy from module template)
- Add `module` field to `package.json` (copy from module template)
- Add `sideEffects` field to `package.json` (copy from module template)
- Remove trailing slash from `dist/` in `files` field

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by comparing package.json with the module template, then inspect build.sh and the current package scripts, dependencies, and package metadata. Done means the listed browserify/build.sh changes, tsup-based build scripts, and package.json fields match the template and the project build completes successfully.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.