Review replication: Difference between revisions
Mitrofanov (talk | contribs) |
|||
(7 intermediate revisions by 2 users not shown) | |||
Line 1: | Line 1: | ||
== Репликация разметки по правилам (экспериментальная функциональность) == | == Репликация разметки по правилам (экспериментальная функциональность) == | ||
При использовании разных версий Svace (или иных анализаторов), разных настроек CI/CD и прочих факторах возможны ситуации, когда снимки (snapshots) в рамках одной ветки (branch) могут иметь различные инварианты и разметка автоматически не переносится. Для решения задачи переноса разметки в таких случаях разработана утилита <code>reviewer</code>, которая позволяет применить декларативно описанные правила переноса к существующей разметке. В дальнейшем данная функциональность будет включена в Svacer и доступна в веб-интерфейсе. | При использовании разных версий Svace (или иных анализаторов), разных настроек CI/CD и прочих факторах возможны ситуации, когда снимки (snapshots) в рамках одной ветки (branch) могут иметь различные инварианты и разметка автоматически не переносится. Для решения задачи переноса разметки в таких случаях разработана утилита <code>reviewer</code>, которая позволяет применить декларативно описанные правила переноса к существующей разметке. В дальнейшем данная функциональность будет включена в Svacer и доступна в веб-интерфейсе. | ||
Использование утилиты подразумевает использование CLI команд <code>svacer markup2 export</code> и <code>svacer markup2 import</code> [[Markup2|(описание markup2)]] для выгрузки и импортирования разметки. Утилита <code>reviewer</code> обеспечивает модификацию выгружаемой разметки с целью копирования существующей разметки на неразмеченные элементы. Правила копирования задаются с использованием булевских выражений и набора предопределенных предикатов. | |||
=== Описание утилиты === | === Описание утилиты === | ||
Утилита <code>reviewer</code> предназначена для Svacer версии 10 и выше. | Утилита <code>reviewer</code> предназначена для Svacer версии 10 и выше. Утилиту можно скачать отсюда [https://nextcloud.ispras.ru/index.php/s/mC8333JNDF2Z8dX Ссылка на nextcloud], утилита собрана под Linux. | ||
NAME: | NAME: | ||
reviewer - Svacer review help tool | reviewer - Svacer review help tool | ||
Line 48: | Line 50: | ||
'''Шаги решения:''' | '''Шаги решения:''' | ||
1. | 1. Опционально, но рекомендуется выгрузить разметку с ветки в файл, используя команду: | ||
svacer markup2 --host=<nowiki>http://localhost:8080</nowiki> --user admin --password admin export --project myproject --branch master --uncompressed --format json --export-all /tmp/myproject.json | svacer markup2 --host=<nowiki>http://localhost:8080</nowiki> --user admin --password admin export --project myproject --branch master --uncompressed --format json --export-all /tmp/myproject.json | ||
2. Написать правила переноса разметки (предикатов переноса) в файл <code>/tmp/map.txt</code> | 2. Написать правила переноса разметки (предикатов переноса) в файл <code>/tmp/map.txt</code> | ||
3. Если разметка была выгружена (шаг 1), то применить утилиту <code>reviewer</code> с загрузкой данных из файла (флаг <code>--html</code> создаст дополнительно HTML с информацией об изменениях , имя файла <code>/tmp/out.json.html</code>): | 3. Если разметка была выгружена (шаг 1 выполнен), то применить утилиту <code>reviewer</code> '''с загрузкой данных из файла''' (флаг <code>--html</code> создаст дополнительно HTML с информацией об изменениях , имя файла <code>/tmp/out.json.html</code>): | ||
reviewer replicate --file /tmp/myproject.json --rule /tmp/map.txt --html /tmp/out.json | reviewer replicate --file /tmp/myproject.json --rule /tmp/map.txt --html /tmp/out.json | ||
4. Если разметка не была выгружена (шаг 1 пропущен) применить утилиту <code>reviewer</code> с загрузкой данных с сервера (флаг <code>--html</code> создаст дополнительно HTML с информацией об изменениях , имя файла <code>/tmp/out.json.html</code>): | 4. Если разметка не была выгружена (шаг 1 пропущен) применить утилиту <code>reviewer</code> '''с загрузкой данных с сервера''' (флаг <code>--html</code> создаст дополнительно HTML с информацией об изменениях , имя файла <code>/tmp/out.json.html</code>): | ||
reviewer --host=http://localhost:8080 --user admin --password admin replicate --project myproject --branch master --rule /tmp/map.txt --html /tmp/out.json | reviewer --host=http://localhost:8080 --user admin --password admin replicate --project myproject --branch master --rule /tmp/map.txt --html /tmp/out.json | ||
5. Проанализировать созданный файл <code>/tmp/out.json.html</code> | 5. Проанализировать созданный файл <code>/tmp/out.json.html</code> | ||
6. Если результат устраивает, то загрузить обновленную разметку в сервер: | 6. Если результат устраивает, то загрузить обновленную разметку в сервер (разметка записалась в файл <code>/tmp/out.json</code>): | ||
svacer markup2 --host=<nowiki>http://localhost:8080</nowiki> --user admin --password admin import --project myproject --branch master /tmp/out.json | svacer markup2 --host=<nowiki>http://localhost:8080</nowiki> --user admin --password admin import --project myproject --branch master /tmp/out.json | ||
'''<u>''Примечания''</u>''' | '''<u>''Примечания''</u>''' | ||
Line 76: | Line 78: | ||
=== Алгоритм переноса разметки === | === Алгоритм переноса разметки === | ||
Входные данные (разметка), полученные из файла или с сервера разбиваются на два вектора source и target. В source включаются все элементы с разметкой отличной от разметки по умолчанию. В target включаются все оставшиеся элементы. | Входные данные (разметка), полученные из файла или с сервера разбиваются на два вектора source и target. В source включаются все элементы с разметкой отличной от разметки по умолчанию. В target включаются все оставшиеся элементы. Порядок элементов в source и target соответствует порядку объектов во входных данных (файле). | ||
Процедура проходит по всем парам <code>(source_item,target_item)</code> и для каждой пары вычисляет предикаты из файла с правилами в порядке их определения. Если какой либо предикат вычислен в true, то происходит копирование статусов разметки и комментариев с <code>source_item</code> на <code>target_item.</code>Во избежание перезаписи разметки можно в правилах использовать предикат <code>unprocessed()</code> описанный ниже. | Если указана опция <code>--process-all</code> , то в source и target включаются все элементы (данную опцию следует использовать для случаев, если мы хотим заместить существующую разметку, а не перенести разметку на неразмеченные элементы) | ||
Процедура проходит по всем парам <code>(source_item,target_item)</code> и для каждой пары вычисляет предикаты из файла с правилами в порядке их определения. Если какой либо предикат вычислен в true, то происходит копирование статусов разметки и комментариев с <code>source_item</code> на <code>target_item.</code> Во избежание перезаписи разметки можно в правилах использовать предикат <code>unprocessed()</code> описанный ниже. | |||
==== Файл задания предикатов переноса ==== | ==== Файл задания предикатов переноса ==== | ||
Line 85: | Line 89: | ||
Правила - это булевы выражение в синтаксисе [https://expr-lang.org/docs/language-definition go-expr] , выражения использует операторы <code>&& , ||, !, скобки</code> и следующие дополнительные предикаты: | Правила - это булевы выражение в синтаксисе [https://expr-lang.org/docs/language-definition go-expr] , выражения использует операторы <code>&& , ||, !, скобки</code> и следующие дополнительные предикаты: | ||
* <code>unprocessed()</code> - проверка, что target еще не содержит разметки. Рекомендуется использовать, чтобы избежать перезаписи разметки, если несколько правил сработали; | * <code>unprocessed()</code> - проверка, что target еще не содержит разметки. Рекомендуется использовать, чтобы избежать перезаписи разметки, если несколько правил сработали; при наличии такого предиката, разметка будет скопирована лишь один раз для первого сработавшего предиката; | ||
* <code>processed()</code> - проверка, что target уже содержит какую-либо не дефолтную разметку; | * <code>processed()</code> - проверка, что target уже содержит какую-либо не дефолтную разметку; | ||
Line 121: | Line 125: | ||
* <code>target_reviewed_by() string</code> - возвращает автора разметки в target. | * <code>target_reviewed_by() string</code> - возвращает автора разметки в target. | ||
==== '''<u>Пример</u>:''' ==== | |||
<u>Пример</u>: | |||
// Перенос разметки для Go, где поменялся формат поля function в последних Svace | // Перенос разметки для Go, где поменялся формат поля function в последних Svace | ||
Latest revision as of 09:33, 5 March 2025
Репликация разметки по правилам (экспериментальная функциональность)
При использовании разных версий Svace (или иных анализаторов), разных настроек CI/CD и прочих факторах возможны ситуации, когда снимки (snapshots) в рамках одной ветки (branch) могут иметь различные инварианты и разметка автоматически не переносится. Для решения задачи переноса разметки в таких случаях разработана утилита reviewer
, которая позволяет применить декларативно описанные правила переноса к существующей разметке. В дальнейшем данная функциональность будет включена в Svacer и доступна в веб-интерфейсе.
Использование утилиты подразумевает использование CLI команд svacer markup2 export
и svacer markup2 import
(описание markup2) для выгрузки и импортирования разметки. Утилита reviewer
обеспечивает модификацию выгружаемой разметки с целью копирования существующей разметки на неразмеченные элементы. Правила копирования задаются с использованием булевских выражений и набора предопределенных предикатов.
Описание утилиты
Утилита reviewer
предназначена для Svacer версии 10 и выше. Утилиту можно скачать отсюда Ссылка на nextcloud, утилита собрана под Linux.
NAME: reviewer - Svacer review help tool USAGE: reviewer [global options] command [command options] [arguments...] COMMANDS: syntax Prints help information about rule syntax replicate Replicate non-empty review to invariants according to rules help, h Shows a list of commands or help for one command GLOBAL OPTIONS: --debug Enabled debug output (default: false) --log-file value Define log file for output --host value Defines URL to svacer server. Format: http://host:port --user value User login. When using env variable SVACER_AUTH_CREDS the format is SVACER_AUTH_CREDS=<login>:<password> (default: admin) [$SVACER_AUTH_CREDS] --password value User password (default: admin) --ldap_server value Server for LDAP authentication [$SVACER_LDAP_SERVER] --token value Defines access token for svacer server [$SVACER_AUTH_TOKEN] --help, -h show help
Основная команда, это replicate
- она позволяет применить декларативные правила к разметке, полученной либо с сервера, либо из файла, куда ранее она была выгружена командой svacer markup2 export
.
NAME: reviewer replicate - Replicate non-empty review to invariants according to rules USAGE: reviewer replicate [command options] <file name> - path to output file with applied rules OPTIONS: --project value Project name or id --branch value Branch name or id (default: master) --file value Load review from file instead of server. File must be in 'json with new line separator' format. File may be gzipped (must have *.gz extension) --rule value Path to file containing replicate rules --html Produce HTML report showing applied changes.Report wil have name <file name>.html (default: false) --verbose Verbose output of rule applications (default: false) --process-all Process all reviews, even with non-default values (default: false) --skip-comments Skip comments while replicating review information (default: false) --skip-status Skip review status and copy only comments while replicating review information (default: false) --help, -h show help
Шаги применения утилиты
Задача: В проекте myproject на ветке master необходимо перенести существующую разметку по некоторым правилам. Сервер запущен на localhost на порте 8080.
Шаги решения:
1. Опционально, но рекомендуется выгрузить разметку с ветки в файл, используя команду:
svacer markup2 --host=http://localhost:8080 --user admin --password admin export --project myproject --branch master --uncompressed --format json --export-all /tmp/myproject.json
2. Написать правила переноса разметки (предикатов переноса) в файл /tmp/map.txt
3. Если разметка была выгружена (шаг 1 выполнен), то применить утилиту reviewer
с загрузкой данных из файла (флаг --html
создаст дополнительно HTML с информацией об изменениях , имя файла /tmp/out.json.html
):
reviewer replicate --file /tmp/myproject.json --rule /tmp/map.txt --html /tmp/out.json
4. Если разметка не была выгружена (шаг 1 пропущен) применить утилиту reviewer
с загрузкой данных с сервера (флаг --html
создаст дополнительно HTML с информацией об изменениях , имя файла /tmp/out.json.html
):
reviewer --host=http://localhost:8080 --user admin --password admin replicate --project myproject --branch master --rule /tmp/map.txt --html /tmp/out.json
5. Проанализировать созданный файл /tmp/out.json.html
6. Если результат устраивает, то загрузить обновленную разметку в сервер (разметка записалась в файл /tmp/out.json
):
svacer markup2 --host=http://localhost:8080 --user admin --password admin import --project myproject --branch master /tmp/out.json
Примечания
Разметка в Svacer привязана к инвариантам. Инвариант это некоторый хэш, формируемый из полей предупреждения. Алгоритм формирования хэша описан здесь: формирование инвариантов. В рамках одного снимка все инварианты всегда уникальны.
Команда markup2 export
с флагом --export-all
выгружает все инварианты с ветки с информацией о разметке и комментариях.
При экспорте, разметка на инварианте представлена как объект соответствующей схеме, описанной здесь Markup2. Технически - это:
- статус (тройка
status, severity, action
) - набор комментариев
- массив location-ов, которые определяют множество мест, имеющих один и тот же инвариант.
Алгоритм переноса разметки
Входные данные (разметка), полученные из файла или с сервера разбиваются на два вектора source и target. В source включаются все элементы с разметкой отличной от разметки по умолчанию. В target включаются все оставшиеся элементы. Порядок элементов в source и target соответствует порядку объектов во входных данных (файле).
Если указана опция --process-all
, то в source и target включаются все элементы (данную опцию следует использовать для случаев, если мы хотим заместить существующую разметку, а не перенести разметку на неразмеченные элементы)
Процедура проходит по всем парам (source_item,target_item)
и для каждой пары вычисляет предикаты из файла с правилами в порядке их определения. Если какой либо предикат вычислен в true, то происходит копирование статусов разметки и комментариев с source_item
на target_item.
Во избежание перезаписи разметки можно в правилах использовать предикат unprocessed()
описанный ниже.
Файл задания предикатов переноса
Формат файла: UTF-8. Правила разделяются ;;
. В качестве комментариев допустимы /**/ и //
Правила - это булевы выражение в синтаксисе go-expr , выражения использует операторы && , ||, !, скобки
и следующие дополнительные предикаты:
unprocessed()
- проверка, что target еще не содержит разметки. Рекомендуется использовать, чтобы избежать перезаписи разметки, если несколько правил сработали; при наличии такого предиката, разметка будет скопирована лишь один раз для первого сработавшего предиката;
processed()
- проверка, что target уже содержит какую-либо не дефолтную разметку;
match_loc(field1, field2, ...)
- проверка, содержат ли locations в source и target хотя бы один общий location, какой совпадает по указанным полям. Возможные поля:warnclass, function, details, invariant, line, file, mtid, tool, lang
Пример: match_loc(warnclass,file,lang,tool,line)
match_loc_strict(field1, field2, ...)
- проверка, содержат ли locations source и target одинаковые location по указанным полям.
Пример: match_loc_strict(warnclass,file,lang,tool,line)
match_source_re (field, value)
- проверка, содержит ли location в source запись, где field совпадает с value. Value - Golang regexp;
match_target_re (field, value)
- проверка, содержит ли location в target запись где field совпадает с value. Value - Golang regexp;
source_review(status[, severity, action])
- проверка, если разметка source соответствует указанным параметрам.severity, action
можно опускать, шаблон"*"
соответствует всему;
target_review(status[, severity, action])
- проверка, если разметка target соответствует указанным параметрам.severity, action
можно опускать, шаблон"*"
соответствует всему;
match_source_range (lower_bound,upper_bound)
- проверка, если в source есть location, где line попадает в указанный диапазон. Значение-1
означает отсутствие границы;
match_target_range (lower_bound,upper_bound)
- проверка, если в target есть location, где line попадает в указанный диапазон. Значение-1
означает отсутствие границы;
match_suffix_re (field, source_prefix[,target_prefix])
- проверка, есть ли location в source и target, какой содержит общий суффикс. Суффикс вычисляется путем отрезания префикса source_prefix и target_prefix. Если target_prefix не указан, то считается, что он равен source_prefix. Префикс определяется как Go regexp;
match_source_id (id1, id2, ....)
- проверка, если внутренний id source разметки совпадает с одним из аргументов;
match_target_id (id1, id2, ....)
- проверка, если внутренний id target разметки совпадает с одним из аргументов;
source_review_ct() time.Time
- возвращает время создания разметки в source;
target_review_ct() time.Time
- возвращает время создания разметки в target;
source_reviewed_by() string
- возвращает автора разметки в source;
target_reviewed_by() string
- возвращает автора разметки в target.
Пример:
// Перенос разметки для Go, где поменялся формат поля function в последних Svace unprocessed() && // ensure target has no review match_loc(warnclass,file,lang,tool,mtid, line) && // ensure main fields are the same match_suffix_re (function, "(.+/)" /* regexp for prefix */ ) // trim prefix and match suffix
HTML отчет о результатах переноса
При использовании опции --html
утилита формирует HTML файл с информацией о разметке и операциях по переносу. Файл предназначен для отладки процесса переноса. В будущем, в Svacer UI будет предусмотрен специальный интерфейс для просмотра результатов переноса.

При наличии переноса разметки в столбце с Review будет запись Review Source с ссылкой на элемент, откуда была перенесена разметка.

При раскрытии элемента Review Source будут представлены основные поля location-ов, соответствующих исходной разметке, откуда был сделан перенос.