Skip to content

Sync upstream v1.31.1 and isolate wicked Go code generation - #16

Closed
Stumble wants to merge 541 commits into
mainfrom
refactor/upstream-sync
Closed

Sync upstream v1.31.1 and isolate wicked Go code generation#16
Stumble wants to merge 541 commits into
mainfrom
refactor/upstream-sync

Conversation

@Stumble

@Stumble Stumble commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Summary

Merge upstream sqlc v1.31.1 into the wicked fork and isolate wpgx code generation
in internal/codegen/wicked, without requiring a separately installed plugin.
Keep the existing sql_package: wpgx, schema ordering and comment-option syntax.

Architecture and key decisions

  • Compiler owns SQL analysis and supplies primary-schema/model and top-level
    SELECT facts through WickedMetadata; the backend owns Go mappings, templates,
    cache keys, timeout, invalidation and replica APIs.
  • Custom options use standard query comments instead of a custom Query.options field.
  • Generic compiler fixes remain independent commits in this fork. No upstream PR
    has been opened; that is a later contribution after whole-fork validation.

Compatibility follow-up

Additional baseline/consumer review exposed gaps not covered by the original
bookstore-only verification. Follow-up commits cead568ed and 62d9e5b26:

  • Restore Google UUID type identity, inherited primary models, legacy exec Row
    declarations and column-scoped JSON tags.
  • Preserve parameter context/nullability by fixing duplicate unaliased relation
    indexing, instead of promoting all assignment contexts. Keep explicit narg
    behavior and true self-join ambiguity checks.
  • Restore consistent Docker/make/CLI/generated-code release versions, tested by
    building and running a real binary.
  • Add a v2.3.4-derived generation fixture with consumer compilation and actual
    old-JSON round trips in CI; broaden PostgreSQL type/import regression tests.

Same-input regeneration in eight local consumer repositories covers 226 query
sets. All newly generated Go files match the v2.3.4 baseline except version
comments. Existing checked-in consumers on older fork versions additionally
pick up the already-existing nil-cache/concurrent-invalidation fixes; one
historically edited generated wrapper normalizes back to the template.
No consumer business code, configuration or production schema was changed.

Verification

  • Focused compiler/backend/cmd/config tests with CGO enabled and disabled.
  • Full examples/upstream suite with owned PostgreSQL/MySQL containers:
    GOMAXPROCS=2 PATH="$PWD/bin:$PATH" go run ./scripts/test-local -- go test -p 2 -parallel 8 -count=1 -tags=examples -timeout 20m ./...
    (endtoend: 144.467s).
  • GOTOOLCHAIN=go1.26.8 make build-endtoend; generated consumer/JSON fixture tests.
  • CGO and non-CGO binaries pass bookstore regeneration/diff; its 17 files remain unchanged.
  • Fresh bookstore tests, race tests and make lint-fix pass with isolated PostgreSQL/Redis.
  • 147 two-version generation cases: 113 successful identical outputs, 23 explained
    differences, and separately classified invalid/pre-existing broken cases.
  • govulncheck using Go 1.26.8 reports zero reachable vulnerabilities (not a claim
    that every imported module is vulnerability-free).
  • Buf/protobuf boundary is unchanged by the follow-up; original generation/lint
    and standard-plugin tests remain covered. Credential scan passes.

Seven consumer repositories and their local workspace modules compile fully.
For trex, regenerated files differ from its checked-in source only by version
comments, its generated repositories compile locally, and the exact existing
source commit has passing CI/lint/image-build checks. Local trex full-build
limitation:
the large CCXT dependency exceeds this environment's memory soft
limit; default and reduced-optimization attempts were cancelled, not counted
as passes. No dependency, business source, or host resource limit was changed.

Breaking changes and retained limits

  • The internal fork plugin protocol changes; old custom plugin binaries must be
    rebuilt. This is not a separately installed plugin migration for normal users.
  • Qualified built-in types that previously degraded to interface{} are now
    recognized. A standard INSERT/SELECT fixture also now retains inferred non-null
    types instead of losing them through a database fallback; both analysis modes
    are tested. These are documented inference improvements, not unexplained rewrites.
  • Legacy top-level SELECT classification is retained, not a complete proof of
    read-only SQL. Ordinary-view primary models, wicked batch/last-ID commands and
    database-only analysis remain unsupported. Copyfrom does not implement cache/invalidate.
  • No production migrations or downstream wpgx/dcache dependency upgrades.

Related PR and merge order

Runtime example: Stumble/bookstore#2.
Merge this sqlc PR first, then the example. No automatic merge or release.

Full design, audit disposition and verification evidence:
docs/changelogs/2026-09-08-sync-upstream-and-separate-wicked-codegen.md.

kyleconroy and others added 30 commits June 5, 2024 09:12
Managed databases only work using database servers hosted by sqlc Cloud. I'm going to add support for using managed databases with any database server you can access.

Database servers will be configured in a separate servers collection in the configuration file.

Each entry will have a mandatory uri field and an optional name. I may add an optional engine field if the URI scheme isn't enough to infer the engine type.

When using a database server not hosted by sqlc Cloud, sqlc will automatically create databases as needed based on the schema for the associated query set. All operations against these databases will be read-only, so they can be created once and re-used.
…3425)

The output still needs work, but this replaces the cloud-based verify
* Updated MySQL copyfrom template to handle strucs with 1 argument

* Added copyfrom 1 arg struct testcase

* Fixed issue where copyfrom structs were not being emitted

* Added testcase to catch not emitting structs used with copyfrom
Bumps golang from 1.22.4 to 1.22.5.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…v#3480)

Bumps [certifi](https://github.com/certifi/python-certifi) from 2024.6.2 to 2024.7.4.
- [Commits](certifi/python-certifi@2024.06.02...2024.07.04)

---
updated-dependencies:
- dependency-name: certifi
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.2.1 to 2.2.2.
- [Release notes](https://github.com/urllib3/urllib3/releases)
- [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst)
- [Commits](urllib3/urllib3@2.2.1...2.2.2)

---
updated-dependencies:
- dependency-name: urllib3
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…y with 9 updates (sqlc-dev#3520)

* build(deps): bump the production-dependencies group across 1 directory with 9 updates

Bumps the production-dependencies group with 9 updates in the / directory:

| Package | From | To |
| --- | --- | --- |
| [github.com/antlr4-go/antlr/v4](https://github.com/antlr4-go/antlr) | `4.13.0` | `4.13.1` |
| [github.com/google/cel-go](https://github.com/google/cel-go) | `0.20.1` | `0.21.0` |
| [github.com/jackc/pgx/v5](https://github.com/jackc/pgx) | `5.5.5` | `5.6.0` |
| [github.com/spf13/cobra](https://github.com/spf13/cobra) | `1.8.0` | `1.8.1` |
| [github.com/tetratelabs/wazero](https://github.com/tetratelabs/wazero) | `1.7.2` | `1.7.3` |
| [golang.org/x/sync](https://github.com/golang/sync) | `0.7.0` | `0.8.0` |
| [google.golang.org/grpc](https://github.com/grpc/grpc-go) | `1.63.2` | `1.65.0` |
| google.golang.org/protobuf | `1.34.1` | `1.34.2` |
| [modernc.org/sqlite](https://gitlab.com/cznic/sqlite) | `1.29.9` | `1.31.1` |



Updates `github.com/antlr4-go/antlr/v4` from 4.13.0 to 4.13.1
- [Release notes](https://github.com/antlr4-go/antlr/releases)
- [Commits](antlr4-go/antlr@v4.13.0...v4.13.1)

Updates `github.com/google/cel-go` from 0.20.1 to 0.21.0
- [Release notes](https://github.com/google/cel-go/releases)
- [Commits](https://github.com/google/cel-go/compare/v0.20.1...v0.21.0)

Updates `github.com/jackc/pgx/v5` from 5.5.5 to 5.6.0
- [Changelog](https://github.com/jackc/pgx/blob/master/CHANGELOG.md)
- [Commits](jackc/pgx@v5.5.5...v5.6.0)

Updates `github.com/spf13/cobra` from 1.8.0 to 1.8.1
- [Release notes](https://github.com/spf13/cobra/releases)
- [Commits](spf13/cobra@v1.8.0...v1.8.1)

Updates `github.com/tetratelabs/wazero` from 1.7.2 to 1.7.3
- [Release notes](https://github.com/tetratelabs/wazero/releases)
- [Commits](wazero/wazero@v1.7.2...v1.7.3)

Updates `golang.org/x/sync` from 0.7.0 to 0.8.0
- [Commits](golang/sync@v0.7.0...v0.8.0)

Updates `google.golang.org/grpc` from 1.63.2 to 1.65.0
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.63.2...v1.65.0)

Updates `google.golang.org/protobuf` from 1.34.1 to 1.34.2

Updates `modernc.org/sqlite` from 1.29.9 to 1.31.1
- [Commits](https://gitlab.com/cznic/sqlite/compare/v1.29.9...v1.31.1)

---
updated-dependencies:
- dependency-name: github.com/antlr4-go/antlr/v4
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: github.com/google/cel-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: github.com/jackc/pgx/v5
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: github.com/spf13/cobra
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: github.com/tetratelabs/wazero
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: google.golang.org/protobuf
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: modernc.org/sqlite
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

* go mod tidy

* Pull in latest github.com/wasilibs/go-pgquery

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kyle Conroy <kyle@sqlc.dev>
…y with 7 updates (sqlc-dev#3519)

* build(deps): bump the production-dependencies group across 1 directory with 7 updates

Bumps the production-dependencies group with 7 updates in the /docs directory:

| Package | From | To |
| --- | --- | --- |
| [alabaster](https://github.com/sphinx-doc/alabaster) | `0.7.16` | `1.0.0` |
| [packaging](https://github.com/pypa/packaging) | `24.0` | `24.1` |
| [sphinxcontrib-applehelp](https://github.com/sphinx-doc/sphinxcontrib-applehelp) | `1.0.8` | `2.0.0` |
| [sphinxcontrib-devhelp](https://github.com/sphinx-doc/sphinxcontrib-devhelp) | `1.0.6` | `2.0.0` |
| [sphinxcontrib-htmlhelp](https://github.com/sphinx-doc/sphinxcontrib-htmlhelp) | `2.0.5` | `2.1.0` |
| [sphinxcontrib-qthelp](https://github.com/sphinx-doc/sphinxcontrib-qthelp) | `1.0.7` | `2.0.0` |
| [sphinxcontrib-serializinghtml](https://github.com/sphinx-doc/sphinxcontrib-serializinghtml) | `1.1.10` | `2.0.0` |



Updates `alabaster` from 0.7.16 to 1.0.0
- [Release notes](https://github.com/sphinx-doc/alabaster/releases)
- [Changelog](https://github.com/sphinx-doc/alabaster/blob/master/docs/changelog.rst)
- [Commits](sphinx-doc/alabaster@0.7.16...1.0.0)

Updates `packaging` from 24.0 to 24.1
- [Release notes](https://github.com/pypa/packaging/releases)
- [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst)
- [Commits](pypa/packaging@24.0...24.1)

Updates `sphinxcontrib-applehelp` from 1.0.8 to 2.0.0
- [Release notes](https://github.com/sphinx-doc/sphinxcontrib-applehelp/releases)
- [Changelog](https://github.com/sphinx-doc/sphinxcontrib-applehelp/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinxcontrib-applehelp@1.0.8...2.0.0)

Updates `sphinxcontrib-devhelp` from 1.0.6 to 2.0.0
- [Release notes](https://github.com/sphinx-doc/sphinxcontrib-devhelp/releases)
- [Changelog](https://github.com/sphinx-doc/sphinxcontrib-devhelp/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinxcontrib-devhelp@1.0.6...2.0.0)

Updates `sphinxcontrib-htmlhelp` from 2.0.5 to 2.1.0
- [Release notes](https://github.com/sphinx-doc/sphinxcontrib-htmlhelp/releases)
- [Changelog](https://github.com/sphinx-doc/sphinxcontrib-htmlhelp/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinxcontrib-htmlhelp@2.0.5...2.1.0)

Updates `sphinxcontrib-qthelp` from 1.0.7 to 2.0.0
- [Release notes](https://github.com/sphinx-doc/sphinxcontrib-qthelp/releases)
- [Changelog](https://github.com/sphinx-doc/sphinxcontrib-qthelp/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinxcontrib-qthelp@1.0.7...2.0.0)

Updates `sphinxcontrib-serializinghtml` from 1.1.10 to 2.0.0
- [Release notes](https://github.com/sphinx-doc/sphinxcontrib-serializinghtml/releases)
- [Changelog](https://github.com/sphinx-doc/sphinxcontrib-serializinghtml/blob/master/CHANGES.rst)
- [Commits](sphinx-doc/sphinxcontrib-serializinghtml@1.1.10...2.0.0)

---
updated-dependencies:
- dependency-name: alabaster
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: packaging
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: sphinxcontrib-applehelp
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: sphinxcontrib-devhelp
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: sphinxcontrib-htmlhelp
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: sphinxcontrib-qthelp
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: sphinxcontrib-serializinghtml
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update a subset of deps

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kyle Conroy <kyle@sqlc.dev>
Also print out more error info by default
If you create a database but can't connect to it, clean up after yourself
…#3481)

When using Goose with numeric migrations, if the migration files aren't in
lexographic order SQLC parses them in the wrong order. This is similar to the issue that occurs
with golang-migrate. This pr documents this behaviour.
* Update endtoend testdata

* Update examples

* Update the rest
It wasn't being used and just cluttered things
Bumps the production-dependencies group in /docs with 1 update: [myst-parser](https://github.com/executablebooks/MyST-Parser).


Updates `myst-parser` from 3.0.1 to 4.0.0
- [Release notes](https://github.com/executablebooks/MyST-Parser/releases)
- [Changelog](https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md)
- [Commits](executablebooks/MyST-Parser@v3.0.1...v4.0.0)

---
updated-dependencies:
- dependency-name: myst-parser
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps golang from 1.22.5 to 1.22.6.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
dependabot Bot and others added 22 commits April 16, 2026 21:55
…y with 11 updates (sqlc-dev#4380)

* build(deps): bump the production-dependencies group across 1 directory with 11 updates

Bumps the production-dependencies group with 11 updates in the /docs directory:

| Package | From | To |
| --- | --- | --- |
| [babel](https://github.com/python-babel/babel) | `2.17.0` | `2.18.0` |
| [certifi](https://github.com/certifi/python-certifi) | `2026.1.4` | `2026.2.25` |
| [chardet](https://github.com/chardet/chardet) | `5.2.0` | `7.4.3` |
| [imagesize](https://github.com/shibukawa/imagesize_py) | `1.4.1` | `2.0.0` |
| [myst-parser](https://github.com/executablebooks/MyST-Parser) | `4.0.1` | `5.0.0` |
| [packaging](https://github.com/pypa/packaging) | `25.0` | `26.1` |
| [pyparsing](https://github.com/pyparsing/pyparsing) | `3.3.1` | `3.3.2` |
| [pytz](https://github.com/stub42/pytz) | `2025.2` | `2026.1.post1` |
| [requests](https://github.com/psf/requests) | `2.33.0` | `2.33.1` |
| [sphinx-favicon](https://github.com/tcmetzger/sphinx-favicon) | `1.0.1` | `1.1.0` |
| [sphinx-rtd-theme](https://github.com/readthedocs/sphinx_rtd_theme) | `3.0.2` | `3.1.0` |



Updates `babel` from 2.17.0 to 2.18.0
- [Release notes](https://github.com/python-babel/babel/releases)
- [Changelog](https://github.com/python-babel/babel/blob/master/CHANGES.rst)
- [Commits](python-babel/babel@v2.17.0...v2.18.0)

Updates `certifi` from 2026.1.4 to 2026.2.25
- [Commits](certifi/python-certifi@2026.01.04...2026.02.25)

Updates `chardet` from 5.2.0 to 7.4.3
- [Release notes](https://github.com/chardet/chardet/releases)
- [Changelog](https://github.com/chardet/chardet/blob/main/docs/changelog.rst)
- [Commits](chardet/chardet@5.2.0...7.4.3)

Updates `imagesize` from 1.4.1 to 2.0.0
- [Commits](shibukawa/imagesize_py@1.4.1...2.0.0)

Updates `myst-parser` from 4.0.1 to 5.0.0
- [Release notes](https://github.com/executablebooks/MyST-Parser/releases)
- [Changelog](https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md)
- [Commits](executablebooks/MyST-Parser@v4.0.1...v5.0.0)

Updates `packaging` from 25.0 to 26.1
- [Release notes](https://github.com/pypa/packaging/releases)
- [Changelog](https://github.com/pypa/packaging/blob/main/CHANGELOG.rst)
- [Commits](pypa/packaging@25.0...26.1)

Updates `pyparsing` from 3.3.1 to 3.3.2
- [Release notes](https://github.com/pyparsing/pyparsing/releases)
- [Changelog](https://github.com/pyparsing/pyparsing/blob/master/CHANGES)
- [Commits](pyparsing/pyparsing@3.3.1...3.3.2)

Updates `pytz` from 2025.2 to 2026.1.post1
- [Release notes](https://github.com/stub42/pytz/releases)
- [Commits](stub42/pytz@release_2025.2...release_2026.1.post1)

Updates `requests` from 2.33.0 to 2.33.1
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](psf/requests@v2.33.0...v2.33.1)

Updates `sphinx-favicon` from 1.0.1 to 1.1.0
- [Release notes](https://github.com/tcmetzger/sphinx-favicon/releases)
- [Changelog](https://github.com/tcmetzger/sphinx-favicon/blob/main/CHANGELOG)
- [Commits](tcmetzger/sphinx-favicon@v1.0.1...v1.1.0)

Updates `sphinx-rtd-theme` from 3.0.2 to 3.1.0
- [Changelog](https://github.com/readthedocs/sphinx_rtd_theme/blob/master/docs/changelog.rst)
- [Commits](readthedocs/sphinx_rtd_theme@3.0.2...3.1.0)

---
updated-dependencies:
- dependency-name: babel
  dependency-version: 2.18.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: certifi
  dependency-version: 2026.2.25
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: chardet
  dependency-version: 7.4.3
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: imagesize
  dependency-version: 2.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: myst-parser
  dependency-version: 5.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: packaging
  dependency-version: '26.1'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: pyparsing
  dependency-version: 3.3.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: pytz
  dependency-version: 2026.1.post1
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: production-dependencies
- dependency-name: requests
  dependency-version: 2.33.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
- dependency-name: sphinx-favicon
  dependency-version: 1.1.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
- dependency-name: sphinx-rtd-theme
  dependency-version: 3.1.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: production-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump Sphinx to 8.2.3 to satisfy myst-parser 5.0.0

myst-parser 5.0.0 requires sphinx>=8,<10, but docs/requirements.txt
was still pinning Sphinx==7.4.7, so pip couldn't resolve the
environment and the Read the Docs build failed before sphinx-build
ran. Bumping to the latest 8.x resolves the conflict.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kyle Conroy <kyle@conroy.org>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add paths-ignore for docs/** and .readthedocs.yaml to the go, buf,
  kotlin, python, and typescript workflows so doc-only PRs don't
  spin up the Go test matrix.
- Add a Read the Docs post_checkout hook that exits 183 (the
  documented "cancel build successfully" code) for PR builds that
  don't touch docs/ or .readthedocs.yaml, mirroring the inverse on
  the RTD side.

Note: if any of the skipped checks are currently listed as required
in branch protection, doc-only PRs will block waiting for them. Flip
those to non-required, or add a matching no-op workflow that runs
under the ignored paths, as a follow-up.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Preserve MySQL optimizer hints in stripped query text

Fixes sqlc-dev#4353. Hint comments (/*+ ... */) on their own line were being
treated as regular block comments and removed from the query string
embedded in generated code. Inline hints already survived because the
surrounding line did not match the block-comment pattern.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Drop unit test; end-to-end coverage is sufficient

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Removed paths-ignore for docs and readthedocs in CI.
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Bump version from 1.30.0 to 1.31.0

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Regenerate examples for 1.31.0

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Regenerate endtoend testdata for 1.31.0

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Stumble

Stumble commented Sep 9, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #17, which applies the verified implementation directly on the exact upstream v1.31.1 commit in two linear commits: core changes, then the wicked backend. The new PR CI passes. Product files are unchanged; only publication documentation was updated. The old refactor/upstream-sync branch and legacy main are retained without force-pushing. Please review #17 instead.

@Stumble Stumble closed this Sep 9, 2026
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.