Skip to content

HF-122: Framework integration guides for React, Angular, Vue, Svelte#1653

Open
marcin-kordas-hoc wants to merge 16 commits intodevelopfrom
feature/hf-122-framework-integration-guides
Open

HF-122: Framework integration guides for React, Angular, Vue, Svelte#1653
marcin-kordas-hoc wants to merge 16 commits intodevelopfrom
feature/hf-122-framework-integration-guides

Conversation

@marcin-kordas-hoc
Copy link
Copy Markdown
Collaborator

@marcin-kordas-hoc marcin-kordas-hoc commented Apr 13, 2026

Summary

Expand the four framework integration pages (React, Angular, Vue, Svelte) from one-line redirects into self-contained guides with code snippets extracted from the respective Stackblitz demos.

Each guide's primary snippet is a simplified version of the demo's framework pattern — same lifecycle hooks, same service architecture, same reactivity approach — with simplified data (buildFromArray instead of Employee Table).

Design rationale

Snippets from demos, not invented patterns

Per review feedback: every code snippet must match what's in the corresponding Stackblitz demo. This ensures the snippets are tested, idiomatic, and consistent with what users see when they click the demo link. Patterns not present in demos (e.g., Angular Signals, Svelte 5 runes) are deliberately excluded until validated by a framework expert.

Framework Demo file Primary pattern in guide
React react-demo/src/lib/employee/employee.provider.tsx useRef + useEffect init/cleanup + useState
Angular angular-demo/src/app/employees/employees.service.ts @Injectable + BehaviorSubject + async pipe
Vue vue-3-demo/src/lib/employees-data-provider.ts Class wrapper with private HF field + ref
Svelte svelte-demo/src/routes/Hyperformula.svelte buildFromArray + getCellValue + on:click + onDestroy

Other decisions

  • TypeScript in all snippets (HF ships .d.ts typings)
  • licenseKey: 'gpl-v3' in every snippet (without it, engine throws license warning)
  • SSR notes for Next.js, Nuxt, SvelteKit (HF depends on browser-only APIs)
  • VuePress template fix — Stackblitz links use <a :href> Vue binding instead of {{ }} interpolation in markdown

Test plan

  • Render docs locally (npm run docs:dev) and verify all four integration pages
  • Click each Stackblitz demo link — confirm it opens the demo
  • Verify snippets match demo patterns (React=useRef, Angular=BehaviorSubject, Vue=class wrapper, Svelte=on:click)
  • Verify no untested patterns remain (no Signals, no $state runes, no Pinia, no NgZone)
  • Confirm licenseKey: 'gpl-v3' present in every snippet
  • Confirm destroy() cleanup present in every applicable snippet

Note

Low Risk
Docs-only changes; primary risk is broken documentation rendering/links or misleading integration guidance if snippets drift from supported patterns.

Overview
Adds AGENTS.md with repo-wide guidance for AI assistants, including HyperFormula usage patterns and framework integration do/don’ts.

Replaces the previously minimal React/Angular/Vue/Svelte integration pages with full, self-contained guides and code snippets (lifecycle/cleanup patterns, SSR notes, and next-steps links), and updates Vue troubleshooting guidance around reactivity (markRaw) and licensing.

Fixes VuePress rendering of Stackblitz demo URLs by switching markdown interpolation to bound <a :href> links across the affected guides (including custom-functions.md).

Reviewed by Cursor Bugbot for commit d6a316c. Bugbot is set up for automated code reviews on this repo. Configure here.

Add idiomatic code examples to React, Angular, Vue, and Svelte
integration pages showing HyperFormula initialization and reading
calculated values. Each guide uses framework-specific patterns
(React hooks, Angular service, Vue ref/markRaw, Svelte reactivity).

Closes HF-122.
Per feedback, snippets now focus on framework-specific integration
patterns (useRef/useEffect, @Injectable, markRaw/ref, Svelte reactivity)
with placeholder comments for data and configuration rather than
concrete values. The Demo section link is reworded from "Explore the
full working example" to "For a more advanced example" to signal that
the Stackblitz demo is a richer, separate project.

Part of HF-122.
Expand React, Angular, Vue and Svelte integration guides to cover
framework-specific concerns that the previous skeleton snippets omitted:

- Convert all snippets to TypeScript with HyperFormula/CellValue imports
- Add licenseKey: 'gpl-v3' to every buildFromArray call
- Inline `npm install hyperformula` install command
- React: runnable JSX with controlled inputs, try/catch around setCellContents,
  React.StrictMode note, Next.js App Router SSR section with 'use client' and
  dynamic import
- Angular: modern standalone component + inject() + DestroyRef + Signals +
  OnPush, RxJS BehaviorSubject variant with AsyncPipe import note, NgZone
  runOutsideAngular section, provider-scope + DestroyRef rationale
- Vue: <script setup lang="ts">, expanded markRaw Proxy explanation and
  shallowRef trap, Nuxt SSR snippet, Pinia store with updateCell action,
  valuesUpdated event hook, inline v-for template
- Svelte: fixes memory leak by adding onDestroy(() => hf.destroy()), Svelte 5
  runes primary example plus Svelte 4 delta, SvelteKit SSR warning banner and
  dedicated onMount section with dynamic import, wired #each template with
  updateCell handler
Point readers from each integration guide to related topics they are
likely to need next: configuration options, basic operations (CRUD),
advanced usage (multi-sheet, named expressions), and custom functions.

Identical section in all four guides so the navigation experience is
consistent when switching between frameworks.
The Stackblitz demo links used \${...} JavaScript template literal syntax,
which is not interpolated by VuePress. VuePress 1 processes markdown as
Vue templates, so the correct syntax for accessing the injected
\$page.buildDateURIEncoded variable is Vue's {{ ... }} interpolation.

config.js line 79 documents the intended syntax explicitly:
  // inject current HF buildDate URI encoded as {{ \$page.buildDateURIEncoded }} variable

Before this fix, the cache-buster query string rendered as a literal
?v=\${\$page.buildDateURIEncoded} in the URL, breaking the cache-busting
behaviour on every Stackblitz demo link.

Note: the same issue exists in docs/guide/custom-functions.md:361 from
an earlier change; that is left for a separate fix.
@qunabu
Copy link
Copy Markdown

qunabu commented Apr 13, 2026

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 13, 2026

Performance comparison of head (d6a316c) vs base (1852159)

                                     testName |   base |   head | change
------------------------------------------------------------------------
                                      Sheet A | 503.72 | 498.99 | -0.94%
                                      Sheet B | 171.88 | 165.27 | -3.85%
                                      Sheet T | 147.09 | 144.58 | -1.71%
                                Column ranges | 532.88 | 527.26 | -1.05%
Sheet A:  change value, add/remove row/column |  17.28 |  16.97 | -1.79%
 Sheet B: change value, add/remove row/column | 150.17 | 138.65 | -7.67%
                   Column ranges - add column | 162.24 |  156.4 | -3.60%
                Column ranges - without batch | 498.37 | 481.11 | -3.46%
                        Column ranges - batch | 126.33 | 126.55 | +0.17%

…ntent)

tsc smoke check found that updateCell(value: unknown) does not satisfy
setCellContents which expects RawCellContent. Replace unknown with the
correct type and add missing RawCellContent imports in Angular, Vue and
Svelte guides.
Add framework-agnostic instruction file for AI coding assistants
(Claude Code, GitHub Copilot, Cursor, Gemini CLI). Covers the core
3-operation API surface, framework integration patterns with critical
rules (markRaw, onDestroy, StrictMode, DestroyRef, SSR guards),
key types, common mistakes to prevent, and project structure pointers.
@marcin-kordas-hoc marcin-kordas-hoc marked this pull request as ready for review April 14, 2026 07:18
@marcin-kordas-hoc
Copy link
Copy Markdown
Collaborator Author

Question: Automated testing for framework demos?

During this PR I ran a manual tsc smoke check on all 10 code snippets extracted from the guides — found and fixed a real type bug (unknownRawCellContent).

This raises a broader question: should we have automated tests for the framework demos (both the guide snippets and the Stackblitz demos in hyperformula-demos)?

The risk

Framework demos can silently break when:

  • A framework releases a breaking change (React 19, Angular 18, Svelte 6...)
  • HyperFormula API changes (new required params, renamed types)
  • Bundler updates (Vite, Webpack config changes)
  • Dependency drift (stale lockfile, removed transitive dep)

Currently there are no automated checks — breakage is discovered only when a user clicks a broken Stackblitz link or copy-pastes a non-compiling snippet.

Options considered

Option Where What it catches Setup cost
A: Scheduled CI in demos repo hyperformula-demos Framework breaks ~20 min
B: Cross-repo job in publish.yml HF main repo HF API breaks (before deploy) ~1h
C: Nightly scheduled in HF repo HF main repo Both (builds demos against develop) ~2h

Option A is lowest cost and most immediately useful — a weekly npm ci && npm run build per framework in the demos repo. Zero changes to HF main CI.

Option B catches the other direction (HF breaking demos) and could gate docs deployment.

Related: demo visibility for AI tools

Stackblitz links are invisible to AI coding assistants (Copilot, Cursor, Claude). AI relies solely on the inline snippets in docs/guide/integration-with-*.md. I've added AGENTS.md with framework integration patterns to make these discoverable.

A potential follow-up: vendoring a subset of demo code into docs/examples/ (where 49 examples already live) so AI tools see working examples during code search. This is a separate discussion — noting it here for visibility.

@sequba What do you think — is option A worth setting up? Or is manual verification sufficient for now?

@sequba
Copy link
Copy Markdown
Contributor

sequba commented Apr 14, 2026

@marcin-kordas-hoc I need to be sure the snippets work correctly and align with each framework's best practices, so I want them to be exact code fragments from the linked Stackblitz demos.

You can achieve this by either:

  • updating the code snippets to match the demo code
  • modifying the demos to match the code snippets (but in this case, please make sure that all demos are really well written - this is the code that other devs copy and use in their products, it must be perfect; also we will ask framework experts to review the demos before publication)

@marcin-kordas-hoc
Copy link
Copy Markdown
Collaborator Author

Following our Slack discussion where you approved keeping snippets focused on framework-specific patterns rather than extracting from the demos directly — that's the approach taken here (useEffect, BehaviorSubject, markRaw, on:click as the core of each guide, data as // your data goes here).

The reason for not extracting from demos directly: they use multi-file setup (buildEmptyaddSheet → named expressions, EmployeeRow types, helper files) — copying them produces either 60+ unreadable lines or decontextualised fragments. The Demo section caption already sets the expectation: "For a more advanced example, check out the demo on Stackblitz".

The demos themselves are unchanged.

@marcin-kordas-hoc marcin-kordas-hoc added the Docs Improvements or additions to documentation label Apr 16, 2026
@marcin-kordas-hoc marcin-kordas-hoc self-assigned this Apr 16, 2026
marcin-kordas-hoc and others added 3 commits April 16, 2026 08:57
Angular: switch primary pattern to BehaviorSubject/Observable (module-based),
matching the angular-demo on Stackblitz. Signal-based variant moved to a
'Signals variant (Angular 17+)' note. Cleanup section expanded with both
ngOnDestroy and DestroyRef options.

Svelte: switch primary pattern to Svelte 4 style (on:click, plain let),
matching the svelte-demo on Stackblitz. Svelte 5 runes variant moved to
a '### Svelte 5 (runes)' subsection. SSR section updated to use Svelte 4
as primary.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
React: fix Next.js SSR pattern — 'use client' belongs in the component
file, not on page.tsx; server page uses dynamic(..., { ssr: false })

Angular: add CommonModule/standalone imports for *ngFor + async pipe;
add @for template for signals variant; add inject() context note

Vue: replace shallowRef with ref in Nuxt SSR snippet (shallowRef vs
markRaw contradiction); add hf.off() teardown to valuesUpdated example;
remove hf from Pinia store return value (devtools serialisation risk)

Svelte: fix CellValue type for result variable; fix SSR snippet to use
onMount return for cleanup instead of separate onDestroy

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Vue: rewrite primary snippet to use class wrapper pattern matching
the vue-3-demo on Stackblitz. markRaw moved to Notes as alternative.

Angular: remove unnecessary `as CellValue[][]` casts (getSheetValues
already returns this type).

React: remove try/catch from handleCellEdit for consistency — no demo
uses error handling around setCellContents.

custom-functions.md: fix VuePress template syntax (${}→{{}}) so the
Stackblitz link renders the build date correctly.

AGENTS.md: update framework table to match actual guide primary
patterns (Angular=BehaviorSubject, Vue=class wrapper, Svelte=Svelte 4).
marcin-kordas-hoc and others added 5 commits April 16, 2026 09:57
…polation

{{ $page.buildDateURIEncoded }} inside markdown link syntax [text](url{{ }})
breaks markdown-it parsing — links render as plain text instead of clickable
links. Switch to <a :href="..."> with Vue v-bind which VuePress handles
correctly since page data is available in template scope.
Per Kuba's review feedback: snippets must match what's in the Stackblitz
demos so we can guarantee they work correctly. Removed all patterns that
have never been tested with HyperFormula:

Angular: removed Signals variant (signal, inject, DestroyRef, OnPush,
@for), NgZone.runOutsideAngular section.

Svelte: removed Svelte 5 runes ($state) section.

Vue: removed markRaw alternative, Nuxt SSR with dynamic import,
valuesUpdated event listener, Pinia store sections.

These patterns can be re-added once validated by a framework consultant
or included in updated demos.
All four Stackblitz demos use the same interaction model: "Run calculations"
and "Reset" buttons with read-only table output. Aligned guides to match:

React: replaced editable <input onChange> with button-driven calculate/reset.
The onChange-per-keystroke pattern was not from the demo and caused issues
with partial formula input.

Angular: replaced standalone component with NgModule pattern (matching demo).
Replaced updateCell with calculate/reset methods matching EmployeesService.
Added buttons to template.

Vue: replaced dead handleUpdate/updateCell with getCalculatedValues/
getRawFormulas matching EmployeesDataProvider. Added Run/Reset buttons.
AGENTS.md: Vue SSR guard updated from "onMounted + dynamic import"
to "<ClientOnly> (Nuxt)" matching the actual guide content.

Svelte: renamed "Calculate" to "Run calculations" and added "Reset"
button for consistency with React/Angular/Vue guides. Reset is pure
UI (result = null), no HF API involved.
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.18%. Comparing base (1852159) to head (d6a316c).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1653      +/-   ##
===========================================
+ Coverage    96.77%   97.18%   +0.41%     
===========================================
  Files          174      174              
  Lines        15086    15086              
  Branches      3223     3223              
===========================================
+ Hits         14599    14662      +63     
+ Misses         487      424      -63     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants