RSHB/arm-core
9
0
Fork 0
Code Issues 26 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

AURORA-1677: arm_statements теперь библиотека #178

Merged
tsygankoviva merged 10 commits from statements-lib into main 2026-11-10 13:46:32 +03:00
Conversation 3 Commits 10 Files Changed 89 +463 -12317
tsygankoviva commented 2026-10-24 11:30:10 +03:00
Owner
Copy Link
No description provided.
tsygankoviva added 1 commit 2026-10-24 11:30:11 +03:00
tsygankoviva requested review from aep 2026-10-26 11:07:27 +03:00
tsygankoviva added 1 commit 2026-10-26 18:55:26 +03:00
tsygankoviva force-pushed statements-lib from fd701b3bda to ed1ad8437f 2026-10-27 10:11:41 +03:00 Compare
aep added 2 commits 2026-10-27 15:55:32 +03:00 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aep added 1 commit 2026-10-27 16:07:36 +03:00 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aep commented 2026-10-27 16:23:40 +03:00
Owner
Copy Link

Ревью, раунд 2.

Нашу часть доделали сами прямо в ветках: влили main, обновили копии схемы в tp/sg, docs #166 переписали под библиотеку (спеки, контекст, sync-statements убрали). В make venv/install добавили сверку checkout либы с версией из requirements. Снимок для контура теперь вырезает install, без этого он падал на uv.

Тебе осталось по либе, одним выпуском 0.1.2:

  1. py.typed в arm_statements, проверь, что попадает в wheel.
  2. CHANGELOG.md, он есть в критериях задачи.
  3. version.py: ветка с чтением arm_statements/VERSION мёртвая, в пакете этого файла нет. Оставь только importlib.metadata.
  4. После тега v0.1.2 подними pylib_ai_arm_statements==0.1.2 в requirements всех трёх сервисов, STATEMENTS_TAG теперь берётся оттуда сам.

И в шапке statement_code_orders.yaml остался комментарий про sync-statements. В docs я его уже поправил (contracts/statement-code-orders.yaml в #166), возьми байтовую копию в тот же выпуск.

Ревью, раунд 2. Нашу часть доделали сами прямо в ветках: влили main, обновили копии схемы в tp/sg, docs #166 переписали под библиотеку (спеки, контекст, sync-statements убрали). В make venv/install добавили сверку checkout либы с версией из requirements. Снимок для контура теперь вырезает install, без этого он падал на uv. Тебе осталось по либе, одним выпуском 0.1.2: 1. py.typed в arm_statements, проверь, что попадает в wheel. 2. CHANGELOG.md, он есть в критериях задачи. 3. version.py: ветка с чтением arm_statements/VERSION мёртвая, в пакете этого файла нет. Оставь только importlib.metadata. 4. После тега v0.1.2 подними pylib_ai_arm_statements==0.1.2 в requirements всех трёх сервисов, STATEMENTS_TAG теперь берётся оттуда сам. И в шапке statement_code_orders.yaml остался комментарий про sync-statements. В docs я его уже поправил (contracts/statement-code-orders.yaml в #166), возьми байтовую копию в тот же выпуск.
tsygankoviva added 1 commit 2026-10-30 14:34:26 +03:00
aep added 2 commits 2026-11-02 13:29:06 +03:00 
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
aep commented 2026-11-02 13:30:06 +03:00
Owner
Copy Link

Раунд 3. 0.1.2 проверил: py.typed, CHANGELOG, мёртвая ветка version.py, пин 0.1.2 в трёх сервисах — всё на месте, make check либы зелёный (284/284).

Шапку statement_code_orders.yaml ты не взял байтовой копией — там всё ещё старый текст про sync-statements/arm-statements. Сам завёл 0.1.3 отдельным PR в pylib_ai_arm_statements (#2: RSHB/pylib_ai_arm_statements#2), после merge подниму пин здесь сам.

Остальное наше: влил main (ветка отставала на 73 коммита), поправил описание statements_version в схеме контракта (убрал «маркер VERSION» — после 0.1.2 библиотека его не читает) и копии схемы в tp/sg. make check здесь зелёный — 1631/1631.

Готов принимать после merge PR #2 в библиотеке и подъёма пина.

Раунд 3. 0.1.2 проверил: py.typed, CHANGELOG, мёртвая ветка version.py, пин 0.1.2 в трёх сервисах — всё на месте, make check либы зелёный (284/284). Шапку statement_code_orders.yaml ты не взял байтовой копией — там всё ещё старый текст про sync-statements/arm-statements. Сам завёл 0.1.3 отдельным PR в pylib_ai_arm_statements (#2: https://git.aepif.ru/RSHB/pylib_ai_arm_statements/pulls/2), после merge подниму пин здесь сам. Остальное наше: влил main (ветка отставала на 73 коммита), поправил описание statements_version в схеме контракта (убрал «маркер VERSION» — после 0.1.2 библиотека его не читает) и копии схемы в tp/sg. make check здесь зелёный — 1631/1631. Готов принимать после merge PR #2 в библиотеке и подъёма пина.
aep added 1 commit 2026-11-02 14:12:27 +03:00 
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
aep commented 2026-11-02 14:12:57 +03:00
Owner
Copy Link

0.1.3 смержен, тег v0.1.3, пин поднят в core/tabular-parser/scan-gateway, make check везде зелёный (core 1631, tp 2898, sg 927, либа 284). Раунд 3 принимаю, со своей стороны готово к merge.

0.1.3 смержен, тег v0.1.3, пин поднят в core/tabular-parser/scan-gateway, make check везде зелёный (core 1631, tp 2898, sg 927, либа 284). Раунд 3 принимаю, со своей стороны готово к merge.
tsygankoviva added 1 commit 2026-11-10 13:46:19 +03:00
tsygankoviva merged commit 994fd1b406 into main 2026-11-10 13:46:32 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
aep
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-core#178
No description provided.
Delete Branch "statements-lib"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?