Monaco Editor에 oxlint 린터 적용하여 에디터에 표기 기능 추가

[Bori-github]

📌 요약

• Monaco Editor에 oxlint 린터 적용하여 에디터에 표기 기능 추가

📝 작업 내용

• Rust 백엔드 oxlint 결과 반환 함수 생성
  • oxlint가 린트를 수행을 위한 에디터의 코드를 임시 파일에 저장하기 위해 tempfile 패키지 추가
  • 린트 결과를 반환하는 함수 생성
• oxlint 결과를 Rust 백엔드에서 전달받아 클라이언트에서 setModelMarkers를 통해 에디터에 표기

🔍 스크린샷

• 사용하지 않은 변수 unused에 대해 밑줄로 표기

💡 참고 사항

• 없음

[Copilot]

Pull Request Overview

This pull request adds oxlint integration to the Monaco code editor for real-time JavaScript/TypeScript linting. The implementation uses a Tauri backend command to execute oxlint via npx and displays lint results as markers in the editor.

• Integrates oxlint for code linting with 500ms debounced validation
• Adds Tauri backend command to execute oxlint on temporary files
• Implements Monaco editor markers to display lint errors and warnings

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
apps/executeJS/src/widgets/code-editor/code-editor.tsx	Implements validation logic with Monaco markers, debouncing, and event listeners for code changes
apps/executeJS/src/shared/types/index.ts	Defines LintResult interface for lint diagnostic data structure
apps/executeJS/src-tauri/src/lib.rs	Registers lint_code command in Tauri's invoke handler
apps/executeJS/src-tauri/src/commands.rs	Implements lint_code command that runs oxlint via npx and parses JSON output
apps/executeJS/src-tauri/Cargo.toml	Adds tempfile dependency for temporary file creation
Cargo.lock	Updates lock file with tempfile dependency

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull Request Overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

apps/executeJS/src/widgets/code-editor/code-editor.tsx:179

• The disposables are being overwritten at line 180, but then conditionally overwritten again at lines 211-215 or 221. This creates a potential memory leak because the original disposables at line 180 may not be properly disposed if the model exists. The assignment at line 180 should be removed, and the disposables should only be assigned once within the if-else block (lines 200-222).

      // Disposable들을 ref에 저장
      disposablesRef.current = [jsDisposable, tsDisposable];

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.