Skip to content

feat: add router model selection API#835

Open
aminsamir45 wants to merge 3 commits into
mainfrom
router-model-selection-api
Open

feat: add router model selection API#835
aminsamir45 wants to merge 3 commits into
mainfrom
router-model-selection-api

Conversation

@aminsamir45

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Claude finished @aminsamir45's task —— View job


  • Run git diff to analyze added comments
  • Identify long comment blocks (3+ lines)
  • Post review with suggestions

Found one 10-line comment block in internal/sqlc/model_router_installations.sql.go (lines 288–297) that repeats the SQL verbatim from the const defined above it. Posted a committable suggestion to trim it to two lines preserving the only non-obvious why (cross-tenant scoping).

@workweave-bot workweave-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Advisory only — comment-length nits. Won't block merge.

Comment thread internal/sqlc/model_router_installations.sql.go Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c67adf7. Configure here.

Comment thread internal/api/admin/model_selection.go
@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Claude encountered an error —— View job


  • Run git diff to analyze added comments
  • Identify long comment blocks (3+ lines)
  • Post review with suggestions (or exit silently if none)

View job

@workweave-bot workweave-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Advisory only — comment-length nits. Won't block merge.

Comment thread internal/sqlc/model_router_installations.sql.go
@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Claude finished @aminsamir45's task —— View job


  • Run git diff to analyze added comments
  • Identify long comment blocks (3+ lines)
  • Post review with suggestions

Found one 9-line comment block in internal/sqlc/model_router_installations.sql.go (lines 289–297) that repeats the SQL verbatim from the const defined above it. Posted a committable suggestion to trim it to two lines preserving the only non-obvious why (cross-tenant scoping).

@workweave-bot workweave-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Advisory only — comment-length nits. Won't block merge.

Comment thread internal/sqlc/model_router_installations.sql.go
@aminsamir45
aminsamir45 force-pushed the router-model-selection-api branch from 870142c to 40bac07 Compare July 24, 2026 21:33
@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Claude finished @aminsamir45's task —— View job


  • Run git diff to analyze added comments
  • Identify long comment blocks (3+ lines)
  • Post review with suggestions

Found one 9-line comment block in internal/sqlc/model_router_installations.sql.go (lines 288–296) that repeats the SQL verbatim from the const defined above it. Posted a committable suggestion to trim it to two lines preserving the only non-obvious why (cross-tenant scoping).

@workweave-bot workweave-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Advisory only — comment-length nit. Won't block merge.

Comment thread internal/sqlc/model_router_installations.sql.go
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