You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
refactor(admin-frontend): tipo das props de cada campo fica no arquivo do campo - #217
O shared/form/fields/fields.types.ts passa a ser genérico. Ele não importa mais nada de shared/ui e não conhece nenhum campo. O tipo das props de cada campo e a lista das props omitidas ficam no arquivo do próprio campo, em components/, ao lado do primitivo que o campo renderiza. É o modelo que o #207 adotou para o SearchSelectField no commit 2ae5e6c.
A mudança é só de estrutura de tipos. Nenhum comportamento muda, e o barril fields/index.tsx continua exportando os mesmos 14 nomes de tipo, agora a partir do arquivo de cada campo. Nenhum consumidor fora de fields/ foi tocado.
Inventário
O que o fields.types.ts importava de shared/ui na main:
Import de shared/ui
Tipo que o usava
Campo
Para onde foi
Input
TextFieldProps, MaskedFieldProps, DateFieldProps
TextField, MaskedField, DateField
text-fields.tsx, masked-field.tsx, date-field.tsx
Textarea
TextareaFieldProps
TextareaField
text-fields.tsx
InputGroupInput
SearchFieldProps
SearchField
search-field.tsx
ColorSwatchPickerOption
ColorFieldProps
ColorField
color-field.tsx
FormFieldControlProps
ControlledFieldProps
ControlledField
controlled-field.tsx
Tipos que pertencem a um campo só, sem import de shared/ui:
Um campo construído sobre outro importa o tipo base do arquivo do campo que ele já renderiza: PhoneField e EmailField usam text-fields, CpfField usa masked-field e BirthDateField usa date-field. O teste do SelectField passou a importar SelectFieldOption de ./select-field.
O que ficou em fields.types.ts, exportado porque vários campos usam:
BaseFieldProps: TextField, TextareaField, MaskedField e DateField;
RegisterOwnedProps e FormFieldOwnedProps: os mesmos quatro, mais o SearchField;
WithOptionalLabel: CpfField, PhoneField, EmailField e BirthDateField;
FieldLabelPlacement: SelectField e SearchField, a união tipada uma vez só, como registra a ADR 0076.
Nenhum tipo exigiu escolha. Os tipos compartilhados são genéricos e ficam em fields.types.ts. Os derivados seguem a dependência que o componente já tem, e a API pública do barril não muda.
Orientação para os próximos campos
.claude/skills/agenza-form-field/SKILL.md: o tipo das props de um campo novo nasce no arquivo do campo e é reexportado de lá pelo barril. O fields.types.ts só guarda as peças compartilhadas e não importa de shared/ui.
apps/admin-frontend/docs/ARCHITECTURE.md: a mesma regra, na seção de componentes e formulários.
Não abri ADR nova. É uma convenção de organização local, reversível e sem efeito em fronteira, contrato, persistência ou operação, então se encaixa no guia de arquitetura e na skill (ADR 0088). Nenhuma ADR na main dizia onde ficam os tipos dos campos. A ADR 0091, que vem no #207, já registra o caso do SearchSelectField.
O #207 (claude/multi-select-component-d9018b) também mexe no fields.types.ts e no index.tsx. Por isso, o PR que entrar por último resolve um conflito pequeno nesses dois arquivos. Os dois exportam as mesmas peças genéricas, BaseFieldProps e FormFieldOwnedProps, e este PR não toca no SearchSelectField. Resolver o conflito é manter o fields.types.ts deste PR e a linha export { SearchSelectField, type SearchSelectFieldProps } ... do #207 no barril.
Este PR substitui a tentativa do #216, que foi fechado por escopo. Ele reaproveita a mesma divisão e vai além em dois pontos: também tira ControlledFieldProps e FieldErrorProps de fields.types.ts, e com isso o arquivo deixa de importar FormFieldControlProps de shared/ui/form-field.
Verificação
Rodado em apps/admin-frontend:
npm run format:check: ok;
npm run lint: 0 erros. Os 6 avisos react-refresh/only-export-components já existiam e estão fora de shared/form/fields; npx eslint src/shared/form/fields sai limpo;
npm run build: ok (tsc + vite build);
npx vitest run --coverage --exclude ".claude/**": 100 arquivos, 1092 testes passando, limiares de cobertura atendidos (linhas 98,08%, ramos 94,65%);
npm run check:docs (raiz): ok.
O fields.types.ts não tem nenhum import de @/shared/ui. Confirmei com grep.
Form field prop types are now organized with their corresponding fields, while shared types remain available centrally. Existing field exports and rendered behavior are unchanged.
Documentation
Updated form development guidance to reflect the organization of field types and shared form types.
…o do campo
fields.types.ts deixa de importar de shared/ui e de conhecer os campos: guarda
só BaseFieldProps, RegisterOwnedProps, FormFieldOwnedProps, WithOptionalLabel e
FieldLabelPlacement. O tipo das props de cada campo e a lista das props
omitidas passam para o arquivo do próprio campo em components/, e o barril
reexporta os mesmos nomes a partir deles. A skill de formulário e o guia de
arquitetura dizem onde nasce o tipo de um campo novo.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No actionable comments were generated in the recent review.
Walkthrough
Field-specific prop types now live in their component files instead of fields.types.ts. The barrel re-exports the component types. Architecture and field-authoring documentation describe the updated type ownership rules.
Changes: Field Prop Type Ownership
Layer / File(s)
Summary
Shared and primitive field contracts apps/admin-frontend/src/shared/form/fields/fields.types.ts, apps/admin-frontend/src/shared/form/fields/components/text-fields.tsx, apps/admin-frontend/src/shared/form/fields/components/masked-field.tsx, apps/admin-frontend/src/shared/form/fields/components/date-field.tsx
fields.types.ts now exports shared field types. Text, masked, and date components declare their prop types using those shared types and the relevant primitive props.
Field-specific prop types move into their component files. SelectFieldOption values use the selected form field’s path value, and the select test imports the type from select-field.
Barrel exports and type-location guidance apps/admin-frontend/src/shared/form/fields/index.tsx, apps/admin-frontend/docs/ARCHITECTURE.md, .claude/skills/agenza-form-field/SKILL.md
The barrel re-exports each component’s prop types, including FieldErrorProps and SelectFieldOption. The architecture documentation and field-authoring guidance describe where field prop types belong.
evertonschuster/Agenza#216: Moves the same field-specific prop types from fields.types.ts into component files.
evertonschuster/Agenza#192: Changes several field prop contracts that this PR relocates into their component files.
evertonschuster/Agenza#207: Adds a field prop type with an explicit later-refactor note to move it into its component file.
Merge Risk: Minimal · up to 7e55d
This refactor moves field prop types next to their components and updates the related documentation, with no behavior change. No merge-blocking risk was found in the changed code. The CI failures are in backend builds caused by a dependency vulnerability advisory and are not connected to these frontend changes.
Pre-merge checks: 5
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title clearly and concisely describes the main change: moving each field's prop type into its field file in admin-frontend.
Docstring Coverage
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 15 files. (2 skipped: 2 unsupported.)
Finishing Touches
Generate docstrings
Commit to this branch
Create a new PR
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Warning
Some tools did not complete. Review the errors below.
ESLint
If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.
Error: typescript-eslint does not support TS 7.0.
at Object. (/.eslint-tmp/node_modules/typescript-eslint/dist/index.js:52:11)
at Module._compile (node:internal/modules/cjs/loader:1929:14)
at Object..js (node:internal/modules/cjs/loader:2060:10)
at Module.load (node:internal/modules/cjs/loader:1651:32)
at Module._load (node:internal/modules/cjs/loader:1443:12)
at wrapModuleLoad (node:internal/modules/cjs/loader:261:19)
at loadCJSModuleWithModuleLoad (node:internal/modules/esm/translators:372:15)
at ModuleWrap. (node:internal/modules/esm/translators:244:9)
at ModuleJob.run (node:internal/modules/esm/module_job:561:25)
at async node:internal/modules/esm/loader:647:26
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O que muda
O
shared/form/fields/fields.types.tspassa a ser genérico. Ele não importa mais nada deshared/uie não conhece nenhum campo. O tipo das props de cada campo e a lista das props omitidas ficam no arquivo do próprio campo, emcomponents/, ao lado do primitivo que o campo renderiza. É o modelo que o #207 adotou para oSearchSelectFieldno commit 2ae5e6c.A mudança é só de estrutura de tipos. Nenhum comportamento muda, e o barril
fields/index.tsxcontinua exportando os mesmos 14 nomes de tipo, agora a partir do arquivo de cada campo. Nenhum consumidor fora defields/foi tocado.Inventário
O que o
fields.types.tsimportava deshared/uinamain:shared/uiInputTextFieldProps,MaskedFieldProps,DateFieldPropsTextField,MaskedField,DateFieldtext-fields.tsx,masked-field.tsx,date-field.tsxTextareaTextareaFieldPropsTextareaFieldtext-fields.tsxInputGroupInputSearchFieldPropsSearchFieldsearch-field.tsxColorSwatchPickerOptionColorFieldPropsColorFieldcolor-field.tsxFormFieldControlPropsControlledFieldPropsControlledFieldcontrolled-field.tsxTipos que pertencem a um campo só, sem import de
shared/ui:CpfFieldProps(deriva deMaskedFieldProps)CpfFieldcpf-field.tsxPhoneFieldProps(deriva deTextFieldProps)PhoneFieldphone-field.tsxEmailFieldProps(deriva deTextFieldProps)EmailFieldemail-field.tsxBirthDateFieldProps(deriva deDateFieldProps)BirthDateFieldbirth-date-field.tsxSelectFieldOption,SelectFieldBaseProps(interno),SelectFieldPropsSelectFieldselect-field.tsxFieldErrorPropsFieldErrorfield-error.tsxUm campo construído sobre outro importa o tipo base do arquivo do campo que ele já renderiza:
PhoneFieldeEmailFieldusamtext-fields,CpfFieldusamasked-fieldeBirthDateFieldusadate-field. O teste doSelectFieldpassou a importarSelectFieldOptionde./select-field.O que ficou em
fields.types.ts, exportado porque vários campos usam:BaseFieldProps:TextField,TextareaField,MaskedFieldeDateField;RegisterOwnedPropseFormFieldOwnedProps: os mesmos quatro, mais oSearchField;WithOptionalLabel:CpfField,PhoneField,EmailFieldeBirthDateField;FieldLabelPlacement:SelectFieldeSearchField, a união tipada uma vez só, como registra a ADR 0076.Nenhum tipo exigiu escolha. Os tipos compartilhados são genéricos e ficam em
fields.types.ts. Os derivados seguem a dependência que o componente já tem, e a API pública do barril não muda.Orientação para os próximos campos
.claude/skills/agenza-form-field/SKILL.md: o tipo das props de um campo novo nasce no arquivo do campo e é reexportado de lá pelo barril. Ofields.types.tssó guarda as peças compartilhadas e não importa deshared/ui.apps/admin-frontend/docs/ARCHITECTURE.md: a mesma regra, na seção de componentes e formulários.Não abri ADR nova. É uma convenção de organização local, reversível e sem efeito em fronteira, contrato, persistência ou operação, então se encaixa no guia de arquitetura e na skill (ADR 0088). Nenhuma ADR na
maindizia onde ficam os tipos dos campos. A ADR 0091, que vem no #207, já registra o caso doSearchSelectField.Relação com o #207
O #207 (
claude/multi-select-component-d9018b) também mexe nofields.types.tse noindex.tsx. Por isso, o PR que entrar por último resolve um conflito pequeno nesses dois arquivos. Os dois exportam as mesmas peças genéricas,BaseFieldPropseFormFieldOwnedProps, e este PR não toca noSearchSelectField. Resolver o conflito é manter ofields.types.tsdeste PR e a linhaexport { SearchSelectField, type SearchSelectFieldProps } ...do #207 no barril.Este PR substitui a tentativa do #216, que foi fechado por escopo. Ele reaproveita a mesma divisão e vai além em dois pontos: também tira
ControlledFieldPropseFieldErrorPropsdefields.types.ts, e com isso o arquivo deixa de importarFormFieldControlPropsdeshared/ui/form-field.Verificação
Rodado em
apps/admin-frontend:npm run format:check: ok;npm run lint: 0 erros. Os 6 avisosreact-refresh/only-export-componentsjá existiam e estão fora deshared/form/fields;npx eslint src/shared/form/fieldssai limpo;npm run build: ok (tsc+vite build);npx vitest run --coverage --exclude ".claude/**": 100 arquivos, 1092 testes passando, limiares de cobertura atendidos (linhas 98,08%, ramos 94,65%);npm run check:docs(raiz): ok.O
fields.types.tsnão tem nenhum import de@/shared/ui. Confirmei comgrep.🤖 Generated with Claude Code
Summary by CodeRabbit
Refactor
Documentation