new build process to allow building native apps #126

2025-02-17T15:50:30Z

trentlarson commented

2025-02-17 15:50:30 +00:00

No description provided.

trentlarson added 41 commits 2025-02-17 15:50:31 +00:00

Refatored vite.config to be a bit more streamlined before adding alternate build options. This did end up requiring me to remove @ from imports for some reason. Tests came out fine. be8ba12df6

Multi-build support; tested successfully for Electron a32c3c7765

Updates for cleanup 0001eb8784

Android Capacitor configurations 9adb8b01ee

Merge fixes 94ee9e24ea

Merge branch 'master' into split_build_process c7ced87845

Fixes for APP_SERVER definition issue f8db626d14

Merge changes b3b01652c4

Merge branch 'master' into split_build_process 6c97cafedb

fix: update import paths from alias to relative

- Replace @ alias imports with relative paths in HiddenDidDialog and UserProfileView
- Maintain consistent import style across components
- Improve build compatibility across different environments

This change helps ensure consistent module resolution across different
build targets including Capacitor and Electron builds.

(chore): merge mostly pathway changes 43983bd993

(chore): cleaning up formatting and relative references d9085ced6d

docs: add comprehensive build instructions for all platforms b1ba3935e6

Add detailed build instructions for web, desktop (Electron), and mobile
(Capacitor) platforms. Include prerequisites, setup steps, and platform-specific
build commands. Add sections for testing, linting, and development workflows.
Document build output directories and PWA features.

feat(electron): improve electron build configuration 562b27851c

- Add dedicated electron build scripts for each platform
- Configure Vite rollup options for electron builds
- Update electron-builder configuration with proper app metadata
- Add clean script and rimraf dependency
- Set proper appId and build settings for production builds
- Enable asar packaging for better security

commit working index.html 47a28ff7ad

fix: improve electron asset path handling e6138bd0e2

- Add extraResources config to package.json to properly include www directory
- Update path rewriting in build-electron.js to handle all asset types
- Fix modulepreload and stylesheet paths in index.html
- Improve path handling in main.js for packaged app
- Clean up file protocol handling and service worker management

This fixes the issue with assets not loading in the packaged Electron app
by ensuring all paths are correctly rewritten to reference the www directory
in the app resources.

chore: update gradle and android build tool versions 976976e2ea

- Update gradle wrapper from 8.9 to 8.10.2
- Update android build tools from 8.7.3 to 8.8.0
- Maintain build compatibility with latest tooling

These updates ensure the Android build process uses the latest stable
versions of the build tools and gradle wrapper.

WIP (fix): improve electron build configuration and service worker handling f0d0f63672

- Properly disable service workers in electron builds
- Add CSP headers for electron security
- Fix path resolution in electron context
- Improve preload script error handling and IPC setup
- Update build scripts for better electron/capacitor compatibility
- Fix router path handling in electron context
- Remove electron-builder dependency
- Streamline build process and output structure

This change improves the stability and security of electron builds while
maintaining PWA functionality in web builds. Service workers are now
properly disabled in electron context, and path resolution issues are
fixed.

docs: move build instructions from README to BUILDING.md 4e0f9235cd

- Move detailed setup and build instructions from README.md to BUILDING.md
- Add concise reference to BUILDING.md in README.md
- Keep testing and other documentation in README.md
- Improve organization of documentation by separating build-specific content

docs: enhance BUILDING.md with environment and deployment details a288496dcf

- Add environment configuration section with .env templates for dev/test/prod
- Add deployment instructions for test and production servers
- Add version management workflow steps
- Add troubleshooting section for common build issues
- Improve documentation organization and completeness

docs: update electron build instructions and fix electron asset paths de017d1a71

- Add web build prerequisite step to electron build docs
- Update electron run command to use npx
- Fix electron asset loading with improved path handling
- Add request interception for proper asset resolution
- Remove service worker files from electron build
- Improve debug logging for electron builds

fix: add Content Security Policy for Electron API connections 9a966ef04d

- Add CSP headers to allow connections to endorser.ch and timesafari.app APIs
- Configure secure content policies for images, scripts, styles and fonts
- Fix API connection errors by allowing required external resources
- Remove duplicate CSP header configuration

chore: linting for build 16bc47921d

chore: update in preparation for production mode 12025d6f21

fix: resolve nostr-tools typescript type conflicts 1045266310

- Update type definitions for nostr-tools
- Fix type compatibility issues
- Add missing type declarations

fix: remove navigator.serviceWorker redefinition in Vite config fa15f7da85

The previous configuration attempted to redefine the read-only navigator.serviceWorker
property, which caused runtime errors. Instead, we now handle service worker
availability through the transform plugin, which safely removes service worker
code in Electron and PyWebView builds.

This fixes the error:
"Cannot set property serviceWorker of #<Navigator> which has only a getter"

Technical changes:
- Removed navigator.serviceWorker from define section
- Rely on existing transform plugin to handle service worker code removal
- Maintains existing PWA functionality for web builds while properly
  disabling it for desktop builds

fix: add missing DEFAULT_PARTNER_API_SERVER import in DiscoverView 8eabf76f56

- Import DEFAULT_PARTNER_API_SERVER constant from app constants
- Fix missing import in DiscoverView component

refactor(tests): enhance contact management test reliability 11d1fae35a

- Add comprehensive error handling with try-catch blocks
- Add TypeScript type annotations for Page parameters
- Add debug logging for test troubleshooting
- Add retry logic for flaky operations
- Add API request routing for port conflicts
- Add detailed JSDoc documentation
- Extract helper functions for better maintainability
- Add timeout constants for different operations
- Improve error messages with better context
- Add load state handling for network operations

feat: improve test stability and API routing a5c86fbc53

- Add proxy configuration for API requests in dev/test
- Target default endorser API server in development
- Add proxy for /api endpoints to handle CORS
- Enhance contact management test reliability
- Add comprehensive error handling and debugging
- Extract test helper functions

Breaking: None

Touch up gitignore 7d225ea7b2

refine instructions & remove duplicate info 777f72f85d

fix(tests): Improve gift recording test reliability f07a2de565

- Add better error handling and logging for gift recording flow
- Add explicit navigation to contacts page before finding gift button
- Add info icon click handling when needed
- Add more comprehensive button detection with multiple selectors
- Add debug logging for page state and navigation
- Add screenshot capture on failures
- Add retry logic with proper state verification
- Fix linter errors in playwright config

The changes help diagnose and handle various UI states that can occur during gift recording, making the tests more reliable especially on Linux.

feat(tests): Add structured logging and screenshot capture 5550d6a411

Enhances test debugging capabilities by adding:
- Structured logging with timestamps and categories (INFO/STEP/SUCCESS/WAIT)
- Systematic screenshot capture at key test steps
- Comprehensive file documentation headers
- Improved error handling with visual state capture
- Organized test step numbering and progress tracking

Key improvements:
- Added screenshot helper function with consistent naming
- Ensured test-results directory exists before captures
- Replaced console.log with structured logging utility
- Added try-catch blocks with failure state capture
- Improved debugging info for gift button not found error

Tests affected:
- 35-record-gift-from-image-share.spec.ts
- 40-add-contact.spec.ts

Note: Screenshots are stored in test-results/ with test-specific prefixes

docs(tests): Add comprehensive test suite documentation a66093028e

- Add detailed header documentation to playwright tests
- Document test categories and flows
- Add key selector documentation
- Document state verification and alert handling
- Include code examples and usage patterns
- Add important checks and requirements

The documentation helps developers understand the foundational tests that verify
basic application functionality before running more complex test suites.

fix image server references for tests (2 tests failing: missing function & looking for registration prompt for unregistered user) c9c3cacfbd

fix where function is not found f2febbd94a

fix error setting the apiServer to "", plus some refactors while fixing test 40 811fddf24a

fix a check for user's registration status 886202e4c8

enable remainder of the devices for full UI tests 03161744b5

chore: add new playwright test results to .gitignore e2b7797e2d

Merge branch 'master' into split_build_process caabaa3257

fix: image server references and test configurations

- Update image server references to use test server by default for local dev
- Fix registration status checks in tests
- Remove verbose console logging
- Update environment configurations for consistent image server usage
- Fix alert handling in contact registration tests
- Clean up component lifecycle logging
- Add clarifying comments about shared image server usage
- Update playwright test configurations for better reliability

This commit ensures consistent image server behavior across environments
and improves test reliability by properly handling registration status
checks and alerts.

wait for some elements before calling 'click' 67afe6a952

trentlarson merged commit 78d7f38aa3 into master

2025-02-17 15:50:48 +00:00

trentlarson referenced this issue from a commit

2025-02-17 15:50:48 +00:00

Merge pull request 'new build process to allow building native apps' (#126) from split_build_process into master

anomalist referenced this issue from a commit

2025-07-22 08:51:06 +00:00

Merge pull request 'new build process to allow building native apps' (#126) from split_build_process into master

Sign in to join this conversation.