-
Notifications
You must be signed in to change notification settings - Fork 44.4k
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(builder, server): Add Builds to Frontend and Packaged assets #7406
Conversation
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
PR Description updated to latest commit (d06d8a2)
|
PR Reviewer Guide 🔍(Review updated until commit f2849b4)
|
✅ Deploy Preview for auto-gpt-docs canceled.
|
PR Code Suggestions ✨Latest suggestions up to f2849b4
Previous suggestionsSuggestions up to commit d06d8a2
|
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
CI Failure Feedback 🧐(Checks updated until commit f2849b4)
✨ CI feedback usage guide:The CI feedback tool (
In addition to being automatically triggered, the tool can also be invoked manually by commenting on a PR:
where Configuration options
See more information about the |
Co-authored-by: Reinier van der Leer <pwuts@agpt.co>
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
Co-authored-by: Reinier van der Leer <pwuts@agpt.co>
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Not going to do any longer |
PR Description updated to latest commit (f2849b4)
|
Persistent review updated to latest commit f2849b4 |
User description
Background
We need to be able to serve a frontend and backend on the installed versions of AutoGPT Server. This fixes errors and adds that capability to the frontend and backend.
Changes 🏗️
Unfortunately, lots of changes were required to get this to work.
First,
next build
runsnext lint,
so all the errors fromnext lint
were fixed.Next, the build path for the frontend was updated to be
/frontend
rather than/example_files.
Then, the frontend was correctly served, if available in the
/frontend.
The routes were also hooked to serve the frontend pages with a higher priority than the API endpoints with a higher priority than the static files.Finally, the packaging script
setup.py
and the ci pipelines were updated.PR Quality Scorecard ✨
+2 pts
+5 pts
+5 pts
+5 pts
-4 pts
+4 pts
+5 pts
-5 pts
agbenchmark
to verify that these changes do not regress performance?+10 pts
PR Type
Enhancement, Bug fix, Tests
Description
Changes walkthrough 📝
12 files
app.py
Handle Prisma executable path and update process setup
rnd/autogpt_server/autogpt_server/app.py
run_processes
to include new setup steps.server.py
Serve frontend files and update routing logic
rnd/autogpt_server/autogpt_server/server/server.py
data.py
Update paths for frontend and Prisma files
rnd/autogpt_server/autogpt_server/util/data.py
setup.py
Build frontend and include in package setup
rnd/autogpt_server/setup.py
page.tsx
Add FlowEditorWrapper and lazy load FlowEditor
rnd/autogpt_builder/src/app/build/page.tsx
layout.tsx
Remove unused imports from layout.tsx
rnd/autogpt_builder/src/app/layout.tsx
CustomNode.tsx
Improve input handling and validation in CustomNode
rnd/autogpt_builder/src/components/CustomNode.tsx
Flow.tsx
Improve node input data preparation
rnd/autogpt_builder/src/components/Flow.tsx
.eslintrc.json
Update ESLint configuration for TypeScript
rnd/autogpt_builder/.eslintrc.json
next.config.mjs
Update Next.js configuration for export and build
rnd/autogpt_builder/next.config.mjs
package.json
Add TypeScript ESLint plugins and update dependencies
rnd/autogpt_builder/package.json
tsconfig.json
Update TypeScript configuration
rnd/autogpt_builder/tsconfig.json
2 files
page.tsx
Fix ESLint warnings in monitor page
rnd/autogpt_builder/src/app/monitor/page.tsx
calendar.tsx
Fix ESLint warnings in calendar component
rnd/autogpt_builder/src/components/ui/calendar.tsx
2 files
index.html
Remove example files HTML
rnd/example_files/index.html
.gitkeep
Add .gitkeep for frontend directory
rnd/frontend/.gitkeep
1 files
autogpt-server-ci.yml
Add frontend build and test steps in CI
.github/workflows/autogpt-server-ci.yml
1 files
pyproject.toml
Update cx-freeze dependency
rnd/autogpt_server/pyproject.toml