Conversation
The thread kept ending in "I couldn't get a response... try again" with no
summary, and every fresh start crashed. Three independent causes:
- master_ai.py: the malformed-directive detector only recognized the
<tool_call> wrapper shape, so a colonless directive glued into prose
("Let me look. RUN find ...</arg_value>") registered as neither a
directive nor a repair trigger. The command silently never ran and the
model's leaked XML fragments were rendered to the user as the answer.
Reuse _ARG_XML_TAG_RE as a second detector so these route to the
existing repair-retry instead of silent passthrough.
- sensei_tui.py: main()'s loop calls _SENSEI_APP.set_chat_id() every turn
but SenseiApp never defined it, so each start died on AttributeError and
the supervisor respawned into the same crash every ~5s. Add the method,
mirroring set_label.
- harvest.py: the privacy fence blanket-blocked ~/Desktop and ~/Documents
by path, dead-ending any cloud turn that merely globbed through them.
Narrow it to Pictures/Downloads/jobseeker; _PRIVATE_TERM_RE still does
content-based detection wherever the content actually lives.
Co-Authored-By: Claude Opus 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_015KpMtUUYRvv5vxQo5Qgino
…tension) Preserving in-progress work from this clone before consolidating the two diverged working copies (~/scripts and ~/master-ai-cli) onto one source of truth. Committed as-is, unreviewed, following this repo's existing WIP convention. Includes: TinyFish + Telegram clients and their command-menu entries, the finished SenseiApp.set_chat_id + chat-id renderer, sessions browse/resume entries, sensei_extension updates, stt_server/setup_wizard/typed_actions changes, and the Pupil Reentry Desk panel wired to the reentry-desk HTTP bridge on :8091. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_015KpMtUUYRvv5vxQo5Qgino
~/scripts and ~/master-ai-cli are clones of the same repo that drifted apart, with canonical location differing per file (some real files, some symlinks). Tonight that drift caused a hard crash loop: SenseiApp.set_chat_id was fully implemented in the master-ai-cli clone while its only caller lives in master_ai.main() in the ~/scripts copy that actually runs, so every turn raised AttributeError and the supervisor respawned into the same crash. Merged so caller and callee live in one tree. Conflict resolutions: - sensei_tui.py: took the clone's real set_chat_id (typed + normalized) over the stub written tonight; its _chat_id init and renderer come with it. - sensei_extension/service_worker.js: comment-only conflicts, kept the fuller version carrying the 2026-09-06 live-test correction. - pupil.html, stt_server.py: kept as symlinks into master-ai-cli, which is where their current content already lives. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_015KpMtUUYRvv5vxQo5Qgino
|
Warning Review limit reachedNext included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (18)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Owner
Author
|
@coderabbitai review |
|
Owner
Author
|
@coderabbitai review |
|
Owner
Author
📋 Dashboard HTML Audit (CodeRabbit Review)SummaryReviewed What’s in it:
Key Notes:
Reviewed by Hermes Agent via CodeRabbit |
Owner
Author
|
@coderabbitai review |
|
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
1 participant
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.
Bug fixes from tonight's session — see diff for details.
Summary by CodeRabbit
New Features
Bug Fixes & Safety