This repository was archived by the owner on Sep 11, 2026. It is now read-only.
Repository navigation
test(parity): fix-контур через продакшн-міст для всіх шести фіксерів - #514
Merged
Merged
Conversation
Гейти перевіряли переважно `detect()`. Порахував до роботи: у `wasm-plugin-parity-php` нуль згадок fix-контуру, у `ci-github` і `python` по дві, у `rust` чотири, у `lang-js` девʼять. А фіксерів у гостях уже шість. Ціна цієї прогалини вже виміряна: #513 знайшов баг спільного napi-мосту — `run_wasm_concern_fix` будував `FixRequest::files` лише з `diagnostic.file`, і whole-batch концерни діставали порожній батч, через що гість затирав конфіги користувача. Знайшлось лише тому, що ОДИН тест ішов через реальний `runWasmConcernFix`. Прямий виклик гостя показав би зелень. Чотири нові сценарії, кожен — повний цикл через МІСТ: `runWasmConcern` → `runWasmConcernFix` → застосування едитів → повторний детект чисто. Покрито `rust/doc_comments`, `rust/cargo_mutants_config`, `python/doc_comments` і `ga/workflows` — останній був єдиним фіксером узагалі без fix-циклу в гейті. Реєстр сам чесно писав про цю дірку («той файл `fix()` не викликає взагалі»), тож задача лише підтвердила записане й закрила. Кожен сценарій доведено здатним падати. Перевірено особисто на `rust/doc_comments`: занулив `fix_doc_comments` до порожнього плану, перезібрав гість — червоніє рівно новий сценарій, сусіди зелені. Повернув — 58/58. Окремо цінне спостереження виконавця: локальний прекомпільований napi-аддон із node_modules виявився старішим за HEAD і відтворив баг #513 ЖИВЦЕМ, без жодної навмисної мутації — ті самі два тести, ті самі назви. Аудит whole-batch: `js/check` лишається єдиним серед портованих. Два кандидати на ризик для майбутніх портів (`rust/check`, file-less гілки `ga/workflows`) задокументовано в §2.49 як застереження. Продакшн-код не змінювався — `git diff` по crates порожній. Тести: чотири гейти разом 402. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Гейти перевіряли переважно
detect(). Порахував до роботи:phpci-githubpythonrustlang-jsА фіксерів у гостях уже шість.
Ціна прогалини вже виміряна
#513 знайшов баг спільного napi-мосту:
run_wasm_concern_fixбудувавFixRequest::filesлише зdiagnostic.file, whole-batch концерни діставали порожній батч, і гість затирав конфіги користувача.Знайшлось лише тому, що один тест ішов через реальний
runWasmConcernFix. Прямий виклик гостя показав би зелень.Чотири нові сценарії — усі через міст
Повний цикл:
runWasmConcern→runWasmConcernFix→ застосування едитів → повторний детект чисто.rust/doc_commentsrust/cargo_mutants_configpython/doc_commentsga/workflowsРеєстр сам чесно писав про цю дірку («той файл
fix()не викликає взагалі») — задача підтвердила записане й закрила.Кожен сценарій доведено здатним падати
Перевірив особисто на
rust/doc_comments: зануливfix_doc_commentsдо порожнього плану, перезібрав гість — червоніє рівно новий сценарій, сусіди зелені. Повернув — 58/58.Спостереження, варте окремої згадки
Локальний прекомпільований napi-аддон із
node_modulesвиявився старішим за HEAD і відтворив баг #513 живцем, без жодної навмисної мутації — ті самі два тести, ті самі назви.Аудит whole-batch
js/checkлишається єдиним таким серед портованих. Один кандидат на ризик для майбутніх портів — file-less гілкиga/workflows— задокументовано в §2.49 як застереження.Продакшн-код не змінювався —
git diffпоcrates/порожній. Тести: чотири гейти разом 402.🤖 Generated with Claude Code