Skip to content

refactor(file): yield instance context in watcher#25205

Merged
kitlangton merged 1 commit into
devfrom
kit/file-watcher-instance-state
May 1, 2026
Merged

refactor(file): yield instance context in watcher#25205
kitlangton merged 1 commit into
devfrom
kit/file-watcher-instance-state

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Yield the file watcher instance context explicitly during state initialization.
  • Use the yielded context for watcher roots, protected paths, logs, and git directory lookup.
  • Route native watcher callbacks through InstanceState.bind so ALS and Effect-provided contexts both work.

Tests

  • bun typecheck
  • bun run test test/file/watcher.test.ts

@kitlangton kitlangton enabled auto-merge (squash) May 1, 2026 01:36
@kitlangton kitlangton disabled auto-merge May 1, 2026 01:45
@kitlangton kitlangton merged commit c2a97a7 into dev May 1, 2026
12 checks passed
@kitlangton kitlangton deleted the kit/file-watcher-instance-state branch May 1, 2026 01:45
caretak3r pushed a commit to caretak3r/opencode that referenced this pull request May 1, 2026
oleksii-honchar pushed a commit to oleksii-honchar/better-opencode that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant