Skip to content

fix: declare missing frontend dependencies in package.json - #92

Open
Haiagari wants to merge 1 commit into
hardbeat920:mainfrom
Haiagari:fix/missing-frontend-dependencies
Open

Haiagari wants to merge 1 commit into
hardbeat920:mainfrom
Haiagari:fix/missing-frontend-dependencies

Conversation

@Haiagari

@Haiagari Haiagari commented Sep 6, 2026

Copy link
Copy Markdown

Problem Description

When building from source or running npm run build / pnpm build, TypeScript and Vite fail with 60+ missing module errors because several packages imported across src/ are not declared in package.json.

Specifically, imports like:

  • @codemirror/language in src/surfaces/editorLanguage.ts
  • @codemirror/state & @codemirror/view across editor surfaces
  • @codemirror/autocomplete & @codemirror/commands
  • @lezer/common
  • mermaid in src/surfaces/mermaidPlugin.ts
  • diff in src/lib/unifiedDiff.ts
  • unified in src/surfaces/AgentMarkdown.tsx

Solution

Declare all missing dependencies in package.json. Verified that pnpm build (tsc && vite build) and tauri dev build cleanly with 0 errors.

@nulljosh nulljosh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified — code already imports @codemirror/autocomplete, @codemirror/commands, mermaid, diff, unified, etc. (editorAutocomplete.ts, FileEditor.tsx, mermaidPlugin.ts, unifiedDiff.ts, and others) but they weren't declared in package.json, only pulled in transitively. This correctly declares the real dependencies.

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.

2 participants