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

Merged

slaid098 merged 1 commit from fix/check-permissions/replace-adr-placeholder into main 2026-08-19 01:13:51 +03:00

Conversation 1 Commits 1 Files changed 1 +2 -2

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

Owner

Copy link

Что сделано

Заменён плейсхолдер ADR-NNN на ADR-019 в обоих reason-строках для spec-status.py в .opencode/scripts/check-permissions.py (строки 26 и 31). Теперь паттерны для spec-status.py симметричны соседним (pipeline-status.py, project-status.py), которые уже ссылаются на ADR-019.

Почему

Плейсхолдер ADR-NNN никогда не был заполнен — нумерация ADR-NN отменена согласно memory skill. Несуществующая ссылка вводит в заблуждение при чтении violation-сообщений: агент видит See ADR-NNN, но такого ADR не существует, и не понимает, куда смотреть. Соседние паттерны используют корректную ссылку ADR-019 — расхождение было несимметричным. Issue #109.

Watch out

—

Pending

—

Closes #109

## Что сделано Заменён плейсхолдер `ADR-NNN` на `ADR-019` в обоих reason-строках для `spec-status.py` в `.opencode/scripts/check-permissions.py` (строки 26 и 31). Теперь паттерны для `spec-status.py` симметричны соседним (`pipeline-status.py`, `project-status.py`), которые уже ссылаются на `ADR-019`. ## Почему Плейсхолдер `ADR-NNN` никогда не был заполнен — нумерация ADR-NN отменена согласно memory skill. Несуществующая ссылка вводит в заблуждение при чтении violation-сообщений: агент видит `See ADR-NNN`, но такого ADR не существует, и не понимает, куда смотреть. Соседние паттерны используют корректную ссылку `ADR-019` — расхождение было несимметричным. Issue #109. ## Watch out — ## Pending — Closes #109

slaid098 added 1 commit 2026-08-19 01:10:57 +03:00

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

slaid098 commented 2026-08-19 01:13:42 +03:00

Author

Owner

Copy link

Code Review Summary

Тривиальный, но корректный фикс: замена stale-плейсхолдера ADR-NNN на ADR-019 в двух reason-строках для spec-status.py в DANGEROUS_PATTERNS. Изменение восстанавливает симметрию с соседними паттернами (pipeline-status.py, project-status.py), которые уже ссылаются на ADR-019.

Positives

• Точный минимальный diff (2 строки, без побочных изменений)
• Ссылка ADR-019 валидна — ADR существует (docs/decisions/019-pr-46-review-posting-tools.md), и это та же ссылка, что используется в reviewer.md и memory-syncer.md для deny-rule оракул-скриптов
• PR body заполнен осмысленно, Closes #109, title в conventional формате
• Cross-file impact проверен: rg "ADR-NNN" не находит вхождений в активном коде (только исторические docs/handoff, которые по конвенции не трогаются); тесты не ассертят старый литерал; оракулы не парсят reason-строки — breakage risk отсутствует

Suggestions (info, not blocking)

• check-permissions.py:26 [style] Reason-строки для всех трёх оракул-скриптов теперь идентичны по структуре — при желании можно вынести общий шаблон сообщения в константу, но для 6 строк это оверинжиниринг (YAGNI), оставляю как есть.

Verdict: APPROVE

## Code Review Summary Тривиальный, но корректный фикс: замена stale-плейсхолдера `ADR-NNN` на `ADR-019` в двух reason-строках для `spec-status.py` в `DANGEROUS_PATTERNS`. Изменение восстанавливает симметрию с соседними паттернами (`pipeline-status.py`, `project-status.py`), которые уже ссылаются на `ADR-019`. ### Positives - Точный минимальный diff (2 строки, без побочных изменений) - Ссылка `ADR-019` валидна — ADR существует (`docs/decisions/019-pr-46-review-posting-tools.md`), и это та же ссылка, что используется в `reviewer.md` и `memory-syncer.md` для deny-rule оракул-скриптов - PR body заполнен осмысленно, `Closes #109`, title в conventional формате - Cross-file impact проверен: `rg "ADR-NNN"` не находит вхождений в активном коде (только исторические docs/handoff, которые по конвенции не трогаются); тесты не ассертят старый литерал; оракулы не парсят reason-строки — breakage risk отсутствует ### Suggestions (info, not blocking) - **check-permissions.py:26** [style] Reason-строки для всех трёх оракул-скриптов теперь идентичны по структуре — при желании можно вынести общий шаблон сообщения в константу, но для 6 строк это оверинжиниринг (YAGNI), оставляю как есть. ### Verdict: APPROVE

slaid098 merged commit 7e71854358 into main 2026-08-19 01:13:51 +03:00

slaid098 deleted branch fix/check-permissions/replace-adr-placeholder 2026-08-19 01:13:52 +03:00

slaid098 referenced this pull request from a commit 2026-08-19 01:13:53 +03:00

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

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something is not working

  

chore

  

config

  

enhancement

  

refactor

  

security

No labels

bug

chore

config

enhancement

refactor

security

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

slaid098/opencode-config!110

No description provided.