youtrack-cli 0.3.0__tar.gz → 0.3.2__tar.gz

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/.claude/settings.local.json

@@ -40,7 +40,8 @@
       "mcp__github__create_issue",
       "mcp__claude-code__Read",
       "mcp__claude-code__Grep",
-      "WebFetch(domain:github.com)"
+      "WebFetch(domain:github.com)",
+      "Bash(git remote get-url:*)"
     ],
     "deny": []
   }

youtrack_cli-0.3.2/CLAUDE.md

@@ -0,0 +1,35 @@
+# CLAUDE.md
+
+This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
+
+## Plan
+
+This is a YouTrack CLI application for interacting with JetBrains YouTrack issue tracking system via command line interface. This cli will offer an ergonomic, best practice cli and will leverage
+
+- rich
+- textual
+- pydantic
+
+We use `uv` for managing dependencies.
+
+## Create
+
+Each new feature must have a corresponding github issue. When working on a new issue a new feature branch must be created with the name of the branch matching the name of the issue with the issue number in it.
+
+For every change that is implemented, the README.md file MUST be updated to reflect that change.
+
+## Test
+
+All tests must pass. We use `pytest` for testing, `ruff` for linting, `ty` for type checking, `tox` for running on various versions of Python. We'll utilize `zizmor` for reviewing our GitHub Actions. Pre-commit hooks ensure code quality before commits. All commands should be run with uv.
+
+## Documentation
+
+Documentation is available in the docs/ folder. Any new functionality should have documentation written for it there. The README.md file shoudl not be used for comprehensive documentation.
+
+## Deploy
+
+Deployment will always be done to a feature branch. When a feature is significant enough, we'll bump the version of the tool and tag it with that version. We will have a github action that deploys this to Test PyPI and PyPI using a `release.yml` GitHub Action.
+
+## Current Configuration
+
+- Claude Code permissions are configured in `.claude/settings.local.json`

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: youtrack-cli
-Version: 0.3.0
+Version: 0.3.2
 Summary: YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system
 Project-URL: Homepage, https://github.com/ryan-murphy/yt-cli
 Project-URL: Documentation, https://yt-cli.readthedocs.io/

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/justfile

@@ -52,8 +52,8 @@ format-check:
 [group('quality')]
 typecheck:
     #!/usr/bin/env bash
-    echo "Running mypy type checker..."
-    uv run mypy youtrack_cli
+    echo "Running ty type checker..."
+    uv run ty youtrack_cli
     echo "✅ Type checking complete"
 
 [group('quality')]

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "youtrack-cli"
-version = "0.3.0"
+version = "0.3.2"
 description = "YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system"
 readme = "README.md"
 requires-python = ">=3.9, <3.14"
@@ -62,7 +62,7 @@ packages = ["youtrack_cli"]
 
 [tool.ruff]
 target-version = "py39"
-line-length = 88
+line-length = 120
 
 [tool.ruff.lint]
 select = [

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/tests/test_admin.py

@@ -51,9 +51,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_settings
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.get_global_settings()
 
@@ -71,20 +69,14 @@ class TestAdminManager:
         assert "Not authenticated" in result["message"]
 
     @pytest.mark.asyncio
-    async def test_get_global_settings_insufficient_permissions(
-        self, admin_manager, auth_manager
-    ):
+    async def test_get_global_settings_insufficient_permissions(self, admin_manager, auth_manager):
         """Test global settings retrieval with insufficient permissions."""
         with patch("httpx.AsyncClient") as mock_client:
             mock_response = Mock()
             mock_response.status_code = 403
             mock_request = Mock()
-            http_error = httpx.HTTPStatusError(
-                "Forbidden", request=mock_request, response=mock_response
-            )
-            mock_client.return_value.__aenter__.return_value.get.side_effect = (
-                http_error
-            )
+            http_error = httpx.HTTPStatusError("Forbidden", request=mock_request, response=mock_response)
+            mock_client.return_value.__aenter__.return_value.get.side_effect = http_error
 
             result = await admin_manager.get_global_settings()
 
@@ -98,9 +90,7 @@ class TestAdminManager:
             mock_response = Mock()
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_response
 
             result = await admin_manager.set_global_setting("server.name", "New Name")
 
@@ -114,12 +104,8 @@ class TestAdminManager:
             mock_response = Mock()
             mock_response.status_code = 400
             mock_request = Mock()
-            http_error = httpx.HTTPStatusError(
-                "Bad Request", request=mock_request, response=mock_response
-            )
-            mock_client.return_value.__aenter__.return_value.post.side_effect = (
-                http_error
-            )
+            http_error = httpx.HTTPStatusError("Bad Request", request=mock_request, response=mock_response)
+            mock_client.return_value.__aenter__.return_value.post.side_effect = http_error
 
             result = await admin_manager.set_global_setting("invalid.key", "value")
 
@@ -142,9 +128,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_license
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.get_license_info()
 
@@ -161,9 +145,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_usage
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.get_license_usage()
 
@@ -186,9 +168,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_health
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.get_system_health()
 
@@ -202,9 +182,7 @@ class TestAdminManager:
             mock_response = Mock()
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_response
 
             result = await admin_manager.clear_caches()
 
@@ -231,12 +209,10 @@ class TestAdminManager:
 
         with patch("httpx.AsyncClient") as mock_client:
             mock_response = Mock()
-            mock_response.json.return_value = mock_groups
+            mock_response.json.return_value = {"usergroups": mock_groups}
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.list_user_groups()
 
@@ -257,9 +233,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_created_group
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_response
 
             result = await admin_manager.create_user_group("New Group", "A new group")
 
@@ -274,12 +248,8 @@ class TestAdminManager:
             mock_response = Mock()
             mock_response.status_code = 400
             mock_request = Mock()
-            http_error = httpx.HTTPStatusError(
-                "Bad Request", request=mock_request, response=mock_response
-            )
-            mock_client.return_value.__aenter__.return_value.post.side_effect = (
-                http_error
-            )
+            http_error = httpx.HTTPStatusError("Bad Request", request=mock_request, response=mock_response)
+            mock_client.return_value.__aenter__.return_value.post.side_effect = http_error
 
             result = await admin_manager.create_user_group("Existing Group")
 
@@ -311,9 +281,7 @@ class TestAdminManager:
             mock_response.json.return_value = mock_fields
             mock_response.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_response
 
             result = await admin_manager.list_custom_fields()
 
@@ -387,9 +355,7 @@ class TestAdminManager:
             # Verify that it shows "No user groups found"
             mock_print.assert_called()
             call_args = [call[0][0] for call in mock_print.call_args_list]
-            no_groups_found = any(
-                "No user groups found" in str(arg) for arg in call_args
-            )
+            no_groups_found = any("No user groups found" in str(arg) for arg in call_args)
             assert no_groups_found
 
     def test_display_user_groups_with_data(self, admin_manager):
@@ -417,9 +383,7 @@ class TestAdminManager:
             # Verify that it shows "No custom fields found"
             mock_print.assert_called()
             call_args = [call[0][0] for call in mock_print.call_args_list]
-            no_fields_found = any(
-                "No custom fields found" in str(arg) for arg in call_args
-            )
+            no_fields_found = any("No custom fields found" in str(arg) for arg in call_args)
             assert no_fields_found
 
     def test_display_custom_fields_with_data(self, admin_manager):
@@ -478,9 +442,7 @@ class TestAdminCommands:
         }
 
         with patch("asyncio.run") as mock_asyncio:
-            result = self.runner.invoke(
-                main, ["admin", "global-settings", "set", "server.name", "New Name"]
-            )
+            result = self.runner.invoke(main, ["admin", "global-settings", "set", "server.name", "New Name"])
 
             assert result.exit_code == 0
             mock_asyncio.assert_called_once()
@@ -525,9 +487,7 @@ class TestAdminCommands:
         }
 
         with patch("asyncio.run") as mock_asyncio:
-            result = self.runner.invoke(
-                main, ["admin", "maintenance", "clear-cache", "--confirm"]
-            )
+            result = self.runner.invoke(main, ["admin", "maintenance", "clear-cache", "--confirm"])
 
             assert result.exit_code == 0
             mock_asyncio.assert_called_once()
@@ -572,9 +532,7 @@ class TestAdminCommands:
         }
 
         with patch("asyncio.run") as mock_asyncio:
-            result = self.runner.invoke(
-                main, ["admin", "user-groups", "create", "NewGroup"]
-            )
+            result = self.runner.invoke(main, ["admin", "user-groups", "create", "NewGroup"])
 
             assert result.exit_code == 0
             mock_asyncio.assert_called_once()

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/tests/test_articles.py

@@ -52,9 +52,7 @@ class TestArticleManager:
             mock_resp.text = '{"id": "123", "summary": "Test Article"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.create_article(
                 title="Test Article",
@@ -72,9 +70,7 @@ class TestArticleManager:
             mock_resp = Mock()
             mock_resp.status_code = 400
             mock_resp.text = "Bad Request"
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.create_article(
                 title="Test Article",
@@ -107,9 +103,7 @@ class TestArticleManager:
             mock_resp.text = '[{"id": "123", "summary": "Article 1"}]'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.list_articles()
 
@@ -133,9 +127,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.get_article("123")
 
@@ -158,9 +150,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.update_article(
                 article_id="123",
@@ -187,9 +177,7 @@ class TestArticleManager:
             mock_resp = Mock()
             mock_resp.status_code = 200
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.delete.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.delete.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.delete_article("123")
 
@@ -212,9 +200,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.publish_article("123")
 
@@ -240,9 +226,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.search_articles("search query")
 
@@ -268,9 +252,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.get_article_comments("123")
 
@@ -293,9 +275,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.add_comment("123", "Test comment")
 
@@ -322,9 +302,7 @@ class TestArticleManager:
             mock_resp.text = '{"mock": "response"}'
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp  # noqa: E501
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp  # noqa: E501
 
             result = await article_manager.get_article_attachments("123")
 
@@ -350,9 +328,7 @@ class TestArticleManager:
             article_manager.console = mock_console.return_value
             article_manager.display_articles_table([])
 
-            mock_console.return_value.print.assert_called_with(
-                "No articles found.", style="yellow"
-            )
+            mock_console.return_value.print.assert_called_with("No articles found.", style="yellow")
 
     def test_display_articles_table_with_data(self, article_manager):
         """Test displaying articles table with data."""
@@ -383,9 +359,7 @@ class TestArticleManager:
             article_manager.console = mock_console.return_value
             article_manager.display_articles_tree([])
 
-            mock_console.return_value.print.assert_called_with(
-                "No articles found.", style="yellow"
-            )
+            mock_console.return_value.print.assert_called_with("No articles found.", style="yellow")
 
     def test_display_articles_tree_with_data(self, article_manager):
         """Test displaying articles tree with data."""
@@ -455,9 +429,7 @@ class TestArticlesCLI:
                 "data": {"id": "123"},
             }
 
-            result = runner.invoke(
-                main, ["articles", "create", "Test Title", "--content", "Test content"]
-            )
+            result = runner.invoke(main, ["articles", "create", "Test Title", "--content", "Test content"])
 
             assert result.exit_code == 0
             assert "Creating article" in result.output
@@ -545,9 +517,7 @@ class TestArticlesCLI:
                 "data": {"id": "comment-1"},
             }
 
-            result = runner.invoke(
-                main, ["articles", "comments", "add", "123", "Test comment"]
-            )
+            result = runner.invoke(main, ["articles", "comments", "add", "123", "Test comment"])
 
             assert result.exit_code == 0
             assert "Adding comment" in result.output

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/tests/test_auth.py

@@ -27,9 +27,7 @@ class TestAuthConfig:
 
     def test_config_without_username(self):
         """Test creating config without username."""
-        config = AuthConfig(
-            base_url="https://example.youtrack.cloud", token="test-token-123"
-        )
+        config = AuthConfig(base_url="https://example.youtrack.cloud", token="test-token-123")
         assert config.base_url == "https://example.youtrack.cloud"
         assert config.token == "test-token-123"
         assert config.username is None
@@ -51,8 +49,7 @@ class TestAuthManager:
 
         # Store original environment variables to restore later
         self.original_env = {
-            key: os.environ.get(key)
-            for key in ["YOUTRACK_BASE_URL", "YOUTRACK_TOKEN", "YOUTRACK_USERNAME"]
+            key: os.environ.get(key) for key in ["YOUTRACK_BASE_URL", "YOUTRACK_TOKEN", "YOUTRACK_USERNAME"]
         }
 
     def teardown_method(self):
@@ -93,9 +90,7 @@ class TestAuthManager:
     def test_save_credentials_without_username(self):
         """Test saving credentials without username."""
         # Force file storage instead of keyring for this test
-        self.auth_manager.save_credentials(
-            "https://example.youtrack.cloud", "test-token-123", use_keyring=False
-        )
+        self.auth_manager.save_credentials("https://example.youtrack.cloud", "test-token-123", use_keyring=False)
 
         with open(self.config_path) as f:
             content = f.read()
@@ -183,13 +178,9 @@ class TestAuthManager:
         mock_response.raise_for_status.return_value = None
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_client.return_value.__aenter__.return_value.get = AsyncMock(
-                return_value=mock_response
-            )
+            mock_client.return_value.__aenter__.return_value.get = AsyncMock(return_value=mock_response)
 
-            result = await self.auth_manager.verify_credentials(
-                "https://example.youtrack.cloud", "test-token-123"
-            )
+            result = await self.auth_manager.verify_credentials("https://example.youtrack.cloud", "test-token-123")
 
         assert result["status"] == "success"
         assert result["username"] == "testuser"
@@ -200,13 +191,9 @@ class TestAuthManager:
     async def test_verify_credentials_failure(self):
         """Test failed credential verification."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_client.return_value.__aenter__.return_value.get = AsyncMock(
-                side_effect=Exception("HTTP Error")
-            )
+            mock_client.return_value.__aenter__.return_value.get = AsyncMock(side_effect=Exception("HTTP Error"))
 
-            result = await self.auth_manager.verify_credentials(
-                "https://example.youtrack.cloud", "invalid-token"
-            )
+            result = await self.auth_manager.verify_credentials("https://example.youtrack.cloud", "invalid-token")
 
         assert result["status"] == "error"
         assert "HTTP Error" in result["message"]

{youtrack_cli-0.3.0 → youtrack_cli-0.3.2}/tests/test_boards.py

@@ -55,9 +55,7 @@ class TestBoardManager:
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp
 
             result = await board_manager.list_boards()
 
@@ -84,9 +82,7 @@ class TestBoardManager:
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp
 
             result = await board_manager.list_boards(project_id="TEST")
 
@@ -124,9 +120,7 @@ class TestBoardManager:
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.get.return_value = (
-                mock_resp
-            )
+            mock_client.return_value.__aenter__.return_value.get.return_value = mock_resp
 
             result = await board_manager.view_board("123")
 
@@ -161,9 +155,7 @@ class TestBoardManager:
             mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
-            mock_client.return_value.__aenter__.return_value.post.return_value = (
-                mock_resp
-            )
+            mock_client.return_value.__aenter__.return_value.post.return_value = mock_resp
 
             result = await board_manager.update_board("123", name="Updated Board Name")
 
@@ -193,9 +185,7 @@ class TestBoardManager:
     async def test_list_boards_general_error(self, board_manager):
         """Test board listing with general error."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_client.return_value.__aenter__.return_value.get.side_effect = (
-                Exception("Connection error")
-            )
+            mock_client.return_value.__aenter__.return_value.get.side_effect = Exception("Connection error")
 
             result = await board_manager.list_boards()
 
@@ -206,9 +196,7 @@ class TestBoardManager:
     async def test_view_board_general_error(self, board_manager):
         """Test board viewing with general error."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_client.return_value.__aenter__.return_value.get.side_effect = (
-                Exception("Connection error")
-            )
+            mock_client.return_value.__aenter__.return_value.get.side_effect = Exception("Connection error")
 
             result = await board_manager.view_board("123")
 
@@ -219,9 +207,7 @@ class TestBoardManager:
     async def test_update_board_general_error(self, board_manager):
         """Test board updating with general error."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_client.return_value.__aenter__.return_value.post.side_effect = (
-                Exception("Connection error")
-            )
+            mock_client.return_value.__aenter__.return_value.post.side_effect = Exception("Connection error")
 
             result = await board_manager.update_board("123", name="New Name")