[Bug] DatasetSegmentListApi in Service API returns invalid has_more and metadata when page or limit is non-positive

api/controllers/service_api/dataset/segment.py の DatasetSegmentListApi.get から始め、api/controllers/service_api/app/annotation.py の AnnotationListApi とページネーション処理を比較します。limit=0 および 0 以下の page 値でリクエストを再現し、返されるデータと has_more、limit、page のメタデータが正規化されたページネーション値を使用していることを確認します。

Self Checks

• I have read the Contributing Guide and Language Policy.
• This is only for bug report, if you would like to ask a question, please head to Discussions.
• I have searched for existing issues search for existing issues, including closed ones.
• I confirm that I am using English to submit this report, otherwise it will be closed.
• 【中文用户 & Non English User】请使用英语提交，否则会被关闭 ：）
• Please do not modify this template :) and fill in all the required fields.

Dify version

1.0.0-dev (main)

Cloud or Self Hosted

Self Hosted (Docker)

Steps to reproduce

1. Inspect DatasetSegmentListApi.get in api/controllers/service_api/dataset/segment.py.
2. Notice page and limit = min(args.limit, 100) are read from query arguments without calling clamp_pagination(args.page, args.limit, 100).
3. Send a GET request to /v1/datasets/{dataset_id}/documents/{document_id}/segments?limit=0 using a valid dataset API key for a document with existing chunks.
4. Inspect the returned JSON response payload.

✔️ Expected Behavior

DatasetSegmentListApi.get should normalize page and limit using clamp_pagination(args.page, args.limit, 100) (matching sibling endpoints like AnnotationListApi in api/controllers/service_api/app/annotation.py), ensuring effective_page and effective_limit are used consistently for query execution and response metadata fields (has_more, limit, page).

❌ Actual Behavior

Unclamped variables are passed directly to the response metadata:

• response["has_more"] evaluates 0 * 0 < total, returning true for any non-empty document (total > 0).
• API consumers using has_more for pagination get trapped in infinite request loops.
• response["limit"] and response["page"] return unclamped non-positive numbers (0 or negative integers), contradicting the actual returned data array.