Artifact Description [84dce9672e]

Artifact 84dce9672ec1413377366837ff7a78c725aaf8d44ccf3f532800ab613f0eb105:

• File db/db.go
  • 2026-10-19 23:10:28 — part of check-in [69c95d4419] on branch feat/cli-snapshot — fix(db): set busy_timeout before any other pragma on open

    The drivers build the DSN's _pragma parameters from a map and run them in that order, so journal_mode or foreign_keys often ran before busy_timeout. An open while another process held a lock then failed at once with "database is locked" instead of waiting: one cause of TestSnapshotConcurrentProcesses failing under -tags test_ncruces. (user: [email protected], size: 16324) [annotate] [blame] [check-ins using]

  • 2026-10-20 01:30:24 — part of check-in [90345545d3] on branch main — feat(cli): checkout-free snapshot, per-path log, cat --at and diff --json (#453)

    * feat(cli): checkout-free snapshot, per-path log, cat --at and diff --json

    repo snapshot commits one file's bytes onto trunk with no checkout, a no-op when they equal the tip. repo log --path lists the check-ins that changed a path, repo cat --at reads a file as a version holds it, and repo diff --json A B [PATH] diffs two versions. Exit 2 means not found. A snapshot re-checks the check-in stamp inside its write transaction and retries, so concurrent processes build one linear trunk.

    Tracked in #451

    * test(cli): cover log across a rename; order exit statuses in the docs

    Tracked in #451

    * fix(db): convert Julian days to the millisecond SQLite gives

    log --json printed most check-in times 1ms before their D card: db.julianToTime truncated where SQLite's date.c rounds with (sqlite3_int64)(r*86400000.0+0.5). fsltype.MillisFromJulianDay now does exactly that, JulianToTime and ScanTime use it, and ScanTime rounds the time.Time ncruces hands back to the millisecond as well.

    * fix(cli): print usage errors on stderr and pin their exit status 80

    The documented status was right for the binary; go run turns it into 1. Usage text went to stdout, so an embedder reading stdout saw it on a failure. The doc now says which status a measurement under go run shows, and a test pins 80 for an unknown flag, a missing flag or argument and an unknown subcommand.

    * fix(cli): refuse a snapshot path holding a control byte or a reserved name

    validateRepoPath now applies fossil's own rules for a new file name: file_is_simple_pathname in strict form (deck.StrictPathname), no byte below 0x20, and file_is_reserved_name, whose existing port in deck is exported as IsReservedName. Each refusal exits 1 and names the reason.

    * fix(cli): diff A B PATH matches fossil's PATH rules

    A PATH in neither version printed [] and exited 0; fossil diff --from A --to B fails with "not found". It now exits 2 (not found), and PATH also matches a directory, as fossil's file_dir_match does. The test runs fossil on the same repository as the oracle.

    * fix(cli): refuse a snapshot into a repository with no main-branch check-in

    When the main branch resolved to nothing, snapshot took the repository for empty and committed a second root beside the existing history. It now treats only a repository with no check-ins as empty, and otherwise fails at exit 1 naming the branch.

    * test(cli): compare log across a rename with fossil finfo

    The rename test built its history through manifest.Checkin and checked our own idea of the result. It now builds the history with the fossil binary and compares repo log with fossil finfo on the same repository: both stop a name's history at the rename. The doc tells an embedder how to follow a file across renames, and where log differs from finfo.

    * fix(db): set busy_timeout before any other pragma on open

    The drivers build the DSN's _pragma parameters from a map and run them in that order, so journal_mode or foreign_keys often ran before busy_timeout. An open while another process held a lock then failed at once with "database is locked" instead of waiting: one cause of TestSnapshotConcurrentProcesses failing under -tags test_ncruces.

    * fix(cli): hold the write lock from snapshot's read of the tip to its commit

    Under -tags test_ncruces, TestSnapshotConcurrentProcesses failed 8 of 20 runs. Besides the open-time lock error fixed in db, a snapshot read the tip's manifest outside any transaction while another process's check-in re-deltaed it, and failed with a hash mismatch. A snapshot now runs in one bound write transaction from the read of the tip to its commit, so no other check-in can land in between; the optimistic retry loop it replaces is gone. (user: [email protected], size: 16324) [annotate] [blame] [check-ins using]

  • 2026-10-20 01:31:37 — part of check-in [7c834d64ec] on branch fix/verify-leaf-rule — Merge remote-tracking branch 'origin/main' into fix/verify-leaf-rule (user: [email protected], size: 16324) [annotate] [blame] [check-ins using]