[Task] Review the file rename path without owner context: move remaining callers to the owner-scoped bulk path where possible, or keep the fallback explicitly legacy with regres... #1

Closed
opened 2026-07-06 11:35:06 +02:00 by zemion · 1 comment
Owner

Imported from the consolidated GovOPlaN product backlog.

  • Source: /mnt/DATA/Nextcloud/ADD ideas UG/Products/govoplan/backlog.md
  • Line: 55
  • Section: Inbox

Imported backlog item:

- [ ] Review the file rename path without owner context: move remaining callers to the owner-scoped bulk path where possible, or keep the fallback explicitly legacy with regression coverage for access checks.
<!-- codex-backlog-fingerprint:3e816db119d38abe5c42eafc --> Imported from the consolidated GovOPlaN product backlog. - Source: `/mnt/DATA/Nextcloud/ADD ideas UG/Products/govoplan/backlog.md` - Line: `55` - Section: `Inbox` Imported backlog item: ```markdown - [ ] Review the file rename path without owner context: move remaining callers to the owner-scoped bulk path where possible, or keep the fallback explicitly legacy with regression coverage for access checks. ```
zemion added this to the Inbox milestone 2026-07-06 13:21:22 +02:00
Author
Owner

Codex State: done

Summary

  • Bulk rename is now owner-scoped at the public API boundary, and the historical no-owner file-only path is isolated behind an explicit legacy helper.

Changed Files

  • src/govoplan_files/backend/schemas.py
  • src/govoplan_files/backend/storage/transfers.py
  • src/govoplan_files/backend/storage/services.py
  • webui/src/api/files.ts
  • README.md
  • ../govoplan-core/tests/test_api_smoke.py

Verification

  • python -m unittest tests.test_api_smoke.ApiSmokeTests.test_managed_file_runtime_flow
  • python -m unittest tests.test_module_system.ModuleSystemTests.test_enabled_module_permutations_register_expected_routes tests.test_module_system.ModuleSystemTests.test_module_permutations_start_when_absent_modules_are_physically_unavailable
  • python scripts/check_dependency_boundaries.py
  • npm run test:module-permutations
## Codex State: done ### Summary - Bulk rename is now owner-scoped at the public API boundary, and the historical no-owner file-only path is isolated behind an explicit legacy helper. ### Changed Files - `src/govoplan_files/backend/schemas.py` - `src/govoplan_files/backend/storage/transfers.py` - `src/govoplan_files/backend/storage/services.py` - `webui/src/api/files.ts` - `README.md` - `../govoplan-core/tests/test_api_smoke.py` ### Verification - `python -m unittest tests.test_api_smoke.ApiSmokeTests.test_managed_file_runtime_flow` - `python -m unittest tests.test_module_system.ModuleSystemTests.test_enabled_module_permutations_register_expected_routes tests.test_module_system.ModuleSystemTests.test_module_permutations_start_when_absent_modules_are_physically_unavailable` - `python scripts/check_dependency_boundaries.py` - `npm run test:module-permutations`
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: add-ideas/govoplan-files#1
No description provided.