Skip to content

shell: .llm -> $agent, ui polish#10010

Merged
shykes merged 7 commits into
dagger:mainfrom
vito:shell-multi-agent
Mar 31, 2025
Merged

shell: .llm -> $agent, ui polish#10010
shykes merged 7 commits into
dagger:mainfrom
vito:shell-multi-agent

Conversation

@vito
Copy link
Copy Markdown
Contributor

@vito vito commented Mar 30, 2025

  • .llm => $agent
  • get rid of llm variables => shell syncing now that it doesn't set vars
  • get rid of obnoxious background highlight in nav mode
  • chain calls by the same actor (i.e. tool calls, thinking spans)
  • show available LLM vars

vito added 5 commits March 30, 2025 16:57
no longer a thing - vars are only set by user, llm returns single object
this var can be reassigned to replace the current LLM, so now you can
use the full API to update it, whereas .llm only supported setting the
current state

Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
currently LLM call loops are always double spaced and calls are always
unchained, which is a shame because they actually are chained calls.

now we'll consider spans chained as long as they have the same actor
emoji. it's a little overzealous but works out for now because the tool
calling scheme is always effectively chained anyway.

Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
really not needed, noticing whether the prompt input is has a focused
background is enough

Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
@vito vito requested a review from a team as a code owner March 30, 2025 20:59
@vito vito requested a review from shykes March 30, 2025 21:00
vito added 2 commits March 30, 2025 17:26
...and refactor key handling along the way

the crux of this is allowing prompt initialization to kick off a
tea.Cmd

Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
Signed-off-by: Alex Suraci <suraci.alex@gmail.com>
Comment thread cmd/dagger/llm.go
}

func (do *dynamicObject) XXX_GraphQLType() string {
func (do *dynamicObject) XXX_GraphQLType() string { //nolint:stylecheck
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lol wut 👀 feels like the linter should have been catching this before

@shykes shykes merged commit 414c006 into dagger:main Mar 31, 2025
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