fix(cli): keep the commit-session trailer dedup working on git 2.39
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 84/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- git, typescript
- Domínio
- cli, testing-qa
Direção de pesquisa
Execute cd cli [?25l&& npx vitest run tests/contexts/telemetry/domain/formats/commit-session-trailer.integration.test.ts e inspecione o fixture e o delegate que escreve o trailer por volta da linha 72. Garanta que o fixture ou o delegate lide com a ausência de uma nova linha final, para que duas passagens deixem um trailer; em seguida, confirme que o teste de integração passa no git 2.39.2.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Observed
cli/tests/contexts/telemetry/domain/formats/commit-session-trailer.integration.test.ts:72 ("never doubles a trailer a prior run already wrote") runs the trailer-writing delegate twice and asserts the trailer token appears once.
On git 2.39.2 the test fails with AssertionError: expected 2 to be 1: the fixture message has no trailing newline, so git appends the trailer without a blank-line separator; the second pass then sees no trailer block and --if-exists doNothing appends again. The same delegate on a message ending in a newline dedupes correctly.
Reproduction
git --version-> 2.39.2 (or any version sharing this behavior).cd cli && npx vitest run tests/contexts/telemetry/domain/formats/commit-session-trailer.integration.test.ts-> fails alone, deterministic, also underenv -i.- The same test passes on newer git (CI's).
Impact
The suite reports one failure per run on machines with this git, masking real regressions in the file and eroding trust in the suite's signal. The test and its source are byte-identical to next, so the failure predates any recent change.
Suggested fix
End the fixture message with a newline, or have the delegate normalize the message before appending.
Standalone repro verified on the machine that reported it: message without trailing newline doubles the trailer; message with one dedupes.
- Linguagem predominante
- TypeScript
- Estrelas
- 481
- Forks
- 45
- Merge médio
- 1d 3h
- PRs com merge (30d)
- 100
Preparar o ambiente
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de ai-driven-dev/framework
-
fix(cli): doctor warns that claude no longer declares aidd-framework right after a clean setupAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
ai-driven-dev/framework#1118 ·
Mantenedores costumam responder em até 1 dia
-
docs(framework): explain how a contributor without push access opens a pull requestTalvez já em andamento @sadhana-s-codez assumiu há 1 dia. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
ai-driven-dev/framework#1117 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
ai-driven-dev/framework#1112 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
ai-driven-dev/framework#1002 ·
Mantenedores costumam responder em até 1 dia
-
feat(aidd-dev/test): enforce test effectiveness and test-double disciplineTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
ai-driven-dev/framework#952 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
Todas as issues de ai-driven-dev/framework
Issues semelhantes
-
Shell: serve the home app's Cookie Policy on a hard reloadTalvez já em andamento @2001anshkaushik assumiu hoje. Abertafeature module:ai
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
rocketride-org/rocketride-server#2899 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
wazuh/wazuh-dashboard-plugins#9551 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
gofish-graphics/gofish-graphics#1252 ·
Mantenedores costumam responder em até 1 dia
-
optimization P3 size:XS
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 88/100
josephfung/curia#2366 ·
Mantenedores costumam responder em até 1 dia
-
stranded-follow-up sweep trusts any issue whose body carries the marker as the tracking issueAbertatype:bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
nightgauge/nightgauge#2688 ·
Mantenedores costumam responder em até 1 dia