Skip to content
zit Artifact [de05152541]
Login

Artifact [de05152541]

Artifact de05152541a83748861b77a338772031591e995d4bca0381ebcd7036c9bf5580:

Manifest of check-in [de05152541] - B5: an add-only merge registers its file, shows as pending, and commits

A merge whose only effect was an ADDED file was silently lost. `merge` wrote the incoming file to the working tree but never inserted it into `.zit-managed`, so the file was simultaneously on disk and untracked: `status` said "nothing to report", `extras` listed it as junk, and `commit` refused with "nothing to commit (no changes)" — dropping the merge parent that `merge` had recorded in `.zit-checkout`. The branch existed on exactly one disk and `undo` was the only escape.

No new state was needed. CheckoutState.merge_parent already carries the pending merge (src/checkout.zig), runMerge already records it, and commit already consumes it as a two-token P card and clears it. The gap was entirely in the managed set and in two refusals that did not know about it.

applyMerge now DERIVES the membership mutation from the same entry stream that drives the filesystem effects, rather than accepting a parallel list from the caller: a `.write` registers its path (insert is idempotent, so an ordinary edit-merge is unaffected) and a `.delete` deregisters it. One stream, one truth. Deriving it there rather than in computeAndApplyMerge also fixes the symmetric adjacent bug for free — a merge-deleted file used to stay managed and invent a phantom "missing" that commit then refused over — and covers cherrypick and backout, which share the same seam. The pre-merge set joins the in-process rollback images so a mid-apply failure restores membership along with the files and the checkout; the undo journal already snapshotted `.zit-managed`, so abort and undo deregistration fall out unchanged.

The set is read twice rather than rebuilt from `old.iterate()`: the on-disk record carries a pending `mv` rename beyond the bare membership, and rebuilding would have silently dropped it.

status now names a pending merge/cherrypick/backout in its header block. A merge whose file effects are already materialized was otherwise indistinguishable from a clean tree, with no way to see that a second parent was queued.

commit's no-op refusal no longer fires while a merge or special op is pending. A convergent-content merge — both sides making the same edit — composes to F cards identical to the parent's, so `unchanged` went true and the pending merge wedged with no commit path at all. The P/Q card is itself the change.

Test-expectation change: the new "zit add of a merge-added file is idempotent" case asserted the "already tracked" notice on stdout. That notice has always been a stderr diagnostic (three shipped e2e cases assert it there); the test had the stream wrong, not the product. The assertion's intent — add sees the merge-registered file as already tracked and does not re-add it — is unchanged.

No technique was borrowed here: the mechanism rides zit's existing applyMerge, ManagedSet and journal seams. Fossil's vmerge/vfile registration was consulted as an analog for the SHAPE of the problem only, and none of its code or algorithm is reproduced.

Verified in this worktree: fmt, build, zig build test, full zig build e2e (420 tests, all eight REGRESSION B5 cases green, contract-partner undo guard still green), and smoke S1-S6. The blocker repro was also driven by hand against the rebuilt binary: merge reports 1 file changed, status shows "pending merge with <hash>" and "added feat.txt", extras is empty, commit lands, and info tip shows both parents. by dmestas on 2026-07-19 15:09:07.


C B5:\san\sadd-only\smerge\sregisters\sits\sfile,\sshows\sas\spending,\sand\scommits\n\nA\smerge\swhose\sonly\seffect\swas\san\sADDED\sfile\swas\ssilently\slost.\s`merge`\swrote\nthe\sincoming\sfile\sto\sthe\sworking\stree\sbut\snever\sinserted\sit\sinto\s`.zit-managed`,\nso\sthe\sfile\swas\ssimultaneously\son\sdisk\sand\suntracked:\s`status`\ssaid\s"nothing\sto\nreport",\s`extras`\slisted\sit\sas\sjunk,\sand\s`commit`\srefused\swith\s"nothing\sto\ncommit\s(no\schanges)"\s—\sdropping\sthe\smerge\sparent\sthat\s`merge`\shad\srecorded\sin\n`.zit-checkout`.\sThe\sbranch\sexisted\son\sexactly\sone\sdisk\sand\s`undo`\swas\sthe\sonly\nescape.\n\nNo\snew\sstate\swas\sneeded.\sCheckoutState.merge_parent\salready\scarries\sthe\spending\nmerge\s(src/checkout.zig),\srunMerge\salready\srecords\sit,\sand\scommit\salready\nconsumes\sit\sas\sa\stwo-token\sP\scard\sand\sclears\sit.\sThe\sgap\swas\sentirely\sin\sthe\nmanaged\sset\sand\sin\stwo\srefusals\sthat\sdid\snot\sknow\sabout\sit.\n\napplyMerge\snow\sDERIVES\sthe\smembership\smutation\sfrom\sthe\ssame\sentry\sstream\sthat\ndrives\sthe\sfilesystem\seffects,\srather\sthan\saccepting\sa\sparallel\slist\sfrom\sthe\ncaller:\sa\s`.write`\sregisters\sits\spath\s(insert\sis\sidempotent,\sso\san\sordinary\nedit-merge\sis\sunaffected)\sand\sa\s`.delete`\sderegisters\sit.\sOne\sstream,\sone\struth.\nDeriving\sit\sthere\srather\sthan\sin\scomputeAndApplyMerge\salso\sfixes\sthe\ssymmetric\nadjacent\sbug\sfor\sfree\s—\sa\smerge-deleted\sfile\sused\sto\sstay\smanaged\sand\sinvent\sa\nphantom\s"missing"\sthat\scommit\sthen\srefused\sover\s—\sand\scovers\scherrypick\sand\nbackout,\swhich\sshare\sthe\ssame\sseam.\sThe\spre-merge\sset\sjoins\sthe\sin-process\nrollback\simages\sso\sa\smid-apply\sfailure\srestores\smembership\salong\swith\sthe\sfiles\nand\sthe\scheckout;\sthe\sundo\sjournal\salready\ssnapshotted\s`.zit-managed`,\sso\sabort\nand\sundo\sderegistration\sfall\sout\sunchanged.\n\nThe\sset\sis\sread\stwice\srather\sthan\srebuilt\sfrom\s`old.iterate()`:\sthe\son-disk\nrecord\scarries\sa\spending\s`mv`\srename\sbeyond\sthe\sbare\smembership,\sand\srebuilding\nwould\shave\ssilently\sdropped\sit.\n\nstatus\snow\snames\sa\spending\smerge/cherrypick/backout\sin\sits\sheader\sblock.\sA\smerge\nwhose\sfile\seffects\sare\salready\smaterialized\swas\sotherwise\sindistinguishable\sfrom\na\sclean\stree,\swith\sno\sway\sto\ssee\sthat\sa\ssecond\sparent\swas\squeued.\n\ncommit's\sno-op\srefusal\sno\slonger\sfires\swhile\sa\smerge\sor\sspecial\sop\sis\spending.\nA\sconvergent-content\smerge\s—\sboth\ssides\smaking\sthe\ssame\sedit\s—\scomposes\sto\sF\ncards\sidentical\sto\sthe\sparent's,\sso\s`unchanged`\swent\strue\sand\sthe\spending\smerge\nwedged\swith\sno\scommit\spath\sat\sall.\sThe\sP/Q\scard\sis\sitself\sthe\schange.\n\nTest-expectation\schange:\sthe\snew\s"zit\sadd\sof\sa\smerge-added\sfile\sis\sidempotent"\ncase\sasserted\sthe\s"already\stracked"\snotice\son\sstdout.\sThat\snotice\shas\salways\nbeen\sa\sstderr\sdiagnostic\s(three\sshipped\se2e\scases\sassert\sit\sthere);\sthe\stest\shad\nthe\sstream\swrong,\snot\sthe\sproduct.\sThe\sassertion's\sintent\s—\sadd\ssees\sthe\nmerge-registered\sfile\sas\salready\stracked\sand\sdoes\snot\sre-add\sit\s—\sis\sunchanged.\n\nNo\stechnique\swas\sborrowed\shere:\sthe\smechanism\srides\szit's\sexisting\sapplyMerge,\nManagedSet\sand\sjournal\sseams.\sFossil's\svmerge/vfile\sregistration\swas\sconsulted\nas\san\sanalog\sfor\sthe\sSHAPE\sof\sthe\sproblem\sonly,\sand\snone\sof\sits\scode\sor\nalgorithm\sis\sreproduced.\n\nVerified\sin\sthis\sworktree:\sfmt,\sbuild,\szig\sbuild\stest,\sfull\szig\sbuild\se2e\s(420\ntests,\sall\seight\sREGRESSION\sB5\scases\sgreen,\scontract-partner\sundo\sguard\sstill\ngreen),\sand\ssmoke\sS1-S6.\sThe\sblocker\srepro\swas\salso\sdriven\sby\shand\sagainst\sthe\nrebuilt\sbinary:\smerge\sreports\s1\sfile\schanged,\sstatus\sshows\s"pending\smerge\swith\n<hash>"\sand\s"added\sfeat.txt",\sextras\sis\sempty,\scommit\slands,\sand\sinfo\stip\sshows\nboth\sparents.\n
D 2026-07-19T15:09:07.784
F .fossil-settings/ignore-glob 3580f973d7097226f91a27c7b61ec13d9d07880d95549f9b90ce86da5d867a23
F .github/workflows/ci.yml 6d343370ac09cc6826509b7c22e045fd7c8b2272504ced464c382a24588d0878
F LICENSE 702ae7ff835a2a971dae11c5cae039803bf394b77a6d8ec9f18693e449eaaf26
F README.md 62e1f936cc2ea0e33d717539bdffd6126a639843069bd6256359f2b15cb06232
F build.zig 43c12a0eaffed3817646b3021862ee1add3b8536a85b531b7cf55319c3b0e95e
F build.zig.zon 0eade544d4d7fc5a620348c6f29cc9219ff8c6d6377a05e756ca89860a524781
F docs/rfcs/draft-fossil-artifact-format-00.txt a6d542a5effeb41d0ea1e916058ccdc0a5819787f167ef1ebad7814f5680e7ad
F docs/rfcs/draft-fossil-delta-format-00.txt d8025d6d75322e33887c3334a4b9cc729960028b3c994ed01e5a474e4b388fd2
F docs/rfcs/draft-fossil-repo-state-00.txt fd413bcb28001a8a07009ca0c769cfe666394ba79dd4f14a6461f2b592049595
F docs/rfcs/draft-fossil-sync-protocol-00.txt aa55791464a54ef4b651df8780ba5c46e95c723c7997d09a5b727fef658f920a
F scripts/mirror-export.sh cc1130129e3c320cb9c1c5a05f505e45ece23d6bb3e0228da8e7ffe97bd5612b x
F src/annotate.zig 0a6b6728099e63c84681d1e5228ebb45fafb55d65df1720358247fd6e6532c63
F src/artifact.zig c630afd9fbbcfaccd7b823105b4d05db1dc85b6af03080a6d58635ed064672ca
F src/artifact_fuzz.zig 8bb359ddfe790818e109a1dd7ac0c1b5cec1a8fdc556f37e73504cdb7f933e17
F src/artifact_test.zig da999476660f7e6e1f50ed9da5c30f1c83d6b358e2cfcd265c6932959c85bdd1
F src/bisect.zig 7bfc45d3e0e339017826941463cc4e9118892d2898fbcd319e6e6ca85c4d3301
F src/checkout.zig ac48e76f22c8065f70d2a827ae00ffb99887af80950a93c1a3451a9a1ad0f403
F src/ci_test.zig 98f79b658d4840ba4807a79b5cc697cd1417015b2fc6521d6b21ba283310cf86
F src/cli.zig ae2ec7787850f9969ba32c4d0453e5eb1359eefe25f1cc2b252ecd95735840f6
F src/container.zig cef1028531f2763f94248b761c8779a1884e44bb854b8c45617494e3ead40e61
F src/container_test.zig a013652b1ed181161d3ad8fc9d8f052f7c27b61e129662632ceaf8cf189445ec
F src/delta.zig e8f3a6297d47549aea146222732ccf514c39688414f0a74ba396ac91066686f3
F src/delta_fuzz.zig 9e0d8dbb8c82ca0ba1a80974c076e74e4f37072b21a10812efd5539875904719
F src/delta_test.zig b18d97e117dce8fbcc05bff6f6954effdb946bab6d36337efbe5ef64d10646a0
F src/dst.zig 7a2e9e522066e5e052090da4b175f8fa5ea51452d30c019b5f46079034fd2f79
F src/e2e.zig 21ef1bd6d07720dd4b7ff0dfebe01b7ffa1e249f61fa0eec30ae69a06238df3d
F src/extras.zig b8b36b7d6a3d348eeaf733124e671de1dba0bd4fe3cf2acd35268dfb3cf6acfb
F src/fuzz_tests.zig 1074473ceb2aba56872e79607ecd4eb19f07be0d2c6c749e19a02ee2db8570d9
F src/logstore.zig a26e7b006f335b0bbb9bb41ac702ea42fab1c5b4ce9b13c7d25a89c84197aed0
F src/logstore_benchmark.zig e245474e1cc4f10617866f718aab3958f13b256c609b3e5fd562f500668388c0
F src/logstore_fuzz.zig 67b7bda806397f92c3b07c94854d9fafca56710090071a1c1c8ec94d137408f8
F src/logstore_test.zig a9baaee2cf009c988832d3632f52a1eeabf36afd52c891315f7673ce28442533
F src/main.zig e69633ce617fc53a3c3970b2c8a81868de6dab85482c3fb4fae4bd5db59c459d
F src/managed.zig 81ccc2d6321fe3dff91f653305ae4e73bea0b3fae7db2ab2f0cf168d9518d601
F src/merge3.zig 7fc52445688751bb8af30462549650660bcdbadabef54d892b6505d628248d0b
F src/merge3_fuzz.zig 83dcd4d9f2f9c208baa15e79ceeea80a0ebf47456a9b045f6d1c1efe3a927582
F src/pack.zig 0ea96cb540b3f43f6ff8ce95a3044ce939879bc6f9bf0eb82b26aab4089009c7
F src/packaging_test.zig b802ada92f1ade1b62e3d1cfc36549c9b8d05be5430bd33d440bc93d7cd3600f
F src/repocache.zig aae232a6b1b86e27c5242c0ee17ce9f2544299d99cf4936d8c2c9000e3166884
F src/repoimage.zig 7d2dc2f3c145e10e2d9ba6997705fd61a0cc0aedde8e171c253163d1279ccdd5
F src/repository.zig 79ffa239b0a37eea0b3161040f045d6ca8c656331c29925f078ac87213a12476
F src/repository_test.zig 17bdf4c7042f846f928570a47821035f936238f70f3d61d8d2ea3729d5f58868
F src/repostate.zig b4944efab53752f12da2871b2bc46251dbdacd4fcba27e38c61c4fe4d40d1df1
F src/repostate_test.zig e5699688b0f43b7480ed5d89f11585c2f61114bbe3ec9dae5061741f37b37e7a
F src/repoview.zig ec2958e1e8fd955c761d240ab80c3b3f62e3473991713c25ed5a603df984e403
F src/rfc_vectors/artifact_a1_1_sha1_manifest.txt 82d8fc78b3fbf85efe80cb05e79db40c910bf0551e55b728c6f9c2ca7a42ddb6
F src/rfc_vectors/artifact_a1_2_sha3_technote.txt 832e850f83aab2e2a754aa97591109f7f05fb41e4126c7e71c88a59d6d17e55d
F src/rfc_vectors/artifact_a3_rcard_manifest.txt 47e412ef211f095e93275fadf527909e581eef42357d8437fc7e299bfc3e62a7
F src/rfc_vectors/sync_a1_round1_reply.txt 638813029b10f81303b88859279afb17095d7c779c993f4267870909acabbeb9
F src/rfc_vectors/sync_a1_round2_file_cards.txt 4fa397741f7521cc4181d5d6b04dc143187f57602384eaa688552115ecda27c2
F src/rfc_vectors/sync_a1_round2_reply.txt af956808190c79b37a384041690c36b78845b471d6c6c01d1cced6e3ee92e6b1
F src/rfc_vectors/sync_a2_reply.txt aa130a127cde210f793c7842960083c765e0ffc7b673374fabc8f0edfe03d763
F src/root.zig c0e74897f57db81c0e4ce57bb992f4fbf52139234317ca00698a2af0c1c1c9b3
F src/serve.zig a072735c12410c63d1d5ace84d7ddf30b84639848ba420207e2432e5b7dcbc27
F src/serve_fuzz.zig 3c39ea768ffa37599e2cc00c588f053870766ab91500aceb0d844a4322174687
F src/sim/faulty_file.zig 8e8a0325d0e987d1dd306588f910af5302e6456561eefae92c73c424fc1a9c94
F src/sim/gen.zig 0e237af570a73bc78e9a717a8ea59867611deb2348287765826ea4ff1942528b
F src/sim/net.zig 8fb057012ce0a33f67c928cd696bc2761fea8ba4012b2daa33871b2ba5dfe772
F src/sim/oracle.zig 04b81d9472a0529d9e9d030779304c3d6c930f4db20b800ae3f9cbc5c178969a
F src/sim/storage_sweep.zig 5472d632347079fb5c7e7ae300b46c6e8202b91202b1c51b181dd3c7bab8a306
F src/sim/sync_gen.zig 238d46942915b5c0e989dfc738b5927b5228468ea123e6f17b6fb593aa5cbf6e
F src/sim/sync_sweep.zig 5458a84be08823d2d0fc9464665f1c3f3f26563185dc9ae24cf73259faa5d40e
F src/smoke.zig d3aecf12565618a6b678eeff534230c88dbe5a02486beafcdbdf1eac9b258292
F src/storage.zig 512205eb629d5af8d50ad336f1ef49f53b455abd52ee2f9b56d20893cafad39f
F src/storage_benchmark.zig 871f00fef1a34a0b2ee7b9656a217a0cf52949fdd710b642a44dc6bf12d6866e
F src/sync.zig b04119b2679887c6705ed4831899ca1911ca20afe0bac865022ac8b7ad879194
F src/sync_benchmark.zig 9a4542401c880750ff6565a2ca9f1d116341d1c5af704ef4a05c2732eeec8849
F src/sync_driver.zig 722ed498ea6b76900661a085d4257e51f6416d43ac1185e8ac15140de265f67e
F src/sync_driver_test.zig b50450d2938fc60aa90cd9067fc45141a965b63dc2194f42d5109919e380d7df
F src/sync_fuzz.zig 3f419ec3e3c0b4ecf99376e46d111c4fae9826d08f9968e649ffdf84d10f9e61
F src/sync_test.zig b47b05d3defeef61a02c9c9817b66288c48430713737c946734dba151bb17e7e
F src/transport.zig 4652b2ab7f43ccd8af5ca6ef7604f8e7cd28353450b1ec90d206b881308f8b5f
F src/transport_fuzz.zig adc4ea5e22f821d868ffa57e3aafd42834f358b26d4b317cd6dbe8c76c6f733b
F src/undo.zig 2e213cbe02118e37b11554620b285deae5dbf5b4d8009ae6dc229b84375b7591
F src/update.zig 4b7b65d372759deed40a5e93b8aad55fa3a499e1d9ef95fadbff7113b7b1ee72
F src/update_test.zig c06e5fb6da0b1b7aa055fb2398bd03844e0dbec67ed97905135db1dffb8a2a14
F src/warmread_benchmark.zig 9421986d49deff3f720e491fb6d1c7162dc9322f4e67dd14198820c3992dc02a
P bc7b91431608ccd856b398c8a5bf6d6dcaf57cbba40ffd1be95967d4b7d4e99d
R 6d8e3655b7b61c86f7db74ac850d89ec
T *branch * blockers
T *sym-blockers *
T -sym-trunk *
U dmestas
Z 76061bb54d33a3572e9dba727cef536a