Skip to content

feat(leitner): return confirmed_chunk + source_sentence on review items for L3+ fill-in #86exnxph5#53

Merged
navidshad merged 4 commits into
devfrom
CU-86exnxph5_Update-get-review-bundle-return-confirmed_chunk-source_sentence-L3-fill-in_Navid-Shad
Jul 3, 2026
Merged

feat(leitner): return confirmed_chunk + source_sentence on review items for L3+ fill-in #86exnxph5#53
navidshad merged 4 commits into
devfrom
CU-86exnxph5_Update-get-review-bundle-return-confirmed_chunk-source_sentence-L3-fill-in_Navid-Shad

Conversation

@navidshad

@navidshad navidshad commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

📋 Summary

This PR enhances the Leitner review system by returning both the confirmed chunk and the source sentence for review items at level 3 and above that use fill-in-the-blank type questions.

🔗 Related Tasks

#86exnxph5 - Return confirmed chunk and source sentence on review items for L3+ fill-in

📝 Additional Details

This change improves the data returned during the review process, enabling better context and validation for advanced-level fill-in-the-blank exercises.

📜 Commit List

2856a3b feat(leitner): return confirmed_chunk + source_sentence on review items for L3+ fill-in #86exnxph5
06cb6e2 Merge pull request #51 from codebridger/dev
d1bcd99 Merge pull request #50 from codebridger/dev
ed3611a Merge pull request #48 from codebridger/dev

navidshad and others added 4 commits June 8, 2026 12:27
Text-only live sessions + Stripe-metadata-driven subscriptions (Council 004)
Fix chunks confusion on translation function #86exw91re
docs: add sibling repos (extension, pilotui) and cross-repo testing workflow
…ms for L3+ fill-in #86exnxph5

Surface two flat fields on every item returned by get-review-session (getDueItems)
and get-custom-review-session (getCustomReviewItems):

- confirmed_chunk: text of the phrase's primary chunk (highest confidence,
  tie-break earliest), or null when the phrase has no chunks so the renderer
  falls back to the recognition card.
- source_sentence: the phrase's context, kept whole (no AI-trim), or null.

Read-path only; no schema change. Ratified by Council 005 (primary-chunk rule).
Adds a ReviewItem type and unit tests covering highest-confidence selection,
tie-break, and the null-fallback cases.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@navidshad

Copy link
Copy Markdown
Contributor Author

@navidshad navidshad merged commit 19c52c9 into dev Jul 3, 2026
6 checks passed
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0-dev.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant