Skip to content

deps(od-ontology): bump ogar-vocab + ogar-adapter-surrealql pin → OGAR main (post-#111)#17

Merged
AdaWorldAPI merged 1 commit into
mainfrom
claude/bump-ogar-pin-to-include-product-accounting-account
Jun 24, 2026
Merged

deps(od-ontology): bump ogar-vocab + ogar-adapter-surrealql pin → OGAR main (post-#111)#17
AdaWorldAPI merged 1 commit into
mainfrom
claude/bump-ogar-pin-to-include-product-accounting-account

Conversation

@AdaWorldAPI

Copy link
Copy Markdown
Owner

What

Bumps the ogar-vocab + ogar-adapter-surrealql pin in
crates/od-ontology/Cargo.toml from 7e67718 (OGAR PR #98 merge,
2026-06-22) to ac4b4162 (current OGAR main, post-PR #111 merge,
2026-06-23).

Why — three OGAR PRs missed between the old pin and current main

OGAR PR Adds Why odoo-rs needs it
#94 OdooPort OdooPort, SmbPort, WoaPort structs + ODOO_ALIASES The Port types I leaned on in PR #14's cross-axis identity check. Currently un-exposed at the pinned commit, so test seeded_classes_have_compatible_ogar_identity (under ogar-emit feature) silently can't import them.
#110 AuthStore ConceptDomain::Auth + 4 auth_* class_ids Future-proof: if od-ontology ever materialises auth-scoped classes, the codebook is ready.
#111 PRODUCT + ACCOUNTING_ACCOUNT PRODUCT 0x0207, ACCOUNTING_ACCOUNT 0x0208 + 4 new ODOO_ALIASES (product.template, product.product, account.account, account.account.template) Direct closure of odoo-rs PR #14's "11 missing OdooPort aliases" finding — 4 of those 11 are now canonical and OdooPort::class_id("product.template") returns Some(0x0207). The cross-axis identity check this crate's tests/alignment_pin.rs runs becomes tighter.

What this is NOT

  • Not a V3 NodeGuid migration. od-ontology emits SurrealQL DDL via
    ogar-adapter-surrealql's formatter; it doesn't materialise
    NodeRows. The V3 (part_of:is_a) tile-bit packing in
    lance_graph_contract::canonical_node would apply if/when
    od-ontology grows a runtime-record path — that's a separate PR
    scoped per the §2.4 generic recipe in
    q2 fma docs/OGAR_CONSUMER_INTEGRATION.md.
  • Not breaking-shape. The Cargo.toml deps for ogar-vocab /
    ogar-adapter-surrealql are byte-identical between 7e67718 and
    ac4b4162 (verified separately), so this is a pure SHA bump — no
    feature-flag changes, no API renames.

Test plan

  • cargo test -p od-ontology --features ogar-emit (the OdooPort-using path) should now resolve product.template / account.account to canonical class_ids in the cross-axis pin check. Informational eprintln should report 7 remaining un-mapped seeded classes (was 11).
  • cargo test -p od-ontology (default features) — no change expected.

Refs

…18 → ac4b416 (OGAR main, +PRODUCT/ACCOUNTING_ACCOUNT/Auth)
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@AdaWorldAPI AdaWorldAPI merged commit 2ece3b7 into main Jun 24, 2026
AdaWorldAPI pushed a commit that referenced this pull request Jul 7, 2026
All five lenses reported; 3-meta synthesis consolidated:
R1 HOLD->RESOLVED (5b45170) · R2 CATCH-LATENT x2 (queued) · R3 TARGETS-CORE
+ triple.rs VIOLATION (Q1, THE next refactor) · R4 3 OPPORTUNITY / 3
WORTH-EXPLORING / 0 DROP · R5 OVERCLAIMED -> core gap RESOLVED same-hour
(real 55-file account addon compiled + verbatim account_move.py fused).

One reviewer premise corrected by synthesis: L6 SPOG IS ruled (operator
2026-07-06); what is missing is the lance-graph ledger paperwork + the
attribute->byte-position assignor, not the ruling.

Queue Q1-Q9 with sizes + owners recorded in docs/COUNCIL-5+3-2026-07-07.md:
triple.rs upstream migration, group_functions EmittedBy arm (smallest
highest-leverage ruff diff), ledger appends x3 repos, od-posting
re-architecture off SurrealQL vocabulary, mint relocation + odoo_actions
authority table, guard-classifier convergence, 5.1%-tail ownership,
carrier asks (#11/#15/#17 + stored verify).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant