Skip to content

fix: gate effect-app declaration facades (effect-app.10, latest upstream)#2

Open
patroza wants to merge 1 commit into
mainfrom
fix/update-tsgo-facade-gate
Open

fix: gate effect-app declaration facades (effect-app.10, latest upstream)#2
patroza wants to merge 1 commit into
mainfrom
fix/update-tsgo-facade-gate

Conversation

@patroza

@patroza patroza commented Jul 9, 2026

Copy link
Copy Markdown
Member

Supersedes #1. Tracks the effect-app .d.ts schema-facade emitter, now gated behind a compiler option and rebuilt against the latest upstream.

What changed

  • Upstream refresh: built on the latest typescript-go (@typescript/native-preview@7.0.0-dev.20260707.2, commit 9977d6d) already present on main. All _patches/ apply cleanly against it.
  • Gated facade emit (new vs _patches: layer the .d.ts-emit schema-facade emitter onto Effect-TS/tsgo #1): the schema-facade .d.ts emitter is now opt-in via a compiler option rather than always-on.
    • 028-checker-emitresolver-effect-schemaCreateTypeOf{Struct,Class}…Property materialization
    • 029-printer-emitresolver-effect-schema — printer hook
    • 030-transformers-declarations-effect-schema — declaration transformer
    • 031-effect-app-dts-facade-gate-option — the gate (compiler option + diagnostics + tsconfig parsing)
    • 032-effect-app-dts-facade-gate-test — coverage for the gate
  • Release tooling: _tools/cut-release.sh cross-compiles every platform binary, packages the tsgo-<platform>[.exe].gz assets + SHA256SUMS, and creates the GitHub release from a source-derived version.
  • Version: 7.0.0-dev+effect-app.10 (bumped from the already-released .9, which shipped the older upstream).

Validation

  • pnpm setup-repo — all 27 patches apply against 9977d6d; diagnostics + shims regenerated.
  • go build ./... — clean.
  • go test ./typescript-go/internal/transformers/declarations/... — passes (facade-gate test).
  • Built tsgo reports Version 7.0.0-dev+effect-app.10.

Release

Binary release cut via _tools/cut-release.shv7.0.0-dev-effect-app.10.

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.

1 participant