Skip to content

Fix missing npm dependencies causing TypeScript lint failures#9

Merged
naheel0 merged 2 commits intocodefrom
copilot/sub-pr-8
Mar 3, 2026
Merged

Fix missing npm dependencies causing TypeScript lint failures#9
naheel0 merged 2 commits intocodefrom
copilot/sub-pr-8

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

Description

Five packages imported in source were absent from package.json, causing tsc --noEmit to fail with TS2307 (module not found) and TS7006 (implicit any) errors.

Added to dependencies:

  • @google/generative-ai@^0.24.1AIService.ts
  • @octokit/rest@^22.0.1GitHubService.ts (also resolves implicit any on pr via bundled types)
  • pino@^10.3.1 + pino-pretty@^13.1.3logger.ts
  • zod@^3.23.8ConfigService.ts

Related Issue

Closes #

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • 🔧 Refactor / internal improvement
  • 🤖 CI/CD or tooling change
  • ⬆️ Dependency update

Checklist

  • I have read the CONTRIBUTING guidelines (if present)
  • My code follows the existing code style
  • I have added or updated tests that cover my changes
  • All existing tests pass locally
  • I have updated the documentation where necessary
  • My changes do not introduce new security vulnerabilities

How Has This Been Tested?

npm run lint (tsc --noEmit) passes with zero errors after npm install.

Screenshots (if applicable)

Additional Notes

All five added packages were verified clean against the GitHub Advisory Database prior to inclusion.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: jaseel0 <225665919+jaseel0@users.noreply.github.com>
Copilot AI changed the title [WIP] Add foundational features for AI-powered Git CLI tool Fix missing npm dependencies causing TypeScript lint failures Mar 3, 2026
@naheel0 naheel0 marked this pull request as ready for review March 3, 2026 16:12
@naheel0 naheel0 merged commit 57adf18 into code Mar 3, 2026
2 checks passed
@naheel0 naheel0 deleted the copilot/sub-pr-8 branch March 3, 2026 16:12
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.

3 participants