creator/cc-1c-skills
1
0
Fork 0
mirror of https://github.com/Nikolay-Shirokov/cc-1c-skills.git synced 2026-08-31 23:40:51 +03:00
Code Issues Packages Projects Releases Wiki Activity

feat(db-repo): порог на длину вывода и функциональные тесты

Browse Source 
Операция над всей конфигурацией перечисляет тысячи объектов, и весь список
уезжал в вывод модели. Списки обрезаются на двадцати позициях, полный уходит
в файл, путь к которому назван. Сырой лог платформы при отказе показывается
хвостом в 200 строк: итог и причину платформа пишет в конце.

Тесты на фейковой платформе — 12 кейсов, разбор лога проверяется без 1С на
логах, снятых со стенда. Ключевой кейс закрывает главное правило навыка:
платформа вернула 1, часть объектов захвачена, навык возвращает 0 с поимённым
предупреждением. Обрезка списка проверена на логе из тридцати объектов —
кейс требует и строку «и ещё 10», и отсутствие двадцать первого объекта
в выводе.

Проверено негативным контролем: намеренно сломанное ожидание роняет кейс,
то есть тесты действительно сверяют вывод, а не проходят вхолостую.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Nick Shirokov
2026-08-23 13:25:11 +03:00
co-authored by Claude Opus 5
parent 67b8ad5b47
commit 4a49eea422
15 changed files with 598 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/skills/db-repo/scripts/db-repo.ps1
+57 -14
View File
@@ -1,4 +1,4 @@
# db-repo v1.4 — 1C configuration repository operations
# db-repo v1.5 — 1C configuration repository operations
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
# NB: движок только 1cv8 — ibcmd работу с хранилищем не поддерживает (нет такого режима).
<#
@@ -718,7 +718,11 @@ function Write-ReceivedWarning {
if (-not $Received -or $Received.Count -eq 0) { return }
Write-Host ""
Write-Host "[warning] локальная конфигурация изменена, получено объектов из хранилища: $($Received.Count)" -ForegroundColor Yellow
foreach ($n in $Received) { Write-Host " $n" -ForegroundColor Yellow }
$recvLimit = [Math]::Min($Received.Count, $script:ListLimit)
foreach ($n in $Received[0..($recvLimit - 1)]) { Write-Host " $n" -ForegroundColor Yellow }
if ($Received.Count -gt $script:ListLimit) {
Write-Host " … и ещё $($Received.Count - $script:ListLimit)" -ForegroundColor Yellow
}
$owners = Get-OwnerObjects $Received
$hasRoot = @($Received | Where-Object { ($_ -split '\.').Count -eq 1 }).Count -gt 0
if ($owners.Count -gt 0) {
@@ -804,11 +808,29 @@ function Get-ObjectKey {
return ($own -join '.').ToLowerInvariant()
}
# Операция над всей конфигурацией перечисляет тысячи объектов. Печатаем начало списка,
# остальное кладём в файл: модели нужен факт и путь, а не простыня.
$script:ListLimit = 20
function Save-ObjectList {
param([string[]]$Names, [string]$Key)
if (-not $Key) { $Key = 'objects' }
$path = Join-Path $env:TEMP "db-repo-$Key.txt"
$utf8Bom = New-Object System.Text.UTF8Encoding($true)
[System.IO.File]::WriteAllLines($path, $Names, $utf8Bom)
return $path
}
function Write-RepoObjects {
param([string]$Title, [string[]]$Names, [string]$Color = 'Green')
param([string]$Title, [string[]]$Names, [string]$Color = 'Green', [string]$FileKey = 'objects')
if (-not $Names -or $Names.Count -eq 0) { return }
Write-Host "$Title ($($Names.Count)):" -ForegroundColor $Color
foreach ($n in $Names) { Write-Host " $n" }
$limit = [Math]::Min($Names.Count, $script:ListLimit)
foreach ($n in $Names[0..($limit - 1)]) { Write-Host " $n" }
if ($Names.Count -gt $script:ListLimit) {
$path = Save-ObjectList $Names $FileKey
Write-Host " … и ещё $($Names.Count - $script:ListLimit); полный список: $path" -ForegroundColor Yellow
}
}
function Write-RepoVerdict {
@@ -816,7 +838,11 @@ function Write-RepoVerdict {
if ($Log.Missing.Count -gt 0) {
Write-Host "Error: objects not found in the configuration:" -ForegroundColor Red
foreach ($n in $Log.Missing) { Write-Host " $n" -ForegroundColor Red }
$missLimit = [Math]::Min($Log.Missing.Count, $script:ListLimit)
foreach ($n in $Log.Missing[0..($missLimit - 1)]) { Write-Host " $n" -ForegroundColor Red }
if ($Log.Missing.Count -gt $script:ListLimit) {
Write-Host " … и ещё $($Log.Missing.Count - $script:ListLimit); полный список: $(Save-ObjectList $Log.Missing 'missing')" -ForegroundColor Red
}
Write-Host "Возможные причины:" -ForegroundColor Yellow
Write-Host " - опечатка в имени. Принимаются обе формы: Справочник.Номенклатура и Catalog.Номенклатура" -ForegroundColor Yellow
Write-Host " - база отстала от хранилища, объект появился позже — выполните /db-repo update" -ForegroundColor Yellow
@@ -829,10 +855,15 @@ function Write-RepoVerdict {
'lock' {
# Порядок вывода: факты, затем вердикт, и только потом совет. Совет длинный, и
# между фактами и вердиктом он прятал бы главную строку.
Write-RepoObjects "Захвачено" $Log.Locked
Write-RepoObjects "Захвачено" $Log.Locked 'Green' 'locked'
if ($Log.LockedByOther.Count -gt 0) {
Write-Host "Не удалось захватить ($($Log.LockedByOther.Count)):" -ForegroundColor Yellow
foreach ($o in $Log.LockedByOther) { Write-Host " $($o.Name) — держит $($o.Holder)" -ForegroundColor Yellow }
$blockedLimit = [Math]::Min($Log.LockedByOther.Count, $script:ListLimit)
foreach ($o in $Log.LockedByOther[0..($blockedLimit - 1)]) { Write-Host " $($o.Name) — держит $($o.Holder)" -ForegroundColor Yellow }
if ($Log.LockedByOther.Count -gt $script:ListLimit) {
$blockedPath = Save-ObjectList @($Log.LockedByOther | ForEach-Object { "$($_.Name) — $($_.Holder)" }) 'blocked'
Write-Host " … и ещё $($Log.LockedByOther.Count - $script:ListLimit); полный список: $blockedPath" -ForegroundColor Yellow
}
}
if ($Log.Locked.Count -eq 0 -and $Log.LockedByOther.Count -gt 0) {
# «Уже захвачено мной» платформа не печатает вовсе, поэтому отличить его от
@@ -877,15 +908,19 @@ function Write-RepoVerdict {
if ($PlatformExit -ne 0) {
if ($Log.Modified.Count -gt 0) {
Write-Host "Отмена захвата не выполнена: у объектов есть локальные изменения ($($Log.Modified.Count)):" -ForegroundColor Red
foreach ($n in $Log.Modified) { Write-Host " $n" -ForegroundColor Red }
$modLimit = [Math]::Min($Log.Modified.Count, $script:ListLimit)
foreach ($n in $Log.Modified[0..($modLimit - 1)]) { Write-Host " $n" -ForegroundColor Red }
if ($Log.Modified.Count -gt $script:ListLimit) {
Write-Host " … и ещё $($Log.Modified.Count - $script:ListLimit); полный список: $(Save-ObjectList $Log.Modified 'modified')" -ForegroundColor Red
}
Write-Host "Поместите их (/db-repo commit) либо откажитесь от них: -Force перезапишет объекты версией из хранилища." -ForegroundColor Yellow
} else {
Write-Host "Отмена захвата не выполнена (код $PlatformExit)$(Get-ExitAnnotation $PlatformExit)" -ForegroundColor Red
}
return 1
}
Write-RepoObjects "Захват отменён" $Log.Unlocked
Write-RepoObjects "Не были захвачены — снимать нечего" $Log.NotLocked 'Yellow'
Write-RepoObjects "Захват отменён" $Log.Unlocked 'Green' 'unlocked'
Write-RepoObjects "Не были захвачены — снимать нечего" $Log.NotLocked 'Yellow' 'not-locked'
if ($Log.Unlocked.Count -eq 0) { Write-Host "Изменений не потребовалось." -ForegroundColor Green }
Write-ReceivedWarning $Log.Received
return 0
@@ -901,8 +936,8 @@ function Write-RepoVerdict {
Write-Host " - есть ли у пользователя хранилища право на помещение" -ForegroundColor Yellow
return 1
}
Write-RepoObjects "Помещено в хранилище" $Log.Committed
Write-RepoObjects "Без изменений — не помещались" $Log.Unchanged 'Yellow'
Write-RepoObjects "Помещено в хранилище" $Log.Committed 'Green' 'committed'
Write-RepoObjects "Без изменений — не помещались" $Log.Unchanged 'Yellow' 'unchanged'
if ($Log.Committed.Count -eq 0) {
Write-Host "Новая версия в хранилище НЕ создана: помещать было нечего." -ForegroundColor Yellow
}
@@ -922,7 +957,7 @@ function Write-RepoVerdict {
Write-Host " всю конфигурацию базы содержимым хранилища. Проверьте состояние базы." -ForegroundColor Red
return 1
}
Write-RepoObjects "Получено из хранилища" $Log.Received
Write-RepoObjects "Получено из хранилища" $Log.Received 'Green' 'received'
if ($Log.Received.Count -eq 0) {
Write-Host "Изменений в хранилище нет — конфигурация уже актуальна." -ForegroundColor Green
return 0
@@ -1154,8 +1189,16 @@ try {
$verdict = Write-RepoVerdict $cmd $log $exitCode $requested
# Разбор мог не покрыть причину (у commit её вовсе нет в логе) — при отказе показываем сырой лог.
if ($verdict -ne 0 -and $logText.Trim()) {
# Лог операции над всей конфигурацией — тысячи строк. Показываем хвост: итог и причина
# отказа платформа пишет в конце.
$logLines = @($logText.TrimEnd() -split "`r?`n")
$logLimit = 200
Write-Host "--- Log ---"
Write-Host $logText.TrimEnd()
if ($logLines.Count -gt $logLimit) {
Write-Host "[... показаны последние $logLimit строк из $($logLines.Count) ...]"
$logLines = $logLines[($logLines.Count - $logLimit)..($logLines.Count - 1)]
}
Write-Host ($logLines -join [Environment]::NewLine)
Write-Host "--- End ---"
}
Write-PlatformOutput $proc.Output
.claude/skills/db-repo/scripts/db-repo.py
+51 -14
View File
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# db-repo v1.4 — 1C configuration repository operations
# db-repo v1.5 — 1C configuration repository operations
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
# NB: движок только 1cv8 — ibcmd работу с хранилищем не поддерживает (нет такого режима).
"""Работа с хранилищем конфигурации 1С.
@@ -622,8 +622,10 @@ def print_received_warning(received):
return
print("")
print("[warning] локальная конфигурация изменена, получено объектов из хранилища: %d" % len(received))
for n in received:
for n in received[:LIST_LIMIT]:
print(" %s" % n)
if len(received) > LIST_LIMIT:
print(" … и ещё %d" % (len(received) - LIST_LIMIT))
owners = owner_objects(received)
has_root = any(len(n.split(".")) == 1 for n in received)
if owners:
@@ -705,19 +707,39 @@ def object_key(name):
return ".".join(own).lower()
def print_repo_objects(title, names):
# Операция над всей конфигурацией перечисляет тысячи объектов. Печатаем начало списка,
# остальное кладём в файл: модели нужен факт и путь, а не простыня.
LIST_LIMIT = 20
def save_object_list(names, key):
if not key:
key = "objects"
path = os.path.join(tempfile.gettempdir(), "db-repo-%s.txt" % key)
with open(path, "w", encoding="utf-8-sig", newline="\n") as f:
f.write("\n".join(names) + "\n")
return path
def print_repo_objects(title, names, file_key="objects"):
if not names:
return
print("%s (%d):" % (title, len(names)))
for n in names:
for n in names[:LIST_LIMIT]:
print(" %s" % n)
if len(names) > LIST_LIMIT:
path = save_object_list(names, file_key)
print(" … и ещё %d; полный список: %s" % (len(names) - LIST_LIMIT, path))
def write_repo_verdict(cmd, log, platform_exit, requested, report_format="", output_file=""):
if log["missing"]:
print("Error: objects not found in the configuration:")
for n in log["missing"]:
for n in log["missing"][:LIST_LIMIT]:
print(" %s" % n)
if len(log["missing"]) > LIST_LIMIT:
print(" … и ещё %d; полный список: %s"
% (len(log["missing"]) - LIST_LIMIT, save_object_list(log["missing"], "missing")))
print("Возможные причины:")
print(" - опечатка в имени. Принимаются обе формы: Справочник.Номенклатура и Catalog.Номенклатура")
print(" - база отстала от хранилища, объект появился позже — выполните /db-repo update")
@@ -728,11 +750,16 @@ def write_repo_verdict(cmd, log, platform_exit, requested, report_format="", out
if cmd == "lock":
# Порядок вывода: факты, затем вердикт, и только потом совет. Совет длинный, и
# между фактами и вердиктом он прятал бы главную строку.
print_repo_objects("Захвачено", log["locked"])
print_repo_objects("Захвачено", log["locked"], "locked")
if log["locked_by_other"]:
print("Не удалось захватить (%d):" % len(log["locked_by_other"]))
for o in log["locked_by_other"]:
for o in log["locked_by_other"][:LIST_LIMIT]:
print(" %s — держит %s" % (o["name"], o["holder"]))
if len(log["locked_by_other"]) > LIST_LIMIT:
path = save_object_list(
["%s — %s" % (o["name"], o["holder"]) for o in log["locked_by_other"]], "blocked")
print(" … и ещё %d; полный список: %s"
% (len(log["locked_by_other"]) - LIST_LIMIT, path))
if not log["locked"] and log["locked_by_other"]:
# «Уже захвачено мной» платформа не печатает вовсе, поэтому отличить его от
# «не захвачено» по логу нельзя — сверяем с тем, что просили. Заняты ВСЕ
@@ -770,15 +797,18 @@ def write_repo_verdict(cmd, log, platform_exit, requested, report_format="", out
if platform_exit != 0:
if log["modified"]:
print("Отмена захвата не выполнена: у объектов есть локальные изменения (%d):" % len(log["modified"]))
for n in log["modified"]:
for n in log["modified"][:LIST_LIMIT]:
print(" %s" % n)
if len(log["modified"]) > LIST_LIMIT:
print(" … и ещё %d; полный список: %s"
% (len(log["modified"]) - LIST_LIMIT, save_object_list(log["modified"], "modified")))
print("Поместите их (/db-repo commit) либо откажитесь от них: -Force перезапишет "
"объекты версией из хранилища.")
else:
print("Отмена захвата не выполнена (код %s)%s" % (platform_exit, describe_exit(platform_exit)))
return 1
print_repo_objects("Захват отменён", log["unlocked"])
print_repo_objects("Не были захвачены — снимать нечего", log["not_locked"])
print_repo_objects("Захват отменён", log["unlocked"], "unlocked")
print_repo_objects("Не были захвачены — снимать нечего", log["not_locked"], "not-locked")
if not log["unlocked"]:
print("Изменений не потребовалось.")
print_received_warning(log["received"])
@@ -794,8 +824,8 @@ def write_repo_verdict(cmd, log, platform_exit, requested, report_format="", out
print(" - не держит ли его другой пользователь")
print(" - есть ли у пользователя хранилища право на помещение")
return 1
print_repo_objects("Помещено в хранилище", log["committed"])
print_repo_objects("Без изменений — не помещались", log["unchanged"])
print_repo_objects("Помещено в хранилище", log["committed"], "committed")
print_repo_objects("Без изменений — не помещались", log["unchanged"], "unchanged")
if not log["committed"]:
print("Новая версия в хранилище НЕ создана: помещать было нечего.")
return 0
@@ -812,7 +842,7 @@ def write_repo_verdict(cmd, log, platform_exit, requested, report_format="", out
print(" Похоже, база НЕ подключена к хранилищу — в этом случае команда заменяет")
print(" всю конфигурацию базы содержимым хранилища. Проверьте состояние базы.")
return 1
print_repo_objects("Получено из хранилища", log["received"])
print_repo_objects("Получено из хранилища", log["received"], "received")
if not log["received"]:
print("Изменений в хранилище нет — конфигурация уже актуальна.")
return 0
@@ -1119,8 +1149,15 @@ def main():
args.ReportFormat, args.OutputFile)
# Разбор мог не покрыть причину (у commit её вовсе нет в логе) — при отказе показываем сырой лог.
if verdict != 0 and log_text.strip():
# Лог операции над всей конфигурацией — тысячи строк. Показываем хвост: итог и причина
# отказа платформа пишет в конце.
log_lines = log_text.rstrip().splitlines()
log_limit = 200
print("--- Log ---")
print(log_text.rstrip())
if len(log_lines) > log_limit:
print("[... показаны последние %d строк из %d ...]" % (log_limit, len(log_lines)))
log_lines = log_lines[-log_limit:]
print("\n".join(log_lines))
print("--- End ---")
print_platform_output(result)
sys.exit(verdict)