fix(rust): repair fresh dependency resolution (#137)

[konard]

🤖 Решение

Исправляет xlabtg/TONAIAgent#453 (LOGIC-43) — read-локи в shared-memory перезаписывали друг друга.

📋 Проблема

Локи хранились в this.locks: Map<key, MemoryLock> — не более одной записи на ключ. Read-локи должны разделяться несколькими держателями, но приобретение второго read-лока просто перезаписывало запись в map новым держателем. Запись первого читателя терялась: при вызове releaseLock его holderId уже не совпадал и release возвращал false, а write-лок мог быть взят после истечения TTL последнего читателя, даже если ранние читатели ещё активны — это нарушало гарантию взаимного исключения read/write.

🔧 Решение

В core/multi-agent/memory/shared-memory.ts хранилище локов заменено на структуру с раздельным учётом:

interface LockEntry {
  write?: MemoryLock;                 // эксклюзивный write-лок
  readers: Map<string, MemoryLock>;   // конкурентные read-локи по holderId
}
private locks: Map<string, LockEntry> = new Map();

• acquireLock: write-лок выдаётся только при отсутствии активного write и активных читателей; read-лок блокируется лишь активным write-локом, несколько читателей сосуществуют.
• releaseLock: освобождает write- или read-лок именно вызвавшего держателя; запись удаляется, когда локов не осталось.
• Добавлен приватный helper pruneLockEntry, который чистит истёкшие write/read-локи и используется как в acquireLock, так и в cleanup.
• set и getStats обновлены под новую структуру (проверка write-лока и подсчёт активных локов с учётом читателей).

✅ Соответствие критериям приёмки

• Несколько одновременных держателей read-лока учитываются независимо; каждый может освободить свой лок.
• Write-лок выдаётся только при отсутствии активных читателей.
• Регрессионный тест берёт два read-лока, освобождает один и проверяет, что второй ещё держится.

🧪 Тесты

Добавлены тесты в tests/multi-agent/multi-agent.test.ts:

• одновременные read-локи нескольких держателей;
• освобождение одного read-лока без влияния на остальных;
• выдача write-лока только после освобождения всех читателей;
• блокировка read-локов при активном write-локе.

Локально: vitest run tests/multi-agent/multi-agent.test.ts — 52 passed; tsc --noEmit — без ошибок; eslint — чисто.

JS/Rust parity

The repo enforces JS↔Rust source/test parity. The same html-to-markdown 3.6
{ width, height } dimensions shape applies to the JS binding
(@kreuzberg/html-to-markdown-node, ^3.5.5 already resolves into the 3.6
line), so the Rust regression is mirrored on the JS side: normalizeStructuredKeys
is exported from js/src/kreuzberg.js and js/tests/unit/kreuzberg.test.js
gains tests asserting inline image dimensions keep { width, height }. A
changeset is included for the JS package.

CI

All workflows green on the latest commit: Rust Checks and Release,
JavaScript Checks and Release, and JS/Rust Parity.

Fixes #137

[konard]

Working session summary

Everything is complete and verified. The .gitkeep is an auto-generated placeholder from the task-setup commit (bc9dc2e), not my change — it's harmless and outside any code/parity path.

Summary

Issue #137 is fully resolved and PR #138 is ready for review with all CI green.

Root causes fixed (fresh cargo add web-capture + cargo check failures):

1. html-to-markdown-rs 3.6 API break — InlineImage.dimensions became a structured ImageDimensions { width, height } instead of a (u32, u32) tuple. Adapted rust/src/kreuzberg.rs and bumped the dep to 3.6.
2. Broken brotli 8.0.3 graph (alloc-no-stdlib E0277 conflict) — dropped the unused brotli reqwest decompression feature (kept gzip), removing the conflicting crate entirely.
3. time/cookie E0119 — resolved naturally by the latest time 0.3.49 in a fresh lockfile.

Per maintainer feedback: bumped rust-version to the latest stable 1.96 (confirmed no newer stable exists) and satisfied the now-stricter MSRV clippy lint via Duration::from_mins(1).

Regression prevention: added a CI Fresh Dependency Resolution job (rm Cargo.lock && cargo generate-lockfile && cargo check/test) that gates the build/release, catching registry drift before publish.

JS/Rust parity: mirrored the dimensions regression on the JS side (exported normalizeStructuredKeys, added tests, included a changeset), since the parity guard requires synchronized changes and @kreuzberg/html-to-markdown-node ^3.5.5 already resolves into the same 3.6 line.

Verification:

• Local: cargo fmt --check, cargo clippy --all-targets --all-features -D warnings, cargo test, fresh-resolution simulation, and JS jest (19/19) all pass.
• CI on 8cee5e4: Rust Checks and Release ✅, JavaScript Checks and Release ✅, JS/Rust Parity ✅.
• Version bumped 0.3.30 → 0.3.31 with CHANGELOG entry; PR marked ready, MERGEABLE, 0 commits behind main.

PR: #138

---

This summary was automatically extracted from the AI working session output.

[konard]

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

• Public pricing estimate: $8.220738
• Calculated by Anthropic: $8.894096
• Difference: $0.673357 (+8.19%)

📊 Context and tokens usage:

Claude Opus 4.8: (2 sub-sessions)

1. 114.8K / 1M (11%) input tokens, 44.3K / 128K (35%) output tokens
2. 89.4K / 1M (9%) input tokens, 21.9K / 128K (17%) output tokens

Total: (25.3K new + 181.5K cache writes + 10.3M cache reads) input tokens, 72.7K output tokens, $8.220738 cost

🤖 Models used:

• Tool: Anthropic Claude Code
• Requested: opus
• Model: Claude Opus 4.8 (claude-opus-4-8)

📎 Log file uploaded as Gist (3809KB)

• View complete solution draft log

---

Now working session is ended, feel free to review and add any feedback on the solution draft.

[konard]

✅ Ready to merge

This pull request is now ready to be merged:

• All CI checks have passed
• No merge conflicts
• No pending changes

---

Monitored by hive-mind with --auto-restart-until-mergeable flag