C9S/mono
3
0
Fork 0
Code Issues 2 Pull Requests 11 Actions Packages Projects 3 Releases Wiki Activity

[105-9] feat: cron-уборка истёкших access_rules (Story 6.7 backend-слайс) #142

Closed
claude wants to merge 1 commits from feat/coopid-epic-6-access-rules-cleanup into feat/coopid-epic-12
pull from: feat/coopid-epic-6-access-rules-cleanup
merge into: C9S:feat/coopid-epic-12
Conversation 1 Commits 1 Files Changed 5
+92 -1
claude commented 2026-06-14 22:32:35 +00:00
Owner
Copy Link
Copy Source

Безопасный не-визуальный бэкенд-слайс заблокированной Story 6.7 (TTL capability).

Истёкшие точечные права (access_rules.expires_at) при истечении уже инертны — read-path их исключает (expires_at <= now), доступа не дают. Мёртвые строки копились вечно. Добавлены:

  • порт IAccessRulesRepository.deleteExpired(now)
  • PostgresAccessRulesRepository: DELETE ... WHERE expires_at IS NOT NULL AND expires_at <= now RETURNING (детерминированный подсчёт, как в capability-sets)
  • AccessRulesCleanupService: @Cron ежедневно (прецедент CriticalActionsService.expireStale), зарегистрирован в AuthorizationModule

Ключевой инвариант: удаление != отзыв — ничьи фактические права не меняются → без инвалидации сессий и аудита. Удаляются только истёкшие с непустым TTL; бессрочные и действующие не трогаются.

Тесты: jest 3/3 (вызов с Date, ноль-кейс, проброс сбоя), ESLint 0, тайп-чек ts-jest.

Base — feat/coopid-epic-12 (текущий тип кумулятивного CoopID-стека; access_rules пришёл из Эпика 6, в dev ещё не слит). Нижние ветки стека не трогались.

🤖 Generated with Claude Code

Безопасный не-визуальный бэкенд-слайс заблокированной Story 6.7 (TTL capability). Истёкшие точечные права (access_rules.expires_at) при истечении уже инертны — read-path их исключает (expires_at <= now), доступа не дают. Мёртвые строки копились вечно. Добавлены: - порт IAccessRulesRepository.deleteExpired(now) - PostgresAccessRulesRepository: DELETE ... WHERE expires_at IS NOT NULL AND expires_at <= now RETURNING (детерминированный подсчёт, как в capability-sets) - AccessRulesCleanupService: @Cron ежедневно (прецедент CriticalActionsService.expireStale), зарегистрирован в AuthorizationModule Ключевой инвариант: удаление != отзыв — ничьи фактические права не меняются → без инвалидации сессий и аудита. Удаляются только истёкшие с непустым TTL; бессрочные и действующие не трогаются. Тесты: jest 3/3 (вызов с Date, ноль-кейс, проброс сбоя), ESLint 0, тайп-чек ts-jest. Base — feat/coopid-epic-12 (текущий тип кумулятивного CoopID-стека; access_rules пришёл из Эпика 6, в dev ещё не слит). Нижние ветки стека не трогались. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
claude added 1 commit 2026-06-14 22:32:36 +00:00
[105-9][@ant] feat: cron-уборка истёкших access_rules — гигиена таблицы прав без влияния на доступ (Story 6.7) d59dc7b796 
Точечные права с TTL (expires_at) при истечении уже инертны — read-path
findForPrincipal/findForCapabilitySets исключает их (expires_at <= now), доступа
они не дают. Но мёртвые строки копились вечно. Добавлен AccessRulesCleanupService
(@Cron ежедневно, прецедент CriticalActionsService.expireStale) + порт-метод
deleteExpired + DELETE ... WHERE expires_at IS NOT NULL AND expires_at <= now
RETURNING (детерминированный подсчёт, как в capability-sets). Удаление != отзыв:
ничьи фактические права не меняются → без инвалидации сессий и аудита.
Зачем: завершает TTL-фичу (6.7) — таблица прав не растёт бесконечно, выборка
прав не деградирует; безопасный не-визуальный бэкенд-слайс заблокированной 6.7.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ant added this to the CoopID project 2026-06-15 13:17:41 +00:00
claude commented 2026-06-15 13:26:35 +00:00
Author
Owner
Copy Link
Copy Source

Схлопнут в единый PR #148 (coopID → dev). Ветка сохранена как резерв, ничего не потеряно.

Схлопнут в единый PR #148 (coopID → dev). Ветка сохранена как резерв, ничего не потеряно.
claude closed this pull request 2026-06-15 13:26:48 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project CoopID
Assignees
Clear assignees
ant (Алексей Муравьев) claude valeriy
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: C9S/mono#142
Delete Branch "feat/coopid-epic-6-access-rules-cleanup"

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?