ENG-1290: Port discourse node config panel

[sid597]

https://www.loom.com/share/09868acba4424e28b0369c96104dbd3d

---

Summary by CodeRabbit

• Bug Fixes
  • Improved relation configuration saving with enhanced block structure and data persistence.
  • Fixed initialization flow to correctly map and reconcile relation keys during settings setup.
  • Enhanced configuration panel refresh to properly display saved relation settings.

[linear]

ENG-1290 Port Discourse node config panels

[sid597]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• ENG-1281: Port discourse node block panel #784 : 3 dependent PRs (#793 , #795 , #805 )
• ENG-1290: Port discourse node config panel #769 👈 (View in Graphite)
• ENG-1291: Port discourse node specification #765
• ENG-1280: Port Discourse node: Query builder and editor component for block props #764
• ENG-1225: Discourse node migration simple settings #696 : 1 other dependent PR (#697 )
• ENG-1273: Migrate small personal settings #691
• ENG-1272 Migrate all small global settings components #688 : 1 other dependent PR (#690 )
• ENG-1345: Modify existing RoamJS component onChange to also change props #745 : 1 other dependent PR (#684 )
• ENG-1190 Replace existing roam-js components with blockProp based ones #689
• ENG-1188: prod: pull watchers for prop based settings #682
• ENG-1187: Prod: Implement get and set for block prop based settings #669
• ENG-1189: Prod: Init block prop based schema #666
• ENG-1330: zod schema for discourse relations #719
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[devin-ai-integration]

✅ Devin Review: No Issues Found

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

View in Devin Review to see 6 additional findings.

[sid597]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.

[coderabbitai]

📝 Walkthrough

Walkthrough

Modifies the Discourse relation configuration panel to support asynchronous block updates and saves computed relation data to global settings. Concurrently refactors initialization logic to reconcile placeholder relation keys with actual block UIDs and updates the block property accessor signature.

Changes

Cohort / File(s)	Summary
Relation Config Panel apps/roam/src/components/settings/DiscourseRelationConfigPanel.tsx	Restructures save flow to wrap state updates in async IIFE with setTimeout; rebuilds block construction logic to assemble "And" blocks with nested relation structures and node positions; introduces extraction and persistence of ifConditions and relation metadata to global settings via setGlobalSetting.
Settings Initialization & Reconciliation apps/roam/src/components/settings/utils/init.ts	Introduces new reconcileRelationKeys function to map placeholder relation keys to actual block UIDs discovered in grammar/relations blocks; updates initializeSettingsBlockProps signature to accept pageUid parameter; extends initialization flow to invoke reconciliation logic for global relations.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

• PR #669: Modifies relations defaults/schema and block-prop accessors within settings plumbing, with direct usage in the init.ts changes and setGlobalSetting calls in the component.
• PR #666: Changes settings initialization logic in init.ts including initializeSettingsBlockProps flow and global relation key handling, overlapping with the reconciliation refactoring in this PR.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'ENG-1290: Port discourse node config panel' directly reflects the main changes: porting/refactoring the DiscourseRelationConfigPanel component with new async state management and block structure logic.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mdroidian]

Most of the audio on the loom is inaudible. Be sure to quickly check the looms/audio before triggering reviews.

[devin-ai-integration]

Devin Review found 1 new potential issue.

View 19 additional findings in Devin Review.

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