feat: consolida RealMapperParser como parser MapperVO canonico (issue #139) - #201
Merged
elson-vinicius-lopes merged 4 commits intoAug 27, 2026
Merged
Conversation
…vs leitura ad-hoc (#139) Roda RealMapperParser em paralelo, sem side effect, dentro de ExtractLayoutGuidsFromDecryptedContent, comparando InputLayoutGuid/TargetLayoutGuid extraídos pelo caminho legado vs. pelo parser B (candidato canônico) e logando divergência (só GUIDs/booleano, sem conteúdo do documento). Falha do RealMapperParser é capturada e nunca propaga - passo 1 do plano em docs/architecture/inventario-parsers-mapperVo-issue-139.md. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…e depreca parser legado (#139) Passo 2: XslGeneratorService.GenerateXslContent agora parseia o MapperVO com XslSynth.Core.RealMapperParser / XslSynth.Model.MapperVo (parser B) em vez do legado LayoutParserApi.Models.Entities.MapperVo.FromXml. Todos os campos usados (Rules.Name/Sequence/ContentValue, LinkMappings.Name/Sequence/DefaultValue/ AllowEmpty/InputGuid/TargetGuid) têm equivalente direto em B. Passo 3: marca MapperVo/MapperRule/LinkMappingItem legados com [Obsolete] (sem remover arquivos). Único uso residual fora de escopo é MapperDatabaseService.ExtractLayoutGuidsFromDecryptedContent (fase de sombra do passo 1, intocada de propósito) — gera 1 warning CS0618 esperado e documentado. Testes sintéticos novos cobrem o caminho tipado (Rule+LinkMappingItem dentro de <MapperVO>) e o fallback via XML bruto (sem estrutura reconhecida pelo parser).
…de arquitetura no README Passos 1-3 do plano de migração de MapperVO (fase de sombra, migração do consumidor de XslGeneratorService, deprecação do parser legado) confirmados como concluídos no inventário de arquitetura; passo 4 permanece fora de escopo (#140/#141). Documenta também a limitação já conhecida de que nenhum parser (legado ou RealMapperParser) captura elementos aninhados. README ganha nota curta apontando RealMapperParser como fonte canônica.
…aseService (issue #139) QA encontrou que ExtractLayoutGuidsFromDecryptedContent ainda populava mapper.XslContent via MapperVo.FromXml (legado) em produção, fora da fase de sombra do passo 1. Migrado para RealMapperParser/XslSynth.Model.MapperVo — extração de XslContent é idêntica entre os dois parsers (mesmo Element("XslContent") com fallback "Xsl"), sem gap de comportamento. GUIDs continuam vindo da leitura ad-hoc, inalterada. Sombra do passo 1 mantida (compara caminho ad-hoc vs RealMapperParser, nunca comparou contra MapperVo.FromXml). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
elson-vinicius-lopes
added a commit
that referenced
this pull request
Aug 28, 2026
…andidates-141 Reconcilia PR #207 (issue #141) com develop, que já absorveu as PRs irmãs #200 (issue #86), #201 (issue #139), #203 (issue #138) e #205 (issue #140) da mesma cadeia de trabalho. Conflitos eram todos overlap real entre PRs desta cadeia tocando os mesmos arquivos, não clash semântico: - LowCodeCandidateResult.cs: DecryptedMapperContent (#141) e MapperDecryptedContent (#138) eram o mesmo dado (mapper.DecryptedContent) sob nomes diferentes — unificado em DecryptedMapperContent, único campo, usado tanto por SysmiddleSectionMappingResolver (#138) quanto por TryComposeFieldMappings (#141). - LowCodeAutoTransformationService.cs: mesma duplicação de atribuição nos dois pontos de criação de LowCodeCandidateResult. - TransformationExecutionController.cs: TransformationCandidate agora preenche FieldMappings (#141) E SectionMappings/XmlNamespaces (#138) no mesmo objeto — funcionalidades complementares, ambas preservadas. - README.md: seções de documentação de fieldMappings (#141) e sectionMappings (#138) são independentes, mantidas as duas em sequência. - security-code-scan-baseline.json: entradas de linha para LowCodeAutoTransformationService.cs reconciliadas para 371/415 (linhas atuais pós-merge) — mesmos 2 achados de sempre (File.WriteAllTextAsync em inPath/metaPath), não vulnerabilidades novas. Nota adicionada ao _readme documentando o ajuste. dotnet build: 0 erros. dotnet test: 413/417 passando — as 4 falhas (SafePathResolverTests, LowCodeRunnerArgsTests) são pré-existentes, específicas de ambiente (assumem paths Windows, falham sob WSL/Linux), não relacionadas aos arquivos deste merge. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This was referenced Aug 28, 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
1 participant
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.
Resumo
Consolida
RealMapperParser(Pathway B) como o parser canônico deMapperVO, migrando o consumidor legado (XslGeneratorService) e depreciando (sem remover) o parser legado (Pathway A) identificado no diagnóstico.RealMapperParser) identificados e comparados — verdocs/architecture/inventario-parsers-mapperVo-issue-139.md.MapperDatabaseService(compara os dois parsers em produção sem afetar comportamento).XslGeneratorServicemigrado para consumirRealMapperParser.[Obsolete]— não removido, para permitir rollback e observação adicional.a9633ab), eliminando a última chamada fora do caminho canônico.Testes
dotnet buildsem erros.dotnet test: 392 passando / 4 falhas pré-existentes de path Windows×Linux (LowCodeRunnerArgsTests, não relacionadas a esta mudança).Referências
docs/architecture/inventario-parsers-mapperVo-issue-139.mdCloses #139