fix: state model threadId

[urmauur]

Describe Your Changes

This pull request refactors the ThreadDetail component in web-app/src/routes/threads/$threadId.tsx to simplify thread data subscription and improve state management. The most important changes include removing the getThreadById function and directly subscribing to thread data using a state selector.

Refactor thread data subscription:

• Removed the getThreadById function from the useThreads hook and replaced it with direct subscription to thread data using useShallow for improved reactivity.

Simplify state management:

• Updated the useThreads destructuring to remove the unused getThreadById function.

Fixes Issues

• Closes #
• Closes #

Self Checklist

• Added relevant comments, esp in complex areas
• Updated docs (for bug fixes / features)
• Created issues for follow-up changes or refactoring needed

---

Important

Refactor ThreadDetail component in $threadId.tsx to remove getThreadById and directly subscribe to thread data using useShallow.

• Refactor:
  • Remove getThreadById from useThreads in ThreadDetail component in $threadId.tsx.
  • Directly subscribe to thread data using useShallow for improved reactivity.
• State Management:
  • Simplify useThreads destructuring by removing unused getThreadById.

^{This description was created by for 716e376. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to 716e376 in 45 seconds. Click for details.

• Reviewed 26 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. web-app/src/routes/threads/$threadId.tsx:31

• Draft comment:
  Removing getThreadById from the destructured state is good for cleanup, but ensure that no other part relies on it.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

2. web-app/src/routes/threads/$threadId.tsx:39

• Draft comment:
  Directly subscribing to the thread data with a selector and useShallow improves reactivity. Verify that useThreads supports selector usage and that the reference in state.threads[threadId] stays stable to avoid extra renders.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_Xt65ZOwyYe9zSKtp

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}