Skip to content

Fix startup#220

Closed
mor39a wants to merge 1 commit into
runcat-dev:mainfrom
mor39a:fix-startup
Closed

Fix startup#220
mor39a wants to merge 1 commit into
runcat-dev:mainfrom
mor39a:fix-startup

Conversation

@mor39a
Copy link
Copy Markdown
Collaborator

@mor39a mor39a commented Aug 8, 2025

Type of Contribution

  • Bug Fix
  • New Feature
  • Refactoring

Summary of the Proposal

I was investigating issue #216 and discovered it wasn't being logged in the correct place. I've fixed it.

Checklist

  • Code follows proper indentation and naming conventions.
  • Works correctly in both dark theme and light theme.
  • Works correctly on any device.

@Kyome22
Copy link
Copy Markdown
Collaborator

Kyome22 commented Aug 8, 2025

My web research suggests that RegistryKey is case-insensitive. Is there a basis for your proposal?

@mor39a
Copy link
Copy Markdown
Collaborator Author

mor39a commented Aug 8, 2025

I checked, you're right. It seems the error doesn't occur when I run RunCat from the source code, but it does when I run it from the app installed from the Microsoft Store.

@mor39a
Copy link
Copy Markdown
Collaborator Author

mor39a commented Aug 8, 2025

It seems that it is because it is packaged app (MSIX) and this has restrictions that do not allow writing directly to the system registry outside of its isolated area. I will investigate further to see if I find the solution.

@Kyome22
Copy link
Copy Markdown
Collaborator

Kyome22 commented Aug 8, 2025

@mor39a
I've done quite a bit of research, and it seems that when distributing an app on the Microsoft Store, this feature cannot be provided. The app requires permission to insert a setting that auto-starts the app beyond the app's sandbox, and such actions are either blocked by binary validation or, if the app is distributed, the feature won't work. Therefore, unfortunately, we will have to remove this feature.

@Kyome22 Kyome22 closed this Aug 8, 2025
@mor39a
Copy link
Copy Markdown
Collaborator Author

mor39a commented Aug 8, 2025

@Kyome22 That's unfortunate :(
After my last comment I was researching possible solutions, although I haven't managed to make it work yet. I'll let you know if I make any significant progress.

@mor39a mor39a mentioned this pull request Aug 9, 2025
6 tasks
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.

2 participants