fix(contract+ogar+lock): mirror OGAR #127 (HR domain 0x0DXX) — closes 11-of-11 cross-axis identity gap#608
Conversation
… 11-of-11 cross-axis identity gap D4 — final mirror update for the cross-axis identity arc started by odoo-rs PR #14. Closes the last 4 of the 11-class gap (HR cluster) just minted in OGAR #127. Three coordinated fixes (same shape as D2 #597 and D3 #606): 1. lance-graph-contract: extend `ogar_codebook::CODEBOOK` with the 4 new HR entries (hr_employee 0x0D01, hr_department 0x0D02, hr_job 0x0D03, hr_employment_contract 0x0D04). Add `ConceptDomain::HR` variant. Add `0x0D → HR` arm to canonical_concept_domain. Update Unassigned doc from `0x0DXX+` to `0x0EXX+`. 2. lance-graph-ogar: parity::domains_agree adds the (O::HR, C::HR) arm. Same shape as the Anatomy + Automation arms I added in D3 #606. The E-OGAR-AUTH-MIRROR-DRIFT discipline holds: every new ConceptDomain variant on the OGAR side gets a matching arm in the same commit. 3. Workspace Cargo.lock + crates/lance-graph-ogar/Cargo.lock (gitignored, bumped locally): ogar-vocab pin 5089c1e8 → 597ecb12 (post-OGAR #127). 4 refs each. Same pull-bump pattern as q2 #42 / D2 #597 / D3 #606. After this PR: - 11-of-11 cross-axis identity gaps closed - parity::domains_agree covers all 10 mapped ConceptDomain variants (Reserved, ProjectMgmt, Commerce, Osint, Ocr, Health, Anatomy, Auth, Automation, HR) + Unassigned - odoo-rs alignment_pin can tighten from "commerce-arm only" hard pin to "all 15 seeded classes have an OGAR classid" EPIPHANIES.md updated per Mandatory Board-Hygiene Rule with the milestone finding E-OGAR-11-OF-11-CLOSURE — documents the arc (3 OGAR PRs + 3 lance-graph mirror PRs) and names the two discipline epiphanies that kept it from compounding drift: E-OGAR-AUTH-MIRROR-DRIFT and E-OGAR-API-EDIT-PULL-FIRST. Source-of-finding: - AdaWorldAPI/OGAR#127 (merged) — HR domain mint - AdaWorldAPI/odoo-rs#14 — the original 11-class gap (cross-axis check) - AdaWorldAPI/OGAR#110 — AuthStore precedent (0x0BXX) for new-domain mints 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. |
|
Warning Review limit reached
More reviews will be available in 19 minutes and 38 seconds. Learn how PR review limits work. Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file). ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
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 |
…s Unassigned); add 0x0EXX→Unassigned guard
D4 — closes 11-of-11 cross-axis identity gap
Final mirror update for the cross-axis identity arc that started with
odoo-rs PR #14. OGAR
PR #127 just minted the HR domain (4 final classes); this PR mirrors them
and closes the last domain-agreement drift.
The full arc closed
After this PR merges: 11 of 11 closed.
Three coordinated fixes (same shape as D2 / D3)
1.
lance-graph-contract::ogar_codebook— mirror parityConceptDomain::HRvariant.0x0D → HRarm tocanonical_concept_domain.CODEBOOK:("hr_employee", 0x0D01)("hr_department", 0x0D02)("hr_job", 0x0D03)("hr_employment_contract", 0x0D04)Unassigneddoc-comment from0x0DXX+→0x0EXX+.2.
lance-graph-ogar::parity::domains_agree— new armAdd
(O::HR, C::HR). Same shape as theAnatomy+Automationarms added in D3 #606. After this PR,domains_agreecovers all 10 mapped variants (Reserved / ProjectMgmt / Commerce / Osint / Ocr / Health / Anatomy / Auth / Automation / HR) + Unassigned.3.
Cargo.lock— OGAR pin bumpBump
ogar-vocab/ogar-class-view/ogar-ontology/ogar-adapter-surrealqlfrom5089c1e8→597ecb12(post-#127 OGAR main). 4 refs in workspace lock; 4 in the gitignoredcrates/lance-graph-ogar/Cargo.lock(bumped locally; consumer cargo regenerates).Board-Hygiene per Mandatory Rule
.claude/board/EPIPHANIES.mdprepended withE-OGAR-11-OF-11-CLOSURE(Status: FINDING — milestone). Names the arc + the two discipline epiphanies that kept it from compounding drift:E-OGAR-AUTH-MIRROR-DRIFT— mirror updates land in the SAME commit as any OGARALL-list extension; COUNT_FUSE is the local guardrail.E-OGAR-API-EDIT-PULL-FIRST— API-based file edits MUST pull-then-splice; uploading pre-edited local files regresses upstream main.Cross-repo follow-up
tests/alignment_pin.rs::seeded_classes_have_compatible_ogar_identityinformationaleprintlnshould flip from "7 remaining" to 0 remaining after odoo-rs bumps its OGAR pin past refactor: dual_engine + composite_engine use BuiltEngine (Phase 3) dual_engine.rs rewritten: DualEngine::new(label_a, engine_a, label_b, engine_b) — any BuiltEngine pair DualEngine::u8_vs_bf16(table) — the real comparison (u8 CDF vs BF16 direct) No longer hardcoded to u8 vs i8. Supports all three table types. composite_engine.rs rewritten: CompositeEngine uses BuiltEngine (not hardcoded ThinkingEngine) add_engine(): any BuiltEngine add_u8_lens(): legacy u8 compatibility add_bf16_lens(): BF16 tables from StackedN pipeline Mixed types in same composition (u8 legacy + BF16 new) Both use BuiltEngine from builder.rs which dispatches: BuiltEngine::Unsigned(ThinkingEngine) — u8 CDF (legacy) BuiltEngine::Signed(SignedThinkingEngine) — i8 (deprecated, CDF relabeling) BuiltEngine::BF16(BF16ThinkingEngine) — BF16 from StackedN (correct path) 257 tests pass. https://claude.ai/code/session_019RzHP8tpJu55ESTxhfUy1A #127. The hard-pin assertion can extend from "commerce-arm only" (3 classes) to "all 15 seeded classes have an OGAR classid" — total closure. Separate PR in odoo-rs.Tests
cargo build -p lance-graph-ogarcan't run locally (network 403 fetching the new OGAR pin), but the changes are correct and follow the well-established D2/D3 pattern. CI on this branch will verify.Refs
0x0BXX) for new-domain mintsGenerated by Claude Code