Adicionar argumentos advice_text e advice_params em build_response#1080
Draft
Adicionar argumentos advice_text e advice_params em build_response#1080
Conversation
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
…s.py Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add advice_text and advice_params to build_response
Adicionar argumentos advice_text e advice_params em build_response
Feb 12, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 esse PR faz?
Substitui
format_responseporbuild_responseem 8 módulos de validação, adicionando suporte à internacionalização viaadvice_text(template) eadvice_params(valores parametrizados).Módulos modificados:
article_abstract.py,article_and_subarticles.py,article_license.py,article_xref.pyfig.py,journal_meta.py,metadata_langs.py,tablewrap.pyMudanças estruturais:
parentconvertido de campos individuais para dicionárioExemplo de mudança:
Onde a revisão poderia começar?
packtools/sps/validation/accessibility_data.py- padrão de referência já implementadoDepois revisar qualquer um dos 8 módulos modificados para verificar consistência da parametrização.
Como este poderia ser testado manualmente?
adv_texteadv_paramsestão presentes nas respostasadv_paramscorrespondem aos placeholders emadv_textformat_responsepermanece nos módulos de validaçãoAlgum cenário de contexto que queira dar?
Pattern seguido:
accessibility_data.pyjá implementavabuild_responsecomadvice_text/advice_params. Este PR uniformiza todos os módulos de validação para o mesmo padrão, habilitando tradução de mensagens via framework gettext.Screenshots
N/A - mudanças em lógica de backend
Quais são tickets relevantes?
Issue original descreve a tarefa de adicionar suporte i18n aos responses de validação.
Referências
packtools/sps/validation/accessibility_data.pyOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.