Skip to content

test(web): align emoji-prefixed monogram expectation - #571

Merged
rynfar merged 1 commit into
pylonfrom
fix/project-monogram-test-expectation
Sep 15, 2026
Merged

rynfar merged 1 commit into
pylonfrom
fix/project-monogram-test-expectation

Conversation

@rynfar

@rynfar rynfar commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

The project monogram test expected LA for an emoji-prefixed Launch, but the shipped algorithm intentionally uses the first and last letters of a single word (LH). Correct that expectation, consistent with the existing Nebula → NA case.

This failure already exists on pylon at 7f10561 and blocked the unrelated Prime transcript PR #570. All 14 focused project identity tests pass. Product behavior is unchanged.

Implemented and reviewed with GPT-6 Astra in Codex.


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 15, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
pylon-marketing Ready Ready Preview Sep 15, 2026 12:37am UTC

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

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.9 KiB 13.9 KiB +14 B (+0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB +2 B (+0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.7 KiB +12 B (+0.2%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 8 8 0 (0.0%) 21 ✅
Claude Total thread wire 13.9 KiB 13.9 KiB −8 B (−0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB +3 B (+0.0%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.7 KiB −11 B (−0.2%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.8 KiB 58.8 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 8 8 0 (0.0%) 21 ✅

Baseline: 9e59445 · PR result: 0b978d4 · 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.3 KiB

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

@rynfar
rynfar merged commit 10b25f1 into pylon Sep 15, 2026
19 checks passed
@rynfar
rynfar deleted the fix/project-monogram-test-expectation branch September 15, 2026 01:06

This branch was successfully deployed

1 active deployment
Preview — 0b978d45 Deployed Sep 15, 2026 by vercel[bot]
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