Skip to content

Conversation

@ichoosetoaccept
Copy link
Member

@ichoosetoaccept ichoosetoaccept commented Dec 30, 2025

Summary

Removes 11 unused upstream dependencies that were never integrated into the main server code.

Removed Dependencies

Production (10):

  • fetch-cookie - Cookie handling for fetch
  • form-data - Form data encoding
  • http-proxy-agent - HTTP proxy support
  • https-proxy-agent - HTTPS proxy support
  • node-fetch - Node.js fetch polyfill
  • open - Open URLs in browser (OAuth flow)
  • pkce-challenge - PKCE auth flow
  • socks-proxy-agent - SOCKS proxy support
  • tough-cookie - Cookie jar implementation
  • zod-to-json-schema - Zod schema conversion

Dev (1):

  • @types/express - No longer needed

Verification

  • Confirmed none of these packages are imported in src/
  • Build passes ✅
  • All 122 tests pass ✅

Notes

These were upstream leftovers for OAuth, proxy support, and other features that were never wired into the main server. The gitlabUseOAuth config option exists but is never consumed - OAuth implementation doesn't exist in this fork.

Copy link
Member Author

ichoosetoaccept commented Dec 30, 2025

Copy link
Member Author

ichoosetoaccept commented Dec 30, 2025

Merge activity

  • Dec 30, 10:00 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 30, 10:00 AM UTC: @ichoosetoaccept merged this pull request with Graphite.

@ichoosetoaccept ichoosetoaccept merged commit 851594f into main Dec 30, 2025
3 checks passed
@github-actions
Copy link

🎉 This PR is included in version 0.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants