fix(meta-info,meta-compile,+5): правки по ревью — регистр, BOM, голый менеджер, порог

Семь находок ревью, все подтверждены воспроизведением:

1. Голый менеджер в источниках печатался по-английски: карта объектных видов
   применялась только к форме с точкой, а meta-compile пишет «cfg:DocumentManager»
   обычным v8:Type. Восемнадцать добавленных *Manager-записей были недостижимы.
   Суффикс «(все)» ему не ставится — это сам тип менеджера, а не класс объектов.
2. Новые регулярки прощающего ввода расходились между портами: -replace в
   PowerShell регистронезависим, re.sub — нет. «ДокументОбъект (Все)» проходил в
   ps1 и падал в py. Добавлен re.IGNORECASE — это задокументированная ловушка
   портирования, и она же снова сработала.
3. В шесть py-портов попал BOM (перекодировка при бампе версии), и перед
   «#!/usr/bin/env python3» он ломает shebang на POSIX. Снят там, где его не было
   в HEAD; в mxl-compile.py он был изначально и оставлен.
4. Правило со стрелкой из Resolve-TypeStr убрано. form-compile режет тип по [|+]
   ДО резолвера, поэтому строка глоссария «A -> B | C» молча превращалась в
   составной тип «A | C» вместо одного A. Половина строки, принятая за тип, —
   хуже громкого отказа. Остались суффикс «(все)» и счётчик «— типов: N».
5. py-порт meta-info не переиспользовал общий климб до корня конфигурации:
   правка тогда молча не применилась, и порты разошлись по структуре.
6. Found выставлялся до разбора состава определяемого типа: падение внутри
   давало «состав пуст» — ложь вместо «файл типа не разобран».
7. Порог сворачивания источников считался по сумме множеств и явных типов, а
   сворачивались только явные: пять множеств плюс один тип прятали этот тип, а
   шесть типов без множеств давали «и ещё» после пустоты. Порог теперь по числу
   явных типов, «и ещё» убрано как ложное.

Плюс формы источника подписки описаны в reference самого навыка: раньше они
были только в docs/meta-dsl-spec.md, а навык обязан быть самодостаточным.

Добавлен кейс на голого менеджера — без него находка 1 вернулась бы незамеченной.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
Nick Shirokov
2026-09-20 18:14:27 +03:00
co-authored by Claude Opus 5
parent 510c9163e3
commit e9d45984dc
27 changed files with 555 additions and 107 deletions
+15 -17
View File
@@ -328,7 +328,6 @@ def resolve_defined_type(dt_name):
dt_root = etree.parse(dt_path, parser_xml).getroot()
tn = find(dt_root, "/md:MetaDataObject/md:DefinedType/md:Properties/md:Type")
if tn is not None:
res["found"] = True
members = []
_in_type_set_expansion["on"] = True
try:
@@ -340,6 +339,9 @@ def resolve_defined_type(dt_name):
finally:
_in_type_set_expansion["on"] = False
res["members"] = members
# found только после успешного разбора: иначе упавшее раскрытие
# печаталось бы как «состав пуст» — ложь вместо «не разобран».
res["found"] = True
except Exception:
res["broken"] = True
if not res["found"]:
@@ -793,6 +795,12 @@ def format_source_type(raw):
name = m.group(2)
if prefix in object_type_map:
return f"{object_type_map[prefix]}.{name}"
# Голый вид без точки тоже бывает источником: meta-compile пишет менеджера (ДокументМенеджер)
# обычным v8:Type. Суффикса «(все)» тут НЕ ставим — это сам тип менеджера, а не класс объектов;
# множеством голый вид приходит через v8:TypeSet, и его форматирует format_single_type_set.
m = re.match(r'^cfg:(\w+)$', raw)
if m and m.group(1) in object_type_map:
return object_type_map[m.group(1)]
m = re.match(r'^cfg:(.+)$', raw)
if m:
return m.group(1)
@@ -883,19 +891,9 @@ def get_object_support_status(obj_uuid):
try:
if _sg_is_external_root(object_path):
return None
d = os.path.dirname(object_path)
bin_path = None
for _ in range(8):
if not d:
break
cand = os.path.join(d, "Ext", "ParentConfigurations.bin")
if os.path.exists(cand) or os.path.exists(os.path.join(d, "Configuration.xml")):
bin_path = cand
break
parent = os.path.dirname(d)
if parent == d:
break
d = parent
# Корень конфигурации ищем тем же климбом, что и состав определяемых типов.
root = get_config_root_dir()
bin_path = os.path.join(root, "Ext", "ParentConfigurations.bin") if root else None
if not bin_path or not os.path.exists(bin_path):
return "не на поддержке"
data = open(bin_path, "rb").read()
@@ -1474,14 +1472,14 @@ if not drill_done:
out(f"Источники ({total}):")
# full печатает всё: режим для этого и нужен, а длину держит постраничник.
# В overview явные типы сворачиваем в счётчик — их бывает больше тысячи.
list_types = mode == "full" or total <= COMPOSED_TYPE_THRESHOLD
list_types = mode == "full" or len(src_types) <= COMPOSED_TYPE_THRESHOLD
if list_types:
for s in src_types:
out(f" {s}")
for s in src_sets:
out(f" {s}")
if not list_types and src_types:
out(f" и ещё явных типов: {len(src_types)} (-Mode full)")
if not list_types:
out(f" явных типов: {len(src_types)} (-Mode full)")
# HTTPService
if md_type == "HTTPService":