Skip to content

R/effect-v4-2#623

Draft
patroza wants to merge 75 commits intomainfrom
r/effect-v4-2
Draft

R/effect-v4-2#623
patroza wants to merge 75 commits intomainfrom
r/effect-v4-2

Conversation

@patroza
Copy link
Member

@patroza patroza commented Feb 25, 2026

No description provided.

@patroza patroza changed the base branch from r/effect-v4 to main February 25, 2026 16:33
…vices resolution\n\n- Replace any return types in makeRpcClient with proper TaggedRequestResult type\n- Add ~decodingServices phantom property to pre-compute services at definition time\n- Use S.Void instead of S.Schema<void> for correct DecodingServices: never\n- Use ReqDecodingServices property access in RequestHandlers instead of generic S.Codec.DecodingServices\n- Update test expectations for v4 (schemas no longer carry service requirements)"
jfet97 and others added 2 commits March 4, 2026 19:46
…/cli (#625)

* fix(cli): patch argv for wrap commands to support unquoted --flags

The new effect/unstable/cli lexer classifies --flags as LongOption tokens
and discards unrecognized ones. This patch auto-inserts "--" after
wrap-enabled subcommands so that `effect-app-cli index-multi tsc --build`
works without quoting, maintaining backward compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(cli): refactor argv patching to join wrap args into a single element

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants