Skip to content

Comments

chore: remove with*Context HOC wrappers#2962

Open
oliverlaz wants to merge 1 commit intomasterfrom
chore/remove-context-hocs
Open

chore: remove with*Context HOC wrappers#2962
oliverlaz wants to merge 1 commit intomasterfrom
chore/remove-context-hocs

Conversation

@oliverlaz
Copy link
Member

@oliverlaz oliverlaz commented Feb 23, 2026

🎯 Goal

BREAKING CHANGE: Remove legacy high-order context wrappers (with*Context) from stream-chat-react since the docs and internal usage are hook-based (use*Context).

πŸ›  Implementation details

  • Removed the following exported HOCs from context modules:
    • withChannelActionContext
    • withChannelStateContext
    • withChatContext
    • withComponentContext
    • withMessageContext
    • withTranslationContext
    • withTypingContext
  • Kept provider/hook APIs intact (*Provider and use*Context).
  • Removed now-unused imports/types tied to HOC implementations (including UnknownType usages in these files).
  • Deleted now-unused helper src/context/utils/getDisplayName.ts.

🎨 UI Changes

N/A (no UI changes).

@github-actions
Copy link

Size Change: -516 B (-0.13%)

Total Size: 405 kB

Filename Size Change
./dist/Channel-C1ckso1b.js 0 B -20.8 kB (removed) πŸ†
./dist/Channel-DGZtjDrh.js 20.7 kB +20.7 kB (new file) πŸ†•
./dist/ChannelPreview-CP_K__VU.js 0 B -7.15 kB (removed) πŸ†
./dist/ChannelPreview-CWSn1O4L.js 7.15 kB +7.15 kB (new file) πŸ†•
./dist/cjs/emojis.js 2.98 kB +3 B (+0.1%)
./dist/cjs/experimental.js 3.62 kB +2 B (+0.06%)
./dist/cjs/index.js 206 kB -139 B (-0.07%)
./dist/limits-CCF1580z.js 62.2 kB +62.2 kB (new file) πŸ†•
./dist/limits-DpR0J_m4.js 0 B -62.4 kB (removed) πŸ†
ℹ️ View Unchanged
Filename Size
./dist/audioProcessing-BbOs2wMd.js 1.32 kB
./dist/cjs/mp3-encoder.js 1.27 kB
./dist/css/index.css 35.2 kB
./dist/css/v2/emoji-mart.css 1.84 kB
./dist/css/v2/emoji-replacement.css 300 B
./dist/css/v2/index.css 39.4 kB
./dist/css/v2/index.layout.css 22.8 kB

compressed-size-action

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