fix(deps): bump pinned ogar-vocab to include OdooPort/SmbPort/WoaPort — fixes Railway build#42
Conversation
…dooPort/SmbPort/WoaPort Root cause: q2's Cargo.lock pinned the four OGAR crates (ogar-vocab, ogar-class-view, ogar-ontology, ogar-adapter-surrealql) to OGAR commit `b6a12a6` (2026-06-21, PR quarto-dev#92 merge). The Railway docker build clones `AdaWorldAPI/lance-graph` HEAD fresh on every build (Dockerfile:55), and lance-graph-ogar's bridges (`bridges/odoo_bridge.rs`, `bridges/smb_bridge.rs`, `bridges/woa_bridge.rs`) import `ogar_vocab::ports::{OdooPort, SmbPort, WoaPort}`. Those three Port types landed in OGAR on 2026-06-21/22: - 9096081 feat(ports): add WoaPort + SmbPort — planner-times align with billable hours - 2fc96e5 feat(ogar-vocab): add OdooPort commerce-arm curator + ERP time bridge Both are AFTER b6a12a6 in the OGAR main timeline, so cargo would resolve the imports against a Port-less `ports.rs` and fail with: error[E0432]: unresolved import `ogar_vocab::ports::OdooPort` error[E0432]: unresolved import `ogar_vocab::ports::SmbPort` error[E0432]: unresolved import `ogar_vocab::ports::WoaPort` Fix: bump all four OGAR-crate pins in Cargo.lock from b6a12a6 → 302c28437c7621957acfd414b80925d9ebccee84 (current OGAR main, PR quarto-dev#110 merge). lance-graph itself pins to 08a9c979 (PR quarto-dev#94) which is the intermediate commit that first added OdooPort; bumping q2 to current main keeps q2 forward-compat with anything in lance-graph HEAD. Verified safe as a pure SHA bump: `git diff b6a12a6..302c284 -- crates/{ogar-vocab,ogar-class-view,ogar-ontology,ogar-adapter-surrealql}/Cargo.toml` returns EMPTY on all four — the dep graphs are byte-identical between the two commits, so the Cargo.lock dependency lines remain valid; only the `source = "git+...?branch=main#<SHA>"` line changes. No code changes, no Cargo.toml changes. Pure lock bump. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
The root /Dockerfile clones lance-graph@main at build time. Two upstream fixes have since landed that unblock the cockpit-server Rust build: - lance-graph #595: contract::ogar_codebook synced to OGAR quarto-dev#110 (Auth domain 0x0B, 43 concepts) — clears the COUNT_FUSE compile panic. - q2 #42: Cargo.lock ogar-vocab b6a12a6 -> 302c284 — resolves the missing OdooPort/SmbPort/WoaPort imports. Railway did not auto-rebuild (no q2 change since #42 merged, which predated #595), so this empty commit forces a fresh build that picks up the now-fixed lance-graph main and redeploys the live Rust stack (700-node OSINT scene).
Root cause (Railway build failure)
q2's
Cargo.lockpinned all four OGAR crates (ogar-vocab,ogar-class-view,ogar-ontology,ogar-adapter-surrealql) to OGAR commitb6a12a6(2026-06-21, PR #92 merge).The Railway docker build clones
AdaWorldAPI/lance-graphHEAD fresh on everybuild (
Dockerfile:55). lance-graph-ogar's bridges (bridges/odoo_bridge.rs,bridges/smb_bridge.rs,bridges/woa_bridge.rs) importogar_vocab::ports::{OdooPort, SmbPort, WoaPort}. Those three Port typeslanded in OGAR on 2026-06-21/22:
9096081— feat(ports): addWoaPort+SmbPort— planner-times align with billable hours2fc96e5— feat(ogar-vocab): addOdooPortcommerce-arm curator + ERP time bridgeBoth commits are AFTER
b6a12a6in the OGAR main timeline, so cargoresolved the imports against a Port-less
ports.rsand failed with:(Railway AI's diagnosis named "OGAR main is missing the types" — that was
the wrong axis. The types are present on current OGAR main; the issue was
q2's lock pin, not OGAR.)
Fix
Bump all four OGAR-crate pins in
Cargo.lockfromb6a12a6→302c28437c7621957acfd414b80925d9ebccee84(current OGARmain, PR #110 merge).
lance-graph itself pins to
08a9c979(PR quarto-dev#94 — the intermediate commit thatfirst added
OdooPort); bumping q2 to current OGAR main keeps q2 forward-compatible with anything lance-graph HEAD references.
Verified safe as a pure SHA bump
$ git -C /tmp/ogar diff b6a12a6..302c284 -- crates/{ogar-vocab,ogar-class-view,ogar-ontology,ogar-adapter-surrealql}/Cargo.toml # returns EMPTY — all four Cargo.tomls byte-identicalThe dep graphs are byte-identical between the two commits, so the
Cargo.lockdependency lines remain valid; only thesource = "git+...?branch=main#<SHA>"line changes (4 occurrences).Diff
Four lines in
Cargo.lock, one per package. Zero code changes, zero Cargo.tomlchanges.
Test plan
cargo check -p cockpit-server --features embed-cockpit,plannersucceedsDockerfile) succeedscargo update -p ogar-vocablocally if a different policy applies)
Related
AdaWorldAPI/q2#33(Phase 1 NARS+GraphSnapshot deletion) — still open since 2026-05-04, blocked on lance-graph#331, independent of this fix.OdooPortconsumer):AdaWorldAPI/odoo-rs#12-15 —
OdooPortis the cross-axis identity codebookfor the alignment table.