Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(platform/library): Library v2 > Agent Runs page #9051

Merged
merged 79 commits into from
Feb 19, 2025
Merged

feat(platform/library): Library v2 > Agent Runs page #9051

merged 79 commits into from
Feb 19, 2025

Conversation

Pwuts
Copy link
Member

@Pwuts Pwuts commented Dec 18, 2024 •
edited
Loading

Changes 🏗️

  • Add new UI components
  • Add /agents/[id] page, with sub-components:
    • AgentRunsSelectorList
      • AgentRunSummaryCard
        • AgentRunStatusChip
    • AgentRunDetailsView
    • AgentRunDraftView
    • AgentScheduleDetailsView

Backend improvements:

  • Improve output of execution-related API endpoints: return GraphExecution instead of NodeExecutionResult[]
  • Reduce log spam from Prisma in tests

General frontend improvements:

  • Hide nav link names on smaller screens to prevent navbar overflow
  • Clean up styling and fix sizing of agptui/Button

Technical frontend improvements:

  • Fix tailwind config size increments
  • Rename font-poppin -> font-poppins
  • Clean up component implementations and usages
    • Yeet all occurrences of variant="default"
    • Remove default button variant as duplicate of outline; make outline the default
  • Fix minor typing issues

DX:

  • Add front end type-check step to pre-commit config
  • Fix logging setup in conftest.py

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • /agents/[id] (new)
      • Go to page -> list of runs loads
      • Create new run -> runs; all I/O is visible
      • Click "Run again" -> runs again with same input
    • /monitoring (existing)
      • Go to page -> everything loads
      • Selecting agents and agent runs works

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end size/l labels Dec 18, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 18, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 13c7692
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67b651da6307a60008535e88

@netlify Netlify
Copy link

netlify bot commented Dec 18, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 13c7692
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67b651daf59e5b00085d8203

@github-actions github-actions bot added size/xl and removed size/l labels Dec 19, 2024
@Pwuts Pwuts force-pushed the pwuts/open-2088-implement-new-agent-runs-page branch from 76b9180 to 9d4ba1e Compare December 19, 2024 00:41
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Dec 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 3, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 3, 2025

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 6, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 6, 2025

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

ntindle and others added 10 commits January 12, 2025 17:11
@ntindle
Re-enable the tests in monitor.spec.ts and then ensure they pass
0da8a3a 
Enable the tests in `monitor.spec.ts`.

* Remove `test.describe.skip` to enable the tests.
* Ensure the tests are now running and passing successfully.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/Significant-Gravitas/AutoGPT?shareId=XXXX-XXXX-XXXX-XXXX).
@Pwuts
fix library.db.update_agent_version_in_library(..)
3d21a12 
- Resolves #9250
@Pwuts
fix other DB calls in v2.library.db
d07be7f
@Pwuts
feat(backend): Library v2 Agents and Presets
577357d 
This re-applies the following commits:
- fd6f28f
- 4b17cc9
@Pwuts
Merge branch 'pwuts/open-2309-fix-db-error-could-not-find-field-at' i…
700f2a3 
…nto pwuts/open-2314-fix-up-and-re-introduce-library-v2-back-end-work
@ntindle
Merge branch 'dev' into ntindle/reenable-tests
6aaed0a
@ntindle
feat: tests
a5b9dc6
@ntindle
Merge remote-tracking branch 'origin/dev' into ntindle/reenable-tests
7ab94c9
@ntindle
feat: break apart the add blocks test
8e82469
@ntindle
fix: stop waiting for network to settle
81a137d
@Pwuts Pwuts marked this pull request as ready for review February 19, 2025 20:18
@Pwuts Pwuts requested a review from a team as a code owner February 19, 2025 20:18
@Pwuts Pwuts requested review from Bentlybro, majdyz and ntindle and removed request for a team February 19, 2025 20:18
@Pwuts Pwuts enabled auto-merge February 19, 2025 21:59
@Pwuts Pwuts changed the title feat(platform): Agent Library v2 / Runs page feat(platform/library): Library v2 > Agent Runs page Feb 19, 2025
@Pwuts Pwuts added this pull request to the merge queue Feb 19, 2025
Merged via the queue into dev with commit 296eee0 Feb 19, 2025
27 checks passed
@Pwuts Pwuts deleted the pwuts/open-2088-implement-new-agent-runs-page branch February 19, 2025 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ntindle ntindle ntindle approved these changes

@Bentlybro Bentlybro Awaiting requested review from Bentlybro Bentlybro is a code owner automatically assigned from Significant-Gravitas/maintainers

@majdyz majdyz Awaiting requested review from majdyz majdyz is a code owner automatically assigned from Significant-Gravitas/maintainers

Assignees
No one assigned
Labels
platform/backend AutoGPT Platform - Back end platform/frontend AutoGPT Platform - Front end size/xl
Projects
Frontend
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Pwuts @ntindle @Swiftyos