[CL-DOCS-UPDATE] docs(architecture): follow-up — post-merge re-verify findings #73

Merged

andrei merged 2 commits from claude-audit/docs-followup-20260519 into master 2026-05-18 22:08:32 +00:00

Conversation 0 Commits 2 Files changed 2 +7 -7

andrei commented 2026-05-18 21:44:01 +00:00

Owner

Copy link

Что сделано

Follow-up к PR #72 (365c44f05). Post-merge final re-verify ARCHITECTURE.md обнаружил 4 оставшихся drift, не пойманных iter 1/2/3.

• L210: Sentry init path logger.ts → instrument.ts (BLOCKER)
• L377: rate-limiter count 17 → 27 (BLOCKER, cross-doc inconsistency с L222)
• L75: route files 127 → 128 (WARNING)
• L120: schema files 31 → 32 (WARNING)

Commit: 883aa13ed. Husky pre-push pass (vitest 189).

План тестирования

• Подтверждены все 4 факта через grep/find/ls
• Husky pre-push pass

Где могу ошибиться

• Другие 7 generated docs могут иметь similar drift не покрытый в этом follow-up.
• Workflow MAX_FIX_ITERATIONS=2 был в основном PR — это independent follow-up scope.

🤖 Generated with Claude Code via /gsd-docs-update (post-merge follow-up)

## Что сделано Follow-up к PR #72 (`365c44f05`). Post-merge final re-verify ARCHITECTURE.md обнаружил 4 оставшихся drift, не пойманных iter 1/2/3. - **L210**: Sentry init path `logger.ts` → `instrument.ts` (BLOCKER) - **L377**: rate-limiter count `17` → `27` (BLOCKER, cross-doc inconsistency с L222) - **L75**: route files `127` → `128` (WARNING) - **L120**: schema files `31` → `32` (WARNING) Commit: `883aa13ed`. Husky pre-push pass (vitest 189). ## План тестирования - [x] Подтверждены все 4 факта через grep/find/ls - [x] Husky pre-push pass ## Где могу ошибиться - Другие 7 generated docs могут иметь similar drift не покрытый в этом follow-up. - Workflow MAX_FIX_ITERATIONS=2 был в основном PR — это independent follow-up scope. 🤖 Generated with Claude Code via /gsd-docs-update (post-merge follow-up)

andrei added 1 commit 2026-05-18 21:44:02 +00:00

[CL-DOCS-UPDATE] docs(architecture): follow-up — post-merge re-verify findings ... 883aa13edc

После merge PR #72 (`365c44f05`) запущен финальный re-verify докumenta
docs/ARCHITECTURE.md обнаружил 4 оставшихся неточности (2 BLOCKERs + 2 WARNINGs),
которые iter 1/2/3 fix loop пропустил.

## Что сделано

### BLOCKER #1 (line 210) — Sentry init path
- Было: `api/src/lib/logger.ts`
- Стало: `api/src/instrument.ts` (Sentry init, imported first in `api/src/index.ts`)
- Source: `logger.ts` не содержит ни одной Sentry-ссылки; реальный init в `instrument.ts`

### BLOCKER #2 (line 377) — cross-doc rate-limiter inconsistency
- Было: "17 Redis-backed limiters"
- Стало: "27 Redis-backed limiters"
- Source: grep `export const ` в api/src/middleware/rateLimit.ts = 27
- L222 уже было 27 (iter 3 patch), но L377 в Security секции пропущен — теперь consistent.

### WARNING #1 (line 75) — route file count
- Было: 127 route files
- Стало: 128 route files
- Source: рекурсивный grep `*routes.ts` в api/src/routes/ = 128

### WARNING #2 (line 120) — schema file count
- Было: 31 API schema files
- Стало: 32 API schema files
- Source: ls api/src/schemas/*.ts = 32

## Зачем

PR #72 merged via squash commit `365c44f05`. Post-merge final re-verify
обнаружил эти drift'ы. Workflow MAX_FIX_ITERATIONS=2 в основном PR
запрещал iter 3+ автоматически, но как follow-up PR это допустимо.

## План тестирования

- [x] Подтверждено `api/src/instrument.ts` содержит Sentry init
- [x] Подтверждено 27 limiters в `api/src/middleware/rateLimit.ts`
- [x] Подтверждено 128 routes.ts через find
- [x] Подтверждено 32 schemas .ts через ls
- [x] Husky pre-push готов запустить tsc + build + vitest

## Где могу ошибиться

- Может остаться ещё drift в ARCHITECTURE.md, не пойманный re-verify
  (например claims пропущенные категориями extraction rules). Iter 4 не
  выполнен — это would be 4th iter, явно вне workflow scope.
- Другие 7 generated docs (CONFIGURATION/API/DEPLOYMENT/DEVELOPMENT/
  app|api|contracts/README) тоже могут иметь stale verify-*.json
  artifacts с inflated failure counts. Re-verify не выполнен на них в
  этом follow-up.

🤖 Generated with Claude Code via /gsd-docs-update (post-merge follow-up)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

andrei added 1 commit 2026-05-18 21:58:15 +00:00

[CL-DOCS-UPDATE] docs(contracts): follow-up — post-merge re-verify contracts/README fixes ... 6a30b44e80

Post-merge re-verify contracts/README.md обнаружил дополнительные drift'ы.

## Что сделано

- **L53**: `BACKEND_SETTLED` / `USER_SETTLED` → `BackendSettled` / `UserSettled` (actual enum values per `EuropaP2PEscrow.sol:54-55`)
- **L175**: gas baseline `153 methods` → `162 methods` (per `python3 -c 'import json; print(len(json.load(open("gas-baseline.json"))["methods"]))'`)
- **L251**: doc paths `docs/SECURITY-AUDIT-*.md` → `../docs/SECURITY-AUDIT-*.md` (correct relative path from `contracts/`)

## Зачем

`contracts/README.md` post-merge re-verify (43 claims, 6 failures). 3 из 6 устранены здесь. 3 пропущены:
- L113 `privateKeyToAccount(key)` — external viem reference (Rule A crypto safety guideline), не contract code
- L113 другой privateKeyToAccount instance — same
- Один минор о docs path duplicate

## План тестирования

- [x] Подтверждено enum values в `EuropaP2PEscrow.sol:54-55`
- [x] Подтверждено 162 methods в `gas-baseline.json`
- [x] Подтверждены оба audit reports в `../docs/` (monorepo root)
- [x] Husky pre-push готов запустить

🤖 Generated with Claude Code via /gsd-docs-update (post-merge follow-up batch 2)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

andrei merged commit 3a543d983d into master 2026-05-18 22:08:32 +00:00

andrei referenced this pull request from a commit 2026-05-18 22:08:34 +00:00

[CL-DOCS-UPDATE] docs: follow-up post-merge re-verify fixes (#73)

andrei referenced this pull request 2026-05-18 22:56:01 +00:00

[CL-DOCS-UPDATE] docs: batch 3 — API routes + DEPLOY flags + app langs #75

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

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

europa-tech-srl/europatech!73

No description provided.