Skip to content

[AI Improvement] [Task] Remove unused production dependencies proxy-agent, universal-analytics, and @apphosting/common - #11217

Draft
joehan wants to merge 4 commits into
mainfrom
ai-improve-568084099-task-remove-unused-production-depen
Draft

joehan wants to merge 4 commits into
mainfrom
ai-improve-568084099-task-remove-unused-production-depen

Conversation

@joehan

@joehan joehan commented Sep 30, 2026

Copy link
Copy Markdown
Member

Resolves Buganizer b/568084099

Proposed Improvement

Removed unused production dependencies proxy-agent, universal-analytics, and @apphosting/common, along with @types/universal-analytics from devDependencies and the universal-analytics uuid override block from package.json. Regenerated npm-shrinkwrap.json using pinned npm@11.9, cleanly removing ~40 unused transitive packages from the published package dependency tree.

Verification

  • npm ci: Installed clean lockfile successfully.
  • npm run build: TypeScript compilation and build steps succeeded.
  • npm run lint:quiet: Passed cleanly across the entire repository (ESLint and Prettier).
  • npm run test:compile: Passed cleanly.
  • npx mocha src/apiv2.spec.ts src/track.spec.ts: All 54 tests passed.
  • Idempotence verification: npx -y npm@11.9 install --package-lock-only --ignore-scripts produces no diff against npm-shrinkwrap.json.

…t, universal-analytics, and @apphosting/common (b/568084099)
@joehan joehan self-assigned this Sep 30, 2026

@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 removes unused dependencies from package.json, including @apphosting/common, proxy-agent, and universal-analytics (along with its types and overrides), and updates CHANGELOG.md accordingly. As there are no review comments, I have no additional feedback to provide.

This branch has not been deployed

No deployments
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