Skip to content

Hernoem package naar nl.rijksoverheid.moz.fbs.<module>#35

Open
ericwout-overheid wants to merge 1 commit intomainfrom
chore/fbs-package-namespace
Open

Hernoem package naar nl.rijksoverheid.moz.fbs.<module>#35
ericwout-overheid wants to merge 1 commit intomainfrom
chore/fbs-package-namespace

Conversation

@ericwout-overheid
Copy link
Copy Markdown
Contributor

Samenvatting

  • Introduceert een expliciete fbs-productnamespace onder de MOZ-groupId
  • Maakt één packageconventie die zowel services (services/<naam>) als libraries (libraries/<naam>) dekt: nl.rijksoverheid.moz.fbs.<module-naam>.*
  • Alleen services/berichtensessiecache raakt geraakt (enige bestaande module in main); libraries/fbs-common en services/berichtenmagazijn landen in PR Issue #410: Berichtenmagazijn module met Aanlever API #31 direct met de nieuwe conventie

Waarom

In PR #31 komt libraries/fbs-common/ met package nl.rijksoverheid.moz.fbs.common.* — dat is incongruent met sessiecache's nl.rijksoverheid.moz.berichtensessiecache.* (zonder .fbs.). Twee mogelijkheden:

  1. De library volgt sessiecache en wordt nl.rijksoverheid.moz.fbscommon.*
  2. Sessiecache trekt bij en krijgt nl.rijksoverheid.moz.fbs.berichtensessiecache.*

Optie 2 is semantisch netter: moz is de MOZ-organisatie, fbs is het Federatief Berichtenstelsel-product, daaronder vallen modules (services én libraries). Dit reserveert ruimte voor toekomstige niet-FBS-projecten onder dezelfde moz-groupId.

Scope

Mechanische hernoeming — geen gedragswijziging:

  • 38 Kotlin-bestanden via git mv verhuisd van nl/rijksoverheid/moz/berichtensessiecache/ naar nl/rijksoverheid/moz/fbs/berichtensessiecache/
  • package-regels en import-statements aangepast (76 occurrences)
  • services/berichtensessiecache/pom.xml: apiPackage, modelPackage en JaCoCo-exclude-pattern bijgewerkt
  • CLAUDE.md: regel over package-conventie herschreven naar nl.rijksoverheid.moz.fbs.<module-naam>.* met toelichting dat fbs bewust een productnamespace reserveert

Volgorde

Merge deze PR vóór PR #31. Daarna rebase feature/berichtenmagazijn-aanlever-api op main en landt de nieuwe berichtenmagazijn-module direct onder nl.rijksoverheid.moz.fbs.berichtenmagazijn.*.

Test plan

  • ./mvnw -pl services/berichtensessiecache compile — BUILD SUCCESS
  • Alle 125 non-Docker tests slagen (./mvnw -pl services/berichtensessiecache surefire:test -Dtest='!RedisBerichtenCacheIntegrationTest')
  • RedisBerichtenCacheIntegrationTest in CI (vereist Testcontainers/Docker; lokaal niet beschikbaar in deze omgeving — dezelfde test faalt óók op onveranderde main, geen rename-regressie)
  • JaCoCo 90%-drempel haalt in CI (lokaal alleen gehaald mét Redis-test)

🤖 Generated with Claude Code

Introduceert een expliciete `fbs`-productnamespace onder de MOZ-groupId,
zodat services en libraries consistent onder hetzelfde prefix leven. Bereidt
voor op de komst van `libraries/fbs-common` en `services/berichtenmagazijn`
(PR #31) zodat die niet een afwijkende packagestructuur introduceren.

Puur mechanische rename: bestanden verhuisd met git mv, package-regels en
imports geüpdatet, POM (apiPackage, modelPackage, JaCoCo exclude) bijgewerkt,
conventie in CLAUDE.md aangepast. Geen gedragswijziging.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ericwout-overheid ericwout-overheid requested a review from a team as a code owner April 16, 2026 15:53
@ericwout-overheid ericwout-overheid requested review from mreuvekamp and removed request for a team April 16, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant