Run any Skill in Manus with one click

plot-type-check

Stars6

Forks2

UpdatedJuly 13, 2026 at 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).

Installation

Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.

Run Skill in Manus

Source

nikiitin

nikiitin/RING-5

View GitHub Repository View Creator Repositories

Download

Run Skill in Manus

Related occupationsSOC

Based on SOC occupation classification

Software DevelopersComputer and Mathematical Occupations·SOC 15-1252

SKILL.md

readonly

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)

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

More from this repository

same repository

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