fix: handle(_:) がパスの短い URL で Index out of range になる問題を修正

[masaru87]

Studyplus.handle(_:) に、パスの要素数が足りない URL を渡すと Index out of range でクラッシュする箇所がありました。

再現

isAcceptableURL(url:) は pathComponents.isEmpty のみを確認していますが、handle(_:) は pathComponents[1] を、success の場合はさらに pathComponents[2] を読みます。そのため要素数が 1 または 2 の URL が通過してしまいます。

URL	pathComponents	結果
studyplus-{key}://auth-result	[]	弾かれる（問題なし）
studyplus-{key}://auth-result/	["/"]	pathComponents[1] でクラッシュ
studyplus-{key}://auth-result/success	["/", "success"]	pathComponents[2] でクラッシュ
studyplus-{key}://auth-result/success/{token}	["/", "success", "{token}"]	正常

README の案内どおり AppDelegate で受け取った URL をそのまま handle(_:) に渡す使い方だと、カスタム URL スキームは他のアプリからも開けるため、上のような URL を渡されるとアプリが終了してしまいます。

変更内容

• isAcceptableURL(url:) で pathComponents.count < 2 を弾くようにしました（handle(_:) が [1] を読むため）
• case "success" で pathComponents.count >= 3 を確認し、足りない場合は他の受け付けられない URL と同じく studyplusLoginFail(error: .unknownUrl(url)) を呼んで false を返すようにしました

確認したこと

手元に Xcode 環境がないため、handle(_:) と isAcceptableURL(url:) のロジックをそのまま書き写したものを Swift 6.3 (macOS) で実行して確認しました。

• 変更前: 上の表の 2 ケースで Fatal error: Index out of range（SIGTRAP で終了）
• 変更後: 同じ 2 ケースが .unknownUrl で false を返します。success/{token}・fail・cancel の 3 ケースは変更前後で挙動が変わりません
• リポジトリの .swiftlint.yml で swiftlint lint を実行し、新たな警告が出ないことも確認しています

テストターゲットが定義されていないようでしたので、この PR にはテストを追加していません。追加した方がよければ対応します。

[GOTO-TSL]

修正ありがとうございます、記述だけ揃えてもらえれば内容は問題なさそうです

Suggested change

	if url.pathComponents.count < 2 {
	return false
	}
	guard url.pathComponents.count >= 2 {
	return false
	}

[masaru87]

ご確認ありがとうございます。guard に揃えました。

guard は else が必須なので、同ファイルの他の条件と同じ 1 行の guard ... else { return false } の形にしています。