chore(bundle-size): measure JavaScript and React bundle sizes with and without Chat - #7146
Draft
anjola-adeuyi wants to merge 11 commits into
Draft
chore(bundle-size): measure JavaScript and React bundle sizes with and without Chat#7146anjola-adeuyi wants to merge 11 commits into
anjola-adeuyi wants to merge 11 commits into
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 402 |
TIP This summary will be updated as you push new changes.
More templates
algoliasearch-helper
instantsearch-ui-components
instantsearch.css
instantsearch.js
react-instantsearch
react-instantsearch-core
react-instantsearch-nextjs
react-instantsearch-router-nextjs
vue-instantsearch
commit: |
|
Size Change: 0 B Total Size: 1.22 MB ℹ️ View Unchanged
|
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
This adds a reproducible way to compare four consumer bundle cases:
Each measurement records minified and gzip sizes, output hashes, resolved input graphs and retained module attribution.
Result
Three clean runs produced identical measurements and hashes across all four consumers.
The experiment successfully removed the intended generic scrolling modules from both Chat bundles. However, the gzip savings were far below the required 1,000 bytes per Chat consumer, so the candidate was rejected.
Compatibility checks were not run because the size gate failed. The candidate implementation was removed afterward, so no production-source change remains in this PR.
Markdown feasibility
A separate local dependency spike confirmed that JavaScript Chat retains React through the default
markdown-to-jsxentry, even though InstantSearch uses Preact.A renderer-neutral Markdown entry removed React from the JavaScript Chat graph and produced these provisional results:
The build, relevant tests and Chromium, Firefox and WebKit checks passed.
This is a one-run local feasibility result, not a production change or verified three-run measurement. InstantSearch uses
markdown-to-jsx7.7.15, while upstream is now on 9.10.1, so the local dependency change cannot be shipped or contributed unchanged.View the complete feasibility report.
What this PR keeps
View the complete verified report.
FX-3801