feat: robust skills management (#610) (#620)

* feat: skills management production-grade (#610)

- Added profile.skill_remove action
- Extracted skills list in profile read
- Handled empty skills dropdown cases
- Added MCP tools for read/remove skills
- Improved reorder robustness

* fix #610: resolve typescript errors in profile seed and mcp tools

Author: sigvardt

packages/cli/src/profileSeed.ts

@@ -27,7 +27,8 @@ const SECTION_KEY_ALIASES = new Map<string, LinkedInProfileSectionType>([
   ["volunteerexperience", "volunteer_experience"],
   ["volunteer_experience", "volunteer_experience"],
   ["honorsawards", "honors_awards"],
-  ["honors_awards", "honors_awards"]
+  ["honors_awards", "honors_awards"],
+  ["skills", "skills"]
 ]);
 
 const INTRO_FIELD_KEYS = new Set(["firstName", "lastName", "headline", "location"]);
@@ -45,7 +46,8 @@ const SECTION_IDENTITY_FIELDS: Record<Exclude<LinkedInProfileSectionType, "about
   languages: ["name"],
   projects: ["title"],
   volunteer_experience: ["role", "organization"],
-  honors_awards: ["title", "issuer"]
+  honors_awards: ["title", "issuer"],
+  skills: ["name"]
 };
 
 type SeedSectionType = Exclude<LinkedInProfileSectionType, "about">;
@@ -119,12 +121,12 @@ export function parseProfileSeedSpec(input: unknown): ProfileSeedSpec {
       continue;
     }
 
-    if (rawKey === "skills") {
+    if (rawKey === "recommendations") {
       if (Array.isArray(rawValue) && rawValue.length > 0) {
         unsupportedFields.push({
-          path: "skills",
-          reason: "Skills are not exposed by the current LinkedIn profile edit automation.",
-          issueNumber: 228
+          path: "recommendations",
+          reason: "Recommendations are not exposed by the current LinkedIn profile edit automation.",
+          issueNumber: 0
         });
       }
       continue;
@@ -636,6 +638,8 @@ function describeSectionValues(
       return joinSummaryParts(values.role, values.organization);
     case "honors_awards":
       return joinSummaryParts(values.title, values.issuer);
+    case "skills":
+      return readString(values.name);
   }
 }
 
@@ -653,6 +657,8 @@ function describeSectionItem(
       return joinSummaryParts(item.primary_text, item.secondary_text);
     case "projects":
       return readString(item.primary_text);
+    case "skills":
+      return readString(item.primary_text);
   }
 }
 

packages/cli/test/profileCli.test.ts

@@ -319,7 +319,7 @@ describe("CLI profile commands", () => {
             customProfileUrl: "avery-automation"
           },
           about: "Building production LLM systems.",
-          skills: ["TypeScript", "Python"]
+          recommendations: ["TypeScript", "Python"]
         },
         null,
         2
@@ -358,9 +358,9 @@ describe("CLI profile commands", () => {
     ]);
     expect(output.unsupported_fields).toEqual([
       {
-        path: "skills",
-        reason: "Skills are not exposed by the current LinkedIn profile edit automation.",
-        issueNumber: 228
+        path: "recommendations",
+        reason: "Recommendations are not exposed by the current LinkedIn profile edit automation.",
+        issueNumber: 0
       }
     ]);
     expect(profileCliMocks.prepareUpdateIntro).toHaveBeenCalledWith(
@@ -401,7 +401,7 @@ describe("CLI profile commands", () => {
           intro: {
             headline: "Automation Engineer at Example Labs"
           },
-          skills: ["TypeScript"]
+          recommendations: ["TypeScript"]
         },
         null,
         2
@@ -420,7 +420,7 @@ describe("CLI profile commands", () => {
         specPath,
         "--yes"
       ])
-    ).rejects.toThrow("Profile seed spec includes unsupported fields: skills (#228)");
+    ).rejects.toThrow("Profile seed spec includes unsupported fields: recommendations (#0)");
 
     expect(profileCliMocks.prepareUpdateIntro).not.toHaveBeenCalled();
     expect(profileCliMocks.confirmByToken).not.toHaveBeenCalled();

packages/cli/test/profileSeed.test.ts

@@ -76,7 +76,7 @@ describe("profile seed planner", () => {
           issueYear: "2024",
         },
       ],
-      skills: ["TypeScript", "Python"],
+      recommendations: ["TypeScript", "Python"],
     });
 
     expect(spec.intro).toMatchObject({
@@ -89,10 +89,10 @@ describe("profile seed planner", () => {
     expect(spec.sections.certifications).toHaveLength(1);
     expect(spec.unsupportedFields).toEqual([
       {
-        path: "skills",
+        path: "recommendations",
         reason:
-          "Skills are not exposed by the current LinkedIn profile edit automation.",
-        issueNumber: 228,
+          "Recommendations are not exposed by the current LinkedIn profile edit automation.",
+        issueNumber: 0,
       },
     ]);
   });

packages/core/src/__tests__/e2e/helpers.ts

@@ -108,6 +108,8 @@ import {
   LINKEDIN_PROFILE_PREPARE_FEATURED_REORDER_TOOL,
   LINKEDIN_PROFILE_PREPARE_ADD_SKILL_TOOL,
   LINKEDIN_PROFILE_PREPARE_REORDER_SKILLS_TOOL,
+  LINKEDIN_PROFILE_READ_SKILLS_TOOL,
+  LINKEDIN_PROFILE_PREPARE_REMOVE_SKILL_TOOL,
   LINKEDIN_PROFILE_PREPARE_ENDORSE_SKILL_TOOL,
   LINKEDIN_PROFILE_PREPARE_REQUEST_RECOMMENDATION_TOOL,
   LINKEDIN_PROFILE_PREPARE_WRITE_RECOMMENDATION_TOOL,
@@ -1060,6 +1062,8 @@ export const MCP_TOOL_NAMES = {
   profilePrepareFeaturedReorder: LINKEDIN_PROFILE_PREPARE_FEATURED_REORDER_TOOL,
   profilePrepareAddSkill: LINKEDIN_PROFILE_PREPARE_ADD_SKILL_TOOL,
   profilePrepareReorderSkills: LINKEDIN_PROFILE_PREPARE_REORDER_SKILLS_TOOL,
+  profileReadSkills: LINKEDIN_PROFILE_READ_SKILLS_TOOL,
+  profilePrepareRemoveSkill: LINKEDIN_PROFILE_PREPARE_REMOVE_SKILL_TOOL,
   profilePrepareEndorseSkill: LINKEDIN_PROFILE_PREPARE_ENDORSE_SKILL_TOOL,
   profilePrepareRequestRecommendation:
     LINKEDIN_PROFILE_PREPARE_REQUEST_RECOMMENDATION_TOOL,

packages/core/src/__tests__/linkedinProfile.test.ts

@@ -1039,6 +1039,7 @@ describe("profile action type constants", () => {
       "projects",
       "volunteer_experience",
       "honors_awards",
+      "skills"
     ]);
     expect(LINKEDIN_PROFILE_SECTION_TYPES).not.toContain("featured");
     expect(LINKEDIN_PROFILE_FEATURED_ITEM_KINDS).toEqual([