Skip to content

Fix broken skills (additionalDirectories) in 0.130.0 #156

Open
NikolaiSviridov wants to merge 8 commits into
mainfrom
codex-update/0.130.0-skills
Open

Fix broken skills (additionalDirectories) in 0.130.0 #156
NikolaiSviridov wants to merge 8 commits into
mainfrom
codex-update/0.130.0-skills

Conversation

@NikolaiSviridov
Copy link
Copy Markdown
Collaborator

No description provided.

@AlexandrSuhinin
Copy link
Copy Markdown
Collaborator

Why this is done on top of update to 0.130.0? I would expect that we first change skills to new implementation and the do codex update (probably to latest 0.132.0) and check that all tests are still green.

@NikolaiSviridov
Copy link
Copy Markdown
Collaborator Author

@AlexandrSuhinin skills were broken in 0.130.0 and there is a new logic related to skill handling in it (recomended by openai). So, I wanted to merge update 0.130.0 and update to 0.132.0 after.

Can move it to 0.132.0

Comment thread src/CodexAcpClient.ts Outdated
Comment thread src/CodexAcpClient.ts
Comment thread src/CodexAcpClient.ts
Comment thread src/SkillDirectoryParser.ts
Comment thread src/CodexAppServerClient.ts
Comment thread src/CodexAcpClient.ts Outdated
@AlexandrSuhinin
Copy link
Copy Markdown
Collaborator

Please add e2e tests if they are missing.

@AlexandrSuhinin
Copy link
Copy Markdown
Collaborator

So there is no marketplace API in 0.128.0 version and you're dependent on update to 0.130.0+?

@AlexandrSuhinin
Copy link
Copy Markdown
Collaborator

Also I don't really understand if it's a bug, but this review doesn't include all related changes.

Comment thread src/__tests__/CodexACPAgent/e2e/acp-e2e.test.ts Outdated
Comment thread src/CodexAcpServer.ts Outdated
@AlexandrSuhinin
Copy link
Copy Markdown
Collaborator

Please rebase this branch on master if marketplace events are supported in 0.128.0.
Reviewing and merging this will be too error-prone otherwise.

@NikolaiSviridov NikolaiSviridov changed the base branch from codex-update/0.130.0 to main May 26, 2026 11:33
@NikolaiSviridov NikolaiSviridov force-pushed the codex-update/0.130.0-skills branch from 5f49e5e to 17f7fee Compare May 26, 2026 12:59
@NikolaiSviridov NikolaiSviridov force-pushed the codex-update/0.130.0-skills branch from 17f7fee to c7b6f36 Compare May 26, 2026 13:03
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.

3 participants