Skip to content

Desktop 1.3#2179

Merged
mschuwalow merged 50 commits intogolemcloud:mainfrom
Myestery:desktop-1.3
Oct 14, 2025
Merged

Desktop 1.3#2179
mschuwalow merged 50 commits intogolemcloud:mainfrom
Myestery:desktop-1.3

Conversation

@Myestery
Copy link
Copy Markdown
Contributor

@Myestery Myestery commented Oct 13, 2025

Issues fixed with this version

  • App new command supports only ts
  • component new command fails with success message
  • invoke fails because requested function name not found (v2:good/assistant-agent.{initialize} missing)
  • expose agent types
  • update component exports
  • Error finding Component Manifest
  • rename ‘workers’ to ‘agents’
  • allow agent creation (worker creation) with constructor parameters
  • fix anything that’s broken due to changed APIs
  • application marked as not responsive when building an application using the golem-cli
  • View yaml shows only top-level golem yaml (not useful with includes)
  • The overview page does not handle undeployed applications well
  • CLI does not handle confirmation dialogs (--yes should be passed everywhere)

/fix #2082
/claim #2082

…ng to call_golem_command with new parameters
@andrueandersoncs andrueandersoncs self-assigned this Oct 13, 2025
@andrueandersoncs andrueandersoncs marked this pull request as ready for review October 13, 2025 12:05
@mschuwalow
Copy link
Copy Markdown
Contributor

mschuwalow commented Oct 13, 2025

Thanks. A couple of notes from trying it out:

  • This is missing some templates:
Screenshot 2025-10-13 at 15 07 11
  • Something weird is going on with component names. I created the component as golem-it:test, but it's shown as golem:it:test in the ui. Also see the toast.
Screenshot 2025-10-13 at 15 08 39
  • The invalid name causes the rest of the flow to break
Screenshot 2025-10-13 at 15 09 45
  • The deployments page is entirely uninteractive (out of scope for this pr I guess)
Screenshot 2025-10-13 at 15 28 19
  • Agent types are listed multiple times
Screenshot 2025-10-13 at 15 29 57
  • Agent constructors need to be kebab-cased
Screenshot 2025-10-13 at 15 33 01
  • Initialize function needs to be hidden here. Additionally all functions not belonging to the initialized / constructed agent type need to be hidden
Screenshot 2025-10-13 at 15 37 38
  • minor, the invocation result is a flashbang when using the dark theme
Screenshot 2025-10-13 at 15 39 06

@Myestery
Copy link
Copy Markdown
Contributor Author

Taking a look 👀

@Myestery
Copy link
Copy Markdown
Contributor Author

I have resolved all the concerns in the PR asides 1
The deployments are not scoped in this PR; its just view only for now

Copy link
Copy Markdown
Contributor

@mschuwalow mschuwalow left a comment

Choose a reason for hiding this comment

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

@Myestery looks great, thanks

@mschuwalow mschuwalow merged commit a4e578f into golemcloud:main Oct 14, 2025
30 checks passed
@BenraouaneSoufiane
Copy link
Copy Markdown

@mschuwalow can you review this: #1976

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Golem Desktop for Golem 1.3

4 participants