Skip to content

[TEST REPLICA #641] feat(codex, claude): bootstrap Codex skills and add skills-first activation - #646

Closed
tallessouza wants to merge 9 commits into
SynkraAI:mainfrom
tallessouza:test/pr641-replica
Closed

[TEST REPLICA #641] feat(codex, claude): bootstrap Codex skills and add skills-first activation#646
tallessouza wants to merge 9 commits into
SynkraAI:mainfrom
tallessouza:test/pr641-replica

Conversation

@tallessouza

@tallessouza tallessouza commented May 5, 2026

Copy link
Copy Markdown
Contributor

TEST REPLICA NOTICE — This PR is a test replica of #641 by @rafaelscosta, opened from a personal fork (tallessouza/aiox-core) to validate UI rendering and CI behavior with a different author. Same commits/diff as the original. Do not merge. Original PR: #641

Summary

This PR now contains two related local-agent activation improvements:

  1. Story 123.9: Codex local skills bootstrap for core agents and squad entry chiefs.
  2. ACORE-SKILLS: Claude skills-first activation for the 12 AIOX core agents, preserving legacy command compatibility.

Story 123.9 - Codex Skills Bootstrap

Adds .aiox-core/infrastructure/scripts/codex-skills-sync/bootstrap.js and npm wrappers:

  • npm run setup:codex-skills
  • npm run setup:codex-skills:dry

The bootstrap command generates local .codex/skills/aiox-* entries for core agents and squad entry chiefs. It keeps the existing incremental sync:skills:codex flow focused on core agents for CI and normal agent edits.

ACORE-SKILLS - Claude Skills-First Core Agents

Ports the controlled commands -> skills and agent.md -> SKILL.md pattern from Sinkra Hub into aiox-core for the 12 core AIOX agents.

Changes:

  • Generates .claude/skills/AIOX/agents/<agent-id>/SKILL.md for all 12 core agents.
  • Converts .claude/commands/AIOX/agents/*.md into legacy shims pointing to the canonical skills.
  • Adds activation_type: pipeline to generated Claude agent skills.
  • Updates Claude sync validation to expect 12 legacy commands + 12 skills.
  • Updates doctor checks to count nested AIOX agent skills.
  • Keeps non-agent Claude commands out of scope (synapse, greet, design-system, cohort-squad).
  • Documents the migration in docs/migration/ACORE-CLAUDE-SKILLS-PREFLIGHT-2026-05-01.md.

Validation

Local gates run on 2026-05-03:

```sh
npm run sync:ide:claude
npm run sync:skills:codex
npm run validate:claude-sync
npm run validate:claude-integration
npm run validate:codex-skills
git diff --check
npm test -- tests/ide-sync/transformers.test.js tests/unit/codex-skills-validate.test.js tests/unit/validate-claude-integration.test.js tests/unit/doctor-skills-count.test.js
```

Results:

  • Claude sync: PASS, 24 expected / 24 synced.
  • Claude integration: PASS, 12 skills + 12 legacy commands.
  • Codex skills: PASS, 12 skills + 12 legacy commands.
  • Codex skills: PASS, 12 skills checked.
  • Focused Jest: 4 suites, 33 tests passed.

Follow-up

Expansion from core agents to squads/*/agents is intentionally not included. The migration preflight recommends a separate ACORE-SQUAD-SKILLS epic to decide namespace, coverage, validation, and sync semantics before implementation.

rafaelscosta and others added 9 commits April 28, 2026 11:40
@rafaelscosta @claude
feat(codex): add bootstrap.js for Codex local skills (squad chiefs) […
94842cd 
…Story 123.9]

Definitive solution for the bug where squad-chief skills did not appear
in Codex's $ menu. Root cause: existing sync:skills:codex (index.js)
only generates skills from .aiox-core/development/agents/, leaving squad
chiefs in squads/*/agents/ uncovered.

Implementation:
- New: .aiox-core/infrastructure/scripts/codex-skills-sync/bootstrap.js
  Standalone, zero-deps (vendored js-yaml fallback). Generates aiox-*
  skills for both core agents AND squad entry chiefs via config.yaml
  resolution (entry_agent | tier=orchestrator | *-chief.md). Marker-aware
  (skips hand-edited skills unless --force, with .bak backup).
- New: README.md — explains the difference between sync:skills:codex
  (incremental, core-only, CI) and setup:codex-skills (full bootstrap,
  squad chiefs included, operator-facing).
- package.json: setup:codex-skills, setup:codex-skills:dry scripts.
- Sample output: .codex/skills/aiox-claude-mastery-chief/SKILL.md
  (only squad whitelisted in this repo; downstream installs with N
  squads will get N chief skills).

Operator usage:
  npm run setup:codex-skills            # full bootstrap
  npm run setup:codex-skills:dry        # preview
  npm run setup:codex-skills -- --force # overwrite hand-edited (with .bak)

Replaces the provisional standalone script previously distributed
manually to students. The script logic is preserved verbatim; only the
header doc and module exports were adapted for the canonical path.

Note: bootstrap.js sits under .aiox-core/infrastructure/scripts/ which
is in eslint global ignore, so no lint impact.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rafaelscosta
feat(claude): add skills-first agent activation [ACORE-SKILLS]
13d7eeb
@rafaelscosta
fix(validation): accept generated squad Codex skills [PR-641]
ad74d49
@rafaelscosta
fix(ci): stabilize skills schema and doctor tests [PR-641]
ebf761c
@rafaelscosta
fix(review): address skills migration review comments [PR-641]
108107e
@rafaelscosta
fix(ci): refresh install manifest for PR merge [PR-641]
d76a369
@rafaelscosta
fix(review): normalize aiox-master command visibility [PR-641]
3ff1eb6
@rafaelscosta
fix(review): address qa behavior and source path guards [PR-641]
75d7d03
@rafaelscosta
test(e2e): add installed skills smoke gate [ACORE-SKILLS.7]
6db7dc6
@vercel

vercel Bot commented May 5, 2026

Copy link
Copy Markdown

@rafaelscosta is attempting to deploy a commit to the SINKRA - AIOX Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added area: agents Agent system related area: workflows Workflow system related squad mcp type: test Test coverage and quality area: core Core framework (.aios-core/core/) area: installer Installer and setup (packages/installer/) area: synapse SYNAPSE context engine area: cli CLI tools (bin/, packages/aios-pro-cli/) area: pro Pro features (pro/) area: health-check Health check system area: docs Documentation (docs/) area: devops CI/CD, GitHub Actions (.github/) labels May 5, 2026
@coderabbitai

coderabbitai Bot commented May 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 07ec71f3-52aa-47eb-9ff4-83cdededc1e4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented May 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 26.88860% with 571 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...rastructure/scripts/codex-skills-sync/bootstrap.js 0.00% 273 Missing and 99 partials ⚠️
scripts/e2e/installed-skills-smoke.js 0.00% 119 Missing and 24 partials ⚠️
...aiox-core/infrastructure/scripts/ide-sync/index.js 41.17% 15 Missing and 5 partials ⚠️
bin/aiox.js 0.00% 8 Missing and 4 partials ⚠️
packages/installer/src/wizard/index.js 64.28% 10 Missing ⚠️
...frastructure/scripts/codex-skills-sync/validate.js 84.00% 4 Missing ⚠️
.aiox-core/core/doctor/checks/skills-count.js 91.17% 2 Missing and 1 partial ⚠️
...rastructure/scripts/validate-claude-integration.js 91.17% 3 Missing ⚠️
.aiox-core/core/doctor/checks/ide-sync.js 95.34% 2 Missing ⚠️
...cture/scripts/ide-sync/transformers/claude-code.js 95.55% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rafaelscosta

rafaelscosta commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Fechando como parte da triagem de manutenção do backlog de PRs abertos do AIOX-Core em 2026-05-07.

Motivo: este PR ficou defasado ou está fora do escopo atual do core, com conflito/estado antigo ou conteúdo de produto/app/superfície legada que não deve entrar diretamente em main.

Se ainda houver uma ideia útil aqui, o caminho correto é abrir um PR novo, pequeno e atualizado contra main, com story/critério de aceite e validação atual. Obrigado pela contribuição.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pedrovaleriolopez Pedrovaleriolopez Awaiting requested review from Pedrovaleriolopez Pedrovaleriolopez will be requested when the pull request is marked ready for review Pedrovaleriolopez is a code owner
@oalanicolas oalanicolas Awaiting requested review from oalanicolas oalanicolas will be requested when the pull request is marked ready for review oalanicolas is a code owner

Assignees

No one assigned

Labels

area: agents Agent system related area: cli CLI tools (bin/, packages/aios-pro-cli/) area: core Core framework (.aios-core/core/) area: devops CI/CD, GitHub Actions (.github/) area: docs Documentation (docs/) area: health-check Health check system area: installer Installer and setup (packages/installer/) area: pro Pro features (pro/) area: synapse SYNAPSE context engine area: workflows Workflow system related mcp squad type: test Test coverage and quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tallessouza @rafaelscosta