Skip to content

🤖 perf: Use tsgo for main process compilation (19.6x faster) #264

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 15, 2025
Merged

🤖 perf: Use tsgo for main process compilation (19.6x faster) #264

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ce5b86f
🤖 perf: Use tsgo for main process compilation (19.6x faster)
ammar-agent Oct 15, 2025
fe06c7a
Remove stray investigation document
ammar-agent Oct 15, 2025
a3af01d
Remove conditional branches from build targets
ammar-agent Oct 15, 2025
3c61331
DRY: Extract tsgo path to TSGO variable
ammar-agent Oct 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 13 additions & 12 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,11 @@
#
# Adding New Targets:
# Add `## Description` after the target to make it appear in `make help`
#
# Build Reproducibility:
# AVOID CONDITIONAL BRANCHES (if/else) IN BUILD TARGETS AT ALL COSTS.
# Branches reduce reproducibility - builds should fail fast with clear errors
# if dependencies are missing, not silently fall back to different behavior.

# Include formatting rules
include fmt.mk
Expand All @@ -31,6 +36,9 @@ include fmt.mk
.PHONY: ensure-deps
.PHONY: check-eager-imports check-bundle-size check-startup

# Build tools
TSGO := bun run node_modules/@typescript/native-preview/bin/tsgo.js

TS_SOURCES := $(shell find src -type f \( -name '*.ts' -o -name '*.tsx' \))

# Default target
Expand All @@ -56,7 +64,7 @@ help: ## Show this help message
## Development
dev: node_modules/.installed build-main ## Start development server (Vite + tsgo watcher for 10x faster type checking)
@bun x concurrently -k \
"bun x concurrently \"bun run node_modules/@typescript/native-preview/bin/tsgo.js -w -p tsconfig.main.json\" \"bun x tsc-alias -w -p tsconfig.main.json\"" \
"bun x concurrently \"$(TSGO) -w -p tsconfig.main.json\" \"bun x tsc-alias -w -p tsconfig.main.json\"" \
"vite"

start: node_modules/.installed build-main build-preload build-static ## Build and start Electron app
Expand All @@ -69,7 +77,7 @@ build-main: node_modules/.installed dist/main.js ## Build main process

dist/main.js: src/version.ts tsconfig.main.json tsconfig.json $(TS_SOURCES)
@echo "Building main process..."
@NODE_ENV=production bun x tsc -p tsconfig.main.json
@NODE_ENV=production $(TSGO) -p tsconfig.main.json
@NODE_ENV=production bun x tsc-alias -p tsconfig.main.json

build-preload: node_modules/.installed dist/preload.js ## Build preload script
Expand Down Expand Up @@ -136,16 +144,9 @@ lint-fix: node_modules/.installed ## Run linter with --fix
@./scripts/lint.sh --fix

typecheck: node_modules/.installed src/version.ts ## Run TypeScript type checking (uses tsgo for 10x speedup)
@if [ -f "node_modules/@typescript/native-preview/bin/tsgo.js" ]; then \
bun x concurrently -g \
"bun run node_modules/@typescript/native-preview/bin/tsgo.js --noEmit" \
"bun run node_modules/@typescript/native-preview/bin/tsgo.js --noEmit -p tsconfig.main.json"; \
else \
echo "⚠️ tsgo not found, falling back to tsc (slower)"; \
bun x concurrently -g \
"tsc --noEmit" \
"tsc --noEmit -p tsconfig.main.json"; \
fi
@bun x concurrently -g \
"$(TSGO) --noEmit" \
"$(TSGO) --noEmit -p tsconfig.main.json"

## Testing
test-integration: node_modules/.installed ## Run all tests (unit + integration)
Expand Down