一键在 Manus 中运行任何 Skill

plot-type-check

星标6

分支2

更新时间2026年7月13日 22:00

Gate that a plot type is correctly implemented in RING-5 — BasePlot contract, engine-agnostic traces, PlotFactory registration, dual-engine render, config UI, serialization. Use to verify a new/changed chart type before merging (complements the add-plot-type build skill).

安装

用 Codex 或 Claude 帮你安装复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。

在 Manus 中运行

来源

nikiitin

nikiitin/RING-5

打开 GitHub 仓库查看创作者相关仓库

下载

在 Manus 中运行

Plot-type check (the chart conformance gate)

A plot type is correct only if it is engine-agnostic and fully wired. To build one use the add-plot-type skill; this skill verifies the result. Types live in src/web/pages/ui/plotting/types/; the registry is src/web/pages/ui/plotting/plot_factory.py::PlotFactory.

Checklist

BasePlot contract — the class subclasses BasePlot and implements both abstract methods:
- create_traces(self, data, config) -> TraceBuildResult
- get_legend_column(self, config) -> str | None create_figure() is inherited — only overridden with good reason. (GroupedStackedBarPlot extends StackedBarPlot, not BasePlot directly — that's allowed.)
Engine-agnostic — the plot builds TraceConfig/TraceBuildResult only. No go.* and no plt.* inside the plot class. Styling is not the plot's job (it lives in FigureConfig + the connectors — see rendering-figureconfig).
plot_type set before super().__init__ in __init__.
Registered (snake_case) — present in PlotFactory._plot_classes with a snake_case key (grouped_bar, not groupedBar — shapers use camelCase, plots use snake_case) and a matching _plot_metadata entry (display_name, icon, category ∈ basic|comparison|distribution).
Exported from src/web/pages/ui/plotting/types/__init__.py (so registration runs).
Dual-engine — renders under both Plotly and Matplotlib (toggle ring5_engine_mode); the trace converters are trace_to_plotly.py / matplotlib_trace_renderer.py.
Config UI — if it needs options beyond the shared X/Y/color selectors, a component under src/web/components/plotting/config/ is wired via plot_config_ui.py/PlotConfigUIMixin, with per-plot-unique widget keys f"{prefix}_{field}_{plot_id}".
Serialization round-trips — BasePlot.to_dict/from_dict + PlotFactory rebuild it; any new state is plain/JSON-serializable so portfolios survive save/load.
Conventions — default palette wong; error-bar column is {y}.sd.
Coverage — there is a unit test exercising create_traces/create_figure with sample_data. dual_axis_bar_dot is WIP and intentionally excluded from coverage — don't count it as a model.

# registry sanity
grep -n "_plot_classes\|_plot_metadata" src/web/pages/ui/plotting/plot_factory.py
# engine-leak smell (should be empty inside types/)
grep -rn "import plotly\|plotly.graph_objects\|matplotlib\|plt\." src/web/pages/ui/plotting/types/

Keep this skill sharp

Canon, not history — edit in place when the plot contract moves:

The abstract methods and registry shape above mirror base_plot.py and plot_factory.py. If a new abstract method is added or the metadata schema changes, update this checklist.
New conformance failure caught in review? Add it here (and to add-plot-type if it's a build step); durable facts → a memory.

Verify after

make arch-check && ./python_venv/bin/mypy src/ && ./python_venv/bin/pytest tests/unit -k plot -q

同仓库更多 Skills

同仓库

add-plot-type

nikiitin/RING-5

Add a new plot/chart type to RING-5 (subclass BasePlot, emit engine-agnostic traces, register in PlotFactory, wire config UI, test). Use when the user asks to add or modify a visualization/chart/plot type.

2026-07-136

add-shaper

nikiitin/RING-5

Add a new data-transformation shaper to RING-5's pipeline (subclass Shaper, add a discriminated-union TypedDict config, register in ShaperFactory with a camelCase key, add the UI config). Use when the user wants a new transform/shaper/pipeline step.

2026-07-136

api-check

nikiitin/RING-5

Gate the ApplicationAPI facade contract — the single Web→Core entry point, its three sub-APIs, and the async parse/scan protocol. Use when verifying API usage, reviewing facade/service changes, or checking that the UI talks to core only through the facade.

2026-07-136

architecture-check

nikiitin/RING-5

Gate the RING-5 architecture — the 7 hard rules and the strict 3-layer dependency direction (Web → Core ← Parsing). Use before merging, after any structural/refactor change, or when asked to "check architecture", "run the arch gate", or verify layer boundaries.

2026-07-136

clean-stale-paths

nikiitin/RING-5

Sweep code and docs for dead, duplicated, or stale paths/symbols and remove them — repoint drifted paths, delete docs for removed subsystems, and enforce canon-only docs (no "was X, now Y" history). Use when asked to clean up stale references, dead paths, doc debt, or legacy shims/aliases.

2026-07-136

e2e-streamlit-testing

nikiitin/RING-5

Write or debug RING-5's Streamlit UI tests — the no-browser AppTest/ui_logic/ui_unit layers and the Playwright browser e2e/visual suite (Page Object Model, data-testid selectors, and Streamlit-specific gotchas). Use for any UI test, Playwright, AppTest, screenshot, or flaky-UI-test task.

2026-07-136

name	plot-type-check
description	Gate that a plot type is correctly implemented in RING-5 — BasePlot contract, engine-agnostic traces, PlotFactory registration, dual-engine render, config UI, serialization. Use to verify a new/changed chart type before merging (complements the add-plot-type build skill).

Plot-type check (the chart conformance gate)

Checklist

BasePlot contract — the class subclasses BasePlot and implements both abstract methods:
- create_traces(self, data, config) -> TraceBuildResult
- get_legend_column(self, config) -> str | None create_figure() is inherited — only overridden with good reason. (GroupedStackedBarPlot extends StackedBarPlot, not BasePlot directly — that's allowed.)
Engine-agnostic — the plot builds TraceConfig/TraceBuildResult only. No go.* and no plt.* inside the plot class. Styling is not the plot's job (it lives in FigureConfig + the connectors — see rendering-figureconfig).
plot_type set before super().__init__ in __init__.
Registered (snake_case) — present in PlotFactory._plot_classes with a snake_case key (grouped_bar, not groupedBar — shapers use camelCase, plots use snake_case) and a matching _plot_metadata entry (display_name, icon, category ∈ basic|comparison|distribution).
Exported from src/web/pages/ui/plotting/types/__init__.py (so registration runs).
Dual-engine — renders under both Plotly and Matplotlib (toggle ring5_engine_mode); the trace converters are trace_to_plotly.py / matplotlib_trace_renderer.py.
Config UI — if it needs options beyond the shared X/Y/color selectors, a component under src/web/components/plotting/config/ is wired via plot_config_ui.py/PlotConfigUIMixin, with per-plot-unique widget keys f"{prefix}_{field}_{plot_id}".
Serialization round-trips — BasePlot.to_dict/from_dict + PlotFactory rebuild it; any new state is plain/JSON-serializable so portfolios survive save/load.
Conventions — default palette wong; error-bar column is {y}.sd.
Coverage — there is a unit test exercising create_traces/create_figure with sample_data. dual_axis_bar_dot is WIP and intentionally excluded from coverage — don't count it as a model.

# registry sanity
grep -n "_plot_classes\|_plot_metadata" src/web/pages/ui/plotting/plot_factory.py
# engine-leak smell (should be empty inside types/)
grep -rn "import plotly\|plotly.graph_objects\|matplotlib\|plt\." src/web/pages/ui/plotting/types/

Keep this skill sharp

Canon, not history — edit in place when the plot contract moves:

The abstract methods and registry shape above mirror base_plot.py and plot_factory.py. If a new abstract method is added or the metadata schema changes, update this checklist.
New conformance failure caught in review? Add it here (and to add-plot-type if it's a build step); durable facts → a memory.

Verify after

make arch-check && ./python_venv/bin/mypy src/ && ./python_venv/bin/pytest tests/unit -k plot -q