Skip to content

Reapply: real-time message writes to Honcho (re-land #42)#54

Draft
ajspig wants to merge 1 commit into
revert-42-realtime-writesfrom
reapply-42-realtime-writes
Draft

Reapply: real-time message writes to Honcho (re-land #42)#54
ajspig wants to merge 1 commit into
revert-42-realtime-writesfrom
reapply-42-realtime-writes

Conversation

@ajspig

@ajspig ajspig commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Re-applies the changes from #42, which were reverted in #53.

#42 made user prompts write to Honcho in real time on UserPromptSubmit (instead of queuing for the SessionEnd flush), and removed the local ~/.honcho/message-queue.jsonl queue. That was reverted in #53 because the real-time path is still blocked on something on the Honcho side.

This PR holds those changes so we can re-land them the moment that's unblocked — do not merge until the Honcho-side blocker is resolved.

Merge order

  1. Merge Revert "Simplifying Adding Messages to Honcho (#42)" #53 first (the revert).
  2. Then merge this. Its base is the revert-42-realtime-writes branch; once Revert "Simplifying Adding Messages to Honcho (#42)" #53 merges and that branch is deleted, GitHub will auto-retarget this PR's base to main.

Contents

bunx tsc --noEmit passes clean.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6fbed391-8cdf-4eb1-9e3a-4ed211dfbd12

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch reapply-42-realtime-writes

Comment @coderabbitai help to get the list of available commands and usage tips.

@ajspig ajspig marked this pull request as draft June 1, 2026 21:28
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