Skip to content

docs: assign corporate-master resolution ownership - #83

Merged
seonghobae merged 1 commit into
cursor/ontology-catalog-plane-90aafrom
codex/corporate-master-resolution-contract
Aug 26, 2026
Merged

docs: assign corporate-master resolution ownership#83
seonghobae merged 1 commit into
cursor/ontology-catalog-plane-90aafrom
codex/corporate-master-resolution-contract

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Outcome

Assigns the ecosystem corporate-master entity-resolution contract to Semantic
Data Portal's existing ontology/catalog governance plane. This PR is stacked on
#73 because that branch introduces the tenant-scoped catalog identity, alias,
concept-binding, stewardship, and persistence boundary that the contract owns.

Contract

  • Adds ADR 0002 and sdp.corporate-master-resolution/v1.
  • Preserves three distinct outcomes: unique, miss, and tie.
  • Allows a binding only for unique; miss and tie remain unbound.
  • Binds request, tenant, catalog snapshot, method, evidence references,
    limitations, candidates, and result digest.
  • Adds no matching algorithm, suffix list, similarity threshold, arbitrary
    weight, first-match tie break, catalog mutation, or executable-endpoint claim.

Ecosystem boundary evidence

  • Semantic Data Portal PRD ONT-001–ONT-006 and ADR 0001 own governed catalog
    identity, aliases, ontology terms, mappings, and stewardship.
  • Keyverse owns account identity, not corporate master data.
  • TEPP supplies temporal/event measurement evidence; it does not own canonical
    corporate binding.
  • RankWeave owns ranking/fusion artifacts; an artifact is evidence, not catalog
    identity authority.
  • LineageWeave owns authorized source reconstruction and may consume this
    contract after an implementation is released and pinned.

Verification

  • uv run --extra dev pytest -q tests/test_catalog_plane.py tests/test_catalog_plane_sql.py tests/test_migrations.py — 25 passed.
  • git diff --check — clean.
  • Standards basis: W3C SKOS Recommendation and DCAT 3 Recommendation, cited in
    APA 7th form in ADR 0002.

Implementation remains a follow-up requiring an accepted method ADR and
synthetic unique/miss/tie recovery tests. Until then consumers fail closed as
unavailable rather than restore local heuristics.


Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: 2f82ab73-3fd3-41f4-a4ea-fed90978190d

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

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

@seonghobae
seonghobae merged commit f592e7d into cursor/ontology-catalog-plane-90aa Aug 26, 2026
3 of 4 checks passed
@seonghobae
seonghobae deleted the codex/corporate-master-resolution-contract branch August 26, 2026 13:24

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread docs/implementation-compliance.md
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