github-starred/yaak
2
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-06-09 12:45:58 -05:00
Code Issues 119 Packages Projects Releases 104 Wiki Activity

[PR #428] [MERGED] Migrate to Vite+ unified toolchain #2964

New Issue
Closed
opened 2026-06-08 22:00:10 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-06-08 22:00:10 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/428
Author: @gschier
Created: 3/13/2026
Status: ✅ Merged
Merged: 3/13/2026
Merged by: @gschier

Base: main ← Head: migrate-vite-plus

📝 Commits (10+)

📊 Changes

166 files changed (+1841 additions, -1598 deletions)

View changed files

📝 .github/workflows/ci.yml (+6 -3)
📝 .github/workflows/release-app.yml (+7 -4)
➕ .node-version (+1 -0)
📝 .vite-hooks/post-checkout (+0 -0)
➕ .vite-hooks/pre-commit (+1 -0)
📝 .vscode/extensions.json (+1 -1)
📝 .vscode/settings.json (+5 -3)
➖ biome.json (+0 -55)
📝 crates-tauri/yaak-license/index.ts (+1 -1)
📝 crates/yaak-git/index.ts (+6 -6)
📝 crates/yaak-models/guest-js/store.ts (+1 -0)
📝 crates/yaak-sync/index.ts (+2 -2)
📝 crates/yaak-templates/build-wasm.cjs (+19 -0)
📝 crates/yaak-templates/pkg/yaak_templates.js (+5 -4)
📝 package-lock.json (+1473 -1261)
📝 package.json (+10 -9)
📝 packages/common-lib/debounce.ts (+2 -2)
📝 packages/plugin-runtime-types/src/plugins/AuthenticationPlugin.ts (+1 -1)
📝 packages/plugin-runtime-types/src/plugins/Context.ts (+1 -1)
📝 packages/plugin-runtime-types/src/plugins/TemplateFunctionPlugin.ts (+1 -1)

...and 80 more files

📄 Description

Migrate to Vite+ for a simplified toolchain

  • Replace Vite, Vitest, biome, and oxlint with Vite+ (vite-plus) unified toolchain
  • Enable type-aware linting with no-explicit-any (error), no-floating-promises, restrict-template-expressions, and other type-aware rules
  • Convert all 96 biome-ignore comments to oxlint-disable equivalents
  • Add fireAndForget utility for fire-and-forget promises with error logging and toast notifications
  • Fix all lint warnings across the codebase (type coercion, floating promises, redundant types, template expressions)
  • Remove biome config, deps, and VS Code integration
  • Add .node-version pin for Node 24 (required by vp)
  • Exclude flatpak, npm, and generated binding files from TS checking and linting

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mountain-loop/yaak/pull/428 **Author:** [@gschier](https://github.com/gschier) **Created:** 3/13/2026 **Status:** ✅ Merged **Merged:** 3/13/2026 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `migrate-vite-plus` --- ### 📝 Commits (10+) - [`49659a3`](https://github.com/mountain-loop/yaak/commit/49659a3da930ac99bdce4c3813295ae371c1b917) Migrate to Vite+ (vite-plus) unified toolchain - [`7f4eedd`](https://github.com/mountain-loop/yaak/commit/7f4eedd6301355f601ca439358ff99c5902e7126) Get lint running - [`e4103f1`](https://github.com/mountain-loop/yaak/commit/e4103f1a4af139eaa8a6a840ad1693b54dae0e6d) Fix some lint - [`be34dfe`](https://github.com/mountain-loop/yaak/commit/be34dfe74a811c38927ee5728da4d3c0dfadf9d5) Fix no-base-to-string - [`7670ab0`](https://github.com/mountain-loop/yaak/commit/7670ab007f53b2a6b90dbdb128978917e4a45df7) Fix lint errors - [`44a3319`](https://github.com/mountain-loop/yaak/commit/44a331929f2b2d78f96052881861302d25a09039) Enable type-aware linting and replace biome-ignore with oxlint-disable - [`21f7757`](https://github.com/mountain-loop/yaak/commit/21f775741a738f9709d2e5d313e61ea47fe4de18) More fixes - [`560c466`](https://github.com/mountain-loop/yaak/commit/560c4667e4ee64763312cafa280ebdbab293c115) More fixes - [`08a64b6`](https://github.com/mountain-loop/yaak/commit/08a64b69387e5b7c51bf3a93e3f7d8556723f90e) VSCode suggestion and CI - [`dde8d61`](https://github.com/mountain-loop/yaak/commit/dde8d61b4b5874ce0e1229471066be40bb2a3b4e) Run bootstrap ### 📊 Changes **166 files changed** (+1841 additions, -1598 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+6 -3) 📝 `.github/workflows/release-app.yml` (+7 -4) ➕ `.node-version` (+1 -0) 📝 `.vite-hooks/post-checkout` (+0 -0) ➕ `.vite-hooks/pre-commit` (+1 -0) 📝 `.vscode/extensions.json` (+1 -1) 📝 `.vscode/settings.json` (+5 -3) ➖ `biome.json` (+0 -55) 📝 `crates-tauri/yaak-license/index.ts` (+1 -1) 📝 `crates/yaak-git/index.ts` (+6 -6) 📝 `crates/yaak-models/guest-js/store.ts` (+1 -0) 📝 `crates/yaak-sync/index.ts` (+2 -2) 📝 `crates/yaak-templates/build-wasm.cjs` (+19 -0) 📝 `crates/yaak-templates/pkg/yaak_templates.js` (+5 -4) 📝 `package-lock.json` (+1473 -1261) 📝 `package.json` (+10 -9) 📝 `packages/common-lib/debounce.ts` (+2 -2) 📝 `packages/plugin-runtime-types/src/plugins/AuthenticationPlugin.ts` (+1 -1) 📝 `packages/plugin-runtime-types/src/plugins/Context.ts` (+1 -1) 📝 `packages/plugin-runtime-types/src/plugins/TemplateFunctionPlugin.ts` (+1 -1) _...and 80 more files_ </details> ### 📄 Description Migrate to [Vite+](https://viteplus.dev/) for a simplified toolchain - Replace Vite, Vitest, biome, and oxlint with Vite+ (`vite-plus`) unified toolchain - Enable type-aware linting with `no-explicit-any` (error), `no-floating-promises`, `restrict-template-expressions`, and other type-aware rules - Convert all 96 `biome-ignore` comments to `oxlint-disable` equivalents - Add `fireAndForget` utility for fire-and-forget promises with error logging and toast notifications - Fix all lint warnings across the codebase (type coercion, floating promises, redundant types, template expressions) - Remove biome config, deps, and VS Code integration - Add `.node-version` pin for Node 24 (required by vp) - Exclude `flatpak`, `npm`, and generated binding files from TS checking and linting --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-06-08 22:00:10 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#2964
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?