Fix undo to skip ignored messages

[gergesh]

Summary

• When using undo/revert, skip messages where all text parts are marked as ignored, so the user reverts to the last meaningful message instead of an ignored one
• Fixes /undo broken with plugin Opencode-DCP/opencode-dynamic-context-pruning#191

Greptile Summary

Enhanced the undo functionality to skip over messages where all text parts are marked as ignored by DCP, ensuring users revert to the last meaningful message.

• Modified the message filtering logic in the undo handler to check for non-ignored text parts
• Uses optional chaining and some() to verify at least one text part exists that isn't ignored
• Addresses issue Desktop app crashes when running shell command as first message in new session #191 where undo would revert to ignored messages

Minor issues:

• Missing safety check when accessing sync.data.part[message.id] on line 543 (should use ?? [] fallback)
• Redo functionality may need similar ignored-message filtering for consistency

Confidence Score: 4/5

• This PR is safe to merge with minor refinements needed
• The core logic correctly implements the feature to skip ignored messages during undo. Two minor issues identified: (1) missing null safety check on line 543 that could cause runtime error, and (2) redo functionality lacks similar ignored-message filtering for consistency. The implementation is targeted and focused on the stated issue.
• Line 543 needs a safety check (?? []), and consider updating redo at line 572 for consistency

Important Files Changed

Filename	Overview
packages/opencode/src/cli/cmd/tui/routes/session/index.tsx	Added logic to skip messages with all ignored text parts during undo; minor safety concern with parts access

_{Last reviewed commit: dc7314f}

[greptile-apps]

_{1 file reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (2)

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx
add fallback to match pattern used elsewhere in this file

        const parts = sync.data.part[message.id] ?? []

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review.
Path: packages/opencode/src/cli/cmd/tui/routes/session/index.tsx
Line: 543

Comment:
add fallback to match pattern used elsewhere in this file

```suggestion
        const parts = sync.data.part[message.id] ?? []
```

<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>

How can I resolve this? If you propose a fix, please make it concise.

---

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx
redo should likely also skip ignored messages for consistency with undo behavior

_{Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!}

Prompt To Fix With AI

This is a comment left during a code review.
Path: packages/opencode/src/cli/cmd/tui/routes/session/index.tsx
Line: 572

Comment:
redo should likely also skip ignored messages for consistency with undo behavior

<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>

How can I resolve this? If you propose a fix, please make it concise.