@josephyan/qingflow-app-user-mcp 1.0.5 → 1.0.6

package/README.md

@@ -3,13 +3,13 @@
 Install:
 
 ```bash
-npm install @josephyan/qingflow-app-user-mcp@1.0.5
+npm install @josephyan/qingflow-app-user-mcp@1.0.6
 ```
 
 Run:
 
 ```bash
-npx -y -p @josephyan/qingflow-app-user-mcp@1.0.5 qingflow-app-user-mcp
+npx -y -p @josephyan/qingflow-app-user-mcp@1.0.6 qingflow-app-user-mcp
 ```
 
 Environment:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josephyan/qingflow-app-user-mcp",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "Operational end-user MCP for Qingflow records, tasks, comments, and directory workflows.",
   "license": "MIT",
   "type": "module",

package/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "qingflow-mcp"
-version = "1.0.5"
+version = "1.0.6"
 description = "User-authenticated MCP server for Qingflow"
 readme = "README.md"
 license = "MIT"

package/skills/qingflow-record-analysis/SKILL.md

@@ -75,7 +75,7 @@ Use `record_access` to fetch detail rows into local CSV shards. It does not anal
 
 Then run Python against every `files[].local_path`. CSV columns are stable: `record_id`, then `field_<field_id>`. Use `fields[]` metadata to map titles and types.
 
-- Never ask for `page`, `page_size`, `limit`, or `max_rows`; `record_access` owns paging internally.
+- Never ask for `page`, `page_size`, `limit`, or `max_rows`; `record_access` owns paging internally and follows the backend's native paging capability.
 - If multiple CSV files are returned, read them all.
 - If `complete=false` or `safe_for_final_conclusion=false`, downgrade the answer and disclose the limitation.
 - `record_export_direct` is only for explicit export/download/Excel requests, not default analysis.

package/skills/qingflow-record-analysis/references/analysis-gotchas.md

@@ -89,7 +89,7 @@ Examples of the right recovery question:
 
 ## Do not try to control paging manually
 
-`record_access` and `record_analyze` hide paging and scan budget on purpose.
+`record_access` hides paging and follows the backend's native paging capability. `record_analyze` hides paging and scan budget on purpose.
 
 - Do not invent `page_size`
 - Do not invent `requested_pages`

package/src/qingflow_mcp/tools/record_tools.py

@@ -32,9 +32,8 @@ from .directory_tools import _directory_has_more, _directory_items
 
 DEFAULT_QUERY_PAGE_SIZE = 50
 DEFAULT_LIST_PAGE_SIZE = 200
+BACKEND_RECORD_ACCESS_PAGE_SIZE = 1000
 DEFAULT_RECORD_ACCESS_SHARD_ROWS = 5000
-DEFAULT_RECORD_ACCESS_HARD_ROWS = 50000
-DEFAULT_RECORD_ACCESS_TIMEOUT_SECONDS = 60
 DEFAULT_ANALYSIS_PAGE_SIZE = 1000
 DEFAULT_SCAN_MAX_PAGES = 10
 DEFAULT_ANALYSIS_SCAN_MAX_PAGES = 100
@@ -1870,7 +1869,6 @@ class RecordTools(ToolBase):
                 primary_search_que_ids = primary_search_que_ids or None
             match_rules = self._resolve_match_rules(context, filters, index)
             sort_rules = self._resolve_sorts(sorts, index)
-            dept_member_cache: dict[int, set[int]] = {}
             used_list_type: int | None = None
             fallback_list_types = (
                 [view_route.list_type if view_route.list_type is not None else DEFAULT_RECORD_LIST_TYPE]
@@ -1934,17 +1932,8 @@ class RecordTools(ToolBase):
             current_page = 1
             has_more = False
             reported_total: int | None = None
-            limit_reached = False
-            timeout_reached = False
-            started_at = time.perf_counter()
             try:
                 while True:
-                    if row_count >= DEFAULT_RECORD_ACCESS_HARD_ROWS:
-                        limit_reached = True
-                        break
-                    if time.perf_counter() - started_at >= DEFAULT_RECORD_ACCESS_TIMEOUT_SECONDS:
-                        timeout_reached = True
-                        break
                     if used_list_type is None:
                         last_error: QingflowApiError | None = None
                         page: JSONObject | None = None
@@ -1955,7 +1944,7 @@ class RecordTools(ToolBase):
                                     app_key=app_key,
                                     view_selection=view_selection,
                                     page_num=current_page,
-                                    page_size=DEFAULT_LIST_PAGE_SIZE,
+                                    page_size=BACKEND_RECORD_ACCESS_PAGE_SIZE,
                                     query_key=None,
                                     match_rules=match_rules,
                                     sorts=sort_rules,
@@ -1982,7 +1971,7 @@ class RecordTools(ToolBase):
                             app_key=app_key,
                             view_selection=view_selection,
                             page_num=current_page,
-                            page_size=DEFAULT_LIST_PAGE_SIZE,
+                            page_size=BACKEND_RECORD_ACCESS_PAGE_SIZE,
                             query_key=None,
                             match_rules=match_rules,
                             sorts=sort_rules,
@@ -1992,25 +1981,15 @@ class RecordTools(ToolBase):
                     page_rows = page.get("list")
                     items = page_rows if isinstance(page_rows, list) else []
                     if reported_total is None:
-                        reported_total = _effective_total(page, DEFAULT_LIST_PAGE_SIZE)
-                    has_more = _page_has_more(page, current_page, DEFAULT_LIST_PAGE_SIZE, len(items))
+                        reported_total = _effective_total(page, BACKEND_RECORD_ACCESS_PAGE_SIZE)
+                    has_more = _page_has_more(page, current_page, BACKEND_RECORD_ACCESS_PAGE_SIZE, len(items))
                     page_apply_order: list[int] = []
                     page_answer_map: dict[int, list[JSONValue]] = {}
                     for item in items:
-                        if row_count + len(page_apply_order) >= DEFAULT_RECORD_ACCESS_HARD_ROWS:
-                            limit_reached = True
-                            break
                         if not isinstance(item, dict):
                             continue
                         answers = item.get("answers")
                         answer_list = answers if isinstance(answers, list) else []
-                        if not self._matches_view_selection(
-                            context,
-                            answer_list,
-                            view_selection=view_selection,
-                            dept_member_cache=dept_member_cache,
-                        ):
-                            continue
                         apply_id = _coerce_count(item.get("applyId")) or _coerce_count(item.get("id"))
                         if apply_id is None:
                             continue
@@ -2023,7 +2002,7 @@ class RecordTools(ToolBase):
                                 app_key=app_key,
                                 view_selection=view_selection,
                                 page_num=current_page,
-                                page_size=DEFAULT_LIST_PAGE_SIZE,
+                                page_size=BACKEND_RECORD_ACCESS_PAGE_SIZE,
                                 query_key=None,
                                 match_rules=match_rules,
                                 sorts=sort_rules,
@@ -2045,11 +2024,8 @@ class RecordTools(ToolBase):
                                     cast(list[JSONValue], extra_answer_list),
                                 )
                     for apply_id in page_apply_order:
-                        if row_count >= DEFAULT_RECORD_ACCESS_HARD_ROWS:
-                            limit_reached = True
-                            break
                         write_record(apply_id, page_answer_map.get(apply_id, []))
-                    if limit_reached or not has_more:
+                    if not has_more:
                         break
                     current_page += 1
             finally:
@@ -2069,21 +2045,7 @@ class RecordTools(ToolBase):
                         ),
                     }
                 )
-            if limit_reached:
-                warnings.append(
-                    {
-                        "code": "RECORD_ACCESS_INTERNAL_LIMIT_REACHED",
-                        "message": "record_access reached its internal row limit before all pages were fetched.",
-                    }
-                )
-            if timeout_reached:
-                warnings.append(
-                    {
-                        "code": "RECORD_ACCESS_TIMEOUT",
-                        "message": "record_access reached its internal time limit before all pages were fetched.",
-                    }
-                )
-            complete = not has_more and not limit_reached and not timeout_reached
+            complete = not has_more
             safe_for_final_conclusion = complete and not any(
                 warning.get("code") == "CUSTOM_VIEW_FILTER_UNVERIFIED" for warning in warnings
             )
@@ -5594,9 +5556,8 @@ class RecordTools(ToolBase):
         source_pages: list[int] = []
         result_amount: int | None = None
         has_more = False
-        dept_member_cache: dict[int, set[int]] = {}
         view_selection = resolved_view.view_selection
-        local_filtering = bool(filters) or bool(view_selection is not None and view_selection.conditions)
+        local_filtering = bool(filters)
         group_stats: dict[tuple[tuple[str, object], ...], JSONObject] = {}
         overall_metrics = self._initialize_metric_states(metrics)
         matched_rows = 0
@@ -5640,13 +5601,6 @@ class RecordTools(ToolBase):
                     continue
                 answers = item.get("answers")
                 answer_list = answers if isinstance(answers, list) else []
-                if not self._matches_view_selection(
-                    context,
-                    answer_list,
-                    view_selection=view_selection,
-                    dept_member_cache=dept_member_cache,
-                ):
-                    continue
                 if not self._matches_analyze_filters(answer_list, filters):
                     continue
                 matched_rows += 1
@@ -6732,7 +6686,6 @@ class RecordTools(ToolBase):
                 if view_selection is not None
                 else self._get_field_index(profile, context, app_key, force_refresh=False)
             )
-            dept_member_cache: dict[int, set[int]] = {}
             result = self._search_page(
                 context,
                 app_key=app_key,
@@ -6747,17 +6700,7 @@ class RecordTools(ToolBase):
             )
             rows = result.get("list")
             raw_rows = rows if isinstance(rows, list) else []
-            filtered_rows = [
-                item
-                for item in raw_rows
-                if isinstance(item, dict)
-                and self._matches_view_selection(
-                    context,
-                    item.get("answers") if isinstance(item.get("answers"), list) else [],
-                    view_selection=view_selection,
-                    dept_member_cache=dept_member_cache,
-                )
-            ]
+            filtered_rows = [item for item in raw_rows if isinstance(item, dict)]
             if isinstance(rows, list):
                 result = dict(result)
                 result["list"] = filtered_rows
@@ -7007,7 +6950,6 @@ class RecordTools(ToolBase):
         def runner(session_profile, context):
             index = self._get_field_index(profile, context, app_key, force_refresh=False)
             view_selection = self._resolve_view_selection(profile, context, app_key, view_key=view_key, view_name=view_name)
-            dept_member_cache: dict[int, set[int]] = {}
             resolved_column_cap = _bounded_column_limit(
                 max_columns,
                 default_limit=MAX_LIST_COLUMN_LIMIT,
@@ -7113,13 +7055,6 @@ class RecordTools(ToolBase):
                         continue
                     answers = item.get("answers")
                     answer_list = answers if isinstance(answers, list) else []
-                    if not self._matches_view_selection(
-                        context,
-                        answer_list,
-                        view_selection=view_selection,
-                        dept_member_cache=dept_member_cache,
-                    ):
-                        continue
                     matched_records += 1
                     apply_id = _coerce_count(item.get("applyId")) or _coerce_count(item.get("id"))
                     row = _build_flat_row(answer_list, selected_fields_from_primary, apply_id=apply_id)