Skip to content

feat: checkpoint PhotoKit duplicate inventory - #321

Open
seonghobae wants to merge 8 commits into
feat/apple-photos-duplicate-library-v1from
feat/photokit-checkpoint-audit-v1
Open

feat: checkpoint PhotoKit duplicate inventory#321
seonghobae wants to merge 8 commits into
feat/apple-photos-duplicate-library-v1from
feat/photokit-checkpoint-audit-v1

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Outcome

  • audits one PhotoKit asset per native completion boundary and records measured page duration
  • validates gap-free, stable-total checkpoints in Rust and only marks an exact-total audit complete
  • renders progress, supports cancellation between assets, and resumes the accepted read-only checkpoint
  • keeps network access disabled and adds no Photos mutation authority

Validation

  • cargo test --manifest-path src-tauri/Cargo.toml photos_library::tests:: --lib (9 passed)
  • npm test -- --run src/lib/photosLibraryState.test.ts (2 passed)
  • npm run build
  • git diff --check

Stacked on #317. No live Photos mutation was performed.


Devin Review

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 23911cff-8858-4486-bf36-ba0cb21640b3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Comment @coderabbitai help to get the list of available commands.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

…pr321-sync

# Conflicts:
#	src-tauri/src/container_orphan_reclaim.rs
@seonghobae seonghobae added enhancement New feature or request priority: medium Normal-priority or P2 work labels Sep 2, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request priority: medium Normal-priority or P2 work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant