build(deps-dev): Bump vitest from 3.2.7 to 4.1.10 in /frontend - #5
Closed
dependabot[bot] wants to merge 1 commit into
Closed
build(deps-dev): Bump vitest from 3.2.7 to 4.1.10 in /frontend#5dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Conversation
dependabot
Bot
added
dependencies
dependabot
Bot
force-pushed
the
dependabot/npm_and_yarn/frontend/vitest-4.1.10
branch
from
August 12, 2026 02:03
61afe6f to
8001460
Compare
Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 3.2.7 to 4.1.10. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Changelog](https://github.com/vitest-dev/vitest/blob/main/docs/releases.md) - [Commits](https://github.com/vitest-dev/vitest/commits/v4.1.10/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-version: 4.1.10 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/npm_and_yarn/frontend/vitest-4.1.10
branch
from
August 12, 2026 20:11
8001460 to
d3cc55c
Compare
8 tasks
alicoding
added a commit
that referenced
this pull request
Aug 12, 2026
Vitest 4 peers on vite ^6||^7||^8 -- Mill's frontend is already on vite 8.2.1, so this is a clean, isolated bump with no vite cascade (the hypothetical vite-5 scenario the goal brief flagged doesn't apply here). vite.config.ts's `test` block is a plain `exclude` array (no `workspace` config), so the workspace->projects rename in vitest 4 doesn't touch this repo either. No source changes needed -- package.json/package-lock.json only. Full local suite green: 231/231 vitest tests (29 files -- the jump from 227/28 reflects goal 0037's seedLifecycle.test.ts merging to main concurrently, not a vitest 4 behavior change, confirmed by diffing the file list), eslint (0 errors, 4 pre-existing warnings), tsc, boundaries, frontend build, go vet, go build (desktop+server), file-loc-limit, root-file-naming. Claude-Session: https://claude.ai/code/session_01FYwojT8GdUbYSoggbvEFft Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
Looks like vitest is up-to-date now, so this is no longer needed. |
dependabot
Bot
deleted the
dependabot/npm_and_yarn/frontend/vitest-4.1.10
branch
August 12, 2026 20:20
6 tasks
alicoding
added a commit
that referenced
this pull request
Aug 13, 2026
A plain ordered workflow-ID list (shared/store.ts's pinnedWorkflowIds, persisted via the existing zustand `persist` localStorage tier, same pattern as goal 0033's activeWorkTabKey) with a togglePinnedWorkflow action. app/workflowFrecency.ts gains sortWorkflowsByPinnedAndFrecency, partitioning pinned rows (in pin order) above the existing frecency-sorted unpinned tail rather than a second ranking algorithm. A subtle Primer PinIcon toggle lands on both the Quick Panel's and the ⌘K palette's workflow rows -- muted outline unpinned, accent-colored once pinned. Along the way, found and fixed a real Primer interaction bug: ActionList.Item's TrailingVisual wraps children in a VisualWrap span with pointer-events: none (trailing visuals are decorative-only by the library's own convention), which silently ate every click on the toggle until pointer-events: auto was added back on the button itself. Vitest covers the pinned-above-frecency/pin-order/dropped-stale-id/ no-mutation cases; quick-panel.spec.ts gained a full pin-sorts-above-frecency -> unpin-reverts -> persists-across-reload e2e case. Claude-Session: https://claude.ai/code/session_018pkViCNAuZp2vBv2K9AbUh Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
alicoding
added a commit
that referenced
this pull request
Aug 13, 2026
A plain ordered workflow-ID list (shared/store.ts's pinnedWorkflowIds, persisted via the existing zustand `persist` localStorage tier, same pattern as goal 0033's activeWorkTabKey) with a togglePinnedWorkflow action. app/workflowFrecency.ts gains sortWorkflowsByPinnedAndFrecency, partitioning pinned rows (in pin order) above the existing frecency-sorted unpinned tail rather than a second ranking algorithm. A subtle Primer PinIcon toggle lands on both the Quick Panel's and the ⌘K palette's workflow rows -- muted outline unpinned, accent-colored once pinned. Along the way, found and fixed a real Primer interaction bug: ActionList.Item's TrailingVisual wraps children in a VisualWrap span with pointer-events: none (trailing visuals are decorative-only by the library's own convention), which silently ate every click on the toggle until pointer-events: auto was added back on the button itself. Vitest covers the pinned-above-frecency/pin-order/dropped-stale-id/ no-mutation cases; quick-panel.spec.ts gained a full pin-sorts-above-frecency -> unpin-reverts -> persists-across-reload e2e case. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018pkViCNAuZp2vBv2K9AbUh
alicoding
added a commit
that referenced
this pull request
Aug 13, 2026
Vitest 4 peers on vite ^6||^7||^8 -- Mill's frontend is already on vite 8.2.1, so this is a clean, isolated bump with no vite cascade (the hypothetical vite-5 scenario the goal brief flagged doesn't apply here). vite.config.ts's `test` block is a plain `exclude` array (no `workspace` config), so the workspace->projects rename in vitest 4 doesn't touch this repo either. No source changes needed -- package.json/package-lock.json only. Full local suite green: 231/231 vitest tests (29 files -- the jump from 227/28 reflects goal 0037's seedLifecycle.test.ts merging to main concurrently, not a vitest 4 behavior change, confirmed by diffing the file list), eslint (0 errors, 4 pre-existing warnings), tsc, boundaries, frontend build, go vet, go build (desktop+server), file-loc-limit, root-file-naming. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FYwojT8GdUbYSoggbvEFft
alicoding
added a commit
that referenced
this pull request
Aug 14, 2026
* feat: user-facing vocabulary says step, not node (goal 0053) -- seed copy drops internal doc refs Sweeps every user-visible surface (locale JSON, NodeType/ConfigField Label/Description/Output, validation and run-failure messages, seeded workflow descriptions) from "node" to "step" per goal 0053, leaving code identifiers (NodeType, composition.Node, RefKind wire values) untouched. Rides the goal 0044 dry-run gap #5 rider: seeded descriptions in builtinworkflows*.go (plus two more found in list/builtin.go and httprequest/builtin.go) no longer cite internal docs/adr/goal references, and check-ui-copy.sh now gates builtinworkflows*.go the same way it already gates locale JSON. Bumps SeedRevision (1 -> 2) on every seeded entity whose Description changed and updates seed_fingerprints.json to match. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ * docs: goal 0053 acceptance recorded; ui-copy gate covers list/http seed descriptions Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
alicoding
added a commit
that referenced
this pull request
Aug 15, 2026
configureservice_test.go, settingsservice_test.go, triggerservice_test.go, executionservice_test.go, and executionchildworkflow_test.go (1000+ lines) previously ran in neither CI's test-go job nor Lefthook's go-test hook -- both scoped to ./internal/... only. Verified locally before wiring it in, not assumed: `go test . ./internal/... -race -cover` passes clean, root package at 57.2% coverage. Two real complications this surfaced, not just adding `.` to a command: - main.go's //go:embed all:frontend/dist makes the root package fail to compile without frontend/dist existing -- test-go didn't build it (deliberately, per its own now-stale comment: "internal/ packages never reference main.go's //go:embed"). Added needs: frontend and the same download-artifact step build-go/lint-go/govulncheck already use. Lefthook needed no equivalent fix: go-vet/go-build-desktop already require frontend/dist locally, so the dependency was already implied. - test-go runs on ubuntu-latest with no CGO_ENABLED override. build-go's own ubuntu-latest+server-tag entry already documents why that's required, not optional, on this runner (Wails3's internal/ operatingsystem/assetserver-webview packages are cgo-gated onto GTK4/webkitgtk-6.0 the runner doesn't have, pulled in regardless of the server tag unless cgo itself is disabled) -- previously moot for test-go since it never built the root package. Now load-bearing here too, so CGO_ENABLED: '0' was added to match. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FYwojT8GdUbYSoggbvEFft
alicoding
added a commit
that referenced
this pull request
Aug 15, 2026
Was 12 direct package-function call sites in package main (Set/Get/ Delete), no interface, no injection -- unlike settings.Store, which is already a real port every ConfigureService/CompositionService/ TriggerService consumer depends on rather than the concrete KVStore type. Confirmed by the architecture research this session: this was the one genuine adapter-shape gap among the three adapters checked (settings.Store already worked as advertised; execution's DSN parameterization, task #4, needed a one-line fix instead of this). credential.Store mirrors settings.Store's exact shape (small interface, a New() constructor callers depend on instead of the concrete type) -- the one difference is go-keyring itself is package-function-based with no existing struct to satisfy the interface structurally, so a small keyringStore{} adapter type exists solely to give those functions a method set. ConfigureService gained a `credentials credential.Store` field, threaded through NewConfigureService's third parameter from main.go (credential.New()). All 12 production call sites across configureservice_requestauth.go, configureservice_requesttest.go, and configureservice_builtin.go now go through c.credentials instead of the package directly, closing the actual gap: package main's tests (1000+ lines, wired into CI/Lefthook by task #5 this same session) were previously the only thing exercising this code path indirectly through keyring.MockInit() -- now the seam itself is real and independently testable. Verified: go vet, golangci-lint (0 issues), ls_lint, go test . ./internal/... -race -cover (root package unchanged at 57.2% coverage, credential adapter now 100%), go build for both desktop and server tags. All 14 test call sites across three _test.go files updated to pass credential.New() -- still exercises the real keyring, backed by the same keyring.MockInit() TestMain already covered the whole package main test binary with, so no test behavior changed, only how the dependency reaches the code under test. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FYwojT8GdUbYSoggbvEFft
alicoding
added a commit
that referenced
this pull request
Aug 15, 2026
Vitest 4 peers on vite ^6||^7||^8 -- Mill's frontend is already on vite 8.2.1, so this is a clean, isolated bump with no vite cascade (the hypothetical vite-5 scenario the goal brief flagged doesn't apply here). vite.config.ts's `test` block is a plain `exclude` array (no `workspace` config), so the workspace->projects rename in vitest 4 doesn't touch this repo either. No source changes needed -- package.json/package-lock.json only. Full local suite green: 231/231 vitest tests (29 files -- the jump from 227/28 reflects goal 0037's seedLifecycle.test.ts merging to main concurrently, not a vitest 4 behavior change, confirmed by diffing the file list), eslint (0 errors, 4 pre-existing warnings), tsc, boundaries, frontend build, go vet, go build (desktop+server), file-loc-limit, root-file-naming. Claude-Session: https://claude.ai/code/session_01FYwojT8GdUbYSoggbvEFft Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
alicoding
added a commit
that referenced
this pull request
Aug 15, 2026
A plain ordered workflow-ID list (shared/store.ts's pinnedWorkflowIds, persisted via the existing zustand `persist` localStorage tier, same pattern as goal 0033's activeWorkTabKey) with a togglePinnedWorkflow action. app/workflowFrecency.ts gains sortWorkflowsByPinnedAndFrecency, partitioning pinned rows (in pin order) above the existing frecency-sorted unpinned tail rather than a second ranking algorithm. A subtle Primer PinIcon toggle lands on both the Quick Panel's and the ⌘K palette's workflow rows -- muted outline unpinned, accent-colored once pinned. Along the way, found and fixed a real Primer interaction bug: ActionList.Item's TrailingVisual wraps children in a VisualWrap span with pointer-events: none (trailing visuals are decorative-only by the library's own convention), which silently ate every click on the toggle until pointer-events: auto was added back on the button itself. Vitest covers the pinned-above-frecency/pin-order/dropped-stale-id/ no-mutation cases; quick-panel.spec.ts gained a full pin-sorts-above-frecency -> unpin-reverts -> persists-across-reload e2e case. Claude-Session: https://claude.ai/code/session_018pkViCNAuZp2vBv2K9AbUh Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
alicoding
added a commit
that referenced
this pull request
Aug 15, 2026
* feat: user-facing vocabulary says step, not node (goal 0053) -- seed copy drops internal doc refs Sweeps every user-visible surface (locale JSON, NodeType/ConfigField Label/Description/Output, validation and run-failure messages, seeded workflow descriptions) from "node" to "step" per goal 0053, leaving code identifiers (NodeType, composition.Node, RefKind wire values) untouched. Rides the goal 0044 dry-run gap #5 rider: seeded descriptions in builtinworkflows*.go (plus two more found in list/builtin.go and httprequest/builtin.go) no longer cite internal docs/adr/goal references, and check-ui-copy.sh now gates builtinworkflows*.go the same way it already gates locale JSON. Bumps SeedRevision (1 -> 2) on every seeded entity whose Description changed and updates seed_fingerprints.json to match. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ * docs: goal 0053 acceptance recorded; ui-copy gate covers list/http seed descriptions Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FJ8wStsHyu7XPLTspNjMnQ --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps vitest from 3.2.7 to 4.1.10.
Release notes
Sourced from vitest's releases.
... (truncated)
Commits
db616d2chore: release v4.1.10 (#10718)bae52b5fix(vm): fix external module resolve error with deps optimizer query for enco...a7a61e7chore: release v4.1.9 (#10598)934b0f5fix(pool): prevent test run hang on worker crash (#10543) [backport to v4] (#...7fb2965fix(browser): wait for orchestrator readiness before resolving browser sessio...a518019fix: fiximportOriginalwith optimizer and query import [backport to v4] (#...e61f2ddchore: release v4.1.8e4067b3fix(browser): disable clientcdpAPI whenallowWrite/allowExec: false[ba...a09d472chore: release v4.1.7a8fd24cchore: release v4.1.6