fix: canonicalize Pro docs and Kimi skills [Story PRO-13.4] - #719
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
WalkthroughRename AIOS branding to AIOX: transformer normalizes IDs to ChangesAIOS to AIOX Naming Migration
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
📊 Coverage ReportCoverage report not available
Generated by PR Automation (Story 6.1) |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
docs/guides/pro/install-gate-setup.md (1)
11-11: ⚡ Quick winClarify legacy repo name vs current service naming.
The sentence mixes historical
aios-license-serverwith 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
📒 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.mddocs/PUBLISHING.mddocs/guides/pro/install-gate-setup.mdtests/ide-sync/kimi-transformer.test.jstests/ide-sync/validator.test.js
There was a problem hiding this comment.
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
📒 Files selected for processing (5)
.aiox-core/infrastructure/scripts/ide-sync/transformers/kimi.js.aiox-core/install-manifest.yaml.kimi/skills/aiox-master/SKILL.mddocs/guides/pro/install-gate-setup.mdtests/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
|
🎉 This PR is included in version 5.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
…#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]
Summary
Validation
Summary by CodeRabbit
Chores
Documentation
Tests