Skip to content

Rename the Shell action type to Open - #67

Merged
danielchalmers merged 1 commit into
mainfrom
claude/shell-actions-naming-control-b79430
Jul 13, 2026
Merged

Rename the Shell action type to Open#67
danielchalmers merged 1 commit into
mainfrom
claude/shell-actions-naming-control-b79430

Conversation

@danielchalmers

Copy link
Copy Markdown
Owner

Summary

Renames the user-facing action type Shell → Open across the UI and the code, with no behavior change.

Why

"Shell" is developer jargon. What the action actually does is open something: an app, file, folder, or URL. The editor's own hint text already describes it that way ("Choose the app, file, folder, URL, or command to open"), and "Open" matches standard Windows vocabulary. This is a naming-clarity change only.

We deliberately did not add a toggle for UseShellExecute. That is a ProcessStartInfo implementation detail, not a user-meaningful choice — turning it off would break URLs, folders, and file-association launches while exposing nothing users need. If a "run a raw command line" scenario ever comes up, it belongs as its own distinct action type, not a boolean named after a .NET property.

What changed

  • ActionType.ShellActionType.Open (enum member).
  • Dropdown label and editor hint text now say Open.
  • Internal identifiers renamed for consistency: CreateShellActionCreateOpenAction, ShellActionDefaultsOpenActionDefaults (file renamed), ApplyShellDefaults/GetShellDefaults/_autoShellDefaultsOpen*, ExecuteShellExecuteOpen, BrowseShellTargetBrowseOpenTarget.
  • Tests and AGENTS.md updated to match.
  • UseShellExecute = true in the launch code is unchanged — that is the .NET API, not the feature name.

Compatibility

Settings serialize Type as an integer (Open is still 2), so existing saved configurations load unchanged. No migration needed.

Testing

  • dotnet build RadialActions.sln
  • dotnet test RadialActions.sln

The action that launches apps, files, folders, and URLs was surfaced to users as 'Shell', which is developer jargon. Rename the user-facing label and all internal identifiers to 'Open', matching the editor's own description and standard Windows vocabulary.

Settings persist Type as an integer (Open is still 2), so existing configurations load unchanged. No behavior change; UseShellExecute launching is untouched.
@danielchalmers
danielchalmers merged commit 7ef98d8 into main Jul 13, 2026
1 check passed
@danielchalmers
danielchalmers deleted the claude/shell-actions-naming-control-b79430 branch July 13, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant