Skip to content

fix: restore update-typescript-go workflow#180

Closed
xesrevinu wants to merge 1 commit intoEffect-TS:mainfrom
xesrevinu:codex/fix-update-typescript-go-workflow
Closed

fix: restore update-typescript-go workflow#180
xesrevinu wants to merge 1 commit intoEffect-TS:mainfrom
xesrevinu:codex/fix-update-typescript-go-workflow

Conversation

@xesrevinu
Copy link
Copy Markdown
Contributor

Summary

Fix the automatic update-typescript-go workflow after the latest upstream typescript-go update.

This PR:

  • refreshes _patches/001-cmd-tsgo-main.patch against the current upstream cmd/tsgo/main.go
  • updates the typescript-go submodule to ee4225a856389fc94c8ce4a5fd9af771dbbc2c0b
  • refreshes generated shim output for the upstream API changes
  • syncs flake.nix / flake.lock with the updated upstream commit
  • hardens _tools/update-flake-vendor-hash.sh for local/staged submodule refreshes and empty NIX_BUILD_ARGS

Why

The scheduled workflow started failing in pnpm setup-repo --ci because _patches/001-cmd-tsgo-main.patch no longer applied after upstream added internal/core imports and core.ApplyDebugStackLimit() to cmd/tsgo/main.go.

Validation

Ran locally:

  • pnpm setup-repo --ci
  • pnpm check
  • pnpm test
  • nix build .#effect-tsgo

pnpm lint still reports pre-existing modernize diagnostics unrelated to this change.

@mattiamanzati
Copy link
Copy Markdown
Contributor

I'm sorry I noticed the PR after I have done the fix, thanks for it anyway! :)

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