Skip to content

feat(instantsearch.js): introduce chat widget#6705

Merged
dhayab merged 115 commits intomasterfrom
feat/js-chat
Sep 24, 2025
Merged

feat(instantsearch.js): introduce chat widget#6705
dhayab merged 115 commits intomasterfrom
feat/js-chat

Conversation

@shaejaz
Copy link
Copy Markdown
Contributor

@shaejaz shaejaz commented Sep 15, 2025

This PR adds the chat widget for JS and scaffolds CTS.

FX-3528

Comment thread packages/instantsearch.js/src/widgets/chat/chat.tsx Outdated
Base automatically changed from feat/chat-design to master September 22, 2025 20:41
Comment thread examples/react/e-commerce-with-chat/App.css Outdated
@dhayab dhayab requested a review from Haroenv September 23, 2025 09:43
Comment thread bundlesize.config.json Outdated
Copy link
Copy Markdown
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

Comment thread packages/algolia-experiences/rollup.config.js Outdated
Comment thread packages/instantsearch.js/scripts/rollup/rollup.config.js Outdated
Comment thread packages/instantsearch.js/src/connectors/index.umd.ts
Comment thread packages/instantsearch.js/src/widgets/index.umd.ts Outdated
Comment thread packages/vue-instantsearch/rollup.config.js Outdated
@Haroenv
Copy link
Copy Markdown
Contributor

Haroenv commented Sep 23, 2025

netlify seems to be failing because of storybook. Could you maybe use the umd entry point for that?

@dhayab dhayab requested a review from Haroenv September 24, 2025 08:57
@dhayab dhayab enabled auto-merge (squash) September 24, 2025 08:58
Copy link
Copy Markdown
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yippee!


<script>
import { connectAutocomplete } from 'instantsearch.js/es/connectors';
import { connectAutocomplete } from 'instantsearch.js/es/connectors/index.umd';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, pretty annoying this is needed, but no way around for now as we still want regular esm users to have access to chat

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, UMD has no relation with this but it technically is helpful. Wondering if we should have a barrel file "index.noAi.ts" or something, don't know if it's better 🤷. In any case we have tests to ensure both barrel files have the same connectors / widgets, so I'm not afraid of drift.

@dhayab dhayab merged commit d7b360b into master Sep 24, 2025
14 checks passed
@dhayab dhayab deleted the feat/js-chat branch September 24, 2025 09:23
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.

4 participants