Skip to content

Add ontology owner decision contract#548

Merged
SoundBlaster merged 3 commits into
mainfrom
codex/0114-ontology-owner-decision-contract
Jun 13, 2026
Merged

Add ontology owner decision contract#548
SoundBlaster merged 3 commits into
mainfrom
codex/0114-ontology-owner-decision-contract

Conversation

@SoundBlaster

Copy link
Copy Markdown
Member

Summary

Motivation

  • SpecGraph can now display closed-loop Ontology evidence, but it needs a stable contract for Ontology owner accepted/rejected decisions before a safe import preview can be built.

Goals

  • Emit runs/ontology_owner_decision_report.json from a typed decision fixture plus closed-loop evidence source refs.
  • Preserve read-only boundaries: no SpecGraph import, no gate close, no canonical mutation.
  • Leave decision import preview to the next bounded slice.

Changes

  • Adds owner decision report layout/contract/fixture to tools/ontology_semantic_control_policy.json.
  • Adds builder/validator/write path in tools/ontology_imports.py.
  • Adds focused tests, proposal/source docs, registry tracking, and operator docs.

Validation

  • Tests added/updated for changed behavior
  • Local checks passed

Commands run:

/opt/homebrew/bin/python3.10 -m pytest -q tests/test_ontology_import_policy.py
make ontology-imports PYTHON=/opt/homebrew/bin/python3.10
make proposal-tracking-gate PYTHON=/opt/homebrew/bin/python3.10
make docc-sync PYTHON=/opt/homebrew/bin/python3.10

Results:

  • Focused ontology tests: 69 passed
  • make ontology-imports: wrote runs/ontology_owner_decision_report.json
  • Proposal tracking gate: passed, 114 tracked proposals
  • DocC sync: passed

Risks / Notes

  • Backward compatibility impact: additive artifact only.
  • Migration/config changes required: none.
  • Known limitations: does not import owner decisions into SpecGraph; # next slice will add the preview contract.

Checklist

  • PR title clearly describes the change
  • Scope is focused and minimal
  • Documentation updated (or N/A)
  • No secrets or sensitive data added

Copilot AI review requested due to automatic review settings June 13, 2026 15:47

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 01378d73ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tools/ontology_imports.py
Comment thread tools/ontology_imports.py
…oard' into codex/tmp-0114-pushfix

# Conflicts:
#	tests/test_ontology_import_policy.py
@SoundBlaster SoundBlaster changed the base branch from codex/0113-ontology-review-dashboard to main June 13, 2026 16:43
@SoundBlaster SoundBlaster merged commit 127a153 into main Jun 13, 2026
6 checks passed
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.

2 participants