Skip to content

fix: canonicalize Pro docs and Kimi skills [Story PRO-13.4] - #719

Merged
rafaelscosta merged 3 commits into
mainfrom
feat/pro-13-4-legacy-cleanup-core
May 9, 2026
Merged

fix: canonicalize Pro docs and Kimi skills [Story PRO-13.4]#719
rafaelscosta merged 3 commits into
mainfrom
feat/pro-13-4-legacy-cleanup-core

Conversation

@rafaelscosta

@rafaelscosta rafaelscosta commented May 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • updates active Pro install/publishing docs for @aiox-squads/core@5.1.17, private @aiox-squads/pro@0.4.2, and signed-artifact acquisition
  • switches Kimi IDE skill projection from legacy aios-* directories to canonical aiox-* directories without keeping duplicate generated skills
  • keeps Claude legacy slash commands as thin shims and Codex skills canonical as aiox-*

Validation

  • npm test -- --runInBand --forceExit tests/ide-sync/kimi-transformer.test.js tests/ide-sync/validator.test.js tests/integration/codex-skills-sync.test.js tests/unit/codex-skills-validate.test.js
  • npm run validate:codex-skills
  • npm run validate:claude-integration
  • npm run sync:ide:check
  • npm run validate:publish
  • npm run lint
  • npm run typecheck
  • npm run validate:manifest
  • npm registry check: @aiox-squads/core=5.1.17, @aiox-squads/pro=0.4.2, @aiox-squads/pro status=private

Summary by CodeRabbit

  • Chores

    • Updated system branding from AIOS to AIOX across agent identifiers and activation wording.
  • Documentation

    • Pro installation and publishing docs revised for authenticated distribution, license validation, and canonical package @aiox-squads/pro; clarified installation and release baseline guidance.
  • Tests

    • Test suites adjusted to expect AIOX naming and to ensure titles aren’t double-prefixed with AIOX.

Review Change Stack

@vercel

vercel Bot commented May 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
aiox-core Ready Ready Preview, Comment May 9, 2026 9:17am

Request Review

@coderabbitai

coderabbitai Bot commented May 9, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 592e9b45-dd55-4b8b-a651-1a0cfaa1cda7

📥 Commits

Reviewing files that changed from the base of the PR and between 8aadf6c and 94cb5f9.

📒 Files selected for processing (2)
  • docs/PUBLISHING.md
  • docs/guides/pro/install-gate-setup.md
✅ Files skipped from review due to trivial changes (1)
  • docs/PUBLISHING.md

Walkthrough

Rename AIOS branding to AIOX: transformer normalizes IDs to aiox-, skill YAML names updated, tests and docs adjusted, and install manifest checksums refreshed.

Changes

AIOS to AIOX Naming Migration

Layer / File(s) Summary
Transformer Logic
.aiox-core/infrastructure/scripts/ide-sync/transformers/kimi.js
getSkillId normalized to aiox- prefix; aios-aiox- conversion; fallback title and activation description updated to AIOX branding; doc path updated to .kimi/skills/aiox-{agent-id}/SKILL.md.
Skill Metadata Frontmatter
.kimi/skills/aiox-*/SKILL.md (12 files)
Front-matter name fields changed from aios-* to aiox-*; several description strings updated to reference AIOX.
Tests / Expectations
tests/ide-sync/kimi-transformer.test.js, tests/ide-sync/validator.test.js
Test assertions and comments updated to expect aiox- normalization, directory derivation, nested layout checks, and to avoid double-AIOX title prefixing.
Distribution & Release Documentation
docs/PUBLISHING.md, docs/guides/pro/install-gate-setup.md
Publishing and Pro install docs updated: canonical package set to @aiox-squads/pro, signed-artifact acquisition described, core dependency/prerequisite versions adjusted, and privacy/transition wording revised.
Manifest & Checksums
.aiox-core/install-manifest.yaml
generated_at timestamp, sha256, and size entries refreshed for the updated transformer file.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

area: core, area: pro, area: agents

Suggested reviewers

  • Pedrovaleriolopez
  • oalanicolas
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: canonicalizing Pro documentation and Kimi skills to AIOX branding, with reference to the story ticket PRO-13.4.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/pro-13-4-legacy-cleanup-core

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.

@github-actions github-actions Bot added type: test Test coverage and quality area: docs Documentation (docs/) labels May 9, 2026
@github-actions

github-actions Bot commented May 9, 2026

Copy link
Copy Markdown
Contributor

📊 Coverage Report

Coverage report not available

📈 Full coverage report available in Codecov


Generated by PR Automation (Story 6.1)

@coderabbitai coderabbitai 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.

Actionable comments posted: 2

🧹 Nitpick comments (1)
docs/guides/pro/install-gate-setup.md (1)

11-11: ⚡ Quick win

Clarify legacy repo name vs current service naming.

The sentence mixes historical aios-license-server with current AIOX branding; add a short clarifier so readers don’t mistake this for a typo.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/guides/pro/install-gate-setup.md` at line 11, The sentence refers to the
historical service name `aios-license-server` while the product uses AIOX
branding; update the sentence in the paragraph so it explicitly calls out that
`aios-license-server` is the legacy/license-server repo name (e.g., “the
historical `aios-license-server` service, now maintained under the AIOX
branding”) to avoid confusion between the legacy repo name and current AIOX
service naming.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.kimi/skills/aiox-master/SKILL.md:
- Line 3: Update the "description" field to remove the duplicated product name
by changing "AIOX AIOX Master Orchestrator & Framework Developer agent (Orion)"
to "AIOX Master Orchestrator & Framework Developer agent (Orion)"; edit the
description string in SKILL.md (the line containing description:) so all trigger
examples and surrounding text remain unchanged aside from the duplicate removal.

In `@docs/guides/pro/install-gate-setup.md`:
- Around line 13-15: The fenced code block containing "Comprar Licença ->
Validar -> Baixar artefato assinado -> Usar Features Pro" is unlabeled and
triggers markdownlint MD040; update that fenced block (the triple-backtick block
around that line) to include a language identifier such as text (i.e., change
``` to ```text) so the markdown block is properly labeled.

---

Nitpick comments:
In `@docs/guides/pro/install-gate-setup.md`:
- Line 11: The sentence refers to the historical service name
`aios-license-server` while the product uses AIOX branding; update the sentence
in the paragraph so it explicitly calls out that `aios-license-server` is the
legacy/license-server repo name (e.g., “the historical `aios-license-server`
service, now maintained under the AIOX branding”) to avoid confusion between the
legacy repo name and current AIOX service naming.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2fab54d7-d0bb-41b6-8b31-efa6a068b342

📥 Commits

Reviewing files that changed from the base of the PR and between 9368432 and a0eb98e.

📒 Files selected for processing (18)
  • .aiox-core/infrastructure/scripts/ide-sync/transformers/kimi.js
  • .aiox-core/install-manifest.yaml
  • .kimi/skills/aiox-analyst/SKILL.md
  • .kimi/skills/aiox-architect/SKILL.md
  • .kimi/skills/aiox-data-engineer/SKILL.md
  • .kimi/skills/aiox-dev/SKILL.md
  • .kimi/skills/aiox-devops/SKILL.md
  • .kimi/skills/aiox-master/SKILL.md
  • .kimi/skills/aiox-pm/SKILL.md
  • .kimi/skills/aiox-po/SKILL.md
  • .kimi/skills/aiox-qa/SKILL.md
  • .kimi/skills/aiox-sm/SKILL.md
  • .kimi/skills/aiox-squad-creator/SKILL.md
  • .kimi/skills/aiox-ux-design-expert/SKILL.md
  • docs/PUBLISHING.md
  • docs/guides/pro/install-gate-setup.md
  • tests/ide-sync/kimi-transformer.test.js
  • tests/ide-sync/validator.test.js

Comment thread .kimi/skills/aiox-master/SKILL.md Outdated
Comment thread docs/guides/pro/install-gate-setup.md Outdated

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/guides/pro/install-gate-setup.md`:
- Line 46: Update the prerequisite version for the package reference
`@aiox-squads/core` in the docs: change the allowed version text from ">= 5.1.0"
to the targeted baseline "5.1.17" (or a constrained range that starts at 5.1.17)
so the guide aligns with the release baseline `@aiox-squads/core@5.1.17`.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f1a402ca-9197-4139-93c8-9c10c0fd58db

📥 Commits

Reviewing files that changed from the base of the PR and between a0eb98e and 8aadf6c.

📒 Files selected for processing (5)
  • .aiox-core/infrastructure/scripts/ide-sync/transformers/kimi.js
  • .aiox-core/install-manifest.yaml
  • .kimi/skills/aiox-master/SKILL.md
  • docs/guides/pro/install-gate-setup.md
  • tests/ide-sync/kimi-transformer.test.js
✅ Files skipped from review due to trivial changes (2)
  • .kimi/skills/aiox-master/SKILL.md
  • .aiox-core/install-manifest.yaml
🚧 Files skipped from review as they are similar to previous changes (2)
  • .aiox-core/infrastructure/scripts/ide-sync/transformers/kimi.js
  • tests/ide-sync/kimi-transformer.test.js

Comment thread docs/guides/pro/install-gate-setup.md Outdated
@rafaelscosta
rafaelscosta merged commit fc68344 into main May 9, 2026
40 checks passed
@rafaelscosta
rafaelscosta deleted the feat/pro-13-4-legacy-cleanup-core branch May 9, 2026 09:22
@github-actions

github-actions Bot commented May 9, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

tuanmedeiros pushed a commit to tuanmedeiros/aios-core-synkraay that referenced this pull request Jun 2, 2026
…#719)

* fix: canonicalize Pro docs and Kimi skills [Story PRO-13.4]

* fix: address Kimi review nits [Story PRO-13.4]

* docs: align Pro release baseline [Story PRO-13.4]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Documentation (docs/) released type: test Test coverage and quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant