fix(check-permissions): replace stale ADR-NNN placeholder in spec-status reasons #109

New issue

Closed

opened 2026-08-19 01:08:53 +03:00 by slaid098 · 0 comments

slaid098 commented

2026-08-19 01:08:53 +03:00

Owner

Контекст

В .opencode/scripts/check-permissions.py паттерны для spec-status.py (строки 26 и 31) содержат reason "use spec-status tool, not bash. See ADR-NNN" — плейсхолдер ADR-NNN, который никогда не был заполнен. Соседние паттерны (pipeline-status.py, project-status.py) используют ADR-019. Согласно memory skill, нумерация ADR-NN отменена — ссылка на несуществующий ADR вводит в заблуждение при чтении violation-сообщений.

Задача

Заменить See ADR-NNN на See ADR-019 в обоих reason для spec-status.py.

Контракты

Строки 26 и 31: reason → "use spec-status tool, not bash. See ADR-019".
Ничего больше не менять.

Инварианты

Regex-паттерны не меняются.
Scope "agent" не меняется.
Тесты зелёные.

Граничные случаи

Тесты, проверяющие reason для spec-status (если есть) — обновить ожидания.

Влияние на связанные компоненты

Только .opencode/scripts/check-permissions.py.
CI permissions-check.yml — black-box (exit code), не затронут.

Вне scope

Рефакторинг DANGEROUS_PATTERNS.
Изменение других reason-строк.

Критерии приемки

rg "ADR-NNN" .opencode/scripts/ — пусто.
uv run pytest tests/test_check_permissions.py -q — зелёные.
python3 .opencode/scripts/check-permissions.py — exit 0.

## Контекст В `.opencode/scripts/check-permissions.py` паттерны для `spec-status.py` (строки 26 и 31) содержат reason `"use spec-status tool, not bash. See ADR-NNN"` — плейсхолдер `ADR-NNN`, который никогда не был заполнен. Соседние паттерны (`pipeline-status.py`, `project-status.py`) используют `ADR-019`. Согласно memory skill, нумерация ADR-NN отменена — ссылка на несуществующий ADR вводит в заблуждение при чтении violation-сообщений. ## Задача Заменить `See ADR-NNN` на `See ADR-019` в обоих reason для `spec-status.py`. ## Контракты - Строки 26 и 31: reason → `"use spec-status tool, not bash. See ADR-019"`. - Ничего больше не менять. ## Инварианты - Regex-паттерны не меняются. - Scope `"agent"` не меняется. - Тесты зелёные. ## Граничные случаи - Тесты, проверяющие reason для spec-status (если есть) — обновить ожидания. ## Влияние на связанные компоненты - Только `.opencode/scripts/check-permissions.py`. - CI `permissions-check.yml` — black-box (exit code), не затронут. ## Вне scope - Рефакторинг `DANGEROUS_PATTERNS`. - Изменение других reason-строк. ## Критерии приемки - `rg "ADR-NNN" .opencode/scripts/` — пусто. - `uv run pytest tests/test_check_permissions.py -q` — зелёные. - `python3 .opencode/scripts/check-permissions.py` — exit 0.

slaid098 added the

bug

label

2026-08-19 01:08:53 +03:00

slaid098 referenced this issue

2026-08-19 01:09:04 +03:00

fix(check-permissions): add project-status.py to DANGEROUS_PATTERNS #108

slaid098 referenced this issue from a pull request that will close it,

2026-08-19 01:10:57 +03:00

fix(check-permissions): replace stale ADR-NNN placeholder in spec-status reasons #110

slaid098 closed this issue

2026-08-19 01:13:52 +03:00