-
Notifications
You must be signed in to change notification settings - Fork 5
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add create-gensx package #66
Conversation
Coverage Report for packages/gensx
File CoverageNo changed files found. |
Deploying gensx with
|
Latest commit: |
7c2090e
|
Status: | ✅ Deploy successful! |
Preview URL: | https://9e862d7d.gensx.pages.dev |
Branch Preview URL: | https://jeremy-create-gensx.gensx.pages.dev |
0929a8b
to
ad9c8a8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! One question on whether or not we should make the example template a little more complete (add some LLM stuff).
); | ||
|
||
const App = gsx.Component<Record<string, never>, string>(async () => { | ||
return await gsx.execute( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the default template include the open AI package (and instantiate a provider + a hello world completion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. That should be straightforward to add.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could also make it a different template, but that might be overkill
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that I've implemented it, my one concern is that it does not Just Work now. If you npm dev
the newly created project, you can an error about a missing API key.
Is it worth that tradeoff to demonstrate the OpenAI functionality vs something that works out of the box?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a hint to the run command, so hopefully that makes this less of a problem.
Successfully created GenSX project in /var/folders/_z/v4ndzzd150x02zpv4_q_dcn40000gn/T/gensx-test-9qyIYk1jJuFY/test-project
To get started:
cd /var/folders/_z/v4ndzzd150x02zpv4_q_dcn40000gn/T/gensx-test-9qyIYk1jJuFY/test-project
OPENAI_API_KEY=<your_api_key> npm run devEdit src/index.tsx to start building your GenSX application.
7d24149
to
d37ba38
Compare
d37ba38
to
7c2090e
Compare
🤖 I have created a release *beep* *boop* --- ## 0.1.0 (2025-01-07) ### ✨ New Features * Add create-gensx package ([#66](#66)) ([9bb7418](9bb7418)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Proposed changes
Add a package to make it easy to boostrap a
gensx
project, usingnpx create-gensx
ornpm create gensx
.