Skip to content

docs(upstream): correct source commit IDs in review index - #738

Merged
rynfar merged 1 commit into
pylonfrom
docs/upstream-ledger-sha-corrections
Sep 24, 2026
Merged

rynfar merged 1 commit into
pylonfrom
docs/upstream-ledger-sha-corrections

Conversation

@rynfar

@rynfar rynfar commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

The upstream review index contains 15 full commit IDs that do not resolve in Git. Their ten-character prefixes and adjacent upstream PR titles identify the intended source commits, but the incorrect suffixes break exact provenance checks.

This corrects those IDs in four existing review rows. Each replacement was checked against the frozen upstream range, the commit title and upstream PR number, and the linked merged Pylon adoption PR (#680, #683/#684, #686/#687, or #690). No disposition or review cursor changes.

Validation: all full commit IDs in the corrected index resolve with git cat-file; the file equals the previous version after only these 15 replacements; git diff --check passes.

Implemented with GPT-6 Sol in Pylon.


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

@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
pylon-marketing Ignored Ignored Preview Sep 24, 2026 7:25am UTC

@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 14.0 KiB 14.0 KiB +30 B (+0.2%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB +12 B (+0.2%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.8 KiB 6.8 KiB +18 B (+0.3%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB +44 B (+0.1%) 66.4 KiB ✅
Codex Live turn messages 9 10 +1 (+11.1%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB +40 B (+0.3%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB +13 B (+0.2%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +27 B (+0.4%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB +44 B (+0.1%) 66.4 KiB ✅
Claude Live turn messages 9 10 +1 (+11.1%) 21 ✅

Baseline: 3868597 · PR result: f8eefb3 · 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: 115.7 KiB
  • Claude decoded thread snapshot: 116.4 KiB

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

@rynfar
rynfar merged commit 88a688b into pylon Sep 24, 2026
19 checks passed
@rynfar
rynfar deleted the docs/upstream-ledger-sha-corrections branch September 24, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 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.

1 participant