Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore: add user details to aibridge interception list endpoint#20397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
deansheather merged 3 commits intomainfromdean/aibridge-list-with-user
Oct 22, 2025

Conversation

@deansheather
Copy link
Member

@deansheatherdeansheather commentedOct 21, 2025
edited
Loading

  • Adds FK fromaibridge_interceptions.initiator_id tousers.id
    • This is enforced by deleting any rows that don't have any users. Since this is an experimental feature AND coder never deletes user rows I think this is acceptable.
  • Addsname as a property oncodersdk.MinimalUser
    • This matches thevisible_users view in the database. I'm unsure whyname wasn't already included given thatusername is.
  • Adds a newinitiator field tocodersdk.AIBridgeInterception which containscodersdk.MinimalUser (ID, username, name, avatar URL)
  • Removesinitiator_id fromcodersdk.AIBridgeInterception
    • Should be fine since we're still in early access

TODO:

  • Thoroughly test the migration in a variety of database scenarios and ensure we get the expected result

Copy link
Contributor

@dannykoppingdannykopping left a comment

Choose a reason for hiding this comment

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

LGTM

@deansheatherdeansheather merged commit69c2c40 intomainOct 22, 2025
37 checks passed
@deansheatherdeansheather deleted the dean/aibridge-list-with-user branchOctober 22, 2025 05:18
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsOct 22, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@dannykoppingdannykoppingdannykopping approved these changes

@pawbanapawbanaAwaiting requested review from pawbana

Assignees

@deansheatherdeansheather

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@deansheather@dannykopping

[8]ページ先頭

©2009-2025 Movatter.jp