fix(opencode): follow symlinks in file scan and directory listing

[mormubis]

Issue for this PR

Closes#29080

Type of change

• Bug fix
• New feature
• Refactor / code improvement
• Documentation

What does this PR do?

@file mentions cannot find files inside symlinked directories because File.scan() calls rg.files() without --follow — ripgrep skips symlinked directories by default.

this adds follow: true to the rg.files() call so ripgrep traverses symlinks when building the file search cache.

the other half of the issue (readDirectoryEntries classifying symlinked dirs as "symlink" instead of "directory") is addressed by #28532 which fixes the root cause in readDirectoryEntries itself.

How did you verify your code works?

• bun typecheck passes
• bun test test/file/index.test.ts — 52 tests pass

Checklist

• I have tested my changes locally
• I have not included unrelated changes in this PR

[github-actions]

The following comment was made by an LLM, it may be inaccurate:

Based on the search results, I found a related PR that addresses a similar concern:

Related PR:

• fix(filesystem): resolve symlink/junction targets to directory type in readDirectoryEntries #28532 - fix(filesystem): resolve symlink/junction targets to directory type in readDirectoryEntries
  • fix(filesystem): resolve symlink/junction targets to directory type in readDirectoryEntries #28532
  • This PR appears to address the same issue of correctly classifying symlinked directories in readDirectoryEntries. Your current PR (fix(opencode): follow symlinks in file scan and directory listing #29102) seems to be a continuation or related fix for the same problem area.

The other PRs found (#27639 and #24973) are related to symlink handling but focus on different aspects (circular symlinks and ripgrep-specific handling respectively).

No exact duplicates were found (the current PR #29102 should not be marked as a duplicate of itself), but you may want to review #28532 to ensure your changes complement rather than conflict with that work.

[github-actions]

Thanks for updating your PR! It now meets our contributing guidelines. 👍

[sghng]

Any updates on this PR? This feature is very needed! @mormubis seems like there are some merge conflicts that need to be resolved?

[mormubis]

Any updates on this PR? This feature is very needed! @mormubis seems like there are some merge conflicts that need to be resolved?

@sghng resolved the conflicts... but not much else I can do ¯_(ツ)_/¯ I hope some contributor sees the PR and approve the CI.

[danielxxomg]

Hey @mormubis! Author of #28532 here.

Thanks for working on the complementary fix. Your follow: true change for ripgrep and my NFS.stat-based classification should cover the symlink behavior from both sides.

Hopefully both fixes can land together or close to each other.