Skip to content

test - #100373

Draft
situchan wants to merge 6 commits into
Expensify:mainfrom
mukhrr:fix/97049
Draft

test#100373
situchan wants to merge 6 commits into
Expensify:mainfrom
mukhrr:fix/97049

Conversation

@situchan

@situchan situchan commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Anchor each pregenerated follow-up turn to its question reportActionID and
use it to detect a newer user message. The follow-up reconciliation no
longer wipes the server indicator for that message, the composer skips its
optimistic clear while a turn is still reconciling, and the pending
follow-up list stops suppressing indicators once the user has sent again.
@codecov

codecov Bot commented Sep 4, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Looks like you've decreased code coverage for some files. Please write tests to increase, or at least maintain, the existing level of code coverage. See our documentation here for how to interpret this table.

Files with missing lines Coverage Δ
src/hooks/usePendingConciergeResponse.ts 94.20% <100.00%> (-0.05%) ⬇️
src/hooks/useShouldSuppressConciergeIndicators.tsx 100.00% <100.00%> (ø)
src/libs/actions/Report/SuggestedFollowup.ts 86.11% <100.00%> (+0.39%) ⬆️
src/pages/inbox/AgentZeroStatusContext.tsx 94.25% <100.00%> (+0.83%) ⬆️
...ox/report/ReportActionCompose/useComposerSubmit.ts 90.36% <100.00%> (+0.36%) ⬆️
src/libs/ReportActionFollowupUtils/index.ts 93.33% <91.66%> (-1.12%) ⬇️
... and 7 files with indirect coverage changes

The code standards check asks for plain comment sentences, so the dashes and the semicolon-joined clause become separate sentences.
A label that is already on when the gate mounts can belong to a reply that has landed and is waiting for the server clear, so recording it as the baseline kept the bubble on top of that reply. Also add the missing Given/When/Then comments to the new suppression tests.

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.

2 participants