PLAN — Migrate sak to the latest sx #2

Merged
agra merged 1 commit from flow/sak-sx-latest/plan into master 2026-09-18 14:34:31 +03:00
Owner

Tracker: #1605

This branch carries sak's sources and tests on the current sx. Every JSON encode site writes through the caller-Writer json.JsonWriter API, and the json.Object.put / json.List.append calls omit the context.allocator argument, which their default supplies. There are no shims and no aliases.

Required sx

sak on this branch requires an sx master carrying these three sx PRs, all three already merged:

  • sx#10 — duplicate server import
  • sx#13 — declaration facts by DeclId
  • sx#20 — caught inferred error shorthand

The evidence below is measured on sx master 03abf022. On an sx without them sak builds but answers no HTTP request (without sx#10), or answers every store refusal with 500 (without sx#20).

Evidence

Measured on this PR's head commit 477c494, with .sx-cache and .sx-tmp removed first:

  • SX_STDLIB_PATH=<sx>/library bash scripts/release.sh builds cold and writes dist/sak-0.2.0-macos-arm64.tar.gz, dist/sak-0.2.0-linux-x86_64.tar.gz and dist/sak-0.2.0-linux-arm64.tar.gz, each binary stamped with 477c494. The SPA rebuild leaves the tracked web/dist byte-identical.
  • sx run tests/step9_release.sx passes every check (STEP9 OK) against those archives, including "the packaged binary reports the checkout commit".
  • The full suite passes: all 20 files under tests/ exit 0, with 1379 ok checks and no failures.

Head: 477c494868

Tracker: [#1605](https://sak.swipelab.com/tasks/1605) This branch carries sak's sources and tests on the current sx. Every JSON encode site writes through the caller-Writer `json.JsonWriter` API, and the `json.Object.put` / `json.List.append` calls omit the `context.allocator` argument, which their default supplies. There are no shims and no aliases. ## Required sx sak on this branch requires an sx master carrying these three sx PRs, all three already merged: - [sx#10](https://git.swipelab.com/sx/sx/pulls/10) — duplicate server import - [sx#13](https://git.swipelab.com/sx/sx/pulls/13) — declaration facts by DeclId - [sx#20](https://git.swipelab.com/sx/sx/pulls/20) — caught inferred error shorthand The evidence below is measured on sx master `03abf022`. On an sx without them sak builds but answers no HTTP request (without sx#10), or answers every store refusal with 500 (without sx#20). ## Evidence Measured on this PR's head commit `477c494`, with `.sx-cache` and `.sx-tmp` removed first: - `SX_STDLIB_PATH=<sx>/library bash scripts/release.sh` builds cold and writes `dist/sak-0.2.0-macos-arm64.tar.gz`, `dist/sak-0.2.0-linux-x86_64.tar.gz` and `dist/sak-0.2.0-linux-arm64.tar.gz`, each binary stamped with `477c494`. The SPA rebuild leaves the tracked `web/dist` byte-identical. - `sx run tests/step9_release.sx` passes every check (`STEP9 OK`) against those archives, including "the packaged binary reports the checkout commit". - The full suite passes: all 20 files under `tests/` exit 0, with 1379 `ok` checks and no failures. Head: 477c4948686c0e4787d772426d96f334984fa4ba
encode_json and the test encoders write through json.JsonWriter; json
Object.put and List.append calls drop the explicit context.allocator
argument that equals their default.
agra merged commit 264d47b583 into master 2026-09-18 14:34:31 +03:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sx/sak!2
No description provided.