fix(store/filters): remove premature emit from onFilterUpdateChips

onFilterUpdateChips was emitting libresign:filters:update before
filter_status was persisted, causing getAllFiles to run with stale
state. The subsequent call from onFilterUpdateChipsAndSave (which
actually persists the new value) was then skipped because loading=true.

Only onFilterUpdateChipsAndSave should trigger a re-fetch, since it is
the action that actually updates filter_status/filter_modified.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

src/store/filters.js

@@ -44,7 +44,6 @@ export const useFiltersStore = defineStore('filter', {
 		async onFilterUpdateChips(event) {
 			this.chips = { ...this.chips, [event.id]: [...event.detail] }
 
-			emit('libresign:filters:update')
 			logger.debug('File list filter chips updated', { chips: event.detail })
 
 		},

src/tests/store/filters.spec.ts

@@ -198,8 +198,8 @@ describe('filters store - filter business rules', () => {
 		})
 	})
 
-	describe('business rule: chips update should emit filter event', () => {
-		it('onFilterUpdateChips should emit libresign:filters:update event', async () => {
+	describe('business rule: chips update should only update UI chips state', () => {
+		it('onFilterUpdateChips should NOT emit libresign:filters:update event', async () => {
 			const store = useFiltersStore()
 			const event = {
 				id: 'status',
@@ -208,7 +208,7 @@ describe('filters store - filter business rules', () => {
 
 			await store.onFilterUpdateChips(event)
 
-			expect(emit).toHaveBeenCalledWith('libresign:filters:update')
+			expect(emit).not.toHaveBeenCalled()
 		})
 
 		it('onFilterUpdateChips should update chips for specific filter', async () => {