docs(dashboards): add agent skills and contributing docs for dashboard widgets (1/4)#60516
Merged
MattPua merged 2 commits intoJun 3, 2026
Merged
Conversation
This was referenced May 28, 2026
Closed
Closed
Member
Author
Closed
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
from
ac7f208 to
ca8308d
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
from
cf1fe09 to
bb15662
Compare
MattPua
changed the title
Member
Author
|
@gewenyu99 if i could get some of your quick scanning if this is the optimum way to write a skill here for the widgets, that would be much appreciated |
Contributor
|
Reviews (1): Last reviewed commit: "chore(dashboards): add agent skills and ..." | Re-trigger Greptile |
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
from
56d18fa to
d374b34
Compare
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
2 times, most recently
from
05b6772 to
9c2ab4d
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
from
a31a1d0 to
ef958f5
Compare
This was referenced May 28, 2026
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
3 times, most recently
from
eb3dc99 to
dcf2941
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
2 times, most recently
from
d88ec05 to
46da118
Compare
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
2 times, most recently
from
0dbf944 to
c7cc186
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
2 times, most recently
from
a97cc56 to
e4d7d2a
Compare
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
2 times, most recently
from
4831687 to
456423e
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
from
1d9143e to
64db5b4
Compare
MattPua
force-pushed
the
05-28-chore_dashboards_add_dashboard_widgets_docs
branch
from
456423e to
8fde0cf
Compare
MattPua
force-pushed
the
05-28-feat_dashboards_add_mcp_tools_for_dashboard_widgets
branch
from
f2c2fa2 to
b960c05
Compare
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
vdekrijger
approved these changes
Jun 3, 2026
Contributor
vdekrijger
left a comment
vdekrijger
left a comment
There was a problem hiding this comment.
Curious to see the value of these kind of skills in due time 😃!
Good call to explore this! In my mind I feel like "readable/maintainable" code helps the LLM reach similar goals and avoids the need for a dedicated skill, but TBH the landscape is so new that anything is possible, thus super curious to see the results over time vs the potential to get outdated:smiley:
vdekrijger
approved these changes
Jun 3, 2026
Contributor
vdekrijger
left a comment
vdekrijger
left a comment
There was a problem hiding this comment.
Not sure if valuable, but here is some output from my review skill:
┌─────┬──────────────────────────────────────────────────────────────────────────┬──────────────────────────────────────┬──────────┬──────────────────────────────────────┐
│ # │ File │ Line │ Sev │ Finding │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ ✅1 │ .agents/skills/dashboard-widgets/references/checklist-new-widget-type.md │ checklist step 5b ↔ composition ~661 │ 🟠 HIGH │ direct contradiction on exporting │
│ │ ↔ .agents/skills/dashboard-widgets/references/composition.md │ │ │ Kea decorators from *.stories.tsx │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ │ │ │ │ points agents to │
│ ✅2 │ .agents/skills/dashboard-widgets/references/mcp.md │ move-tile rows (diff ~1044/1068) │ 🔵 LOW │ dashboards-move-tile-partial-update, │
│ │ │ │ │ which is enabled: false │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ ✅3 │ .agents/skills/dashboard-widgets/ (whole skill) │ general │ 🟠 HIGH │ per-type touch-list duplicated │
│ │ │ │ (xp) │ across 5 files │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ │ │ │ │ required_scopes docs-only / RBAC │
│ ✅ │ .agents/skills/dashboard-widgets/references/permissions-and-sharing.md │ general │ 🟡 MED │ rule restated ~8× (twice in single │
│ │ │ │ │ files) │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ ✅ │ .agents/skills/dashboard-widgets/references/layout-and-ux.md │ general │ 🟡 MED │ tile min/max-size rule restated ~7× │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ ✅ │ .agents/skills/dashboard-widgets/references/architecture.md │ ~167–180 │ 🔵 LOW │ registry code snippet duplicated │
│ │ │ │ │ verbatim with checklist §7 │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
│ │ │ │ │ frontmatter description / "when to │
│ ✅ │ .agents/skills/dashboard-widgets/SKILL.md │ ~9–14 / ~25–39 │ 🔵 LOW │ use" overlap; hardcoded counts │
│ │ │ │ │ (11/11, migration anchor, constants) │
│ │ │ │ │ will rot │
├─────┼──────────────────────────────────────────────────────────────────────────┼──────────────────────────────────────┼──────────┼──────────────────────────────────────┤
In general it claims that there is a lot of duplication that can be improved. But again as things change fast, I don't know how "bad" this in terms of context clutter is, or if this helps agents give priority to things.
This was referenced Jun 3, 2026
Contributor
MCP UI Apps size report
|
Member
Author
|
Re-enabled |
Contributor
Query snapshots: Backend query snapshots updatedChanges: 1 snapshots (1 modified, 0 added, 0 deleted) What this means:
Next steps:
|
2 tasks
Contributor
|
⏭️ Skipped snapshot commit because branch advanced to The new commit will trigger its own snapshot update workflow. If you expected this workflow to succeed: This can happen due to concurrent commits. To get a fresh workflow run, either:
|
Contributor
|
⏭️ Skipped snapshot commit because branch advanced to The new commit will trigger its own snapshot update workflow. If you expected this workflow to succeed: This can happen due to concurrent commits. To get a fresh workflow run, either:
|
Contributor
Query snapshots: Backend query snapshots updatedChanges: 1 snapshots (1 modified, 0 added, 0 deleted) What this means:
Next steps:
|
Contributor
|
🎭 Playwright report · View test results →
These issues are not necessarily caused by your changes. |
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
3 participants
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.
Problem
Agents and contributors need a single playbook for adding dashboard widget types — file paths, catalog/registry wiring, availability gating, client-side composition patterns, and a checklist for new types.
This PR is 1/4 in the open dashboard widgets stack (docs → placeholders → SLOs → OpenAPI), rebased onto
masterafter #60497 merged.Changes
.agents/skills/dashboard-widgets/skill with architecture, composition, availability, permissions, pitfalls, and new-widget checklist referencesproducts/dashboards/CONTRIBUTING.mdfor widget type authorsupdateWidgetTile/updateDashboardWidgetTile+buildWidgetTileMetadataPatch(not split config/metadata loaders)tryGetDashboardWidgetCatalogEntryheader fallback + body-onlyErrorBoundarywidgetProductAccess.ts/WIDGET_PRODUCT_ACCESS_CHECKS(not a switch inDashboardWidgetItem)dashboard tile addedanddashboard widget addedon first insertrefreshDashboardWidgetsWidgetCardProductIntroductionfor catalog setup promptseditWidgetModalBuilders.ts(spread actions, inline reducers); shared validation viawidgetConfigValidation.tsgroupId+DASHBOARD_WIDGET_GROUP_LABELS; omit redundantheaderLayout/headerMetadefaults; previews inDASHBOARD_WIDGET_PREVIEWSHow did you test this code?
error_tracking_listandsession_replay_listimplementations; no runtime doc verification👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Automatic notifications
Docs update
This PR is the docs update. Add the
skip-inkeep-docslabel.🤖 Agent context
masterafter feat(dashboards): add MCP tools for dashboard widgets (10/11) #60497 merged; downstream PRs are placeholders (feat(dashboards): show widget placeholders on shared dashboards (2/4) #61416), SLOs (feat(dashboards): add slo for widget data delivery (3/4) #61431), and OpenAPI (feat(dashboards): discriminate widget batch-add openapi schema (4/4) #61457)