Skip to content

fix: align markdown rootEntity override with promote identity#126

Merged
Jurij89 merged 7 commits intotest/devnet-e2e-sections-18-24from
fix/issue-122-root-entity-identity
Apr 12, 2026
Merged

fix: align markdown rootEntity override with promote identity#126
Jurij89 merged 7 commits intotest/devnet-e2e-sections-18-24from
fix/issue-122-root-entity-identity

Conversation

@Jurij89
Copy link
Copy Markdown
Contributor

@Jurij89 Jurij89 commented Apr 11, 2026

Summary

  • fix issue documentIri pinning causes frontmatter rootEntity override to disagree with promote partitioning #122 by retargeting markdown import extraction to the resolved
    ootEntity when frontmatter overrides the default assertion-UAL identity
  • keep _meta rows keyed by �ssertionUri while aligning data-graph rows 1/2/3 and content triples with the actual promotable entity subject
  • update the mirrored import orchestration test harness and add an issue-focused regression test proving the override no longer stays informational-only

Verification

Comment thread packages/cli/src/daemon.ts
Comment thread packages/cli/src/daemon.ts Outdated
Comment thread packages/cli/src/daemon.ts Outdated
Comment thread packages/cli/src/daemon.ts Outdated
Comment thread packages/cli/test/import-file-integration.test.ts
Comment thread packages/cli/src/daemon.ts Outdated
Comment thread packages/cli/src/daemon.ts Outdated
Comment thread packages/cli/src/daemon.ts
Comment thread packages/cli/src/extraction/markdown-extractor.ts Outdated
Comment thread packages/cli/src/daemon.ts
Comment thread packages/cli/test/import-file-integration.test.ts Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Codex review completed — no issues found.

@Jurij89 Jurij89 merged commit d614164 into test/devnet-e2e-sections-18-24 Apr 12, 2026
1 check 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.

1 participant