検索クエリで ?. 記法のモジュール関数指定を受け付ける#278
Merged
Merged
Conversation
rurema#250 のフォローアップ(rurema#277 の PR body に記録した件)。 クエリ入力は表示と違い版に紐付かないため、動的 /search(SimpleSearcher)で .# / ?. の両記法を常に受理する。 parse_method_spec_pattern の逆転文字列正規表現は "#."/",." と同じ選択肢に "?" を含んでいなかったため、"Kernel?.open" は "?" がクラス名側に付いて 0 件になっていた。"?" を追加し、tr で従来どおり正準形 .# に正規化する。 Kernel.#open・Kernel#open・Kernel.open・Kernel::open・裸の ?.open の 挙動は回帰テストで確認済み。 静的検索(search_ranker.js)側は Aliki テーマからの verbatim vendor 方針の ため本 PR では変更しない。調査の過程で、静的検索では修飾付き検索 (File.open や Kernel.#open など。モジュール関数に限らない)がクエリの 一律 "."→"::" 書き換えとインデックスの full_name 形式の不一致により 従来から 0 件になることが判明しており、vendored 方針下での対応方針も 含めて別 issue で扱う。 refs rurema#250 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
#250 のフォローアップ(#277 の PR body に記録した件): 動的
/search(SimpleSearcher)で?.記法のモジュール関数指定を受け付けます(refs #250)。クエリ入力は表示と違い版に紐付かないため、.#/?.の両記法を常に受理する方針です。parse_method_spec_patternの逆転文字列正規表現に?が無く、Kernel?.openは?がクラス名側に付いて 0 件でした。#./,.と同じ選択肢に?を追加し、従来どおり正準形.#へ正規化しますKernel.#open・Kernel#open・Kernel.open・Kernel::open・裸の?.openは挙動不変静的検索(search_ranker.js)を触らない理由
search_ranker.jsは Aliki テーマからの verbatim vendor 方針のため、本 PR では変更しません。なお調査の過程で、静的検索では修飾付き検索(File.openやKernel.#openなど。モジュール関数に限らない)が従来から 0 件になること(クエリの一律.→::書き換えとインデックスのfull_name形式の不一致)が判明したため、vendored 方針下での対応方針も含めて別 issue として起票します。検証
?.・インスタンスメソッド無影響)= 17788 tests / 0 failures・rbs validate・steep checkクリーン・rake test:js(vendored 挙動のベースライン)も greenDummy?.mf/Dummy.#mfとも同一ヒットrefs #250
🤖 Generated with Claude Code