Repository navigation
Update dependency @effect/tsgo to v0.63.0 - #125
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
0.62.0→0.63.0Release Notes
Effect-TS/tsgo (@effect/tsgo)
v0.63.0Compare Source
Minor Changes
b961835: AddfailCauseToConstructor, a style suggestion for Effect v3 and v4 that rewritesEffect.failCause/Effect.failCauseSyncwrapping a singleCause.fail/Cause.dieinto the direct constructor:Piped spellings are detected through piping flows, and later steps are kept. On v3, a lazy
Cause.diebecomesEffect.dieSync; v4 has nodieSync, so it is not reported there. Forwarded, stored, composed or transformed causes, explicitfailCause<E>type arguments,asassertions and annotated thunks are skipped. Lazy forms written as apipe(...)subject or with a piped body are reported without a quick fix, as are lazy values whose error type would change once returned from afailSyncthunk (such ascond ? "A" : "B", whichCause.failwidens tostring) and named-import calls where no valueEffectmodule binding is in scope (type-only imports do not count).cbf958c: AddmanualMetricTapToTrack, an Effect v4 style suggestion with a quickfix that rewritesEffect.tap/tapError/tapDefectcalls which only callMetric.updateintoEffect.trackSuccesses/trackErrors/trackDefects:Inside callbacks, only metrics that are stable at the call site (earlier
constbindings,declare const, enum members) are moved, becausetrack*evaluates the metric once while a tap callback re-evaluates it on every run. A computed input stays inside a mapper so it is still evaluated on every run.e75ca43: AddmapMapErrorToMapBothstyle diagnostic (defaultoff, Effect v3 and v4) with a quickfix. AdjacentEffect.map+Effect.mapErrorsteps (either order) anywhere in one piping flow are reported: in apipe/.pipe/Effect.fnpipeline, across chained.pipe(...).pipe(...)calls, and in nested data-first, curried or mixed data-first/pipeable calls. The quickfix merges them intoEffect.mapBothand removes the other step, collapsing a.pipe(...)left without steps to its subject; on v4,Effect.mapEager+Effect.mapErrorEagersuggestsEffect.mapBothEager. The quickfix is withheld when it cannot keep behavior: a step callee that is not a plain or dotted name (such asgetEffect().map), explicit type arguments on any call between and including the two steps (a curried application, a containing or emptypipe/.pipe), no binding in scope that reachesmapBoth, a rewrite that would newly start an expression statement with(or another token that continues the previous line, or a pair split across calls where merging would move a callback that is not a function literal, or the work of an eager first step, across other code.Patch Changes
a2751c7:catchAllToMapErrorandcatchToOrElseSucceednow name the catch variant the callee resolves to in their message. Previously a named import, renamed import or local alias ofEffect.catchAllwas reported asEffect.catch:a2751c7: Quick fixes, refactors and completions now build references to Effect modules (Effect,Layer,Schema,Data,Context, ...) from the file's imports instead of reusing the receiver of the call being rewritten, and support namespace imports of the package barrel.When the call site reaches Effect through something other than an import, the fix now uses the import:
5ebba91: Perf CI: base and head are now measured on the same runner, and only when a maintainer opts in with theperflabel. Builds still run in parallel, one job per channel and side, each with that commit's own tooling. A separatemeasurejob per channel downloads both binaries and runs 9 rounds of stock, base and head compiles in a rotating order. The report's main column is now head vs base directly, flagged by a paired Wilcoxon signed-rank test (p < 0.01), with base and PR overhead over stock tsgo kept as context.repoctl perf citakes--base-bin/--base-commit/--base-version,--head-bin/--head-commit/--head-versionand--roundsinstead of--side/--commit/--version/--patched-bin/--pairs, and writes version 2 cells:611fae3: Rust port bookkeeping: each origin marker now carries the hash of the Go declaration it was last synced to, soporting/manifest.tsvandporting/ported.tsv, which caused constant merge conflicts, are gone. Therepoctl tsrs port-*commands compute the manifest in memory,port-checkno longer fails on a stale marker line number, andport-ackwrites the hash (and current line) into the markers:// go: internal/typeparser/effect_type.go:45 TypeParser.EffectType 8a9c52f8a845e081fd5ec0a:preferSchemaOverJson's Effect v4 hint namesSchema.fromJsonString(Schema.Unknown)for unknown shapes instead ofSchema.UnknownFromJsonString, which effect marks@internaland leaves out of its published types:1b8a25b: Update the TypeScript next tag totypescript@next, which shipsTypeScriptcommit97aedd3b75e8e18a3cf2786ffbdb2e6c88aea23f, and the Rust port totsrs@0.20.1-ts7.1.0-beta. Symbol tables are read through per-provider helpers, since TypeScript next replaces the map-basedSymbolTablewith an insertion-ordered one.Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.