ENG-2084 Add schema import UI for Obsidian

[trangdoan982]

Summary

Adds the import modal UI and wires it into the command palette and settings.

• ImportSchemaPreviewSummary.tsx — per-category new/existing counts shown before the user applies the import
• ImportSpecsModal.tsx — two-screen modal: file picker screen → preview + selection screen (reuses SchemaSelectionModalBody + SchemaSelectionPanel from ENG-2083); delegates apply logic to applySchemaImportSelection from ENG-1977
• GeneralSettings.tsx — adds "Import discourse graph schema" entry to the settings UI
• registerCommands.ts — registers import-dg-schema command in the command palette

Stack

PR 5 of 5 for FEE-840. Stacks on #1264.

1. Shared foundation — ENG-1975 (ENG-1975 Define schema file contract and shared foundation for Obsidian export/import #1180)
2. Schema selection panel UI — ENG-2083 (ENG-2083 Add schema selection panel UI for Obsidian export/import #1262)
3. Schema export command — ENG-1976 (ENG-1976 Add schema export command to Obsidian #1263)
4. Schema import data layer — ENG-1977 (ENG-1977 Add schema import data layer for Obsidian #1264)
5. This PR — schema import UI (ENG-2084)

Test plan

• pnpm --filter @discourse-graphs/obsidian check-types passes
• Command palette: Import discourse graph schema opens the import modal
• Settings entry opens the import modal
• Picking a non-schema JSON or wrong-version file shows a clear error notice
• Preview summary shows correct new/existing counts per category
• Selection panel enforces dependency constraints
• Import applies correctly and creates new items as provisional
• Re-importing the same file creates no duplicates

---

[linear-code]

ENG-2084

[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 30, 2026 3:18am

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

[devin-ai-integration]

✅ Devin Review: No Issues Found

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

[graphite-app]

React state update on unmounted component. When the import succeeds, onClose() is called at line 102, which closes the modal and unmounts the component. Then the finally block at line 107 attempts to call setIsApplyingImport(false) on an already unmounted component, causing a React error.

Fix: Move onClose() after the finally block or restructure the flow:

try {
  const result = await applySchemaImportSelection({
    plugin,
    loadedSchemaFile,
    selection: {
      nodeTypeIds: selected.nodeTypeIds,
      relationTypeIds: selected.relationTypeIds,
      discourseRelationIds: selected.relationIds,
      templateNames: selected.templateNames,
    },
  });

  const { created } = result;
  new Notice(
    `Import complete: ${created.nodeTypes} node type(s), ${created.relationTypes} relation type(s), ${created.discourseRelations} relation triple(s), and ${created.templates} template(s) created.`,
    7000,
  );

  if (result.warnings.length > 0) {
    new Notice(
      `Import completed with ${result.warnings.length} warning(s).`,
      6000,
    );
    for (const warning of result.warnings) {
      new Notice(warning, 6000);
    }
  }
} catch (error) {
  const message = error instanceof Error ? error.message : String(error);
  new Notice(`Failed to import schema: ${message}`, 6000);
} finally {
  setIsApplyingImport(false);
}
onClose(); // Move outside try/catch/finally

Suggested change

	onClose();
	} catch (error) {
	const message = error instanceof Error ? error.message : String(error);
	new Notice(`Failed to import schema: ${message}`, 6000);
	} finally {
	setIsApplyingImport(false);
	}
	} catch (error) {
	const message = error instanceof Error ? error.message : String(error);
	new Notice(`Failed to import schema: ${message}`, 6000);
	} finally {
	setIsApplyingImport(false);
	}
	onClose();

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.