Skip to content

refactor: replace dynamic imports with static imports in app/src#111

Merged
graycyrus merged 1 commit into
mainfrom
fix/remove-dynamic-imports
Mar 31, 2026
Merged

refactor: replace dynamic imports with static imports in app/src#111
graycyrus merged 1 commit into
mainfrom
fix/remove-dynamic-imports

Conversation

@graycyrus
Copy link
Copy Markdown
Contributor

Summary

  • Replace all 8 dynamic import() calls across 6 files in app/src/ with static top-level imports
  • Remove stale "Tech debt" note from CLAUDE.md since all listed files have been migrated
  • Existing try/catch guards preserved for Tauri/non-Tauri environment safety

Files changed

File Change
lib/skills/skillEvents.ts await import('@tauri-apps/api/event') → static import
lib/skills/manager.ts await import('coreRpcClient') x2 → static import
components/skills/SkillSetupWizard.tsx await import('skillsApi') x2 → static import
components/settings/panels/MessagingPanel.tsx import().then() chain → static import + direct call
hooks/useToolsUpdates.ts await import('file-watcher') → static import
utils/desktopDeepLinkListener.ts await import('skillsApi') → static import, merged duplicate
CLAUDE.md Removed stale tech debt note

Test plan

  • yarn typecheck — passes
  • yarn lint — passes (0 errors)
  • yarn format:check — passes
  • yarn build — passes
  • yarn tauri dev — app launches, core connects
  • Zero dynamic imports remaining in app/src/ (verified via grep)

Closes #102

@graycyrus graycyrus merged commit fca77f3 into main Mar 31, 2026
14 checks passed
@senamakel senamakel deleted the fix/remove-dynamic-imports branch March 31, 2026 18:12
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.

[Tech debt] Remove dynamic imports from app/src (static imports only)

1 participant