Repository navigation
chat: a landed card quotes the report's first prose line, not its literal first line - #930
Merged
Merged
Conversation
…eral first line A report that opens with a code fence is the model spelling its answer correctly; the card reading it literally quoted the fence marker itself and spent the one row that carries the work's own sentence on punctuation. firstProseLine skips leading blanks and fence markers and quotes the first line that says something; a fence-only report quotes its first line inside, and a report with no prose keeps the emptiness law it already had. Co-Authored-By: aforge <agentfield-bot@users.noreply.github.com>
…t on The first-prose-line reader arrived as two functions and a fallthrough — a walk that stepped into a fenced block, a second walk inside it, and a comment about a fence still being written. Every answer it gave is the answer one sentence gives: a blank line says nothing, a fence marker says nothing, and the first line that is neither is what the work came to. So it is one loop now, and [mdFenceOpen] is still the only opinion the surface has about what a fence is. The manual page moved with it. The rule was a paragraph inside the section about when a task started, which is not a heading anybody asking "why does my card show three backticks" would ever reach — headings are the search index, and a page that is correct and unreachable is a page the chat talks over. It has its own heading now, in the asker's own words, with the card it draws and the row it used to draw. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
santoshkumarradha
force-pushed
the
af/issue-889
branch
from
September 11, 2026 20:09
51bc96e to
fbce6ca
Compare
The lane numbered its entry 889, which is the issue. The number in the filename and in `pr:` is what the released changelog links, so an entry filed under an issue number points a reader at the report rather than at the change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
santoshkumarradha
added
the
area:chat
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.
Closes #889.
A report that opens with a code fence is the model spelling its answer correctly. The card read that literally and quoted the fence marker itself, so the one row that carries the work's own sentence drew
"```".firstProseLineskips leading blank lines and fence markers, with or without a language word after them, and quotes the first line beneath that says something. Two cases it handles on purpose:A report with no prose at all returns nothing and the row falls back to the subtitle, then to the start stamp, which
doneUnderalready knew how to do.Where this came from
This change was written by an aforge lane, not by hand.
~/af-lane.sh 889ranaforge doonz-ai/glm-5.3through OpenRouter, unattended on the Spark, and produced the fix, the test, the change entry and the manual pages in about ten minutes for $3.13. I read the diff, rebased it onto dev, and ran the suites. The runbook for that isspark:~/AFORGE-LANES-HOWTO.md.Proof
Full
internal/tui3on the Spark, plus the untagged e2e words gate and the manual gates:Re-run after the rebase onto
eaa3d69de: build, vet, e2e and manual green;touched packagescovers tui3 here.🤖 Generated with Claude Code