Skip to content

feat: 起動時に管理者権限を要求する (uacPrivileges: administrator)#17

Merged
jiuya merged 4 commits into
mainfrom
copilot/request-admin-permission-on-startup
Apr 21, 2026
Merged

feat: 起動時に管理者権限を要求する (uacPrivileges: administrator)#17
jiuya merged 4 commits into
mainfrom
copilot/request-admin-permission-on-startup

Conversation

Copilot AI commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

HKLM\...\PersonalizationCSP への書き込みには管理者権限が必要だが、アプリは権限なしで起動を続け、ロックスクリーン変更が毎サイクル失敗していた。

Changes

  • tauri.conf.jsonbundle.windowsuacPrivileges: "administrator" を追加。起動時に常に UAC プロンプトを表示し、管理者権限を保証する
"windows": {
  "uacPrivileges": "administrator"
}
  • lib.rs — 起動時の check_permission() チェックブロックを削除(UAC で常に管理者として動作するため不要)
  • scheduler/mod.rs — 権限エラー専用の Windows 固有ブランチを削除し、エラー処理を log::error! に統一
  • commands/mod.rs — 不要になった AppState.permission_notified: Mutex<bool> フィールドと Mutex インポートを削除

Copilot AI linked an issue Apr 14, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Require administrator permission at startup for core functionality feat: 起動時に管理者権限を要求する (uacPrivileges: administrator) Apr 14, 2026
Copilot AI requested a review from jiuya April 14, 2026 09:27
- build.rs: リリースビルドのみ requireAdministrator マニフェストを埋め込む
  デバッグビルドで適用すると cargo run が os error 740 で失敗するため
- locksun.exe.manifest: requireAdministrator マニフェストファイルを追加
- tauri.conf.json: uacPrivileges: administrator を削除(build.rs で制御)
- renderer/mod.rs: render_to_bytes() 関数を追加
- commands/mod.rs, scheduler/mod.rs: enhance_image の引数を所有権移転に変更
@jiuya jiuya marked this pull request as ready for review April 21, 2026 12:09
@jiuya

jiuya commented Apr 21, 2026

Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts in this pull request

…tation from main

Co-authored-by: jiuya <6884687+jiuya@users.noreply.github.com>

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Resolved. The only conflict was in src-tauri/src/renderer/mod.rs where both branches had added a render_to_bytes function. I kept the main branch version that uses PngEncoder::write_image (commit 8171e88).

@jiuya jiuya merged commit 243898f into main Apr 21, 2026
1 check failed
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