build: require Go 1.26 everywhere at once

The newest go-json-experiment/json, which wails/v3's application
package imports, declares go 1.26, so taking it raises our go
directive with it.  Every other place that names a Go version moves in
the same commit: GO_VERSION in ci, desktop-assets and android-apk, the
Arch makedepends, and CONTRIBUTING's table.

index-artifact's container image moves too, and it is the one that
matters most.  The official golang images set GOTOOLCHAIN=local, so a
golang:1.25 container refuses a go 1.26 module outright rather than
fetching a newer toolchain, and that job owns the ~205 GB checkpoint.

Closes #265

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017HJiuc3ZZhxsPXz3ozTirT

.gitea/workflows/android-apk.yml

@@ -68,7 +68,7 @@ jobs:
       SHA: ${{ github.sha }}
       REF_NAME: ${{ github.ref_name }}
       DEBIAN_FRONTEND: noninteractive
-      GO_VERSION: '1.25.0'
+      GO_VERSION: '1.26.0'
       npm_config_store_dir: /cache/pnpm-store
       # The Go half wants the NDK; the Gradle half wants a platform.
       ANDROID_HOME: /cache/android-sdk

.gitea/workflows/ci.yml

@@ -36,7 +36,7 @@ concurrency:
   cancel-in-progress: true
 
 env:
-  GO_VERSION: '1.25.0'
+  GO_VERSION: '1.26.0'
   # Shared by all three Playwright consumers (@playwright/cli, e2e/'s
   # @playwright/test, frontend/'s Vitest provider).  See the browsers
   # step in job 2 for why that is not the whole story.
@@ -53,7 +53,7 @@ jobs:
   check:
     runs-on: ubuntu-latest
     container:
-      # Not golang:1.25 — this job runs `make ui-test`, which is Vitest
+      # Not golang:1.26 — this job runs `make ui-test`, which is Vitest
       # *browser* mode and needs a Chromium and its system libraries
       # anyway, so the "fast job needs no browser" split does not hold.
       # Not the Playwright image either: e2e/ pins @playwright/test
@@ -107,32 +107,15 @@ jobs:
       # Conventional Commits.  `.releaserc.yml` has always derived the
       # version from the commit type; until now nothing checked that the
       # type was one it recognises, so a malformed subject silently meant
-      # "no release".
-      #
-      # **On a `pull_request` there is no `before`.**  Gitea leaves
-      # `github.event.before` empty for one, so this step fell through to
-      # bare `make commit-check`, which lints `git log -1` — the tip
-      # alone.  Every other commit the branch would bring was first
-      # examined by *main's* post-merge run, which is a green PR that
-      # stops being true after the merge, and which happened twice (#254).
-      # The PR's base is the stand-in: the range below already excludes
-      # what the base shares with the branch, because base advances on
-      # main and those commits stay reachable from it.
-      #
-      # Both are handed to the shell rather than chosen in an expression:
-      # `github.event.issue.number` in unclaim.yml is this repo's proof
-      # that payload fields resolve, and the shell then falls back to
-      # today's behaviour for a dispatch run or a missing field instead of
-      # depending on how `&&`/`||` treat an absent context.
+      # "no release".  BEFORE is the push's previous tip and is absent or
+      # all-zeros for a new branch, in which case only the tip is linted.
       - name: Commit messages
         working-directory: /src
         env:
-          PR_BASE: ${{ github.event.pull_request.base.sha }}
-          PUSH_BEFORE: ${{ github.event.before }}
+          BEFORE: ${{ github.event.before }}
         run: |
           set -eu
-          BEFORE="${PR_BASE:-${PUSH_BEFORE:-}}"
-          if [ -n "$BEFORE" ] && [ "${BEFORE#0000000}" = "$BEFORE" ] \
+          if [ -n "${BEFORE:-}" ] && [ "${BEFORE#0000000}" = "$BEFORE" ] \
              && git cat-file -e "$BEFORE^{commit}" 2>/dev/null; then
             make commit-check RANGE="$BEFORE..$SHA"
           else

.gitea/workflows/desktop-assets.yml

@@ -48,7 +48,7 @@ jobs:
       SHA: ${{ github.sha }}
       REF_NAME: ${{ github.ref_name }}
       DEBIAN_FRONTEND: noninteractive
-      GO_VERSION: '1.25.0'
+      GO_VERSION: '1.26.0'
       npm_config_store_dir: /cache/pnpm-store
     steps:
       # The same set ci.yml's check job installs: the app is cgo, and

.gitea/workflows/index-artifact.yml

@@ -68,7 +68,7 @@ jobs:
       # claim with a test behind it now (cmd/indexbuild/deps_test.go),
       # because the v3 migration quietly broke it and this job was where
       # that surfaced.
-      image: golang:1.25
+      image: golang:1.26
       # This host path must exist on the runner and be listed verbatim in
       # act_runner's container.valid_volumes. It holds explore-staging/
       # (counts.bin + state.json) and yj.db — the checkpoint that makes

.pi/skills/yellowjacket-dev/references/harness.md

@@ -57,21 +57,6 @@ behind `YJ_TESTCTL=1`, which `scripts/dev-headless.sh` sets and
   staging the work that would produce it — job progress, download
   progress, scan progress. It calls `events.Deliver`, which *errors*
   when the event reaches nobody, so a `200` means it really arrived.
-- **State you stage, you own** (#168). Nothing resets those stores, so
-  clear yours in `test.afterEach` with the same event that staged it
-  (`emit('JobsChanged', [])`) — the store replaces its list from every
-  snapshot, so `testctl` needs no special case. **Measured: this does
-  not currently cross a spec boundary**, because every test gets a fresh
-  page and `JobStore.init()` refetches `GetJobs()` from a backend
-  registry that `/__test/emit` never writes to. Stated anyway, because
-  it costs one line and the leak needs only one spec that keeps a page
-  alive — but do not cite #168 for a symptom you have not reproduced.
-- **Measure against the thing next to you, not an absolute
-  coordinate.** An absolute number in a shell measurement is also a
-  claim about everything above it — `contentTop === 0` quietly asserts
-  "and no background job is running", which is not what that spec was
-  about or could arrange, while `contentTop === jobBandBottom` is true
-  either way. This is the half of #168 that stands on its own.
 - **`restore` is slow** (~40 s in the suite) because it copies every
   table. Prefer snapshotting once and restoring only when a spec
   genuinely mutates state.

CLAUDE.md

@@ -845,22 +845,6 @@ that is quietly empty.
   top-N, exact match, FTS search, popularity batch, the CAA map — and
   asserts each returns something with a dashed id. A missed conversion
   site shows up there and essentially nowhere else.
-- **A comparison is typed on *both* sides, and a parameter is the half
-  that gets forgotten.** The paragraph above is about a literal; the
-  artifact merge positioned its batch walk with a Go `string` cursor
-  against the artifact's byte column, and SQLite answered rather than
-  complained: `mbid > ?` with a text key is true of every row, so the
-  bound the walk looked up was the same every time and the cursor
-  never advanced, while `mbid <= ?` is false of every row, so no batch
-  merged at all. The import looped indefinitely at 100% CPU behind a
-  progress bar reading "0 of 1,077,893 rows", merged nothing and
-  raised nothing (#258). Nothing caught it because the fixture that
-  guards the walk writes the old text form and the only compact one is
-  a single row — below `artifactMergeBatch`, so the bound query never
-  ran. `artifactKey` types the cursor to the artifact's own encoding
-  now, and the walk fails loudly when its bound does not strictly
-  advance, because the failure mode here is silence rather than a
-  wrong answer.
 
 **The artifact is read in either encoding.** A published artifact
 carries whichever form the exporter that built it used, and there is one

CONTRIBUTING.md

@@ -13,7 +13,7 @@ frontend, bridged by [Wails v3](https://wails.io/).
 
 | Tool | Version |
 |------|---------|
-| Go | 1.25+ |
+| Go | 1.26+ |
 | Node.js | 22+ |
 | pnpm | 10+ |
 | Wails CLI | v3 — vendored, no install needed (`go tool wails3`) |

backend/explore/artifactimport.go

@@ -1,10 +1,8 @@
 package explore
 
 import (
-	"bytes"
 	"context"
 	"database/sql"
-	"database/sql/driver"
 	"errors"
 	"fmt"
 	"os"
@@ -350,12 +348,8 @@ func (si *SearchIndex) analyzeIndex() {
 // is an index range scan and a cancelled import leaves committed work
 // behind rather than rolling it all back.
 func (si *SearchIndex) mergeArtifactRows(ctx context.Context, total int) (int, error) {
-	// Asked once, because it is a property of the file and it decides
-	// how the walk's own comparisons are typed.  See artifactKey.
-	storesText := si.artifactStoresText()
-
 	selectColumns := artifactSelectColumns(
-		storesText, si.artifactHasTotals(),
+		si.artifactStoresText(), si.artifactHasTotals(),
 	)
 
 	insertSQL := `
@@ -373,7 +367,7 @@ func (si *SearchIndex) mergeArtifactRows(ctx context.Context, total int) (int, e
 		WHERE mbid > ? AND mbid <= ?` + upsertIndexConflictSQL
 
 	var (
-		cursor artifactKey
+		cursor string
 		merged int
 	)
 
@@ -382,30 +376,17 @@ func (si *SearchIndex) mergeArtifactRows(ctx context.Context, total int) (int, e
 			return merged, err
 		}
 
-		upper, hasUpper, err := si.artifactBatchBound(storesText, cursor)
+		upper, hasUpper, err := si.artifactBatchBound(cursor)
 		if err != nil {
 			return merged, err
 		}
 
-		if hasUpper && bytes.Compare(upper, cursor) <= 0 {
-			// The predicate matched the cursor itself, so the walk can
-			// never advance.  SQLite says nothing when a comparison is
-			// made between types it will not coerce - the query simply
-			// answers wrongly - so a mismatch here would otherwise spin
-			// forever behind an unmoving progress bar.  Fail instead.
-			return merged, fmt.Errorf(
-				"%w: artifact walk did not advance past %x",
-				ErrArtifactUnusable, []byte(cursor),
-			)
-		}
-
 		var res sql.Result
 
 		if hasUpper {
-			res, err = si.db.ExecContext(insertRangeSQL,
-				cursor.bind(storesText), upper.bind(storesText))
+			res, err = si.db.ExecContext(insertRangeSQL, cursor, upper)
 		} else {
-			res, err = si.db.ExecContext(insertSQL, cursor.bind(storesText))
+			res, err = si.db.ExecContext(insertSQL, cursor)
 		}
 
 		if err != nil {
@@ -432,65 +413,26 @@ func (si *SearchIndex) mergeArtifactRows(ctx context.Context, total int) (int, e
 	}
 }
 
-// artifactKey is one MBID as the attached artifact stores it: 16 raw
-// bytes in a compact artifact, the dashed 36-character form in one
-// published before that storage change.
-//
-// It is a type with a bind method rather than a string because the
-// comparison it feeds is typed, and the wrong type is silent.  SQLite
-// does not coerce between TEXT and BLOB and orders every blob after
-// every text value, so a cursor bound as text against a byte column
-// makes `mbid > ?` true of the whole table - the walk rediscovers the
-// same batch bound forever, and `mbid <= ?` false of the whole table,
-// so no batch merges at all.  Nothing errors; the import simply never
-// finishes.  bind is the one place that knows which form the column is
-// in, decided by artifactStoresText, which asks the artifact rather than
-// trusting a version number.
-type artifactKey []byte
-
-// bind renders the key as a statement argument in the artifact's own
-// encoding.
-func (k artifactKey) bind(storesText bool) driver.Value {
-	if storesText {
-		return string(k)
-	}
-
-	// Never nil.  database/sql converts a nil []byte to SQL NULL, and
-	// `mbid > NULL` is NULL for every row - so an unset cursor would
-	// agree with nothing and import nothing, which is the same silently
-	// empty merge this type exists to prevent, one type over.
-	if k == nil {
-		return []byte{}
-	}
-
-	return []byte(k)
-}
-
 // artifactBatchBound returns the MBID that ends the next batch, and
 // whether one exists — no bound means the remainder is the last batch.
-//
-// The bound is read out of the artifact and handed back as an
-// artifactKey, because it becomes the next comparison the walk makes.
-func (si *SearchIndex) artifactBatchBound(
-	storesText bool, cursor artifactKey,
-) (artifactKey, bool, error) {
-	var bound []byte
+func (si *SearchIndex) artifactBatchBound(cursor string) (string, bool, error) {
+	var bound string
 
 	err := si.db.QueryRowWriter(
 		`SELECT mbid FROM core.explore_index
 		 WHERE mbid > ? ORDER BY mbid LIMIT 1 OFFSET ?`,
-		cursor.bind(storesText), artifactMergeBatch-1,
+		cursor, artifactMergeBatch-1,
 	).Scan(&bound)
 
 	if errors.Is(err, sql.ErrNoRows) {
-		return nil, false, nil
+		return "", false, nil
 	}
 
 	if err != nil {
-		return nil, false, fmt.Errorf("%w: batch bound: %w", ErrArtifactUnusable, err)
+		return "", false, fmt.Errorf("%w: batch bound: %w", ErrArtifactUnusable, err)
 	}
 
-	return artifactKey(bound), true, nil
+	return bound, true, nil
 }
 
 // stampArtifactMeta records what the merge established: the catalog half

backend/explore/artifactimport_test.go

@@ -1,7 +1,6 @@
 package explore
 
 import (
-	"bytes"
 	"context"
 	"database/sql"
 	"encoding/hex"
@@ -72,7 +71,13 @@ func writeTestArtifact(
 		}
 	}
 
-	stampArtifactMeta(t, db, meta)
+	for k, v := range meta {
+		if _, err := db.Exec(
+			`INSERT INTO artifact_meta (key, value) VALUES (?, ?)`, k, v,
+		); err != nil {
+			t.Fatalf("stamp artifact meta: %v", err)
+		}
+	}
 
 	for _, r := range rows {
 		if _, err := db.Exec(`
@@ -88,101 +93,6 @@ func writeTestArtifact(
 	return path
 }
 
-// compactArtifactSchema is the artifact cmd/indexexport publishes: the
-// catalog's ids as 16 raw bytes, its entity types as codes, and the
-// per-release-group total_tracks the exporter added after the first
-// artifact was shipped.
-//
-// It matters that a fixture carries this encoding and not the older
-// text one, because SQLite does not coerce between TEXT and BLOB and
-// every comparison the importer makes against an mbid is therefore
-// encoding-sensitive.  writeTestArtifact above is the *other* fixture:
-// it still writes the text form, which is what the first published
-// artifact carries and what the importer must keep reading.
-var compactArtifactSchema = []string{
-	`CREATE TABLE explore_index (
-		entity_type      INTEGER NOT NULL,
-		mbid             BLOB NOT NULL,
-		title            TEXT NOT NULL,
-		artist_name      TEXT NOT NULL,
-		artist_mbid      BLOB NOT NULL,
-		aliases          TEXT NOT NULL DEFAULT '',
-		popularity       INTEGER NOT NULL DEFAULT 0,
-		listener_count   INTEGER NOT NULL DEFAULT 0,
-		duration         INTEGER NOT NULL DEFAULT 0,
-		caa_release_mbid BLOB NOT NULL DEFAULT x'',
-		release_name     TEXT NOT NULL DEFAULT '',
-		primary_type     TEXT NOT NULL DEFAULT '',
-		secondary_types  TEXT NOT NULL DEFAULT '',
-		release_date     TEXT NOT NULL DEFAULT '',
-		total_tracks     INTEGER NOT NULL DEFAULT 0,
-		artist_type      TEXT NOT NULL DEFAULT '',
-		country          TEXT NOT NULL DEFAULT '',
-		disambiguation   TEXT NOT NULL DEFAULT '',
-		sort_name        TEXT NOT NULL DEFAULT '',
-		discog_fetched   INTEGER NOT NULL DEFAULT 0,
-		PRIMARY KEY (mbid)
-	) WITHOUT ROWID`,
-	`CREATE TABLE artifact_meta (
-		key   TEXT PRIMARY KEY,
-		value TEXT NOT NULL
-	)`,
-}
-
-// writeCompactTestArtifact builds the artifact the exporter publishes
-// today, in its own encoding, so the importer is exercised against what
-// a client actually downloads rather than against what it was written
-// for.
-func writeCompactTestArtifact(
-	t *testing.T, meta map[string]string, rows []artifactRow,
-) string {
-	t.Helper()
-
-	path := filepath.Join(t.TempDir(), "core-index.db")
-
-	db, err := sql.Open("sqlite", "file:"+path)
-	if err != nil {
-		t.Fatalf("open artifact: %v", err)
-	}
-
-	defer func() { _ = db.Close() }()
-
-	for _, stmt := range compactArtifactSchema {
-		if _, err := db.Exec(stmt); err != nil {
-			t.Fatalf("create artifact schema: %v", err)
-		}
-	}
-
-	stampArtifactMeta(t, db, meta)
-
-	for _, r := range rows {
-		if _, err := db.Exec(`
-			INSERT INTO explore_index
-				(entity_type, mbid, title, artist_name, artist_mbid, popularity)
-			VALUES (?, ?, ?, ?, ?, ?)`,
-			entityCode(r.entityType), mbidBytes(r.mbid), r.title,
-			r.artistName, mbidBytes(r.artistMBID), r.popularity,
-		); err != nil {
-			t.Fatalf("insert artifact row: %v", err)
-		}
-	}
-
-	return path
-}
-
-// stampArtifactMeta writes the artifact_meta rows a fixture declares.
-func stampArtifactMeta(t *testing.T, db *sql.DB, meta map[string]string) {
-	t.Helper()
-
-	for k, v := range meta {
-		if _, err := db.Exec(
-			`INSERT INTO artifact_meta (key, value) VALUES (?, ?)`, k, v,
-		); err != nil {
-			t.Fatalf("stamp artifact meta: %v", err)
-		}
-	}
-}
-
 // validMeta is the artifact_meta a well-formed artifact carries.
 func validMeta() map[string]string {
 	return map[string]string{
@@ -360,71 +270,6 @@ func TestImportCoreArtifactBatchWalkCoversAllRows(t *testing.T) {
 	}
 }
 
-// TestImportCoreArtifactBatchWalkCoversAllRowsCompact is the batch walk
-// on the encoding the exporter actually publishes.
-//
-// The walk positions itself by comparing the artifact's own mbid column
-// against the last id it reached, and that column holds 16 raw bytes.
-// SQLite does not coerce between TEXT and BLOB, and a blob sorts after
-// every text value, so a cursor bound as text is a predicate that either
-// matches every row or none: `mbid > ?` with an empty text key is true
-// of the whole table, so
-// the 100th row is always the 100th row and the bound never advances,
-// while `mbid <= <text>` is false of the whole table, so no batch ever
-// merges.  The result is not a wrong import but an unbounded loop that
-// merges nothing and never fails.
-//
-// Both encodings are covered on purpose.  The walk was only ever tested
-// against the text fixture above, which is why it shipped broken on the
-// one the clients download.
-func TestImportCoreArtifactBatchWalkCoversAllRowsCompact(t *testing.T) {
-	db := database.NewTestDB(t)
-	si := NewSearchIndex(db, nil, nil, testLogger())
-
-	original := artifactMergeBatch
-	artifactMergeBatch = 100
-
-	t.Cleanup(func() { artifactMergeBatch = original })
-
-	const total = 337
-
-	rows := make([]artifactRow, 0, total)
-	for i := range total {
-		rows = append(rows, artifactRow{
-			entityType: EntityRecording,
-			mbid:       syntheticMBID(i),
-			title:      "Song",
-			artistName: "Artist",
-			artistMBID: artA,
-			popularity: i,
-		})
-	}
-
-	path := writeCompactTestArtifact(t, validMeta(), rows)
-
-	if err := si.importCoreArtifact(context.Background(), path); err != nil {
-		t.Fatalf("importCoreArtifact: %v", err)
-	}
-
-	var got, top int
-
-	if err := db.QueryRowWriter(
-		"SELECT COUNT(*), MAX(popularity) FROM explore_index",
-	).Scan(&got, &top); err != nil {
-		t.Fatalf("count rows: %v", err)
-	}
-
-	if got != total {
-		t.Errorf("merged %d rows, want %d", got, total)
-	}
-
-	// A count alone would pass if the walk re-merged the same first
-	// batch forever, so the far end of the artifact is checked too.
-	if top != total-1 {
-		t.Errorf("highest popularity = %d, want %d", top, total-1)
-	}
-}
-
 func TestImportCoreArtifactRejectsBadArtifacts(t *testing.T) {
 	tests := []struct {
 		name string
@@ -609,9 +454,61 @@ func TestArtifactColumnsMatchExporter(t *testing.T) {
 // the importer decides by asking the artifact, not by trusting a
 // version number, and both must land identically.
 func TestImportCoreArtifactAcceptsBothEncodings(t *testing.T) {
-	compact := writeCompactTestArtifact(t, validMeta(), []artifactRow{
-		{EntityArtist, artA, "Artist A", "Artist A", artA, 5000},
-	})
+	compact := filepath.Join(t.TempDir(), "core-index.db")
+
+	db, err := sql.Open("sqlite", "file:"+compact)
+	if err != nil {
+		t.Fatalf("open artifact: %v", err)
+	}
+
+	if _, err := db.Exec(`CREATE TABLE explore_index (
+		entity_type      INTEGER NOT NULL,
+		mbid             BLOB NOT NULL,
+		title            TEXT NOT NULL,
+		artist_name      TEXT NOT NULL,
+		artist_mbid      BLOB NOT NULL,
+		aliases          TEXT NOT NULL DEFAULT '',
+		popularity       INTEGER NOT NULL DEFAULT 0,
+		listener_count   INTEGER NOT NULL DEFAULT 0,
+		duration         INTEGER NOT NULL DEFAULT 0,
+		caa_release_mbid BLOB NOT NULL DEFAULT x'',
+		release_name     TEXT NOT NULL DEFAULT '',
+		primary_type     TEXT NOT NULL DEFAULT '',
+		secondary_types  TEXT NOT NULL DEFAULT '',
+		release_date     TEXT NOT NULL DEFAULT '',
+		artist_type      TEXT NOT NULL DEFAULT '',
+		country          TEXT NOT NULL DEFAULT '',
+		disambiguation   TEXT NOT NULL DEFAULT '',
+		sort_name        TEXT NOT NULL DEFAULT '',
+		discog_fetched   INTEGER NOT NULL DEFAULT 0,
+		PRIMARY KEY (mbid)
+	)`); err != nil {
+		t.Fatalf("create artifact table: %v", err)
+	}
+
+	if _, err := db.Exec(
+		`CREATE TABLE artifact_meta (key TEXT PRIMARY KEY, value TEXT NOT NULL)`,
+	); err != nil {
+		t.Fatalf("create artifact meta: %v", err)
+	}
+
+	for k, v := range validMeta() {
+		if _, err := db.Exec(
+			"INSERT INTO artifact_meta (key, value) VALUES (?, ?)", k, v,
+		); err != nil {
+			t.Fatalf("write artifact meta: %v", err)
+		}
+	}
+
+	if _, err := db.Exec(`
+		INSERT INTO explore_index (entity_type, mbid, title, artist_name, artist_mbid, popularity)
+		VALUES (1, ?, 'Artist A', 'Artist A', ?, 5000)`,
+		mbidBytes(artA), mbidBytes(artA),
+	); err != nil {
+		t.Fatalf("write artifact row: %v", err)
+	}
+
+	_ = db.Close()
 
 	live := database.NewTestDB(t)
 	si := NewSearchIndex(live, nil, nil, testLogger())
@@ -851,53 +748,3 @@ func TestImportCoreArtifactWithoutCredits(t *testing.T) {
 		t.Errorf("credit refs = %d, want 0", refs)
 	}
 }
-
-// TestArtifactKeyBindsInTheArtifactsOwnEncoding pins the one place the
-// batch walk's comparison type is decided.
-//
-// Every wrong answer is silent, which is why it is worth pinning all
-// four.  SQLite does not coerce TEXT to BLOB and orders every blob after
-// every text value, so a text key against a byte column makes
-// `mbid > ?` true of the whole artifact - the cursor never advances and
-// the walk spins forever without merging a row - while a byte key
-// against a text column makes it false of the whole artifact, so every
-// batch merges nothing and the import "succeeds" empty.  An unset cursor
-// is the same fault once more: database/sql converts a nil []byte to
-// SQL NULL, and `mbid > NULL` matches no row at all.
-func TestArtifactKeyBindsInTheArtifactsOwnEncoding(t *testing.T) {
-	raw := mbidBytes(artA)
-
-	for _, tt := range []struct {
-		name string
-		key  artifactKey
-		want []byte
-	}{
-		{"unset", nil, []byte{}},
-		{"set", artifactKey(raw), raw},
-	} {
-		t.Run("bytes/"+tt.name, func(t *testing.T) {
-			got, ok := tt.key.bind(false).([]byte)
-			if !ok {
-				t.Fatalf("bind(false) = %T, want []byte", tt.key.bind(false))
-			}
-
-			if got == nil {
-				t.Fatal("bound to SQL NULL, which matches no row")
-			}
-
-			if !bytes.Equal(got, tt.want) {
-				t.Errorf("bind(false) = %x, want %x", got, tt.want)
-			}
-		})
-	}
-
-	// The dashed form is what an artifact published before the storage
-	// change carries, and it has to compare as text against text.
-	if got := artifactKey(nil).bind(true); got != "" {
-		t.Errorf("bind(true) on an unset cursor = %#v, want an empty string", got)
-	}
-
-	if got := artifactKey(artA).bind(true); got != artA {
-		t.Errorf("bind(true) = %#v, want %q", got, artA)
-	}
-}

e2e/specs/jobs-on-a-phone.spec.ts

@@ -60,23 +60,6 @@ const PHONE = { width: 424, height: 439 };
 const DESKTOP = { width: 1100, height: 800 };
 
 test.describe('background jobs on a phone', () => {
-  /**
-   * **State a spec stages is the spec's to clear.** `/__test/emit` writes
-   * to a store nothing resets, so the event that staged a job is the
-   * event that clears it — `JobStore` replaces its whole list from every
-   * snapshot, so `testctl` needs no special case.
-   *
-   * **Measured on #168: this does not currently outlive the page.** Every
-   * test gets a fresh page, and `JobStore.init()` refetches `GetJobs()`
-   * from a backend registry that `/__test/emit` never writes to, so the
-   * staged job is gone before the next spec starts. Ownership is stated
-   * rather than a live leak repaired — the leak needs a page that
-   * survives its own spec, and there is none today.
-   */
-  test.afterEach(async ({ testctl }) => {
-    await testctl.emit('JobsChanged', []);
-  });
-
   test('are shown in the band, without opening anything', async ({
     app,
     testctl,

e2e/specs/top-bar-fit.spec.ts

@@ -102,22 +102,6 @@ const collapsed = (page: Page) =>
   }));
 
 test.describe('the top bar fits the window', () => {
-  /**
-   * **State a spec stages is the spec's to clear** (#168). `/__test/emit`
-   * writes to a store nothing resets, and this file stages the widest job
-   * in the app, so it puts it back — with the same event, since the store
-   * replaces its whole list from every snapshot.
-   *
-   * **Measured: it does not currently outlive the page.** Every test gets
-   * a fresh page and `JobStore.init()` refetches `GetJobs()` from a
-   * backend registry `/__test/emit` never writes to, so nothing is being
-   * repaired here; the rule is stated because it costs one line and the
-   * leak would need only one spec that keeps a page alive.
-   */
-  test.afterEach(async ({ testctl }) => {
-    await testctl.emit('JobsChanged', []);
-  });
-
   /**
    * The phone's answer, which is not "it fits" (#57).
    *

go.mod

@@ -1,6 +1,6 @@
 module yellowjacket
 
-go 1.25.0
+go 1.26
 
 require (
 	github.com/BurntSushi/toml v1.6.0
@@ -144,7 +144,7 @@ require (
 	github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 // indirect
 	github.com/go-git/go-billy/v5 v5.9.0 // indirect
 	github.com/go-git/go-git/v5 v5.19.2 // indirect
-	github.com/go-json-experiment/json v0.0.0-20251027170946-4849db3c2f7e // indirect
+	github.com/go-json-experiment/json v0.0.0-20260820222146-c27c302e5fc3 // indirect
 	github.com/go-ole/go-ole v1.3.0 // indirect
 	github.com/go-resty/resty/v2 v2.17.1 // indirect
 	github.com/go-sql-driver/mysql v1.9.3 // indirect

go.sum

@@ -362,8 +362,8 @@ github.com/go-git/go-git/v5 v5.19.2/go.mod h1:QqCBE1EFN5ddFmrliLQ3/ntRCUjZU3EJuw
 github.com/go-gl/glfw v0.0.0-20190409004039-e6da0acd62b1/go.mod h1:vR7hzQXu2zJy9AVAgeJqvqgH9Q5CA+iKCZ2gyEVpxRU=
 github.com/go-gl/glfw/v3.3/glfw v0.0.0-20191125211704-12ad95a8df72/go.mod h1:tQ2UAYgL5IevRw8kRxooKSPJfGvJ9fJQFa0TUsXzTg8=
 github.com/go-gl/glfw/v3.3/glfw v0.0.0-20200222043503-6f7a984d4dc4/go.mod h1:tQ2UAYgL5IevRw8kRxooKSPJfGvJ9fJQFa0TUsXzTg8=
-github.com/go-json-experiment/json v0.0.0-20251027170946-4849db3c2f7e h1:Lf/gRkoycfOBPa42vU2bbgPurFong6zXeFtPoxholzU=
-github.com/go-json-experiment/json v0.0.0-20251027170946-4849db3c2f7e/go.mod h1:uNVvRXArCGbZ508SxYYTC5v1JWoz2voff5pm25jU1Ok=
+github.com/go-json-experiment/json v0.0.0-20260820222146-c27c302e5fc3 h1:UADEEmDKgfXbtnGJZ97beY5XLo9ZechG1nlU4KnRrkE=
+github.com/go-json-experiment/json v0.0.0-20260820222146-c27c302e5fc3/go.mod h1:tphK2c80bpPhMOI4v6bIc2xWywPfbqi1Z06+RcrMkDg=
 github.com/go-kit/kit v0.8.0/go.mod h1:xBxKIO96dXMWWy0MnWVtmwkA9/13aqxPnvrjFYMA2as=
 github.com/go-kit/kit v0.9.0/go.mod h1:xBxKIO96dXMWWy0MnWVtmwkA9/13aqxPnvrjFYMA2as=
 github.com/go-kit/log v0.1.0/go.mod h1:zbhenjAZHb184qTLMA9ZjW7ThYL0H2mk7Q6pNt4vbaY=

packaging/arch/PKGBUILD

@@ -18,7 +18,7 @@ arch=('x86_64')
 url="https://git.ljones.me/yonlu/yellowjacket"
 license=('custom')
 depends=('webkitgtk-6.0' 'gtk4' 'alsa-lib' 'hicolor-icon-theme')
-makedepends=('go>=1.25' 'nodejs>=22' 'pnpm' 'git')
+makedepends=('go>=1.26' 'nodejs>=22' 'pnpm' 'git')
 options=('!lto')
 
 # Source is overridable so the same PKGBUILD works two ways:

scripts/skill-check.sh

@@ -82,69 +82,23 @@ targets="$({ make -pqRr 2>/dev/null || true; } |
 # happened to break there, and a check that fails on reflow gets
 # disabled rather than fixed.
 #
-# **An inline span may be hard-wrapped, and then the mention is split
-# across two lines.**  `make` at the end of one line and its target at
-# the start of the next is one code span to Markdown and two strings to
-# a per-line regex, so the target was invisible — and these docs are
-# mostly hard-wrapped prose, so the wrap is what the author does not
-# think about.  Lines are therefore joined while the span is still open,
-# which is what an odd number of backticks means.
-#
-# Joining re-opens the reflow trap above unless it is bounded, so it is
-# bounded three ways: a fence flushes first (a fenced command is already
-# whole, and joining inside one would break the line-start rule), a
-# blank line flushes (CommonMark does not allow a blank line inside a
-# code span, so nothing legitimate is split by one), and so does a file
-# boundary.  A stray odd backtick in prose therefore costs one paragraph
-# of over-matching rather than the rest of the file.
-#
 # AGENTS.md is deliberately not in this list: it is a symlink to
 # CLAUDE.md, asserted above, so scanning it would report every failure
 # twice under two names.
 mentioned="$(printf '%s\n' "$docs" |
 	xargs awk '
-		function scan(text,    rest) {
-			rest = text
+		FNR == 1 { fence = 0 }
+		/^```/   { fence = !fence; next }
+		{
+			rest = $0
 			while (match(rest, /`make [a-z][a-z0-9-]*/)) {
 				print substr(rest, RSTART + 6, RLENGTH - 6)
 				rest = substr(rest, RSTART + RLENGTH)
 			}
-		}
-
-		function lineStart(text) {
-			if (match(text, /^make [a-z][a-z0-9-]*/)) {
-				print substr(text, 6, RLENGTH - 5)
+			if (fence && match($0, /^make [a-z][a-z0-9-]*/)) {
+				print substr($0, 6, RLENGTH - 5)
 			}
 		}
-
-		function ticks(s,    n, i) {
-			n = 0
-			for (i = 1; i <= length(s); i++) {
-				if (substr(s, i, 1) == "`") n++
-			}
-			return n
-		}
-
-		function flush() {
-			if (buf == "") return
-			scan(buf)
-			if (fence) lineStart(buf)
-			buf = ""
-		}
-
-		FNR == 1 { flush(); fence = 0 }
-
-		/^```/ { flush(); fence = !fence; next }
-
-		/^[[:space:]]*$/ { flush(); next }
-
-		{
-			if (fence) { scan($0); lineStart($0); next }
-			buf = (buf == "" ? $0 : buf " " $0)
-			if (ticks(buf) % 2 == 0) flush()
-		}
-
-		END { flush() }
 	' | sort -u)"
 
 missing=""
@@ -159,16 +113,7 @@ if [ -n "$missing" ]; then
 	echo "skill-check: the docs name make targets that do not exist:" >&2
 	for t in $missing; do
 		echo "  make $t" >&2
-		# `make <t>` on one line first, because that is where a target is
-		# normally named and it is the precise answer.  The bare name is the
-		# fallback, and it exists because the parser above can now find a
-		# mention that *this* grep cannot: a wrapped span has `make` and its
-		# target on different lines.  Without it a missing target reported no
-		# file at all, and `set -o pipefail` turned the empty grep into exit
-		# 123, before the line telling the author what to do.
-		hits="$(printf '%s\n' "$docs" | xargs grep -ln "make $t" 2>/dev/null || true)"
-		[ -n "$hits" ] || hits="$(printf '%s\n' "$docs" | xargs grep -ln -- "$t" 2>/dev/null || true)"
-		[ -n "$hits" ] && printf '%s\n' "$hits" | sed 's/^/      /' >&2
+		printf '%s\n' "$docs" | xargs grep -ln "make $t" | sed 's/^/      /' >&2
 	done
 	echo "Fix the docs, or restore the target." >&2
 	exit 1