GeezerVison

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 4, 2026 7:05am

PR Summary

Medium Risk
Medium risk because it changes how chat resources are persisted/removed (new DELETE side-effect) and adds automatic removal behavior on 404s, which could incorrectly drop resources if errors are misclassified.

Overview
Prevents get_workflow_logs tool results from auto-creating persisted log resources, so logs only appear in the resource panel when explicitly opened.

Adds a self-healing path for the log panel: EmbeddedLog detects a 404 from useLogDetail, triggers an onNotFound callback, and MothershipView removes the stale log resource.

Makes resource removals persistent by extending useChat.removeResource to also DELETE /api/mothership/chat/resources, keeping local UI state and stored chat resources in sync.

^{Reviewed by Cursor Bugbot for commit 6ce9f59. Configure here.}

Greptile Summary

This PR fixes a bug where Mothership's resource panel displayed stale "Log not found" entries caused by three cooperating issues: get_workflow_logs auto-persisting every returned log ID, no TTL on persisted resources, and the panel auto-selecting the last resource on load. The fix removes GetWorkflowLogs from the auto-persist list, adds self-healing (EmbeddedLog fires onNotFound on 404 which removes the resource from both local state and the DB), and makes removeResource symmetric with addResource by persisting deletions.

Confidence Score: 5/5

Safe to merge — all three cooperating causes of the stale-resource bug are addressed correctly with no new defects introduced.

No P0 or P1 findings. The self-heal flow is correct: 404 fires exactly once before the component unmounts, local state and DB are both cleaned up symmetrically, and the retry guard prevents unnecessary delay. The extraction.ts removal is clean. All changed paths are narrow and well-contained.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Panel as ResourcePanel
    participant EmbeddedLog
    participant RQ as React Query (useLogDetail)
    participant API as /api/mothership/chat/resources
    participant DB as Database

    Panel->>EmbeddedLog: render log (stale logId)
    EmbeddedLog->>RQ: useLogDetail(logId)
    RQ->>API: GET log detail
    API-->>RQ: 404 (log deleted)
    Note over RQ: retry: skip on 404
    RQ-->>EmbeddedLog: error (404)
    EmbeddedLog->>EmbeddedLog: useEffect detects 404
    EmbeddedLog->>Panel: onNotFound(resourceId)
    Panel->>Panel: removeResource('log', id) / setResources / setActiveResourceId
    Panel->>API: DELETE /api/mothership/chat/resources {chatId, resourceType, resourceId}
    API->>DB: UPDATE copilotChats remove resource from JSONB array
    DB-->>API: updated resources
    Note over Panel: EmbeddedLog unmounts, stale entry gone permanently

_{Reviews (3): Last reviewed commit: "fix(mothership): simplify onNotFoundRef ..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review