- Notifications
You must be signed in to change notification settings - Fork905
refactor: reduce the default workspace name for a task#18193
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@hugodutka just to keep in mind when creating modules, we want to keep the app names smaller as possible. |
Can we AI generate the name based on the task using aisdk-go if configured? |
Probably yes, but you are probably talking about a diff prop. In this PR we are talking about the unique workspace name that is used as a slug. In this case, we can't simply only generate it with AI, we need to make sure it is unique and handle possible database errors. I see AI generated value for the display name 👍 |
We could generate the slug with AI too. If it's already taken then append |
1d27d4f intomainUh oh!
There was an error while loading.Please reload this page.
Subdomains should have 63 max characters, so we don't want to have a long default workspace name that could overflow this limit. With that in mind, I'm reducing 3 characters from the default name.
PS: I've been facing issues with that already. Eg: