feat: add robust profile banner upload/remove management (#614)

- Implement prepareRemoveBanner and RemoveProfileBannerActionExecutor
- Add banner file size and dimension checks locally
- Update CLI and MCP commands
- Add unit/E2E tests for banner operations

Author: sigvardt

package-lock.json

@@ -7319,6 +7319,40 @@ async function runProfilePrepareUploadPhoto(
   }
 }
 
+async function runProfilePrepareRemoveBanner(
+  input: {
+    profileName: string;
+    operatorNote?: string;
+  },
+  cdpUrl?: string,
+): Promise<void> {
+  const runtime = createRuntime(cdpUrl);
+
+  try {
+    runtime.logger.log("info", "cli.profile.prepare_remove_banner.start", {
+      profileName: input.profileName,
+    });
+
+    const prepared = await runtime.profile.prepareRemoveBanner({
+      profileName: input.profileName,
+      ...(input.operatorNote ? { operatorNote: input.operatorNote } : {}),
+    });
+
+    runtime.logger.log("info", "cli.profile.prepare_remove_banner.done", {
+      profileName: input.profileName,
+      preparedActionId: prepared.preparedActionId,
+    });
+
+    printPrepareResult({
+      run_id: runtime.runId,
+      profile_name: input.profileName,
+      ...prepared,
+    });
+  } finally {
+    runtime.close();
+  }
+}
+
 async function runProfilePrepareUploadBanner(
   input: {
     profileName: string;
@@ -13383,6 +13417,31 @@ export function createCliProgram(): Command {
       },
     );
 
+  profileCommand
+    .command("prepare-remove-banner")
+    .description("Prepare to remove a LinkedIn profile banner (two-phase)")
+    .option("-p, --profile <profile>", "Profile name", "default")
+    .option(
+      "--operator-note <note>",
+      "Optional note attached to the prepared action",
+    )
+    .action(
+      async (options: {
+        operatorNote?: string;
+        profile: string;
+      }) => {
+        await runProfilePrepareRemoveBanner(
+          {
+            profileName: options.profile,
+            ...(options.operatorNote
+              ? { operatorNote: options.operatorNote }
+              : {}),
+          },
+          readCdpUrl(),
+        );
+      },
+    );
+
   profileCommand
     .command("apply-spec")
     .description(

packages/cli/src/bin/linkedin.ts

@@ -49,6 +49,7 @@
   "dependencies": {
     "better-sqlite3": "^12.8.0",
     "fflate": "^0.8.2",
+    "image-size": "^2.0.2",
     "playwright-core": "npm:patchright-core@^1.58.0",
     "playwright-extra": "^4.3.6",
     "pngjs": "^7.0.0",

packages/core/package.json

@@ -109,6 +109,7 @@ import {
   LINKEDIN_PROFILE_PREPARE_REQUEST_RECOMMENDATION_TOOL,
   LINKEDIN_PROFILE_PREPARE_WRITE_RECOMMENDATION_TOOL,
   LINKEDIN_PROFILE_PREPARE_UPLOAD_BANNER_TOOL,
+  LINKEDIN_PROFILE_PREPARE_REMOVE_BANNER_TOOL,
   LINKEDIN_PROFILE_PREPARE_UPLOAD_PHOTO_TOOL,
   LINKEDIN_PROFILE_PREPARE_UPDATE_PUBLIC_PROFILE_TOOL,
   LINKEDIN_PROFILE_PREPARE_UPDATE_SETTINGS_TOOL,
@@ -1052,6 +1053,7 @@ export const MCP_TOOL_NAMES = {
     LINKEDIN_PROFILE_PREPARE_UPSERT_SECTION_ITEM_TOOL,
   profilePrepareUploadPhoto: LINKEDIN_PROFILE_PREPARE_UPLOAD_PHOTO_TOOL,
   profilePrepareUploadBanner: LINKEDIN_PROFILE_PREPARE_UPLOAD_BANNER_TOOL,
+  profilePrepareRemoveBanner: LINKEDIN_PROFILE_PREPARE_REMOVE_BANNER_TOOL,
   profilePrepareFeaturedAdd: LINKEDIN_PROFILE_PREPARE_FEATURED_ADD_TOOL,
   profilePrepareFeaturedRemove: LINKEDIN_PROFILE_PREPARE_FEATURED_REMOVE_TOOL,
   profilePrepareFeaturedReorder: LINKEDIN_PROFILE_PREPARE_FEATURED_REORDER_TOOL,

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

@@ -232,6 +232,15 @@ describe("Profile E2E", () => {
     expectRateLimitPreview(prepared.preview, "linkedin.profile.upload_banner");
   }, 60_000);
 
+  it("prepareRemoveBanner returns valid preview", async (context) => {
+    skipIfE2EUnavailable(e2e, context);
+    const runtime = e2e.runtime();
+    const prepared = await runtime.profile.prepareRemoveBanner({});
+
+    expectPreparedAction(prepared);
+    expectRateLimitPreview(prepared.preview, "linkedin.profile.remove_banner");
+  }, 60_000);
+
   it("prepareFeaturedAdd for link returns valid preview", async (context) => {
     skipIfE2EUnavailable(e2e, context);
     const runtime = e2e.runtime();