Skip to content

fix(chat): trim padding around internal thread links - #81

Merged
ScriptedAlchemy merged 1 commit into
mainfrom
t3/normalize-thread-markdown
Nov 8, 2026
Merged

ScriptedAlchemy merged 1 commit into
mainfrom
t3/normalize-thread-markdown

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Model-written thread links with padding, such as < t3-thread://v2/environment/thread >, appear as raw internal URLs instead of thread links. Trim that padding before rendering assistant messages on mobile and web/desktop, while keeping strict link parsing and stored messages unchanged.

The shared normalization preserves code spans, fenced and indented code, invalid addresses, and blank-line boundaries. Regression coverage includes the opaque delegated-thread ID from the incident, spaces/tabs, LF/CRLF padding, labeled destinations, and title substitution.

Validation: focused regression tests and vp run check:changed main; iPhone 18 Pro simulator before/after verification confirms a titled thread chip, successful navigation to its target, and unchanged literal code. Web is covered through the shared parser checks; no live web session was run.

Before/after captures are saved locally at /tmp/t3-link-padding-evidence/before.png and after.png. GitHub image upload is unavailable in this session; no evidence assets are committed.

Implemented with Claude Opus 5.5 through the Claude Agent harness; reviewed, hardened, and verified by GPT-6-Astra through Codex.

@changeset-bot

changeset-bot Bot commented Nov 8, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3cdc098

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) November 8, 2026 06:15
@ScriptedAlchemy
ScriptedAlchemy merged commit d1c7b58 into main Nov 8, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant