[ICC-47] quiz parsing

[ChanJinYeon]

📢 설명

파일의 URL, 생성할 퀴즈의 개수를 받아 문제를 생성 및 API response 형태까지 맞췄습니다.

✅ 체크 리스트

• Swagger API 요청 확인

{
  "file_url": "https://d1tqvep3lsyhyj.cloudfront.net/20250520_223833921_12. chap11_component2.pdf",
  "quiz_count": 20,
  "type": "string"
}

• 퀴즈 생성 시 JSON 객체로 파싱 되었는지 확인
• 퀴즈 생성 시 번호 확인 (1번부터 N번까지)
• 퀴즈 생성 시 generate_response 형태 갖췄는지 확인

[Copilot]

Pull Request Overview

This PR implements end-to-end quiz generation by fetching and parsing input files, summarizing content, splitting it into chunks, and orchestrating calls to Bedrock for question creation.

• Add process_file to extract text from PDF and PPTX files.
• Introduce create_chunks to divide text into segments based on requested quiz count.
• Update GenerateService to integrate parsing, summarization, chunking, and DTO changes.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
app/util/parsing.py	New function to download and extract text from files.
app/util/create_chunks.py	Async chunking utility based on quiz count.
app/service/generate_service.py	Service orchestration for parsing, summary, and quiz.
app/dto/response/generate_response.py	Refactor response DTO to quiz field.
app/dto/request/generate_request.py	Add type field to request.
app/adapter/summary_bedrock.py	Adapter for generating summaries via AWS Lambda.
app/adapter/request_to_bedrock.py	Timeout update and debug print disabled.

Comments suppressed due to low confidence (1)

app/dto/request/generate_request.py:7

• [nitpick] Using type as a field name shadows the built-in type. Consider renaming to something more descriptive like quiz_type or file_type.

    type: str

[Copilot]

Calculating chunk_count as quiz_count // 5 can yield zero when quiz_count < 5, leading to a division by zero on the next line. Consider validating or clamping chunk_count to at least 1.

Suggested change

	chunk_count = quiz_count // 5
	chunk_count = max(quiz_count // 5, 1)

[Copilot]

[nitpick] The broad try/except simply re-raises the exception without adding context. Consider removing it or logging additional details to improve error diagnosis.

Suggested change

	import tempfile
	from app.dto.request.generate_request import GenerateRequest
	import tempfile
	import logging
	from app.dto.request.generate_request import GenerateRequest
	logging.basicConfig(level=logging.ERROR)
	logger = logging.getLogger(__name__)

[Copilot]

[nitpick] The try/except block here just rethrows the caught exception. It can be removed to simplify the code unless you plan to add specific error handling.

Suggested change

	try:
	chunks = []
	chunk_count = quiz_count // 5
	chunk_size = len(text) // chunk_count
	for i in range(chunk_count):
	start = i * chunk_size
	end = (i + 1) * chunk_size if i < chunk_count - 1 else len(text)
	chunks.append(text[start:end])
	return chunks
	except Exception as e:
	raise e
	chunks = []
	chunk_count = quiz_count // 5
	chunk_size = len(text) // chunk_count
	for i in range(chunk_count):
	start = i * chunk_size
	end = (i + 1) * chunk_size if i < chunk_count - 1 else len(text)
	chunks.append(text[start:end])
	return chunks

[Copilot]

Imported process_file is unused in this module. Consider removing this import to avoid confusion.

Suggested change

from app.util.parsing import process_file

[Copilot]

Imported GenerateRequest is unused. You can remove this import to keep the code clean.

Suggested change

from app.dto.request.generate_request import GenerateRequest

[Copilot]

Leftover commented-out debug print. Remove it or replace with structured logging if needed.

[GulSauce]

잘 설명해주셔서 감사합니다 고생하셨습니다