From bbc41ab307ddd5a56316d4b8421dee5d70dc2e0e Mon Sep 17 00:00:00 2001 From: a1chiz Date: Thu, 5 Feb 2026 16:24:22 +0800 Subject: [PATCH] fix: Add missing body parameter to skill publish endpoint The publish endpoint was ignoring the skillContent sent by CLI because the controller wasn't accepting or passing the request body to the service. This caused: - CLI's skillContent with workflowId to be discarded - Service to fall back to generating SKILL.md with placeholder 'see-workflow-mapping' - GitHub PRs to have incorrect workflowId - Metadata updates from SKILL.md to be skipped Changes: - Add PublishSkillDto to controller imports - Add @Body() parameter to publishSkill method - Pass body to skillPackageService.publishSkillPackage() Now when CLI sends { skillContent }, it will be properly received and used to update skill metadata and create GitHub PRs with correct workflowId. Fixes: All CLI skill publish operations since 2026-01-22 (commit afc49fed4) Co-Authored-By: Claude Opus 4.5 --- .../api/src/modules/skill-package/skill-package.controller.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/apps/api/src/modules/skill-package/skill-package.controller.ts b/apps/api/src/modules/skill-package/skill-package.controller.ts index b8c0d42802..6e7322d10f 100644 --- a/apps/api/src/modules/skill-package/skill-package.controller.ts +++ b/apps/api/src/modules/skill-package/skill-package.controller.ts @@ -41,6 +41,7 @@ import { SkillExecutionResult, CreateSkillPackageCliDto, CreateSkillPackageCliResponse, + PublishSkillDto, } from './skill-package.dto'; import { SKILL_CLI_ERROR_CODES, throwCliError, mapErrorToCliCode } from './skill-package.errors'; @@ -143,8 +144,9 @@ export class SkillPackageController { async publishSkill( @Req() req: any, @Param('skillId') skillId: string, + @Body() body?: PublishSkillDto, ): Promise { - return this.skillPackageService.publishSkillPackage(req.user, skillId); + return this.skillPackageService.publishSkillPackage(req.user, skillId, body); } @Post(':skillId/unpublish')