Repository navigation
Исправления по аудиту: обезличивание отчёта, нечитаемые файлы, CI - #2
Merged
Merged
Conversation
…иком Виндовый путь обрывался на первом пробеле, и хвост вроде "Ivan Petrov\Desktop\..." или "Общая папка\Иванов И.И\..." уходил в отчёт, который по шаблону issue прикладывают публично. Теперь путь тянется до кавычки или конца строки. Имя файла из пути остаётся, только если оно безопасно (исходник .py или уже PRG_NN / PART_NN). Иначе остаётся FILE.<расширение>: при падении без загруженного задания подстановке не из чего строиться, и реальный номер программы попадал в отчёт как есть. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CW3WfPRekjK4x6M8VNRckz
…ишнего - Программа, файл которой не читается, остаётся на своём месте в задании, в расчёт не идёт и даёт замечание warn.unreadable. Раньше один заблокированный файл не давал открыть всю смену. - Сверка пропускает нечитаемые программы и отчёты наладки вместо падения. - Окно считает сверку до смены состояния: если она упадёт, отметки прошлой смены не перейдут на новую. - isdecimal вместо isdigit в поиске программ и разборе номера: «²» — digit, но int() на нём падал. - Убрано неиспользуемое: FONT_BIG, var_lang, status_at (тесты переведены на status_for + positions_upto), лишняя проверка в _save_report. Одинаковые dimension() из двух парсеров слиты в reader.search_float. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CW3WfPRekjK4x6M8VNRckz
Раньше в CI были только workflow Claude, тесты не запускались. Windows + 3.8 x86 ближе всего к цеховой машине. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CW3WfPRekjK4x6M8VNRckz
This was referenced Sep 29, 2026
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Исправления по итогам security review, code review и аудита кода.
Отчёт об ошибке (приватность)
Отчёт по шаблону issue прикладывают публично, поэтому утечки здесь важнее всего.
Ivan Petrov\Desktop\...илиОбщая папка\Иванов И.И\...уходил в отчёт. Теперь путь тянется до кавычки или конца строки (report.py).000101zz201006.ncоставался в пути. Теперь от небезопасного имени остаётсяFILE.<расширение>; исходники.pyи уже обезличенныеPRG_NN/PART_NNсохраняются.Устойчивость
.nc(заблокирован, недокопирован) больше не мешает открыть смену: программа остаётся на своём месте в задании, в расчёт не идёт, оператор видит замечаниеwarn.unreadable.isdecimalвместоisdigit).Чистка лишнего кода
FONT_BIG,var_lang,status_at(тесты переведены наstatus_for+positions_upto), лишняя проверка в_save_report.dimension()из двух парсеров слиты вreader.search_float.CI
Добавлен
.github/workflows/tests.yml: pytest на Windows + Python 3.8 x86 (ближе всего к цеховой машине) и на Linux + 3.13. Раньше тесты в CI не запускались.Проверка
gui.pyпроверен компиляцией.Не вошло
ProgramNest.numberоставлено.--done 0, экранирование формул в CSV.🤖 Generated with Claude Code
https://claude.ai/code/session_01CW3WfPRekjK4x6M8VNRckz
Generated by Claude Code