Skip to content

feat: support web console opener, deprecate sandbox creation#821

Open
rektdeckard wants to merge 1 commit intomainfrom
tobiasfried/grw-234-cli-prints-console-link-instead-of-sandbox-link
Open

feat: support web console opener, deprecate sandbox creation#821
rektdeckard wants to merge 1 commit intomainfrom
tobiasfried/grw-234-cli-prints-console-link-instead-of-sandbox-link

Conversation

@rektdeckard
Copy link
Copy Markdown
Member

  • Adds support for Web Console opener in lk token create, lk room join
  • lk agent init no longer defaults to creating a sandbox (and the flag is deprecated and hidden)
  • lk agent init and lk app create now print Web Console link on completion
  • lk agent init and lk app create now prompt to auto-install dependencies

@rektdeckard rektdeckard requested review from a team, real-danm and yoonsio April 10, 2026 20:53
Copy link
Copy Markdown
Contributor

@yoonsio yoonsio left a comment

Choose a reason for hiding this comment

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

Do we still support template projects?

@rektdeckard
Copy link
Copy Markdown
Member Author

rektdeckard commented Apr 10, 2026

@yoonsio yes, the templates are unchanged except that the agent name is now used to print a link to the web-based console instead of a sandbox (for python and node agent templates):

Screenshot 2026-04-09 at 11 02 01 PM

fmt.Println("Installing template...")
if err := doInstall(ctx, bootstrap.TaskInstall, appName, verbose); err != nil {
return err
fmt.Fprintf(os.Stderr, "Warning: installation failed: %v\n", err)
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.

Was the intent here to make install failures non fatal? I’m asking because it looks like we continue into post_create and cleanup even if doInstall fails, and I wasn’t sure if that could leave someone with a partially initialized app that’s harder to deal with?

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.

4 participants