Commit graph

7 commits

Author SHA1 Message Date
Peter Steinberger
bc9f505026
Fix Ghostty terminal spawn issues with dynamic delays (#408)
* Fix Ghostty terminal spawn issues with dynamic delays

- Add isTerminalRunning() helper to check if terminal app is running
- Implement dynamic delays for Ghostty based on running state
  - 0.5s delay for warm start (already running)
  - 2.0s delay for cold start (needs to launch)
- Add window count checking to ensure UI is ready
- Fix issue where commands weren't executed when Ghostty had no windows

Fixes #371

* Fix CI: Skip Node.js check when using pre-built web artifacts

- Add SKIP_NODE_CHECK=true environment variable to Mac CI build step
- Prevents install-node.sh from failing when pnpm is not available
- CI downloads pre-built web artifacts, so Node.js/pnpm are not needed

* Fix CI: Properly handle pre-built web artifacts in Mac build

- Add early exit in build-web-frontend.sh when CI has pre-built artifacts
- Set CI=true environment variable in all Xcode build steps
- Update node-path-setup.sh to skip Node.js check in CI
- Copy pre-built artifacts directly without attempting rebuild
- This prevents pnpm dependency errors in CI environment

* Fix SwiftFormat modifier order issue

- Change 'static weak' to 'weak static' in AppDelegate
- SwiftFormat requires consistent modifier ordering

* Fix CI: Include native binaries in web artifacts

- Add web/native/ directory to uploaded artifacts
- Add web/bin/vt script to uploaded artifacts
- This ensures Mac tests can find the vibetunnel executable
- Fixes test failures due to missing server binary

* Fix CI: Copy native binaries from web artifacts in Mac CI

- Update artifact extraction to copy web/native/ directory
- Also copy web/bin/ directory for vt script
- Add debugging output to show native contents
- This ensures tests can find the vibetunnel executable
2025-07-18 17:24:30 +02:00
Peter Steinberger
9c81ed9969 Fix unbound variable error in node-path-setup.sh
The script was failing with "VIBETUNNEL_BUILD_CLEAN_ENV: unbound variable"
when running under strict bash error checking (set -u). Fixed by using
parameter expansion with default value ${VAR:-} to handle cases where
the variable is not set.
2025-07-08 10:10:42 +01:00
Peter Steinberger
6b93fdbf41
Fix Homebrew library dependencies in release builds (#269) 2025-07-08 09:55:52 +01:00
Peter Steinberger
53b4aff571 Fix build warnings
- Fix 'variable was never mutated' warning in ScreencapService.swift
- Fix fnm env command syntax error in node-path-setup.sh
- Remove incorrect --use-on-cd flag usage

The duplicate rpath warning remains (needs Xcode project fix)
2025-07-08 01:16:42 +01:00
Raghav Sethi
48ea8898fa
fix: resolve Xcode Node.js detection issues with fnm/homebrew conflicts (#246) (#253) 2025-07-08 00:44:05 +01:00
hewigovens
89240a09a9
cleanup 2025-07-06 11:32:16 +09:00
hewigovens
826d8de4c1
refactor: consolidate Node.js path setup into shared script and add fnm support 2025-07-06 11:09:32 +09:00