Skip to content

fix(compareBar): require configurationId when the widget is created - #7306

Draft
Haroenv wants to merge 1 commit into
masterfrom
fix/compare-bar-configuration-id
Draft

Haroenv wants to merge 1 commit into
masterfrom
fix/compare-bar-configuration-id

Conversation

@Haroenv

@Haroenv Haroenv commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • compareBar (JS) throws The configurationId option is required. when the widget is created without a configurationId, like it already does for container.
  • <CompareBar> (React) throws The configurationId prop is required. on render, before the hook is used.
  • The JS widget passes configurationId to the connector as a required string. The compare() guard in connectCompare is unchanged, since toggles-only setups still rely on it.

Result

A missing configurationId fails when the widget is created, instead of inside the Compare button's click handler. Tests cover the JS widget creation error and the React render error; the all-widgets test util now passes a configurationId to CompareBar.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 9 complexity

Metric Results
Complexity 9

View in Codacy

TIP This summary will be updated as you push new changes.

@pkg-pr-new

pkg-pr-new Bot commented Oct 16, 2026

Copy link
Copy Markdown
More templates

algoliasearch-helper

npm i https://pkg.pr.new/algolia/instantsearch/algoliasearch-helper@7306

instantsearch-ui-components

npm i https://pkg.pr.new/algolia/instantsearch/instantsearch-ui-components@7306

instantsearch.css

npm i https://pkg.pr.new/algolia/instantsearch/instantsearch.css@7306

instantsearch.js

npm i https://pkg.pr.new/algolia/instantsearch/instantsearch.js@7306

react-instantsearch

npm i https://pkg.pr.new/algolia/instantsearch/react-instantsearch@7306

react-instantsearch-core

npm i https://pkg.pr.new/algolia/instantsearch/react-instantsearch-core@7306

react-instantsearch-nextjs

npm i https://pkg.pr.new/algolia/instantsearch/react-instantsearch-nextjs@7306

react-instantsearch-router-nextjs

npm i https://pkg.pr.new/algolia/instantsearch/react-instantsearch-router-nextjs@7306

vue-instantsearch

npm i https://pkg.pr.new/algolia/instantsearch/vue-instantsearch@7306

commit: b062c46

@github-actions

Copy link
Copy Markdown

Size Change: +90 B (+0.01%)

Total Size: 1.04 MB

📦 View Changed
Filename Size Change
packages/instantsearch.js/dist/instantsearch.development.js 334 kB +45 B (+0.01%)
packages/instantsearch.js/dist/instantsearch.production.min.js 152 kB +18 B (+0.01%)
packages/react-instantsearch/dist/umd/ReactInstantSearch.min.js 125 kB +27 B (+0.02%)
ℹ️ View Unchanged
Filename Size Change
packages/algoliasearch-helper/dist/algoliasearch.helper.js 44.5 kB 0 B
packages/algoliasearch-helper/dist/algoliasearch.helper.min.js 14.1 kB +1 B (+0.01%)
packages/instantsearch.css/components/ai-mode-button.css 1.76 kB 0 B
packages/instantsearch.css/components/autocomplete-min.css 4.33 kB 0 B
packages/instantsearch.css/components/autocomplete.css 4.68 kB 0 B
packages/instantsearch.css/components/button.css 1.98 kB 0 B
packages/instantsearch.css/components/chat-min.css 7.4 kB 0 B
packages/instantsearch.css/components/chat.css 7.91 kB 0 B
packages/instantsearch.css/components/filter-suggestions.css 1.89 kB 0 B
packages/instantsearch.css/themes/algolia-min.css 11.9 kB 0 B
packages/instantsearch.css/themes/algolia.css 12.6 kB 0 B
packages/instantsearch.css/themes/nova-min.css 12 kB 0 B
packages/instantsearch.css/themes/nova.css 12.8 kB 0 B
packages/instantsearch.css/themes/reset-min.css 1.3 kB 0 B
packages/instantsearch.css/themes/reset.css 1.38 kB 0 B
packages/instantsearch.css/themes/satellite-min.css 12.8 kB 0 B
packages/instantsearch.css/themes/satellite.css 13.7 kB 0 B
packages/react-instantsearch-core/dist/umd/ReactInstantSearchCore.min.js 73.8 kB 0 B
packages/vue-instantsearch/vue2/cjs/index.js 19.9 kB 0 B
packages/vue-instantsearch/vue2/umd/index.js 74.3 kB -4 B (-0.01%)
packages/vue-instantsearch/vue3/cjs/index.js 20.6 kB 0 B
packages/vue-instantsearch/vue3/umd/index.js 74.7 kB -6 B (-0.01%)

compressed-size-action

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant