Repository navigation
docs(D-DT64-6): implementation plan for the OGIT predicate expansion - #1832
Conversation
Details #1831's D-DT64-6-1..4 and adds D-DT64-6-1a. ttl_parse classifies only the 12 rdfs:Class verbs (511 of 523 OGIT verbs are owl:ObjectProperty) and skips root ogit: names; ogit:allowed sits on entity files as blank-node cells that walk_rdf_list does not read. Stages give files, tests that fail on main, silence twins, disable runs and acceptance counts. Docs only. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
Takes #1831's STATUS_BOARD section as merged and adds a dated note on AdaWorldAPI/OGIT#11 to the plan. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f793a2a26c
ℹ️ 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".
…iption vs ON/WHEN/DO Adds plan sections 6-9, measured on OGIT b5731a8 against OGAR main's reader code and OGIT#11's full Turtle parse: - OGAR rejects 102 files (12 rdfs:Class verbs, 90 column-aligned type lines); accepting both forms measured 102 -> 90 -> 0. - extract_allowed_block loses 116 cells in 5 files (a ')' in a comment inside the list; ogit:allowed in a description). No file has two cells on a line or a cell spanning lines. - File universes 960/953/952/939/931 and allowed vs from-to rows kept apart. - Table ownership recorded as OPEN: OGAR, lance-graph-ontology, or a generated TSV snapshot both read (no dependency edge between them). - Corrections to merged #1831 section 2 and #1833 section 2 (the "successor" attribution); description is not a fourth axis; crai-rs CR-H2 waits on the D-SAG-6-1 audit. Housekeeping: parent link to main, the per-verb LinkSpec replacement is #1831's decision, OGIT#11 counts re-measured on its head, Calendar example shows the file's real layout. Merges main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4
#1831's entry already owns D-DT64-6; entries_index --check rejects the duplicate. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4
… §10 corrections - D-DT64-6-2 now states its precondition: it is a second ogit:allowed parser, which merged #1831 rules out unless the owner picks §6 option B and amends that guard. The stage is scoped to option B. - Item 5 resolves references across the whole checkout in a second pass, not per file. - §10 records corrections to the merged parent plan from the D-CFA, EAW and #1819 research docs, each citing the section it corrects. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4
Summary
#1831 (D-DT64-6, squash-merged as
42b4b83) describes the OGIT predicate stages D-DT64-6-1 to 6-4 in one line each. This PR adds an implementation plan for those stages,.claude/plans/d-dt64-6-ogit-predicates-impl-v1.md, with a board entry and board rows. It is documentation only: no code, tests or ABI, and it authorizes no work. Each stage still needs its own owner approval.The plan rests on measurements against OGIT
masterb5731a8:ttl_parseaccepts a verb only when it is declared as a class withrdfs:subClassOf ogit:Verb. 511 of OGIT's 523 verb files declareowl:ObjectPropertyinstead, so only the 12NTO/WorkOrderverbs come through.OgitUri::parsealso requires anogit.<Domain>:prefix, so names in the rootogit:namespace are skipped (183 SGO verbs and the SGO core entities).rdfs:Classverbs, plus 90 files whose type line has column-aligned whitespace aftera.extract_allowed_blockloses 116 cells in 5 files. In 4 of them a)inside a comment ends the list early; in the fifth a description contains the textogit:allowed.ogit:allowedlives in entity files. 960 files contain it, and none of them are underverbs/. The plan keeps the file counts apart (960 / 953 / 952 / 939 / 931), and keepsallowedrows apart fromfrom-torows;OgitTables::Allowedreads only the former.Stages
classifyacceptsowl:ObjectPropertyverbs; rootogit:subjects are kept; anything unclassified is reportedparse_verbanddetect_kindaccept therdfs:Classverb form and column-aligned type linestests/ontology_cache.rs: both readers, one OGIT checkout, one run(src, verb, dst)table, checked pair for pair. Applies only under option B (see below)Reconciliation with merged #1831 and #1833
OgitTables::Allowed; this plan's 6-2 proposes a second expansion inlance-graph-ontology. The reverse dependency is ruled out, becauseogar-action-handleralready depends onlance-graph-ontology. Three options:lance-graph-ontologyowns it, and OGAR runs the parity check.parse_ttl_directory_with_provenanceparses one file at a time, so 6-2 item 5 resolves references in a second pass, after every file's declarations have been collected.LinkSpec: the per-verb placeholder is replaced from the one table, as docs(D-DT64-6): DR64 routing, DT64 bind store, per-cell WHEN×ON×DO, OGIT reference predicates #1831 decides. This plan no longer treats it as open.extract_allowed_blockincompletely; the 2,894 / 523 figures come from a count made on the consumer side; docs(D-SAG-6-1): reference ledger and stale-status corrections for the D-SAG-6 audit #1833 §2 says docs(D-DT64-6): DR64 routing, DT64 bind store, per-cell WHEN×ON×DO, OGIT reference predicates #1831 calls crai-rs a "successor", but docs(D-DT64-6): DR64 routing, DT64 bind store, per-cell WHEN×ON×DO, OGIT reference predicates #1831 calls it a consumer of docs(D-SAG-6): Rust Codegraph → Mathematics → Logic → Tracert, prompt and board #1760.d-dt64-6-routing-bindstore-behavior-v1.mdit corrects:PredicateRole::RELATES), not a fourth axis. Its ON, WHEN and DO cells stay UNKNOWN until a source attests behaviour.docs/ROADMAP.mdstage CR-H2 lists the D-SAG-6-1 audit as a dependency.Board changes
entries/2026-10-28-d-dt64-6-ogit-predicates-impl.md, updated with the measurements.STATUS_BOARD.mdandINTEGRATION_PLANS.md.entries/README.mdregenerated; the supersession index is regenerated too and comes out unchanged.Validation
append_only_gate.py origin/mainpasses: 11 files checked, none shrank.entries_index.py --checkpasses (491 rows).OgitTables::load; it was not committed. The full-parse counts come from OGIT#11'stools/import_check.py.🤖 Generated with Claude Code
https://claude.ai/code/session_01XAqKMPWTwovjYJEUMuGPb4