Skip to content

effectify Command service - #18568

Merged
kitlangton merged 10 commits into
devfrom
kit/effectify-command-dev-rebase
Mar 22, 2026
Merged

kitlangton merged 10 commits into
devfrom
kit/effectify-command-dev-rebase

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

Summary

  • migrate Command from Instance.state to the current Effect service pattern with InstanceState and makeRunPromise
  • preserve lazy initialization behavior while making command loading and error handling explicit in the service layer
  • publish this as a fresh rebased draft branch so the original PR stays available for reference

Constrain BusEvent.define to ZodObject instead of ZodType so TS knows
event properties are always a record. Type GlobalBus payload as
{ type: string; properties: Record<string, unknown> } instead of any.

Refactor watcher test to use Bus.subscribe instead of raw GlobalBus
listener, removing hand-rolled event types and unnecessary casts.
@kitlangton
kitlangton marked this pull request as ready for review March 22, 2026 00:14
@kitlangton
kitlangton force-pushed the kit/effectify-command-dev-rebase branch from e7903e9 to 68409c9 Compare March 22, 2026 00:15
@kitlangton
kitlangton enabled auto-merge (squash) March 22, 2026 00:16
@kitlangton
kitlangton merged commit 0e0e7a4 into dev Mar 22, 2026
8 checks passed
@kitlangton
kitlangton deleted the kit/effectify-command-dev-rebase branch March 22, 2026 00:21
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request May 5, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
Rwanbt pushed a commit to Rwanbt/unifia that referenced this pull request Jun 14, 2026
MarsQiu007 referenced this pull request in MarsQiu007/openNovel Jul 31, 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