remove npm api calls (#6177)

rochdev · web-flow · commit aed28b208395 · 2025-07-28T10:30:18.000-04:00
* remove npm show call for llmobs tests

* disable job that may require npm show to work

* remove npm api call for upstream tests

* remove github api call for upstream tests

* add missing axios
diff --git a/.github/workflows/project.yml b/.github/workflows/project.yml
@@ -89,13 +89,14 @@ jobs:
       - run: yarn type:test
       - run: yarn type:doc
 
-  verify-yaml:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-      - uses: ./.github/actions/node/active-lts
-      - uses: ./.github/actions/install
-      - run: node scripts/verify-ci-config.js
+  # TODO: Remove need for `npm show` before re-enabling to avoid rate limit errors.
+  # verify-yaml:
+  #   runs-on: ubuntu-latest
+  #   steps:
+  #     - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+  #     - uses: ./.github/actions/node/active-lts
+  #     - uses: ./.github/actions/install
+  #     - run: node scripts/verify-ci-config.js
 
   yarn-dedupe:
     runs-on: ubuntu-latest
diff --git a/packages/dd-trace/test/llmobs/sdk/typescript/index.spec.js b/packages/dd-trace/test/llmobs/sdk/typescript/index.spec.js
@@ -67,22 +67,14 @@ const testCases = [
   }
 ]
 
-// a bit of devex to show the version we're actually testing
-// so we don't need to know ahead of time
-function getLatestVersion (range) {
-  const command = `npm show typescript@${range} version`
-  const output = execSync(command, { encoding: 'utf-8' }).trim()
-  const versions = output.split('\n').map(line => line.split(' ')[1].replace(/'/g, ''))
-  return versions[versions.length - 1]
-}
-
 describe('typescript', () => {
   let agent
   let proc
   let sandbox
 
   for (const version of testVersions) {
-    context(`with version ${getLatestVersion(version)}`, () => {
+    // TODO: Figure out the real version without using `npm show` as it causes rate limit errors.
+    context(`with version ${version}`, () => {
       before(async function () {
         this.timeout(20000)
         sandbox = await createSandbox(
diff --git a/packages/dd-trace/test/plugins/suite.js b/packages/dd-trace/test/plugins/suite.js
@@ -7,82 +7,16 @@ const fs = require('fs')
 const util = require('util')
 const os = require('os')
 const path = require('path')
-const https = require('https')
-const url = require('url')
 const { once } = require('events')
 const { expect } = require('chai')
+const latests = require('../plugins/versions/package.json').dependencies
 
 process.env.DD_INSTRUMENTATION_TELEMETRY_ENABLED = 'false'
 
 const mkdtemp = util.promisify(fs.mkdtemp)
 
 const ddTraceInit = path.resolve(__dirname, '../../../../init')
 
-const latestCache = []
-async function getLatest (modName, repoUrl) {
-  if (latestCache[modName]) {
-    return latestCache[modName]
-  }
-  const { stdout } = await retry(() => exec(`npm view ${modName} dist-tags --json`), 1000)
-  const { latest } = JSON.parse(stdout)
-  const tags = await get(`https://api.github.com/repos/${repoUrl}/git/refs/tags`)
-  for (const tag of tags) {
-    if (tag.ref.includes(latest)) {
-      return tag.ref.split('/').pop()
-    }
-  }
-}
-
-function get (theUrl) {
-  return new Promise((resolve, reject) => {
-    // eslint-disable-next-line n/no-deprecated-api
-    const options = url.parse(theUrl)
-    options.headers = {
-      'user-agent': 'dd-trace plugin test suites'
-    }
-    https.get(options, res => {
-      if (res.statusCode === 403) {
-        console.log('403')
-        for (const header in res.headers) {
-          if (header.startsWith('x-ratelimit')) {
-            console.log(header, res.headers[header])
-          }
-        }
-        const resetTime = Number(res.headers['x-ratelimit-reset']) * 1000
-        const waitTime = 1000 + resetTime - Date.now()
-        console.log('Waiting', waitTime / 1000, 'seconds for retry')
-        setTimeout(() => {
-          get(theUrl).then(resolve, reject)
-        }, waitTime)
-        return
-      }
-      if (res.statusCode >= 300) {
-        res.pipe(process.stderr)
-        reject(new Error(res.statusCode))
-        return
-      }
-      const data = []
-      res.on('data', d => data.push(d))
-      res.on('end', () => {
-        resolve(JSON.parse(Buffer.concat(data).toString('utf8')))
-      })
-    }).on('error', reject)
-  })
-}
-
-async function retry (fn, delay) {
-  let result
-  try {
-    result = fn()
-  } catch (e) {
-    console.log(e)
-    console.log('Retrying after', delay, 'ms')
-    await new Promise(resolve => setTimeout(resolve, delay))
-    result = retry(fn, delay)
-  }
-  return result
-}
-
 function exec (cmd, opts = {}) {
   const date = new Date()
   const time = [
@@ -127,11 +61,18 @@ function getTmpDir () {
 
 async function setup (modName, repoName, commitish) {
   if (commitish === 'latest') {
-    commitish = await getLatest(modName, repoName)
+    commitish = latests[modName]
   }
   const repoUrl = `https://github.com/${repoName}.git`
   const cwd = await getTmpDir()
-  await execOrError(`git clone ${repoUrl} --branch ${commitish} --single-branch ${cwd}`)
+  const clone = `git clone ${repoUrl} --single-branch ${cwd}`
+
+  try {
+    await execOrError(`${clone} --branch ${commitish}`)
+  } catch {
+    // Exact version doesn't exist, try with a `v` prefix for example `v1.2.3`.
+    await execOrError(`${clone} --branch v${commitish}`)
+  }
 
   try {
     await execOrError('npm install --legacy-peer-deps', { cwd })
diff --git a/packages/dd-trace/test/plugins/versions/package.json b/packages/dd-trace/test/plugins/versions/package.json
@@ -36,6 +36,7 @@
     "apollo-server-core": "3.13.0",
     "avsc": "5.7.9",
     "aws-sdk": "2.1692.0",
+    "axios": "1.11.0",
     "bluebird": "3.7.2",
     "body-parser": "2.2.0",
     "bunyan": "2.0.5",
