コミットを比較
9 コミット
| 作成者 | SHA1 | 日付 | |
|---|---|---|---|
| 245a712827 | |||
| f3d3f2d301 | |||
| 7ab46f907f | |||
| 791b52e894 | |||
| 6750692445 | |||
| 01b063f473 | |||
| c06d73fc6c | |||
| 3b5ad3b805 | |||
| e94720941c |
@@ -158,6 +158,13 @@ npm run preview
|
|||||||
- Keep page-level code under `frontend/src/pages` and shared UI/feature code
|
- Keep page-level code under `frontend/src/pages` and shared UI/feature code
|
||||||
under `frontend/src/components` unless existing patterns point elsewhere.
|
under `frontend/src/components` unless existing patterns point elsewhere.
|
||||||
- Match existing Tailwind, component, and import alias conventions.
|
- Match existing Tailwind, component, and import alias conventions.
|
||||||
|
- In TypeScript and TSX, prefer direct comparison operators such as `===` and
|
||||||
|
`!==` over negating a comparison like `!(a === b)`.
|
||||||
|
- In TypeScript and TSX, prefer `++i` or `--i` over `i += 1` or `i -= 1` for
|
||||||
|
simple unit-step counter updates.
|
||||||
|
- For user-facing Japanese text, prefer modern kana usage and natural current
|
||||||
|
phrasing over historical spellings or awkward literal wording.
|
||||||
|
- For user-facing Japanese ellipses, prefer `……` over ASCII `...`.
|
||||||
|
|
||||||
### Frontend TSX style
|
### Frontend TSX style
|
||||||
|
|
||||||
@@ -179,6 +186,9 @@ npm run preview
|
|||||||
single physical line.
|
single physical line.
|
||||||
- Always add braces around `if`, `else`, or `for` bodies when the body spans
|
- Always add braces around `if`, `else`, or `for` bodies when the body spans
|
||||||
two or more physical lines, even if it is one statement.
|
two or more physical lines, even if it is one statement.
|
||||||
|
- Do not use a leading semicolon for expression statements such as
|
||||||
|
`;([...]).forEach(...)`; rewrite the expression to avoid ASI hazards
|
||||||
|
explicitly, for example with `void`.
|
||||||
|
|
||||||
Preferred:
|
Preferred:
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
class GekanatorGamesController < ApplicationController
|
class GekanatorGamesController < ApplicationController
|
||||||
def create
|
def create
|
||||||
return head :not_found unless current_user&.admin?
|
return head :unauthorized unless current_user
|
||||||
|
|
||||||
guessed_post_id = params.require(:guessed_post_id)
|
guessed_post_id = params.require(:guessed_post_id)
|
||||||
correct_post_id = params[:correct_post_id].presence
|
correct_post_id = params[:correct_post_id].presence
|
||||||
@@ -22,10 +22,8 @@ class GekanatorGamesController < ApplicationController
|
|||||||
end
|
end
|
||||||
|
|
||||||
def extra_questions
|
def extra_questions
|
||||||
return head :not_found unless current_user&.admin?
|
game = find_owned_game
|
||||||
|
return if performed?
|
||||||
game = GekanatorGame.find_by(id: params[:id])
|
|
||||||
return head :not_found unless game
|
|
||||||
|
|
||||||
questions =
|
questions =
|
||||||
GekanatorQuestion
|
GekanatorQuestion
|
||||||
@@ -45,10 +43,8 @@ class GekanatorGamesController < ApplicationController
|
|||||||
end
|
end
|
||||||
|
|
||||||
def extra_question_answers
|
def extra_question_answers
|
||||||
return head :not_found unless current_user&.admin?
|
game = find_owned_game
|
||||||
|
return if performed?
|
||||||
game = GekanatorGame.find_by(id: params[:id])
|
|
||||||
return head :not_found unless game
|
|
||||||
|
|
||||||
answer_params = params.require(:answers)
|
answer_params = params.require(:answers)
|
||||||
if !answer_params.is_a?(Array)
|
if !answer_params.is_a?(Array)
|
||||||
@@ -137,4 +133,16 @@ class GekanatorGamesController < ApplicationController
|
|||||||
|
|
||||||
question.priority_weight.to_f / (1.0 + sample_count * 0.15)
|
question.priority_weight.to_f / (1.0 + sample_count * 0.15)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def find_owned_game
|
||||||
|
return head :unauthorized unless current_user
|
||||||
|
|
||||||
|
game = GekanatorGame.find_by(id: params[:id])
|
||||||
|
return head :not_found unless game
|
||||||
|
if !current_user.admin? && game.user_id != current_user.id
|
||||||
|
return head :not_found
|
||||||
|
end
|
||||||
|
|
||||||
|
game
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -1,7 +1,5 @@
|
|||||||
class GekanatorPostsController < ApplicationController
|
class GekanatorPostsController < ApplicationController
|
||||||
def index
|
def index
|
||||||
return head :not_found unless current_user&.admin?
|
|
||||||
|
|
||||||
posts =
|
posts =
|
||||||
Post
|
Post
|
||||||
.preload(tags: :tag_name)
|
.preload(tags: :tag_name)
|
||||||
|
|||||||
@@ -1,9 +1,12 @@
|
|||||||
class GekanatorQuestionSuggestionsController < ApplicationController
|
class GekanatorQuestionSuggestionsController < ApplicationController
|
||||||
def create
|
def create
|
||||||
return head :not_found unless current_user&.admin?
|
return head :unauthorized unless current_user
|
||||||
|
|
||||||
game = GekanatorGame.find_by(id: params.require(:gekanator_game_id))
|
game = GekanatorGame.find_by(id: params.require(:gekanator_game_id))
|
||||||
return head :not_found unless game
|
return head :not_found unless game
|
||||||
|
if !current_user.admin? && game.user_id != current_user.id
|
||||||
|
return head :not_found
|
||||||
|
end
|
||||||
|
|
||||||
suggestion = GekanatorQuestionSuggestion.new(
|
suggestion = GekanatorQuestionSuggestion.new(
|
||||||
gekanator_game: game,
|
gekanator_game: game,
|
||||||
|
|||||||
@@ -1,7 +1,5 @@
|
|||||||
class GekanatorQuestionsController < ApplicationController
|
class GekanatorQuestionsController < ApplicationController
|
||||||
def index
|
def index
|
||||||
return head :not_found unless current_user&.admin?
|
|
||||||
|
|
||||||
questions =
|
questions =
|
||||||
GekanatorQuestion
|
GekanatorQuestion
|
||||||
.accepted
|
.accepted
|
||||||
@@ -49,6 +47,8 @@ class GekanatorQuestionsController < ApplicationController
|
|||||||
"title:length-at-least:#{ condition[:length].to_i + 1 }"
|
"title:length-at-least:#{ condition[:length].to_i + 1 }"
|
||||||
when 'title-has-ascii'
|
when 'title-has-ascii'
|
||||||
'title:ascii'
|
'title:ascii'
|
||||||
|
when 'title-contains'
|
||||||
|
"title:contains:#{ condition[:text] }"
|
||||||
when 'post-similarity'
|
when 'post-similarity'
|
||||||
"post-similarity:#{ question.id }"
|
"post-similarity:#{ question.id }"
|
||||||
else
|
else
|
||||||
@@ -77,6 +77,8 @@ class GekanatorQuestionsController < ApplicationController
|
|||||||
case condition[:type]
|
case condition[:type]
|
||||||
when 'title-length-at-least'
|
when 'title-length-at-least'
|
||||||
"タイトルは #{ condition[:length] } 文字以上?"
|
"タイトルは #{ condition[:length] } 文字以上?"
|
||||||
|
when 'title-contains'
|
||||||
|
"題名に「#{ condition[:text] }」が含まれる?"
|
||||||
else
|
else
|
||||||
question.text
|
question.text
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -1,5 +1,15 @@
|
|||||||
module Gekanator
|
module Gekanator
|
||||||
class QuestionSuggestionAiConverter
|
class QuestionSuggestionAiConverter
|
||||||
|
# Temporary heuristic converter for #361.
|
||||||
|
# This creates pending ai_generated questions without external LLM calls;
|
||||||
|
# accepted questions are still distributed only after admin approval.
|
||||||
|
TITLE_LENGTH_RE = /\Aタイトルは\s*(\d+)\s*文字以上[??]\z/
|
||||||
|
ORIGINAL_YEAR_RE = /\Aオリジナルの投稿年は\s*(\d{4})\s*年[??]\z/
|
||||||
|
ORIGINAL_MONTH_RE = /\Aオリジナルの投稿月は\s*(\d{1,2})\s*月[??]\z/
|
||||||
|
ORIGINAL_MONTH_DAY_RE = /\Aオリジナルの投稿日は\s*(\d{1,2})\s*月\s*(\d{1,2})\s*日[??]\z/
|
||||||
|
TITLE_CONTAINS_RE = /\A題名に「(.+?)」が含まれる[??]\z/
|
||||||
|
SOURCE_RE = /\A(.+?)\s+の投稿を思[ひい]浮かべて[ゐい]る[??]\z/
|
||||||
|
|
||||||
def self.call(...) = new(...).call
|
def self.call(...) = new(...).call
|
||||||
|
|
||||||
def initialize suggestion:, user:
|
def initialize suggestion:, user:
|
||||||
@@ -8,11 +18,151 @@ module Gekanator
|
|||||||
end
|
end
|
||||||
|
|
||||||
def call
|
def call
|
||||||
raise NotImplementedError, 'AI question conversion is not implemented yet.'
|
suggestion.with_lock do
|
||||||
|
existing = existing_generated_question
|
||||||
|
return existing if existing
|
||||||
|
|
||||||
|
run = suggestion.gekanator_ai_runs.create!(
|
||||||
|
model: 'heuristic_converter_v1',
|
||||||
|
status: 'running',
|
||||||
|
input_tokens: 0,
|
||||||
|
output_tokens: 0,
|
||||||
|
estimated_cost_jpy: 0)
|
||||||
|
|
||||||
|
question_attributes = build_question
|
||||||
|
question =
|
||||||
|
question_attributes &&
|
||||||
|
GekanatorQuestion.create!(
|
||||||
|
**question_attributes,
|
||||||
|
source: 'ai_generated',
|
||||||
|
status: 'pending',
|
||||||
|
gekanator_question_suggestion: suggestion,
|
||||||
|
created_by: user)
|
||||||
|
|
||||||
|
run.update!(status: question ? 'succeeded' : 'failed')
|
||||||
|
question
|
||||||
|
end
|
||||||
|
rescue => error
|
||||||
|
run&.update!(status: 'failed') if run&.persisted? && run.status != 'failed'
|
||||||
|
raise error
|
||||||
end
|
end
|
||||||
|
|
||||||
private
|
private
|
||||||
|
|
||||||
attr_reader :suggestion, :user
|
attr_reader :suggestion, :user
|
||||||
|
|
||||||
|
def existing_generated_question
|
||||||
|
suggestion
|
||||||
|
.gekanator_questions
|
||||||
|
.where(source: 'ai_generated')
|
||||||
|
.order(id: :desc)
|
||||||
|
.first
|
||||||
|
end
|
||||||
|
|
||||||
|
def build_question
|
||||||
|
text = normalized_text
|
||||||
|
return nil if text.blank?
|
||||||
|
|
||||||
|
structured_question_for(text) || post_similarity_question_for(text)
|
||||||
|
end
|
||||||
|
|
||||||
|
def normalized_text
|
||||||
|
suggestion.question_text.to_s.gsub(/[[:space:]]+/, ' ').strip
|
||||||
|
end
|
||||||
|
|
||||||
|
def structured_question_for text
|
||||||
|
case text
|
||||||
|
when TITLE_LENGTH_RE
|
||||||
|
length = Regexp.last_match(1).to_i
|
||||||
|
return nil if length <= 0
|
||||||
|
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'title',
|
||||||
|
condition: {
|
||||||
|
type: 'title-length-at-least',
|
||||||
|
length:
|
||||||
|
},
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when /\A題名に英数字が混じって[ゐい]る[??]\z/
|
||||||
|
{
|
||||||
|
text: '題名に英数字が混じってゐる?',
|
||||||
|
kind: 'title',
|
||||||
|
condition: { type: 'title-has-ascii' },
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when ORIGINAL_YEAR_RE
|
||||||
|
year = Regexp.last_match(1).to_i
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'original_date',
|
||||||
|
condition: { type: 'original-year', year: },
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when ORIGINAL_MONTH_RE
|
||||||
|
month = Regexp.last_match(1).to_i
|
||||||
|
return nil unless month.between?(1, 12)
|
||||||
|
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'original_date',
|
||||||
|
condition: { type: 'original-month', month: },
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when ORIGINAL_MONTH_DAY_RE
|
||||||
|
month = Regexp.last_match(1).to_i
|
||||||
|
day = Regexp.last_match(2).to_i
|
||||||
|
return nil unless month.between?(1, 12) && day.between?(1, 31)
|
||||||
|
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'original_date',
|
||||||
|
condition: {
|
||||||
|
type: 'original-month-day',
|
||||||
|
monthDay: "#{ month }-#{ day }"
|
||||||
|
},
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when TITLE_CONTAINS_RE
|
||||||
|
title_text = Regexp.last_match(1).to_s.strip
|
||||||
|
return nil if title_text.blank?
|
||||||
|
|
||||||
|
{
|
||||||
|
text: "題名に「#{ title_text }」が含まれる?",
|
||||||
|
kind: 'title',
|
||||||
|
condition: { type: 'title-contains', text: title_text },
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
when SOURCE_RE
|
||||||
|
host = Regexp.last_match(1).to_s.strip
|
||||||
|
return nil if host.blank?
|
||||||
|
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'source',
|
||||||
|
condition: { type: 'source', host: },
|
||||||
|
priority_weight: 0.95
|
||||||
|
}
|
||||||
|
else
|
||||||
|
nil
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
def post_similarity_question_for text
|
||||||
|
return nil if suggestion.answer == 'unknown'
|
||||||
|
|
||||||
|
{
|
||||||
|
text:,
|
||||||
|
kind: 'post_similarity',
|
||||||
|
condition: {
|
||||||
|
type: 'post-similarity',
|
||||||
|
postId: suggestion.gekanator_game.correct_post_id,
|
||||||
|
answer: suggestion.answer,
|
||||||
|
threshold: 0.65
|
||||||
|
},
|
||||||
|
priority_weight: 1.0
|
||||||
|
}
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -52,16 +52,16 @@ RSpec.describe 'Gekanator games API', type: :request do
|
|||||||
expect(response).to have_http_status(:unprocessable_entity)
|
expect(response).to have_http_status(:unprocessable_entity)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'returns not found without an admin user' do
|
it 'returns unauthorized without a user' do
|
||||||
post '/gekanator/games', params: {
|
post '/gekanator/games', params: {
|
||||||
guessed_post_id: guessed_post.id,
|
guessed_post_id: guessed_post.id,
|
||||||
correct_post_id: guessed_post.id,
|
correct_post_id: guessed_post.id,
|
||||||
answers: [] }
|
answers: [] }
|
||||||
|
|
||||||
expect(response).to have_http_status(:not_found)
|
expect(response).to have_http_status(:unauthorized)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'returns not found for a non-admin user' do
|
it 'stores a game for a non-admin user' do
|
||||||
sign_in_as user
|
sign_in_as user
|
||||||
|
|
||||||
post '/gekanator/games', params: {
|
post '/gekanator/games', params: {
|
||||||
@@ -69,7 +69,8 @@ RSpec.describe 'Gekanator games API', type: :request do
|
|||||||
correct_post_id: guessed_post.id,
|
correct_post_id: guessed_post.id,
|
||||||
answers: [{ question_id: 'tag:1', answer: 'yes' }] }
|
answers: [{ question_id: 'tag:1', answer: 'yes' }] }
|
||||||
|
|
||||||
expect(response).to have_http_status(:not_found)
|
expect(response).to have_http_status(:created)
|
||||||
|
expect(GekanatorGame.find(json['id']).user).to eq(user)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -129,7 +129,7 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
expect(response).to have_http_status(:unprocessable_entity)
|
expect(response).to have_http_status(:unprocessable_entity)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'returns not found for a non-admin user' do
|
it 'stores a game result for a non-admin user' do
|
||||||
sign_in_as member
|
sign_in_as member
|
||||||
|
|
||||||
post '/gekanator/games', params: {
|
post '/gekanator/games', params: {
|
||||||
@@ -138,7 +138,18 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
answers: []
|
answers: []
|
||||||
}
|
}
|
||||||
|
|
||||||
expect(response).to have_http_status(:not_found)
|
expect(response).to have_http_status(:created)
|
||||||
|
expect(GekanatorGame.find(json['id']).user).to eq(member)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns unauthorized without a user' do
|
||||||
|
post '/gekanator/games', params: {
|
||||||
|
guessed_post_id: guessed_post.id,
|
||||||
|
correct_post_id: correct_post.id,
|
||||||
|
answers: []
|
||||||
|
}
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:unauthorized)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@@ -261,17 +272,57 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
expect(response).to have_http_status(:unprocessable_entity)
|
expect(response).to have_http_status(:unprocessable_entity)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'returns not found for a non-admin user' do
|
it 'allows a non-admin user to suggest a question for their own game' do
|
||||||
|
member_game = GekanatorGame.create!(
|
||||||
|
user: member,
|
||||||
|
guessed_post: guessed_post,
|
||||||
|
correct_post: correct_post,
|
||||||
|
won: false,
|
||||||
|
question_count: 1,
|
||||||
|
answers: [{ 'question_id' => 'tag:1', 'answer' => 'yes' }]
|
||||||
|
)
|
||||||
sign_in_as member
|
sign_in_as member
|
||||||
|
|
||||||
|
expect {
|
||||||
|
post '/gekanator/question_suggestions', params: {
|
||||||
|
gekanator_game_id: member_game.id,
|
||||||
|
question_text: 'member question?',
|
||||||
|
answer: 'yes'
|
||||||
|
}
|
||||||
|
}.to change { GekanatorQuestionSuggestion.count }.by(1)
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:created)
|
||||||
|
expect(GekanatorQuestionSuggestion.last).to have_attributes(
|
||||||
|
gekanator_game_id: member_game.id,
|
||||||
|
user_id: member.id
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns not found for another user game' do
|
||||||
|
sign_in_as member
|
||||||
|
|
||||||
|
expect {
|
||||||
post '/gekanator/question_suggestions', params: {
|
post '/gekanator/question_suggestions', params: {
|
||||||
gekanator_game_id: game.id,
|
gekanator_game_id: game.id,
|
||||||
question_text: 'member question?',
|
question_text: 'member question?',
|
||||||
answer: 'yes'
|
answer: 'yes'
|
||||||
}
|
}
|
||||||
|
}.not_to change { GekanatorQuestionSuggestion.count }
|
||||||
|
|
||||||
expect(response).to have_http_status(:not_found)
|
expect(response).to have_http_status(:not_found)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it 'returns unauthorized without a user' do
|
||||||
|
expect {
|
||||||
|
post '/gekanator/question_suggestions', params: {
|
||||||
|
gekanator_game_id: game.id,
|
||||||
|
question_text: 'member question?',
|
||||||
|
answer: 'yes'
|
||||||
|
}
|
||||||
|
}.not_to change { GekanatorQuestionSuggestion.count }
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:unauthorized)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe 'GET /gekanator/games/:id/extra_questions' do
|
describe 'GET /gekanator/games/:id/extra_questions' do
|
||||||
@@ -377,6 +428,38 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
expect(response).to have_http_status(:ok)
|
expect(response).to have_http_status(:ok)
|
||||||
expect(json['questions'].map { _1['id'] }).to contain_exactly(accepted.id)
|
expect(json['questions'].map { _1['id'] }).to contain_exactly(accepted.id)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it 'allows a non-admin user to fetch extra questions for their own game' do
|
||||||
|
member_game = GekanatorGame.create!(
|
||||||
|
user: member,
|
||||||
|
guessed_post: guessed_post,
|
||||||
|
correct_post: correct_post,
|
||||||
|
won: false,
|
||||||
|
question_count: 1,
|
||||||
|
answers: [{ 'question_id' => 'tag:1', 'answer' => 'yes' }]
|
||||||
|
)
|
||||||
|
accepted = create_post_similarity_question!(text: 'accepted?')
|
||||||
|
sign_in_as member
|
||||||
|
|
||||||
|
get "/gekanator/games/#{member_game.id}/extra_questions"
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:ok)
|
||||||
|
expect(json['questions'].map { _1['id'] }).to include(accepted.id)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns not found for another user game' do
|
||||||
|
sign_in_as member
|
||||||
|
|
||||||
|
get "/gekanator/games/#{game.id}/extra_questions"
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:not_found)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns unauthorized without a user' do
|
||||||
|
get "/gekanator/games/#{game.id}/extra_questions"
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:unauthorized)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe 'POST /gekanator/games/:id/extra_question_answers' do
|
describe 'POST /gekanator/games/:id/extra_question_answers' do
|
||||||
@@ -503,6 +586,69 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
|
|
||||||
expect(response).to have_http_status(:unprocessable_entity)
|
expect(response).to have_http_status(:unprocessable_entity)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it 'allows a non-admin user to answer extra questions for their own game' do
|
||||||
|
member_game = GekanatorGame.create!(
|
||||||
|
user: member,
|
||||||
|
guessed_post: guessed_post,
|
||||||
|
correct_post: correct_post,
|
||||||
|
won: false,
|
||||||
|
question_count: 1,
|
||||||
|
answers: [{ 'question_id' => 'tag:1', 'answer' => 'yes' }]
|
||||||
|
)
|
||||||
|
question = create_post_similarity_question!(text: 'extra?')
|
||||||
|
sign_in_as member
|
||||||
|
|
||||||
|
expect {
|
||||||
|
post "/gekanator/games/#{member_game.id}/extra_question_answers", params: {
|
||||||
|
answers: [
|
||||||
|
{
|
||||||
|
question_id: question.id,
|
||||||
|
answer: 'yes'
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}.to change { GekanatorQuestionExample.count }.by(1)
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:created)
|
||||||
|
expect(GekanatorQuestionExample.last).to have_attributes(
|
||||||
|
user_id: member.id,
|
||||||
|
gekanator_game_id: member_game.id
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns not found for another user game' do
|
||||||
|
question = create_post_similarity_question!(text: 'extra?')
|
||||||
|
sign_in_as member
|
||||||
|
|
||||||
|
expect {
|
||||||
|
post "/gekanator/games/#{game.id}/extra_question_answers", params: {
|
||||||
|
answers: [
|
||||||
|
{
|
||||||
|
question_id: question.id,
|
||||||
|
answer: 'yes'
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}.not_to change { GekanatorQuestionExample.count }
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:not_found)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns unauthorized without a user' do
|
||||||
|
question = create_post_similarity_question!(text: 'extra?')
|
||||||
|
|
||||||
|
post "/gekanator/games/#{game.id}/extra_question_answers", params: {
|
||||||
|
answers: [
|
||||||
|
{
|
||||||
|
question_id: question.id,
|
||||||
|
answer: 'yes'
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:unauthorized)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe 'GET /gekanator/questions' do
|
describe 'GET /gekanator/questions' do
|
||||||
@@ -608,5 +754,33 @@ RSpec.describe 'Gekanator learning API', type: :request do
|
|||||||
'length' => 21
|
'length' => 21
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it 'returns title-contains questions without authentication' do
|
||||||
|
GekanatorQuestion.create!(
|
||||||
|
text: '題名に「結束バンド」が含まれる?',
|
||||||
|
kind: 'title',
|
||||||
|
source: 'ai_generated',
|
||||||
|
status: 'accepted',
|
||||||
|
priority_weight: 0.95,
|
||||||
|
condition: {
|
||||||
|
type: 'title-contains',
|
||||||
|
text: '結束バンド'
|
||||||
|
},
|
||||||
|
created_by: admin
|
||||||
|
)
|
||||||
|
|
||||||
|
get '/gekanator/questions'
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:ok)
|
||||||
|
question_json = json['questions'].find { _1['id'] == 'title:contains:結束バンド' }
|
||||||
|
expect(question_json).to include(
|
||||||
|
'text' => '題名に「結束バンド」が含まれる?',
|
||||||
|
'kind' => 'title'
|
||||||
|
)
|
||||||
|
expect(question_json['condition']).to include(
|
||||||
|
'type' => 'title-contains',
|
||||||
|
'text' => '結束バンド'
|
||||||
|
)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -0,0 +1,112 @@
|
|||||||
|
require 'rails_helper'
|
||||||
|
|
||||||
|
RSpec.describe Gekanator::QuestionSuggestionAiConverter do
|
||||||
|
let(:user) { create(:user, :member) }
|
||||||
|
let(:guessed_post) { Post.create!(title: 'guess', url: 'https://example.com/guess') }
|
||||||
|
let(:correct_post) { Post.create!(title: 'correct', url: 'https://example.com/correct') }
|
||||||
|
let(:game) do
|
||||||
|
GekanatorGame.create!(
|
||||||
|
user: user,
|
||||||
|
guessed_post: guessed_post,
|
||||||
|
correct_post: correct_post,
|
||||||
|
won: false,
|
||||||
|
question_count: 1,
|
||||||
|
answers: [{ 'question_id' => 'tag:1', 'answer' => 'yes' }]
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
def create_suggestion!(question_text:, answer: 'yes')
|
||||||
|
GekanatorQuestionSuggestion.create!(
|
||||||
|
gekanator_game: game,
|
||||||
|
user: user,
|
||||||
|
question_text: question_text,
|
||||||
|
answer: answer
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'converts title-contains suggestions to pending ai-generated questions' do
|
||||||
|
suggestion = create_suggestion!(question_text: '題名に「結束バンド」が含まれる?')
|
||||||
|
|
||||||
|
expect {
|
||||||
|
described_class.call(suggestion: suggestion, user: user)
|
||||||
|
}.to change { GekanatorQuestion.count }.by(1)
|
||||||
|
.and change { GekanatorAiRun.count }.by(1)
|
||||||
|
|
||||||
|
question = GekanatorQuestion.last
|
||||||
|
expect(question).to have_attributes(
|
||||||
|
text: '題名に「結束バンド」が含まれる?',
|
||||||
|
kind: 'title',
|
||||||
|
source: 'ai_generated',
|
||||||
|
status: 'pending',
|
||||||
|
priority_weight: 0.95,
|
||||||
|
gekanator_question_suggestion_id: suggestion.id,
|
||||||
|
created_by_id: user.id
|
||||||
|
)
|
||||||
|
expect(question.condition).to include(
|
||||||
|
'type' => 'title-contains',
|
||||||
|
'text' => '結束バンド'
|
||||||
|
)
|
||||||
|
expect(GekanatorAiRun.last).to have_attributes(
|
||||||
|
gekanator_question_suggestion_id: suggestion.id,
|
||||||
|
model: 'heuristic_converter_v1',
|
||||||
|
status: 'succeeded'
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'converts concrete non-unknown suggestions to post-similarity questions' do
|
||||||
|
suggestion = create_suggestion!(
|
||||||
|
question_text: '喜多ちゃんが泣いてる?',
|
||||||
|
answer: 'partial'
|
||||||
|
)
|
||||||
|
|
||||||
|
question = described_class.call(suggestion: suggestion, user: user)
|
||||||
|
|
||||||
|
expect(question).to have_attributes(
|
||||||
|
text: '喜多ちゃんが泣いてる?',
|
||||||
|
kind: 'post_similarity',
|
||||||
|
source: 'ai_generated',
|
||||||
|
status: 'pending',
|
||||||
|
priority_weight: 1.0
|
||||||
|
)
|
||||||
|
expect(question.condition).to include(
|
||||||
|
'type' => 'post-similarity',
|
||||||
|
'postId' => correct_post.id,
|
||||||
|
'answer' => 'partial',
|
||||||
|
'threshold' => 0.65
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'records a failed run when the suggestion cannot be converted' do
|
||||||
|
suggestion = create_suggestion!(
|
||||||
|
question_text: 'よく分からない質問?',
|
||||||
|
answer: 'unknown'
|
||||||
|
)
|
||||||
|
|
||||||
|
expect {
|
||||||
|
expect(described_class.call(suggestion: suggestion, user: user)).to be_nil
|
||||||
|
}.not_to change { GekanatorQuestion.count }
|
||||||
|
|
||||||
|
expect(GekanatorAiRun.last).to have_attributes(
|
||||||
|
gekanator_question_suggestion_id: suggestion.id,
|
||||||
|
status: 'failed'
|
||||||
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns an existing generated question without creating a duplicate run' do
|
||||||
|
suggestion = create_suggestion!(question_text: 'タイトルは 10 文字以上?')
|
||||||
|
existing = GekanatorQuestion.create!(
|
||||||
|
text: 'タイトルは 10 文字以上?',
|
||||||
|
kind: 'title',
|
||||||
|
source: 'ai_generated',
|
||||||
|
status: 'pending',
|
||||||
|
priority_weight: 0.95,
|
||||||
|
condition: { type: 'title-length-at-least', length: 10 },
|
||||||
|
gekanator_question_suggestion: suggestion,
|
||||||
|
created_by: user
|
||||||
|
)
|
||||||
|
|
||||||
|
expect {
|
||||||
|
expect(described_class.call(suggestion: suggestion, user: user)).to eq(existing)
|
||||||
|
}.not_to change { GekanatorAiRun.count }
|
||||||
|
end
|
||||||
|
end
|
||||||
|
変更後 幅: | 高さ: | サイズ: 559 KiB |
|
変更後 幅: | 高さ: | サイズ: 146 KiB |
|
変更後 幅: | 高さ: | サイズ: 1.2 MiB |
|
変更後 幅: | 高さ: | サイズ: 188 KiB |
|
変更後 幅: | 高さ: | サイズ: 201 KiB |
|
変更後 幅: | 高さ: | サイズ: 196 KiB |
|
変更後 幅: | 高さ: | サイズ: 179 KiB |
|
変更後 幅: | 高さ: | サイズ: 559 KiB |
|
変更後 幅: | 高さ: | サイズ: 146 KiB |
|
変更後 幅: | 高さ: | サイズ: 1.2 MiB |
|
変更後 幅: | 高さ: | サイズ: 188 KiB |
|
変更後 幅: | 高さ: | サイズ: 201 KiB |
|
変更後 幅: | 高さ: | サイズ: 196 KiB |
|
変更後 幅: | 高さ: | サイズ: 179 KiB |
@@ -40,7 +40,7 @@ import WikiHistoryPage from '@/pages/wiki/WikiHistoryPage'
|
|||||||
import WikiNewPage from '@/pages/wiki/WikiNewPage'
|
import WikiNewPage from '@/pages/wiki/WikiNewPage'
|
||||||
import WikiSearchPage from '@/pages/wiki/WikiSearchPage'
|
import WikiSearchPage from '@/pages/wiki/WikiSearchPage'
|
||||||
|
|
||||||
import type { Dispatch, FC, ReactNode, SetStateAction } from 'react'
|
import type { Dispatch, FC, SetStateAction } from 'react'
|
||||||
|
|
||||||
import type { User } from '@/types'
|
import type { User } from '@/types'
|
||||||
|
|
||||||
@@ -81,10 +81,7 @@ const RouteTransitionWrapper = ({ user, setUser }: {
|
|||||||
<Route path="/users/settings" element={<SettingPage user={user} setUser={setUser}/>}/>
|
<Route path="/users/settings" element={<SettingPage user={user} setUser={setUser}/>}/>
|
||||||
<Route path="/settings" element={<Navigate to="/users/settings" replace/>}/>
|
<Route path="/settings" element={<Navigate to="/users/settings" replace/>}/>
|
||||||
<Route path="/tos" element={<TOSPage/>}/>
|
<Route path="/tos" element={<TOSPage/>}/>
|
||||||
<Route path="/gekanator" element={
|
<Route path="/gekanator" element={<GekanatorPage user={user}/>}/>
|
||||||
<AdminOnly user={user}>
|
|
||||||
<GekanatorPage/>
|
|
||||||
</AdminOnly>}/>
|
|
||||||
<Route path="/more" element={<MorePage/>}/>
|
<Route path="/more" element={<MorePage/>}/>
|
||||||
<Route path="*" element={<NotFound/>}/>
|
<Route path="*" element={<NotFound/>}/>
|
||||||
</Routes>
|
</Routes>
|
||||||
@@ -92,16 +89,6 @@ const RouteTransitionWrapper = ({ user, setUser }: {
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
const AdminOnly = ({ user, children }: {
|
|
||||||
user: User | null
|
|
||||||
children: ReactNode }) => {
|
|
||||||
if (user?.role !== 'admin')
|
|
||||||
return <NotFound/>
|
|
||||||
|
|
||||||
return <>{children}</>
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
const PostDetailRoute = ({ user }: { user: User | null }) => {
|
const PostDetailRoute = ({ user }: { user: User | null }) => {
|
||||||
const location = useLocation ()
|
const location = useLocation ()
|
||||||
const key = location.pathname
|
const key = location.pathname
|
||||||
|
|||||||
@@ -66,7 +66,8 @@ export const menuOutline = ({ tag, wikiId, user, pathName }: {
|
|||||||
{ name: '履歴', to: `/wiki/changes?id=${ wikiId }`, visible: wikiPageFlg },
|
{ name: '履歴', to: `/wiki/changes?id=${ wikiId }`, visible: wikiPageFlg },
|
||||||
{ name: '編輯', to: `/wiki/${ wikiId || wikiTitle }/edit`, visible: wikiPageFlg }] },
|
{ name: '編輯', to: `/wiki/${ wikiId || wikiTitle }/edit`, visible: wikiPageFlg }] },
|
||||||
{ name: 'おたのしみ', visible: false, subMenu: [
|
{ name: 'おたのしみ', visible: false, subMenu: [
|
||||||
{ name: '上映会 (β)', to: '/theatres/1' }] },
|
{ name: '上映会 (β)', to: '/theatres/1' },
|
||||||
|
{ name: 'グカネータ (β)', to: '/gekanator' }] },
|
||||||
{ name: 'ユーザ', to: '/users/settings', visible: false, subMenu: [
|
{ name: 'ユーザ', to: '/users/settings', visible: false, subMenu: [
|
||||||
{ name: '一覧', to: '/users', visible: false },
|
{ name: '一覧', to: '/users', visible: false },
|
||||||
{ name: 'お前', to: `/users/${ user?.id }`, visible: false },
|
{ name: 'お前', to: `/users/${ user?.id }`, visible: false },
|
||||||
|
|||||||
@@ -4,6 +4,7 @@ import { apiPost } from '@/lib/api'
|
|||||||
import {
|
import {
|
||||||
buildGekanatorQuestions,
|
buildGekanatorQuestions,
|
||||||
expectedAnswerForQuestion,
|
expectedAnswerForQuestion,
|
||||||
|
questionIdForCondition,
|
||||||
restoreGekanatorQuestion,
|
restoreGekanatorQuestion,
|
||||||
saveGekanatorExtraQuestionAnswers,
|
saveGekanatorExtraQuestionAnswers,
|
||||||
saveGekanatorGame,
|
saveGekanatorGame,
|
||||||
@@ -164,6 +165,27 @@ describe('expectedAnswerForQuestion', () => {
|
|||||||
|
|
||||||
expect(expectedAnswerForQuestion(question, post({ id: 1, title: 'short' }))).toBe('no')
|
expect(expectedAnswerForQuestion(question, post({ id: 1, title: 'short' }))).toBe('no')
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('returns yes for matching title-contains questions', () => {
|
||||||
|
const question: StoredGekanatorQuestion = {
|
||||||
|
id: 'title:contains:結束バンド',
|
||||||
|
text: '題名に「結束バンド」が含まれる?',
|
||||||
|
kind: 'title',
|
||||||
|
condition: {
|
||||||
|
type: 'title-contains',
|
||||||
|
text: '結束バンド',
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
expect(expectedAnswerForQuestion(
|
||||||
|
question,
|
||||||
|
post({ title: '結束バンドのライブ' }),
|
||||||
|
)).toBe('yes')
|
||||||
|
expect(expectedAnswerForQuestion(
|
||||||
|
question,
|
||||||
|
post({ title: '後藤ひとりの休日' }),
|
||||||
|
)).toBe('no')
|
||||||
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
describe('restoreGekanatorQuestion', () => {
|
describe('restoreGekanatorQuestion', () => {
|
||||||
@@ -248,6 +270,21 @@ describe('restoreGekanatorQuestion', () => {
|
|||||||
expect(question.test(post({ title: 'x'.repeat(20) }))).toBe(false)
|
expect(question.test(post({ title: 'x'.repeat(20) }))).toBe(false)
|
||||||
expect(question.test(post({ title: 'x'.repeat(21) }))).toBe(true)
|
expect(question.test(post({ title: 'x'.repeat(21) }))).toBe(true)
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('restores title-contains questions with a title matcher', () => {
|
||||||
|
const question = restoreGekanatorQuestion({
|
||||||
|
id: 'title:contains:結束バンド',
|
||||||
|
text: '題名に「結束バンド」が含まれる?',
|
||||||
|
kind: 'title',
|
||||||
|
condition: {
|
||||||
|
type: 'title-contains',
|
||||||
|
text: '結束バンド',
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
expect(question.test(post({ title: '結束バンドのライブ' }))).toBe(true)
|
||||||
|
expect(question.test(post({ title: '後藤ひとりの休日' }))).toBe(false)
|
||||||
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
describe('buildGekanatorQuestions', () => {
|
describe('buildGekanatorQuestions', () => {
|
||||||
@@ -264,6 +301,59 @@ describe('buildGekanatorQuestions', () => {
|
|||||||
expect(titleQuestion?.text).toMatch(/^タイトルは \d+ 文字以上\?$/)
|
expect(titleQuestion?.text).toMatch(/^タイトルは \d+ 文字以上\?$/)
|
||||||
expect(titleQuestion?.id).toMatch(/^title:length-at-least:\d+$/)
|
expect(titleQuestion?.id).toMatch(/^title:length-at-least:\d+$/)
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('builds title-contains questions from repeated title words', () => {
|
||||||
|
const questions = buildGekanatorQuestions([
|
||||||
|
post({ id: 1, title: '結束バンド ライブ' }),
|
||||||
|
post({ id: 2, title: '結束バンド 新曲' }),
|
||||||
|
post({ id: 3, title: '後藤ひとり 練習' }),
|
||||||
|
post({ id: 4, title: '伊地知虹夏 練習' }),
|
||||||
|
])
|
||||||
|
|
||||||
|
const titleContainsQuestion = questions.find(question =>
|
||||||
|
question.condition.type === 'title-contains'
|
||||||
|
&& question.condition.text === '結束バンド')
|
||||||
|
|
||||||
|
expect(titleContainsQuestion).toMatchObject({
|
||||||
|
id: 'title:contains:結束バンド',
|
||||||
|
text: '題名に「結束バンド」が含まれる?',
|
||||||
|
kind: 'title',
|
||||||
|
source: 'default',
|
||||||
|
priorityWeight: .96,
|
||||||
|
})
|
||||||
|
expect(titleContainsQuestion?.test(post({ title: '結束バンドのライブ' }))).toBe(true)
|
||||||
|
expect(titleContainsQuestion?.test(post({ title: '廣井きくりのライブ' }))).toBe(false)
|
||||||
|
})
|
||||||
|
|
||||||
|
it('honors question caps and title-contains toggles', () => {
|
||||||
|
const posts = [
|
||||||
|
post({ id: 1, title: '結束バンド ライブ' }),
|
||||||
|
post({ id: 2, title: '結束バンド 新曲' }),
|
||||||
|
post({ id: 3, title: '後藤ひとり 練習' }),
|
||||||
|
post({ id: 4, title: '伊地知虹夏 練習' }),
|
||||||
|
]
|
||||||
|
|
||||||
|
const capped = buildGekanatorQuestions(posts, {
|
||||||
|
titleContainsCap: 1,
|
||||||
|
totalQuestionCap: 1,
|
||||||
|
})
|
||||||
|
const withoutTitleContains = buildGekanatorQuestions(posts, {
|
||||||
|
includeTitleContains: false,
|
||||||
|
})
|
||||||
|
|
||||||
|
expect(capped).toHaveLength(1)
|
||||||
|
expect(withoutTitleContains.some(question =>
|
||||||
|
question.condition.type === 'title-contains')).toBe(false)
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
|
describe('questionIdForCondition', () => {
|
||||||
|
it('builds stable ids for title-contains questions', () => {
|
||||||
|
expect(questionIdForCondition({
|
||||||
|
type: 'title-contains',
|
||||||
|
text: '結束バンド',
|
||||||
|
})).toBe('title:contains:結束バンド')
|
||||||
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
describe('Gekanator API writers', () => {
|
describe('Gekanator API writers', () => {
|
||||||
|
|||||||
@@ -13,6 +13,7 @@ export type GekanatorAnswerLog = {
|
|||||||
questionId: string
|
questionId: string
|
||||||
questionText: string
|
questionText: string
|
||||||
questionCondition?: GekanatorQuestionCondition
|
questionCondition?: GekanatorQuestionCondition
|
||||||
|
questionMode?: 'normal' | 'winning_run'
|
||||||
answer: GekanatorAnswerValue
|
answer: GekanatorAnswerValue
|
||||||
originalAnswer: GekanatorAnswerValue }
|
originalAnswer: GekanatorAnswerValue }
|
||||||
|
|
||||||
@@ -29,6 +30,8 @@ export type GekanatorQuestionSource =
|
|||||||
| 'ai_generated'
|
| 'ai_generated'
|
||||||
| 'admin_curated'
|
| 'admin_curated'
|
||||||
|
|
||||||
|
export type GekanatorPerformanceMode = 'lite' | 'normal'
|
||||||
|
|
||||||
export type GekanatorQuestionCondition =
|
export type GekanatorQuestionCondition =
|
||||||
| { type: 'tag'; key: string }
|
| { type: 'tag'; key: string }
|
||||||
| { type: 'source'; host: string }
|
| { type: 'source'; host: string }
|
||||||
@@ -38,6 +41,7 @@ export type GekanatorQuestionCondition =
|
|||||||
| { type: 'title-length-at-least'; length: number }
|
| { type: 'title-length-at-least'; length: number }
|
||||||
| { type: 'title-length-greater-than'; length: number }
|
| { type: 'title-length-greater-than'; length: number }
|
||||||
| { type: 'title-has-ascii' }
|
| { type: 'title-has-ascii' }
|
||||||
|
| { type: 'title-contains'; text: string }
|
||||||
| {
|
| {
|
||||||
type: 'post-similarity'
|
type: 'post-similarity'
|
||||||
postId: number
|
postId: number
|
||||||
@@ -76,6 +80,13 @@ export type GekanatorQuestion = {
|
|||||||
exampleAnswers?: Record<`${ number }`, GekanatorAnswerValue>
|
exampleAnswers?: Record<`${ number }`, GekanatorAnswerValue>
|
||||||
test: (post: Post) => boolean }
|
test: (post: Post) => boolean }
|
||||||
|
|
||||||
|
export type BuildGekanatorQuestionsOptions = {
|
||||||
|
includeTitleContains?: boolean
|
||||||
|
tagQuestionCap?: number
|
||||||
|
titleContainsCap?: number
|
||||||
|
totalQuestionCap?: number
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
export const normalizeTitleLengthCondition = (
|
export const normalizeTitleLengthCondition = (
|
||||||
condition: GekanatorQuestionCondition,
|
condition: GekanatorQuestionCondition,
|
||||||
@@ -127,6 +138,8 @@ export const questionIdForCondition = (
|
|||||||
return `title:length-at-least:${ titleLengthMinimumForCondition (condition) }`
|
return `title:length-at-least:${ titleLengthMinimumForCondition (condition) }`
|
||||||
case 'title-has-ascii':
|
case 'title-has-ascii':
|
||||||
return 'title:ascii'
|
return 'title:ascii'
|
||||||
|
case 'title-contains':
|
||||||
|
return `title:contains:${ condition.text }`
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -292,6 +305,8 @@ const questionMatches = (
|
|||||||
return (post.title?.length ?? 0) > question.condition.length
|
return (post.title?.length ?? 0) > question.condition.length
|
||||||
case 'title-has-ascii':
|
case 'title-has-ascii':
|
||||||
return /[A-Za-z0-9]/.test (post.title ?? '')
|
return /[A-Za-z0-9]/.test (post.title ?? '')
|
||||||
|
case 'title-contains':
|
||||||
|
return (post.title ?? '').includes (question.condition.text)
|
||||||
case 'post-similarity':
|
case 'post-similarity':
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
@@ -319,6 +334,7 @@ export const expectedAnswerForQuestion = (
|
|||||||
case 'title-length-at-least':
|
case 'title-length-at-least':
|
||||||
case 'title-length-greater-than':
|
case 'title-length-greater-than':
|
||||||
case 'title-has-ascii':
|
case 'title-has-ascii':
|
||||||
|
case 'title-contains':
|
||||||
return questionMatches (post, question) ? 'yes' : 'no'
|
return questionMatches (post, question) ? 'yes' : 'no'
|
||||||
case 'post-similarity':
|
case 'post-similarity':
|
||||||
return null
|
return null
|
||||||
@@ -382,7 +398,16 @@ export const fetchGekanatorExtraQuestions = async (
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
export const buildGekanatorQuestions = (
|
||||||
|
posts: Post[],
|
||||||
|
options: BuildGekanatorQuestionsOptions = { },
|
||||||
|
): GekanatorQuestion[] => {
|
||||||
|
const {
|
||||||
|
includeTitleContains = true,
|
||||||
|
tagQuestionCap = 192,
|
||||||
|
titleContainsCap = 24,
|
||||||
|
totalQuestionCap = Number.POSITIVE_INFINITY,
|
||||||
|
} = options
|
||||||
const tagCounts = countBy (posts.flatMap (post =>
|
const tagCounts = countBy (posts.flatMap (post =>
|
||||||
post.tags
|
post.tags
|
||||||
.filter (tag =>
|
.filter (tag =>
|
||||||
@@ -404,17 +429,31 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
.map (originalMonthDayOf)
|
.map (originalMonthDayOf)
|
||||||
.filter ((monthDay): monthDay is string => monthDay !== null))
|
.filter ((monthDay): monthDay is string => monthDay !== null))
|
||||||
const titleLengthMedian = median (posts.map (post => post.title?.length ?? 0))
|
const titleLengthMedian = median (posts.map (post => post.title?.length ?? 0))
|
||||||
|
const titleWordCounts =
|
||||||
|
includeTitleContains
|
||||||
|
? countBy (
|
||||||
|
posts.flatMap (post =>
|
||||||
|
Array.from (
|
||||||
|
new Set (
|
||||||
|
(post.title ?? '')
|
||||||
|
.match (
|
||||||
|
/[\p{Script=Han}\p{Script=Hiragana}\p{Script=Katakana}A-Za-z0-9]{2,}/gu)
|
||||||
|
?? []))))
|
||||||
|
: new Map<string, number> ()
|
||||||
|
|
||||||
const usefulEntries = <T extends string | number> (counts: Map<T, number>) =>
|
const usefulEntries = <T extends string | number> (
|
||||||
|
counts: Map<T, number>,
|
||||||
|
cap: number,
|
||||||
|
) =>
|
||||||
[...counts.entries ()]
|
[...counts.entries ()]
|
||||||
.filter (([, count]) => count > 0 && count < posts.length)
|
.filter (([, count]) => count > 0 && count < posts.length)
|
||||||
.sort ((a, b) => Math.abs (posts.length / 2 - a[1])
|
.sort ((a, b) => Math.abs (posts.length / 2 - a[1])
|
||||||
- Math.abs (posts.length / 2 - b[1]))
|
- Math.abs (posts.length / 2 - b[1]))
|
||||||
.slice (0, 80)
|
.slice (0, cap)
|
||||||
|
|
||||||
const tagQuestions = usefulEntries (tagCounts)
|
const tagQuestions = usefulEntries (tagCounts, Math.max (tagQuestionCap, 80))
|
||||||
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
||||||
.slice (0, 80)
|
.slice (0, tagQuestionCap)
|
||||||
.map (([key]) => {
|
.map (([key]) => {
|
||||||
const { category, name } = tagFromQuestionKey (String (key))
|
const { category, name } = tagFromQuestionKey (String (key))
|
||||||
const label = category === 'nico' ? nicoTagLabel (name) : name
|
const label = category === 'nico' ? nicoTagLabel (name) : name
|
||||||
@@ -429,7 +468,7 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
test: (post: Post) => questionableTag (post, String (key)) }
|
test: (post: Post) => questionableTag (post, String (key)) }
|
||||||
})
|
})
|
||||||
|
|
||||||
const sourceQuestions = usefulEntries (hosts)
|
const sourceQuestions = usefulEntries (hosts, 20)
|
||||||
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
||||||
.slice (0, 20)
|
.slice (0, 20)
|
||||||
.map (([host]) => ({
|
.map (([host]) => ({
|
||||||
@@ -441,7 +480,7 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
priorityWeight: 1,
|
priorityWeight: 1,
|
||||||
test: (post: Post) => hostOf (post) === host }))
|
test: (post: Post) => hostOf (post) === host }))
|
||||||
|
|
||||||
const originalYearQuestions = usefulEntries (originalYears)
|
const originalYearQuestions = usefulEntries (originalYears, 20)
|
||||||
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
||||||
.slice (0, 20)
|
.slice (0, 20)
|
||||||
.map (([year]) => ({
|
.map (([year]) => ({
|
||||||
@@ -453,7 +492,7 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
priorityWeight: 1,
|
priorityWeight: 1,
|
||||||
test: (post: Post) => originalYearOf (post) === year }))
|
test: (post: Post) => originalYearOf (post) === year }))
|
||||||
|
|
||||||
const originalMonthQuestions = usefulEntries (originalMonths)
|
const originalMonthQuestions = usefulEntries (originalMonths, 20)
|
||||||
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
||||||
.slice (0, 20)
|
.slice (0, 20)
|
||||||
.map (([month]) => ({
|
.map (([month]) => ({
|
||||||
@@ -465,7 +504,7 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
priorityWeight: 1,
|
priorityWeight: 1,
|
||||||
test: (post: Post) => originalMonthOf (post) === month }))
|
test: (post: Post) => originalMonthOf (post) === month }))
|
||||||
|
|
||||||
const originalMonthDayQuestions = usefulEntries (originalMonthDays)
|
const originalMonthDayQuestions = usefulEntries (originalMonthDays, 20)
|
||||||
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
.filter (([, count]) => count >= 2 && count <= Math.max (2, posts.length * .7))
|
||||||
.slice (0, 20)
|
.slice (0, 20)
|
||||||
.map (([monthDay]) => {
|
.map (([monthDay]) => {
|
||||||
@@ -505,6 +544,23 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
const no = posts.length - yes
|
const no = posts.length - yes
|
||||||
return yes >= 2 && no >= 2 && yes <= posts.length * .7 && no <= posts.length * .7
|
return yes >= 2 && no >= 2 && yes <= posts.length * .7 && no <= posts.length * .7
|
||||||
})
|
})
|
||||||
|
const titleContainsQuestions =
|
||||||
|
includeTitleContains
|
||||||
|
? usefulEntries (titleWordCounts, titleContainsCap)
|
||||||
|
.filter (([word, count]) =>
|
||||||
|
String (word).length <= 24
|
||||||
|
&& count >= 2
|
||||||
|
&& count <= Math.max (2, posts.length * .7))
|
||||||
|
.slice (0, titleContainsCap)
|
||||||
|
.map (([word]) => ({
|
||||||
|
id: `title:contains:${ word }`,
|
||||||
|
text: `題名に「${ word }」が含まれる?`,
|
||||||
|
kind: 'title' as const,
|
||||||
|
condition: { type: 'title-contains' as const, text: String (word) },
|
||||||
|
source: 'default' as const,
|
||||||
|
priorityWeight: .96,
|
||||||
|
test: (post: Post) => (post.title ?? '').includes (String (word)) }))
|
||||||
|
: []
|
||||||
|
|
||||||
return [
|
return [
|
||||||
...sourceQuestions,
|
...sourceQuestions,
|
||||||
@@ -512,7 +568,8 @@ export const buildGekanatorQuestions = (posts: Post[]): GekanatorQuestion[] => {
|
|||||||
...originalMonthQuestions,
|
...originalMonthQuestions,
|
||||||
...originalMonthDayQuestions,
|
...originalMonthDayQuestions,
|
||||||
...titleQuestions,
|
...titleQuestions,
|
||||||
...tagQuestions]
|
...titleContainsQuestions,
|
||||||
|
...tagQuestions].slice (0, totalQuestionCap)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -145,7 +145,30 @@ describe('recoverCandidatePosts', () => {
|
|||||||
|
|
||||||
expect(recovered?.recoveryStepCount).toBe (1)
|
expect(recovered?.recoveryStepCount).toBe (1)
|
||||||
expect([...(recovered?.recoveredCandidatePosts.keys () ?? [])])
|
expect([...(recovered?.recoveredCandidatePosts.keys () ?? [])])
|
||||||
.toEqual ([8, 7, 6, 5, 4, 3, 2])
|
.toEqual ([8, 7, 6, 5, 4])
|
||||||
expect(recovered?.recoveredCandidatePosts.get (7)).toBe (2)
|
expect(recovered?.recoveredCandidatePosts.get (7)).toBe (2)
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('does not add posts when recovered and eligible candidates already hit the target', () => {
|
||||||
|
const posts = Array.from ({ length: 10 }, (_value, index) => post (index + 1))
|
||||||
|
const scores = new Map (posts.map (candidate => [candidate.id, candidate.id]))
|
||||||
|
|
||||||
|
const recovered = recoverCandidatePosts ({
|
||||||
|
posts,
|
||||||
|
scores,
|
||||||
|
rejectedPostIds: new Set (),
|
||||||
|
recoveredCandidatePosts: new Map ([
|
||||||
|
[1, 1],
|
||||||
|
[2, 1],
|
||||||
|
[3, 1],
|
||||||
|
]),
|
||||||
|
eligiblePostIds: new Set ([4, 5, 6]),
|
||||||
|
answerCountAtRecovery: 2,
|
||||||
|
recoveryStepCount: 0,
|
||||||
|
})
|
||||||
|
|
||||||
|
expect(recovered?.recoveryStepCount).toBe (1)
|
||||||
|
expect([...(recovered?.recoveredCandidatePosts.keys () ?? [])])
|
||||||
|
.toEqual ([1, 2, 3])
|
||||||
|
})
|
||||||
})
|
})
|
||||||
|
|||||||
@@ -100,7 +100,7 @@ export const allConcreteAnswerOptionsExhausted = (
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
const nextRecoveryBatchSize = (recoveryStepCount: number): number =>
|
const nextRecoveryTargetSize = (recoveryStepCount: number): number =>
|
||||||
6 * (2 ** recoveryStepCount)
|
6 * (2 ** recoveryStepCount)
|
||||||
|
|
||||||
|
|
||||||
@@ -125,6 +125,16 @@ export const recoverCandidatePosts = ({
|
|||||||
recoveryStepCount: number
|
recoveryStepCount: number
|
||||||
} | null => {
|
} | null => {
|
||||||
const recovered = new Map (recoveredCandidatePosts)
|
const recovered = new Map (recoveredCandidatePosts)
|
||||||
|
const targetSize = nextRecoveryTargetSize (recoveryStepCount)
|
||||||
|
const countedPostIds = new Set ([
|
||||||
|
...eligiblePostIds,
|
||||||
|
...recovered.keys ()])
|
||||||
|
const addCount = targetSize - countedPostIds.size
|
||||||
|
if (addCount <= 0)
|
||||||
|
return {
|
||||||
|
recoveredCandidatePosts: recovered,
|
||||||
|
recoveryStepCount: recoveryStepCount + 1 }
|
||||||
|
|
||||||
const candidates = posts
|
const candidates = posts
|
||||||
.filter (post =>
|
.filter (post =>
|
||||||
!(rejectedPostIds.has (post.id))
|
!(rejectedPostIds.has (post.id))
|
||||||
@@ -133,7 +143,7 @@ export const recoverCandidatePosts = ({
|
|||||||
.sort ((a, b) =>
|
.sort ((a, b) =>
|
||||||
(scores.get (b.id) ?? Number.NEGATIVE_INFINITY)
|
(scores.get (b.id) ?? Number.NEGATIVE_INFINITY)
|
||||||
- (scores.get (a.id) ?? Number.NEGATIVE_INFINITY))
|
- (scores.get (a.id) ?? Number.NEGATIVE_INFINITY))
|
||||||
.slice (0, nextRecoveryBatchSize (recoveryStepCount))
|
.slice (0, addCount)
|
||||||
|
|
||||||
if (candidates.length === 0)
|
if (candidates.length === 0)
|
||||||
return null
|
return null
|
||||||
|
|||||||