Skip to content

feat(markdown,usage): adopt five remaining upstream corrections - #450

Merged
rynfar merged 5 commits into
pylonfrom
upstream/2026-09-10-misc-fixes
Sep 10, 2026
Merged

rynfar merged 5 commits into
pylonfrom
upstream/2026-09-10-misc-fixes

Conversation

@rynfar

@rynfar rynfar commented Sep 10, 2026 •

Copy link
Copy Markdown
Collaborator

Five remaining clean-applying sources from the upstream catch-up range: the
GitHub mark on github.com markdown links, Claude usage limits on retried turns,
email-bearing account labels hidden in usage limits, and two review-agent label
additions to the existing .macroscope config.

Adopted from T3 Code

  • 6ba15c027a6d411c7f75cc0ca59b601a295c3633 fix: hide email-bearing account labels in usage limits (#10668)
  • 5a18fb95e49ea8352963329f32d5e85cf3125e24 Add 'macroscope-review' label to UI consistency
  • f0bd43eaf9d8b5e3df2b913532bbee74bba58314 Add 'macroscope-review' label to conventions
  • 3bf74eb6dce3e1372cfafc2ca84825f4b13bae6a fix(claude): report usage limits on retried turns (#10549)
  • 272d6d747ef214f50dbd9d0051120de12ffa3591 feat(markdown): show the GitHub mark for github.com links (#10324)

Pylon adaptations

None. Every source applied unchanged.

Verification

  • Tests: 191 passing across three suites.
  • vp check: 0 errors across 3362 files.
  • Unused exports and files: knip:check clean.

The markdown link module gains a ./links export, so a reinstall is needed
before its tests resolve; that is a local workflow note rather than a change.

Model and harness: Claude Opus 5 via Claude Code.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Source traceability verified during cycle #414 closure: the source list now uses frozen upstream SHAs. The previous list used Pylon port commit SHAs. All listed port commits are merged ancestors; source/port changes match (some patch IDs differ only because surrounding context changed).

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.8 KiB 13.6 KiB −180 B (−1.3%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB 0 B (0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.5 KiB −180 B (−2.6%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 57.4 KiB 56.5 KiB −910 B (−1.5%) 66.4 KiB ✅
Codex Live turn messages 10 8 −2 (−20.0%) 21 ✅
Claude Total thread wire 13.8 KiB 13.8 KiB +29 B (+0.2%) 15.1 KiB ✅
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +7 B (+0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.7 KiB +22 B (+0.3%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.2 KiB 58.2 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 10 10 0 (0.0%) 21 ✅

Baseline: d7aaa61 · PR result: 1df2aba · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 111.3 KiB
  • Claude decoded thread snapshot: 111.9 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@rynfar
rynfar merged commit 2018bb8 into pylon Sep 10, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants