Skip to content

Adicionar argumentos advice_text e advice_params em build_response#1080

Draft
Copilot wants to merge 10 commits intomasterfrom
copilot/add-advice-text-and-params
Draft

Adicionar argumentos advice_text e advice_params em build_response#1080
Copilot wants to merge 10 commits intomasterfrom
copilot/add-advice-text-and-params

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

O que esse PR faz?

Substitui format_response por build_response em 8 módulos de validação, adicionando suporte à internacionalização via advice_text (template) e advice_params (valores parametrizados).

Módulos modificados:

  • article_abstract.py, article_and_subarticles.py, article_license.py, article_xref.py
  • fig.py, journal_meta.py, metadata_langs.py, tablewrap.py

Mudanças estruturais:

  • Parâmetro parent convertido de campos individuais para dicionário
  • Todas as mensagens de advice agora parametrizadas para suporte i18n via gettext

Exemplo de mudança:

# Antes
yield format_response(
    title="Journal ISSN",
    parent='article',
    parent_id=None,
    advice=f'Mark {name} ISSN with <issn pub-type="{tp}">{issn}</issn>',
    # ...
)

# Depois
yield build_response(
    title="Journal ISSN",
    parent={"parent": "article", "parent_id": None, ...},
    advice=f'Mark {name} ISSN with <issn pub-type="{tp}">{issn}</issn>',
    advice_text='Mark {name} ISSN with <issn pub-type="{tp}">{issn}</issn>',
    advice_params={"name": name, "tp": tp, "issn": issn},
    # ...
)

Onde a revisão poderia começar?

packtools/sps/validation/accessibility_data.py - padrão de referência já implementado

Depois revisar qualquer um dos 8 módulos modificados para verificar consistência da parametrização.

Como este poderia ser testado manualmente?

  1. Executar validações existentes e verificar que adv_text e adv_params estão presentes nas respostas
  2. Verificar que os valores em adv_params correspondem aos placeholders em adv_text
  3. Confirmar que nenhuma chamada a format_response permanece nos módulos de validação

Algum cenário de contexto que queira dar?

Pattern seguido: accessibility_data.py já implementava build_response com advice_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

  • Implementação de referência: packtools/sps/validation/accessibility_data.py
  • Framework gettext para internacionalização Python
Original prompt

This section details on the original issue you should resolve

<issue_title>Adicionar os argumentos advice_text e advice_params em build_response</issue_title>
<issue_description>### Descrição da tarefa
Adicionar os argumentos advice_text e advice_params em build_response

Subtarefas

  • Acesse os módulos presentes em packtools/sps/validation
  • Onde houver format_response troque por build_response e ajuste seus argumentos, faça um commit para cada arquivo modificado
  • Adicione os argumentos advice_text e advice_params nas chamadas a build_response, considere o exemplo encontrado em packtools/sps/validation/accessibility_data.py. Observe que são argumentos para termos as mensagens traduzidas
    </issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 9 commits February 12, 2026 11:45
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
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.

Adicionar os argumentos advice_text e advice_params em build_response

2 participants