-
Notifications
You must be signed in to change notification settings - Fork 0
Build CrewAI asset management dashboard #16
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| # Change Log - 2025-11-24 15:58:34 | ||
|
|
||
| ## web/src/app/page.js | ||
| - Rebuilt the home page into a CrewAI Asset Manager layout with richer hero content, control-room stats, operational reads, and environment/storage overview cards. | ||
| - Added structured resource metadata (agents, tools, toolsets, crews, crew configs) plus section descriptions, search fields, action sets, and operational alerts. | ||
| - Wired dynamic listings and controls so the page feeds expanded data into reusable table rendering with actions. | ||
|
|
||
| ## web/src/components/Navigation.js | ||
| - Updated navigation to reflect CrewAI asset types (agents, tools, toolsets, crews, crew configs) with refreshed iconography and helper copy. | ||
|
|
||
| ## web/src/components/FileListing.js | ||
| - Replaced file-centric table with asset-aware listing that supports configurable columns, status badges, multi-value chips, and contextual action buttons. | ||
| - Added flexible search coverage across configurable fields and improved empty-state messaging. | ||
|
|
||
| ## web/src/components/SearchBar.js | ||
| - Tuned placeholder text to match CrewAI asset search and retained submit handling for query updates. | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,6 @@ | ||||||||||||||||||||||||||||
| # Recommendations - 2025-11-24 15:58:34 | ||||||||||||||||||||||||||||
| - Add API-backed data fetching and mutations (e.g., Next.js route handlers) so the asset lists reflect real CrewAI definitions instead of mock data. | ||||||||||||||||||||||||||||
| - Introduce per-asset detail drawers or modals with markdown rendering for configs, run history, and dependency graphs. | ||||||||||||||||||||||||||||
| - Layer in filtering by status, environment, and owner plus saved views for release/incident responders. | ||||||||||||||||||||||||||||
| - Wire quick actions to actual workflows: trigger crew runs via queue/webhook, scaffold new assets from templates, and surface audit logs. | ||||||||||||||||||||||||||||
| - Add persistence for operational reads with timestamps, severity, and links to source documents. | ||||||||||||||||||||||||||||
|
Comment on lines
+1
to
+6
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix markdown heading spacing to satisfy lint
-# Recommendations - 2025-11-24 15:58:34
-- Add API-backed data fetching and mutations (e.g., Next.js route handlers) so the asset lists reflect real CrewAI definitions instead of mock data.
+# Recommendations - 2025-11-24 15:58:34
+
+- Add API-backed data fetching and mutations (e.g., Next.js route handlers) so the asset lists reflect real CrewAI definitions instead of mock data.📝 Committable suggestion
Suggested change
🧰 Tools🪛 markdownlint-cli2 (0.18.1)3-3: Headings should be surrounded by blank lines (MD022, blanks-around-headings) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||
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.
Clean up markdown heading spacing for the change log
All the
##section headings are missing a blank line after the heading, which is whatmarkdownlintis complaining about. Easy formatting win.📝 Committable suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
3-3: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
8-8: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
11-11: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
15-15: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
🤖 Prompt for AI Agents