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

feat(cfe-patch-method): устойчивый якорь ресинка, прозрачность, merge-имена файлов

Browse Source 
Упрочнение актуализации &ИзменениеИКонтроль (v2.1):

- Якорь вставки теперь двусторонний (контекст до+после, окно 3) с расширением:
  Тир A — уникальная смежная пара; Тир B — одиночная уникальность before/after.
  Срезает ложные конфликты, когда строка перед вставкой generic/повторяется
  (пустая, КонецЦикла;, КонецЕсли; и т.п.), сохраняя безопасность (не уверены → конфликт).
- Прозрачность: на [АКТУАЛИЗИРОВАН] и ЧАСТИЧНО печатается сводка перенесённого.
- Файлы-версии переименованы в конвенцию git-mergetool: base/local/remote/merged
  (+diff base->remote) вместо v1/v2/current; merged.bsl добавлен. Комментарий в
  модуле и SKILL.md обновлены.

Тесты: +resync-reanchor (generic-строка -> авто), resync-conflict переделан на
настоящий конфликт (окружающий блок исчез). 13 кейсов, оба рантайма зелёные,
байтовый паритет ps1<->py включая merge-воркспейс.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Nick Shirokov
2026-07-18 17:16:11 +03:00
co-authored by Claude Opus 4.8
parent a125464caa
commit 39fc1b977d
17 changed files with 682 additions and 51 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/skills/cfe-patch-method/scripts/cfe-patch-method.ps1
+99 -18
View File
@@ -1,4 +1,4 @@
# cfe-patch-method v2.0 — Source-aware method interceptor for 1C extension (CFE)
# cfe-patch-method v2.1 — Source-aware method interceptor for 1C extension (CFE)
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
param(
[Parameter(Mandatory)]
@@ -419,6 +419,62 @@ function Find-UniqueRun {
return $found
}
# Resolve where an insertion lands in v2 using two-sided context.
# Returns index to "insert after" (-1 = top of body), or $null if ambiguous/conflict.
function Resolve-InsertionPoint {
param($v2norm, $beforeLines, $afterLines)
$nb = $beforeLines.Count; $na = $afterLines.Count
# Tier A: adjacent pair (before[-1] immediately followed by after[0] in v2), widening symmetrically
if ($nb -ge 1 -and $na -ge 1) {
$cands = @()
for ($k = 0; $k -lt ($v2norm.Count - 1); $k++) {
if ($v2norm[$k] -eq $beforeLines[$nb - 1] -and $v2norm[$k + 1] -eq $afterLines[0]) { $cands += $k }
}
if ($cands.Count -eq 1) { return $cands[0] }
if ($cands.Count -gt 1) {
$w = 1
while ($cands.Count -gt 1 -and ($w -lt $nb -or $w -lt $na)) {
$w++
$filtered = @()
foreach ($k in $cands) {
$ok = $true
if ($w -le $nb) {
if (($k - ($w - 1)) -lt 0 -or $v2norm[$k - ($w - 1)] -ne $beforeLines[$nb - $w]) { $ok = $false }
}
if ($ok -and $w -le $na) {
if (($k + $w) -ge $v2norm.Count -or $v2norm[$k + $w] -ne $afterLines[$w - 1]) { $ok = $false }
}
if ($ok) { $filtered += $k }
}
if ($filtered.Count -eq 0) { break }
$cands = $filtered
}
if ($cands.Count -eq 1) { return $cands[0] }
return $null
}
}
# Tier B: one side changed -> single-side uniqueness
if ($nb -ge 1) {
$bk = Find-UniqueIndex $v2norm $beforeLines[$nb - 1]
if ($bk -ge 0) { return $bk }
}
if ($na -ge 1) {
$ak = Find-UniqueIndex $v2norm $afterLines[0]
if ($ak -ge 0) { return ($ak - 1) }
}
return $null
}
# Truncate a line for compact summary output
function Get-Truncated {
param([string]$s, [int]$n = 60)
$t = $s.Trim()
if ($t.Length -gt $n) { return $t.Substring(0, $n) + "…" }
return $t
}
# ============================================================================
# Main
# ============================================================================
@@ -556,17 +612,30 @@ if ($dup) {
$delStart = @{} # v2 index -> $true
$delEnd = @{}
$disputed = @()
$transferred = @() # for the summary: @{ Kind; Anchor }
foreach ($op in $ops) {
if ($op.Kind -eq 'insert') {
if ($op.After -lt 0) { $insertTop += ,$op.Lines; continue }
$anchorKey = $v1norm[$op.After]
$k = Find-UniqueIndex $v2norm $anchorKey
if ($k -ge 0) {
# two-sided context anchor
$beforeLines = @()
if ($op.After -ge 0) {
$bs = [Math]::Max(0, $op.After - 2)
for ($m = $bs; $m -le $op.After; $m++) { $beforeLines += $v1norm[$m] }
}
$afterLines = @()
$ae = [Math]::Min($v1norm.Count - 1, $op.After + 3)
for ($m = $op.After + 1; $m -le $ae; $m++) { $afterLines += $v1norm[$m] }
$k = Resolve-InsertionPoint $v2norm $beforeLines $afterLines
if ($null -eq $k) {
$disputed += @{ Kind = 'insert'; Lines = $op.Lines }
} elseif ($k -lt 0) {
$insertTop += ,$op.Lines
$transferred += @{ Kind = 'insert'; Anchor = '(в начало метода)' }
} else {
if (-not $insertAfter.ContainsKey($k)) { $insertAfter[$k] = @() }
$insertAfter[$k] += ,$op.Lines
} else {
$disputed += @{ Kind = 'insert'; Lines = $op.Lines }
$transferred += @{ Kind = 'insert'; Anchor = $v2[$k] }
}
} else {
$keys = @()
@@ -575,6 +644,7 @@ if ($dup) {
if ($p -ge 0) {
$delStart[$p] = $true
$delEnd[$p + $keys.Count - 1] = $true
$transferred += @{ Kind = 'delete'; Anchor = $op.Lines[0] }
} else {
$disputed += @{ Kind = 'delete'; Lines = $op.Lines }
}
@@ -599,7 +669,7 @@ if ($dup) {
# disputed blocks appended with conflict markers (never lost)
if ($disputed.Count -gt 0) {
$newBody += "`t// [РЕСИНК-КОНФЛИКТ] перенесите блоки ниже вручную — исходный якорь изменился в новой версии оригинала."
$newBody += "`t// Материалы для анализа см. в выводе команды (файлы v1/v2/current/diff)."
$newBody += "`t// Материалы для анализа см. в выводе команды (файлы base/local/remote/merged)."
foreach ($d in $disputed) {
if ($d.Kind -eq 'insert') {
$newBody += "#Вставка"; foreach ($l in $d.Lines) { $newBody += $l }; $newBody += "#КонецВставки"
@@ -632,26 +702,37 @@ if ($dup) {
[System.IO.File]::WriteAllText($extBsl, (($out -join "`r`n") + "`r`n"), $enc)
# transferred summary (shared by full-auto and partial)
function Write-TransferSummary {
param($items)
foreach ($t in $items) {
if ($t.Kind -eq 'insert') { Write-Host " • вставка после «$(Get-Truncated $t.Anchor)»" }
else { Write-Host " • удаление «$(Get-Truncated $t.Anchor)»" }
}
}
if ($disputed.Count -gt 0) {
# write version files
# merge workspace (git-mergetool convention: base/local/remote/merged)
$tmpRoot = Join-Path ([System.IO.Path]::GetTempPath()) ("cfe-resync\" + ($ModulePath -replace '[\\/:*?"<>|]', '_') + "." + $MethodName)
if (-not (Test-Path $tmpRoot)) { New-Item -ItemType Directory -Path $tmpRoot -Force | Out-Null }
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "v1.bsl"), (($v1 -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "v2.bsl"), (($v2 -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "current.bsl"), (($markedBody -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "base.bsl"), (($v1 -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "local.bsl"), (($markedBody -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "remote.bsl"), (($v2 -join "`r`n") + "`r`n"), $enc)
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "merged.bsl"), (($newBlock -join "`r`n") + "`r`n"), $enc)
$diff = @()
$diff += "--- v1 (что было скопировано) vs v2 (новый оригинал) ---"
$diff += "--- base -> remote (что изменилось в оригинале) ---"
foreach ($l in $v1) { if ($v2norm -notcontains (Get-Normalized $l)) { $diff += "- $l" } }
foreach ($l in $v2) { if ($v1norm -notcontains (Get-Normalized $l)) { $diff += "+ $l" } }
[System.IO.File]::WriteAllText((Join-Path $tmpRoot "diff.txt"), (($diff -join "`r`n") + "`r`n"), $enc)
Write-Host "[АКТУАЛИЗИРОВАН-ЧАСТИЧНО] &ИзменениеИКонтроль(`"$MethodName`")"
Write-Host " Перенесено автоматически, конфликтов: $($disputed.Count) (помечены // [РЕСИНК-КОНФЛИКТ])"
Write-Host " Файлы-версии для анализа:"
Write-Host "[АКТУАЛИЗИРОВАН-ЧАСТИЧНО] &ИзменениеИКонтроль(`"$MethodName`") — перенесено: $($transferred.Count), конфликтов: $($disputed.Count)"
Write-TransferSummary $transferred
Write-Host " Конфликтные блоки помечены // [РЕСИНК-КОНФЛИКТ] — разместите вручную."
Write-Host " Merge-воркспейс (base/local/remote/merged/diff):"
Write-Host " $tmpRoot"
Write-Host " Проверьте конфликтные блоки и разместите их вручную."
} else {
Write-Host "[АКТУАЛИЗИРОВАН] &ИзменениеИКонтроль(`"$MethodName`") — тело обновлено по новому оригиналу, правки перенесены."
Write-Host "[АКТУАЛИЗИРОВАН] &ИзменениеИКонтроль(`"$MethodName`") — тело обновлено, перенесено правок: $($transferred.Count)"
Write-TransferSummary $transferred
}
Write-Host " Файл: $extBsl"
exit 0
.claude/skills/cfe-patch-method/scripts/cfe-patch-method.py
+87 -18
View File
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# cfe-patch-method v2.0 — Source-aware method interceptor for 1C extension (CFE)
# cfe-patch-method v2.1 — Source-aware method interceptor for 1C extension (CFE)
# Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
import argparse
@@ -410,6 +410,57 @@ def find_unique_run(v2norm, keys):
return found
def resolve_insertion_point(v2norm, before_lines, after_lines):
"""Where an insertion lands in v2 using two-sided context.
Returns index to insert-after (-1 = top), or None if ambiguous/conflict."""
nb = len(before_lines)
na = len(after_lines)
# Tier A: adjacent pair (before[-1] then after[0] in v2), widening symmetrically
if nb >= 1 and na >= 1:
cands = [k for k in range(len(v2norm) - 1)
if v2norm[k] == before_lines[-1] and v2norm[k + 1] == after_lines[0]]
if len(cands) == 1:
return cands[0]
if len(cands) > 1:
w = 1
while len(cands) > 1 and (w < nb or w < na):
w += 1
filtered = []
for k in cands:
ok = True
if w <= nb:
if k - (w - 1) < 0 or v2norm[k - (w - 1)] != before_lines[nb - w]:
ok = False
if ok and w <= na:
if k + w >= len(v2norm) or v2norm[k + w] != after_lines[w - 1]:
ok = False
if ok:
filtered.append(k)
if not filtered:
break
cands = filtered
if len(cands) == 1:
return cands[0]
return None
# Tier B: one side changed -> single-side uniqueness
if nb >= 1:
bk = find_unique_index(v2norm, before_lines[-1])
if bk >= 0:
return bk
if na >= 1:
ak = find_unique_index(v2norm, after_lines[0])
if ak >= 0:
return ak - 1
return None
def truncate(s, n=60):
t = s.strip()
return (t[:n] + "…") if len(t) > n else t
def die(msg):
print(msg, file=sys.stderr)
sys.exit(1)
@@ -677,23 +728,29 @@ def resync(ext_bsl, ext_lines, dup, method, method_name, module_path):
del_start = set()
del_end = set()
disputed = []
transferred = [] # for the summary: {"kind", "anchor"}
for op in ops:
if op["kind"] == "insert":
if op["after"] < 0:
insert_top.append(op["lines"])
continue
k = find_unique_index(v2norm, v1norm[op["after"]])
if k >= 0:
insert_after.setdefault(k, []).append(op["lines"])
else:
after = op["after"]
before_lines = v1norm[max(0, after - 2):after + 1] if after >= 0 else []
after_lines = v1norm[after + 1:after + 4]
k = resolve_insertion_point(v2norm, before_lines, after_lines)
if k is None:
disputed.append({"kind": "insert", "lines": op["lines"]})
elif k < 0:
insert_top.append(op["lines"])
transferred.append({"kind": "insert", "anchor": "(в начало метода)"})
else:
insert_after.setdefault(k, []).append(op["lines"])
transferred.append({"kind": "insert", "anchor": v2[k]})
else:
keys = v1norm[op["start"]:op["end"] + 1]
p = find_unique_run(v2norm, keys)
if p >= 0:
del_start.add(p)
del_end.add(p + len(keys) - 1)
transferred.append({"kind": "delete", "anchor": op["lines"][0]})
else:
disputed.append({"kind": "delete", "lines": op["lines"]})
@@ -716,7 +773,7 @@ def resync(ext_bsl, ext_lines, dup, method, method_name, module_path):
if disputed:
new_body.append("\t// [РЕСИНК-КОНФЛИКТ] перенесите блоки ниже вручную — исходный якорь изменился в новой версии оригинала.")
new_body.append("\t// Материалы для анализа см. в выводе команды (файлы v1/v2/current/diff).")
new_body.append("\t// Материалы для анализа см. в выводе команды (файлы base/local/remote/merged).")
for d in disputed:
if d["kind"] == "insert":
new_body.append("#Вставка")
@@ -747,14 +804,23 @@ def resync(ext_bsl, ext_lines, dup, method, method_name, module_path):
out.extend(ext_lines[block_end + 1:])
write_bsl(ext_bsl, out)
def print_transfer_summary(items):
for t in items:
if t["kind"] == "insert":
print(" • вставка после «%s»" % truncate(t["anchor"]))
else:
print(" • удаление «%s»" % truncate(t["anchor"]))
if disputed:
# merge workspace (git-mergetool convention: base/local/remote/merged)
safe = re.sub(r'[\\/:*?"<>|]', '_', module_path)
tmp_root = os.path.join(tempfile.gettempdir(), "cfe-resync", safe + "." + method_name)
os.makedirs(tmp_root, exist_ok=True)
write_bsl(os.path.join(tmp_root, "v1.bsl"), v1)
write_bsl(os.path.join(tmp_root, "v2.bsl"), v2)
write_bsl(os.path.join(tmp_root, "current.bsl"), marked_body)
diff = ["--- v1 (что было скопировано) vs v2 (новый оригинал) ---"]
write_bsl(os.path.join(tmp_root, "base.bsl"), v1)
write_bsl(os.path.join(tmp_root, "local.bsl"), marked_body)
write_bsl(os.path.join(tmp_root, "remote.bsl"), v2)
write_bsl(os.path.join(tmp_root, "merged.bsl"), new_block)
diff = ["--- base -> remote (что изменилось в оригинале) ---"]
for l in v1:
if normalize(l) not in v2norm:
diff.append("- " + l)
@@ -763,13 +829,16 @@ def resync(ext_bsl, ext_lines, dup, method, method_name, module_path):
diff.append("+ " + l)
write_bsl(os.path.join(tmp_root, "diff.txt"), diff)
print('[АКТУАЛИЗИРОВАН-ЧАСТИЧНО] &ИзменениеИКонтроль("%s")' % method_name)
print(" Перенесено автоматически, конфликтов: %d (помечены // [РЕСИНК-КОНФЛИКТ])" % len(disputed))
print(" Файлы-версии для анализа:")
print('[АКТУАЛИЗИРОВАН-ЧАСТИЧНО] &ИзменениеИКонтроль("%s") — перенесено: %d, конфликтов: %d'
% (method_name, len(transferred), len(disputed)))
print_transfer_summary(transferred)
print(" Конфликтные блоки помечены // [РЕСИНК-КОНФЛИКТ] — разместите вручную.")
print(" Merge-воркспейс (base/local/remote/merged/diff):")
print(" %s" % tmp_root)
print(" Проверьте конфликтные блоки и разместите их вручную.")
else:
print('[АКТУАЛИЗИРОВАН] &ИзменениеИКонтроль("%s") — тело обновлено по новому оригиналу, правки перенесены.' % method_name)
print('[АКТУАЛИЗИРОВАН] &ИзменениеИКонтроль("%s") — тело обновлено, перенесено правок: %d'
% (method_name, len(transferred)))
print_transfer_summary(transferred)
print(" Файл: %s" % ext_bsl)