Skip to content

Rebase Re-Rethink fork onto upstream celzero/rethink-app#2813

Open
Mayanknishad9 wants to merge 1 commit into
celzero:mainfrom
Mayanknishad9:main
Open

Rebase Re-Rethink fork onto upstream celzero/rethink-app#2813
Mayanknishad9 wants to merge 1 commit into
celzero:mainfrom
Mayanknishad9:main

Conversation

@Mayanknishad9

@Mayanknishad9 Mayanknishad9 commented Jun 24, 2026

Copy link
Copy Markdown

Summary

  • Rebases the Re-Rethink fork (Material 3 Expressive / Jetpack Compose UI rewrite and related changes) onto latest celzero/rethink-app main
  • Where histories diverged: UI/adapters/themes prefer the fork; core VPN, firewall, database, and wireguard services prefer upstream
  • Includes a large squashed commit bringing the fork work current with upstream

Test plan

  • Build and install the app (full/play flavors as applicable)
  • Smoke-test main screens (home, firewall, DNS, logs, settings)
  • Verify VPN connect/disconnect and basic firewall/DNS behavior still works
  • Review UI regressions vs upstream where Compose/Material 3 changes apply

Squashed Material 3 Expressive / Jetpack Compose UI rewrite and related
fork changes onto latest upstream main. UI/adapters/themes prefer the
fork; core VPN, firewall, database, and wireguard services prefer
upstream where histories diverged.
val scope = rememberCoroutineScope()
val explanation =
rememberDnsStatusExplanation(
key = "${endpoint.url}:${endpoint.blocklistCount}",
val isLastInGroup = nextInitial == null || currentInitial != nextInitial

if (isFirstInGroup) {
stickyHeader(key = "proxy_header_$currentInitial") {
}
}

item(key = "proxy_app_${item.uid}_${item.packageName}") {
}

if (index == 0 || isFirstInGroup) {
stickyHeader(key = "header_$currentInitial") {
}
}

item(key = "app_${item.uid}_${item.packageName}") {
verticalArrangement = Arrangement.spacedBy(2.dp)
) {
if (isLoading) {
item(key = "logs_loading") {
LogsLoadingState()
}
} else if (isEmpty) {
item(key = "logs_empty") {
LogsEmptyState()
}
} else if (hasLoadError) {
item(key = "logs_load_error") {
@hussainmohd-a

Copy link
Copy Markdown
Collaborator

@Mayanknishad9, Thanks for the pull request!! I added two comments that caught my attention. I will get back to you after we wrap up the upcoming release.

@Mayanknishad9

Mayanknishad9 commented Jun 26, 2026

Copy link
Copy Markdown
Author

@Mayanknishad9, Thanks for the pull request!! I added two comments that caught my attention. I will get back to you after we wrap up the upcoming release.

@hussainmohd-a
Hope to hear from you soon

@hussainmohd-a

Copy link
Copy Markdown
Collaborator

@Mayanknishad9, Thanks for the pull request!! I added two comments that caught my attention. I will get back to you after we wrap up the upcoming release.

@hussainmohd-a Hope to hear from you soon

Meanwhile, could you please address the MobSF scan findings, build failures, and the review comments? Also, just curious which AI agent(s) did you use for reviewing/coding for this PR.

@Mayanknishad9

Copy link
Copy Markdown
Author

@hussainmohd-a
So i am in xAI as an Software engineer expert we are testing our grok Cli which is under development. So not available for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants