Enhancement/cli description#157
Merged
Merged
Conversation
- Migrate add_page, admin_page, feed_page to pmui.Page layout - Add hv_list.py app for HoloViews element listing - Polish pn_packages, hvplot_list apps with chips instead of raw JSON - Fix skill_get imports to use core.skills module - Fix FastMCP Root object serialization in client.py - Improve admin table UX: styled status, truncated description, error in expandable row content, clickable view links - Fix feed page CodeEditor height in tabs - Add hv_list thumbnail Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refreshes end-user UX and messaging across the CLI and the built-in Panel UI apps, aligning help text/tests and modernizing the display server pages with Panel Material UI components.
Changes:
- Updated CLI help text, command grouping/panels, and adjusted CLI tests to match.
- Refreshed display server pages (
/feed,/admin,/add) to usepanel_material_uilayouts and added “About” dialogs. - Added/updated UI demo apps (e.g., new
hv_listapp) and thumbnails; tweaked search UI default max results.
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/test_cli.py | Updates assertions to match new CLI help/command descriptions. |
| src/holoviz_mcp/thumbnails/hv_list.png | Adds a new thumbnail asset for the hv_list demo app. |
| src/holoviz_mcp/display_mcp/pages/feed_page.py | Migrates feed UI to Material UI components; adds About/GitHub header actions. |
| src/holoviz_mcp/display_mcp/pages/admin_page.py | Improves snippet manager table UX (formatters, row content, error display) and migrates to Material UI page layout. |
| src/holoviz_mcp/display_mcp/pages/add_page.py | Migrates add-visualization form to Material UI components; improves status/link UX; adds About/GitHub header actions. |
| src/holoviz_mcp/client.py | Adds post-processing to normalize FastMCP Root objects in result.data for serialization. |
| src/holoviz_mcp/cli.py | Reworks CLI descriptions, help panels, and several command docstrings/help text. |
| src/holoviz_mcp/apps/skill_get.py | Switches to core.skills APIs and adapts to new list_skills() return shape; fixes a docstring typo. |
| src/holoviz_mcp/apps/search.py | Increases default max results from 2 to 5. |
| src/holoviz_mcp/apps/pn_packages.py | Replaces JSON output with styled chip-based UI content. |
| src/holoviz_mcp/apps/hvplot_list.py | Replaces JSON output with styled chip-based UI content. |
| src/holoviz_mcp/apps/hv_list.py | New Material UI demo app for the hv_list tool. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.