ENG-2372 Record the source triple's RID on triples created by Obsidian relation import

[maparent]

https://entire.io/gh/DiscourseGraphs/discourse-graph/trails/75

Reviewer brief

• Size: About 415 non-test lines. The fix for new triples and the repair of existing ones share the source-triple lookup in sourceTriple.ts, so splitting them would ship that lookup without a user, or the repair without the fix that stops new bad triples.
• Review entry points: selectSourceTripleRids in sourceTriple.ts, which picks the source triple (a Roam relation's schema is already the triple; an Obsidian relation's triple is matched with the shared rule from ENG-2373 Share the relation triple matching rule in packages/database #1527). Then repairImportedTripleRids in repairTripleRids.ts, which detects bad RIDs by the relation subtype or by matching an imported instance's RID. Then the wiring in importRelations.ts and initializeSupabaseSync.
• Review focus: with no unique match, both paths keep the instance RID. Removing it would make isAcceptedSchema treat the imported triple as local, and sync would publish it as this space's own.
• Testing path: unit tests for selection and detection. The Loom shows the bug reproduced with a main build (Test -informs-> Question from an Obsidian space) and then repaired by this build. The Roam path is unit-tested only.

Loom video

https://www.loom.com/share/17eeaf4cc9cc4096a26b8bb360b24f11

Scope check

• Ran $scope-check against ENG-2372 and the final diff.
• Scope beyond Done When: Repaired triples get a new modified, so the next sync uploads the accepted ones. The ticket's Out of Scope excludes the triple values already published to the database.
  • Required now: Not required by Done When. It keeps published triples consistent with the vault, as a side effect of the repair.
  • Anyone affected or consulted: Ticket owner.
  • Decision: https://linear.app/discourse-graphs/issue/ENG-2372/record-the-source-triples-rid-on-triples-created-by-obsidian-relation#comment-a8689730

Standards check

• Ran $dg-pr-adherence-check against the final diff and PR metadata.

Local delegated full review

• Ran a comprehensive review of the entire final diff in a subagent with a fresh context. Use $dg-delegated-full-review when no other full-review workflow is available.

https://linear.app/discourse-graphs/issue/ENG-2372/record-the-source-triples-rid-on-triples-created-by-obsidian-relation

---

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
discourse-graph	Skipped		Oct 7, 2026 5:08pm UTC

[supabase]

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[linear-code]

ENG-2372

[devin-ai-integration]

Devin Review found 2 potential issues.

2 flags not posted on this PR by your GitHub settings — view them in Devin Review. (Configure)

[mdroidian]

@maparent see devin comments