cli_test.v

1module tests23import os4import time56const root = @VMODROOT7const binary = os.join_path(root, 'build', 'koh3')89struct Result {10mut:11	code int12	out  string13	err  string14}1516fn testsuite_begin() {17	os.mkdir_all(os.join_path(root, 'build')) or { panic(err) }18	build := os.execute('v -o ${q(binary)} ${q(os.join_path(root, 'src', 'koh3.v'))}')19	if build.exit_code != 0 {20		panic('could not build koh3: ${build.output}')21	}22}2324fn q(s string) string {25	return "'${s.replace("'", "'\\''")}'"26}2728fn temp_repo(name string) string {29	dir := os.join_path(os.temp_dir(), 'koh3-cli-${name}-${time.now().unix_nano()}')30	os.mkdir_all(dir) or { panic(err) }31	return dir32}3334fn run(repo string, args ...string) Result {35	home := temp_repo('home')36	defer { os.rmdir_all(home) or {} }37	return run_home(repo, home, ...args)38}3940fn run_home(repo string, home string, args ...string) Result {41	stamp := '${os.getpid()}-${time.now().unix_nano()}'42	base := os.join_path(os.temp_dir(), 'koh3-cli-${stamp}')43	out_path := '${base}.out'44	err_path := '${base}.err'45	code_path := '${base}.code'46	mut command := 'HOME=${q(home)} ${q(binary)}'47	for arg in args {48		command += ' ${q(arg)}'49	}50	shell := 'cd ${q(repo)} && ${command} </dev/null >${q(out_path)} 2>${q(err_path)}; printf "%s" $? >${q(code_path)}'51	result := os.execute(shell)52	if result.exit_code != 0 {53		panic('could not run test command: ${result.output}')54	}55	code := os.read_file(code_path) or { panic(err) }56	out := os.read_file(out_path) or { '' }57	err := os.read_file(err_path) or { '' }58	os.rm(out_path) or {}59	os.rm(err_path) or {}60	os.rm(code_path) or {}61	return Result{62		code: code.int()63		out:  out64		err:  err65	}66}6768fn init_repo(repo string) {69	r := run(repo, 'init')70	assert r.code == 071	assert os.exists(os.join_path(repo, '.koh'))72	assert os.exists(os.join_path(repo, '.kohignore'))73}7475fn write(repo string, path string, text string) {76	full := os.join_path(repo, path)77	os.mkdir_all(os.dir(full)) or { panic(err) }78	os.write_file(full, text) or { panic(err) }79}8081fn read_json_has(text string, key string) {82	if !text.contains(key) {83		panic('missing JSON key ${key}: ${text}')84	}85}8687fn test_init_and_usage() {88	dir := temp_repo('init')89	defer { os.rmdir_all(dir) or {} }90	r := run(dir, 'init', '--json')91	assert r.code == 092	read_json_has(r.out, 'repo')93	read_json_has(r.out, 'ignore_created')94	assert run(dir, 'init').code == 195	assert run(dir, 'unknown-command').code == 296	assert run(dir, 'unknown-command').err.contains('unknown command')97}9899fn test_save_checkpoint_and_status() {100	dir := temp_repo('save')101	defer { os.rmdir_all(dir) or {} }102	init_repo(dir)103	write(dir, 'tracked.txt', 'one\n')104	r := run(dir, 'save', 'first message', '--agent', 'agent-7', '--session', 'session-a', '--json')105	assert r.code == 0106	for key in ['id', 'parent', 'message', 'created_at', 'machine', 'is_checkpoint', 'agent',107		'session', 'files', 'size_bytes'] {108		read_json_has(r.out, key)109	}110	assert r.out.contains('"message":"first message"')111	assert r.out.contains('"agent":"agent-7"')112	assert r.out.contains('"session":"session-a"')113	assert r.out.contains('"is_checkpoint":false')114	assert run(dir, 'save', 'again').code == 1115	assert run(dir, 'save', 'again').err.contains('nothing has changed since the last save')116	assert run(dir, 'save').code == 2117	assert run(dir, 'save', 'x', '--bogus').code == 2118	cp := run(dir, 'checkpoint', 'still first', '--json')119	assert cp.code == 0120	assert cp.out.contains('"is_checkpoint":true')121	assert cp.out.contains('"parent":')122	clean := run(dir, 'status')123	assert clean.code == 0124	assert clean.out.contains('no changes')125	write(dir, 'tracked.txt', 'two\n')126	write(dir, 'new.txt', 'new\n')127	os.rm(os.join_path(dir, 'tracked.txt')) or { panic(err) }128	write(dir, '.kohignore', '*.log\n')129	status := run(dir, 'status')130	assert status.code == 0131	assert status.out.contains('+')132	assert status.out.contains('~') || status.out.contains('-')133	assert status.out.contains('added')134	js := run(dir, 'status', '--json')135	assert js.code == 0136	for key in ['head_id', 'head_msg', 'dirty', 'modified', 'added', 'deleted', 'ignore_changed'] {137		read_json_has(js.out, key)138	}139	assert js.out.contains('"ignore_changed":true')140}141142fn test_status_requires_repo_and_ignore_rules() {143	dir := temp_repo('status')144	defer { os.rmdir_all(dir) or {} }145	assert run(dir, 'status').code == 1146	init_repo(dir)147	write(dir, '.kohignore', '*.log\nbuild/\n')148	write(dir, 'keep.txt', 'keep\n')149	write(dir, 'ignored.log', 'ignore\n')150	write(dir, 'build/output.txt', 'ignore\n')151	assert run(dir, 'save', 'keep').code == 0152	assert run(dir, 'status').out.contains('no changes')153	assert !run(dir, 'status').out.contains('ignored.log')154	assert !run(dir, 'status').out.contains('build/output.txt')155	assert !run(dir, 'status').out.contains('LOG.md')156}157158// LOG.md is a convenience written after the save commits. If it cannot be159// written, the save still happened, and an agent that retried on a failure160// would only be told nothing had changed.161fn test_a_save_succeeds_when_log_md_cannot_be_written() {162	dir := temp_repo('logmd')163	defer { os.rmdir_all(dir) or {} }164	init_repo(dir)165	os.mkdir_all(os.join_path(dir, 'LOG.md')) or { panic(err) }166	write(dir, 'a.txt', 'one\n')167	r := run(dir, 'save', 'first', '--json')168	assert r.code == 0169	assert r.out.starts_with('{')170	assert r.err.contains('LOG.md')171	assert run(dir, 'log', '--json').out.contains('"message":"first"')172}173174// A folder of ignored files can sit where a restored file should go. Koh175// never touches ignored files, so the restore must stop and say so, not drop176// the file inside the folder under a temporary name.177fn test_a_restore_refuses_a_folder_of_ignored_files_in_the_way() {178	dir := temp_repo('dir-in-way')179	defer { os.rmdir_all(dir) or {} }180	init_repo(dir)181	write(dir, 'out', 'a file\n')182	first := run(dir, 'save', 'out is a file', '--json').out.all_after('"id":"')[..12]183	os.rm(os.join_path(dir, 'out')) or { panic(err) }184	write(dir, 'out/build.log', 'ignored\n')185	assert run(dir, 'save', 'out is gone').code == 0186	r := run(dir, 'load', first, '--yes')187	assert r.code == 1188	assert r.err.contains('out')189	assert os.ls(os.join_path(dir, 'out'))! == ['build.log']190}191192// Koh never follows a symlink, so a folder replaced by a link to somewhere193// else is not the repo's to write into: restoring through it would overwrite194// files outside the repo. The refusal comes before anything is recorded.195fn test_a_restore_never_writes_through_a_symlinked_folder() {196	dir := temp_repo('link-in-way')197	outside := temp_repo('link-target')198	defer {199		os.rmdir_all(dir) or {}200		os.rmdir_all(outside) or {}201	}202	init_repo(dir)203	write(dir, 'docs/a.md', 'saved\n')204	first := run(dir, 'save', 'docs', '--json').out.all_after('"id":"')[..12]205	os.rmdir_all(os.join_path(dir, 'docs')) or { panic(err) }206	write(outside, 'a.md', 'precious\n')207	os.symlink(outside, os.join_path(dir, 'docs')) or { panic(err) }208	assert run(dir, 'save', 'docs is a link now').code == 0209	head := run(dir, 'status', '--json').out210	r := run(dir, 'load', first, '--yes')211	assert r.code == 1212	assert r.err.contains('docs')213	assert os.read_file(os.join_path(outside, 'a.md'))! == 'precious\n'214	assert run(dir, 'status', '--json').out == head215}216217// A restore records its save first, then writes files. When the writing stops218// part way, the half-written tree is not somebody's work: saving it would219// record (and offer) a tree that undoes changes nobody asked to undo. Every220// command that would build on it points at revert, which finishes the job.221fn test_a_restore_cut_short_points_at_revert() {222	dir := temp_repo('cut-short')223	defer {224		os.chmod(os.join_path(dir, 'zz'), 0o755) or {}225		os.rmdir_all(dir) or {}226	}227	init_repo(dir)228	write(dir, 'zero', '0\n')229	assert run(dir, 'save', 'zero').code == 0230	write(dir, 'a.txt', 'v1\n')231	write(dir, 'zz/f', 'v1\n')232	assert run(dir, 'save', 'first').code == 0233	write(dir, 'a.txt', 'v2\n')234	write(dir, 'zz/f', 'v2\n')235	assert run(dir, 'save', 'second').code == 0236	// a.txt is written, then the read-only folder stops the restore: the237	// tree is genuinely half one save and half the other.238	os.chmod(os.join_path(dir, 'zz'), 0o555)!239	assert run(dir, 'undo', '--yes').code == 1240	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'v1\n'241	assert os.read_file(os.join_path(dir, 'zz/f'))! == 'v2\n'242	// Even a status that cannot read the tree says what happened.243	os.chmod(os.join_path(dir, 'zz'), 0o000)!244	assert run(dir, 'status').err.contains('koh revert')245	os.chmod(os.join_path(dir, 'zz'), 0o755)!246247	assert run(dir, 'status', '--json').out.contains('"restore_unfinished":true')248	for args in [['save', 'half'], ['checkpoint', 'half'], ['undo', '--yes']] {249		r := run(dir, ...args)250		assert r.code == 1251		assert r.err.contains('koh revert')252	}253	assert run(dir, 'revert', '--yes').code == 0254	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'v1\n'255	assert os.read_file(os.join_path(dir, 'zz/f'))! == 'v1\n'256	assert run(dir, 'status', '--json').out.contains('"restore_unfinished":false')257	write(dir, 'a.txt', 'v3\n')258	assert run(dir, 'save', 'carry on').code == 0259}260261// Doctor clears peek folders left from earlier days, never one somebody may262// be reading right now.263fn test_doctor_sweeps_only_old_peek_folders() {264	dir := temp_repo('peek-sweep')265	// V's temp_dir is /tmp on macOS whatever TMPDIR says, so these live in266	// the real temp directory under names no save could have.267	fresh := os.join_path(os.temp_dir(), 'koh-peek-fresh${os.getpid()}')268	old := os.join_path(os.temp_dir(), 'koh-peek-old${os.getpid()}')269	defer {270		os.rmdir_all(dir) or {}271		os.rmdir_all(fresh) or {}272		os.rmdir_all(old) or {}273	}274	init_repo(dir)275	write(dir, 'a.txt', 'one\n')276	assert run(dir, 'save', 'one').code == 0277	os.mkdir_all(fresh) or { panic(err) }278	os.mkdir_all(old) or { panic(err) }279	two_days_ago := time.now().unix() - 2 * 86400280	os.utime(old, two_days_ago, two_days_ago) or { panic(err) }281	assert run(dir, 'doctor', '--json').code == 0282	assert os.exists(fresh)283	assert !os.exists(old)284}285286// A file saved long ago and ignored since holds work no save has. Loading the287// old save must not write over it; the document says a restore never touches288// ignored files.289fn test_a_restore_never_overwrites_an_ignored_file() {290	dir := temp_repo('ignored-restore')291	defer { os.rmdir_all(dir) or {} }292	init_repo(dir)293	write(dir, '.env', 'KEY=saved long ago\n')294	first := run(dir, 'save', 'with env', '--json').out.all_after('"id":"')[..12]295	write(dir, '.kohignore', '.env\n')296	assert run(dir, 'save', 'ignore env').code == 0297	write(dir, '.env', 'KEY=never saved\n')298	assert run(dir, 'load', first, '--yes').code == 0299	assert os.read_file(os.join_path(dir, '.env'))! == 'KEY=never saved\n'300301	// Where the ignored file is missing, the restore still brings it back.302	other := temp_repo('ignored-restore-missing')303	defer { os.rmdir_all(other) or {} }304	init_repo(other)305	write(other, '.env', 'KEY=saved long ago\n')306	again := run(other, 'save', 'with env', '--json').out.all_after('"id":"')[..12]307	write(other, '.kohignore', '.env\n')308	assert run(other, 'save', 'ignore env').code == 0309	os.rm(os.join_path(other, '.env')) or { panic(err) }310	assert run(other, 'load', again, '--yes').code == 0311	assert os.read_file(os.join_path(other, '.env'))! == 'KEY=saved long ago\n'312}313314// A going-back preview lists paths only. Running diff(1) per file to draw it315// cost 27 s for 5,000 changed files, so it must not run diff at all.316fn test_a_going_back_preview_does_not_run_diff() {317	dir := temp_repo('preview-nodiff')318	home := temp_repo('preview-nodiff-home')319	defer {320		os.rmdir_all(dir) or {}321		os.rmdir_all(home) or {}322	}323	init_repo(dir)324	write(dir, 'a.txt', 'one\n')325	assert run_home(dir, home, 'save', 'one').code == 0326	write(dir, 'a.txt', 'two\n')327	assert run_home(dir, home, 'save', 'two').code == 0328	fake := os.join_path(home, 'bin')329	os.mkdir_all(fake) or { panic(err) }330	marker := os.join_path(home, 'diff-ran')331	os.write_file(os.join_path(fake, 'diff'), '#!/bin/sh\ntouch ${q(marker)}\nexit 2\n') or {332		panic(err)333	}334	os.chmod(os.join_path(fake, 'diff'), 0o755) or { panic(err) }335	assert run_home_with_path(dir, home, fake, 'undo', '--yes').code == 0336	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'one\n'337	assert !os.exists(marker)338}339340// A restore rewrites only what differs. Rewriting every file resets every341// mtime, so the next status has to hash the whole tree again.342fn test_a_restore_leaves_unchanged_files_alone() {343	dir := temp_repo('restore-skip')344	defer { os.rmdir_all(dir) or {} }345	init_repo(dir)346	write(dir, 'a.txt', 'one\n')347	write(dir, 'same.txt', 'never changes\n')348	assert run(dir, 'save', 'one').code == 0349	write(dir, 'a.txt', 'two\n')350	assert run(dir, 'save', 'two').code == 0351	long_ago := i64(1700000000)352	os.utime(os.join_path(dir, 'same.txt'), long_ago, long_ago) or { panic(err) }353	assert run(dir, 'undo', '--yes').code == 0354	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'one\n'355	assert os.file_last_mod_unix(os.join_path(dir, 'same.txt')) == long_ago356}357358// `save` records the executable bit, so `status` has to see it too, or a359// chmod would be invisible until it turned up in a save.360fn test_status_sees_an_executable_bit_change() {361	dir := temp_repo('chmod')362	defer { os.rmdir_all(dir) or {} }363	init_repo(dir)364	write(dir, 'run.sh', 'echo hi\n')365	assert run(dir, 'save', 'script').code == 0366	os.chmod(os.join_path(dir, 'run.sh'), 0o755) or { panic(err) }367	status := run(dir, 'status', '--json')368	assert status.out.contains('"dirty":true')369	assert status.out.contains('"modified":["run.sh"]')370}371372fn test_log_and_diff() {373	dir := temp_repo('history')374	defer { os.rmdir_all(dir) or {} }375	init_repo(dir)376	write(dir, 'a.txt', 'one\n')377	first := run(dir, 'save', 'one', '--session', 'alpha', '--json')378	id1 := first.out.all_after('"id":"')[..12]379	write(dir, 'a.txt', 'two\n')380	second := run(dir, 'save', 'two', '--session', 'beta', '--json')381	id2 := second.out.all_after('"id":"')[..12]382	cp := run(dir, 'checkpoint', 'checkpoint', '--json')383	assert cp.code == 0384	log := run(dir, 'log')385	assert log.code == 0386	assert log.out.contains('▸')387	assert log.out.contains('◐')388	lj := run(dir, 'log', '--json')389	assert lj.code == 0390	assert lj.out.contains('"head_id"')391	assert lj.out.contains('"saves"')392	assert run(dir, 'log', '--since', 'today').out.contains('two')393	session := run(dir, 'log', '--session', 'alpha')394	assert session.out.contains('one')395	assert !session.out.contains('two')396	write(dir, 'a.txt', 'three\n')397	d := run(dir, 'diff')398	assert d.code == 0399	assert d.out.contains('--- a/')400	assert d.out.contains('+++ b/')401	assert run(dir, 'diff', id1).code == 0402	assert run(dir, 'diff', id1, id2).code == 0403	dj := run(dir, 'diff', '--json')404	assert dj.code == 0405	assert dj.out.contains('"from"')406	assert dj.out.contains('"to"')407	assert dj.out.contains('"files"')408	assert dj.out.contains('"change":"modified"')409	assert run(dir, 'diff', 'not-a-real-id').code == 1410	assert run(dir, 'diff', id1, id2, 'third').code == 1411	write(dir, 'blob.bin', '\x00\x01\x02')412	diff_binary := run(dir, 'diff', '--json')413	assert diff_binary.out.contains('"binary":true')414	assert diff_binary.out.contains('Binary files differ')415}416417fn test_undo_revert_and_load() {418	dir := temp_repo('restore')419	defer { os.rmdir_all(dir) or {} }420	init_repo(dir)421	write(dir, 'a.txt', 'one\n')422	one := run(dir, 'save', 'one', '--json').out423	id1 := one.all_after('"id":"')[..12]424	write(dir, 'a.txt', 'two\n')425	write(dir, 'gone.txt', 'gone\n')426	second := run(dir, 'save', 'two', '--json').out427	id2 := second.all_after('"id":"')[..12]428	write(dir, 'a.txt', 'three\n')429	third := run(dir, 'save', 'three', '--json').out430	id3 := third.all_after('"id":"')[..12]431	// Unsaved work — here a file never saved — blocks every going-back but432	// revert, as in Koh 2: a restore must never overwrite what only the disk has.433	write(dir, 'never.txt', 'keep me\n')434	log_before := run(dir, 'log').out435	for args in [['undo', '--yes'], ['load', id1, '--yes'], ['pluck', id2, '--yes']] {436		blocked := run(dir, ...args)437		assert blocked.code == 1438		assert blocked.err.contains('unsaved changes')439	}440	assert run(dir, 'log').out == log_before441	os.rm(os.join_path(dir, 'never.txt')) or { panic(err) }442	write(dir, 'a.txt', 'edited, never saved\n')443	assert run(dir, 'undo', '--yes').err.contains('unsaved changes')444	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'edited, never saved\n'445	write(dir, 'a.txt', 'three\n')446	os.rm(os.join_path(dir, 'gone.txt')) or { panic(err) }447	assert run(dir, 'load', id1, '--yes').err.contains('unsaved changes')448	write(dir, 'gone.txt', 'gone\n')449	undo := run(dir, 'undo', '--yes', '--json')450	assert undo.code == 0451	if !undo.out.contains(id2) {452		panic('undo JSON did not name ${id2}: ${undo.out}')453	}454	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'two\n'455	assert os.exists(os.join_path(dir, 'gone.txt'))456	assert run(dir, 'load', id3, '--yes').code == 0457	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'three\n'458	assert run(dir, 'undo', '2', '--yes').code == 0459	assert run(dir, 'load', id2, '--yes').code == 0460	assert run(dir, 'undo', '99', '--yes').err.contains('nothing before')461	write(dir, 'a.txt', 'dirty\n')462	before := run(dir, 'log')463	refuse := run(dir, 'revert')464	assert refuse.code == 1465	assert refuse.err.contains('needs --yes when not interactive')466	assert run(dir, 'log').out == before.out467	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'dirty\n'468	write(dir, 'untracked.txt', 'remove\n')469	rv := run(dir, 'revert', '--yes')470	assert rv.code == 0471	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'two\n'472	assert !os.exists(os.join_path(dir, 'untracked.txt'))473	assert run(dir, 'revert', '--yes').out.contains('working tree already matches')474	assert run(dir, 'load', '--checkpoint', '--yes').code == 0475	assert run(dir, 'load', id2, '--json').code == 1476	assert run(dir, 'load', id2, '--json').err.contains('error')477	assert run(dir, 'load').code == 2478	assert run(dir, 'load', 'unknown-id', '--yes').code == 1479}480481fn test_flag_and_flagged_log() {482	dir := temp_repo('flag')483	defer { os.rmdir_all(dir) or {} }484	init_repo(dir)485	write(dir, 'flag.txt', 'one\n')486	saved := run(dir, 'save', 'flag me', '--json')487	id := saved.out.all_after('"id":"')[..12]488	flagged := run(dir, 'flag', id, '--reason', 'unsafe', '--json')489	assert flagged.code == 0490	assert flagged.out.contains('"flagged":true')491	assert flagged.out.contains('"reason":"unsafe"')492	list := run(dir, 'flag', '--json')493	assert list.code == 0494	assert list.out.contains(id)495	assert list.out.contains('unsafe')496	assert run(dir, 'log').out.contains('⚑')497	assert run(dir, 'flag', '--clear', id).code == 0498	assert run(dir, 'flag').out.contains('nothing is flagged')499	assert run(dir, 'flag', 'unknown-id').code == 1500	assert run(dir, 'flag').code == 0501}502503fn test_pluck_forward_reverse_dry_run_and_conflict() {504	dir := temp_repo('pluck')505	defer { os.rmdir_all(dir) or {} }506	init_repo(dir)507	write(dir, 'a.txt', 'one\n')508	first := run(dir, 'save', 'one', '--json').out509	id1 := first.all_after('"id":"')[..12]510	write(dir, 'a.txt', 'two\n')511	second := run(dir, 'save', 'two', '--json').out512	id2 := second.all_after('"id":"')[..12]513	assert run(dir, 'load', id1, '--yes').code == 0514	dry := run(dir, 'pluck', id2, '--dry-run')515	assert dry.code == 0516	assert dry.out.contains('a.txt')517	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'one\n'518	assert run(dir, 'pluck', id2, '--yes').code == 0519	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'two\n'520	assert run(dir, 'pluck', '--reverse', id2, '--yes').code == 0521	assert os.read_file(os.join_path(dir, 'a.txt'))! == 'one\n'522	write(dir, 'a.txt', 'different\n')523	assert run(dir, 'pluck', id2, '--dry-run').code == 1524	assert run(dir, 'save', 'different').code == 0525	conflict := run(dir, 'pluck', id2, '--yes')526	assert conflict.code == 1527	assert conflict.err.contains('a.txt')528	assert run(dir, 'pluck').code == 2529	assert run(dir, 'pluck', 'unknown-id').code == 1530}531532fn test_peek_and_doctor() {533	dir := temp_repo('peek-doctor')534	defer { os.rmdir_all(dir) or {} }535	init_repo(dir)536	write(dir, 'peek.txt', 'peek me\n')537	assert run(dir, 'save', 'peekable').code == 0538	id := run(dir, 'log', '--json').out.all_after('"id":"')[..12]539	before_log := run(dir, 'log', '--json').out540	peek := run(dir, 'peek', id, '--json')541	assert peek.code == 0542	peek_path := peek.out.all_after('"path":"').before('"')543	assert peek.out.contains('"id":"${id}"')544	assert os.read_file(os.join_path(peek_path, 'peek.txt'))! == 'peek me\n'545	assert run(dir, 'status').out.contains('no changes')546	assert run(dir, 'log', '--json').out == before_log547	assert run(dir, 'peek').code == 2548	assert run(dir, 'peek', 'unknown-id').code == 1549	backup := os.join_path(dir, 'doctor.backup')550	healthy := run(dir, 'doctor')551	assert healthy.code == 0552	assert healthy.out.contains('is sound')553	report := run(dir, 'doctor', '--json')554	assert report.code == 0555	for key in ['problems', 'objects', 'saves', 'checkpoints', 'cached_files', 'peek_folders_removed',556		'backup'] {557		read_json_has(report.out, key)558	}559	backed := run(dir, 'doctor', '--backup', backup)560	assert backed.code == 0561	assert os.exists(backup)562	assert backed.out.contains(backup)563	assert run(dir, 'doctor', '--backup', backup).code == 1564	assert run(dir, 'peek', id, '--json').code == 0565	// A peek made today is left for whoever is reading it.566	kept := run(dir, 'doctor', '--json')567	assert kept.code == 0568	assert os.exists(peek_path)569	os.rmdir_all(peek_path) or {}570}571572fn test_strip_refuses_without_terminal() {573	dir := temp_repo('strip')574	defer { os.rmdir_all(dir) or {} }575	init_repo(dir)576	write(dir, 'secret.txt', 'keep\n')577	assert run(dir, 'save', 'secret').code == 0578	before_log := run(dir, 'log', '--json').out579	refused := run(dir, 'strip', 'secret.txt')580	assert refused.code == 1581	assert refused.err.contains('needs a terminal')582	assert run(dir, 'log', '--json').out == before_log583	assert os.read_file(os.join_path(dir, 'secret.txt'))! == 'keep\n'584	assert run(dir, 'log', '--json').out.contains('secret')585}586587fn test_saves_and_checkpoints_aliases() {588	dir := temp_repo('aliases')589	defer { os.rmdir_all(dir) or {} }590	init_repo(dir)591	write(dir, 'alias.txt', 'one\n')592	assert run(dir, 'save', 'plain save').code == 0593	assert run(dir, 'checkpoint', 'checkpoint save').code == 0594	saves := run(dir, 'saves')595	assert saves.code == 0596	assert saves.out.contains('plain save')597	assert saves.out.contains('checkpoint save')598	checkpoints := run(dir, 'checkpoints')599	assert checkpoints.code == 0600	assert checkpoints.out.contains('checkpoint save')601	assert !checkpoints.out.contains('plain save')602	json_result := run(dir, 'checkpoints', '--json')603	assert json_result.code == 0604	assert json_result.out.contains('"head_id"')605	assert json_result.out.contains('"saves"')606	assert json_result.out.contains('"is_checkpoint":true')607	assert !json_result.out.contains('"is_checkpoint":false')608}609610fn test_remote_configuration_is_isolated_and_atomic() {611	dir := temp_repo('remote')612	home := temp_repo('remote-home')613	defer {614		os.rmdir_all(dir) or {}615		os.rmdir_all(home) or {}616	}617	empty := run_home(dir, home, 'remote', '--json')618	assert empty.code == 0619	assert empty.out.contains('"configured":false')620	target := os.join_path(home, 'remote-store')621	set := run_home(dir, home, 'remote', 'folder', target)622	assert set.code == 0623	assert os.is_dir(target)624	state := run_home(dir, home, 'remote', '--json')625	assert state.code == 0626	assert state.out.contains('"kind":"folder"')627	assert state.out.contains(target)628	assert run_home(dir, home, 'remote', 's3', 'http://127.0.0.1:1', 'bucket', 'region').code == 1629	assert run_home(dir, home, 'remote', '--json').out.contains(target)630	assert run_home(dir, home, 'remote', 's3', 'http://127.0.0.1:1', 'bucket', 'region').err.contains('KOH_ACCESS_KEY')631	assert run_home(dir, home, 'remote', '--forget').code == 0632	assert run_home(dir, home, 'remote', '--json').out.contains('"configured":false')633}634635// A clone belongs to the repo it was stolen from, whatever its folder is636// called later: renaming it must not start a second repo on the remote.637fn test_a_renamed_clone_keeps_offering_to_its_repo() {638	syncroot := temp_repo('rename-clone')639	home := temp_repo('rename-clone-home')640	defer {641		os.rmdir_all(syncroot) or {}642		os.rmdir_all(home) or {}643	}644	remote := os.join_path(syncroot, 'remote')645	a := os.join_path(syncroot, 'A', 'proj')646	b_parent := os.join_path(syncroot, 'B')647	os.mkdir_all(a) or { panic(err) }648	os.mkdir_all(b_parent) or { panic(err) }649	init_repo(a)650	write(a, 'a.txt', 'from A\n')651	assert run_home(a, home, 'remote', 'folder', remote).code == 0652	assert run_home(a, home, 'save', 'A first').code == 0653	assert run_home(b_parent, home, 'steal', 'proj').code == 0654	renamed := os.join_path(b_parent, 'renamed')655	os.mv(os.join_path(b_parent, 'proj'), renamed) or { panic(err) }656	write(renamed, 'b.txt', 'from B\n')657	assert run_home(renamed, home, 'save', 'B first').code == 0658	assert !os.exists(os.join_path(remote, 'renamed'))659	assert run_home(a, home, 'steal').code == 0660	assert os.read_file(os.join_path(a, 'b.txt'))! == 'from B\n'661}662663fn test_offer_steal_folder_remote() {664	syncroot := temp_repo('sync')665	home := temp_repo('sync-home')666	defer {667		os.rmdir_all(syncroot) or {}668		os.rmdir_all(home) or {}669	}670	remote := os.join_path(syncroot, 'remote')671	a := os.join_path(syncroot, 'A', 'proj')672	b_parent := os.join_path(syncroot, 'B')673	os.mkdir_all(a) or { panic(err) }674	os.mkdir_all(b_parent) or { panic(err) }675	init_repo(a)676	write(a, 'a.txt', 'from A\n')677	assert run_home(a, home, 'remote', 'folder', remote).code == 0678	assert run_home(a, home, 'save', 'A first').code == 0679	clone := run_home(b_parent, home, 'steal', 'proj')680	assert clone.code == 0681	b := os.join_path(b_parent, 'proj')682	assert os.read_file(os.join_path(b, 'a.txt'))! == 'from A\n'683	write(b, 'b.txt', 'from B\n')684	assert run_home(b, home, 'save', 'B first').code == 0685	write(a, 'a2.txt', 'second\n')686	assert run_home(a, home, 'save', 'A second').code == 0687	assert run_home(a, home, 'steal').code == 0688	assert os.exists(os.join_path(a, 'b.txt'))689	assert run_home(b_parent, home, 'steal', '--list').out.contains('proj')690	write(a, 'clash.txt', 'base\n')691	assert run_home(a, home, 'save', 'clash base').code == 0692	assert run_home(b, home, 'steal').code == 0693	write(a, 'clash.txt', 'A version\n')694	assert run_home(a, home, 'save', 'A clash').code == 0695	write(b, 'clash.txt', 'B version\n')696	assert run_home(b, home, 'save', 'B clash').code == 0697	refused := run_home(b, home, 'steal')698	assert refused.code == 1699	assert refused.err.contains('--mine') || refused.err.contains('--theirs')700	b_head := run_home(b, home, 'status', '--json').out.all_after('"head_id":"')[..12]701	theirs := run_home(b, home, 'steal', '--theirs', '--json')702	assert theirs.code == 0703	// The pre-combine tree is off the line now; the way back must be named,704	// or an agent without the history view cannot find it.705	assert theirs.out.contains('"previous_head":"${b_head}"')706	assert os.read_file(os.join_path(b, 'clash.txt'))! == 'A version\n'707	assert run_home(b, home, 'load', b_head, '--yes').code == 0708	assert os.read_file(os.join_path(b, 'clash.txt'))! == 'B version\n'709	assert run_home(b, home, 'load', theirs.out.all_after('"head":"')[..12], '--yes').code == 0710	assert os.read_file(os.join_path(b, 'clash.txt'))! == 'A version\n'711	write(a, 'clash.txt', 'A second\n')712	assert run_home(a, home, 'save', 'A second clash').code == 0713	write(b, 'clash.txt', 'B second\n')714	assert run_home(b, home, 'save', 'B second clash').code == 0715	assert run_home(b, home, 'steal', '--mine').code == 0716	assert os.read_file(os.join_path(b, 'clash.txt'))! == 'B second\n'717	assert run_home(b, home, 'log').out.contains('combine: kept mine for')718}719720// A clone brings in the whole history, flags included; both output forms must721// say which saves came in flagged, and why.722fn test_a_clone_reports_flagged_saves() {723	syncroot := temp_repo('clone-flags')724	home := temp_repo('clone-flags-home')725	defer {726		os.rmdir_all(syncroot) or {}727		os.rmdir_all(home) or {}728	}729	a := os.join_path(syncroot, 'A', 'proj')730	os.mkdir_all(a) or { panic(err) }731	init_repo(a)732	assert run_home(a, home, 'remote', 'folder', os.join_path(syncroot, 'remote')).code == 0733	write(a, 'a.txt', 'one\n')734	id := run_home(a, home, 'save', 'risky', '--json').out.all_after('"id":"')[..12]735	assert run_home(a, home, 'flag', id, '--reason', 'broke the build').code == 0736	assert run_home(a, home, 'offer').code == 0737	for name in ['B', 'C'] {738		os.mkdir_all(os.join_path(syncroot, name)) or { panic(err) }739	}740	json_clone := run_home(os.join_path(syncroot, 'B'), home, 'steal', 'proj', '--json')741	assert json_clone.code == 0742	assert json_clone.out.contains('"flagged":["${id}"]')743	text_clone := run_home(os.join_path(syncroot, 'C'), home, 'steal', 'proj')744	assert text_clone.code == 0745	assert text_clone.out.contains('[${id}] came in flagged — broke the build')746}747748fn test_man_and_koh2_handoff() {749	dir := temp_repo('man')750	home := temp_repo('man-home')751	defer {752		os.rmdir_all(dir) or {}753		os.rmdir_all(home) or {}754	}755	whole := run_home(dir, home, 'man')756	assert whole.code == 0757	assert whole.out.contains('NAME')758	assert whole.out.contains('COMMANDS')759	strip := run_home(dir, home, 'man', 'strip')760	assert strip.code == 0761	assert strip.out.contains('strip')762	assert !strip.out.contains('koh save ')763	assert run_home(dir, home, 'man', 'nonsense').code == 2764	legacy := temp_repo('legacy')765	defer { os.rmdir_all(legacy) or {} }766	os.mkdir_all(os.join_path(legacy, '.koh')) or { panic(err) }767	fake := os.join_path(home, 'bin')768	os.mkdir_all(fake) or { panic(err) }769	fake_koh := os.join_path(fake, 'koh2')770	os.write_file(fake_koh, '#!/bin/sh\nprintf "handoff:%s\\n" "$*"\nexit 7\n') or { panic(err) }771	os.chmod(fake_koh, 0o755) or { panic(err) }772	handoff := run_home_with_path(legacy, home, fake, 'status', '--json')773	assert handoff.code == 7774	assert handoff.out.contains('handoff:status --json')775	// One notice line, on stderr, so koh2's own output stays as it was.776	assert handoff.err.contains('Koh 2 repo')777	assert handoff.err.trim_space().split_into_lines().len == 1778	assert handoff.out.trim_space() == 'handoff:status --json'779}780781fn run_home_with_path(repo string, home string, bin_dir string, args ...string) Result {782	stamp := '${os.getpid()}-${time.now().unix_nano()}'783	base := os.join_path(os.temp_dir(), 'koh3-cli-${stamp}')784	out_path := '${base}.out'785	err_path := '${base}.err'786	code_path := '${base}.code'787	mut command := 'HOME=${q(home)} PATH=${q(bin_dir + ':' + os.getenv('PATH'))} ${q(binary)}'788	for arg in args {789		command += ' ${q(arg)}'790	}791	shell := 'cd ${q(repo)} && ${command} </dev/null >${q(out_path)} 2>${q(err_path)}; printf "%s" $? >${q(code_path)}'792	result := os.execute(shell)793	if result.exit_code != 0 {794		panic('could not run test command: ${result.output}')795	}796	code := os.read_file(code_path) or { panic(err) }797	out := os.read_file(out_path) or { '' }798	err := os.read_file(err_path) or { '' }799	os.rm(out_path) or {}800	os.rm(err_path) or {}801	os.rm(code_path) or {}802	return Result{803		code: code.int()804		out:  out805		err:  err806	}807}808809// --- migration -------------------------------------------------------------810811fn koh2_binary() string {812	return os.find_abs_path_of_executable('koh') or { '' }813}814815// run_koh2 runs the Koh 2 binary in a fixture; its own output is not under816// test here.817fn run_koh2(dir string, command string) {818	os.execute('cd ${q(dir)} && ${command} >/dev/null 2>&1')819}820821fn save_ids(dir string) []string {822	mut ids := []string{}823	for name in os.ls(os.join_path(dir, '.koh', 'snapshots')) or { []string{} } {824		if name.ends_with('.json') {825			ids << name.trim_right('.json')826		}827	}828	ids.sort()829	return ids830}831832fn test_migrate_converts_a_koh2_repo_and_keeps_every_save() {833	koh2 := koh2_binary()834	if koh2 == '' {835		println('skipping the migration tests: koh (Koh 2) is not on PATH')836		return837	}838	dir := temp_repo('migrate')839	defer { os.rmdir_all(dir) or {} }840	home := temp_repo('migrate-home')841	defer { os.rmdir_all(home) or {} }842	run_koh2(dir, '${q(koh2)} init')843	// One file large enough to be an object, one small enough to be inlined.844	write(dir, 'code.txt', 'x'.repeat(700))845	write(dir, 'tiny.txt', 'tiny\n')846	run_koh2(dir, '${q(koh2)} save "first save"')847	run_koh2(dir, '${q(koh2)} checkpoint "before something"')848	write(dir, 'code.txt', 'y'.repeat(700))849	run_koh2(dir, '${q(koh2)} save "second save"')850	original := save_ids(dir)851	assert original.len == 3852853	report := run_home(dir, home, 'migrate')854	assert report.code == 0855	assert report.out.contains('migrated 3 saves')856	// The swap: `.koh` is a file, the old data is archived, nothing is staging.857	assert os.is_file(os.join_path(dir, '.koh'))858	assert os.is_dir(os.join_path(dir, '.koh-v2'))859	assert os.is_file(os.join_path(dir, '.kohignore'))860	assert !os.exists(os.join_path(dir, '.koh-migrate'))861862	// Every save kept its original ID, and the new repo is sound.863	logged := run_home(dir, home, 'log', '--json')864	assert logged.code == 0865	for id in original {866		assert logged.out.contains(id)867	}868	assert run_home(dir, home, 'doctor').code == 0869	assert run_home(dir, home, 'status').code == 0870871	// Migrating again has nothing to convert.872	assert run_home(dir, home, 'migrate').code == 1873}874875fn test_migrate_refuses_a_tree_that_does_not_match_the_last_save() {876	koh2 := koh2_binary()877	if koh2 == '' {878		return879	}880	dir := temp_repo('migrate-dirty')881	defer { os.rmdir_all(dir) or {} }882	home := temp_repo('migrate-dirty-home')883	defer { os.rmdir_all(home) or {} }884	run_koh2(dir, '${q(koh2)} init')885	write(dir, 'code.txt', 'x'.repeat(700))886	run_koh2(dir, '${q(koh2)} save "only save"')887	write(dir, 'code.txt', 'unsaved change\n')888889	refused := run_home(dir, home, 'migrate')890	assert refused.code == 1891	assert refused.err.contains('save with koh2 first')892	// Nothing moved: still Koh 2, no archive, no staging file.893	assert os.is_dir(os.join_path(dir, '.koh'))894	assert !os.exists(os.join_path(dir, '.koh-v2'))895	assert !os.exists(os.join_path(dir, '.koh-migrate'))896}