[REF-1455]fix: Add missing body parameter to skill publish endpoint

[alchemistklk]

Problem Description

The skill publish API Controller was not accepting the skillContent parameter from the request body, causing:

1. ❌ CLI's skillContent was ignored
2. ❌ SKILL.md metadata could not be updated in the database
3. ❌ GitHub PR used placeholder workflowId: see-workflow-mapping
4. ❌ All CLI publish operations affected (since 2026-01-22)

Fix Details

Modified Files

• apps/api/src/modules/skill-package/skill-package.controller.ts

Changes

1. ✅ Add PublishSkillDto to imports
2. ✅ Add @Body() body?: PublishSkillDto parameter
3. ✅ Pass body to publishSkillPackage() service method

+ import PublishSkillDto from './skill-package.dto'

  @Post(':skillId/publish')
  async publishSkill(
    @Req() req: any,
    @Param('skillId') skillId: string,
+   @Body() body?: PublishSkillDto,
  ): Promise<SkillPackageResponse> {
-   return this.skillPackageService.publishSkillPackage(req.user, skillId);
+   return this.skillPackageService.publishSkillPackage(req.user, skillId, body);
  }

Impact

Before Fix

CLI → Controller (ignores body) → Service (receives undefined)
                                      ↓
                                Uses placeholder ❌

After Fix

CLI → Controller (receives body) → Service (receives skillContent)
                                      ↓
                                Updates DB metadata ✅
                                GitHub PR has correct workflowId ✅

Scope

• ✅ All skill publish operations via CLI
• ✅ Metadata now syncs correctly: name, description, triggers, tags, version
• ✅ GitHub registry contains correct workflowId

Testing

• ✅ TypeScript compilation passes
• ✅ Lint checks pass
• ✅ Code logic verified
• ✅ Service processing logic confirmed

Related Issue

Fixes bug introduced in commit afc49fed4 (2026-01-22), which created the publish endpoint but missed the body parameter.

---

Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com

Summary by CodeRabbit

• New Features
  • The skill publishing endpoint now supports optional configuration parameters for enhanced publishing capabilities.

[coderabbitai]

📝 Walkthrough

Walkthrough

A PublishSkillDto parameter was added as an optional argument to the publishSkill endpoint in the skill-package controller, forwarding it to the service layer. The PublishSkillDto type was also exported from the skill-package.dto module.

Changes

Cohort / File(s)	Summary
Skill Package Endpoint Enhancement apps/api/src/modules/skill-package/skill-package.controller.ts	Added optional PublishSkillDto parameter to publishSkill endpoint and updated the service call to forward the parameter. Exported PublishSkillDto from the dto module for external use.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• nettee
• CH1111

Poem

🐰 A parameter springs forth, optional and light,
The endpoint now flexible, more dynamically bright!
With PublishSkillDto exported with care,
The service receives what the client does share. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a missing body parameter to the skill publish endpoint.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/skill-publish-body

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[linear]

REF-1455 [后端]【cli】publish skill缺少参数