Various tooling changes for Lite by OliverJAsh · Pull Request #12763 · gitbutlerapp/gitbutler

OliverJAsh · 2026-03-10T15:42:21Z

No description provided.

vercel · 2026-03-10T15:42:29Z

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
gitbutler-web	Skipped		Mar 13, 2026 1:47pm

Copilot

Pull request overview

This PR updates the monorepo tooling and the apps/lite Electron+React app setup, adding new lint/typecheck tooling (Knip + Oxc/Oxlint + tsgo) and expanding Lite’s IPC surface to call directly into @gitbutler/but-sdk N-API functions.

Changes:

Add Knip config + scripts and introduce Oxlint-based linting for apps/lite (and exclude Lite from the root ESLint run).
Expand Lite’s Electron IPC API/handlers and wire them directly to @gitbutler/but-sdk N-API functions.
Update Lite UI tooling/config (Vite/Vitest/TS config) and add TanStack React Query (plus devtools).

Reviewed changes

Copilot reviewed 20 out of 22 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
pnpm-lock.yaml	Locks new tooling/runtime deps (knip, oxlint, react-query, electron-devtools-installer, zod, etc.).
packages/but-sdk/src/generated/index.d.ts	Updates generated types (ProjectHandleOrLegacyProjectId) for `listProjectsNapi`.
package.json	Adds Knip scripts; extends `globallint` to run Lite’s `oxlint`; adds new dev deps.
knip.jsonc	Adds Knip configuration focused on `apps/lite`.
eslint.config.js	Switches to `defineConfig` and ignores `apps/lite/**` (Lite linting handled separately).
apps/lite/ui/vitest.config.ts	Removes path alias config from Vitest.
apps/lite/ui/vite.config.ts	Adds React compiler Babel plugin; removes Vite alias config.
apps/lite/ui/tsconfig.json	Tightens TS checks; removes path mappings in favor of package `imports`.
apps/lite/ui/src/router.tsx	Minor typing/style updates in `ProjectsList`.
apps/lite/ui/src/main.tsx	Adds React Query provider and devtools; switches router import to `#ui/*`.
apps/lite/package.json	Adds `imports` maps, switches to `tsgo`, adds Oxlint scripts and new deps.
apps/lite/electron/tsconfig.json	Tightens TS checks; removes path mappings.
apps/lite/electron/src/preload.cts	Expands exposed preload API to include more workspace/commit operations.
apps/lite/electron/src/model/workspace.ts	Removes thin wrapper in favor of calling N-API directly.
apps/lite/electron/src/model/projects.ts	Removes thin wrapper in favor of calling N-API directly.
apps/lite/electron/src/main.ts	Registers additional IPC handlers; installs React DevTools; refactors startup.
apps/lite/electron/src/ipc.ts	Expands IPC types and channel constants for new API surface.
apps/lite/AGENTS.md	Adds Lite-specific agent guidance for typechecking.
apps/lite/.oxlintrc.json	Adds Lite-specific Oxlint configuration/plugins/rules.
AGENTS.md	Removes outdated “Skills” section.
.vscode/settings.json	Updates ESLint validation and adds Oxc-related editor settings + `*.cts` association.
.vscode/extensions.json	Recommends the Oxc VS Code extension.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

apps/lite/electron/src/main.ts

apps/lite/electron/src/preload.cts

apps/lite/ui/src/main.tsx

Copilot

Pull request overview

Copilot reviewed 18 out of 19 changed files in this pull request and generated 5 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

apps/lite/electron/src/main.ts

apps/lite/electron/src/preload.cts

apps/lite/.oxlintrc.json

Copilot

Pull request overview

Copilot reviewed 18 out of 19 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

apps/lite/electron/src/preload.cts

apps/lite/electron/src/main.ts

apps/lite/electron/src/ipc.ts

For now we don't need any abstractions around the Node APIs.

This avoids false positives from the `no-unbound-method` lint rule.

Subpath imports are superior over TS paths because they're part of the standard, meaning more tools can use them.

Without this, the agent would sometimes run the wrong command, or not typecheck at all.

Copilot

Pull request overview

Copilot reviewed 21 out of 22 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

apps/lite/electron/src/ipc.ts

apps/lite/.oxlintrc.json

apps/lite/README.md

apps/lite/electron/src/main.ts

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 21 out of 22 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

apps/lite/electron/src/preload.cts

apps/lite/electron/src/main.ts

package.json

 		"lint": "turbo run //#globallint --no-daemon",
-		"globallint": "prettier --check . && eslint .",
+		"globallint": "prettier --check . && eslint . && pnpm --filter @gitbutler/lite run oxlint",


apps/lite/electron/src/ipc.ts

Copilot AI review requested due to automatic review settings March 10, 2026 15:42

Copilot started reviewing on behalf of OliverJAsh March 10, 2026 15:43 View session

Base automatically changed from push-zzktozzmkzyl to master March 10, 2026 15:44

Copilot AI reviewed Mar 10, 2026

View reviewed changes

apps/lite/electron/src/main.ts Outdated Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/preload.cts Show resolved Hide resolved

apps/lite/ui/src/main.tsx Show resolved Hide resolved

OliverJAsh force-pushed the lite-prep branch from 761781f to 5f1e779 Compare March 10, 2026 15:55

OliverJAsh mentioned this pull request Mar 10, 2026

Lite POC #12752

Merged

Copilot AI review requested due to automatic review settings March 11, 2026 21:04

OliverJAsh force-pushed the lite-prep branch from 5f1e779 to 7312c6f Compare March 11, 2026 21:04

Copilot started reviewing on behalf of OliverJAsh March 11, 2026 21:05 View session

Copilot AI reviewed Mar 11, 2026

View reviewed changes

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/preload.cts Show resolved Hide resolved

apps/lite/.oxlintrc.json Show resolved Hide resolved

OliverJAsh force-pushed the lite-prep branch from 7312c6f to 10297c2 Compare March 12, 2026 12:38

Copilot AI review requested due to automatic review settings March 12, 2026 14:43

OliverJAsh force-pushed the lite-prep branch from 10297c2 to 6593dd7 Compare March 12, 2026 14:43

Copilot started reviewing on behalf of OliverJAsh March 12, 2026 14:43 View session

Copilot AI reviewed Mar 12, 2026

View reviewed changes

apps/lite/electron/src/preload.cts Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

apps/lite/electron/src/ipc.ts Outdated Show resolved Hide resolved

OliverJAsh marked this pull request as draft March 13, 2026 08:19

OliverJAsh force-pushed the lite-prep branch 2 times, most recently from 4d73a1a to 6a7b677 Compare March 13, 2026 13:01

OliverJAsh added 11 commits March 13, 2026 13:02

Fix broken references

5cf8586

Use defineConfig as per ESLint docs

4b579ae

Format (via pnpm)

66dc736

Switch to tsgo

83345ff

Enable noUncheckedIndexedAccess

88b1079

Enable noImplicitReturns

5544c75

Add React Compiler

815bc8d

Add React DevTools

f8131da

Enable eslint-plugin-react-hooks

b85a7a7

Use Oxlint

f55a2ac

Add oxlint-tsgolint to root for VS Code extension

ba782d4

OliverJAsh added 17 commits March 13, 2026 13:07

Fix position of IPC handler registrations

f5c91df

Remove redundant wrappers

8b64e92

For now we don't need any abstractions around the Node APIs.

Remove return types

f1999a4

Remove redundant async/await

529d114

Sort

4c39b62

Add endpoints to Electron

8768557

Named params

f8b641c

Infer

1e12b28

Assert

a649cda

Methods -> properties

4027f1b

This avoids false positives from the `no-unbound-method` lint rule.

Remove redundant async/await

5d7865a

Add endpoints to Electron

9f79b38

Use subpath imports

2ddd1c7

Subpath imports are superior over TS paths because they're part of the standard, meaning more tools can use them.

Add agents for Lite

ea06e01

Without this, the agent would sometimes run the wrong command, or not typecheck at all.

VS Code: migrate deprecated config

08c82d2

VS Code: add config for typescriptreact files

541cd98

VS Code: configure cts as ts

6e53e59

OliverJAsh force-pushed the lite-prep branch from 6a7b677 to 6e53e59 Compare March 13, 2026 13:17

OliverJAsh marked this pull request as ready for review March 13, 2026 13:19

Copilot AI review requested due to automatic review settings March 13, 2026 13:19

Copilot started reviewing on behalf of OliverJAsh March 13, 2026 13:19 View session

Copilot AI reviewed Mar 13, 2026

View reviewed changes

apps/lite/electron/src/ipc.ts Show resolved Hide resolved

apps/lite/.oxlintrc.json Show resolved Hide resolved

apps/lite/README.md Outdated Show resolved Hide resolved

apps/lite/electron/src/main.ts Show resolved Hide resolved

Update docs

a4d62b9

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings March 13, 2026 13:47

vercel bot temporarily deployed to Preview March 13, 2026 13:47 Inactive

OliverJAsh enabled auto-merge March 13, 2026 13:48

Copilot started reviewing on behalf of OliverJAsh March 13, 2026 13:48 View session

Copilot AI reviewed Mar 13, 2026

View reviewed changes

OliverJAsh merged commit 5ead140 into master Mar 13, 2026
41 checks passed

OliverJAsh deleted the lite-prep branch March 13, 2026 13:58

Conversation

OliverJAsh commented Mar 10, 2026

Uh oh!

vercel bot commented Mar 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel bot commented Mar 10, 2026 •

edited

Loading