docs(release): record h4 pro boundary approval

Author: iexitdev

docs/internal/completion-audit.md

@@ -5,19 +5,21 @@
     {
       "id": "h4",
       "name": "Pro/free boundary approval",
-      "status": "open",
+      "status": "approved",
       "requiredFor": ["H5", "H6"],
       "evidence": [
         "docs/release/h4-pro-scope.md",
         "docs/release/h4-owner-decision-memo.md"
       ],
-      "pendingDecisions": [
-        "final free-vs-Pro feature boundary",
-        "@chart-kit/pro package split",
-        "@chart-kit/skia-renderer packaging",
-        "license enforcement timing",
-        "which preview interactions remain free at beta",
-        "CandlestickChart beta visibility"
+      "approvedAt": "2026-05-06",
+      "approvedBy": "owner",
+      "decisions": [
+        "Free v2 remains useful with safe layout, SVG renderer, baseline charts, themes, accessibility helpers, and compatibility facade.",
+        "@chart-kit/pro will be a separate package later.",
+        "@chart-kit/skia-renderer remains a separate optional preview renderer package until native install, parity, and benchmark evidence are complete.",
+        "No license enforcement before beta.",
+        "Advanced interactions, range selector, pan/zoom, animation, financial charts, Skia, and large-data workflows stay preview/Pro-candidate for beta.",
+        "CandlestickChart remains Financial Preview."
       ]
     },
     {

docs/release/evidence/owner-gates.json

@@ -1,16 +1,16 @@
 # H4 Owner Decision Memo
 
-This memo converts the Pro/free boundary gate into explicit choices. It is a recommendation, not approval.
+This memo records the H4 Pro/free boundary approval from May 6, 2026.
 
-## Recommended Decision
+## Approved Decision
 
-Approve the Pro direction as a package-boundary plan, but do not add license enforcement yet.
+The owner approved the Pro direction as a package-boundary plan, with no license enforcement before beta.
 
 Reason: the current implementation keeps the free/baseline public package useful, exposes current advanced workflows through `@chart-kit/react-native/pro-preview`, and records Pro-candidate feature buckets plus prop-level triggers in `packages/pro`. The final paid package and license behavior should wait until native evidence and beta feedback are stronger.
 
 ## Decision 1: Free Baseline
 
-Recommendation:
+Approved:
 
 - keep `react-native-chart-kit` as the compatibility package
 - keep `@chart-kit/react-native` as the clean modern free API
@@ -21,21 +21,21 @@ Rationale: existing users need a low-friction upgrade path, and new adopters nee
 
 ## Decision 2: Pro Package
 
-Recommendation: ship Pro as a separate package later.
+Approved: ship Pro as a separate package later.
 
 Suggested package direction:
 
 ```text
 @chart-kit/react-native         # modern free package
 @chart-kit/pro                  # paid workflows and composition helpers
-@chart-kit/skia-renderer        # optional renderer package, used by Pro docs/examples when approved
+@chart-kit/skia-renderer        # separate optional preview renderer until native evidence is complete
 ```
 
 Rationale: a separate package keeps license checks, paid workflows, and optional native dependencies out of the free runtime.
 
 ## Decision 3: Pro Scope
 
-Recommendation: keep the owner-proposed commercial buckets as Pro candidates:
+Approved: keep the owner-proposed commercial buckets as Pro candidates:
 
 - production layout engine depth
 - production interactions
@@ -50,7 +50,7 @@ The current registry records these as `chartKitProPreviewFeatures`.
 
 ## Decision 4: Prop-Level Triggers
 
-Recommendation: approve the trigger metadata as the working split map, not as runtime gating.
+Approved: use the trigger metadata as the working split map, not as runtime gating.
 
 Examples currently tracked:
 
@@ -73,43 +73,43 @@ Rationale: this lets the project decide later whether a component, a prop, or a
 
 ## Decision 5: License Timing
 
-Recommendation: no license enforcement before beta.
+Approved: no license enforcement before beta.
 
 Rationale:
 
 - API feedback is more important than monetization code at this stage
-- free/pro boundaries are still under owner review
+- paid implementation and native evidence are still incomplete
 - native Skia and performance evidence are not complete
 - license checks should never live in `packages/core`
 
 ## Decision 6: Beta Labels
 
-Recommendation:
+Approved:
 
 - label advanced touch workflows, range selector, pan/zoom, animation previews, financial charts, Skia, and large-data features as preview or Pro candidate
 - keep `CandlestickChart` visible only as Financial Preview
 - keep `@chart-kit/react-native/pro-preview` as the temporary review entrypoint
 
-## Recommended H4 Outcome
+## H4 Outcome
 
-Approve the strategy, not the final paid implementation:
+Approved the strategy, not the final paid implementation:
 
 1. Free v2 stays useful and trustworthy.
 2. Pro focuses on production pain and commercial workflows.
 3. Pro and Skia remain preview packages for beta.
 4. License enforcement is deferred.
-5. Final free-vs-Pro labels are revisited after native evidence and beta feedback.
+5. Final paid implementation details are revisited after native evidence and beta feedback.
 
-Record approval only after the owner accepts or edits these decisions:
+Recorded approval:
 
 ```sh
 npm run release:owner:record -- \
   --gate h4 \
-  --approved-by <owner> \
-  --approved-at <yyyy-mm-dd> \
+  --approved-by owner \
+  --approved-at 2026-05-06 \
   --decision "Free v2 remains useful with safe layout, SVG renderer, baseline charts, themes, accessibility helpers, and compatibility facade." \
   --decision "@chart-kit/pro will be a separate package later." \
-  --decision "@chart-kit/skia-renderer remains an optional renderer package until native evidence is complete." \
+  --decision "@chart-kit/skia-renderer remains a separate optional preview renderer package until native install, parity, and benchmark evidence are complete." \
   --decision "No license enforcement before beta." \
   --decision "Advanced interactions, range selector, pan/zoom, animation, financial charts, Skia, and large-data workflows stay preview/Pro-candidate for beta." \
   --decision "CandlestickChart remains Financial Preview."

docs/release/h4-owner-decision-memo.md

@@ -1,14 +1,14 @@
 # H4 Pro Scope Decision Packet
 
-Status on May 5, 2026: owner monetization direction captured, package separation not approved for release yet.
+Status on May 6, 2026: H4 approved with the recommendations in this packet. Pro remains a separate future package, Skia remains a separate optional preview renderer, and no license enforcement is added before beta.
 
-This packet defines the recommended free-vs-Pro boundary for the v2 beta path. It does not add license checks. The implementation now keeps the main modern package barrel focused on the free/baseline API and exposes current Pro-candidate workflows through an explicit preview subpath until the owner approves the final package split and release timing.
+This packet defines the approved free-vs-Pro boundary for the v2 beta path. It does not add license checks. The implementation keeps the main modern package barrel focused on the free/baseline API and exposes current Pro-candidate workflows through an explicit preview subpath while paid package implementation and native evidence continue.
 
-## Recommended H4 Decision
+## Approved H4 Decision
 
 Keep the free library useful and trustworthy. Make Pro about the places real production apps bleed: layout certainty, touch workflows, commercial chart types, export, premium templates, and performance.
 
-Recommended beta decision:
+Approved beta decision:
 
 - ship free v2 with SVG renderer, modern baseline charts, compatibility facade, themes, accessibility helpers, safe default layout, and basic examples
 - keep currently implemented advanced interactions, range selector, pan/zoom, animation, and financial charts visible only as preview/pro-candidate features until API feedback stabilizes
@@ -59,15 +59,15 @@ react-native-chart-kit          # compatibility package and migration bridge
 @chart-kit/pro                  # paid workflow package that composes free primitives
 ```
 
-`packages/skia-renderer` currently exposes capability metadata, install guidance, and an injected Skia primitive adapter. `LineChart` can accept the injected renderer for its main plot, range selector, sticky-axis, path-local area gradients, threshold rect clips, measured text anchors, default marker, default legend, default tooltip, and debug-layout surfaces. `BarChart` can accept the injected renderer for its chart body, sticky Y-axis, and default tooltip overlay. `PieChart` and `DonutChart` can accept the injected renderer for slices, connector lines, and SVG text labels. `ProgressChart`, `ProgressRing`, `ContributionGraph`, and `CalendarHeatmap` can accept the injected renderer for their chart primitives and SVG text labels. `CombinedChart` can accept the injected renderer for bars, lines, axes, legend, selection overlay, and default tooltip. `CandlestickChart` can accept the injected renderer for its OHLC body, volume bars, session markers, selection overlay, default tooltip, and range selector. `npm run skia:parity` covers the local Skia primitive tests plus LineChart, BarChart, PieChart, ProgressChart, ContributionGraph, CombinedChart, and CandlestickChart renderer contracts. Sticky-axis labels require a Skia font, and native install evidence plus native renderer parity evidence remain pending in [skia-renderer-matrix.json](evidence/skia-renderer-matrix.json). The package does not add license gating until H4 approves the Pro/package split.
+`packages/skia-renderer` currently exposes capability metadata, install guidance, and an injected Skia primitive adapter. `LineChart` can accept the injected renderer for its main plot, range selector, sticky-axis, path-local area gradients, threshold rect clips, measured text anchors, default marker, default legend, default tooltip, and debug-layout surfaces. `BarChart` can accept the injected renderer for its chart body, sticky Y-axis, and default tooltip overlay. `PieChart` and `DonutChart` can accept the injected renderer for slices, connector lines, and SVG text labels. `ProgressChart`, `ProgressRing`, `ContributionGraph`, and `CalendarHeatmap` can accept the injected renderer for their chart primitives and SVG text labels. `CombinedChart` can accept the injected renderer for bars, lines, axes, legend, selection overlay, and default tooltip. `CandlestickChart` can accept the injected renderer for its OHLC body, volume bars, session markers, selection overlay, default tooltip, and range selector. `npm run skia:parity` covers the local Skia primitive tests plus LineChart, BarChart, PieChart, ProgressChart, ContributionGraph, CombinedChart, and CandlestickChart renderer contracts. Sticky-axis labels require a Skia font, and native install evidence plus native renderer parity evidence remain pending in [skia-renderer-matrix.json](evidence/skia-renderer-matrix.json). Per H4, Skia remains a separate optional preview renderer package and does not add license gating before beta.
 
 `packages/pro` currently exists only as a preview feature-registry, surface-boundary, and injected composition package. It has no license checks, no runtime activation, and no gated chart implementations. Its registry now mirrors the owner-proposed commercial buckets: `pro-layout-engine`, `pro-interactions`, `pro-chart-types`, `pro-export`, `pro-theme-templates`, `pro-performance`, `skia-renderer`, and `accessibility-reports`. Each feature entry records the included paid capabilities, the commercial rationale, and the free guardrail that keeps baseline v2 useful. Its package-boundary metadata classifies current exports as compatibility, free baseline, or Pro candidate, and it now tracks prop-level Pro candidate triggers such as `LineChart.rangeSelector`, `LineChart.interaction=scrub`, `BarChart.mode=grouped`, `BarChart.orientation=horizontal`, `CombinedChart`, `CandlestickChart`, active donut selection, and injected renderer props. This makes the final split reviewable before moving runtime code. Its React Native preview helper accepts an injected `@chart-kit/react-native/pro-preview` module and selects Pro-candidate exports without statically importing the free runtime package.
 
 `@chart-kit/react-native/pro-preview` now exists as an explicit preview entrypoint for current Pro-candidate workflows. It exposes the current advanced LineChart, BarChart, DonutChart, CombinedChart, CandlestickChart, and chart-selection surface while the main `@chart-kit/react-native` barrel stays focused on the free/baseline API. This is not license gating and does not decide the final paid boundary; it gives demos, docs, and `@chart-kit/pro` a clean target before runtime code moves.
 
 `npm run surface:check` now also verifies that public docs, showcase source, and docs-example type fixtures do not import explicit Pro-preview-only exports from the main modern package barrel.
 
-The beta publish list is controlled by [package-manifest.json](evidence/package-manifest.json). `@chart-kit/skia-renderer` and `@chart-kit/pro` remain pack-checked preview packages with `publishInBeta: false` until H4 approves the final Pro/package split.
+The beta publish list is controlled by [package-manifest.json](evidence/package-manifest.json). Per H4, `@chart-kit/skia-renderer` and `@chart-kit/pro` remain pack-checked preview packages with `publishInBeta: false` until paid-package implementation and native evidence are ready.
 
 Implementation principles:
 
@@ -80,7 +80,7 @@ Implementation principles:
 
 ## Preview Feature Labels
 
-Until H4 implementation is approved, label these as preview or Pro candidate in docs/showcase language instead of calling them final free or final Pro:
+After H4 approval, keep these labeled as preview or Pro candidate in docs/showcase language instead of calling them final free or final Pro:
 
 - advanced tooltip/crosshair/scrub workflows
 - LineChart range selector and brush selection
@@ -90,9 +90,9 @@ Until H4 implementation is approved, label these as preview or Pro candidate in
 - Combo/Dual-axis polish and advanced grouped/horizontal bar variants
 - future Skia renderer experiments
 
-## Owner Decisions Needed
+## Approved Owner Decisions
 
-H4 approval needs explicit answers:
+H4 approved these answers on May 6, 2026:
 
 1. Does `@chart-kit/pro` ship as a separate package?
 2. Does `@chart-kit/skia-renderer` ship as a separate optional package or inside Pro?
@@ -101,11 +101,9 @@ H4 approval needs explicit answers:
 5. Should combo/dual-axis, grouped bars, horizontal stacked bars, and advanced contribution heatmap move to Pro candidates?
 6. Should the first beta publish without any license enforcement?
 
-Recommended answers:
-
-1. Yes, separate package.
-2. Separate optional package, with Pro docs and examples depending on it.
-3. Keep simple press handlers and baseline selection examples free; label production touch workflows as Pro candidates until pricing is final.
-4. Yes, but only as Financial Preview until native performance and the free-vs-Pro boundary are approved.
-5. Yes, where they are commercial workflows rather than baseline chart primitives.
-6. Yes. Add license enforcement only after API feedback and package boundaries settle.
+7. Yes, separate package.
+8. Separate optional preview package, with Pro docs and examples depending on it after native evidence is complete.
+9. Keep simple press handlers and baseline selection examples free; label production touch workflows as Pro candidates until pricing is final.
+10. Yes, but only as Financial Preview until native performance evidence is complete.
+11. Yes, where they are commercial workflows rather than baseline chart primitives.
+12. Yes. Add license enforcement only after API feedback and package boundaries settle.