Skip to content

fix(ci): publish model feeds with vendored gitlinks - #280

Merged
rynfar merged 1 commit into
pylonfrom
fix/model-feed-checkout
Sep 6, 2026
Merged

fix(ci): publish model feeds with vendored gitlinks#280
rynfar merged 1 commit into
pylonfrom
fix/model-feed-checkout

Conversation

@rynfar

@rynfar rynfar commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

The model feed publisher fails during checkout before validation: removing checkout credentials traverses a vendored gitlink that has no .gitmodules entry. This prevented #270 from reaching the public feeds.

Keep the job-scoped read-only checkout token until post-job cleanup, matching the working CI checkout path. Publication credentials remain confined to the later public-repository steps, after manifest validation and the current-revision check.

Validation: actionlint, workflow formatting, and git diff checks pass. The failed publication run is https://github.com/pylon-code/pylon/actions/runs/34010217243; verify both published feeds after this lands.

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:XS labels Sep 6, 2026
@github-actions

github-actions Bot commented Sep 6, 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 0 B (0.0%) 15.1 KiB
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −8 B (−0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 6.6 KiB +8 B (+0.1%) 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 +6 B (+0.0%) 15.1 KiB
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +12 B (+0.2%) 7.3 KiB
Claude Live turn WebSocket wire 6.6 KiB 6.6 KiB −6 B (−0.1%) 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: aa12838 · PR result: 3c0afcb · 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.

@rynfar
rynfar merged commit 38ae86e into pylon Sep 6, 2026
17 checks passed
@rynfar
rynfar deleted the fix/model-feed-checkout 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: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