Add starter compatibility matrix e2e + version.ts floor guard#67
Draft
demtario wants to merge 9 commits into
Draft
Add starter compatibility matrix e2e + version.ts floor guard#67demtario wants to merge 9 commits into
demtario wants to merge 9 commits into
Conversation
…dation DEV-2100: namespace docs-example buckets
Generate release and next buckets in CI while pinning imported dependencies to concrete npm versions for reproducible snapshots.
…e-ci feat(docs): automate versioned example imports
…-css-link fix(runtime): sync CDN CSS version
chore(docs): update versioned docs examples
… guard Empirically verifies every starter against Handsontable majors 15-19 against a live instance (DEV-2102 / ADR-0021 decision 10), opt-in via `pnpm e2e:matrix`. Findings (docs/reports/starter-matrix-2026-07-17.md): 5 React-UI-lib container starters and Angular break at older majors, Astro fails version-independently (infra issue, not compatibility). Adds a DEFAULT_MIN_MAJOR=15 floor to validateHandsontableVersion, mirroring the existing ceiling — previously only GET /api/versions enforced the 15 floor, so a direct session/API call bypassing the version dropdown could still request an untested major. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
1 participant
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.
Summary
runner/e2e/starter-matrix.spec.ts— opt-in e2e (E2E_STARTER_MATRIX=1) that empirically boots every starter inexamples/at every supported Handsontable major (15-19) against a live instance, per DEV-2102 / ADR-0021 decision 10.runner/scripts/starter-matrix-report.mjsto turn the run's JSON into a markdown compatibility table (pnpm e2e:matrix:report, supports merging chunked runs).DEFAULT_MIN_MAJOR = 15floor tovalidateHandsontableVersioninpackages/runtime/src/version.ts, mirroring the existing ceiling — previously onlyGET /api/versionsenforced the 15 floor, so a direct session/API call bypassing the version dropdown could still request an untested major.runner/docs/starter-compat-matrix.md(how to run it, prod concurrency cap, limitations) + pointer inrun-and-deploy.md.Findings (docs/reports/starter-matrix-2026-07-17.md)
mui's console log points at an npm package-exports resolution error.angularbreaks at majors 15-17, passes only at 18 (never reaches "Live" — container never finishes booting).astrofails at every tested major with an identical, version-independent 504 error — reads as an Astro-container infra issue, not a Handsontable compatibility problem.Test plan
pnpm test(unit, includes new floor-guard cases) — 34/34 passpnpm typecheck— cleanpnpm e2e— deterministic suite unaffected (matrix all-skipped without the flag; pre-existing docs-examples failures confirmed unrelated via baseline comparison)pnpm e2e:matrixrun againsthttps://demos.handsontable.com(chunked), report generated and reviewed🤖 Generated with Claude Code