Skip to content

chore: update typescript-go submodule#905

Merged
graphite-app[bot] merged 1 commit into
mainfrom
codex/update-typescript-go-submodule-20260424-2
Apr 24, 2026
Merged

chore: update typescript-go submodule#905
graphite-app[bot] merged 1 commit into
mainfrom
codex/update-typescript-go-submodule-20260424-2

Conversation

@camc314

@camc314 camc314 commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update the typescript-go submodule from 83b8d2aa23b2d385087dabe5a5a8afd5e296013d to ba858e5c6852160ff69184d7fd7db77b348e246c
  • refresh the local patches/*.patch series against the new upstream base
  • regenerate shims for upstream API surface changes in checker, lsproto, and tsoptions

Verification

  • just shim
  • go test ./internal/... against the refreshed patched submodule state
  • CI workflows apply patches/*.patch before build/test via .github/actions/setup

@camc314 camc314 marked this pull request as ready for review April 24, 2026 09:18
Copilot AI review requested due to automatic review settings April 24, 2026 09:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the vendored typescript-go submodule and refreshes this repo’s compatibility layer (patch series + generated shims) to match upstream API changes.

Changes:

  • Bump typescript-go submodule and refresh patches/*.patch against the new upstream base.
  • Regenerate shim exports for updated upstream surfaces (checker, lsproto, tsoptions).
  • Update workspace dependency checksums in go.work.sum.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
shim/tsoptions/shim.go Removes a shim export (InverseJsxOptionMap) to match upstream surface changes.
shim/lsp/lsproto/shim.go Updates LSP protocol shim aliases/constants (removes closing tag completion, adds multi-document highlight + VS auto-insert).
shim/checker/shim.go Exposes newly added checker flags via shim constants.
patches/0001-Adapt-project-service-for-single-run-mode.patch Patch series metadata refreshed for new upstream base.
patches/0002-patch-expose-more-functions-via-the-shim-with-type-f.patch Patch series metadata refreshed for new upstream base.
patches/0003-fix-early-return-from-invalid-tsconfig-for-better-er.patch Patch series metadata refreshed for new upstream base.
patches/0004-fix-collections-avoid-internal-json-import-in-ordere.patch Patch series metadata refreshed for new upstream base.
patches/0005-perf-vfs-cache-ReadFile-results-in-cachedvfs.patch Patch series metadata refreshed for new upstream base.
go.work.sum Workspace checksum updates; currently missing sums for some versions pinned in go.mod.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread go.work.sum

camc314 commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

Merge activity

## Summary
- update the `typescript-go` submodule from `83b8d2aa23b2d385087dabe5a5a8afd5e296013d` to `ba858e5c6852160ff69184d7fd7db77b348e246c`
- refresh the local `patches/*.patch` series against the new upstream base
- regenerate shims for upstream API surface changes in checker, lsproto, and tsoptions

## Verification
- `just shim`
- `go test ./internal/...` against the refreshed patched submodule state
- CI workflows apply `patches/*.patch` before build/test via `.github/actions/setup`
@graphite-app graphite-app Bot force-pushed the codex/update-typescript-go-submodule-20260424-2 branch from c901d43 to dbfba7b Compare April 24, 2026 09:26
@graphite-app graphite-app Bot merged commit dbfba7b into main Apr 24, 2026
9 checks passed
@graphite-app graphite-app Bot removed the 0-merge label Apr 24, 2026
@graphite-app graphite-app Bot deleted the codex/update-typescript-go-submodule-20260424-2 branch April 24, 2026 09:33
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