Skip to content

Add duplicate detection using file hash and source ref - #48

Open
prachishelke1312 wants to merge 5 commits into
madhav2348:developfrom
prachishelke1312:fix-duplicate-detection-clean
Open

Add duplicate detection using file hash and source ref#48
prachishelke1312 wants to merge 5 commits into
madhav2348:developfrom
prachishelke1312:fix-duplicate-detection-clean

Conversation

@prachishelke1312

Copy link
Copy Markdown
Contributor

Summary

Implemented duplicate detection support for screenshot ingestion.

Changes

  • Added fileHash to ScreenshotInput
  • Added repository methods for hash and source reference lookup
  • Added file_hash storage support in SQLite repository
  • Added duplicate detection checks before screenshot ingestion
  • Updated ingestion services to include fileHash in payloads

Fixes #24

@github-actions

Copy link
Copy Markdown

🎉 Thanks for submitting a PR, @prachishelke1312!

Please confirm the following checklist before review:

  • ⭐ I have starred this repository
  • 🍴 I have forked this repository
  • 📖 I have read the Contributing Guidelines
  • 🔀 My changes are pushed to the develop branch

A maintainer will review your PR shortly. Thank you! 🚀

@prachishelke1312

Copy link
Copy Markdown
Contributor Author

Hi @madhav2348,

The merge conflicts have been resolved and the PR is now up to date with the develop branch. The remaining workflow appears to be awaiting maintainer approval. Could you please review the PR and approve the workflow when you have a chance?

Thank you!

@prachishelke1312

Copy link
Copy Markdown
Contributor Author

@madhav2348 ???

@madhav2348

Copy link
Copy Markdown
Owner

Please resolve merge conflict

@prachishelke1312

Copy link
Copy Markdown
Contributor Author

Done @madhav2348

@prachishelke1312

Copy link
Copy Markdown
Contributor Author

@madhav2348 anything else is needed?

@madhav2348

madhav2348 commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Okay , so all i could see in files changed are , screenshot meta data and ss.ai image files , are you sure about your PR and commits @prachishelke1312
Because we need to gitignore t and remove those files

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants