feat(graph): persist exploration state and update navigation flow (#88)

[komodgn]

• Implement rememberRetained to preserve selected images during exploration
• Change navigation destination to PhotoDetailScreen on image click

Summary by CodeRabbit

• Improvements
  • Enhanced state persistence to preserve user data during app configuration changes
  • Updated image selection flow to navigate to the correct detail screen

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Verification

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR modifies GraphPresenter.kt to replace local state storage from remember to rememberRetained for webViewUrl and selectedImages, and updates the image click navigation target from GraphDetailScreen to PhotoDetailScreen. No public APIs or external contracts are altered.

Changes

Cohort / File(s)	Summary
State Retention & Navigation Update feature/graph/src/main/java/com/metasearch/android/feature/graph/GraphPresenter.kt	Changes state retention semantics from remember to rememberRetained for webViewUrl and selectedImages; switches navigation target on image click from GraphDetailScreen to PhotoDetailScreen; adds corresponding imports.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• AI 기반 이미지 설명 기능을 추가한다 #73: Extends and modifies GraphPresenter alongside adding/enhancing PhotoDetail screen and presenter implementations, directly aligned with this PR's navigation target changes.

Poem

🐰 State now remembers across the hop,
While photos dance to a new routed top,
No screens forgotten in the retained embrace,
Navigation flows to a brighter place! 📸

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: replacing remember with rememberRetained for state persistence and updating navigation from GraphDetailScreen to PhotoDetailScreen.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🤖 Android CI Summary

Step Results:

• Unit Test: ✅ Success (2m 53s)
• Debug Build: ✅ Success (1m 14s)
• Code Style Check: ✅ Success (3m 52s)

Total Time: 7m 59s

🎉 All steps completed successfully!