Skip to content

feat: add option to disable message preprocessing in the internal appconfiguration#21275

Open
MohamadJaara wants to merge 5 commits into
devfrom
mo/feat/add-option-to-disable-message-preprocessing
Open

feat: add option to disable message preprocessing in the internal appconfiguration#21275
MohamadJaara wants to merge 5 commits into
devfrom
mo/feat/add-option-to-disable-message-preprocessing

Conversation

@MohamadJaara
Copy link
Copy Markdown
Member

Adds an internal developer-menu flag to disable message preprocessing and markdown preview behavior in the composer.

The current preprocessing can alter messages in ways that make composing feel unpredictable, especially when typing or pasting exact text/code or URLs. This flag is a temporary bandage so messages can be sent closer to what was typed while we sort out the underlying composer/preprocessing behavior properly.

Notes

  • Keeps mentions and link previews working.
  • Disables markdown/format preprocessing in the input.
  • Sends raw text without automatic trimming when enabled.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

❌ Patch coverage is 32.72727% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.94%. Comparing base (c5dd9d8) to head (7031fbc).
⚠️ Report is 529 commits behind head on dev.

Files with missing lines Patch % Lines
...gins/PlainTextPastePlugin/PlainTextPastePlugin.tsx 6.66% 14 Missing ⚠️
.../script/components/ConfigToolbar/ConfigToolbar.tsx 0.00% 5 Missing ⚠️
...r/InputBarEditor/RichTextEditor/RichTextEditor.tsx 42.85% 1 Missing and 3 partials ⚠️
...webapp/src/script/components/InputBar/InputBar.tsx 70.00% 2 Missing and 1 partial ⚠️
...p/src/script/util/debugMessagePreprocessingUtil.ts 62.50% 3 Missing ⚠️
...BarEditor/RichTextEditor/utils/transformMessage.ts 33.33% 2 Missing ⚠️
...Editor/RichTextEditor/utils/useEditorDraftState.ts 33.33% 0 Missing and 2 partials ⚠️
apps/webapp/src/script/util/debugUtil.ts 0.00% 2 Missing ⚠️
.../components/ConfigToolbar/ConfigToolbar.styles.tsx 0.00% 1 Missing ⚠️
...seMessageHandling/useMessageSend/useMessageSend.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev   #21275      +/-   ##
==========================================
- Coverage   44.95%   44.94%   -0.01%     
==========================================
  Files        1705     1707       +2     
  Lines       44906    44952      +46     
  Branches     9399     9412      +13     
==========================================
+ Hits        20187    20205      +18     
- Misses      22423    22447      +24     
- Partials     2296     2300       +4     
Flag Coverage Δ
app_webapp 43.08% <32.72%> (-0.01%) ⬇️
lib_api_client 51.72% <ø> (ø)
lib_core 58.81% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ponents/InputBar/InputBarEditor/InputBarEditor.tsx 100.00% <ø> (ø)
.../InputBar/useMessageHandling/useMessageHandling.ts 39.65% <ø> (ø)
.../components/ConfigToolbar/ConfigToolbar.styles.tsx 0.00% <0.00%> (ø)
...seMessageHandling/useMessageSend/useMessageSend.ts 23.88% <0.00%> (+0.35%) ⬆️
...BarEditor/RichTextEditor/utils/transformMessage.ts 40.00% <33.33%> (-10.00%) ⬇️
...Editor/RichTextEditor/utils/useEditorDraftState.ts 73.33% <33.33%> (-12.39%) ⬇️
apps/webapp/src/script/util/debugUtil.ts 0.00% <0.00%> (ø)
...webapp/src/script/components/InputBar/InputBar.tsx 60.56% <70.00%> (+2.49%) ⬆️
...p/src/script/util/debugMessagePreprocessingUtil.ts 62.50% <62.50%> (ø)
...r/InputBarEditor/RichTextEditor/RichTextEditor.tsx 62.96% <42.85%> (-7.88%) ⬇️
... and 2 more

... and 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.

@MohamadJaara MohamadJaara force-pushed the mo/feat/add-option-to-disable-message-preprocessing branch from 4cd860a to 5f5ffc7 Compare May 9, 2026 09:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 9, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 15
  • Failed: 0
  • Skipped: 0
  • 🔁 Flaky: 0
  • 📊 Total: 15
  • Total Runtime: 3342.2s (~ 55 min 42 sec)

Comment thread apps/webapp/src/script/components/ConfigToolbar/ConfigToolbar.tsx Outdated
Comment thread apps/webapp/src/script/components/InputBar/InputBar.tsx Outdated
MohamadJaara and others added 3 commits June 2, 2026 00:01
…TextEditor/RichTextEditor.tsx

Co-authored-by: Zafar Saeed Khan <zafarsaeed@outlook.com>
Co-authored-by: Zafar Saeed Khan <zafarsaeed@outlook.com>
@MohamadJaara MohamadJaara requested a review from zskhan June 1, 2026 22:17
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

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.

2 participants