From a6b4f0fbcfc176effe2c4888a7d190fb270b2d72 Mon Sep 17 00:00:00 2001 From: K1ndWha1e Date: Fri, 9 Oct 2026 00:10:40 +0300 Subject: [PATCH] =?UTF-8?q?*=D0=BF=D0=B5=D1=80=D0=B5=D0=B7=D0=B0=D0=BF?= =?UTF-8?q?=D0=B8=D1=81=D1=8C=20=D1=82=D1=8D=D0=B3=D0=BE=D0=B2=20=D0=BD?= =?UTF-8?q?=D0=B0=20=D0=BC=D0=B5=D1=81=D1=82=D0=B5:=20=D1=84=D0=B0=D0=B9?= =?UTF-8?q?=D0=BB=20=D1=81=D0=BE=D1=85=D1=80=D0=B0=D0=BD=D1=8F=D0=B5=D1=82?= =?UTF-8?q?=20inode=20(Audiobookshelf=20=D0=BE=D0=BF=D0=BE=D0=B7=D0=BD?= =?UTF-8?q?=D0=B0=D1=91=D1=82=20=D1=84=D0=B0=D0=B9=D0=BB=D1=8B=20=D0=BF?= =?UTF-8?q?=D0=BE=20inode,=20=D0=B7=D0=B0=D0=BC=D0=B5=D0=BD=D0=B0=20=D1=87?= =?UTF-8?q?=D0=B5=D1=80=D0=B5=D0=B7=20mv=20=D0=BF=D1=83=D1=82=D0=B0=D0=BB?= =?UTF-8?q?=D0=B0=20=D0=B5=D0=B3=D0=BE=20=D0=B1=D0=B8=D0=B1=D0=BB=D0=B8?= =?UTF-8?q?=D0=BE=D1=82=D0=B5=D0=BA=D1=83)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Opus 5.5 --- metadata/audiobook.v | 77 +++++++++++++++++++++++++++++-------------- test/audiobook_test.v | 20 +++++++++++ 2 files changed, 73 insertions(+), 24 deletions(-) diff --git a/metadata/audiobook.v b/metadata/audiobook.v index 3aca81b..6ae5f38 100644 --- a/metadata/audiobook.v +++ b/metadata/audiobook.v @@ -558,30 +558,68 @@ fn build_tag(parsed ParsedTag, t BookTags, opts WriteOptions) []u8 { return out } -// Запись тэгов: новый тэг + аудио из исходного файла, через временный файл. +// Запись тэгов: новый тэг + аудио из исходного файла собираются во временном файле, +// затем копируются обратно в исходный. Файл остаётся тем же (inode, владелец, права): +// Audiobookshelf опознаёт файлы по inode, и замена файла новым путает его библиотеку. // Файл без ID3v2 тоже поддерживается — тэг просто ставится в начало fn write_book_tags_impl(path string, t BookTags, opts WriteOptions) ! { parsed := parse_id3v2(path)! tag := build_tag(parsed, t, opts) - // Владелец и права исходного файла: перезапись через временный файл иначе их потеряет - // (на NAS файлы библиотеки принадлежат пользователю приложения, а агент может работать от root) - orig := os.stat(path) or { os.Stat{} } tmp_path := path + '.tagtmp' - mut src := os.open(path)! - mut dst := os.create(tmp_path)! - mut ok := false - defer { - src.close() - dst.close() - if !ok { + mut buf := []u8{len: 1024 * 1024} + { + mut src := os.open(path)! + defer { + src.close() + } + mut dst := os.create(tmp_path)! + defer { + dst.close() + } + dst.write(tag) or { os.rm(tmp_path) or {} + return err + } + src.seek(i64(parsed.audio_start), .start)! + for { + n := src.read(mut buf) or { 0 } + if n <= 0 { + break + } + dst.write(buf[..n]) or { + os.rm(tmp_path) or {} + return err + } } } + copy_back(tmp_path, path, mut buf) or { + // Исходный файл мог остаться наполовину переписанным — полная копия во временном + orig := os.stat(path) or { os.Stat{} } + os.mv(tmp_path, path, overwrite: true) or { + return error('${err}; новое содержимое сохранено в ${tmp_path}') + } + if orig.mode != 0 { + os.chmod(path, int(orig.mode & 0o7777)) or {} + $if !windows { + os.chown(path, int(orig.uid), int(orig.gid)) or {} + } + } + return + } + os.rm(tmp_path) or {} +} - dst.write(tag)! - src.seek(i64(parsed.audio_start), .start)! - mut buf := []u8{len: 1024 * 1024} +// Содержимое from — в существующий файл to (с обрезкой), не заменяя сам файл +fn copy_back(from string, to string, mut buf []u8) ! { + mut src := os.open(from)! + defer { + src.close() + } + mut dst := os.open_file(to, 'w')! + defer { + dst.close() + } for { n := src.read(mut buf) or { 0 } if n <= 0 { @@ -589,16 +627,7 @@ fn write_book_tags_impl(path string, t BookTags, opts WriteOptions) ! { } dst.write(buf[..n])! } - src.close() - dst.close() - ok = true - if orig.mode != 0 { - os.chmod(tmp_path, int(orig.mode & 0o7777)) or {} - $if !windows { - os.chown(tmp_path, int(orig.uid), int(orig.gid)) or {} - } - } - os.mv(tmp_path, path, overwrite: true)! + dst.flush() } // ---------- публичный API с защитой от паники ---------- diff --git a/test/audiobook_test.v b/test/audiobook_test.v index 57312cd..59a0278 100644 --- a/test/audiobook_test.v +++ b/test/audiobook_test.v @@ -131,3 +131,23 @@ fn test_write_keeps_mode() { assert after.mode & 0o777 == before.mode & 0o777 assert after.uid == before.uid } + +// Файл остаётся тем же: Audiobookshelf опознаёт файлы по inode +fn test_write_keeps_inode() { + path := tmp_copy('test_id3v23.mp3') + defer { + os.rm(path) or {} + } + before := os.stat(path) or { panic(err) } + metadata.write_book_tags(path, metadata.BookTags{ + title: 'Тот же файл' + comment: 'длинное описание '.repeat(500) + }) or { panic(err) } + after := os.stat(path) or { panic(err) } + $if !windows { + assert after.inode == before.inode + } + assert !os.exists(path + '.tagtmp') + t := metadata.read_book_tags(path) or { panic(err) } + assert t.tags.title == 'Тот же файл' +}