ENG-1856 Store source identity metadata for Roam imported nodes

[sid597]

Summary

• store Roam imported-node provenance under discourse-graph.importedFrom
• use sourceNodeRid as the canonical cross-app identity and keep sourceModifiedAt as refresh state
• add consistent read, write, bulk lookup, and single-node lookup helpers
• route Roam shared-node discovery through the shared bulk lookup helper
• preserve sibling block props when imported metadata is written or refreshed

Why

Roam needs a stable way to recognize an imported shared node after its page title or content changes. Matching by display title would make duplicate prevention and refresh dependent on presentation and namespace decisions.

Obsidian already uses importedFromRid as the canonical imported identity and stores the imported source timestamp as lastModified. This implementation follows the same boundary in Roam:

• sourceNodeRid is the identity
• sourceModifiedAt is refresh state
• source app, source space, and source-local ID are not duplicated because they are encoded by the RID
• source title is not persisted as identity

This supersedes the closed v1 PR, #1161, and addresses its review feedback about the redundant six-field identity shape.

Implementation

The metadata is stored in page/block props rather than page content:

{
  "discourse-graph": {
    importedFrom: {
      sourceNodeRid,
      sourceModifiedAt,
    },
  },
}

Writing merges the existing discourse-graph object so unrelated metadata, including relation migration state, survives. Content replacement during import or refresh does not touch these block props.

The helper exposes:

• parsing and reading imported source metadata
• writing updated source metadata
• collecting all imported source RIDs for discovery duplicate prevention
• finding an imported local node UID by canonical source RID

Dependency

This PR is intentionally stacked on ENG-2019 / #1216, which is itself stacked on ENG-1855. ENG-1855 introduced the Roam discovery consumer and ENG-2019 extracted the cross-space listing logic while leaving already-imported detection app-side.

Once #1216 merges, this PR can be retargeted to main without changing the ENG-1856 implementation diff.

Validation

• pnpm --filter roam test -- src/utils/__tests__/importedSourceIdentity.test.ts src/utils/__tests__/discoverSharedNodes.test.ts
  • 11 test files passed
  • 51 tests passed
• pnpm --filter roam check-types
• targeted ESLint for the changed files
• Prettier
• git diff --check

The local shell reports Node 20 while the repository declares Node 22+, but all checks above completed successfully.

[linear-code]

ENG-1856

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
discourse-graph	Skipped		Jul 10, 2026 5:28pm

[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 ↗︎.

[sid597]

@codex review

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.