Skip to content

fix: preserve dynamic model headers - #6

Merged
bcho merged 1 commit into
mainfrom
fix/catalog-model-headers
Jul 10, 2026
Merged

fix: preserve dynamic model headers#6
bcho merged 1 commit into
mainfrom
fix/catalog-model-headers

Conversation

@bcho

@bcho bcho commented Jul 10, 2026

Copy link
Copy Markdown
Member

What changed

  • Preserve generated per-model headers in the dynamic catalog cache.
  • Propagate assistant API failures instead of returning blank successful output.
  • Verify cached headers reach the model registry request configuration.

Why

Dynamic GitHub Copilot models replaced built-ins without required headers such as Editor-Version. Copilot rejected prompts with HTTP 400, while the CLI suppressed the assistant error and printed a blank line.

Validation

  • npm test
  • npm run check
  • Built and installed the SEA binary locally
  • Refreshed the dynamic catalog
  • Confirmed a real GitHub Copilot prompt returns a response

@bcho
bcho merged commit b0869d4 into main Jul 10, 2026
1 check passed
@bcho
bcho deleted the fix/catalog-model-headers branch July 10, 2026 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant