chore(cleanup): remove stale references to deprecated docs infrastructure (#212)

Co-authored-by: opencode-agent <agent@opencode.local>
This commit is contained in:
Sergey 2026-08-01 04:43:10 +03:00 committed by GitHub
parent b2329210d5
commit 37ebcd0e08
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 7 additions and 69 deletions

View file

@ -408,7 +408,7 @@ def _extract_comment_bodies(json_str: str) -> list[str]:
def check_review(pr_number: int) -> PhaseResult:
"""Phase 5: REVIEW — APPROVE found in PR comments from code reviewer.
Looks for '## Code Review Summary' heading (NOT '## Docs Review Summary')
Looks for '## Code Review Summary' heading
with '### Verdict: APPROVE'. Only the latest reviewer comment counts
if reviewer changed from APPROVE to REQUEST_CHANGES, NOT_DONE.
"""

View file

@ -17,65 +17,3 @@ description: Use when you need to view or update the current project folder/file
1. Запусти указанную команду в терминале. Она выведет дерево каталогов и список файлов с их размерами прямо в stdout.
2. Изучи полученную структуру воркспейсов, чтобы точно знать расположение файлов и пакетов.
3. Не сохраняй вывод в файлы на диск — читай его напрямую из вывода терминала.
## Handoff файлы (docs/handoff/)
Контекст передаётся между сессиями через handoff-файлы — один файл на PR.
### Структура
- `docs/handoff/pr-<N>-<slug>.md` — handoff для PR #N
### Шаблон
```markdown
---
pr: <N>
title: <PR title>
---
## Что сделано
<2-3 строки>
## Почему
<1-2 строки>
## Pending
<что осталось, или "">
## Watch out
<gotchas, или "">
```
## ADR файлы (docs/decisions/)
Архитектурные решения сохраняются в ADR (Architecture Decision Records).
### Структура
- `docs/decisions/<NN>-pr-<N>-<slug>.md` — один файл на решение
- Numbering: `001`, `002`, `003`, ... (zero-padded, sequential)
### Шаблон
```markdown
# ADR-<NN>: <title>
## Статус
Accepted (<YYYY-MM-DD>)
## Контекст
<почему нужно было решение>
## Решение
<что решили>
## Альтернативы
- <вариант>: <почему не подошёл>
```
### Когда создавать ADR
- Новый паттерн или конвенция
- Архитектурное изменение (новый модуль, изменённые зависимости)
- Неочевидное решение (почему X, а не Y)
### Когда НЕ создавать ADR
- Bug fixes
- Refactoring without architectural change
- Documentation updates

View file

@ -55,7 +55,7 @@ function makeZodShim() {
boolean: chain,
array: chain,
object: chain,
// enum(values) — used by post-review.ts / post-docs-review.ts for
// enum(values) — used by post-review.ts for
// verdict validation. Like the other methods, the shim returns a
// chainable builder without actually validating the value (validation
// happens at the opencode zod layer, not inside execute()).
@ -74,8 +74,8 @@ function stripTs(src) {
// 5) `args: z.ZodObject` -> the args are referenced inside execute as
// `args.pr_number`; the schema itself is unused at runtime here.
// 6) Strip `: type` annotations and `async execute(args)` stays.
// 7) Strip `as const` assertions (TS-only, used by post-review.ts /
// post-docs-review.ts for tuple literal types) -> plain array literal.
// 7) Strip `as const` assertions (TS-only, used by post-review.ts
// for tuple literal types) -> plain array literal.
// 8) Strip `type <Name> = ...;` type alias declarations (TS-only) -> removed.
// 9) Strip relative imports (`import { X } from "./_shared"`) — the
// referenced module is inlined by loadTool() via inlineShared(). The

View file

@ -1,7 +1,7 @@
"""Tests for .opencode/scripts/pipeline-status.py — pipeline oracle.
All gh/git calls are mocked via monkeypatch on the module's ``run_cmd``
helper. Filesystem checks (handoff, ADR, memory) use tmp_path.
helper. Filesystem checks (PR body headings, memory) use tmp_path.
``get_repo_full_name`` is cached via ``functools.cache`` and now called by
every ``gh pr view``/``gh pr list``/``gh issue view`` site (``--repo`` flag,
@ -347,8 +347,8 @@ def test_check_review_not_done_error(monkeypatch):
assert result.status == ps.PhaseStatus.NOT_DONE
def test_check_review_false_positive_docs_reviewer_comment(monkeypatch):
"""docs-reviewer comment with '### Verdict: APPROVE' must NOT trigger check_review DONE."""
def test_check_review_false_positive_unrelated_summary_comment(monkeypatch):
"""Unrelated review-summary comment must NOT trigger check_review DONE."""
monkeypatch.setattr(
ps,
"run_cmd",