Ground YouTube comment replies in per-comment context - #1478
Draft
Foundup wants to merge 3 commits into
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
#FFCPLNcontext-sensitive instead of automatically treating it as a troll marker.resolve_general_model_path()instead of the coding-role model and passes embedded Qwen the same full grounded prompt as the service route.YT_ALLOW_GENERIC_REPLY_FALLBACK=true.Why
The active daemon showed Grok unavailable, LM Studio consuming most of the 10-second budget, embedded Qwen receiving a reduced context-free prompt, and generic templates such as “Appreciate the comment!” being posted and mislabeled as contextual output. Studio rows also lost their video context before generation.
OpenRouter / RedDog decision
RedDog's production Fusion calls are autonomous and do not require 012 to approve individual calls. They consume policy and signed runtime/model bindings and persist durable, content-free provider-call evidence.
This PR does not import the private
advisory_model_once.pybridge because the currently exposed production runners are typed for repository audits, backend-architect decisions, and bounded artifact generation—not short-form YouTube replies. The next RedDog slice should expose a provider-neutral reply runner that consumes preconfigured daemon policy and returns reply text plus a durable call receipt. 012 remains out of the per-comment loop.Validation
python -m pytest modules/communication/video_comments/tests/test_reply_context_pipeline.py modules/communication/video_comments/tests/test_ytr1_runtime_hardening.py modules/communication/video_comments/tests/test_classifier_pipeline.py -q— 23 passedpython -m py_compileon the three changed runtime modulesgit diff --check