Skip to content

feat: prepare mobileGF2logger v2.2.0 - #31

Merged
1window2 merged 5 commits into
mainfrom
temp
Aug 11, 2026
Merged

feat: prepare mobileGF2logger v2.2.0#31
1window2 merged 5 commits into
mainfrom
temp

Conversation

@1window2

@1window2 1window2 commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Supersedes #30 after its source branch was renamed to comply with the repository branch-naming rule. All commits and resolved Codex review fixes are preserved.

Summary

  • add tappable weekly evidence explanations, Evidence Health, and guided capture auto-stop
  • add bounded CSV import preview with automatic checkpoint/undo, plus privacy-controlled weekly PNG save/share
  • add encrypted, canonical Discord incoming-webhook delivery for parsed original CSV payloads
  • refactor weekly inference/state/snapshot boundaries and harden backup, import, parser, CI, and release workflows
  • include every dependency update from superseded PR chore(deps): Bump the "all-dependencies" group with 2 updates across multiple ecosystems #29 with immutable GitHub Action commit pins

Verification

  • gradlew clean :protocol:test :app:testDebugUnitTest :app:lintDebug :app:assembleDebug :app:assembleRelease
  • gradlew :app:connectedDebugAndroidTest - 36/36 passed on SM-N976N
  • real-device packet ingestion, guided capture, weekly navigation/evidence, PNG privacy sharing, CSV preview/checkpoint/undo, and Discord configured/unconfigured UI flows
  • v2.2.0 test-host cold launch: no crash or ANR
  • release APK SHA-256: 78D33A63CB8DDF8B93E3CBD6882805C78D0578296AB31C64DC7BE32260EBBEA9
  • APK Signature Scheme v2/v3 verified; established certificate SHA-256 3b6af9a1707675494efd042a17d723b23ecb96c9ff53db9255779cf5a5fcbf01

Copilot AI lite review requested due to automatic review settings August 11, 2026 12:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@1window2 1window2 self-assigned this Aug 11, 2026
@1window2

Copy link
Copy Markdown
Owner Author

@codex review

Please review exact head f5206a0. This replacement PR preserves PR #30's history after the source branch was renamed to temp. The five prior Codex findings were fixed, regression-tested, replied to, and resolved on #30; please perform a clean final review focused on CSV preview/commit equivalence, checkpoint crash recovery, weekly inference, sharing privacy, and Discord webhook boundaries.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f5206a06f8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/src/main/java/dev/gf2log/app/MainActivity.kt
@1window2

Copy link
Copy Markdown
Owner Author

@codex review

Please perform a final clean review of exact head 1a5e032. The P1 recovery-order finding is fixed, regression-tested on-device, replied to, and resolved. Verify there are no remaining actionable issues before squash merge.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1a5e032b32

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/src/main/java/dev/gf2log/app/WeeklyReportActivity.kt
@1window2

Copy link
Copy Markdown
Owner Author

@codex review

Please perform the final clean review of exact head 5ee8f1b. The pending-PNG recreation finding is fixed, covered by bounded-state unit tests and an on-device ActivityScenario recreation test, replied to, and resolved. Verify no actionable issues remain before squash merge.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: 5ee8f1b029

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@1window2 1window2 added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request dependencies Pull requests that update a dependency file labels Aug 11, 2026
@1window2
1window2 merged commit 0c23c34 into main Aug 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants