Skip to content

refactor(git): remove runtime facade wrappers#21982

Merged
kitlangton merged 2 commits intodevfrom
kit/facade-git-collapse
Apr 11, 2026
Merged

refactor(git): remove runtime facade wrappers#21982
kitlangton merged 2 commits intodevfrom
kit/facade-git-collapse

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • remove the async Git.run() and Git.defaultBranch() facade wrappers from packages/opencode/src/git/index.ts
  • migrate the remaining CLI edge callers to AppRuntime.runPromise(Git.Service.use(...))
  • keep the existing Git service behavior unchanged while narrowing access to the Effect service surface

Verification

  • bun run typecheck
  • bun run test test/git/git.test.ts

Use AppRuntime at CLI edges so the Git service only exposes its Effect service surface.
@kitlangton kitlangton force-pushed the kit/facade-git-collapse branch from 68bf235 to 06c8d63 Compare April 11, 2026 03:49
@kitlangton kitlangton marked this pull request as ready for review April 11, 2026 03:50
@kitlangton kitlangton merged commit fe4dfb9 into dev Apr 11, 2026
7 of 9 checks passed
@kitlangton kitlangton deleted the kit/facade-git-collapse branch April 11, 2026 03:52
mrsimpson pushed a commit to mrsimpson/opencode that referenced this pull request Apr 14, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 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