/talk?about= names its subject: one line above the composer, only for what the viewer can read #10614

/talk?about=owner/repo and /talk?about=owner/repo#N now show one line above the composer. It reads "About owner/repo" or "About owner/repo#N: <title>", with the title escaped. I ran gate, tsc, test/talk-page.test.ts (38 pass) and design-lint; all pass. I did not run the full suite; the forge runs the check on this push.

• Parsing: parseAbout in src/web/pages/talk.ts is a small pure function. It takes a plain string and returns {owner, repo, number?} or null.
• What shows the line: only a repository the viewer can read, plus an issue or pull number that exists in it. The page then looks up the repo and the issue.
• What does not: a malformed value (../x, a/b#x, 300 characters), an unknown repo or number, or a private repo the viewer cannot read all get the same page as plain /talk. A signed-out viewer gets no line either.
• Tests: test/talk-page.test.ts covers a readable issue showing its title, the private-repo case for another person and for a signed-out viewer, and the malformed values. The private-repo response equals the one for a repo that does not exist.

The earlier pull went red on two unrelated tests: test/child-quota.test.ts and scripts/unreached-allow.json. This branch carried its own fixes for both, which conflicted with main. I merged main and dropped my versions of both files, so the diff against main is only talk.ts and talk-page.test.ts.

I changed no links. Only the Talk page learns about, as the issue asks.

To try it, open /talk?about=mason/forge signed in and read the line above the composer.

🤖 Generated with Claude Code

Closes #10494

/talk?about=owner/repo and /talk?about=owner/repo#N now show one line above the composer. It reads "About owner/repo" or "About owner/repo#N: <title>", with the title escaped. I ran gate, tsc, test/talk-page.test.ts (38 pass) and design-lint; all pass. I did not run the full suite; the forge runs the check on this push.

• Parsing: parseAbout in src/web/pages/talk.ts is a small pure function. It takes a plain string and returns {owner, repo, number?} or null.
• What shows the line: only a repository the viewer can read, plus an issue or pull number that exists in it. The page then looks up the repo and the issue.
• What does not: a malformed value (../x, a/b#x, 300 characters), an unknown repo or number, or a private repo the viewer cannot read all get the same page as plain /talk. A signed-out viewer gets no line either.
• Tests: test/talk-page.test.ts covers a readable issue showing its title, the private-repo case for another person and for a signed-out viewer, and the malformed values. The private-repo response equals the one for a repo that does not exist.

The earlier pull went red on two unrelated tests: test/child-quota.test.ts and scripts/unreached-allow.json. This branch carried its own fixes for both, which conflicted with main. I merged main and dropped my versions of both files, so the diff against main is only talk.ts and talk-page.test.ts.

I changed no links. Only the Talk page learns about, as the issue asks.

To try it, open /talk?about=mason/forge signed in and read the line above the composer.

🤖 Generated with Claude Code

All four points from the review are addressed and pushed. bun x tsc --noEmit, bun scripts/gate.ts, bun scripts/design-lint.ts, test/talk-page.test.ts and test/conversation-talk.test.ts pass. I did not run the full suite; the forge runs the full check on this push. scripts/fast.ts found the diff too broad to name a smaller set.

1. Both doors.

- parseAbout and the readable check now live in one core module, src/core/talk_about.ts. Its aboutLine(db, text, viewer) returns the plain line, or null when the value is malformed, unknown or unreadable. The web page's line is drawn from it. - The terminal gets ssh <forge> talk --about owner/repo[#N] [words] in src/tui/ssh-talk.ts. It prints the same gated line first, and with no words it prints only that line. - Nothing is printed for a guest or for another person on a private repo. Control characters are stripped from the printed line.

1. Docs. docs/features/talk.md has a new section on ?about= and --about.
2. Design. talk-about stays as a marker class with no rule of its own. The line is plain small muted text, like .talk-meta. I recorded it in docs/design/areas/talk.md and added no CSS.
3. Tests.

- A pull of a private repo shows the signed-in non-reader nothing and gives a page identical to the one for a nonexistent repo. The owner does see its title. I made the "pull" by setting kind = 'pull' on an issue row rather than opening a real pull, which needs branches. The lookup does not branch on kind, so this exercises the same path. - aboutLine is tested for owner, other person, signed out and malformed values. - The talk --about terminal path is tested for owner, other person, signed out and malformed values. - The signed-out web test still checks only the 303 to /signin, which you accepted.

I did not make --about pass the subject to the agent. It only names the subject, as the page does, and passing it on would be a feature the issue doesn't ask for.

🤖 Generated with Claude Code