new build process to allow building native apps #126

Merged

trentlarson merged 41 commits from split_build_process into master 3 days ago

trentlarson commented 3 days ago

Owner

There is no content yet.

trentlarson added 41 commits 3 days ago

be8ba12df6 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.

a32c3c7765 Multi-build support; tested successfully for Electron

0001eb8784 Updates for cleanup

9adb8b01ee Android Capacitor configurations

94ee9e24ea Merge fixes

c7ced87845 Merge branch 'master' into split_build_process

f8db626d14 Fixes for APP_SERVER definition issue

b3b01652c4 Merge changes

6c97cafedb Merge branch 'master' into split_build_process

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.

43983bd993 (chore): merge mostly pathway changes

d9085ced6d (chore): cleaning up formatting and relative references

b1ba3935e6 docs: add comprehensive build instructions for all platforms

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.

562b27851c feat(electron): improve electron build configuration

- 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

47a28ff7ad commit working index.html

e6138bd0e2 fix: improve electron asset path handling

- 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.

976976e2ea chore: update gradle and android build tool versions

- 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.

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

- 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.

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

- 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

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

- 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

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

- 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

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

- 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

16bc47921d chore: linting for build

12025d6f21 chore: update in preparation for production mode

1045266310 fix: resolve nostr-tools typescript type conflicts

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

fa15f7da85 fix: remove navigator.serviceWorker redefinition in Vite config

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

8eabf76f56 fix: add missing DEFAULT_PARTNER_API_SERVER import in DiscoverView

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

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

- 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

a5c86fbc53 feat: improve test stability and API routing

- 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

7d225ea7b2 Touch up gitignore

777f72f85d refine instructions & remove duplicate info

f07a2de565 fix(tests): Improve gift recording test reliability

- 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.

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

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

a66093028e docs(tests): Add comprehensive test suite documentation

- 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.

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

f2febbd94a fix where function is not found

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

886202e4c8 fix a check for user's registration status

03161744b5 enable remainder of the devices for full UI tests

e2b7797e2d chore: add new playwright test results to .gitignore

caabaa3257 Merge branch 'master' into split_build_process

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.

67afe6a952 wait for some elements before calling 'click'

trentlarson merged commit 78d7f38aa3 into master 3 days ago

trentlarson referenced this issue from a commit 3 days ago

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

The pull request has been merged as 78d7f38aa3.

No reviewers

No Label

No Milestone

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.