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.

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

Implement new Agent Library to replace Monitor page #8774

Closed
7 of 15 tasks
Pwuts opened this issue Nov 26, 2024 — with Linear · 2 comments
Closed
7 of 15 tasks

Implement new Agent Library to replace Monitor page #8774

Pwuts opened this issue Nov 26, 2024 — with Linear · 2 comments
Assignees
Labels
platform/frontend AutoGPT Platform - Front end UI

Comments

Copy link
Member

Pwuts commented Nov 26, 2024

Wireframes: Library & Monitor

Notes

  • The designs come directly from the design & product department. Engineering feedback can still cause them to change. So:
    • Functionality first
      • We're going for a functional MVP first, without paying too much attention to the styling.
    • Design last
      • Once it all works we'll turn to the design and styling to make it all look nice and consistent.
  • I have created this Discord thread to keep in sync on the development work
@Abhi1992002
Copy link
Contributor

Hey @Pwuts, can I give one of the issues a try?

Copy link
Member Author

Pwuts commented Nov 26, 2024

Hi @Abhi1992002, please hold off for now as I'm still fleshing out these issues. I'll let you know when they're ready to be picked up.

Update: thread on Discord here to sync on development

@Pwuts Pwuts self-assigned this Dec 12, 2024
@Pwuts Pwuts changed the title Implement new Library page to replace Monitor page Implement new Library to replace Monitor page Jan 3, 2025
@Pwuts Pwuts changed the title Implement new Library to replace Monitor page Implement new Agent Library to replace Monitor page Jan 3, 2025
github-merge-queue bot pushed a commit that referenced this issue Feb 19, 2025
- Resolves #8780
- Part of #8774

### 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:
- [x] I have clearly listed my changes in the PR description
- [x] I have made a test plan
- [x] 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

---------

Co-authored-by: Nicholas Tindle <[email protected]>
Co-authored-by: Nicholas Tindle <[email protected]>
Co-authored-by: Swifty <[email protected]>
Co-authored-by: Zamil Majdy <[email protected]>
ntindle added a commit that referenced this issue Feb 25, 2025
- Resolves #8774
  - Resolves #8775
- Includes back-end work for #9168
- Partially implements #8776
- Partially implements #8777

### Changes 🏗️

- Add `/library` page
- Change target of "Library" navigation link from `/monitoring` to
`/library`
- Move `/agents/[id]` page to `/library/agents/[id]`
- Set application background color to `bg-neutral-50`
- Redirect to new library agent's "runs" page (`/library/agents/[id]`)
after adding from marketplace

Further (technical) frontend changes:
- Add types and client methods for all library API endpoints
- Added `primary` variant to `agptui/Button` component

Backend changes:
- Add functionality to library backend
  - Aggregate agent status
  - Image generation for use in library view
    - Add `LibraryAgent.imageUrl` column to DB schema
  - Sorting & pagination
  - Explicit relation between library agents and their graph's creator
- Refactor & update API endpoints for DX
- Other minor refactoring
- Add missing but required `MEDIA_GCS_BUCKET_NAME` to `.env.example`

### Checklist 📋

#### For code changes:
- [x] I have clearly listed my changes in the PR description
- [x] I have made a test plan
- [x] I have tested my changes according to the test plan:
  - `/library`
    - [x] Create agent from scratch -> should show up on `/library`
- [x] Add agent to library from marketplace -> should show up on
`/library`
- [x] Click on agent in `/library` -> should go to
`/library/agents/[id]`
    - [x] Test sorting & pagination functionality
  - `/library/agents/[id]`
    - [x] Fill out inputs and click "Run" -> should run like normally
    - [x] Select completed agent run -> should show all inputs & outputs
- [x] Click "run again" on a completed agent run -> should run
successfully with same input
  - [x] `/monitoring` should still work the same as before

---------

Co-authored-by: abhi1992002 <[email protected]>
Co-authored-by: Reinier van der Leer <[email protected]>
Co-authored-by: Nicholas Tindle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/frontend AutoGPT Platform - Front end UI
Projects
None yet
Development

No branches or pull requests

3 participants