fw git commit validates only the last -m fragment, refusing the conventional subject/body/trailer form
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Leggi agents/git/lib/commit.sh:39-95 per seguire come vengono analizzati gli argomenti -m ripetuti, quindi esamina la riga 143, dove extract_task_id riceve il messaggio. Confronta la gestione del parser con l’unione da parte di Git di più frammenti tramite righe vuote. Il lavoro è completo quando un riferimento a un’attività in un oggetto o nel corpo viene accettato con argomenti -m ripetuti; nell’issue non è indicato alcun file di test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
fw git commit extracts the task reference from only the last -m fragment, so the
conventional multi-paragraph form is refused:
fw git commit -m "T-NNNN: repair the glossar adapter" \
-m "Longer explanation..." \
-m "Co-Authored-By: Claude ..."
ERROR: No task reference found in commit message
The error quotes the trailer back, which is the only part it looked at.
Code path
agents/git/lib/commit.sh:39-95 — the argument parser has -m) message="$2" with no
accumulation, so each repeated -m overwrites the previous one. Line 143 then runs
extract_task_id "$message" against whatever survived, i.e. the last fragment.
Confirmed still present on 1.6.769.
Why it bites
git itself joins repeated -m fragments with blank lines, so this is the standard way to
write subject/body/trailer. It bites hardest in projects where an attribution trailer is
mandatory on every commit — there, the last fragment is always the trailer, so the
conventional form never works and every commit must be built as a single -m with embedded
newlines via printf.
Suggested fix
Either concatenate the fragments the way git does before extracting, or test the first
fragment, which is the subject and is where a task reference conventionally lives.
Concatenating is strictly better: it also accepts a reference that legitimately appears in
the body.
Filed from project 1409-sprind under its observation-inbox triage task, confirmed by
reading the parser rather than by re-running the original repro.
- Lingua principale
- Shell
- Stelle
- 16
- Fork
- 6
- Merge medio
- 6h 24m
- PR unite (30g)
- 5
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di DimitriGeelen/agentic-engineering-framework
-
area:cli bug severity:low status:fixed-on-bleeding-edge triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
DimitriGeelen/agentic-engineering-framework#100 · 5 commenti ·
-
area:cli bug severity:medium status:fixed-on-bleeding-edge triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
DimitriGeelen/agentic-engineering-framework#97 · 5 commenti ·
-
area:context bug severity:low status:accepted triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
DimitriGeelen/agentic-engineering-framework#96 · 3 commenti ·
-
area:cli bug severity:medium status:accepted triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
DimitriGeelen/agentic-engineering-framework#95 · 3 commenti ·
-
area:watchtower bug severity:low status:released triaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
DimitriGeelen/agentic-engineering-framework#94 · 4 commenti ·
Tutte le issue di DimitriGeelen/agentic-engineering-framework
Issue simili
-
bug raised-by:verifier
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
medici-finance/assay#4008 ·
I maintainer di solito rispondono entro 1 giorno
-
bug: `curl NAME` exports an unquoted, uninterpolated command that is not runnableForse già presa @dannyward630 l’ha presa oggi. Apertabug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
yunaremaia/git-api#61 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
cloudflare/skills#247 ·
I maintainer di solito rispondono entro 1 giorno
-
Update nwg-panel to 0.12.3Forse già presa @zen0bit l’ha presa oggi. Apertapackage-update
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno