Skip to content

fix known issues in issue list (model name error, chat-completions and doc)#212

Open
hitdavid wants to merge 14 commits intoericc-ch:masterfrom
hitdavid:master
Open

fix known issues in issue list (model name error, chat-completions and doc)#212
hitdavid wants to merge 14 commits intoericc-ch:masterfrom
hitdavid:master

Conversation

@hitdavid
Copy link
Copy Markdown

No description provided.

StarryKira and others added 12 commits February 13, 2026 06:39
- translateModelName: strip [1m]/bracket suffixes, 8-digit date stamps,
  and convert hyphen minor versions to dot notation (e.g. claude-haiku-4-5
  → claude-haiku-4.5) to match Copilot's expected model ID format
- HTTPError: carry pre-read response body to avoid "Body already read" crash
- forwardError: use stored body instead of re-reading response; remove async
- get-models: pre-read body before throwing HTTPError
- route.ts: remove unnecessary optional chain on non-nullable supports/limits

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Set file permissions to 0o600 for admin.json and accounts.json to
  restrict access to sensitive credential and token data
- Remove githubToken from all account API responses to prevent token
  exposure to browser/frontend
- Fix Bearer token parsing to be case-insensitive (use regex instead
  of string replace)
- Auto-cleanup expired device flow sessions from memory after 60s
- Remove .idea/ IDE files from git tracking and add to .gitignore

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@caozhiyuan
Copy link
Copy Markdown
Contributor

@hitdavid The responses API needs to return the thought signature in the request body; otherwise, it will affect model usage and degrade performance. The chat-completions open compatibility interface cannot transmit the thought signature, so it is not recommended to adapt the responses API to chat-completions. you can use this https://github.com/caozhiyuan/copilot-api . This support uses the message API for GPT and the native Claude message API for the Claude model.

@caozhiyuan
Copy link
Copy Markdown
Contributor

@hitdavid Similar to this , look here #205 (comment)

@theguy000
Copy link
Copy Markdown

you can use this https://github.com/caozhiyuan/copilot-api

Nice of you to do all these new updates. my question is - a
re you updating the commits? or you are taking hte commits from here and merging them in your fork?

is the maintainer of this project has stopped updating the project?

Thank you.

@caozhiyuan
Copy link
Copy Markdown
Contributor

@theguy000 This project hasn't been maintained for six months. What do you want to ask about ? this is my pr https://github.com/ericc-ch/copilot-api/pull/170 , that first create at October 2025

@theguy000
Copy link
Copy Markdown

@caozhiyuan I wanted to know if you were updating your fork using some of the commits from here or all the commits were yours.

I will be using your fork.

@caozhiyuan
Copy link
Copy Markdown
Contributor

caozhiyuan commented Mar 24, 2026

@caozhiyuan I wanted to know if you were updating your fork using some of the commits from here or all the commits were yours.

I will be using your fork.

@theguy000 no code has been merged from this repository's PR. you can see commits history.

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.

4 participants