fix: code quality improvements & pre-refactor stabilization

[devin-ai-integration]

Summary

Comprehensive code quality improvements and pre-refactor stabilization for GeoSylva. These changes prepare the codebase for the planned major refactor of the database, internet sync, and analysis systems — without touching those systems.

Phase 1 — Critical fixes

• DB safety: BackupWorker and PriceSyncWorker no longer create their own Room instances — reuse the Application's shared instance (prevents SQLite corruption)
• Startup performance: Replaced runBlocking in Application.onCreate() with async coroutine (no longer blocks main thread)
• Migration logging: Created execSafe() helper so migration failures are logged instead of silently swallowed (catch (_: Throwable) {})
• CI pipeline: GitHub Actions workflow for unit tests + lint + debug build

Phase 2 — Pre-refactor stabilization

• Permission safety: Replaced all @SuppressLint("MissingPermission") with runtime ContextCompat.checkSelfPermission() checks in IbpEvaluationScreen, GpsAverager, GpsDistanceMeasureDialog — prevents SecurityException crashes on the field
• @Suppress cleanup: Replaced 11 @Suppress("UNUSED_PARAMETER") with Kotlin underscore-prefix convention across ForestryCalculator, ExportDataUseCase, ImportDataUseCase, FormulaParser, SettingsScreen
• File organization: Moved IBP reference files (PDF + 2 JPEGs) from repo root to docs/assets/
• Lint baseline: Configured lint-baseline.xml in build.gradle.kts — CI auto-generates baseline so only NEW lint issues are reported
• Migration tests: Added structural unit tests verifying all 14 Room migrations are present, contiguous (v1→v15), and correctly ordered
• Domain layer refactor: Moved MartelageModels.kt (533 lines of pure business logic) from presentation/screens/forestry/ to domain/calculation/MartelageStatsCalculator.kt — prevents loss during UI refactor
• Import cleanup: Removed unused Mic and Bluetooth icon imports from MartelageScreen
• Documentation: Added CONTRIBUTING.md with code conventions, project structure, PR process, and testing guidelines
• Repo cleanup: Removed committed build logs and crash logs, updated .gitignore

Not touched (preserved for user's planned refactor)

• Database schema
• Internet/sync mechanisms
• Analysis/correlation systems

Review & Testing Checklist for Human

• Verify permission checks work on device: Test GPS capture in IBP evaluation screen and GPS distance measure — confirm no SecurityException crash when permission is granted, and graceful handling when denied
• Verify BackupWorker still works: Trigger a backup and confirm it completes without DB errors in logcat
• Run unit tests locally: ./gradlew :app:testDebugUnitTest — confirm the new DatabaseMigrationsTest passes along with existing tests
• Check MartelageScreen still compiles and functions: The computeMartelageStats import path changed — verify martelage synthesis displays correctly
• Review CONTRIBUTING.md: Confirm conventions match your project standards

Notes

• The i18n hardcoded strings (mostly in IBP diagnostic/analysis screens) were deferred since those screens will be refactored. They are documented for the refonte.
• The lint baseline will be auto-generated on the first CI run since no Android SDK was available locally to generate it.

Link to Devin session: https://app.devin.ai/sessions/e5c5a376148a4413b8280d24b18a5175

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring