Commit Graph

14 Commits

Author SHA1 Message Date
Matthew Raymer
c63799999a chore: resolve all TypeScript lint warnings in PlatformServiceMixin
- Fix remaining type warnings by specifying Settings for cache and using this._setCached directly
- Add eslint-disable-next-line comments for unavoidable (this as any) usages required for Vue context access
- All @typescript-eslint/no-explicit-any warnings are now suppressed or resolved
- Lint passes with zero warnings or errors
- No functional changes; improves code clarity and developer experience
2025-07-03 11:00:27 +00:00
Matthew Raymer
1d8b63b52a refactor: add strict types to PlatformServiceMixin interfaces
- Replace all `any` types in IPlatformServiceMixin and ComponentCustomProperties with proper types:
  - Use PlatformCapabilities for capabilities
  - Use QueryExecResult, DatabaseExecResult, and generics for database methods
  - Use Contact[] and Settings for contact and settings methods
- Improves IDE type inference and type safety for all consumers of the mixin
- No functional changes, interface/type declarations only
2025-07-03 10:46:05 +00:00
Matthew Raymer
d99c8e993c refactor: improve type safety in PlatformServiceMixin with database and cache improvements
- Add proper return types for database methods: QueryExecResult, DatabaseExecResult, Contact[]
- Improve cache system typing with VueComponentWithMixin interface and CacheEntry<unknown>
- Fix $refreshContacts() return type from Promise<any[]> to Promise<Contact[]>
- Update database interfaces with DatabaseExecResult and generic DatabaseResult types
- Maintain runtime compatibility by keeping necessary (this as any) for Vue component context
- Reduces TypeScript warnings from 81 to 23 while preserving all functionality
- Improves type safety for database operations, caching, and contact management
2025-07-03 10:39:26 +00:00
Matthew Raymer
54e30c1b48 refactor: improve cache system typing in PlatformServiceMixin
- Replace WeakMap<any, Map<string, CacheEntry<any>>> with proper VueComponentWithMixin interface
- Add VueComponentWithMixin interface with _platformService, $options, activeDid, and platformService() method
- Use CacheEntry<unknown> instead of CacheEntry<any> for better type safety
- Replace (this as any) with (this as unknown as VueComponentWithMixin) for safer type assertions
- Reduces TypeScript warnings from 53 to 29 (24 warnings eliminated)
- Maintains all existing functionality while improving type safety
2025-07-03 10:24:21 +00:00
Matthew Raymer
829edaf7d7 refactor: add strict typing to core database methods in PlatformServiceMixin
- Replace all `any` types in $db, $exec, $one, $query, $first, and $contacts with proper interfaces:
  - Use QueryExecResult and DatabaseExecResult for raw database operations
  - Use Contact[] for contact queries
  - Add generics for mapped query results
- Update PlatformService and database interfaces to match new types
- Remove unused type imports and fix linter errors
- Reduces risk of runtime errors and improves type safety for all database access
2025-07-03 10:14:35 +00:00
Matthew Raymer
f839b65c56 fix: resolve SVG loading error and improve TypeScript typing
- Fix EntityIcon.vue to import blank-square.svg as module instead of using relative path
- Update template to use imported SVG path for proper Electron compatibility
- Make contact prop optional in EntityIcon component to fix TypeScript error
- Add proper Settings type imports and method signatures in PlatformServiceMixin.ts
- Replace console statements with logger calls across multiple files
- Resolves SVG loading failures in Electron builds while maintaining web compatibility
- Reduces TypeScript 'any' type warnings from 81 to 53
2025-07-03 10:03:58 +00:00
Matthew Raymer
e39f6a2d37 Replace console statements with structured logger calls
- Replace 30 console.log/error/warn statements with appropriate logger calls
- Add logger imports where missing (PlatformServiceFactory, PlatformServiceMixin)
- Maintain eslint-disable comments for critical error logging in worker context
- Improve error handling consistency across platform services
- Clean up commented-out logger statements in WebPlatformService

Files affected:
- src/db/databaseUtil.ts: 4 console statements → logger calls
- src/main.electron.ts: 3 console statements → logger calls
- src/registerSQLWorker.js: 9 console statements → logger calls
- src/services/PlatformServiceFactory.ts: 1 console statement → logger call
- src/services/platforms/WebPlatformService.ts: 8 console statements → logger calls
- src/utils/PlatformServiceMixin.ts: 5 console statements → logger calls

Reduces ESLint warnings from 111 to 82 (eliminates all no-console warnings)
2025-07-03 08:32:41 +00:00
Matthew Raymer
a5eb5cacaf Clean up console logging and fix platform detection issues
- Reduce migration/platform logging verbosity in development mode (~80-90% less noise)
- Fix AbsurdSQL platform check to support 'development' alongside 'web'
- Add missing WebPlatformService methods (isWorker, initSharedArrayBuffer)
- Fix Electron API endpoint resolution to prevent JSON parsing errors
- Prevent circular database logging that caused [DB-PREVENTED-INFO] spam

Console now shows only essential information while preserving all errors/warnings
2025-07-03 06:31:43 +00:00
Matthew Raymer
7d5e81263c # Commit Message for SharedArrayBuffer Platform Exclusion
fix: eliminate SharedArrayBuffer checks on non-web platforms

* Add platform guard in AbsurdSqlDatabaseService to only initialize on web
* Change singleton pattern from eager to lazy instantiation
* Update worker import to use lazy singleton pattern
* Prevents absurd-sql initialization on Electron/Capacitor platforms
* Reduces console noise and memory footprint on desktop/mobile
* Maintains full web platform functionality and performance

Resolves SharedArrayBuffer-related console output on Electron platform
while preserving all web features and maintaining clean architecture.
2025-07-03 05:15:57 +00:00
Matthew Raymer
145cb11f38 Implement smart caching layer in PlatformServiceMixin v4.0.0
- Add TTL-based caching infrastructure with WeakMap for memory safety
- Add component-level cache management with 30s settings / 60s contacts TTL
- Implement automatic cache invalidation on settings updates
- Add settings update shortcuts: (), (), ()
- Add cached specialized methods: (), (), ()
- Add cache management methods: (), (), ()
- Upgrade comprehensive TypeScript declarations for enhanced IDE support
- Deliver massive performance gains for repeated database operations
- Reduce settings update boilerplate code by 90% with ultra-concise shortcuts
2025-07-02 10:52:20 +00:00
Matthew Raymer
da60cad799 Add smart caching layer and settings shortcuts to PlatformServiceMixin
- Add TTL-based caching infrastructure with WeakMap for memory safety
- Add (), (), () shortcuts
- Add cached (), (), () with 30s/60s TTL
- Add cache management methods: (), (), ()
- Implement automatic cache invalidation on settings updates
- Upgrade to v4.0.0 with massive performance gains for repeated operations
- Reduce settings update boilerplate by 90% with ultra-concise shortcuts
2025-07-02 10:46:49 +00:00
Matthew Raymer
4b7badcdd2 Add ultra-concise database methods to PlatformServiceMixin
- Add (), (), () for shortest possible database calls
- Add (), () for query+mapping combinations
- Add (), (), () specialized shortcuts
- Apply TypeScript declaration merging to eliminate type assertions
- Reduce HomeView database calls from 3 lines to 1 line (64% reduction)
- Convert settings loading from 6 lines to 4 lines (44% reduction)
- Replace all (this as any) type assertions with proper typing
2025-07-02 10:31:59 +00:00
Matthew Raymer
fecaa59a2a Enhance PlatformServiceMixin with utility methods and apply to TopMessage
Add $getSettings, $getMergedSettings utilities with built-in error handling. Reduce TopMessage code by 57% while eliminating Vue property conflicts.
2025-07-02 10:09:12 +00:00
Matthew Raymer
be61ba1bce Fix Vue property conflicts in PlatformServiceMixin implementation
- Remove duplicate property declarations from TopMessage component
- Use (this as any) type assertion for mixin methods
- Resolves 'Data property already defined' warnings
- Fixes 'this.dbQuery is not a function' runtime errors
2025-07-02 09:58:07 +00:00