From 9b6950373cfaa1a2c0d28287bfadd4a421afb9e9 Mon Sep 17 00:00:00 2001 From: Kaoru0822 <208281322+kaoru0822-kitauji@users.noreply.github.com> Date: Fri, 9 Oct 2026 09:32:24 +0000 Subject: [PATCH] fix(native): name the CLI flag that selects another plist rea inspect-plist takes the target as its positional path and selects a plist with --relative-path, so "Pass path" pointed CLI callers at the wrong argument. Name the MCP path input and the CLI flag. Co-Authored-By: Claude Opus 5.5 --- src/native/NativeMacOSProvider.ts | 2 +- tests/boundary/cli/nativePlistSelectionFailureCli.test.ts | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/native/NativeMacOSProvider.ts b/src/native/NativeMacOSProvider.ts index a0b499f197..cf8099b28c 100644 --- a/src/native/NativeMacOSProvider.ts +++ b/src/native/NativeMacOSProvider.ts @@ -820,7 +820,7 @@ const unreadablePlist = ( ): AnalysisError => { const sentence = reason.endsWith(".") ? reason : `${reason}.`; // Name the default that is missing and the selection that still works. - const missingDefault = `The target has no readable Contents/Info.plist at ${path}: ${sentence} Pass path to inspect another plist.`; + const missingDefault = `The target has no readable Contents/Info.plist at ${path}: ${sentence} Select another plist in the target with the path input (CLI: --relative-path).`; return requested === undefined ? new AnalysisCapabilityUnavailableError( IDENTITY.id, diff --git a/tests/boundary/cli/nativePlistSelectionFailureCli.test.ts b/tests/boundary/cli/nativePlistSelectionFailureCli.test.ts index 642f1011ea..e14ca4cc38 100644 --- a/tests/boundary/cli/nativePlistSelectionFailureCli.test.ts +++ b/tests/boundary/cli/nativePlistSelectionFailureCli.test.ts @@ -113,12 +113,10 @@ describe.skipIf(process.platform !== "darwin")( expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ code: "capability_unavailable", - message: expect.stringContaining( - "Pass path to inspect another plist", - ), + message: expect.stringContaining("(CLI: --relative-path)"), remediation: { action: expect.stringContaining( - "Pass path to inspect another plist", + "Select another plist in the target with the path input (CLI: --relative-path).", ), }, details: {