Skip to content

docs(readme): macOS install を --no-quarantine 必須に更新#194

Merged
shomatan merged 1 commit into
mainfrom
docs/readme-no-quarantine
May 13, 2026
Merged

docs(readme): macOS install を --no-quarantine 必須に更新#194
shomatan merged 1 commit into
mainfrom
docs/readme-no-quarantine

Conversation

@shomatan
Copy link
Copy Markdown
Collaborator

@shomatan shomatan commented May 13, 2026

概要

v1.2.0 リリースで .app 配布が開始されたが、未署名配布のため Homebrew Cask 経由で
インストールすると初回起動で「Ark は壊れているため開けません」と表示される。
README の Quick Start に対処手順を明記する。

変更点

macOS セクション

  • 過時 NOTE (.app 配布は実装中 (F1-F7 進行中)) を削除
  • 標準コマンド brew install --cask ignission/tap/ark を主推奨に維持
  • 初回起動で "damaged" 表示が出た場合の対処を別小見出しに分離:
    • 対処 A: xattr -dr com.apple.quarantine (事後削除)
    • 対処 B: brew reinstall --cask --no-quarantine (再インストール時に quarantine 付与 skip)
  • 入手元の真正性確認手順 (sha256 自動照合) を CAUTION 注記で明示
  • 追跡 issue (feat(release): macOS .app の Developer ID 署名 + 公証 (notarization) を有効化 #193) へのリンクを追加
  • macOS 要件を Cask 定義に合わせて >= Monterey (12) に修正 (元: Sonoma 14)

設計判断

codex review で --no-quarantine を主推奨にする初案を 「Gatekeeper 保護を default 推奨してしまう」 と
[high] 指摘されたため、標準フローを通常コマンドに戻し、"damaged" 表示時の暫定回避策として
別小見出しに分離する構成に変更済み。署名対応 (#193) 完了時には本節を簡素化する。

関連

Summary by CodeRabbit

リリースノート

  • ドキュメント
    • macOSインストール手順を更新し、Homebrew Caskサポートを追加しました
    • GitHubリリースページからの直接ダウンロードオプションを追加しました
    • Gatekeeper/Quarantine関連のトラブルシューティング手順を追加し、具体的な解決方法を提供しました

Review Change Stack

v1.2.0 リリースで .app 配布が開始されたため、未署名配布の制約を
README の Quick Start に明示する:

- brew install コマンドに --no-quarantine フラグを追加
- なぜ必要かを IMPORTANT note で説明 (壊れている表示の対処)
- xattr による事後修復もフォールバックとして案内
- 署名対応の追跡 issue (#193) へリンク
- 過時 NOTE (F1-F7 進行中) を削除
- macOS 要件を Cask 定義に合わせて Monterey 以降に修正
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c458a972-dee6-4522-9102-e182b95e50c4

📥 Commits

Reviewing files that changed from the base of the PR and between 9c07124 and ed716d0.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

概要

README.md の macOS Quick Start セクションが更新され、Homebrew Cask インストール手順とシステム要件が追加され、GitHub Releases からの直接ダウンロードリンクが含まれ、Gatekeeper/quarantine の問題と回避方法に関する新しい CAUTION セクションが追加されました。

変更内容

macOS Quick Start 更新

レイヤー / ファイル 概要
Homebrew Cask インストールと Gatekeeper トラブルシューティング
README.md
macOS Quick Start セクションが Homebrew Cask によるインストール手順 (タップ指定とシステム要件 macOS/arm64 を含む) と GitHub Releases からの直接ダウンロードリンクで更新されました。署名されていない .app ビルドの Gatekeeper/quarantine の動作を説明する新しい CAUTION ブロックが追加され、com.apple.quarantine 属性の手動削除、または Homebrew の --no-quarantine フラグを使用した再インストールという 2 つの具体的な回避方法が提供されています。

推定レビュー工数

🎯 2 (Simple) | ⏱️ ~10 minutes

関連の可能性がある問題

  • feat(release): macOS .app の Developer ID 署名 + 公証 (notarization) を有効化 #193: macOS の Gatekeeper/quarantine トラブルシューティングと Homebrew Cask インストール手順の追加に関するメインドキュメント。この PR は提案されたこれらの機能を直接実装しており、Developer ID 署名と公証によるこれらの回避方法の必要性を排除するアプローチと関連しています。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルはマクOS環境でのインストール手順の更新と --no-quarantine フラグの必須化についての変更を正確に反映しており、プルリクエストの主要な目的を明確に要約しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/readme-no-quarantine

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shomatan shomatan merged commit 1010bad into main May 13, 2026
2 checks passed
@shomatan shomatan deleted the docs/readme-no-quarantine branch May 13, 2026 05:55
shomatan added a commit that referenced this pull request May 13, 2026
Homebrew 4.5 で --no-quarantine switch と HOMEBREW_CASK_OPTS=--no-quarantine
の両方が代替なしで廃止された (Homebrew/brew#19046)。
PR #194 で書いた対処 B (--no-quarantine 再インストール) は現行 Homebrew では
'Calling the --[no-]quarantine switch is disabled!' エラーになるため削除。

xattr -dr による事後削除のみが残された正規対処であることを明示し、
NOTE で switch 廃止の事実と参考リンクを補足。
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