docs: DISCOVERY-MAP D-OGAR-DOC-LAYER entry (charter #191 follow-up)#193
Conversation
The ogar-doc charter (docs/OGAR-DOC-LAYER-PROPOSAL.md, merged #191) mandated a DISCOVERY-MAP ledger entry (D9). Appending D-OGAR-DOC-LAYER [S] — the document persistence + reconstruction layer: raw bytes as a KV reference, awareness as a GUID-keyed SoA subtree, reconstruct_document ActionDef re-emits an updated PDF. Fires the v2-deferred typed_field mint (0x080A) + adds document (0x080B). Grade [S] until the 5+3 council + a probe promote it; the canon mints land WITH the council-verified build, never ahead of it. Append-only canon; no concept minted in this commit. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016b33swuXE23hKtqxsHu9p1
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_799db516-7936-4baf-9a14-12a40ff842c0) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 123d67d6a0
ℹ️ 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".
| `document` mint (0x080B); three facts-only ActionDefs (`persist` / `read` / | ||
| `reconstruct_document`, External kausal). Consumers inherit it via the classid |
There was a problem hiding this comment.
Keep ActionDef names aligned with the charter
The charter this entry points to names the facts-only capabilities persist_document, read_document, and reconstruct_document in docs/OGAR-DOC-LAYER-PROPOSAL.md D4, so recording persist / read here as ActionDefs creates a conflicting canonical name for future implementers following the discovery map. Please use the charter’s exact names here to avoid minting or documenting the wrong predicates later.
Useful? React with 👍 / 👎.
…append collision (chronological: #192 entries before the 07-12 doc-layer entry; stale 'D-entry above' ref named explicitly)
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_e229fdba-3ff5-458d-8539-406199793403) |
DISCOVERY-MAP
D-OGAR-DOC-LAYER— the charter's mandated ledger entryThe
ogar-doccharter (docs/OGAR-DOC-LAYER-PROPOSAL.md, merged #191) specified aDISCOVERY-MAPentry (§D9) as a merge requirement — closing that gap here (append-only canon, no concept minted in this commit).D-OGAR-DOC-LAYER[S]records the document persistence + reconstruction layer:documentnode value carries{sha256, kv_key, mime, counts}, never the bytes)reconstruct_documentActionDef re-emits an updated PDF — mutate a field node + re-fire ⇒ the doc re-issues with updated knowledgeFires the v2-deferred
typed_fieldmint (0x080A, the trigger the prior D-entry recorded) + addsdocument(0x080B). Grade[S]until the 5+3 council + a probe promote it; the canon mints land with the council-verified build, never ahead of it.🤖 Generated with Claude Code
https://claude.ai/code/session_016b33swuXE23hKtqxsHu9p1