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

feat: support devcontainer agents in ui and unify backend#18332

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

Open
mafredri wants to merge17 commits intomain
base:main
Choose a base branch
Loading
frommafredri/feat-agent-devcontainer-injection-6

Conversation

mafredri
Copy link
Member

@mafredrimafredri commentedJun 11, 2025
edited
Loading

This PR adds consolidates two container endpoints on the backend and improves the frontend devcontainer support by showing names and displaying apps as appropriate.

With this change, the frontend now has knowledge of the subagent and we can also display things like port forwards.

The frontend was updated to show dev container labels on the border as well as subagent connection status. The recreation flow was also adjusted a bit to show placeholder app icons when relevant.

Support forapps was also added, although these are still WIP on the backend. And the port forwarding utility was added in since the sub agents now provide the necessary info.

Fixescoder/internal#666

image

@mafredrimafredriforce-pushed themafredri/feat-agent-devcontainer-injection-6 branch fromc9f4ca4 to18e1593CompareJune 13, 2025 17:35
@mafredrimafredri changed the titlefeat: expand devcontainer subagent support in ui and improve backendfeat: support devcontainer agents in ui and unify backendJun 13, 2025
@mafredrimafredriforce-pushed themafredri/feat-agent-devcontainer-injection-6 branch 2 times, most recently fromfc1a236 to414b6f4CompareJune 13, 2025 18:44
@mafredrimafredriforce-pushed themafredri/feat-agent-devcontainer-injection-6 branch from414b6f4 toecfe483CompareJune 13, 2025 19:06
@mafredrimafredri marked this pull request as ready for reviewJune 13, 2025 20:00
@mafredrimafredriforce-pushed themafredri/feat-agent-devcontainer-injection-6 branch from8f12d26 to79e1844CompareJune 13, 2025 20:08
Copy link
Contributor

@DanielleMaywoodDanielleMaywood left a comment

Choose a reason for hiding this comment

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

haven't managed to read all the Go yet but will read it by Monday

return (
<HelpTooltip>
<HelpTooltipTrigger>
<span role="status" css={{ cursor: "pointer" }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: css

Suggested change
<spanrole="status"css={{cursor:"pointer"}}>
<spanrole="status"class="cursor-pointer">

https://tailwindcss.com/docs/cursor#basic-example

Comment on lines +38 to +41
const versionLabelStyles = {
fontWeight: 600,
color: theme.palette.text.primary,
};
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: is this used anywhere?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@DanielleMaywoodDanielleMaywoodDanielleMaywood requested changes

@BrunoQuaresmaBrunoQuaresmaAwaiting requested review from BrunoQuaresma

@johnstcnjohnstcnAwaiting requested review from johnstcn

Requested changes must be addressed to merge this pull request.

Assignees

@mafredrimafredri

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ConsolidateGET /containers andGET /containers/devcontainers and render devcontainers (instead of containers) in the UI
2 participants
@mafredri@DanielleMaywood

[8]ページ先頭

©2009-2025 Movatter.jp