SF-3704 Do not include deleted books in project progress

[pmachapman]

This PR excludes text documents that do not have an ops array from the project progress query.

---

---

This change is 

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.30%. Comparing base (4bcfdeb) to head (3297fcb).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
.../SIL.XForge.Scripture/Services/SFProjectService.cs	0.00%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3716      +/-   ##
==========================================
- Coverage   81.32%   81.30%   -0.02%     
==========================================
  Files         620      620              
  Lines       39026    39032       +6     
  Branches     6341     6341              
==========================================
  Hits        31736    31736              
- Misses       6320     6326       +6     
  Partials      970      970              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Nateowami]

@Nateowami reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on pmachapman).

---

src/SIL.XForge.Scripture/Services/SFProjectService.cs line 1886 at r1 (raw file):

        BsonDocument verseSegmentOpsFilterExpression = new BsonDocument
        {
            { "input", new BsonDocument("$ifNull", new BsonArray { "$ops", new BsonArray() }) },

These changes look good, but I think we can remove both instances of $ifNull now that we're filtering out documents with no ops earlier in the process.

[pmachapman]

@pmachapman made 1 comment and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on pmachapman).

---

src/SIL.XForge.Scripture/Services/SFProjectService.cs line 1886 at r1 (raw file):

Previously, Nateowami wrote…

These changes look good, but I think we can remove both instances of $ifNull now that we're filtering out documents with no ops earlier in the process.

Done. Good idea.

[Nateowami]

@Nateowami reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on pmachapman).