Skip to content

refactor: corrige violações de arquitetura (PopcornGP) - #99

Open
gabrielbmoro wants to merge 1 commit into
masterfrom
gmoro/solve-arc-violations
Open

refactor: corrige violações de arquitetura (PopcornGP)#99
gabrielbmoro wants to merge 1 commit into
masterfrom
gmoro/solve-arc-violations

Conversation

@gabrielbmoro

Copy link
Copy Markdown
Collaborator

Descrição

Correção de violações de arquitetura detectadas pelo PopcornGP:

  • Extrai interfaces DetailStreamRepository e ProfilePickerStreamRepository para a camada domain
  • Move SearchMapper de domain para presentation
  • Atualiza popcorngp-config.json para refletir as novas regras

Módulos afetados

  • feature-detail
  • feature-profile
  • feature-search
  • build-logic / config
  • gradle / libs.versions.toml

Testes Realizados

  • Build local passa
  • Detekt / PopcornGP sem novas violações

Checklist

  • Os testes foram executados e passaram com sucesso.
  • As alterações de código seguem as diretrizes de estilo do projeto.
  • Foram adicionados testes, se aplicável.
  • Se inscreveu no canal?😛

@gabrielbmoro
gabrielbmoro requested a review from a team as a code owner August 3, 2026 20:25
"forbiddenDependencies": [
"^.*/domain/.+\\.kt$",
"^.*/presentation/.+\\.kt$"
".*\\.presentation\\..+"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

codigo que tem 'data' no path não pode depender de presentation

"filePattern": "^.*/domain/.+\\.kt$",
"forbiddenDependencies": [
"^.*/presentation/.+\\.kt$"
".*\\.presentation\\..+"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

codigo que tem 'domain' no path não pode depender de presentation

]
},
{
"filePattern": "^.*/.+Handler\\.kt$",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*/Handler.kt não podem ter dependencias internas (de projeto)

Comment thread gradle/libs.versions.toml

#Test
popcornguineapigDetektRule = "1.0.0"
popcornguineapigDetektRule = "1.0.1"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bump lib

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