Skip to content

Friend-share: pre-generate derivative for crawl (feed) shares too - #696

Merged
mircealungu merged 1 commit into
masterfrom
wt/friend-share-original
Aug 1, 2026
Merged

Friend-share: pre-generate derivative for crawl (feed) shares too#696
mircealungu merged 1 commit into
masterfrom
wt/friend-share-original

Conversation

@mircealungu

Copy link
Copy Markdown
Member

Completes the multiplexer for the common case (sharing a feed article). When there's no upload, generate the recipient's copy by simplifying the canonical crawled article to their level — so the email deep-link opens a ready, adapted copy instead of prompting 'adapt to your level?'.

Same-language only for now; cross-language crawl translation is deferred (needs a same-language guard in the recommender overlay). Returns None (open-original-and-adapt fallback) for cross-language, when no simplification is needed, or on any error.

Needs a device test — the LLM generation only verifies live: share a feed article to a friend, confirm they open a pre-adapted copy with no level prompt.

🤖 Generated with Claude Code

…uage)

Completes the multiplexer for feed shares. When there's no upload, generate
the recipient's copy by simplifying the CANONICAL crawled article to their
level (create_recipient_derivative_from_article -> create_user_specific_
simplified_version, which makes a proper simplified child: parent_url,
is_simplified, target_cefr_level all correct). The email deep-link then opens
a ready, adapted copy — no 'adapt to your level?' prompt.

Cross-language crawl TRANSLATION is deferred (a translated child needs a
same-language guard in the recommender overlay first); it returns None there,
and the recipient opens the original and adapts, as before. Also None when no
simplification is needed (recipient's level >= the article's).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@mircealungu
mircealungu merged commit d447e2e into master Aug 1, 2026
1 check failed
@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown

ArchLens - No architecturally relevant changes to the existing views

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant