alanshaw/david

Use npm-registry-client instead of npm

Open

#33 opened on Nov 20, 2013

View on GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (962 stars) (81 forks)batch import
help wanted

Description

Since you're only using npm.commands.view it would be better to just use something like:

var RegClient = require('npm-registry-client')
var client = new RegClient(config)

client.get("npm", "latest", 1000, function (er, data, raw, res) {
  // error is an error if there was a problem.
  // data is the parsed data object
  // raw is the json string
  // res is the response from couch
})

npm-registry-client is the library depended on by npm to fetch things from the registry. Why is this better?

  1. Reduces dependencies: As a library about dependencies this should be self-explanatory :-D
  2. More configurable: If I wanted to use david to check against two different registries I am currently limited by what is on disk in .npmrc when you call npm.load

I would make a PR for this if you're interested.

Contributor guide

Tech stack
javascriptnodejs
Domain
backendtoolingdeveloper experience
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Node.jsnpm knowledge
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
First, locate where `npm` is currently used (likely via `npm.commands.view`) in the codebase, for example in a module that checks package versions. Then, replace that usage with `npm registry client` as shown in the issue. Ensure the client is instantiated with appropriate configuration (e.g., from `.npmrc` or passed options). Update any tests accordingly. The repository's main file is `index.js` or `lib/david.js`.