feat(authoring): rename document types, stock and templates natively #3

Merged
james.maes merged 1 commits from feat/definition-rename-20261106 into develop 2026-11-06 15:32:52 +00:00
Owner

Carried locally by GG Forge as definition-rename.patch (development pin 0.1.0-forge.ee782b7d6ea8-rename1). Upstreaming it so a published release can carry the change and Forge can drop that pin.

Change

  • DocumentDefinitionService.rename(table, id, name) plus native authoring actions document_generation_{type,stock,template}_author_rename ("Rename"). The form loads the current name and saves only the trimmed name: nonblank, at most 200 characters, no control characters.
  • document-generation.configure is checked at discovery/preparation and again at save; credential/channel write ceilings still apply. Native UpdateAction keeps the host's field-level audit.
  • GenerationStore.update admits an id+name-only write for type and stock rows; geometry, bindings, drafts, published revisions and queued pins remain immutable. No schema change.
  • Prepare-step selection message generalised from "document template" to "document definition" since type/stock rows now use it.
  • Docs: docs/AUTHORING.md describes the action; README.md and docs/RELEASE-STATUS.md record the addition and verification. The repo keeps no CHANGELOG.

Verification

./mvnw -pl foundation -am verify on this branch (Java 21, wrapper, offline local repository): 232 tests, 0 failures/errors/skips, Checkstyle clean, JaCoCo gates met, including the Docker MySQL 8.4 and PostgreSQL 17 suites. Forge's prior evidence: 37 rename-related tests on H2, MySQL and PostgreSQL 17 (rename-linux-verify.log, rename-linux-db-verify.log).

🤖 Generated with Claude Code

Carried locally by GG Forge as `definition-rename.patch` (development pin `0.1.0-forge.ee782b7d6ea8-rename1`). Upstreaming it so a published release can carry the change and Forge can drop that pin. ## Change - `DocumentDefinitionService.rename(table, id, name)` plus native authoring actions `document_generation_{type,stock,template}_author_rename` ("Rename"). The form loads the current name and saves only the trimmed name: nonblank, at most 200 characters, no control characters. - `document-generation.configure` is checked at discovery/preparation and again at save; credential/channel write ceilings still apply. Native `UpdateAction` keeps the host's field-level audit. - `GenerationStore.update` admits an id+name-only write for type and stock rows; geometry, bindings, drafts, published revisions and queued pins remain immutable. No schema change. - Prepare-step selection message generalised from "document template" to "document definition" since type/stock rows now use it. - Docs: `docs/AUTHORING.md` describes the action; `README.md` and `docs/RELEASE-STATUS.md` record the addition and verification. The repo keeps no CHANGELOG. ## Verification `./mvnw -pl foundation -am verify` on this branch (Java 21, wrapper, offline local repository): 232 tests, 0 failures/errors/skips, Checkstyle clean, JaCoCo gates met, including the Docker MySQL 8.4 and PostgreSQL 17 suites. Forge's prior evidence: 37 rename-related tests on H2, MySQL and PostgreSQL 17 (`rename-linux-verify.log`, `rename-linux-db-verify.log`). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
james.maes added 1 commit 2026-11-06 15:00:02 +00:00
feat(authoring): rename document types, stock and templates natively
QRun-IO/ci-policy: ci / fork-guard (pull_request) Skipped
QRun-IO/ci-policy: native / fork-deny (pull_request) Skipped
QRun-IO/ci-policy: native / dotnet-0 (pull_request) Skipped
QRun-IO/ci-policy: ci / Scoped workflow [workflow 1e25bb5a243dfdcd95d4b9a4f780200ff8c8778a] (pull_request) Successful in 12s
QRun-IO/ci-policy: ci / head-profile (pull_request) Skipped
QRun-IO/ci-policy: ci / repository-policy (pull_request) Skipped
QRun-IO/ci-policy: ci / application-template-0 (pull_request) Skipped
QRun-IO/ci-policy: ci / gradle-1 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-core-2-head (pull_request) Skipped
QRun-IO/ci-policy: ci / java-docker-3 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-docker-3-head (pull_request) Skipped
QRun-IO/ci-policy: ci / java-docker-4 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-frontend-browser-5 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-maven-6 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-maven-6-head (pull_request) Skipped
QRun-IO/ci-policy: ci / java-maven-7 (pull_request) Skipped
QRun-IO/ci-policy: ci / java-maven-7-head (pull_request) Skipped
QRun-IO/ci-policy: ci / node-8 (pull_request) Skipped
QRun-IO/ci-policy: ci / node-9 (pull_request) Skipped
QRun-IO/ci-policy: native / native-result (pull_request) Successful in 12s
QRun-IO/ci-policy: ci / node-10 (pull_request) Skipped
QRun-IO/ci-policy: ci / node-playwright-11 (pull_request) Skipped
QRun-IO/ci-policy: ci / repository-policy-12 (pull_request) Skipped
QRun-IO/ci-policy: ci / source-config-13 (pull_request) Skipped
QRun-IO/ci-policy: ci / source-config-14 (pull_request) Skipped
QRun-IO/ci-policy: ci / android-0 (pull_request) Skipped
QRun-IO/ci-policy: ci / original-business (pull_request) Skipped
QRun-IO/ci-policy: ci / original-mcp (pull_request) Skipped
QRun-IO/ci-policy: ci / original-voyage (pull_request) Skipped
QRun-IO/ci-policy: ci / original-cli (pull_request) Skipped
QRun-IO/ci-policy: ci / shared-pipeline (pull_request) Skipped
QRun-IO/ci-policy: ci / not-enrolled (pull_request) Failing after 9s
QRun-IO/ci-policy: ci / ci-result (pull_request) Failing after 11s
313f081026
Add DocumentDefinitionService.rename and the native authoring actions
document_generation_{type,stock,template}_author_rename. The form loads
the current name and saves only the trimmed name; names must be nonblank,
at most 200 characters and free of control characters. The configure
permission is checked at discovery and again at save, and the native
UpdateAction keeps the host's field-level audit. GenerationStore.update
now admits an id+name-only write for type and stock rows, so geometry,
bindings, drafts, published revisions and queued pins stay immutable.
No schema change.

Carried locally by GG Forge as definition-rename.patch (pin
0.1.0-forge.ee782b7d6ea8-rename1); upstreamed so a published release can
replace that pin. The prepare-step selection message is generalised to
cover type and stock rows. Docs: AUTHORING.md describes the action,
README and RELEASE-STATUS record the addition and its verification
(./mvnw -pl foundation -am verify: 232 tests, Checkstyle and coverage
gates met, including the Docker MySQL and PostgreSQL suites).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Signed-off-by: James Maes <james@kof22.com>
james.maes merged commit d3c0062343 into develop 2026-11-06 15:32:52 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: QRun-IO/qbit-document-generation#3