refactor(v027): OllamaSettingsModal 제거 + onOpenOllamaSettings 채널 cleanup

src/preload/index.ts

@@ -47,11 +47,6 @@ const api: InklingApi = {
     emitRecallSnoozed: (id: string) => ipcRenderer.invoke('inbox:emitRecallSnoozed', id),
     loadOllamaSettings: () => ipcRenderer.invoke('inbox:loadOllamaSettings'),
     saveOllamaSettings: (v: { endpoint: string; model: string }) => ipcRenderer.invoke('inbox:saveOllamaSettings', v),
-    onOpenOllamaSettings: (cb: () => void) => {
-      const handler = () => cb();
-      ipcRenderer.on('inbox:openOllamaSettings', handler);
-      return () => ipcRenderer.removeListener('inbox:openOllamaSettings', handler);
-    },
     // v0.2.7 Task 13 — 외부 (트레이) 에서 view 전환 요청 listener.
     onNavigate: (cb: (view: 'inbox' | 'trash' | 'settings') => void) => {
       const listener = (_e: unknown, view: 'inbox' | 'trash' | 'settings') => cb(view);

src/renderer/inbox/App.tsx

@@ -12,7 +12,6 @@ import { TagUndoToast } from './components/TagUndoToast.js';
 import { ExpiryBanner } from './components/ExpiryBanner.js';
 import { FailedBanner } from './components/FailedBanner.js';
 import { RecallBanner } from './components/RecallBanner.js';
-import { OllamaSettingsModal } from './components/OllamaSettingsModal.js';
 import { SettingsPage } from './components/SettingsPage.js';
 
 export function App(): React.ReactElement {
@@ -25,7 +24,6 @@ export function App(): React.ReactElement {
   const showSettings = useInbox((s) => s.showSettings);
   const setShowSettings = useInbox((s) => s.setShowSettings);
   const [recoveryDismissed, setRecoveryDismissed] = useState(isRecoveryDismissedToday());
-  const [ollamaSettingsOpen, setOllamaSettingsOpen] = useState(false);
 
   useEffect(() => {
     void loadInitial();
@@ -36,7 +34,6 @@ export function App(): React.ReactElement {
     const unsubOllama = inboxApi.onOllamaStatus((status) => {
       useInbox.setState({ ollamaStatus: status });
     });
-    const unsubOllamaSettings = inboxApi.onOpenOllamaSettings(() => setOllamaSettingsOpen(true));
     const unsubNav = inboxApi.onNavigate((view) => {
       if (view === 'settings') {
         useInbox.getState().setShowSettings(true);
@@ -50,7 +47,7 @@ export function App(): React.ReactElement {
     });
     const onFocus = () => { void refreshMeta(); };
     window.addEventListener('focus', onFocus);
-    return () => { unsubNote(); unsubOllama(); unsubOllamaSettings(); unsubNav(); window.removeEventListener('focus', onFocus); };
+    return () => { unsubNote(); unsubOllama(); unsubNav(); window.removeEventListener('focus', onFocus); };
     // onOllamaStatus 콜백은 useInbox.setState 직접 호출 — store reference 가 안정적이라
     // deps array 에 추가 불필요. mount 시 1회 구독 + unmount 시 해제.
   }, [loadInitial, refreshMeta, upsertNote]);
@@ -112,7 +109,7 @@ export function App(): React.ReactElement {
       <main className="main">
         {!showTrash && (
           <>
-            <OllamaBanner onOpenSettings={() => setOllamaSettingsOpen(true)} />
+            <OllamaBanner onOpenSettings={() => setShowSettings(true)} />
             <RecoveryToast
               show={showRecovery}
               onDismiss={() => { markRecoveryDismissed(); setRecoveryDismissed(true); }}
@@ -187,10 +184,6 @@ export function App(): React.ReactElement {
         )}
       </main>
       <TagUndoToast />
-      <OllamaSettingsModal
-        open={ollamaSettingsOpen}
-        onClose={() => setOllamaSettingsOpen(false)}
-      />
     </>
   );
 }

src/renderer/inbox/components/OllamaSettingsModal.tsx

@@ -1,140 +0,0 @@
-import React, { useEffect, useState } from 'react';
-import { z } from 'zod';
-import { inboxApi } from '../api.js';
-import { DEFAULT_OLLAMA_ENDPOINT, DEFAULT_OLLAMA_MODEL } from '../../../shared/constants.js';
-
-const EndpointSchema = z.string().url();
-
-interface Props {
-  open: boolean;
-  onClose: () => void;
-}
-
-export function OllamaSettingsModal({ open, onClose }: Props): React.ReactElement | null {
-  const [endpoint, setEndpoint] = useState(DEFAULT_OLLAMA_ENDPOINT);
-  const [model, setModel] = useState(DEFAULT_OLLAMA_MODEL);
-  const [error, setError] = useState<string | null>(null);
-  const [saving, setSaving] = useState(false);
-
-  // 마운트/open 시 현재 설정 fetch
-  useEffect(() => {
-    if (!open) return;
-    void inboxApi.loadOllamaSettings().then((s) => {
-      if (s) {
-        setEndpoint(s.endpoint);
-        setModel(s.model);
-      }
-      setError(null);
-    });
-  }, [open]);
-
-  if (!open) return null;
-
-  async function handleSave() {
-    if (saving) return;  // m4 fix: synchronous double-click 가드
-    setSaving(true);
-    setError(null);
-    try {
-      // v0.2.6 #42 — client-side URL validation, server-side healthCheck 전에 명확한 메시지
-      const parseResult = EndpointSchema.safeParse(endpoint);
-      if (!parseResult.success) {
-        setError('유효한 URL 형식이 아닙니다 (예: http://localhost:11434)');
-        return;
-      }
-      if (model.trim().length === 0) {
-        setError('모델명을 입력하세요');
-        return;
-      }
-      const r = await inboxApi.saveOllamaSettings({ endpoint, model });
-      if (r.ok) {
-        onClose();
-      } else {
-        setError(r.reason);
-      }
-    } catch (e) {
-      setError(String(e));
-    } finally {
-      setSaving(false);
-    }
-  }
-
-  return (
-    <div
-      onKeyDown={(e) => {
-        if (e.key === 'Escape' && !saving) onClose();
-        if (e.key === 'Enter' && !saving) void handleSave();
-      }}
-      tabIndex={-1}
-      style={{
-        position: 'fixed', inset: 0, background: 'rgba(0,0,0,0.4)',
-        display: 'flex', alignItems: 'center', justifyContent: 'center', zIndex: 1000
-      }}
-    >
-      <div style={{
-        background: '#fff', borderRadius: 8, padding: 20, minWidth: 400, maxWidth: 500,
-        boxShadow: '0 4px 16px rgba(0,0,0,0.2)'
-      }}>
-        <h2 style={{ margin: '0 0 12px 0', fontSize: 16 }}>Ollama 설정</h2>
-        <div style={{ marginBottom: 12 }}>
-          <label style={{ display: 'block', fontSize: 12, color: '#666', marginBottom: 4 }}>
-            Endpoint URL
-          </label>
-          <input
-            type="text"
-            value={endpoint}
-            onChange={(e) => setEndpoint(e.target.value)}
-            placeholder="http://localhost:11434"
-            autoFocus
-            style={{ width: '100%', padding: '6px 8px', fontSize: 13, border: '1px solid #ccc', borderRadius: 4 }}
-            disabled={saving}
-          />
-        </div>
-        <div style={{ marginBottom: 12 }}>
-          <label style={{ display: 'block', fontSize: 12, color: '#666', marginBottom: 4 }}>
-            Model
-          </label>
-          <input
-            type="text"
-            value={model}
-            onChange={(e) => setModel(e.target.value)}
-            placeholder="gemma4:e4b"
-            style={{ width: '100%', padding: '6px 8px', fontSize: 13, border: '1px solid #ccc', borderRadius: 4 }}
-            disabled={saving}
-          />
-        </div>
-        {error && (
-          <div style={{
-            background: '#fce4e4', color: '#a33', padding: '6px 10px', borderRadius: 4,
-            fontSize: 12, marginBottom: 12
-          }}>
-            저장 실패: {error}
-          </div>
-        )}
-        <div style={{ display: 'flex', gap: 8, justifyContent: 'flex-end' }}>
-          <button
-            onClick={onClose}
-            disabled={saving}
-            style={{
-              background: 'transparent', color: '#666',
-              border: '1px solid #ccc', borderRadius: 4,
-              padding: '6px 14px', fontSize: 12, cursor: saving ? 'not-allowed' : 'pointer'
-            }}
-          >
-            취소
-          </button>
-          <button
-            onClick={() => void handleSave()}
-            disabled={saving}
-            style={{
-              background: saving ? '#999' : '#0a4b80', color: '#fff',
-              border: 'none', borderRadius: 4,
-              padding: '6px 14px', fontSize: 12, cursor: saving ? 'not-allowed' : 'pointer'
-            }}
-          >
-            {saving ? '검증 중...' : '저장'}
-          </button>
-        </div>
-      </div>
-    </div>
-  );
-}

src/shared/types.ts

@@ -105,7 +105,6 @@ export interface InboxApi {
   emitRecallSnoozed(id: string): Promise<void>;
   loadOllamaSettings(): Promise<{ endpoint: string; model: string } | null>;
   saveOllamaSettings(v: { endpoint: string; model: string }): Promise<{ ok: true } | { ok: false; reason: string }>;
-  onOpenOllamaSettings(cb: () => void): () => void;
   // v0.2.7 Task 13 — 외부 (트레이 등) 에서 view 전환 요청 구독.
   onNavigate(cb: (view: 'inbox' | 'trash' | 'settings') => void): () => void;
   // v0.2.7 자동 실행 (Task 22 통일) — 진단 정보 포함 응답

tests/unit/App.test.tsx

@@ -19,7 +19,6 @@ vi.mock('../../src/renderer/inbox/api.js', () => ({
     listRecallCandidate: vi.fn(async () => null),
     onNoteUpdated: vi.fn(() => () => undefined),
     onOllamaStatus: vi.fn(() => () => undefined),
-    onOpenOllamaSettings: vi.fn(() => () => undefined),
     onNavigate: vi.fn(() => () => undefined),
     // 4 섹션 mounted 시 호출되는 stub
     loadOllamaSettings: vi.fn(async () => ({ endpoint: '', model: '' })),