Skip to content

Commit a746bbd

Browse files
iexitdeviexitdev
committed
docs(release): record npm token publish blocker
1 parent c100d0a commit a746bbd

3 files changed

Lines changed: 24 additions & 18 deletions

File tree

‎docs/internal/completion-audit.md‎

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -46,29 +46,29 @@ Concrete success criteria:
4646

4747
## Missing Or Not Fully Verified
4848

49-
| Requirement | Missing evidence | Why it matters |
50-
| ---------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------- |
51-
| Native release workflow | Local iOS and Android release builds passed, and the recorder/checker require a run URL, commit, and iOS/Android artifact links before marking workflow evidence complete; no green native workflow artifact is recorded yet. A May 6, 2026 manual dispatch attempt failed because `native-release.yml` is only on `next`, while GitHub Actions exposes workflow dispatches from the default `master` branch. | Required before production beta/RC confidence |
52-
| Native e2e/runtime QA | iOS simulator and Android emulator launch smoke exist, and `docs/release/evidence/native-runtime-qa.json` records partial status. `docs/release/native-qa-checklists.md` provides row-by-row execution guidance. `docs/release/evidence/native-runtime-matrix.json` has 16 pending page/platform rows | Gestures, scroll conflicts, text rendering, and release behavior can differ on device |
53-
| RN CLI native projects | No generated RN CLI `ios/` or `android/` projects are checked in | Source-level RN CLI example exists, but release-build evidence still requires native projects |
54-
| Pro paid implementation | H4 approved the Pro/free boundary, but `@chart-kit/pro` remains a preview registry/composition package without license enforcement, paid implementations, or runtime activation | Required before selling or labeling Pro as a finished paid package |
55-
| Skia renderer | `packages/skia-renderer` exposes an injected primitive adapter. Baseline free chart surfaces can accept it: LineChart, BarChart, PieChart/DonutChart, ProgressChart/ProgressRing, and ContributionGraph/CalendarHeatmap. CombinedChart and CandlestickChart also accept it for Pro-preview workflows. `npm run skia:parity` covers local primitive and renderer-contract parity; `docs/release/evidence/skia-renderer-evidence.json` and `docs/release/evidence/skia-renderer-matrix.json` record the remaining gaps | Native install, native renderer parity, and native Skia performance evidence are not complete |
56-
| Native performance benchmark | Protocol exists, `docs/release/evidence/native-performance-benchmark.json` records partial status, `docs/release/native-qa-checklists.md` provides row-by-row execution guidance, and one Android release-emulator line-animation timing/memory sample is captured. `docs/release/evidence/native-performance-matrix.json` has 18 pending platform/scenario rows | Full release-device matrix is still missing |
57-
| Screen-reader QA | Protocol exists, `docs/release/native-qa-checklists.md` provides row-by-row execution guidance, and `docs/release/evidence/native-accessibility-qa.json` records missing status. `docs/release/evidence/native-accessibility-matrix.json` has 16 pending assistive-tech/page rows | Accessibility helpers exist, but native assistive-tech behavior is not verified |
58-
| Developer Preview publish | `react-native-chart-kit@7.0.0-next.0` is published under `next`, but `@chart-kit/core`, `@chart-kit/svg-renderer`, and `@chart-kit/react-native` are not published because npm scope/token access for `@chart-kit` is blocked. `@chart-kit/pro` and `@chart-kit/skia-renderer` remain unpublished as intended. | The modern package path for new adopters is not installable until npm scope access is fixed |
59-
| H6 approval | No RC approval, final semver, final release notes, final visual freeze | Release candidate cannot be claimed |
49+
| Requirement | Missing evidence | Why it matters |
50+
| ---------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------- |
51+
| Native release workflow | Local iOS and Android release builds passed, and the recorder/checker require a run URL, commit, and iOS/Android artifact links before marking workflow evidence complete; no green native workflow artifact is recorded yet. A May 6, 2026 manual dispatch attempt failed because `native-release.yml` is only on `next`, while GitHub Actions exposes workflow dispatches from the default `master` branch. | Required before production beta/RC confidence |
52+
| Native e2e/runtime QA | iOS simulator and Android emulator launch smoke exist, and `docs/release/evidence/native-runtime-qa.json` records partial status. `docs/release/native-qa-checklists.md` provides row-by-row execution guidance. `docs/release/evidence/native-runtime-matrix.json` has 16 pending page/platform rows | Gestures, scroll conflicts, text rendering, and release behavior can differ on device |
53+
| RN CLI native projects | No generated RN CLI `ios/` or `android/` projects are checked in | Source-level RN CLI example exists, but release-build evidence still requires native projects |
54+
| Pro paid implementation | H4 approved the Pro/free boundary, but `@chart-kit/pro` remains a preview registry/composition package without license enforcement, paid implementations, or runtime activation | Required before selling or labeling Pro as a finished paid package |
55+
| Skia renderer | `packages/skia-renderer` exposes an injected primitive adapter. Baseline free chart surfaces can accept it: LineChart, BarChart, PieChart/DonutChart, ProgressChart/ProgressRing, and ContributionGraph/CalendarHeatmap. CombinedChart and CandlestickChart also accept it for Pro-preview workflows. `npm run skia:parity` covers local primitive and renderer-contract parity; `docs/release/evidence/skia-renderer-evidence.json` and `docs/release/evidence/skia-renderer-matrix.json` record the remaining gaps | Native install, native renderer parity, and native Skia performance evidence are not complete |
56+
| Native performance benchmark | Protocol exists, `docs/release/evidence/native-performance-benchmark.json` records partial status, `docs/release/native-qa-checklists.md` provides row-by-row execution guidance, and one Android release-emulator line-animation timing/memory sample is captured. `docs/release/evidence/native-performance-matrix.json` has 18 pending platform/scenario rows | Full release-device matrix is still missing |
57+
| Screen-reader QA | Protocol exists, `docs/release/native-qa-checklists.md` provides row-by-row execution guidance, and `docs/release/evidence/native-accessibility-qa.json` records missing status. `docs/release/evidence/native-accessibility-matrix.json` has 16 pending assistive-tech/page rows | Accessibility helpers exist, but native assistive-tech behavior is not verified |
58+
| Developer Preview publish | `react-native-chart-kit@7.0.0-next.0` is published under `next`, but `@chart-kit/core`, `@chart-kit/svg-renderer`, and `@chart-kit/react-native` are not published because the publish workflow currently receives no usable npm auth token. `@chart-kit/pro` and `@chart-kit/skia-renderer` remain unpublished as intended. | The modern package path for new adopters is not installable until GitHub Actions npm auth and `@chart-kit` scope access are fixed |
59+
| H6 approval | No RC approval, final semver, final release notes, final visual freeze | Release candidate cannot be claimed |
6060

6161
## Current Gate Position
6262

6363
- H0/H1: covered by prior decisions.
6464
- H2/H3: preview implementation and API are reviewable; final design/API freeze still belongs to beta/RC review.
6565
- H4: approved on May 6, 2026. Pro stays a separate future package, Skia stays a separate optional preview renderer until native evidence is complete, advanced interactions/financial/Skia/large-data workflows stay preview or Pro-candidate for beta, and no license enforcement is added before beta.
66-
- H5: approved on May 6, 2026 for Developer Preview only. Publishable packages are `react-native-chart-kit`, `@chart-kit/core`, `@chart-kit/svg-renderer`, and `@chart-kit/react-native` under the `next` dist-tag; `@chart-kit/pro` and `@chart-kit/skia-renderer` stay unpublished. Current npm publish evidence is partial: only the root package is published until `@chart-kit` scope/token access is fixed.
66+
- H5: approved on May 6, 2026 for Developer Preview only. Publishable packages are `react-native-chart-kit`, `@chart-kit/core`, `@chart-kit/svg-renderer`, and `@chart-kit/react-native` under the `next` dist-tag; `@chart-kit/pro` and `@chart-kit/skia-renderer` stay unpublished. Current npm publish evidence is partial: only the root package is published until GitHub Actions npm auth and `@chart-kit` scope access are fixed.
6767
- H6: not started. RC requires green native workflow evidence, final semver, final changelog, docs freeze, visual baseline freeze, deprecation policy approval, and owner approval. An owner decision memo is available.
6868

6969
## Recommended Next Decisions
7070

7171
1. Merge or otherwise expose `.github/workflows/native-release.yml` on the default `master` branch, then run the native release workflow on the release candidate commit to turn local iOS/Android build evidence into CI artifact evidence.
72-
2. Fix npm `@chart-kit` scope/token access, then rerun the publish workflow with npm dist-tag `next`; it should skip the already-published root package and publish the remaining scoped free packages.
72+
2. Add or repair the GitHub Actions `NPM_TOKEN` secret, make sure it can create and publish public packages under the npm `@chart-kit` scope, then rerun the publish workflow with npm dist-tag `next`; it should skip the already-published root package and publish the remaining scoped free packages.
7373
3. Run the native runtime, accessibility, performance, and Skia evidence matrices.
7474
4. If production beta is the target, prioritize native device QA and accessibility QA before adding more chart features.

‎docs/release/evidence/npm-publish-evidence.json‎

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"schemaVersion": 1,
33
"lastUpdated": "2026-05-06",
44
"status": "partial",
5-
"summary": "Developer Preview npm publish is partial: the root compatibility package is published, but scoped @chart-kit packages are blocked by npm scope/token access.",
5+
"summary": "Developer Preview npm publish is partial: the root compatibility package is published, but scoped @chart-kit packages are blocked because the publish workflow currently receives no npm auth token.",
66
"requiredFor": ["H5", "H6"],
77
"completedEntries": [
88
{
@@ -19,13 +19,19 @@
1919
"https://github.com/indiespirit/react-native-chart-kit/commit/47487b5",
2020
"https://github.com/indiespirit/react-native-chart-kit/commit/8bf2dad"
2121
]
22+
},
23+
{
24+
"result": "A May 6, 2026 publish rerun on next failed before package installation because GitHub Actions received an empty NODE_AUTH_TOKEN for npm.",
25+
"artifacts": [
26+
"https://github.com/indiespirit/react-native-chart-kit/actions/runs/25415346511"
27+
]
2228
}
2329
],
2430
"missingEvidence": [
25-
"@chart-kit/core@7.0.0-next.0 is not published because npm returned 404 for the @chart-kit scope publish.",
31+
"@chart-kit/core@7.0.0-next.0 is not published because the publish workflow is missing a usable NPM_TOKEN.",
2632
"@chart-kit/svg-renderer@7.0.0-next.0 is not published.",
2733
"@chart-kit/react-native@7.0.0-next.0 is not published.",
2834
"GitHub prerelease v7.0.0-next.0 was not created because the publish workflow stops before release creation while scoped package publishing is blocked.",
29-
"GitHub Actions NPM_TOKEN must be able to create and publish public packages under the @chart-kit scope."
35+
"GitHub Actions must have an NPM_TOKEN secret, and that token must be able to create and publish public packages under the @chart-kit scope."
3036
]
3137
}

0 commit comments

Comments
 (0)