chore: align backend engineering workflow (WealthLedger/main#27) #76
No Assignees
hermes-agent
hermes-agent
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: WealthLedger/backend#76
Reference in New Issue
Block a user
Summary
Подключить backend к единому issue-first Engineering Workflow.
Linked Issue
Refs WealthLedger/main#27; канонический документ — WealthLedger/main#28.
Changes
AGENTS и локальная Git/SDD convention, краткие Issue/PR templates; существующий commitlint проверяет ссылку на Issue с исключением для release.
Verification
node --check commitlint.config.cjs,git diff --checkи staged diff — успешно.Breaking Changes
Обычные будущие commits/PR titles требуют ссылку на Issue.
Ревью: замечание по enforcement
CI на актуальном HEAD зелёный;
git diff --checkиnode --check commitlint.config.cjsпроходят.В
.ai/conventions/engineering.md:83-88описаны issue-linked ветки и обязательная Issue reference, но.gitea/PULL_REQUEST_TEMPLATE.md:4содержит необязательныйRefs, а.gitea/workflows/verify.yml:30-37проверяет только commitlint. Это оставляет правило advisory: PR без реальной связи с Issue пройдёт CI.Либо добавьте соразмерную проверку branch/PR body, либо явно назовите это ограничение enforcement в документации, чтобы критерий #27 не создавал ложного ощущения автоматической защиты.
Проверено hermes-agent.
По замечанию review: branch naming и фактическая связь PR с существующей Issue прямо обозначены как manual/advisory. Commitlint проверяет только Conventional Commit и наличие parser-recognized reference; он не валидирует branch ↔ Issue, числовой формат или существование Issue.
npm run verifyи локальный commitlint прошли; CI на последнем HEAD запущен.