Repository navigation
Migrate Studio snackbar to KDS - #6144
Monier-Ayman wants to merge 10 commits into
Conversation
|
👋 Hi @Monier-Ayman, thanks for contributing! For the review process to begin, please verify that the following is satisfied:
Also check that issue requirements are satisfied & you ran Pull requests that don't follow the guidelines will be closed. Reviewer assignment can take up to 2 weeks. |
|
Hi @akolson! The PR is ready for review. I’ve completed the migration and verified the affected tests pass. 🙌 |
6194683 to
892fda1
Compare
|
Hi @Monier-Ayman! Sure, I’ll take a look. Also, please be sure to always include the References section in the PR description so the assigned issue and PR can be auto-linked. Thank you! |
|
📢✨ Before we assign a reviewer, we'll turn on |
🟡 Waiting for changesOutcome: reviewed Last updated: 2026-10-28 12:52 UTC |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: removing the Vuex snackbar module left consumers behind, and several specs got weaker. Call-site migrations match KDS 5.9.0 useKSnackbar and keep the old 6000 ms timing.
CI failing: Frontend tests, linting and build. Locally, ReviewSubmissionSidePanel.spec.js fails 3 tests. SubmitToCommunityLibrarySidePanel.spec.js fails 1.
Manual QA in the browser passed for trash undo, import/search, sharing, account settings, admin actions and the notifications error, including RTL. Redis MISCONF on the QA stack blocked duplicate, clipboard, quick edit, move and channel list. Those passed in scratch Jest against the real GlobalSnackbar. They still need a browser pass.
- blocking: unmigrated consumers of the removed store module (
baseStore.js) - blocking: unused test bindings plus vacuous assertions (
studioCopyToken.spec.js,trashModal.spec.js) - suggestion: other specs dropped their text assertions
- suggestion: snackbar options repeated at every call site
- suggestion: time-limited Cancel/Undo not reachable by keyboard
- suggestion: no test for
GlobalSnackbarwiring
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
Ran a phased review pipeline over the pull request diff:
- Classified the diff to select review passes (core, frontend, backend) and whether manual QA was required
- Core review pass checked correctness, design, architecture, testing, completeness, and DRY/SRP/Rule-of-Three principles
- Specialized frontend/backend review passes applied framework-specific lenses where those files changed
- For UI changes: manual QA and an accessibility audit against a live dev server, when available
- Checked CI status and linked issue acceptance criteria
- Synthesized one review from those passes and chose the verdict from the findings, CI status, and QA evidence
|
Hi @rtibblesbot, thanks for the detailed review. I went through the feedback and pushed the fixes. I fixed the remaining Vuex snackbar usage, restored the snackbar text assertions, added autofocus for the temporary Cancel/Undo actions, kept the 6000ms duration where needed, and fixed the I also cleaned up the unused test variables. The affected tests are now 123/123 passing, ESLint is clean, and Thanks again for pointing these out! |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: 2 of 9 prior findings are still open; 7 are resolved or acknowledged.
CI passing. Manual QA did not run.
Still open:
studioCopyToken.spec.js:79,90—mockStoreis still destructured and unused. Drop it; it no longer blocks.useCommunityLibraryUpdates.js:178— the error snackbar still dismisses at the 5000 ms default. Addduration: 6000, asNotificationsModal/index.vuedoes.
New suggestions are inline:
GlobalSnackbarcallshideCallbacktwice.- Its spec mocks the composable.
- The
EditSourceModaltext assertions never run.
Prior-finding status
RESOLVED — contentcuration/contentcuration/frontend/shared/vuex/baseStore.js:52 — removing module left unmigrated consumers
UNADDRESSED — contentcuration/contentcuration/frontend/shared/views/StudioCopyToken/tests/studioCopyToken.spec.js:90 — mockStore unused
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/views/trash/tests/trashModal.spec.js — dispatchSpy unused
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/tests/EditTitleDescriptionModal.spec.js — snackbar text not checked
UNADDRESSED — contentcuration/contentcuration/frontend/shared/views/NotificationsModal/composables/useCommunityLibraryUpdates.js:178 — 5000 ms duration
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/components/sidePanels/SubmitToCommunityLibrarySidePanel/index.vue:604 — Cancel unreachable by keyboard
RESOLVED — contentcuration/contentcuration/frontend/shared/views/GlobalSnackbar.vue — callback wiring untested
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/views/CurrentTopicView.vue — deferral unneeded
ACKNOWLEDGED — contentcuration/contentcuration/frontend/shared/composables/useToken.js:42 — setup-time useKSnackbar() fixes TDZ (praise)
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
Ran an automatic code-only delta review triggered by new commits on a previously reviewed PR:
- Retrieved prior bot reviews via the GitHub API
- Classified each prior finding as RESOLVED, UNADDRESSED, ACKNOWLEDGED, or CONTESTED
- Only raised NEW findings for newly introduced code
- Core review pass only — specialized frontend/backend lenses and manual QA run when a review is explicitly requested
- Synthesized one review from the passes and chose the verdict from the findings, CI status, and QA evidence
Superseded by my review of 7f8524b: no blocking findings remain.
|
@rtibblesbot Thanks for the review! I addressed all the points you mentioned and pushed the fixes in I removed the unused I also ran the affected tests and ESLint, and everything passed. |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: all 13 prior findings resolved or acknowledged. Two new test-value suggestions inline.
CI passing. Manual QA did not run. The UI is untested.
Prior-finding status
RESOLVED — contentcuration/contentcuration/frontend/shared/vuex/baseStore.js:52 — Removing module leaves unmigrated consumers
RESOLVED — contentcuration/contentcuration/frontend/shared/views/StudioCopyToken/tests/studioCopyToken.spec.js — mockStore now unused
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/views/trash/tests/trashModal.spec.js — dispatchSpy now unused
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/tests/EditTitleDescriptionModal.spec.js — Test no longer checks snackbar text
RESOLVED — contentcuration/contentcuration/frontend/shared/views/NotificationsModal/composables/useCommunityLibraryUpdates.js:178 — Snackbar dismisses after 5000 ms
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/components/sidePanels/SubmitToCommunityLibrarySidePanel/index.vue:604 — Keyboard users can't reach Cancel before it expires
RESOLVED — contentcuration/contentcuration/frontend/shared/views/GlobalSnackbar.vue — Nothing tests callback wiring
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/views/CurrentTopicView.vue — Deferral isn't needed
ACKNOWLEDGED — contentcuration/contentcuration/frontend/shared/composables/useToken.js:42 — useKSnackbar() at setup fixes latent TDZ bug
RESOLVED — contentcuration/contentcuration/frontend/shared/views/GlobalSnackbar.vue — hideCallback runs twice on close
RESOLVED — contentcuration/contentcuration/frontend/shared/views/tests/GlobalSnackbar.spec.js — Spec can't catch double hideCallback
RESOLVED — contentcuration/contentcuration/frontend/channelEdit/components/QuickEditModal/tests/EditSourceModal.spec.js — Assertion runs after the test has passed
ACKNOWLEDGED — contentcuration/contentcuration/frontend/channelEdit/views/CurrentTopicView.vue:917 — autofocus: true on Undo/Cancel snackbars
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
Ran an automatic code-only delta review triggered by new commits on a previously reviewed PR:
- Retrieved prior bot reviews via the GitHub API
- Classified each prior finding as RESOLVED, UNADDRESSED, ACKNOWLEDGED, or CONTESTED
- Only raised NEW findings for newly introduced code
- Core review pass only — specialized frontend/backend lenses and manual QA run when a review is explicitly requested
- Synthesized one review from the passes and chose the verdict from the findings, CI status, and QA evidence
|
@rtibblesbot Thanks for the review! I addressed the two suggestions:
The affected tests are passing. |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: all 15 prior findings resolved. CI passing. Manual QA did not run. The UI is unverified.
Prior-finding status
RESOLVED — shared/vuex/baseStore.js:52 — Removing module leaves unmigrated consumers
RESOLVED — shared/views/StudioCopyToken/tests/studioCopyToken.spec.js — unused mockStore
RESOLVED — channelEdit/views/trash/tests/trashModal.spec.js — unused dispatchSpy
RESOLVED — QuickEditModal/tests/EditTitleDescriptionModal.spec.js — snackbar text unchecked
RESOLVED — useCommunityLibraryUpdates.js:178 — 5000 ms vs 6000 ms duration
RESOLVED — SubmitToCommunityLibrarySidePanel/index.vue:604 — Cancel unreachable by keyboard
RESOLVED — shared/views/GlobalSnackbar.vue:15 — callback wiring untested
RESOLVED — channelEdit/views/CurrentTopicView.vue — unneeded hideCallback deferral
RESOLVED — useToken.js:42 — praise: useKSnackbar() at setup fixes TDZ bug
RESOLVED — shared/views/GlobalSnackbar.vue — hideCallback runs twice on close
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — mock can't catch double hideCallback
RESOLVED — QuickEditModal/tests/EditSourceModal.spec.js — assertion runs after test completes
RESOLVED — QuickEditModal/tests/EditSourceModal.spec.js — duplicate imported-node test
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — :text assertion reads back the ref
RESOLVED — CurrentTopicView.vue:917 — praise: autofocus: true on Undo/Cancel
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
Ran an automatic code-only delta review triggered by new commits on a previously reviewed PR:
- Retrieved prior bot reviews via the GitHub API
- Classified each prior finding as RESOLVED, UNADDRESSED, ACKNOWLEDGED, or CONTESTED
- Only raised NEW findings for newly introduced code
- Core review pass only — specialized frontend/backend lenses and manual QA run when a review is explicitly requested
- Synthesized one review from the passes and chose the verdict from the findings, CI status, and QA evidence
|
@rtibblesbot Could you please run the review/checks again on the latest commit? I’ve fixed the formatting issues and pushed the update. Thanks! |
|
Thanks @Monier-Ayman — the re-review picks up new commits automatically; the status comment above updates when it runs. CI shows no checks on |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: all 16 prior findings resolved. The new commits are formatting only. No new findings. CI passing. UI unverified: manual QA did not run.
Prior-finding status
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — assertion passes with :text binding deleted
RESOLVED — channelEdit/components/QuickEditModal/tests/EditSourceModal.spec.js — duplicate imported-node test
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js:56 — praise: real useKSnackbar singleton with toHaveBeenCalledTimes(1)
RESOLVED — shared/vuex/baseStore.js:52 — removing module leaves three unmigrated consumers
RESOLVED — shared/views/StudioCopyToken/tests/studioCopyToken.spec.js — unused mockStore
RESOLVED — channelEdit/views/trash/tests/trashModal.spec.js — unused dispatchSpy
RESOLVED — channelEdit/components/QuickEditModal/tests/EditTitleDescriptionModal.spec.js — no longer checks snackbar text
RESOLVED — shared/views/NotificationsModal/composables/useCommunityLibraryUpdates.js:178 — 5000 ms duration
RESOLVED — channelEdit/components/sidePanels/SubmitToCommunityLibrarySidePanel/index.vue:604 — keyboard users can't reach Cancel
RESOLVED — shared/views/GlobalSnackbar.vue:15 — callback wiring untested
RESOLVED — channelEdit/views/CurrentTopicView.vue — deferral not needed
RESOLVED — shared/composables/useToken.js:42 — praise: setup-time useKSnackbar() fixes TDZ bug
RESOLVED — shared/views/GlobalSnackbar.vue — hideCallback runs twice on close
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — spec mocks useKSnackbar, can't catch double call
RESOLVED — channelEdit/components/QuickEditModal/tests/EditSourceModal.spec.js — assertion runs after synchronous test ends
RESOLVED — channelEdit/views/CurrentTopicView.vue:917 — praise: autofocus: true on Undo/Cancel snackbars
Written by rtibblesbot, an LLM-based coding agent.
|
Hi @AlexVelezLl! Could you please take a look at this? |
|
Hey @Monier-Ayman! Seems like there are some failing checks, could you take a look at them, please? 👐 |
|
Hey @Monier-Ayman! Could you please review and address the feedback provided? If we don't hear back from you, we'll assume you're unable to proceed with the requested changes, and the issue will be unassigned from you and the PR probably closed. |
|
Yes, of course @akolson ! I’ll take a look at the feedback and work on the requested changes. Thanks for the reminder! |
- Drop explicit duration: 6000 and autoDismiss: true across all call sites to use KDS defaults - Remove duration: null from persistent snackbar callers - Remove color: 'error' from StagingTreePage - Set announce: false for ChannelSetModal - Migrate upstream Organization views from deprecated useSnackbar to useKSnackbar - Remove useKSnackbar jest mocks across 18 test specs and assert against real composable - Assert snackbar text in studioCopyToken.spec.js and clean up mockStore - Rewrite GlobalSnackbar.spec.js with Vue Testing Library - Rename test descriptions still referencing showSnackbarSimple
|
Hi @AlexVelezLl, @akolson I have merged the latest
Please take another look when you have a moment. Thanks! |
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
PR #6144: 16 of 16 prior findings resolved; one new suggestion inline.
CI passing. Manual QA did not run, so the UI is unverified.
Prior-finding status
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — :text assertion passes without the binding
RESOLVED — QuickEditModal/tests/EditSourceModal.spec.js — duplicate of :261
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js:56 — praise: real useKSnackbar singleton
RESOLVED — shared/vuex/baseStore.js:52 — unmigrated consumers of removed module
RESOLVED — StudioCopyToken/tests/studioCopyToken.spec.js — unused mockStore
RESOLVED — trash/tests/trashModal.spec.js — unused dispatchSpy
RESOLVED — QuickEditModal/tests/EditTitleDescriptionModal.spec.js — snackbar text not checked
RESOLVED — NotificationsModal/composables/useCommunityLibraryUpdates.js:178 — 5000 ms duration
RESOLVED — SubmitToCommunityLibrarySidePanel/index.vue:601 — Cancel unreachable by keyboard
RESOLVED — shared/views/GlobalSnackbar.vue:15 — callback wiring untested
RESOLVED — channelEdit/views/CurrentTopicView.vue — unneeded deferral
RESOLVED — shared/composables/useToken.js:42 — praise: TDZ fix
RESOLVED — shared/views/GlobalSnackbar.vue — hideCallback runs twice
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — mock hides double hideCallback
RESOLVED — QuickEditModal/tests/EditSourceModal.spec.js — assertion runs after test finishes
RESOLVED — channelEdit/views/CurrentTopicView.vue:916 — praise: autofocus: true
Next step: Needs a full code review by a human engineer, then manual QA once that reviewer sets Needs QA.
- The author is outside the team: confirm this should be built at all.
- User-visible files changed:
contentcuration/contentcuration/frontend/administration/components/ClipboardChip.vue,contentcuration/contentcuration/frontend/administration/components/sidePanels/ReviewSubmissionSidePanel.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelActionsDropdown.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelItem.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelTable.vue…. - Check that snackbars still auto-dismiss after about 6 s now that explicit
duration/autoDismissrely on the KDS defaults, and that the "creating copies" snackbars stay up. - Check that the Organization views' snackbars appear and are announced to screen readers.
Written by rtibblesbot, an LLM-based coding agent.
…l specs - Make EditAudienceModal tests async and await flushPromises() instead of requestAnimationFrame - Call useKSnackbar().clearSnackbar() in beforeEach across QuickEditModal specs to prevent state leakage - Trigger KModal events via findComponent to correctly invoke modal actions
|
@rtibblesbot Done in commit ce53d03. Made the tests in EditAudienceModal.spec.js async and replaced requestAnimationFrame with await flushPromises(). Also added useKSnackbar().clearSnackbar() in beforeEach across the QuickEditModal specs to ensure clean state isolation between tests. |
|
📢✨ Before we assign a reviewer, we'll turn on |
|
@AlexVelezLl, any other thoughts on this? |
There was a problem hiding this comment.
PR #6144: no new findings. 11 of 17 prior findings resolved; 6 acknowledged. CI pending.
Prior-finding status
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js — :text assertion passes without the binding
RESOLVED — channelEdit/components/QuickEditModal/tests/EditSourceModal.spec.js — test duplicates :261
RESOLVED — shared/views/tests/GlobalSnackbar.spec.js:56 — praise: real useKSnackbar singleton
RESOLVED — shared/vuex/baseStore.js:52 — unmigrated consumers of removed module
RESOLVED — shared/views/StudioCopyToken/tests/studioCopyToken.spec.js — unused mockStore
RESOLVED — channelEdit/views/trash/tests/trashModal.spec.js — unused dispatchSpy
RESOLVED — channelEdit/components/QuickEditModal/tests/EditTitleDescriptionModal.spec.js — snackbar text not checked
ACKNOWLEDGED — shared/views/NotificationsModal/composables/useCommunityLibraryUpdates.js:178 — 5000 ms duration
ACKNOWLEDGED — channelEdit/components/sidePanels/SubmitToCommunityLibrarySidePanel/index.vue:601 — keyboard reach of Cancel
ACKNOWLEDGED — shared/views/GlobalSnackbar.vue:15 — callback wiring untested
ACKNOWLEDGED — channelEdit/views/CurrentTopicView.vue — deferral unneeded
RESOLVED — shared/composables/useToken.js:42 — praise: TDZ fix
ACKNOWLEDGED — shared/views/GlobalSnackbar.vue — hideCallback runs twice on close
ACKNOWLEDGED — shared/views/tests/GlobalSnackbar.spec.js — mock cannot catch double hideCallback
RESOLVED — channelEdit/components/QuickEditModal/tests/EditSourceModal.spec.js — assertion after test completes
RESOLVED — channelEdit/views/CurrentTopicView.vue:916 — praise: autofocus on Undo/Cancel
RESOLVED — channelEdit/components/QuickEditModal/tests/EditAudienceModal.spec.js — assertion can never fail
Next step: Needs a full code review by a human engineer, then manual QA once that reviewer sets Needs QA.
- The author is outside the team: confirm this should be built at all.
- User-visible files changed:
contentcuration/contentcuration/frontend/administration/components/ClipboardChip.vue,contentcuration/contentcuration/frontend/administration/components/sidePanels/ReviewSubmissionSidePanel.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelActionsDropdown.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelItem.vue,contentcuration/contentcuration/frontend/administration/pages/Channels/ChannelTable.vue…. - Decide whether the acknowledged double
hideCallbackcall on close inGlobalSnackbar.vueis acceptable, since clipboard and undochangeTracker.cleanUp()now depend on it. - Confirm the
useKSnackbarmigration pattern and the removal of the Vuex snackbar module are what other code should copy. - Decide whether to fix or drop
ImportFromChannelsModal'sbeforeRouteUpdateclear; it never runs, because the modal is not a route component (same as on base).
QA captures (43)
Edit a resource's audience via the quick-edit Audience modal (select resource, choose role and For beginners, save) — "Changes saved" snackbar appears once, the modal closes, and the new role/beginner values persist after reopening
| Step | Screenshot |
|---|---|
| Audience modal coach beginners |  |
| Audience modal initial |  |
| Audience persisted after reload |  |
| Changes saved snackbar |  |
| Keyboard save snackbar |  |
| Snackbar small screen |  |
1-audience-save-flow.webm
Edit categories or another boolean-map field (e.g. levels) via its quick-edit modal on one and on multiple selected resources, save — single "Changes saved" snackbar, values applied to every selected node, no stale snackbar left from a previous edit
| Step | Screenshot |
|---|---|
| Multi categories mixed modal |  |
| Multi categories saved snackbar |  |
| Multi levels modal |  |
| Multi levels saved snackbar |  |
| Single categories saved snackbar |  |
2-multi-categories-save.webm
Edit language via the quick-edit Language modal on a topic with "apply to all descendants" and on a single resource, save — snackbar text, modal closes, language shown on the node and its children
| Step | Screenshot |
|---|---|
| Child video language fr |  |
| Rtl saved snackbar |  |
| Single resource edit panel language |  |
| Single resource modal es |  |
| Single resource saved snackbar |  |
| Topic modal fr descendants |  |
| Topic saved snackbar |  |
3-single-resource-language-save.webm
3-topic-language-descendants-save.webm
Open Import from other channels, search, apply a saved search or browse into a channel, then move between search results, browse and review pages — any snackbar shown earlier (e.g. saved-search confirmation) is cleared on route change and the import flow still completes with the imported-items snackbar in the channel
| Step | Screenshot |
|---|---|
| After import tree |  |
| Import browse |  |
| Review selections |  |
| Saved searches modal with snackbar |  |
| Search deleted snackbar |  |
| Search saved snackbar |  |
4-saved-search-snackbar-route-change.webm
Move resources to trash, open the trash modal, restore one item and permanently delete another — snackbar text for each action, list updates, restored node reappears in the tree
| Step | Screenshot |
|---|---|
| Delete confirm |  |
| Delete snackbar empty trash |  |
| Go to location |  |
| Removed snackbar single |  |
| Removed snackbar toolbar |  |
| Restore snackbar |  |
| Restored in tree |  |
| Trash modal two items |  |
5-trash-restore-delete.webm
From My channels, delete an editable channel and remove a view-only channel from the list — confirmation modal, success snackbar text for each, card disappears from the list
| Step | Screenshot |
|---|---|
| Channel removed snackbar |  |
| Delete confirm modal |  |
| Remove confirm modal |  |
6-delete-channel.webm
6-remove-view-only-channel.webm
Written by rtibblesbot, an LLM-based coding agent.
Summary
This PR migrates Studio's snackbar implementation from Vuetify/Vuex to the Kolibri Design System's
KSnackbaranduseKSnackbar, as part of the effort to remove Vuetify from Studio.What changed
Snackbar infrastructure
VSnackbarimplementation inGlobalSnackbar.vuewith KDSKSnackbar.useKSnackbar.shared/vuex/snackbarmodule.useSnackbarcomposable and its mock.VSnackbarregistration fromshared/app.js.Snackbar callers
Migrated all existing Studio snackbar callers to
useKSnackbar, including:Existing snackbar behavior was preserved where applicable, including:
hideCallbackbehaviorThe existing
showSnackbarOnCompleteSyncmethod was kept because its name is part of the surrounding workflow; its snackbar implementation now usesuseKSnackbar.Accessibility
The new
KSnackbarimplementation uses the accessibility features provided by the Kolibri Design System, including the existingannounceandassertiveoptions where appropriate.Tests
Updated affected tests and mocks to use
useKSnackbarinstead of the removed snackbar implementation.The affected test suite was run with Jest in-band:
Also verified with:
with no whitespace errors.
References
Closes #5955
Notes
The old
forceReuseoption had no actual callers in Studio. The KDS snackbar handles reactive text updates directly, so no additionalforceReusemigration was necessary.This PR is focused on replacing the snackbar infrastructure while preserving the existing user-facing behavior.
AI Usage
I used AI (Antigravity) as a coding assistant while working on this migration. I planned and implemented the changes myself, with AI helping me investigate the existing implementation, work through some of the changes, and review parts of the implementation.
I reviewed and refined all changes myself, verified the behavior against the issue requirements, checked the existing code and history, and ran the affected Jest tests and local pre-commit checks.