Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 10, 2026

O que esse PR faz?

Substitui todas as chamadas format_response por build_response nos módulos de validação, adicionando suporte à internacionalização através dos argumentos advice_text e advice_params.

Arquivos modificados (12):

  • alternatives.py, app_group.py, article_abstract.py, article_and_subarticles.py
  • article_doi.py, article_license.py, article_xref.py, fig.py
  • formula.py, journal_meta.py, metadata_langs.py, tablewrap.py

Mudanças técnicas:

# Antes
yield format_response(
    title="validation",
    parent="article",
    parent_id=None,
    parent_article_type=article_type,
    parent_lang=lang,
    advice=f"Fix {element} for {lang}",
    ...
)

# Depois
advice = f"Fix {element} for {lang}"
advice_text = _("Fix {element} for {lang}")
advice_params = {"element": element, "lang": lang}

parent_data = {
    "parent": "article",
    "parent_id": None,
    "parent_article_type": article_type,
    "parent_lang": lang,
}

yield build_response(
    title="validation",
    parent=parent_data,
    advice=advice,
    advice_text=advice_text,
    advice_params=advice_params,
    ...
)

Onde a revisão poderia começar?

packtools/sps/validation/accessibility_data.py - referência do padrão implementado
packtools/sps/validation/alternatives.py - exemplo mais simples de migração

Como este poderia ser testado manualmente?

  1. Importar módulos de validação: from packtools.sps.validation import alternatives, fig, formula
  2. Executar validações existentes e verificar estrutura dos responses
  3. Confirmar presença de advice_text e advice_params nos resultados de validação

Algum cenário de contexto que queira dar?

37 chamadas migradas mantendo compatibilidade retroativa com advice original. Parâmetros parent consolidados em dicionário único. Gettext configurado para futuras traduções.

Screenshots

N/A - mudanças internas na estrutura de dados

Quais são tickets relevantes?

Issue: Adicionar os argumentos advice_text e advice_params em build_response

Referências

Padrão estabelecido em packtools/sps/validation/accessibility_data.py

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)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 13 commits February 10, 2026 12:37
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>
…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>
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 advice_text e advice_params em build_response Feb 10, 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