fix(core): break filesystem/search runtime import cycle

[tstachl]

Every prompt on the nixpkgs build (1.18.30, 1.18.31) dies before the model call with TypeError: undefined is not an object (evaluating 'node.name'), surfaced as 'Unexpected server error' (ref err_* in server logs).

Root cause: packages/core/src/filesystem.ts captures FileSystemSearch.node in its layer deps at module scope, while filesystem/search.ts imports ../filesystem at runtime. When the bundle evaluates search.ts first, the namespace is still uninitialized, so the dep is captured as undefined and the layer-tree walk (walk/recur -> resolve -> node.name) throws on first use via Agent.defaultInfo. (Minified stacks misattribute it to SystemPrompt.environment; an unminified build shows the true Agent.state path.)

Fix: make the search.ts import type-only and take the Entry/Match values straight from @opencode-ai/schema/filesystem (the same objects core re-exports). No behavior change; verified a bundle built from the patched source answers prompts again.

[github-actions]

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

• PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

[github-actions]

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

1. Open an issue describing the bug/feature (if one doesn't exist)
2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

[github-actions]

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

Potential Duplicate Found:

• PR fix(core): break filesystem/search import cycle #48877: fix(core): break filesystem/search import cycle
  • This appears to be a nearly identical fix for the same issue. The main difference is that PR fix(core): break filesystem/search runtime import cycle #49298 explicitly mentions "runtime" in the title and has more detail about the bundle evaluation order issue.

Related PRs that may be relevant context:

• PR fix(core): reject undefined layer node dependencies #48818: fix(core): reject undefined layer node dependencies - Addresses a related issue where undefined layer dependencies cause errors
• PR fix(core): break filesystem cycle in compiled prompts #48397: fix(core): break filesystem cycle in compiled prompts - Another filesystem cycle fix attempt

Recommendation: Check if PR #48877 was closed or if this is a reopened/revised attempt at the same fix. If PR #48877 is still open, this is likely a duplicate that should reference or potentially be combined with that PR.

[github-actions]

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.