Initial Integration of CI Pipeline into Main for Further Testing

[polygeist111]

The purpose of this PR is to integrate the CI work from the Mesh Reliability & Structure Team into main. This is not expected to be the final commit/stable version of these features, but certain aspects require placement on the main branch for further testing.

The PR brings in files in a few different areas, but hinges on the iml based GitHub Actions workflows. The report outputs of many of these also feed a viewing site via github pages (e.g. https://grey-box.github.io/Project-Mesh/build_reports/index.html), so a large volume of automatically generated files (and a few critical manual ones) exist in the new Build Reports directory.

This pipeline can run APK builds, super-linter, in-depth kotlin linting, tests, test coverage, and documentation generation. It additionally submits to GitHub's dependency graph, and can be easily adapted to include future CI/CD tasks, including Play Store deployment. The linters, where possible, automatically enforce style and formatting consistency. They will flag errors when issues arise that they cannot automatically resolve.

As stated above, this is a working commit. A followup will be made to trim any nonessential files once testing confirms them as such.

This work is also responsible for the automated checks that now run alongside PRs :) My next task is to address as many of the linter issues as possible (and existing testing errors) so as to get much clearer feedback from the system on codebase quality.

[polygeist111]

I had forgotten my teammate's work hadn't been integrated to main yet either, so this PR will also introduce the unit tests they've written so far, as well as all the KDoc comments. The issues identified by gemini have to do with changes made by the test writing team, so I can discuss with them in the next few days how to resolve.

[github-actions]

Code Coverage

File	Coverage [94.39%]	✅
app/src/main/java/com/greybox/projectmesh/messaging/network/MessageNetworkHandler.kt	50.00%	❌
app/src/main/java/com/greybox/projectmesh/messaging/network/MessageService.kt	56.00%	❌
app/src/main/java/com/greybox/projectmesh/messaging/repository/ConversationRepository.kt	97.73%	✅
app/src/main/java/com/greybox/projectmesh/messaging/utils/MessageMigrationUtils.kt	90.00%	✅
app/src/main/java/com/greybox/projectmesh/testing/TestDeviceService.kt	4.76%	❌
app/src/main/java/com/greybox/projectmesh/user/UserRepository.kt	100.00%	✅
app/src/main/java/com/greybox/projectmesh/viewModel/SelectDestNodeScreenViewModel.kt	68.18%	❌
app/src/test/java/com/greybox/projectmesh/DeviceStatusManagerTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/debug/CrashHandlerTest.kt	96.00%	✅
app/src/test/java/com/greybox/projectmesh/extension/ContentResolverExtensionTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/data/dao/ConversationDaoTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/data/dao/MessageDaoTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/network/MessageNetworkHandlerTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/network/MessageServiceTest.kt	95.12%	✅
app/src/test/java/com/greybox/projectmesh/messaging/repository/ConversationRepositoryTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/repository/MessageRepositoryTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/messaging/utils/MessageMigrationUtilsTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/user/UserDaoTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/user/UserRepositoryTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/HomeScreenViewModelTest.kt	98.63%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/LogScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/NetworkScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/OnboardingViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/PingScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/ReceiveScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/SelectDestNodeScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/SendScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/SettingsScreenViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/viewModel/SharedUriViewModelTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/views/LogScreenUILogicTest.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/views/NetworkScreenUiLogic.kt	96.47%	✅
app/src/test/java/com/greybox/projectmesh/views/OnboardingUiLogic.kt	100.00%	✅
app/src/test/java/com/greybox/projectmesh/views/PingScreenUiLogic.kt	98.15%	✅
app/src/test/java/com/greybox/projectmesh/views/SelectDestNodeScreenUiLogic.kt	100.00%	✅

Total Project Coverage	17.86%	❌

[gemini-code-assist]

Code Review

This pull request significantly improves the project's robustness by introducing an extensive suite of unit tests for core components, including repositories, viewmodels, and utility classes. It also standardizes the codebase with KDoc documentation and integrates Kover and Dokka for coverage and documentation generation. Feedback identifies critical issues in FileEncoder.kt regarding potential OutOfMemoryError when reading large files and failures due to URL length limits when transmitting file content as query parameters. Additionally, the review points out redundant dependencies in build.gradle.kts, potential resource leaks due to unclosed streams, and risks associated with direct serialization of Throwable objects in the CrashHandler.

[gemini-code-assist]

The entire Base64-encoded file content is being passed as a query parameter in the URL. Most HTTP servers and Android's HttpURLConnection have strict limits on URL length (typically between 2KB and 8KB). This approach will fail for all but the smallest files. Additionally, sensitive or large data should always be sent in the request body, not the URL.

[gemini-code-assist]

Using readBytes() on an InputStream without a size limit reads the entire content into memory. This will cause an OutOfMemoryError if the file being encoded is large. For a mesh network application where files might be shared, it is better to process the stream in chunks or use a more memory-efficient encoding approach.

[gemini-code-assist]

The dependency junit:junit:4.13.2 is being added explicitly here, but it is already included via libs.junit on line 90. This is redundant and can lead to dependency resolution issues.

    testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.8.1")

[gemini-code-assist]

Serializing a Throwable object directly using Gson is generally discouraged. Throwable objects often contain circular references (e.g., through the cause field or suppressed exceptions) and platform-specific internal fields that can cause StackOverflowError or JsonIOException during serialization. It is safer to extract and serialize only the necessary information, such as the message and stack trace.

[gemini-code-assist]

The InputStream and OutputStream are not being closed reliably. If an exception occurs during the while loop (line 138), the close() calls at lines 140 and 141 will be skipped, leading to a resource leak. Use the .use { ... } extension function or a finally block to ensure resources are always released.