Merge pull request #71 from martindurant/bits

CLI missing bits

Author: martindurant

src/projspec/__main__.py

@@ -200,6 +200,10 @@ def list(json_out):
 @library.command("delete")
 @click.argument("url")
 def delete(url):
+    """Delete the project at the given URL from the library.
+
+    URL must be given as shows in `list`.
+    """
     from projspec.library import ProjectLibrary
 
     library = ProjectLibrary()
@@ -230,6 +234,18 @@ def show():
     print(conf)
 
 
+@config.command("defaults")
+def defaults():
+    """Show default config settings for all available values and their definitions"""
+    from projspec.config import defaults, config_doc
+    import os
+
+    print("PROJSPEC_CONFIG_DIR", os.environ.get("PROJSPEC_CONFIG_DIR", "unset"))
+    print()
+    for k, v, d in zip(defaults(), defaults().values(), config_doc.values()):
+        print(f"{k}: {v} -- {d}")
+
+
 @config.command("unset")
 @click.argument("key")
 def unset(key):

src/projspec/artifact/base.py

@@ -35,7 +35,7 @@ def _check_runner(self):
 
     @property
     def state(self) -> Literal["clean", "done", "pending", ""]:
-        if get_conf("remote_artifact_status"):
+        if get_conf("remote_artifact_status") or self.proj.is_local():
             if self._is_clean():
                 return "clean"
             elif self._is_done():
@@ -47,7 +47,7 @@ def state(self) -> Literal["clean", "done", "pending", ""]:
 
     def make(self, *args, **kwargs):
         """Create the artifact and any runtime it depends on"""
-        if not isinstance(self.proj.fs, fsspec.implementations.local.LocalFileSystem):
+        if not self.proj.is_local():
             # Later, will implement download-and-make, although some tools
             # can already do this themselves.
             raise RuntimeError("Can't run local command on remote project")

src/projspec/config.py

@@ -30,7 +30,7 @@ def defaults():
     "scan_max_size": "don't scan files bigger than this (in bytes)",
     "remote_artifact_status": "whether to check status for remote artifacts",
     "capture_artifact_output": (
-        "if True, capture and enqueue output from spawned Process artifacts."
+        "if True, capture and enqueue output from spawned Process artifacts. "
         "Otherwise, output appears on stdout/err."
     ),
 }

vsextension/src/extension.ts

@@ -1267,7 +1267,7 @@ function getTreeWebviewContent(treeData: TreeNode, specNames: string[] = [], scr
                         command: 'selectProject',
                         item: itemData
                     });
-                } else if (itemData.projectUrl && !itemData.children) {
+                } else if (itemData.projectUrl && (!itemData.children || itemData.itemType === 'spec')) {
                     vscode.postMessage({
                         command: 'selectItem',
                         item: itemData