Skip to content

Refine structured skill tool flow#45

Merged
chandra447 merged 3 commits into
mainfrom
codex/structured-skill-tool-flow
May 21, 2026
Merged

Refine structured skill tool flow#45
chandra447 merged 3 commits into
mainfrom
codex/structured-skill-tool-flow

Conversation

@chandra447
Copy link
Copy Markdown
Owner

Summary

  • remove background skill auto-extraction so procedural skills are managed explicitly through the main skill tool flow
  • add structured create/update inputs for skills (when_to_use, procedure_steps, pitfalls, verification_steps) and render valid SKILL.md sections in the tool
  • tighten the tool contract and complete the edit -> update guidance cleanup while keeping the legacy edit alias working
  • update docs, prompt policy text, and the source-architecture diagram to match the new inline skill flow

Testing

  • npm run check
  • npx tsx --test tests/tools/skill-tool.test.ts tests/handlers/prompt-context.test.ts tests/handlers/background-review.test.ts tests/store/skill-store.test.ts

Notes

  • full npm test is still blocked in this environment by the existing better-sqlite3 Node ABI mismatch in unrelated SQLite-backed tests

@chandra447 chandra447 merged commit 4547fcf into main May 21, 2026
3 checks passed
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