[codex] Split hive/runtime stabilization#228
Draft
jbcupps wants to merge 12 commits into
Draft
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Fix race condition in AbnormalBrainScreen where the onReady callback never fired because parent re-renders invalidated the useEffect timeout. Use useRef for the callback and useCallback in the parent. Auto-install Ollama when not found: start_managed_ollama now downloads and silently installs Ollama, starts the server, and pulls the bundled model (llama3.2:3b) in a background task instead of failing immediately. Add InstallingOllama lifecycle state so the frontend can show install progress (0-40%) distinct from model download progress (40-100%). Fix Windows file-handle contention: explicitly drop the tokio File before executing the downloaded installer. Include split-app MVP: per-entity provider config, streaming chat in entity-runtime-app, message-bus observability, and Hive provider UI. Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
…N env var These tests spawn hive-daemon and entity-daemon binaries which must be pre-built. The stability CI job handles this explicitly; the generic cargo test --workspace run skips them to avoid build-order races. Made-with: Cursor
Both hive-daemon and entity-daemon integration tests spawn daemon binaries that must be pre-built. Skip them in the generic workspace test run; the stability job builds binaries and sets the env var. Made-with: Cursor
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
2 participants
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
This PR establishes the split Abigail Hive + Abigail Entity Runtime architecture and makes the new Windows local-launch path workable on a managed machine.
hive-appandentity-runtime-appdesktop roots%LOCALAPPDATA%\Abigail\cargo-targetlaunch flowWhy
The old super-app path was carrying too much alpha-era coupling, and local Windows development was blocked by Application Control when Cargo executed generated build-script binaries from the repo-local target directory. This change turns the split architecture into a working local path and gives us both a desktop and browser fallback for pre-MVP validation.
Developer impact
pwsh ./scripts/dev/launch_split_stack.ps1for the split-stack local launch flowpwsh ./scripts/diagnose_windows_build_policy.ps1to capture JSON diagnostics when Windows policy blocks desktop builds%LOCALAPPDATA%\Abigail\cargo-targetValidation
cargo check -p abigail-hive-app -p abigail-entity-runtime-appcargo test -p hive-daemoncargo test -p entity-daemoncargo clippy -p entity-daemon -p hive-daemon -p abigail-hive-app -p abigail-entity-runtime-app --tests -- -D warningsnode scripts/check_unsigned_stabilization.mjspwsh -NoProfile -File scripts/diagnose_windows_build_policy.ps1pwsh -NoProfile -File scripts/dev/launch_split_stack.ps1 -OpenBrowser:$false