gabriel / musehub public
fix patch wire-push feat/fix-object-refs-on-repush #4 / 4
AI Agent gabriel · 5 days ago · Jun 8, 2026 · Diff

fix(wire-push): write object_refs and mpack_index for all mpack objects on repush

Bug A: _upsert_object_refs was called with _new_oids (globally-new only), so repos pushing pre-existing objects got zero object_refs rows. Fix: call with _cc_oids (all objects in the mpack) so every repo gets its own object_refs rows regardless of whether objects are globally new.

Bug B: inline mpack_index byte-range inserts had the same _new_oids filter and used session.add_all which has no ON CONFLICT clause. Fix: use _pg_insert with ON CONFLICT DO UPDATE (byte_offset IS NULL guard) covering all _cc_oids so byte ranges are written even on repush.

Also repairs a stale orphaned list-comprehension fragment on line 549 that caused an IndentationError and prevented the module from loading.

ORP-1 through ORP-5 all green.

sha256:8ba2515ec3cb2a4089d54422f342b5a47161df29d5db72cb7356591d128fba6a sha
sha256:94ef169c149a452bff7c604ded8b280b19bd477c2dabcb56972780b0b784c7aa parent
~1 symbols
1 changed · 966 in snapshot files
sha256:194f41a1ddfb02d4692307c260a4ab534afacc5a5bf65699d05337d5ddb67cd9 snapshot
~1
symbol modified
1
file changed
966
files in snapshot
0
dead code introduced
Semantic Changes 1 symbol
+ .vscode/
+ alembic/
+ alembic/versions/
+ deploy/
+ deploy/load-tests/
+ docs/
+ docs/guides/
+ docs/legal/
+ docs/reference/
+ docs/roadmaps/
+ musehub/
+ musehub/api/
+ musehub/api/routes/
+ musehub/api/routes/api/
+ musehub/api/routes/musehub/
+ musehub/auth/
+ musehub/core/
+ musehub/crypto/
+ musehub/db/
+ musehub/debug/
+ musehub/graph/
+ musehub/maintenance/
+ musehub/mcp/
+ musehub/mcp/tools/
+ musehub/mcp/write_tools/
+ musehub/middleware/
+ musehub/models/
+ musehub/muse_cli/
+ musehub/proposals/
+ musehub/protocol/
+ musehub/security/
+ musehub/services/
+ musehub/storage/
+ musehub/templates/
+ musehub/templates/mcp/
+ musehub/templates/musehub/
+ musehub/templates/musehub/fragments/
+ musehub/templates/musehub/macros/
+ musehub/templates/musehub/pages/
+ musehub/templates/musehub/partials/
+ musehub/templates/musehub/static/
+ musehub/templates/musehub/static/vendor/
+ musehub/types/
+ scripts/
+ src/
+ src/scss/
+ src/scss/components/
+ src/scss/pages/
+ src/scss/patterns/
+ src/scss/theme/
+ src/ts/
+ src/ts/pages/
+ tests/
+ tests/unit/
+ tools/
+ workflows/
~ musehub/services/musehub_wire_push.py .py 1 symbol modified
~ wire_push_unpack_mpack
Files Changed
~1
966 in snapshot
~ musehub/services/musehub_wire_push.py .py
← Older Oldest on feat/fix-object-refs-on-repush
All commits Newer test(mpack-index-job): Phase 3 failing test for Bug D — job neve… sha256:d8a98ffcade6226a977047fb68396a940db91fa1a2f6590b0e55bb1a0f0d2735

0 comments

No comments yet. Be the first to start the discussion.

To add a comment, use the Muse CLI: muse hub commit comment sha256:8ba2515ec3cb2a4089d54422f342b5a47161df29d5db72cb7356591d128fba6a --body "your comment"