Add <response> element validation for SPS 1.10 compliance by Copilot · Pull Request #1138 · scieloorg/packtools

Copilot · 2026-03-19T11:50:19Z

O que esse PR faz?

Implementa 7 regras de validação para o elemento <response> conforme SPS 1.10, cobrindo atributos obrigatórios, unicidade de IDs e estrutura mínima esperada.

#	Regra	Nível
1	`@response-type` presença	CRITICAL
2	`@response-type` valor = `"reply"`	ERROR
3	`@xml:lang` presença	CRITICAL
4	`@id` presença	CRITICAL
5	`@id` unicidade	ERROR
6	`<front-stub>` presença	WARNING
7	`<body>` presença	WARNING

Onde a revisão poderia começar?

packtools/sps/validation/response.py — contém a classe ResponseValidation com todos os 7 métodos de validação.

Como este poderia ser testado manualmente?

from lxml import etree
from packtools.sps.validation.response import ResponseValidation

xml = """<article article-type="letter" xml:lang="en">
    <front><article-meta/></front>
    <body><p>Content</p></body>
    <response response-type="reply" xml:lang="en" id="S1">
        <front-stub/>
        <body><p>Response</p></body>
    </response>
</article>"""

tree = etree.fromstring(xml)
for result in ResponseValidation(tree).validate():
    print(result["title"], result["response"])

Testar também via orchestrator:

from packtools.sps.validation.xml_validator import get_validation_results
for r in get_validation_results(tree, {}):
    if r.get("group") == "response":
        print(r["title"], r["response"])

31 testes unitários em tests/sps/validation/test_response.py cobrem: atributos ausentes/vazios/whitespace, valores incorretos, IDs duplicados, respostas em <sub-article>, error levels customizáveis.

Algum cenário de contexto que queira dar?

O <response> identifica respostas a cartas/comentários publicadas no mesmo documento. Segue o mesmo padrão arquitetural dos validadores existentes (HistoryValidation, XMLPeerReviewValidation): classe de validação + rules JSON + integração no orchestrator via xml_validations.py / xml_validator.py.

Arquivos criados:

packtools/sps/validation/response.py — ResponseValidation
packtools/sps/validation_rules/response_rules.json — configuração de níveis de erro
tests/sps/validation/test_response.py — 31 testes

Arquivos modificados:

packtools/sps/validation/xml_validations.py — import + validate_response()
packtools/sps/validation/xml_validator.py — yield do grupo "response"

Screenshots

N/A — validação backend sem componente visual.

Quais são tickets relevantes?

Referências

Especificação SPS 1.10 — <response>

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

api.crossref.org
- Triggering command: `/usr/bin/python python -c
  from lxml import etree
  from packtools.sps.validation.xml_validator import get_validation_results
  from packtools.sps.validation.xml_validator_rules import get_default_rules

Test that response_rules loads from the JSON

rules = get_default_rules()
assert` (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled

Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

This section details on the original issue you should resolve

<issue_title>Criar validações para o elemento </issue_title>
<issue_description>## Objetivo

Implementar validações para o elemento <response> conforme a especificação SPS 1.10, aumentando a conformidade de X% para 70% (7 de 10 regras).

Nota: Algumas validações para <response> podem já estar parcialmente implementadas no repositório. Este Issue visa reavaliar, complementar e garantir cobertura completa das regras SPS 1.10.

Contexto

O elemento <response> identifica um conjunto de respostas referente a uma carta ou comentário, obrigatoriamente publicadas juntamente com a carta/comentário. É usado quando há múltiplas respostas no mesmo documento. Validações corretas garantem presença de atributos obrigatórios, unicidade de IDs, e estrutura mínima necessária.

Conformidade atual: X de 10 regras implementadas (X%)
Meta após implementação: 7 de 10 regras (70%)

Documentação SPS

Referência oficial: https://docs.google.com/document/d/1GTv4Inc2LS_AXY-ToHT3HmO66UT0VAHWJNOIqzBNSgA/edit?tab=t.0#heading=h.response

Regras principais conforme SPS 1.10:

Ocorrência:
- <response> pode aparecer zero ou mais vezes em <article> e <sub-article>
Contexto de uso:
- Tag para identificar conjunto de respostas referente a carta ou comentário
- Obrigatoriamente publicadas juntamente com carta ou comentário
- Se apenas uma resposta: usar <sub-article article-type="reply">
- Se publicado com DOI separadamente: usar <article article-type="reply">
Atributos obrigatórios:
- @response-type="reply" (obrigatório)
- @xml:lang (obrigatório)
- @id (obrigatório)
Unicidade de IDs:
- Deve-se usar um @id diferente para cada <response>
Estrutura esperada:
- <front-stub> - Metadados da resposta
- <body> - Conteúdo da resposta
- <back> - Seção final (opcional, pode conter <ref-list>)

Regras a Implementar

P0 – Críticas (implementar obrigatoriamente)

#	Regra	Nível	Descrição
1	Validar presença de `@response-type`	CRITICAL	O atributo `@response-type` é obrigatório em `<response>`
2	Validar valor de `@response-type`	ERROR	O valor de `@response-type` deve ser `"reply"`
3	Validar presença de `@xml:lang`	CRITICAL	O atributo `@xml:lang` é obrigatório em `<response>`
4	Validar presença de `@id`	CRITICAL	O atributo `@id` é obrigatório em `<response>`
5	Validar unicidade de `@id`	ERROR	Cada `<response>` deve ter um `@id` único (não pode repetir valores)

P1 – Importantes (implementar se possível)

#	Regra	Nível	Descrição
6	Validar presença de `<front-stub>`	WARNING	Recomenda-se que `<response>` contenha `<front-stub>` com metadados da resposta
7	Validar presença de `<body>`	WARNING	Recomenda-se que `<response>` contenha `<body>` com o conteúdo da resposta

P2 – Futuras (fora do escopo deste Issue)

#	Regra	Motivo de exclusão
8	Validar contexto de uso (carta/comentário)	Média complexidade - requer validação do article-type do documento pai
9	Validar uso de sub-article para resposta única	Alta complexidade - requer análise de quantidade e contexto
10	Validar formato de @id conforme sugestões SPS	Baixa prioridade - formato livre permitido

Arquivos a Criar/Modificar

Avaliar existentes (podem ter validações parciais):

packtools/sps/models/response.py ou similar – Verificar se modelo existe
packtools/sps/validation/response.py – Verificar validações existentes
packtools/sps/validation/rules/response_rules.json ou similar – Verificar configuração

Criar (se não existirem):

packtools/sps/models/response.py – Modelo de extração de dados
packtools/sps/validation/response.py – Validações
packtools/sps/validation/rules/response_rules.json – Configuração de níveis de erro
tests/sps/validation/test_response.py – Testes unitários

Referenciar (implementações similares):

packtools/sps/validation/sub_article.py – Validação de estrutura similar
packtools/sps/validation/utils.py – Funções auxiliares (build_response)

Exemplos de XML

XML Válido (deve passar sem erros):

<!-- Exemplo 1: Duas respostas em um documento -->
<article article-type="letter">
    <front>
        <!-- metadados da carta -->
    </front>
    <body>
        <!-- conteúdo da carta -->
    </body>
    <response response-type="reply" xml:lang="en" id="S1">
        <front-stub>
            <contrib-group>
                <contrib contrib-type="author">
                    <name>
                        <surname>Smith</surname>
                        <given-names>John</given-names>
                    </name>
                </contrib>
            </contrib-grou...

</details>



<!-- START COPILOT CODING AGENT SUFFIX -->

- Fixes scieloorg/packtools#1108

<!-- START COPILOT CODING AGENT TIPS -->
---

📱 Kick off Copilot coding agent tasks wherever you are with [GitHub Mobile](https://gh.io/cca-mobile-docs), available on iOS and Android.

@xml

Implements 7 validation rules for <response> element: - Rule 1: @response-type presence (CRITICAL) - Rule 2: @response-type value must be "reply" (ERROR) - Rule 3: @xml:lang presence (CRITICAL) - Rule 4: @id presence (CRITICAL) - Rule 5: @id uniqueness (ERROR) - Rule 6: <front-stub> presence (WARNING) - Rule 7: <body> presence (WARNING) New files: - packtools/sps/validation/response.py - Validation class - packtools/sps/validation_rules/response_rules.json - Rules config - tests/sps/validation/test_response.py - 31 unit tests Modified files: - packtools/sps/validation/xml_validations.py - Import + wrapper function - packtools/sps/validation/xml_validator.py - Orchestrator integration Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>

Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>

Initial plan

3156335

Copilot AI assigned Copilot and robertatakenaka Mar 19, 2026

Copilot started work on behalf of robertatakenaka March 19, 2026 11:59 View session

Copilot AI and others added 2 commits March 19, 2026 12:07

Address code review feedback: clean URL and grammar fix

760a362

Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Implement validations for the response element~~ Add <response> element validation for SPS 1.10 compliance Mar 19, 2026

Copilot AI requested a review from robertatakenaka March 19, 2026 12:10

Copilot finished work on behalf of robertatakenaka March 19, 2026 12:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add <response> element validation for SPS 1.10 compliance#1138

Add <response> element validation for SPS 1.10 compliance#1138
Copilot wants to merge 3 commits intomasterfrom
copilot/create-validations-for-response-element

Copilot AI commented Mar 19, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

O que esse PR faz?

Onde a revisão poderia começar?

Como este poderia ser testado manualmente?

Algum cenário de contexto que queira dar?

Screenshots

Quais são tickets relevantes?

Referências

I tried to connect to the following addresses, but was blocked by firewall rules:

Test that response_rules loads from the JSON

Contexto

Documentação SPS

Regras principais conforme SPS 1.10:

Regras a Implementar

P0 – Críticas (implementar obrigatoriamente)

P1 – Importantes (implementar se possível)

P2 – Futuras (fora do escopo deste Issue)

Arquivos a Criar/Modificar

Avaliar existentes (podem ter validações parciais):

Criar (se não existirem):

Referenciar (implementações similares):

Exemplos de XML

XML Válido (deve passar sem erros):

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Mar 19, 2026 •

edited

Loading