maxc-cli 0.5.0__tar.gz → 0.5.1__tar.gz

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maxc-cli
-Version: 0.5.0
+Version: 0.5.1
 Summary: Agent-native MaxCompute CLI for external coding agents
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.9

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/src/maxc_cli/__init__.py

@@ -2,4 +2,4 @@
 
 __all__ = ["__version__"]
 
-__version__ = "0.5.0"
+__version__ = "0.5.1"

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/src/maxc_cli/state_permissions.py

@@ -135,10 +135,18 @@ def open_private_directory(
             except FileNotFoundError:
                 if not create:
                     raise
-                os.mkdir(component, PRIVATE_DIRECTORY_MODE, dir_fd=descriptor)
+                try:
+                    os.mkdir(component, PRIVATE_DIRECTORY_MODE, dir_fd=descriptor)
+                except FileExistsError:
+                    # Another process may have created the same component
+                    # after our open() observed it missing. Re-open it with
+                    # O_DIRECTORY/O_NOFOLLOW below so the usual ownership and
+                    # link checks still apply.
+                    pass
+                else:
+                    if is_leaf:
+                        created_leaf = True
                 next_descriptor = os.open(component, flags, dir_fd=descriptor)
-                if is_leaf:
-                    created_leaf = True
             os.close(descriptor)
             descriptor = next_descriptor
 

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/src/maxc_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maxc-cli
-Version: 0.5.0
+Version: 0.5.1
 Summary: Agent-native MaxCompute CLI for external coding agents
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.9

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/tests/test_setting_parser.py

@@ -366,17 +366,37 @@ def test_translate_odps_error_type_error_not_readonly():
     assert not isinstance(result, ReadOnlyError)
 
 
-def test_cli_readonly_error_has_agent_hints():
+def test_cli_readonly_error_has_agent_hints(tmp_path):
     """Verify that client-side write detection returns WRITE_OPERATION_REQUIRES_FORCE with hints."""
     import io
     import json
 
     from maxc_cli.cli import run
 
+    config_path = tmp_path / "config.yaml"
+    config_path.write_text(
+        "default_project: test_project\n"
+        f"state_dir: {tmp_path / 'state'}\n"
+        f"cache_dir: {tmp_path / 'cache'}\n"
+        "auth:\n"
+        "  provider: access_key\n"
+        "  access_id: test_access_id\n"
+        "  secret_access_key: test_secret\n"
+        "  project: test_project\n"
+        "  endpoint: https://service.example.invalid/api\n",
+        encoding="utf-8",
+    )
     stdout = io.StringIO()
 
     exit_code = run(
-        ["query", "CREATE TABLE t (id BIGINT)", "--json"],
+        [
+            "--config",
+            str(config_path),
+            "query",
+            "CREATE TABLE t (id BIGINT)",
+            "--json",
+        ],
+        cwd=tmp_path,
         stdout=stdout,
     )
 
@@ -386,3 +406,7 @@ def test_cli_readonly_error_has_agent_hints():
     assert output["error"]["recoverable"] is False
     assert "--force" not in output["error"].get("suggestion", "")
     assert "outside" in output["error"].get("suggestion", "")
+    assert any(
+        "SELECT-only" in warning
+        for warning in output["agent_hints"].get("warnings", [])
+    )

{maxc_cli-0.5.0 → maxc_cli-0.5.1}/tests/test_state_permissions.py

@@ -11,6 +11,7 @@ import pytest
 
 import maxc_cli.audit as audit_module
 import maxc_cli.cache as cache_module
+import maxc_cli.state_permissions as state_permissions
 from maxc_cli.app import MaxCApp
 from maxc_cli.audit import AuditLogger
 from maxc_cli.cache import LocalCache
@@ -161,6 +162,36 @@ def test_new_cache_and_state_directories_are_private(tmp_path: Path) -> None:
     assert _mode(state_dir / "jobs.json") == 0o600
 
 
+@pytest.mark.skipif(
+    not state_permissions._USE_DIRECTORY_FDS,
+    reason="exercises descriptor-relative directory creation",
+)
+def test_private_directory_accepts_a_concurrently_created_component(
+    tmp_path: Path,
+    monkeypatch,
+) -> None:
+    target = tmp_path / "state"
+    original_mkdir = state_permissions.os.mkdir
+    raced = False
+
+    def racing_mkdir(path, mode=0o777, *, dir_fd=None):
+        nonlocal raced
+        if path == target.name and dir_fd is not None and not raced:
+            raced = True
+            original_mkdir(path, mode, dir_fd=dir_fd)
+            raise FileExistsError(path)
+        return original_mkdir(path, mode, dir_fd=dir_fd)
+
+    monkeypatch.setattr(state_permissions.os, "mkdir", racing_mkdir)
+
+    directory = state_permissions.open_private_directory(target)
+    state_permissions.close_private_directory(directory)
+
+    assert raced is True
+    assert target.is_dir()
+    assert _mode(target) == 0o700
+
+
 def test_custom_audit_path_does_not_chmod_existing_shared_parent(tmp_path: Path) -> None:
     shared_dir = tmp_path / "shared"
     shared_dir.mkdir(mode=0o755)