From a9697b613df36665170ed9eda65f1c64cf35910f Mon Sep 17 00:00:00 2001 From: Nick Shirokov Date: Mon, 10 Aug 2026 21:36:07 +0300 Subject: [PATCH] =?UTF-8?q?fix(mxl-decompile):=20=D0=BD=D0=B5=20=D0=BF?= =?UTF-8?q?=D0=B8=D1=81=D0=B0=D1=82=D1=8C=20style=20=D0=BF=D0=BE=20=D1=83?= =?UTF-8?q?=D0=BC=D0=BE=D0=BB=D1=87=D0=B0=D0=BD=D0=B8=D1=8E,=20=D1=87?= =?UTF-8?q?=D0=B8=D0=BD=D0=B8=D1=82=D1=8C=20=D0=BF=D0=BE=D1=82=D0=B5=D1=80?= =?UTF-8?q?=D1=8E=20=D1=81=D1=82=D0=B8=D0=BB=D1=8F=20=D0=BF=D1=80=D0=B8=20?= =?UTF-8?q?rowStyle?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Одно условие делает две вещи. Сокращение: у 56% ячеек (19 737 из 35 190) стоял "style": "default", хотя компилятор и так подставляет default, когда ключа нет и у строки нет rowStyle. Чистый шум; объём декомпилированного JSON на пилоте упал на 7%. Починка: при ЗАДАННОМ rowStyle ячейка со стилем default, которая его не наследует, теряла стиль вовсе — условие содержало -not rowStyleName, — и при обратной сборке такая ячейка наследовала rowStyle вместо своего умолчания. Проверка на потери: из 40 макетов пилота 23 собрались байт в байт как прежде. У 17 XML изменился, и это разобрано поячеечно: во всех 181 разошедшейся ячейке НИ ОДНА версия не совпадает с оригиналом — там backColor, фон ячейки, который DSL не поддерживает вовсе (числится в ограничениях). То есть правка переключает между двумя одинаково неверными отображениями неподдерживаемой конструкции, не улучшая и не ухудшая совпадение: фактов, совпавших с оригиналом, было 8183 и осталось 8183. Проверено: тесты 54/54 на обоих рантаймах. Дрейф снэпшота один и ожидаемый — из back.json ушёл "style": "default". Co-Authored-By: Claude Opus 5 (1M context) --- .claude/skills/mxl-decompile/scripts/mxl-decompile.ps1 | 9 +++++++-- .claude/skills/mxl-decompile/scripts/mxl-decompile.py | 9 +++++++-- .../snapshots/output-absolute-path/back.json | 2 +- 3 files changed, 15 insertions(+), 5 deletions(-) diff --git a/.claude/skills/mxl-decompile/scripts/mxl-decompile.ps1 b/.claude/skills/mxl-decompile/scripts/mxl-decompile.ps1 index 999d03e5..f2425b6a 100644 --- a/.claude/skills/mxl-decompile/scripts/mxl-decompile.ps1 +++ b/.claude/skills/mxl-decompile/scripts/mxl-decompile.ps1 @@ -1,4 +1,4 @@ -# mxl-decompile v1.6 — Decompile 1C spreadsheet to JSON +# mxl-decompile v1.7 — Decompile 1C spreadsheet to JSON # Source: https://github.com/Nikolay-Shirokov/cc-1c-skills param( [Parameter(Mandatory)] @@ -638,8 +638,13 @@ foreach ($area in $blocks) { if ($rowStyleKey -and $cellStyleKey -eq $rowStyleKey) { # Inherits rowStyle } else { + # Стиль пишем, только когда он отличается от того, что подставит компилятор: + # без rowStyle умолчание и есть "default", поэтому такой ключ — шум (56% ячеек). + # А при заданном rowStyle стиль "default" писать ОБЯЗАТЕЛЬНО: раньше здесь стояло + # `-not $rowStyleName`, и такая ячейка теряла стиль вовсе — при обратной сборке + # она наследовала rowStyle. $sn = Get-StyleName $cell.FormatIdx - if ($sn -ne "default" -or -not $rowStyleName) { + if ($sn -ne "default" -or $rowStyleName) { $dslCell["style"] = $sn } } diff --git a/.claude/skills/mxl-decompile/scripts/mxl-decompile.py b/.claude/skills/mxl-decompile/scripts/mxl-decompile.py index 4a03139f..0ef1792f 100644 --- a/.claude/skills/mxl-decompile/scripts/mxl-decompile.py +++ b/.claude/skills/mxl-decompile/scripts/mxl-decompile.py @@ -1,5 +1,5 @@ #!/usr/bin/env python3 -# mxl-decompile v1.6 — Decompile 1C spreadsheet to JSON +# mxl-decompile v1.7 — Decompile 1C spreadsheet to JSON # Source: https://github.com/Nikolay-Shirokov/cc-1c-skills import argparse @@ -807,8 +807,13 @@ def main(): if row_style_key and cell_style_key == row_style_key: pass # Inherits rowStyle else: + # Стиль пишем, только когда он отличается от того, что подставит компилятор: + # без rowStyle умолчание и есть "default", поэтому такой ключ — шум (56% ячеек). + # А при заданном rowStyle стиль "default" писать ОБЯЗАТЕЛЬНО: раньше здесь было + # `not row_style_name`, и такая ячейка теряла стиль вовсе — при обратной сборке + # она наследовала rowStyle. sn = get_style_name(cell["FormatIdx"]) - if sn != "default" or not row_style_name: + if sn != "default" or row_style_name: dsl_cell["style"] = sn # Content diff --git a/tests/skills/cases/mxl-decompile/snapshots/output-absolute-path/back.json b/tests/skills/cases/mxl-decompile/snapshots/output-absolute-path/back.json index e11b0818..7d36d508 100644 --- a/tests/skills/cases/mxl-decompile/snapshots/output-absolute-path/back.json +++ b/tests/skills/cases/mxl-decompile/snapshots/output-absolute-path/back.json @@ -1 +1 @@ -{ "columns": 2, "defaultWidth": 10, "fonts": { "default": { "face": "Arial", "size": 10 } }, "styles": {}, "areas": [{ "name": "Test", "rows": [{ "cells": [{ "col": 1, "style": "default", "text": "A" }, { "col": 2, "style": "default", "text": "B" }] }] }] } \ No newline at end of file +{ "columns": 2, "defaultWidth": 10, "fonts": { "default": { "face": "Arial", "size": 10 } }, "styles": {}, "areas": [{ "name": "Test", "rows": [{ "cells": [{ "col": 1, "text": "A" }, { "col": 2, "text": "B" }] }] }] } \ No newline at end of file