[AdaWorldAPI]

Summary

Follow-up to #122 (merged) — closes the gap that PR deliberately left open: the map is now actually served from lance-graph storage, not just converted into it.

• osm_lance::locate_row_column() productionizes the soa_to_lance "P-CACHE-4" mechanism into a boot-time resolver: searches the first few megabytes of each Lance data file for the slab's own row-0 bytes (row 0 is never gigabytes in — never reads a multi-GiB file whole), and reports the exact (offset, length) of the verbatim row column within it.
• osm_features::open_slab() now prefers OSM_SLAB_ROW_FILE (set at boot once the resolver succeeds) over OSM_SLAB_PATH for the row-byte mmap, bounded to exactly OSM_SLAB_ROW_OFFSET..+OSM_SLAB_ROW_LEN — required because the Lance data file carries its own header/footer bytes around the row run, which RowSlab::new would otherwise reject outright (it demands an exact multiple of NODE_ROW_STRIDE).
• OSM_SLAB_PATH is never repointed — it still anchors the .books/.chains sidecar paths, which live beside the raw bake, not inside the Lance dataset's data/ directory.
• Fail-closed throughout: a missing dataset, an unlocatable row column, or any doubt at all leaves OSM_SLAB_ROW_FILE unset and the raw .soa file keeps serving the map exactly as before.

Branch note: this session's branch was reset onto fresh main after #122 merged mid-session (per this repo's merged-branch policy) — only this one commit is new.

Test plan

• cargo check -p cockpit-server — clean
• cargo clippy -p cockpit-server --bins — zero warnings in the new code (same pre-existing warnings as before, unrelated)
• cargo test -p cockpit-server --bin q2-cockpit osm_lance — could not be executed this session. The monolithic test-binary link repeatedly crashed the linker (SIGBUS, ld terminated with signal 7) under this environment's disk pressure, even after aggressive cleanup (freed target/debug/incremental multiple times). cargo check confirms the new code type-checks correctly, including the async lance API usage; the 4 new tests (search_head_for_probe_finds_the_exact_offset, search_head_for_probe_declines_when_the_probe_is_absent, locate_row_column_finds_the_fragment_and_reports_the_slabs_own_length, locate_row_column_declines_when_no_fragment_matches) are straightforward tempfile-based fixtures with no exotic dependencies, but they have genuinely not been run — flagging honestly rather than claiming a pass I didn't observe.

---

Generated by Claude Code

[coderabbitai]

Important

Review skipped

Draft detected.

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 Plus

Run ID: 283d71ac-bea5-4532-97ed-ae8884c21781

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

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[cursor]

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_8c2708b1-74cd-48d3-8040-5dc13a6b7c77)