Skip to content

fix(models): publish Astra through compatible model feeds - #270

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-05-astra
Sep 6, 2026
Merged

fix(models): publish Astra through compatible model feeds#270
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-05-astra

Conversation

@rynfar

@rynfar rynfar commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

GPT-6 Astra is discovered by Codex but appears under Legacy because Pylon's current-model list omits it. The hosted manifest has also been stale since September 2: publication rejects the provider catalog field introduced by the Claude model-catalog integration.

Add Astra to the current-model list, porting upstream #9762 (bc03c3640d6d3bb44e5fb477bfd78d7484cd0e00). Repair publication by using the server's manifest decoder and publishing two files in one commit: the original classification feed for older strict readers, and a complete catalog at model-catalog.json for current servers. Both outputs keep the 32 KiB limit. The workflow prepares the files from the approved revision before checking whether it is still current and accessing the release credential.

Web, desktop, and mobile consume the same server classification, including remote and tunnel connections. This change preserves explicit model selections and the existing default-model preference. Update the publishing runbook and record the targeted adoption without advancing the full upstream review cursor.

Validation: 16 focused manifest/publication and Claude catalog tests pass; server typecheck passes with existing suggestions; targeted lint and formatting pass; actionlint passes. Ran the publication CLI under Node 24.13.1 and compared both generated JSON files with the source. No browser or simulator was used. Publication to the hosted feed happens after merge and still needs its workflow result checked.

Implemented with GPT-6 Astra through the Codex harness.


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

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

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.6 KiB 13.6 KiB −13 B (−0.1%) 15.1 KiB
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −2 B (−0.0%) 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 6.5 KiB −11 B (−0.2%) 7.8 KiB
Codex Live turn WebSocket decoded 56.5 KiB 56.5 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 8 8 0 (0.0%) 21
Claude Total thread wire 13.6 KiB 13.6 KiB +8 B (+0.1%) 15.1 KiB
Claude Thread snapshot wire 7.1 KiB 7.1 KiB −5 B (−0.1%) 7.3 KiB
Claude Live turn WebSocket wire 6.6 KiB 6.6 KiB +13 B (+0.2%) 7.8 KiB
Claude Live turn WebSocket decoded 57.3 KiB 57.3 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 8 8 0 (0.0%) 21

Baseline: 2f02e71 · PR result: 1ee780b · 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.2 KiB
  • Claude decoded thread snapshot: 111.9 KiB

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

Port current-model classification from pingdotgg/t3code#9762 (bc03c3640d6d3bb44e5fb477bfd78d7484cd0e00). Reuse runtime manifest validation and publish a separate catalog while retaining the strict legacy classification feed.
@rynfar
rynfar force-pushed the upstream/2026-09-05-astra branch from 8130e52 to 1ee780b Compare September 6, 2026 03:47
@rynfar
rynfar merged commit aa12838 into pylon Sep 6, 2026
18 checks passed
@rynfar
rynfar deleted the upstream/2026-09-05-astra branch September 6, 2026 04:13
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.

1 participant