Skip to content

Comments

feat: add NanoGPT as a provider#5

Open
0xGingi wants to merge 10 commits intonumman-ali:mainfrom
0xGingi:nanogpt-provider
Open

feat: add NanoGPT as a provider#5
0xGingi wants to merge 10 commits intonumman-ali:mainfrom
0xGingi:nanogpt-provider

Conversation

@0xGingi
Copy link
Contributor

@0xGingi 0xGingi commented Jan 4, 2026

Description

This adds https://nano-gpt.com as a provider (This PR also has some changes to tui.test.ts as it was failing a test when attempting to commit to my repo)

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 not work as expected)
  • Documentation update

How Has This Been Tested?

  • Ran npm run check (typecheck + lint + test)
  • Tested the TUI flow manually
  • Tested CLI commands

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have updated the documentation if needed
  • My changes generate no new warnings
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally

@numman-ali
Copy link
Owner

Thanks for the PR!

You probably noticed I put in the new Claude Mirror provider

So you'll have conflicts

Do you mind if I do the work myself later or do you care of the contribution badge?

I think lots of feedback will come in and I need to coordinate and fix a few things most likely

@0xGingi
Copy link
Contributor Author

0xGingi commented Jan 4, 2026

Yeah all good, whatever works best for you!

@0xGingi
Copy link
Contributor Author

0xGingi commented Jan 4, 2026

I went ahead and dealt with the conflicts

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