Skip to content

Conversation

@marcosholgado
Copy link
Contributor

@marcosholgado marcosholgado commented Nov 19, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/1125189844152671/task/1211661961548831?focus=true

Description

This PR prepares duck ai to handle the migration to the new domain via JS message handlers.

Steps to test this PR

  • Use Duck.ai as normal, adding some chats, upload images, etc
  • Settings → Feature Flag Inventory → Enabled standaloneMigration
  • Close and open the app
  • Go to duck ai
  • You should see a popup asking to refresh, click there
  • Eventually you should see a popup saying "success"
  • You should be taken to duck ai
  • Your chats, history, etc should all be there.
  • Click the back button or back arrow
  • Duck ai WebView should close
  • Close and reopen the app
  • Go to duck.ai you should not see the refresh popup again
  • To re-test, clear all the cache and data.

Copy link
Contributor Author

marcosholgado commented Nov 19, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch 2 times, most recently from fbd8f97 to 38045a5 Compare November 20, 2025 09:55
@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch 2 times, most recently from b3bb53e to 3a18af1 Compare November 20, 2025 10:59
@ContributesBinding(AppScope::class)
@SingleInstanceIn(AppScope::class)
class InMemoryStandaloneStore @Inject constructor() : StandaloneDuckChatStore {
private val migrationItems = mutableListOf<String>()
Copy link
Contributor

Choose a reason for hiding this comment

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

I still have a concern that this could cause issues but we can deactivate the feature if we see an uptick in OOMs

@joshliebe
Copy link
Contributor

As stated here, this has an issue where it gets stuck in an "Update complete!” loop.

Once that is fixed, I will re-check.

@marcosholgado marcosholgado force-pushed the feature/marcos/duckai_migration branch from 3a18af1 to 10ec9f3 Compare November 25, 2025 10:04
Copy link
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

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

Performance is still slow for larger / many images but the previously mentioned issue is fixed.

@marcosholgado marcosholgado merged commit 2370946 into develop Nov 26, 2025
12 checks passed
@marcosholgado marcosholgado deleted the feature/marcos/duckai_migration branch November 26, 2025 13:57
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.

4 participants