Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@shuyangli
Copy link
Member

@shuyangli shuyangli commented Dec 11, 2025

Important

Migrates model count functionality to gateway, adding a new endpoint and updating client and server code.

  • Behavior:
    • Adds /internal/models/count route in internal.rs to gateway, handled by count_models_handler.
    • Removes countDistinctModelsUsed from DatabaseClient in internal/tensorzero-node/src/database.rs.
    • Updates TensorZeroClient in tensorzero.ts to use new endpoint for model count.
  • Types:
    • Adds CountModelsResponse type in CountModelsResponse.ts and types.rs.
  • Tests:
    • Adds E2E test test_count_models_endpoint in models.rs to verify new endpoint.

This description was created by Ellipsis for b51c9b4. You can customize this summary. It will automatically update as commits are pushed.

@shuyangli shuyangli force-pushed the sl/migrate-count-model branch 2 times, most recently from 2c84ec8 to bad532f Compare December 11, 2025 03:35
@shuyangli shuyangli marked this pull request as ready for review December 11, 2025 03:44
@shuyangli shuyangli mentioned this pull request Dec 11, 2025
52 tasks
Base automatically changed from sl/get-model-inference-by-id to main December 11, 2025 04:21
@shuyangli shuyangli force-pushed the sl/migrate-count-model branch from bad532f to 16a3610 Compare December 11, 2025 04:37
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
Merged via the queue into main with commit c45309f Dec 11, 2025
55 checks passed
@shuyangli shuyangli deleted the sl/migrate-count-model branch December 11, 2025 19:25
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