Skip to content

Normalize Codex MCP, skill, and collab agent tool items#201

Merged
SDSLeon merged 1 commit into
masterfrom
lightcode/bright-falcon-7f268f95
Jun 18, 2026
Merged

Normalize Codex MCP, skill, and collab agent tool items#201
SDSLeon merged 1 commit into
masterfrom
lightcode/bright-falcon-7f268f95

Conversation

@SDSLeon

@SDSLeon SDSLeon commented Jun 18, 2026

Copy link
Copy Markdown
Owner

Type: Bugfix

  • Map Codex mcpToolCall and collabAgentToolCall items to canonical MCP and subagent tool shapes so they render and complete with the right metadata in chat
  • Recognize skill invocations from file-path reads (e.g. SKILL.md) and show them as skill tools instead of generic file reads
  • Unwrap MCP result content text blocks so tool output displays formatted JSON instead of raw objects
  • Classify MCP, skill, and collab agent usage events consistently in the usage recorder for Codex threads
  • Fix cross-platform test assumptions for project directory creation and CommandCode plugin install wrappers on Windows

- Map Codex mcpToolCall and collabAgentToolCall notifications to canonical payloads
- Classify skills, MCP servers, and subagents in usage recording
- Show SKILL.md reads as skill displays in chat tool UI
- Unwrap MCP result content text blocks for ACP tool output
@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
lightcode-landing Skipped Skipped Jun 18, 2026 4:34am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 82cfb30009

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

/^(loaded|using) skill\b/i.test(name) ||
/^(loaded|using) skill\b/i.test(title)
/^(loaded|using) skill\b/i.test(title) ||
readSkillName(p, args) !== undefined

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid classifying args.name as a skill

When any non-skill tool carries an args.name value, this predicate now succeeds because readSkillName() returns str(args, "name"); for example existing Workflow tool events with args.name never reach the subagent branch below and are recorded as kind: "skill" instead of subagent, corrupting usage stats. Restrict this check to actual Skill tools or SKILL.md paths, and only use args.name after the item is known to be a skill.

Useful? React with 👍 / 👎.

@SDSLeon SDSLeon merged commit 2d8524c into master Jun 18, 2026
6 checks passed
@SDSLeon SDSLeon deleted the lightcode/bright-falcon-7f268f95 branch June 18, 2026 04:39
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