File Picker for Attaching Files

[FadhlanR]

Screen.Recording.2025-02-11.at.15.33.02.mov

• Refactored directory and file tree component so it can reused in the attach file modal.
• Added feature flag for attaching files in AI Assitant.
• Added attach file modal component.

[github-actions]

Host Test Results

    1 files  ±0      1 suites  ±0   24m 43s ⏱️ + 1m 49s
762 tests +2  759 ✔️ +4  3 💤 ±0  0 ❌ ±0 
767 runs  +2  764 ✔️ +6  3 💤 ±0  0 ❌  - 2 

Results for commit f45e1fa. ± Comparison against base commit 9fe7e54.

♻️ This comment has been updated with latest results.

[jurgenwerk]

@FadhlanR I have a draft PR up for displaying FileDefs in the AI assistant (auto-attached file in code mode). Perhaps you can rebase work on top of that. It is also ok if you'd like to merge this first without integrating with my PR, and do the integration in the AI assistant later. But let's think about how to minimize merge conflicts.

#2135

[FadhlanR]

@FadhlanR I have a draft PR up for displaying FileDefs in the AI assistant (auto-attached file in code mode). Perhaps you can rebase work on top of that. It is also ok if you'd like to merge this first without integrating with my PR, and do the integration in the AI assistant later. But let's think about how to minimize merge conflicts.

#2135

@jurgenwerk if this PR is ok to merge, I'll merge it and will rebase your PR. The feature flag introduced in this PR also can be used in yours.

[burieberry]

Looks good and works as expected.

[jurgenwerk]

@FadhlanR I was testing this and see this PR already includes the autoattached behaviour from #2135. So I see no point in maintaining a separate PR anymore since your PR now has that functionality almost in whole. I suggest you merge (or cherry pick the new commits if that is easier) my latest changes (my PR branch is green and ready to go). The latest changes include removing files from being attached, a test, and some type adjustments. I chose not to implement a resource for the autoattached files like we do with attached cards because here the situation is much simpler since we don't have any stacks, just the currently opened file. Let me know if you are ok with rebasing and then we can close my PR and move the rest of the todo into a separate ticket.

[FadhlanR]

@FadhlanR I was testing this and see this PR already includes the autoattached behaviour from #2135. So I see no point in maintaining a separate PR anymore since your PR now has that functionality almost in whole. I suggest you merge (or cherry pick the new commits if that is easier) my latest changes (my PR branch is green and ready to go). The latest changes include removing files from being attached, a test, and some type adjustments. I chose not to implement a resource for the autoattached files like we do with attached cards because here the situation is much simpler since we don't have any stacks, just the currently opened file. Let me know if you are ok with rebasing and then we can close my PR and move the rest of the todo into a separate ticket.

Sounds good. I'll include your latest updates to this PR.

[FadhlanR]

@jurgenwerk I've rebased this PR to your latest updates. And I made some updates for the auto-attached file.

[jurgenwerk]

I think using "all" in the name is redundant. We can just use filesToAttach. It should include all files shown in the attachments

[FadhlanR]

I merged autoAttachedFile and filesToAttach directly in the sendMessage function, following the same approach we use for autoAttachedCards and cardsToAttach.