Fix TypeError: Object of type __proxy__ is not JSON serializable em task_migrate_and_publish_articles

[Copilot]

O que esse PR faz?

Corrige o TypeError: Object of type __proxy__ is not JSON serializable que interrompia a tarefa task_migrate_and_publish_articles. O objeto __proxy__ (gerado por gettext_lazy) era adicionado à lista de eventos da tarefa e não era tratado por sanitize_for_json, chegando inalterado ao psycopg2 durante o save().

Mudanças:

• core/utils/sanitize.py — sanitize_for_json agora usa try/except json.dumps para testar serializabilidade diretamente: se o objeto já for serializável, é retornado inalterado; caso contrário, é convertido para str() e re-passado recursivamente pela própria função, garantindo também o tratamento de surrogates que __str__() possa produzir. Um import json foi adicionado ao módulo.

# Antes: objetos __proxy__ passavam sem tratamento
return obj  # TypeError em runtime no psycopg2

# Depois: testa serializabilidade via json.dumps; caso falhe, converte e re-sanitiza
try:
    json.dumps(obj)
    return obj
except (TypeError, ValueError):
    return sanitize_for_json(str(obj))

• proc/tests.py — 4 novos testes cobrindo lazy proxy isolado, em lista, em dict e objetos customizados não-serializáveis. A asserção de igualdade usa str(lazy_text) em vez de literal inglês fixo, evitando brittleness sob qualquer locale ativo. O bloco __main__ foi removido pois os testes dependem do Django e devem ser executados via pytest.

Onde a revisão poderia começar?

core/utils/sanitize.py — a mudança está na seção final da função sanitize_for_json (bloco try/except), seguida dos novos testes em proc/tests.py.

Como este poderia ser testado manualmente?

Executar a tarefa task_migrate_and_publish_articles com os parâmetros journal_acron=rpe, collection_acron=rve, issue_folder=v25n1 e verificar que nenhum UnexpectedEvent do tipo TypeError é registrado ao final da execução.

Algum cenário de contexto que queira dar?

O fluxo de falha: add_event(_("...")) → json.dumps falha → sanitize_for_json chamada mas não trata __proxy__ → self.save() falha no psycopg2. A correção cobre não apenas __proxy__, mas qualquer tipo não-serializável que possa chegar ao detail no futuro, inclusive strings com surrogates geradas por __str__().

Screenshots

N/A

Quais são tickets relevantes?

[Migração de Artigos] Erro TypeError: Object of type proxy is not JSON serializable

Referências

• Django lazy translation objects

[Copilot]

Pull request overview

Corrige a falha na persistência de detail em eventos/tarefas quando o payload contém objetos não serializáveis em JSON (ex.: gettext_lazy / __proxy__), evitando TypeError: Object of type __proxy__ is not JSON serializable durante a execução da task_migrate_and_publish_articles.

Changes:

• Ajusta sanitize_for_json para manter escalares JSON nativos e converter demais tipos não-serializáveis para str().
• Atualiza a docstring do sanitizador para refletir a nova responsabilidade (JSON-serializável, não só surrogates).
• Adiciona testes cobrindo lazy proxy em valor isolado, em lista, em dict e em objeto customizado.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
core/utils/sanitize.py	Expande o sanitizador para tornar payloads JSON-serializáveis (inclui lazy proxies), reduzindo falhas ao salvar detalhes de eventos no banco.
proc/tests.py	Inclui cobertura de testes para garantir serialização de lazy proxies e objetos não-serializáveis após sanitização.

[Copilot]

These new tests import Django (gettext_lazy) but the file still exposes a __main__ runner (python proc/tests.py). Running it directly will now fail unless Django settings are configured. Either remove the __main__ block or configure DJANGO_SETTINGS_MODULE/django.setup() there to keep the standalone runner functional.

[Copilot]

sanitize_for_json now converts unknown objects with str(obj), but the resulting string is returned without re-running the surrogate sanitization. If __str__() (or a lazy translation) yields a string containing surrogate code points, this reintroduces the original PostgreSQL/JSON rejection. Consider sanitizing the string result as well (e.g., pass str(obj) back through sanitize_for_json).

Suggested change

	# objects from gettext_lazy) to its string representation.
	return str(obj)
	# objects from gettext_lazy) to its string representation, then sanitize
	# that string as well in case __str__() produces surrogate code points.
	return sanitize_for_json(str(obj))

[robertatakenaka]

@copilot teste se obj é serializavel usando try/except e json.dumps, aí caso contário, sanitiza

[Copilot]

Implementado em fc84f73. Agora sanitize_for_json usa try/except json.dumps(obj) para testar serializabilidade: se o objeto já for serializável, é retornado inalterado; caso contrário, é convertido para str() e re-passado recursivamente pela própria função (para também tratar surrogates que __str__() possa produzir).

[Copilot]

This test asserts a hard-coded English value for a gettext_lazy string. Since the project default LANGUAGE_CODE is pt-BR and this msgid has translations in locale/*, str(lazy_text) can legitimately be non-English when translations are compiled/active, making the test brittle. Prefer asserting equality to str(lazy_text) (or using override('en')/translation.activate('en')) instead of a fixed literal.

Suggested change

	self.assertEqual(result, "Select journals by collection")
	self.assertEqual(result, str(lazy_text))