From 44be0353df6a283fc9e254c9f9ac4814ce1b9126 Mon Sep 17 00:00:00 2001 From: v-byte-cpu <65545655+v-byte-cpu@users.noreply.github.com> Date: Sat, 6 Jun 2026 18:53:52 +0400 Subject: [PATCH] ci(dependabot): split update groups by stack Split Dependabot groups so codegen, UI testing, mock-api tooling, and general tooling no longer overlap. Also make the trash refresh-status test/story use a non-retryable error so the stale status assertion is stable in CI. --- .github/dependabot.yml | 25 ++++++++++++++++++- .../trash/pages/TrashPage.stories.tsx | 4 +-- .../features/trash/pages/TrashPage.test.tsx | 4 +-- 3 files changed, 28 insertions(+), 5 deletions(-) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index 151acaf..b5eba41 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -39,6 +39,9 @@ updates: - minor patterns: - "@tanstack/*" + exclude-patterns: + - "@tanstack/router-plugin" + - "@tanstack/router-cli" ui-runtime: applies-to: version-updates @@ -64,6 +67,19 @@ updates: - zustand - lucide-react + mock-api-tooling: + applies-to: version-updates + update-types: + - patch + - minor + patterns: + - "@hono/node-server" + - hono + - tsx + - yaml + - esbuild + - "@types/node" + codegen: applies-to: version-updates update-types: @@ -74,7 +90,7 @@ updates: - "@tanstack/router-plugin" - "@tanstack/router-cli" - tooling: + ui-testing: applies-to: version-updates update-types: - patch @@ -88,6 +104,13 @@ updates: - jsdom - msw - playwright + + tooling: + applies-to: version-updates + update-types: + - patch + - minor + patterns: - eslint* - typescript* - vite diff --git a/ui/src/features/trash/pages/TrashPage.stories.tsx b/ui/src/features/trash/pages/TrashPage.stories.tsx index 82c45db..68980e3 100644 --- a/ui/src/features/trash/pages/TrashPage.stories.tsx +++ b/ui/src/features/trash/pages/TrashPage.stories.tsx @@ -14,7 +14,7 @@ import { } from '@/test/storybook/page-services' import { withStorybookApp } from '@/test/storybook/providers' import { dayMs, timestampAgo } from '@/test/storybook/fixtures' -import { err, type DomainError } from '@shared/errors' +import { domainError, err, type DomainError } from '@shared/errors' import type { TrashService } from '../services/trashService' import type { TrashState } from '../types/trash.types' @@ -221,7 +221,7 @@ export const RefreshTrashError: Story = { services: () => createStorybookServices({ trash: createTrashServiceWithPostMutationRefreshError({ - error: unavailableError('Trash storage is temporarily unavailable.'), + error: domainError.unexpected('Trash storage is temporarily unavailable.'), }), workspaces: createWorkspaceService(), }), diff --git a/ui/src/features/trash/pages/TrashPage.test.tsx b/ui/src/features/trash/pages/TrashPage.test.tsx index 0cebc9b..cc0c403 100644 --- a/ui/src/features/trash/pages/TrashPage.test.tsx +++ b/ui/src/features/trash/pages/TrashPage.test.tsx @@ -166,7 +166,7 @@ describe('TrashPage', () => { it('shows a stale refresh status when delete succeeds but trash refetch fails', async () => { const user = userEvent.setup() const baseServices = createAppServices('mock') - const refreshError = domainError.unavailable( + const refreshError = domainError.unexpected( 'Trash storage is temporarily unavailable.', ) let failFutureList = false @@ -204,7 +204,7 @@ describe('TrashPage', () => { expect(deleteItem).toHaveBeenCalledWith('sampling-error-notes') expect(await screen.findByText('Trash may be out of date')).toBeInTheDocument() - expect(screen.getByText('The service is temporarily unavailable.')).toBeInTheDocument() + expect(screen.getByText('Trash storage is temporarily unavailable.')).toBeInTheDocument() expect(screen.getByText('Sampling Error Notes')).toBeInTheDocument() await waitFor(() => { expect(list.mock.calls.length).toBeGreaterThanOrEqual(2)