Handle hidden (folderless) ClickUp folders transparently

 - Filter hidden projects from interactive selection menu
 - Auto-merge lists from hidden siblings into selection/lookup
 - Add comprehensive test coverage for both paths
 - Fixes folderless lists now appearing seamlessly alongside real folders

Author: matagus

keyup/cli/api_client.py

@@ -141,24 +141,28 @@ def get_project_for(space, argv, interactive=False):
         if len(projects) == 0:
             raise ProjectNotFoundError()
 
-        if len(projects) > 1:
+        visible = [p for p in projects if not getattr(p, "hidden", False)]
+        if not visible:
+            visible = projects
+
+        if len(visible) > 1:
             if interactive:
                 questions = [
                     inquirer.List(
                         "project",
                         message=f"Select a {Color.GREEN}Project{Color.OFF}",
-                        choices=[f"{project.name} [{project.id}]" for project in projects],
+                        choices=[f"{project.name} [{project.id}]" for project in visible],
                     )
                 ]
 
                 answers = inquirer.prompt(questions)
 
                 if answers:
-                    for project in projects:
+                    for project in visible:
                         if f"{project.name} [{project.id}]" == answers["project"]:
                             return project
 
-        return projects[0]
+        return visible[0]
 
     except ProjectNotFoundError:
         raise
@@ -184,10 +188,21 @@ def get_list_for(space_obj, argv, interactive=False):
         index = sys.argv.index("--list")
         list_id = sys.argv[index + 1]
         lists = get_lists_data(space_obj)
+        # Also include lists from hidden (folderless) sibling projects
+        if hasattr(space_obj, "space"):
+            for p in get_projects_data(space_obj.space):
+                if getattr(p, "hidden", False) and p.id != space_obj.id:
+                    lists = lists + get_lists_data(p)
         return next(li for li in lists if str(li.id) == list_id)
 
     except ValueError:
         lists = get_lists_data(space_obj)
+        # Also include lists from hidden (folderless) sibling projects
+        if hasattr(space_obj, "space"):
+            for p in get_projects_data(space_obj.space):
+                if getattr(p, "hidden", False) and p.id != space_obj.id:
+                    lists = lists + get_lists_data(p)
+
         if len(lists) == 0:
             raise ListNotFoundError()
 

keyup/cli/cache.py

@@ -175,23 +175,24 @@ def get_lists_data(project) -> list:
     return lists
 
 
-def get_tasks_data(team, list_id: str) -> list:
+def get_tasks_data(team, list_id: str, include_closed: bool = False) -> list:
     """Get tasks data for a list from cache or fetch from API.
 
     Args:
         team: Team object.
         list_id: ClickUp list ID.
+        include_closed: If True, include closed/done tasks.
 
     Returns:
         List of task objects.
     """
     cache = get_cache()
-    cache_key = f"tasks:{list_id}"
+    cache_key = f"tasks:{list_id}:closed" if include_closed else f"tasks:{list_id}"
 
     if cache_key in cache:
         return cache.get(cache_key)  # type: ignore[no-any-return]
 
-    tasks = team.get_all_tasks(subtasks=False, list_ids=[list_id])
+    tasks = team.get_all_tasks(subtasks=False, list_ids=[list_id], include_closed=include_closed)
     cache.set(cache_key, tasks, expire=TASKS_TTL)
     cache.set(f"team_for_list:{list_id}", team.id, expire=TEAMS_TTL)
     return tasks

keyup/cli/main.py

@@ -30,6 +30,7 @@ def list_tasks(
     group_by: Annotated[
         str, Parameter(name="--group-by", help="Group by: status (default), assignee, priority")
     ] = "status",
+    closed: Annotated[bool, Parameter(name="--closed", help="Include closed/done tasks")] = False,
     no_cache: Annotated[bool, Parameter(name="--no-cache", help="Bypass cache")] = False,
     interactive: Annotated[bool, Parameter(name="-i", help="Enable interactive mode")] = False,
 ) -> None:
@@ -79,6 +80,7 @@ def list_tasks(
         priority=priority,
         due_before=due_before,
         group_by=group_by,
+        include_closed=closed,
         team=team or team_obj.id,
         space=space or space_obj.id,
         project=project or project_obj.id,
@@ -117,6 +119,7 @@ def sprint(
     group_by: Annotated[
         str, Parameter(name="--group-by", help="Group by: status (default), assignee, priority")
     ] = "status",
+    closed: Annotated[bool, Parameter(name="--closed", help="Include closed/done tasks")] = False,
     no_cache: Annotated[bool, Parameter(name="--no-cache", help="Bypass cache")] = False,
     interactive: Annotated[bool, Parameter(name="-i", help="Enable interactive mode")] = False,
 ) -> None:
@@ -167,6 +170,7 @@ def sprint(
         priority=priority,
         due_before=due_before,
         group_by=group_by,
+        include_closed=closed,
         team=team or team_obj.id,
         space=space or space_obj.id,
         project=project or project_obj.id,

keyup/cli/renderer.py

@@ -145,6 +145,7 @@ def render_list(
     priority=None,
     due_before=None,
     group_by="status",
+    include_closed: bool = False,
     team=None,
     space=None,
     project=None,
@@ -160,6 +161,7 @@ def render_list(
         priority: Filter by priority level (low, normal, high, urgent).
         due_before: Filter tasks due before date (YYYY-MM-DD).
         group_by: Group by "status" (default), "assignee", or "priority".
+        include_closed: If True, include closed/done tasks.
         team: Team ID from command line.
         space: Space ID from command line.
         project: Project ID from command line.
@@ -179,14 +181,16 @@ def render_list(
         filters.append(f"due_before={due_before}")
     if group_by != "status":
         filters.append(f"group_by={group_by}")
+    if include_closed:
+        filters.append("closed=true")
 
     filter_info = f" {Effect.DIM}[{', '.join(filters)}]{Effect.DIM_OFF}" if filters else ""
     print(f"{styled_list_name} :: Team: {team_name}{filter_info}")
 
     if no_cache:
-        task_list = team_obj.get_all_tasks(subtasks=False, list_ids=[list_obj.id])
+        task_list = team_obj.get_all_tasks(subtasks=False, list_ids=[list_obj.id], include_closed=include_closed)
     else:
-        task_list = get_tasks_data(team_obj, list_obj.id)
+        task_list = get_tasks_data(team_obj, list_obj.id, include_closed=include_closed)
 
     # Apply filters
     task_list = _filter_tasks(task_list, assignee=assignee, priority=priority, due_before=due_before)
@@ -240,6 +244,8 @@ def render_list(
         cmd_parts.append(f"--due-before {due_before}")
     if group_by != "status":
         cmd_parts.append(f"--group-by {group_by}")
+    if include_closed:
+        cmd_parts.append("--closed")
     if no_cache:
         cmd_parts.append("--no-cache")
 

tests/test_api_client.py

@@ -413,10 +413,13 @@ def setup_method(self):
         """Set up test fixtures."""
         self.mock_space = Mock()
         self.patch_get_lists = patch("keyup.cli.api_client.get_lists_data", side_effect=lambda p: p.lists)
+        self.patch_get_projects = patch("keyup.cli.api_client.get_projects_data", return_value=[])
         self.patch_get_lists.start()
+        self.patch_get_projects.start()
 
     def teardown_method(self):
         self.patch_get_lists.stop()
+        self.patch_get_projects.stop()
 
     @patch("sys.argv", ["keyup", "--list", "list-000"])
     def test_with_list_flag(self):

tests/test_cache.py

@@ -216,10 +216,26 @@ def test_cache_miss(self, mock_get_cache):
         result = get_tasks_data(mock_team, "list-000")
 
         assert result == ["api_task"]
-        mock_team.get_all_tasks.assert_called_once_with(subtasks=False, list_ids=["list-000"])
+        mock_team.get_all_tasks.assert_called_once_with(subtasks=False, list_ids=["list-000"], include_closed=False)
         mock_cache.set.assert_any_call("tasks:list-000", ["api_task"], expire=TASKS_TTL)
         mock_cache.set.assert_any_call("team_for_list:list-000", mock_team.id, expire=TEAMS_TTL)
 
+    @patch("keyup.cli.cache.get_cache")
+    def test_cache_miss_with_include_closed(self, mock_get_cache):
+        """Test cache miss with include_closed uses separate cache key."""
+        mock_cache = Mock()
+        mock_cache.__contains__ = Mock(return_value=False)
+        mock_get_cache.return_value = mock_cache
+
+        mock_team = Mock()
+        mock_team.get_all_tasks.return_value = ["closed_task"]
+
+        result = get_tasks_data(mock_team, "list-000", include_closed=True)
+
+        assert result == ["closed_task"]
+        mock_team.get_all_tasks.assert_called_once_with(subtasks=False, list_ids=["list-000"], include_closed=True)
+        mock_cache.set.assert_any_call("tasks:list-000:closed", ["closed_task"], expire=TASKS_TTL)
+
 
 class TestFindTaskInCache:
     """Tests for find_task_in_cache function."""