fix: native compress telemetry crashes with UnboundLocalError - #68
Draft
cursor[bot] wants to merge 13 commits into
Draft
fix: native compress telemetry crashes with UnboundLocalError#68cursor[bot] wants to merge 13 commits into
cursor[bot] wants to merge 13 commits into
Conversation
Tier 1: Add uv.lock, pre-commit, Dependabot, ruff config, Python 3.13 CI, and fix HLC preservation on snapshot restore and gossip replay. Tier 2: Add server/mcp/agents/http optional extras, long-context compression eval script, and CI smoke jobs for MCP and long-context workloads. Tier 3: Wire HIVE_BACKEND in HiveStack, implement LinUCB policy updater, and add httpx async LLM client support. Tier 4: Refresh enterprise roadmap and improvement plan, add pip-audit/SBOM CI jobs, bump Jetson Docker base image, and enable nightly GPU smoke CI. Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Reflect PR #62 changes: new optional extras, HIVE_BACKEND, LinUCB, MCP/server deployment, 200-test CI matrix, uv.lock/pre-commit, and upcoming PFN busyBee training-mode integration. Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
- README outcome table: HLC fix, MCP, long-context eval, HIVE_BACKEND, LinUCB - New docs/WHATS_NEW.md with tier summary and Twitter-ready copy - CHANGELOG [Unreleased] docs entry; dedupe [full] install block in README Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
- Rewrite hive.mcp_server for MCP 2.x MCPServer API (hive-mcp console script) - Add client config generator, installer, and project .cursor/mcp.json - Document setup in docs/MCP_SETUP.md and harness wiring in docs/HARNESS_SETUP.md - Add hive.harness.load_routing_policy() for SWE-bench eval (busyBee or rules) - Add integration tests; 214 tests passing Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
- Add python -m hive.mcp (server default, install subcommand) - Test MCP list_tools/call_tool over stdio subprocess - Validate bundled Claude/Codex snippets against config generator - Run MCP integration tests in CI; update docs to 218 tests Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Explain that HLC tracks event order and cause-and-effect, not just wall-clock time, so memory stays correctly ordered when messages arrive late or out of sequence. Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
HiveStack.compress() native path built `result` but telemetry still referenced Python-only `out`, causing UnboundLocalError when backend=native, an explicit rust_brain, and telemetry are enabled. Record compression metrics from `result` and add a regression test. Co-authored-by: Daniel <DJLougen@users.noreply.github.com>
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 and impact
HiveStack.compress()crashes withUnboundLocalErrorwhen all of the following are true:backend="native"(orHIVE_BACKEND=nativewith hive-cpp installed)rust_brainis passed (so_nativestays enabled)This breaks native-backend deployments that record compression metrics — a common production configuration.
Root cause
The native compress path builds a
resultCompressedTurn, but telemetry recording still referenced the Python-onlyoutvariable from the non-native branch.Fix
Record compression telemetry from
resultfields instead ofout.Validation
test_native_compress_with_telemetry_does_not_crashpytest tests/test_stack.py tests/test_backend.py— 17 passed, 1 skipped