Skip to content

refactor(seed): split seeder.go by resource#29

Merged
fmenesesg merged 1 commit into
mainfrom
refactor/seeder-split
Jun 11, 2026
Merged

refactor(seed): split seeder.go by resource#29
fmenesesg merged 1 commit into
mainfrom
refactor/seeder-split

Conversation

@fmenesesg

Copy link
Copy Markdown
Collaborator

Proposed change

Split monolithic seeder.go (~628 lines) into resource-focused files for readability and easier review. No API or behavior changes.

Type of change

  • New feature (refactor — no behavior change)

RHCL checklist

  • Tests added or updated (go test ./... or mvn test / npm test)
  • CI does not introduce -DskipTests without justification
  • No secrets in diff (tokens, kubeconfigs, OIDC secrets)
  • rhcl-ai docs updated if export/import contract changes
  • PR description and docs in English
  • Tested in local lab if applicable (seed → export → analyze)

Test plan

  • go test ./internal/seed/... — all existing tests pass
  • go test ./... — 119 tests; coverage 82.2%

Changes

File Change
internal/seed/seeder.go Core types, Run, shared helpers (~80 lines)
internal/seed/backend_seeder.go Backend + account seeding
internal/seed/service_seeder.go Product, proxy, plans, metrics
internal/seed/application_seeder.go Applications + OIDC refresh

Closes #28

Made with Cursor

Move backend/account, service/plan, and application logic into
dedicated files. Public Seeder API and behavior unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>
@fmenesesg fmenesesg requested a review from pcastelo as a code owner June 11, 2026 20:39
@fmenesesg fmenesesg added area/tests Tests unitarios e integracion type:feature New feature or enhancement labels Jun 11, 2026
@fmenesesg fmenesesg merged commit 7569072 into main Jun 11, 2026
1 check passed
@fmenesesg fmenesesg deleted the refactor/seeder-split branch June 11, 2026 20:41
fmenesesg added a commit that referenced this pull request Jun 11, 2026
refactor(seed): split seeder.go by resource
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tests Tests unitarios e integracion type:feature New feature or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] Split seeder.go by resource

1 participant