chore: align specs engineering workflow (WealthLedger/main#27) #32

Merged
codex-bot merged 3 commits from chore/27-engineering-workflow into develop 2026-10-12 11:00:37 +03:00
Owner

Summary

Подключить specs к единому issue-first Engineering Workflow, сохранив OpenSpec approval lifecycle.

Linked Issue

Refs WealthLedger/main#27; канонический документ — WealthLedger/main#28. Отдельная specs#1 остаётся за нормативной lifecycle spec.

Changes

AGENTS/README, краткие Issue/PR templates, существующий commitlint с проверкой Issue reference и исключением для release. Штатные OpenSpec skills и schemas не менялись.

Verification

  • npm run verify, node --check commitlint.config.cjs, staged diff/git diff --check — успешно.
  • Commitlint принимает cross-repo reference, отклоняет сообщение без Issue и принимает release title.
  • Native OpenSpec validation не требуется: artifacts/schema не изменялись; финальную ревизию проверяет CI.

Breaking Changes

Обычные будущие commits/PR titles требуют ссылку на Issue.

## Summary Подключить specs к единому issue-first Engineering Workflow, сохранив OpenSpec approval lifecycle. ## Linked Issue Refs WealthLedger/main#27; канонический документ — WealthLedger/main#28. Отдельная specs#1 остаётся за нормативной lifecycle spec. ## Changes AGENTS/README, краткие Issue/PR templates, существующий commitlint с проверкой Issue reference и исключением для release. Штатные OpenSpec skills и schemas не менялись. ## Verification - `npm run verify`, `node --check commitlint.config.cjs`, staged diff/`git diff --check` — успешно. - Commitlint принимает cross-repo reference, отклоняет сообщение без Issue и принимает release title. - Native OpenSpec validation не требуется: artifacts/schema не изменялись; финальную ревизию проверяет CI. ## Breaking Changes Обычные будущие commits/PR titles требуют ссылку на Issue.
codex-bot added 1 commit 2026-10-12 09:22:48 +03:00
chore: align specs engineering workflow (WealthLedger/main#27)
Conventional commits / commits (pull_request) Successful in 11s
Specs security / security (pull_request) Successful in 7s
Specs verify / verify (pull_request) Successful in 9s
a441b08378
hermes-agent was assigned by codex-bot 2026-10-12 09:32:21 +03:00
codex-bot added this to the Инженерная основа и поставка milestone 2026-10-12 09:32:22 +03:00
codex-bot added the Kind/Process
Priority
Medium
3
labels 2026-10-12 09:32:43 +03:00
codex-bot added this to the WealthLedger — Delivery Board project 2026-10-12 09:36:09 +03:00
ksv741 moved this to На проверке in WealthLedger — Delivery Board on 2026-10-12 09:51:55 +03:00
hermes-agent left a comment
Member

Ревью: требуются изменения

Блокеры

  1. AGENTS.md:7 ссылается на workflow в WealthLedger/main@develop, который сейчас отвечает 404. Документ появляется только в открытом main#28; закрепите зависимость и порядок merge либо используйте уже доступный путь.
  2. commitlint.config.cjs:6: references-empty принимает произвольные foo#bar и WealthLedger/main#abc, поэтому не гарантирует ссылку на реальную Issue в требуемом формате. Добавьте строгую проверку #<число> / <owner>/<repo>#<число> либо явно сузьте обещание политики.
  3. commitlint.config.cjs:3: release-исключение не заякорено в конце; chore(release): 1.2.3-malformed тоже игнорируется. Зафиксируйте допустимый формат полным regex.

CI на актуальном HEAD зелёный, но он не покрывает указанные негативные сценарии.

Проверено hermes-agent.

## Ревью: требуются изменения ### Блокеры 1. `AGENTS.md:7` ссылается на workflow в `WealthLedger/main@develop`, который сейчас отвечает 404. Документ появляется только в открытом `main#28`; закрепите зависимость и порядок merge либо используйте уже доступный путь. 2. `commitlint.config.cjs:6`: `references-empty` принимает произвольные `foo#bar` и `WealthLedger/main#abc`, поэтому не гарантирует ссылку на реальную Issue в требуемом формате. Добавьте строгую проверку `#<число>` / `<owner>/<repo>#<число>` либо явно сузьте обещание политики. 3. `commitlint.config.cjs:3`: release-исключение не заякорено в конце; `chore(release): 1.2.3-malformed` тоже игнорируется. Зафиксируйте допустимый формат полным regex. CI на актуальном HEAD зелёный, но он не покрывает указанные негативные сценарии. _Проверено hermes-agent._
ksv741 added 1 commit 2026-10-12 10:21:36 +03:00
fix: align specs commitlint workflow (WealthLedger/main#27)
Conventional commits / commits (pull_request) Canceled after 0s
Specs security / security (pull_request) Canceled after 0s
Specs verify / verify (pull_request) Canceled after 0s
a29cbf6aae
ksv741 added 1 commit 2026-10-12 10:22:17 +03:00
docs: pin specs workflow reference (WealthLedger/main#27)
Conventional commits / commits (pull_request) Successful in 10s
Specs security / security (pull_request) Successful in 8s
Specs verify / verify (pull_request) Successful in 10s
7f12327c37
Author
Owner

Исправления по review: workflow link pinned на доступный main@03b3b59 (main#28), commitlint исключает только точный SemVer release title (defaultIgnores:false отключает широкое встроенное SemVer-ignore), malformed suffix отвергается. references-empty не проверяет numeric syntax/existence; политика явно ограничена parser-recognized reference и numeric/correct-repo связь оставлена reviewer/manual, без custom validator. Проверены malformed reference examples как ограничение parser, malformed release fail, точный release pass, Gitea merge subject pass. Локальный npm run verify прошёл; CI на новом HEAD запущен.

Исправления по review: workflow link pinned на доступный `main@03b3b59` (main#28), commitlint исключает только точный SemVer release title (`defaultIgnores:false` отключает широкое встроенное SemVer-ignore), malformed suffix отвергается. `references-empty` не проверяет numeric syntax/existence; политика явно ограничена parser-recognized reference и numeric/correct-repo связь оставлена reviewer/manual, без custom validator. Проверены malformed reference examples как ограничение parser, malformed release fail, точный release pass, Gitea merge subject pass. Локальный `npm run verify` прошёл; CI на новом HEAD запущен.
codex-bot merged commit 78f01066d2 into develop 2026-10-12 11:00:37 +03:00
ksv741 moved this to Готово in WealthLedger — Delivery Board on 2026-10-12 18:06:37 +03:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/specs#32