making-with-code-cli 5.2.0__py3-none-any.whl → 5.4.0__py3-none-any.whl

making_with_code_cli/curriculum.py

@@ -11,7 +11,7 @@ def get_curriculum(mwc_site_url, course_name=None):
     """Fetches curriculum metadata from the site url specified in settings.
     Returns the curriculum metadata for course_name.
     """
-    url = mwc_site_url + "/manifest"
+    url = mwc_site_url.rstrip("/") + "/manifest"
     try:
         response = requests.get(url, timeout=15)
     except requests.exceptions.RequestException as e:

making_with_code_cli/errors.py

@@ -26,3 +26,15 @@ class SoftwareInstallationError(MWCError):
 
 class NoCurriculaAvailable(MWCError):
     pass
+
+class GitError(MWCError):
+    pass
+
+class GitNetworkError(GitError):
+    pass
+
+class GitAuthError(GitError):
+    pass
+
+class GitCorruptError(GitError):
+    pass

making_with_code_cli/git_backend/base_backend.py

@@ -2,7 +2,6 @@ from pathlib import Path
 from subprocess import run, CalledProcessError
 import click
 from making_with_code_cli.helpers import cd
-from making_with_code_cli.setup.tasks import WORK_DIR_PERMISSIONS
 from making_with_code_cli.styles import (
     address,
     info,

making_with_code_cli/git_backend/mwc_backend.py

@@ -8,6 +8,12 @@ import json
 import os
 import requests
 from making_with_code_cli.helpers import cd
+from making_with_code_cli.git_wrapper import (
+    abort_in_progress_operation,
+    recover_detached_head,
+    clear_stale_index_lock,
+    _get_default_branch,
+)
 from making_with_code_cli.styles import (
     address,
     confirm,
@@ -18,6 +24,10 @@ from making_with_code_cli.styles import (
 from making_with_code_cli.errors import (
     MissingSetting,
     GitServerNotAvailable,
+    GitError,
+    GitNetworkError,
+    GitAuthError,
+    GitCorruptError,
 )
 
 EDITOR_COMMANDS = {
@@ -29,17 +39,55 @@ EDITOR_COMMANDS = {
     "emacs": "emacs",
 }
 
+# Phrases in git stderr that identify each failure mode
+_NETWORK_PHRASES = [
+    "Could not resolve host",
+    "Connection timed out",
+    "unable to connect",
+    "Failed to connect",
+    "network is unreachable",
+    "Connection refused",
+]
+_AUTH_PHRASES = [
+    "Authentication failed",
+    "403",
+    "invalid credentials",
+    "could not read Username",
+    "terminal prompts disabled",
+    "Repository not found",
+]
+_CORRUPT_PHRASES = [
+    "corrupt",
+    "bad object",
+    "loose object",
+    "fatal: bad",
+    "object file is empty",
+]
+_NO_UPSTREAM_PHRASES = [
+    "no tracking information",
+    "has no upstream",
+    "There is no tracking information",
+]
+
+
 class MWCBackend(GitBackend):
-    """A Github backend. Students own their own repos and grant teachers access via token.
-    """
+    """A Github backend. Students own their own repos and grant teachers access via token."""
 
     MWC_GIT_PROTOCOL = "https"
     MWC_GIT_SERVER = "git.makingwithcode.org"
     COMMIT_TEMPLATE = ".commit_template"
 
+    @property
+    def git_server(self):
+        """Returns the git server hostname, respecting git_server_url setting if set."""
+        url = self.settings.get('git_server_url')
+        if url:
+            parsed = urlparse(url)
+            return parsed.netloc
+        return self.MWC_GIT_SERVER
+
     def init_module(self, module, modpath):
-        """Creates the named repo from a template.
-        """
+        """Creates the named repo from a template."""
         self.check_settings()
         server, repo_owner, repo_name = self.parse_repo_url(module["repo_url"])
 
@@ -53,8 +101,9 @@ class MWCBackend(GitBackend):
         if (modpath / self.COMMIT_TEMPLATE).exists():
             run(["git", "config", "--local", "commit.template", self.COMMIT_TEMPLATE],
                     check=True, cwd=modpath)
-        run("uv venv", shell=True, check=True, cwd=modpath, capture_output=True)
-        self.init_direnv(modpath)
+        if self.settings.get('manage_python_environment', True):
+            run("uv venv", shell=True, check=True, cwd=modpath, capture_output=True)
+            self.init_direnv(modpath)
 
     def configure_git(self, modpath):
         """Configures git user settings locally for the repo."""
@@ -69,25 +118,117 @@ class MWCBackend(GitBackend):
 
     def update(self, module, modpath, install=True):
         if (modpath / ".git").is_dir():
-            with cd(modpath):
-                relpath = self.relative_path(modpath)
-                click.echo(address(f"Checking {relpath} for updates.", preformatted=True))
-                gitresult = run("git pull", shell=True, capture_output=True, text=True)
-                if gitresult.returncode != 0:
-                    click.echo(error(
-                        f"Could not update {relpath}: {gitresult.stderr.strip()}\n"
-                        "You may have local changes that conflict. Ask a teacher for help."
-                    ))
-                    return
-                click.echo(info(gitresult.stdout))
-                if install and Path("pyproject.toml").exists():
-                    result = run("uv sync", shell=True, capture_output=True, text=True, cwd=modpath)
-                    if result.returncode != 0:
-                        click.echo(debug("Error running uv sync:"))
-                        click.echo(debug(result.stderr, preformatted=True))
-                        click.echo(error(f"There was a problem updating {relpath}. Ask a teacher."))
-                    else:
-                        self.init_direnv(modpath)
+            relpath = self.relative_path(modpath)
+            click.echo(address(f"Checking {relpath} for updates.", preformatted=True))
+            pull_output = self.safe_pull(modpath)
+            if pull_output:
+                click.echo(info(pull_output))
+            if install and (modpath / "pyproject.toml").exists() and self.settings.get('manage_python_environment', True):
+                result = run("uv sync", shell=True, capture_output=True, text=True, cwd=modpath)
+                if result.returncode != 0:
+                    click.echo(debug("Error running uv sync:"))
+                    click.echo(debug(result.stderr, preformatted=True))
+                    click.echo(error(f"There was a problem updating {relpath}. Ask a teacher."))
+                else:
+                    self.init_direnv(modpath)
+
+    def safe_pull(self, modpath):
+        """Pulls from origin with robust recovery for common git error states.
+        Returns the git output string on success. Raises GitError subclasses on
+        unrecoverable failures.
+        """
+        # Fix bad repo states before attempting the pull
+        aborted = abort_in_progress_operation(modpath)
+        if aborted:
+            click.echo(confirm(
+                f"Cleared an incomplete git {aborted} — this shouldn't affect your work."
+            ))
+
+        branch = recover_detached_head(modpath)
+        if branch:
+            click.echo(confirm(
+                f"Your repo was not on any branch. Switched back to '{branch}'."
+            ))
+
+        clear_stale_index_lock(modpath)
+
+        # U1: uncommitted changes that would be overwritten — auto-commit first
+        status = run(
+            "git status --porcelain",
+            shell=True, capture_output=True, text=True, cwd=modpath
+        )
+        if status.stdout.strip():
+            probe = run(
+                "git fetch origin",
+                shell=True, capture_output=True, text=True, cwd=modpath
+            )
+            merge_check = run(
+                "git merge --no-commit --no-ff FETCH_HEAD",
+                shell=True, capture_output=True, text=True, cwd=modpath
+            )
+            # Always abort the probe merge so we start clean
+            run("git merge --abort", shell=True, capture_output=True, cwd=modpath)
+            if "CONFLICT" in merge_check.stdout or "CONFLICT" in merge_check.stderr:
+                run(["git", "add", "--all"], capture_output=True, cwd=modpath)
+                run(
+                    ["git", "commit", "-m", "[auto-saved before update]"],
+                    capture_output=True, cwd=modpath
+                )
+                click.echo(confirm(
+                    "Your unsaved changes were auto-saved in git history "
+                    "before applying the update."
+                ))
+
+        result = self._run_pull(modpath)
+
+        if result.returncode == 0:
+            return result.stdout
+
+        combined = result.stderr + result.stdout
+
+        # U8: no upstream tracking branch — set it and retry
+        if any(p in combined for p in _NO_UPSTREAM_PHRASES):
+            default_branch = _get_default_branch(modpath)
+            run(
+                f"git branch --set-upstream-to=origin/{default_branch} {default_branch}",
+                shell=True, capture_output=True, cwd=modpath
+            )
+            result = self._run_pull(modpath)
+            if result.returncode == 0:
+                return result.stdout
+            combined = result.stderr + result.stdout
+
+        click.echo(debug(combined.strip(), preformatted=True))
+        self._raise_pull_error(combined)
+
+    def _run_pull(self, modpath):
+        # -X theirs resolves merge conflicts in favor of incoming (teacher) changes,
+        # and also handles diverged-history cases (U2, U3) without extra configuration.
+        return run(
+            "git pull -X theirs",
+            shell=True, capture_output=True, text=True, cwd=modpath
+        )
+
+    def _raise_pull_error(self, stderr):
+        if any(p in stderr for p in _NETWORK_PHRASES):
+            raise GitNetworkError(
+                "Could not connect to the MWC server. "
+                "Check your internet connection and try again."
+            )
+        if any(p in stderr for p in _AUTH_PHRASES):
+            raise GitAuthError(
+                "Your login credentials are no longer valid. "
+                "Run `mwc setup` to refresh them, then run `mwc update` again."
+            )
+        if any(p in stderr for p in _CORRUPT_PHRASES):
+            raise GitCorruptError(
+                "Your local copy of this module appears to be damaged. "
+                "Ask your teacher for help."
+            )
+        raise GitError(
+            "Could not update this module. "
+            "The details are shown above. If you're not sure what to do, ask your teacher."
+        )
 
     def init_direnv(self, modpath):
         if not (modpath / ".envrc").exists():
@@ -95,8 +236,7 @@ class MWCBackend(GitBackend):
         run("direnv allow", shell=True, check=True, cwd=modpath)
 
     def user_has_repo(self, repo_name, username=None):
-        """Checks to see whether a user already has the named repo.
-        """
+        """Checks to see whether a user already has the named repo."""
         username = username or self.settings['mwc_username']
         url = f"/repos/{username}/{repo_name}"
         response = self.authenticated_mwc_request('get', url)
@@ -115,7 +255,7 @@ class MWCBackend(GitBackend):
     def clone_repo(self, repo_name):
         user = self.settings['mwc_username']
         auth = user + ':' + self.settings['mwc_git_token']
-        url = f"{self.MWC_GIT_PROTOCOL}://{auth}@{self.MWC_GIT_SERVER}/{user}/{repo_name}.git"
+        url = f"{self.MWC_GIT_PROTOCOL}://{auth}@{self.git_server}/{user}/{repo_name}.git"
         run(["git", "clone", url], check=True, capture_output=True)
 
     def check_settings(self):
@@ -135,7 +275,7 @@ class MWCBackend(GitBackend):
         return (server, repo_owner, repo_name)
 
     def authenticated_mwc_request(self, method_name, urlpath, data=None, params=None):
-        server = self.MWC_GIT_PROTOCOL + '://' + self.MWC_GIT_SERVER
+        server = self.MWC_GIT_PROTOCOL + '://' + self.git_server
         args = {
             'url': server + "/api/v1" + urlpath,
             'auth': (self.settings['mwc_username'], self.settings['mwc_git_token']),
@@ -151,8 +291,7 @@ class MWCBackend(GitBackend):
         return response
 
     def relocate_existing_directory(self, path):
-        """Moves an existing directory out of the way.
-        """
+        """Moves an existing directory out of the way."""
         new_path = path.parent / path.name + '_old'
         while new_path.exists():
             new_path = new_path.parent / new_path.name + '_old'

making_with_code_cli/git_wrapper.py

@@ -1,6 +1,8 @@
+from pathlib import Path
 from subprocess import run, CalledProcessError
 from making_with_code_cli.helpers import cd
 
+
 def in_repo():
     """Checks whether currently in repo"""
     try:
@@ -9,9 +11,11 @@ def in_repo():
     except CalledProcessError:
         return False
 
+
 def repo_has_changes():
     return len(changed_files()) > 0
 
+
 def changed_files():
     "Returns a list of (status, filename) tuples for changed files"
     cmd = "git status --porcelain"
@@ -23,3 +27,98 @@ def changed_files():
         return files
     else:
         return []
+
+
+def has_unpushed_commits(cwd=None):
+    """Returns True if there are committed but unpushed local changes."""
+    result = run(
+        "git log @{u}..HEAD --oneline",
+        shell=True, capture_output=True, text=True, cwd=cwd
+    )
+    if result.returncode != 0:
+        return False
+    return bool(result.stdout.strip())
+
+
+def get_repo_state(cwd=None):
+    """Returns 'merge', 'rebase', 'cherry-pick', 'detached', or 'clean'."""
+    git_dir = _git_dir(cwd)
+    if (git_dir / "MERGE_HEAD").exists():
+        return "merge"
+    if (git_dir / "rebase-merge").exists() or (git_dir / "rebase-apply").exists():
+        return "rebase"
+    if (git_dir / "CHERRY_PICK_HEAD").exists():
+        return "cherry-pick"
+    result = run(
+        "git branch --show-current",
+        shell=True, capture_output=True, text=True, cwd=cwd
+    )
+    if result.returncode == 0 and not result.stdout.strip():
+        return "detached"
+    return "clean"
+
+
+def abort_in_progress_operation(cwd=None):
+    """Aborts any in-progress merge, rebase, or cherry-pick.
+    Returns the operation name that was aborted, or None if nothing to abort.
+    """
+    state = get_repo_state(cwd)
+    commands = {
+        "merge": "git merge --abort",
+        "rebase": "git rebase --abort",
+        "cherry-pick": "git cherry-pick --abort",
+    }
+    if state in commands:
+        run(commands[state], shell=True, capture_output=True, cwd=cwd)
+        return state
+    return None
+
+
+def recover_detached_head(cwd=None):
+    """If in detached HEAD, checks out the default branch and returns its name.
+    Returns None if no action was needed.
+    """
+    if get_repo_state(cwd) != "detached":
+        return None
+    branch = _get_default_branch(cwd)
+    # Stash any uncommitted work so the checkout doesn't fail
+    stash_result = run(
+        "git stash", shell=True, capture_output=True, text=True, cwd=cwd
+    )
+    had_stash = "No local changes to save" not in stash_result.stdout
+    run(["git", "checkout", branch], capture_output=True, cwd=cwd)
+    if had_stash:
+        pop_result = run(
+            "git stash pop", shell=True, capture_output=True, text=True, cwd=cwd
+        )
+        # If pop conflicts (extreme edge case), drop the stash
+        if pop_result.returncode != 0:
+            run("git stash drop", shell=True, capture_output=True, cwd=cwd)
+    return branch
+
+
+def clear_stale_index_lock(cwd=None):
+    """Removes .git/index.lock if present. Returns True if a lock was cleared."""
+    lock = _git_dir(cwd) / "index.lock"
+    if lock.exists():
+        lock.unlink()
+        return True
+    return False
+
+
+def _get_default_branch(cwd=None):
+    """Returns the remote default branch name, falling back to 'main'."""
+    result = run(
+        "git symbolic-ref refs/remotes/origin/HEAD",
+        shell=True, capture_output=True, text=True, cwd=cwd
+    )
+    if result.returncode == 0:
+        return result.stdout.strip().replace("refs/remotes/origin/", "")
+    return "main"
+
+
+def _git_dir(cwd=None):
+    """Returns the Path to the .git directory for the given working dir."""
+    if cwd is None:
+        return Path(".git")
+    return Path(cwd) / ".git"

making_with_code_cli/mwc_accounts_api.py

@@ -45,6 +45,26 @@ class MWCAccountsAPI:
         response = self.put("/students", data=params, token=token)
         return self.handle_response(response)
 
+    def sync_workspace(self, token, workspace_id):
+        response = self.post(f"/workspaces/{workspace_id}/sync", token=token)
+        return self.handle_response(response)
+
+    def list_workspaces(self, token):
+        response = self.get("/workspaces", token=token)
+        return self.handle_response(response)
+
+    def create_workspace(self, token, params):
+        response = self.post("/workspaces", data=params, token=token)
+        return self.handle_response(response)
+
+    def update_workspace(self, token, workspace_id, params):
+        response = self.put(f"/workspaces/{workspace_id}", data=params, token=token)
+        return self.handle_response(response)
+
+    def delete_workspace(self, token, workspace_id):
+        response = self.delete(f"/workspaces/{workspace_id}", token=token)
+        return self.handle_response(response)
+
     def create_section(self, token, params):
         response = self.post("/sections", data=params, token=token)
         return self.handle_response(response)

making_with_code_cli/settings.py

@@ -5,7 +5,7 @@ import os
 
 def get_settings_path(settings_path=None):
     """Determines the settings path, based on what is passed and environment.
-    "The default location for the MWC configuration file is ~/.mwc.
+    "The default location for the MWC configuration file is ~/.config/mwc/settings.toml.
     This can be specified (e.g. to support parallel installations) using
     the MWC_CONFIG environment variable or the --config flag.
     """
@@ -14,7 +14,7 @@ def get_settings_path(settings_path=None):
     elif "MWC_CONFIG" in os.environ:
         return Path(os.environ["MWC_CONFIG"])
     else:
-        xdg_config_home = os.environ.get("XDG_CONFIG_HOME", str(Path.home() / ".config"))
+        xdg_config_home = Path(os.environ.get("XDG_CONFIG_HOME", Path.home() / ".config"))
         return xdg_config_home / "mwc" / "settings.toml"
 
 def read_settings(settings_path=None):

making_with_code_cli/setup/__init__.py

@@ -4,7 +4,6 @@ from pprint import pformat
 import traceback
 from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
 from making_with_code_cli.git_backend import get_backend
-from making_with_code_cli.update import update
 from making_with_code_cli.decorators import handle_mwc_errors
 from making_with_code_cli.settings import (
     get_settings_path,
@@ -86,6 +85,8 @@ def setup(ctx, config, debug, mwc_accounts_url):
             token = prompt_mwc_password(settings['mwc_username'], api)
             settings['mwc_accounts_token'] = token
     settings['mwc_git_token'] = status['git_token']
+    if status.get('git_server_url'):
+        settings['git_server_url'] = status['git_server_url']
     settings['work_dir'] = str(choose_work_dir(settings.get("work_dir")).resolve())
     settings['editor'] = choose_editor(settings.get('editor', 'code'))
     if debug:
@@ -128,4 +129,5 @@ def setup(ctx, config, debug, mwc_accounts_url):
         for task in errors:
             click.echo(error(f"- {task.description}"))
     else:
+        from making_with_code_cli.update import update
         ctx.invoke(update, config=config)

making_with_code_cli/setup/tasks.py

@@ -403,10 +403,28 @@ class InstallVSCode(InstallPackage):
     brew_name = "visual-studio-code"
     brew_cask = True
 
+    def is_complete(self):
+        if Platform.detect() == Platform.MAC:
+            app_locations = [
+                Path("/Applications/Visual Studio Code.app"),
+                Path.home() / "Applications/Visual Studio Code.app",
+            ]
+            return self.executable_on_path("code") or any(p.exists() for p in app_locations)
+        return self.executable_on_path("code")
+
     def run_task(self):
         if Platform.detect() in (Platform.UBUNTU, Platform.WSL):
             run("sudo snap install --classic code", shell=True, check=True)
         else:
+            # If brew has a stale record of VS Code but the app is missing,
+            # the upgrade will fail. Zap the cask first to clear brew's state.
+            stale = run(
+                "brew list --cask visual-studio-code",
+                shell=True, capture_output=True
+            ).returncode == 0
+            if stale:
+                run("brew uninstall --cask --zap --force visual-studio-code",
+                    shell=True, check=True)
             super().run_task()
 
 class InstallImageMagick(InstallPackage):

making_with_code_cli/submit.py

@@ -3,16 +3,35 @@ from subprocess import run, CalledProcessError
 from making_with_code_cli.git_wrapper import (
     in_repo,
     repo_has_changes,
+    has_unpushed_commits,
+    abort_in_progress_operation,
+    recover_detached_head,
+    clear_stale_index_lock,
 )
 from making_with_code_cli.styles import (
     address,
-    question,
     info,
-    debug as debug_fmt,
     confirm,
+    debug as debug_fmt,
     error,
 )
 
+_NETWORK_PHRASES = [
+    "Could not resolve host",
+    "Connection timed out",
+    "unable to connect",
+    "Failed to connect",
+    "network is unreachable",
+]
+_AUTH_PHRASES = [
+    "Authentication failed",
+    "403",
+    "invalid credentials",
+    "could not read Username",
+    "terminal prompts disabled",
+]
+
+
 @click.command()
 def submit():
     """Submit your work.
@@ -21,17 +40,123 @@ def submit():
     if not in_repo():
         click.echo(error("You are not in a lab, problem set, or project folder."))
         return
+
+    _preflight_repo()
+    clear_stale_index_lock()
+
+    # S1: committed but not yet pushed — skip straight to push
+    if not repo_has_changes() and has_unpushed_commits():
+        click.echo(confirm(
+            "Your work was already saved locally but not yet sent to the server. "
+            "Submitting now..."
+        ))
+        _push()
+        click.echo(address("Nice job! All your work in this module has been submitted."))
+        return
+
     if not repo_has_changes():
         click.echo(info("Everything is already up to date."))
         return
+
     run("git add --all", shell=True, capture_output=True, check=True)
     run("git --no-pager diff --staged", shell=True, check=True)
+
     if not click.confirm(address("Here are the current changes. Looks OK?")):
+        run("git restore --staged .", shell=True, capture_output=True)
         click.echo(info("Cancelled the submit for now."))
         return
+
     click.echo(info("Write your commit message, then save and exit the window..."))
-    run("git commit", shell=True, capture_output=True, check=True)
-    run("git push", shell=True, capture_output=True, check=True)
+    commit_result = run("git commit", shell=True, capture_output=True, text=True)
+
+    if commit_result.returncode != 0:
+        combined = commit_result.stderr + commit_result.stdout
+        run("git restore --staged .", shell=True, capture_output=True)
+        # S2: student saved without writing a message
+        if "empty commit message" in combined or "Aborting commit" in combined:
+            click.echo(error(
+                "You didn't write a commit message. "
+                "Run `mwc submit` again and describe what you worked on before saving."
+            ))
+        else:
+            click.echo(debug_fmt(combined.strip(), preformatted=True))
+            click.echo(error(
+                "Could not save your work. "
+                "If you're not sure what to do, ask your teacher."
+            ))
+        return
+
+    _push()
     click.echo(address("Nice job! All your work in this module has been submitted."))
 
 
+def _preflight_repo():
+    """Abort any in-progress git operations and recover from detached HEAD."""
+    aborted = abort_in_progress_operation()
+    if aborted:
+        click.echo(confirm(
+            f"Your repo was in an incomplete state (interrupted {aborted}). "
+            "This has been cleared automatically."
+        ))
+    branch = recover_detached_head()
+    if branch:
+        click.echo(confirm(
+            f"Your repo was not on any branch. Switched back to '{branch}'."
+        ))
+
+
+def _push():
+    """Push to origin, with automatic recovery from non-fast-forward rejection."""
+    push_result = run("git push", shell=True, capture_output=True, text=True)
+
+    if push_result.returncode == 0:
+        return
+
+    combined = push_result.stderr + push_result.stdout
+
+    # S3: teacher pushed changes since the student last pulled
+    if "non-fast-forward" in combined or (
+        "rejected" in combined and "fetch first" in combined
+    ):
+        click.echo(confirm(
+            "Your teacher made some changes while you were working. "
+            "Merging them in and resubmitting..."
+        ))
+        # -X ours: student's committed code wins any content conflict
+        pull_result = run(
+            "git pull -X ours", shell=True, capture_output=True, text=True
+        )
+        if pull_result.returncode != 0:
+            click.echo(debug_fmt(pull_result.stderr.strip(), preformatted=True))
+            click.echo(error(
+                "Could not automatically merge your teacher's changes. "
+                "Ask your teacher for help."
+            ))
+            return
+        retry = run("git push", shell=True, capture_output=True, text=True)
+        if retry.returncode == 0:
+            return
+        combined = retry.stderr + retry.stdout
+
+    # S4: network error — commit is safe locally, tell them so
+    if any(p in combined for p in _NETWORK_PHRASES):
+        click.echo(error(
+            "Your work was saved locally but could not be sent to the server "
+            "(network error). Check your internet connection and run "
+            "`mwc submit` again — you won't lose any work."
+        ))
+        return
+
+    # S5: authentication error
+    if any(p in combined for p in _AUTH_PHRASES):
+        click.echo(error(
+            "Your login credentials are no longer valid. "
+            "Run `mwc setup` to refresh them, then run `mwc submit` again."
+        ))
+        return
+
+    click.echo(debug_fmt(combined.strip(), preformatted=True))
+    click.echo(error(
+        "Could not send your work to the server. "
+        "If the problem persists, ask your teacher for help."
+    ))

making_with_code_cli/teach/__init__.py

@@ -7,6 +7,7 @@ from making_with_code_cli.teach.patch import patch
 from making_with_code_cli.teach.check import check
 from making_with_code_cli.teach.section import section
 from making_with_code_cli.teach.student import student
+from making_with_code_cli.teach.workspace import workspace
 
 @click.group()
 def teach():
@@ -20,3 +21,4 @@ teach.add_command(patch)
 teach.add_command(check)
 teach.add_command(section)
 teach.add_command(student)
+teach.add_command(workspace)

making_with_code_cli/teach/section/edit.py

@@ -19,7 +19,7 @@ from making_with_code_cli.styles import (
 @click.option("--curriculum-site-url", help="URL for curriculum website. e.g. https://makingwithcode.org")
 @click.option("--course-name", help="MWC course name")
 @click.option("--code", help="Code students can use to join the section")
-@click.option("--roster", click.Path(exists=True), help="csv file containing student information")
+@click.option("--roster", type=click.Path(exists=True), help="csv file containing student information")
 @click.option('-r', "--remove", is_flag=True, help="When using a roster, remove existing students not on roster")
 @click.option('-f', "--force", is_flag=True, help="When using a roster, update existing student passwords")
 @handle_mwc_errors

making_with_code_cli/teach/setup.py

@@ -72,6 +72,8 @@ def setup(config, debug, mwc_accounts_url):
             token = prompt_mwc_password(settings['mwc_username'], api)
             settings['mwc_accounts_token'] = token
     settings['mwc_git_token'] = status['git_token']
+    if status.get('git_server_url'):
+        settings['git_server_url'] = status['git_server_url']
     settings['work_dir'] = str(choose_work_dir(settings.get("work_dir")))
     settings['editor'] = choose_editor(settings.get('editor', 'code'))
     settings['teacher_work_dir'] = str(choose_work_dir(

making_with_code_cli/teach/student/create.py

@@ -16,18 +16,39 @@ def create_student(username, password, section, config):
     "Create a student user"
     settings = read_settings(config)
     api = MWCAccountsAPI(settings.get('mwc_accounts_url'))
+    token = settings.get('mwc_accounts_token')
     params = {
-        "username": username, 
-        "password": password, 
+        "username": username,
+        "password": password,
         "section": section,
     }
     try:
-        response = api.create_student(settings.get('mwc_accounts_token'), params)
+        response = api.create_student(token, params)
         click.echo(info(f"Created student {response['username']} in {section}."))
     except api.RequestFailed as err:
         click.echo(error(f"Could not create {params['username']}:"))
         print(err)
         for field, problems in err.data.items():
             click.echo(error(f" - {field}: {'; '.join(problems)}"))
+        return
+
+    workspace_id = _get_section_workspace_id(api, token, section)
+    if workspace_id:
+        try:
+            api.sync_workspace(token, workspace_id)
+            click.echo(info(f"Provisioned {username} on workspace."))
+        except (api.RequestFailed, api.ServerError) as err:
+            click.echo(error(f"Could not provision {username} on workspace: {err}"))
+
+
+def _get_section_workspace_id(api, token, section_slug):
+    try:
+        roster = api.get_roster(token)
+        for sec in roster.get('teacher_sections', []):
+            if sec['slug'] == section_slug:
+                return sec.get('workspace_id')
+    except Exception:
+        pass
+    return None
 
 

making_with_code_cli/teach/workspace/__init__.py

@@ -0,0 +1,31 @@
+import click
+from tabulate import tabulate
+from making_with_code_cli.settings import read_settings
+from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
+from making_with_code_cli.styles import info, error, question
+from making_with_code_cli.teach.setup import check_required_teacher_settings
+from making_with_code_cli.teach.workspace.create import create_workspace
+from making_with_code_cli.teach.workspace.edit import edit_workspace
+from making_with_code_cli.teach.workspace.delete import delete_workspace
+
+
+@click.group(invoke_without_command=True)
+@click.pass_context
+@click.option("--config", help="Path to config file (default: ~/.mwc)")
+def workspace(ctx, config):
+    "Manage workspaces"
+    if ctx.invoked_subcommand is None:
+        settings = read_settings(config)
+        if not check_required_teacher_settings(settings):
+            return
+        api = MWCAccountsAPI(settings.get('mwc_accounts_url'))
+        workspaces = api.list_workspaces(settings['mwc_accounts_token'])
+        if workspaces:
+            click.echo(info(tabulate(workspaces, headers="keys"), preformatted=True))
+        else:
+            click.echo(info("You have no workspaces."))
+
+
+workspace.add_command(create_workspace)
+workspace.add_command(edit_workspace)
+workspace.add_command(delete_workspace)

making_with_code_cli/teach/workspace/create.py

@@ -0,0 +1,28 @@
+import click
+from tabulate import tabulate
+from making_with_code_cli.settings import read_settings
+from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
+from making_with_code_cli.styles import info, error, question
+
+
+@click.command('create')
+@click.option("--config", help="Path to config file (default: ~/.mwc)")
+@click.option("--name", help="Workspace name")
+@click.option("--url", help="JupyterHub URL (e.g. https://hub.example.org)")
+@click.option("--api-token", help="JupyterHub API token")
+def create_workspace(config, name, url, api_token):
+    "Create a new workspace"
+    settings = read_settings(config)
+    api = MWCAccountsAPI(settings.get('mwc_accounts_url'))
+    params = {
+        "name": name or click.prompt(question("Workspace name")),
+        "url": url or click.prompt(question("JupyterHub URL")),
+        "api_token": api_token or click.prompt(question("JupyterHub API token"), hide_input=True),
+    }
+    try:
+        response = api.create_workspace(settings.get('mwc_accounts_token'), params)
+        click.echo(info(tabulate([response], headers="keys"), preformatted=True))
+    except api.RequestFailed as err:
+        click.echo(error("Could not create workspace:"))
+        for field, problems in err.data.items():
+            click.echo(error(f" - {field}: {'; '.join(problems)}"))

making_with_code_cli/teach/workspace/delete.py

@@ -0,0 +1,21 @@
+import click
+from making_with_code_cli.settings import read_settings
+from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
+from making_with_code_cli.styles import info, error, confirm as confirm_fmt
+
+
+@click.command('delete')
+@click.argument("workspace_id", type=int)
+@click.option("--config", help="Path to config file (default: ~/.mwc)")
+@click.option("--yes", is_flag=True, help="Skip confirmation prompt")
+def delete_workspace(workspace_id, config, yes):
+    "Delete a workspace"
+    settings = read_settings(config)
+    api = MWCAccountsAPI(settings.get('mwc_accounts_url'))
+    if not yes:
+        click.confirm(confirm_fmt(f"Delete workspace {workspace_id}?"), abort=True)
+    try:
+        api.delete_workspace(settings.get('mwc_accounts_token'), workspace_id)
+        click.echo(info(f"Deleted workspace {workspace_id}."))
+    except api.RequestFailed as err:
+        click.echo(error(f"Could not delete workspace {workspace_id}."))

making_with_code_cli/teach/workspace/edit.py

@@ -0,0 +1,34 @@
+import click
+from tabulate import tabulate
+from making_with_code_cli.settings import read_settings
+from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
+from making_with_code_cli.styles import info, error, question
+
+
+@click.command('edit')
+@click.argument("workspace_id", type=int)
+@click.option("--config", help="Path to config file (default: ~/.mwc)")
+@click.option("--name", help="Workspace name")
+@click.option("--url", help="JupyterHub URL")
+@click.option("--api-token", help="JupyterHub API token")
+def edit_workspace(workspace_id, config, name, url, api_token):
+    "Edit a workspace"
+    settings = read_settings(config)
+    api = MWCAccountsAPI(settings.get('mwc_accounts_url'))
+    params = {}
+    if name:
+        params["name"] = name
+    if url:
+        params["url"] = url
+    if api_token:
+        params["api_token"] = api_token
+    if not params:
+        click.echo(error("No fields to update. Provide at least one option."))
+        return
+    try:
+        response = api.update_workspace(settings.get('mwc_accounts_token'), workspace_id, params)
+        click.echo(info(tabulate([response], headers="keys"), preformatted=True))
+    except api.RequestFailed as err:
+        click.echo(error("Could not update workspace:"))
+        for field, problems in err.data.items():
+            click.echo(error(f" - {field}: {'; '.join(problems)}"))

making_with_code_cli/update/__init__.py

@@ -7,6 +7,7 @@ from making_with_code_cli.curriculum import get_curriculum
 from making_with_code_cli.git_backend import get_backend
 from making_with_code_cli.mwc_accounts_api import MWCAccountsAPI
 from making_with_code_cli.setup.tasks import WORK_DIR_PERMISSIONS
+from making_with_code_cli.errors import GitError
 from making_with_code_cli.styles import (
     address,
     question,
@@ -61,8 +62,10 @@ def update(config, debug):
                 if module_dir.exists():
                     try:
                         git_backend.update(module, module_dir)
+                    except GitError as e:
+                        click.echo(error(str(e)))
                     except Exception as e:
-                        msg =  traceback.format_exception(type(e), e, e.__traceback__)
+                        msg = traceback.format_exception(type(e), e, e.__traceback__)
                         click.echo(error(''.join(msg), preformatted=True))
                 else:
                     rel_dir = module_dir.resolve().relative_to(mwc_home)
@@ -70,6 +73,8 @@ def update(config, debug):
                     click.echo(info(f"See {module['url']} for details."))
                     try:
                         git_backend.init_module(module, module_dir)
+                    except GitError as e:
+                        click.echo(error(str(e)))
                     except Exception as e:
-                        msg =  traceback.format_exception(type(e), e, e.__traceback__)
+                        msg = traceback.format_exception(type(e), e, e.__traceback__)
                         click.echo(error(''.join(msg), preformatted=True))

{making_with_code_cli-5.2.0.dist-info → making_with_code_cli-5.4.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: making-with-code-cli
-Version: 5.2.0
+Version: 5.4.0
 Summary: Courseware for Making With Code
 Author: Chris Proctor
 Author-email: Chris Proctor <chris@chrisproctor.net>
@@ -42,7 +42,7 @@ Making With Code, there is a separate bootstrapping process you should follow.
 Please consult your curriculum website (or ask your teacher) for detailed instructions 
 on how and why to use this tool. 
 
-    pip3 install making-with-code-cli
+    uv tool install making-with-code-cli
 
 Once installed, you can set up your computer using the `mwc setup` command. Run 
 `mwc --help` to see all the commands available.  

{making_with_code_cli-5.2.0.dist-info → making_with_code_cli-5.4.0.dist-info}/RECORD

@@ -1,20 +1,20 @@
 making_with_code_cli/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 making_with_code_cli/cli.py,sha256=KuSWKXYKK651tIXpeZ3nIVynRATal8NV-jSArYXEbo0,443
-making_with_code_cli/curriculum.py,sha256=gp0ValDb3ZfKfHV8B33-i16Q7u49PZfWpNFRFgQO2vQ,1180
+making_with_code_cli/curriculum.py,sha256=LewUkk2BHZsQhbZMXi3JiYm1ROcxpQL1u-nsd43x_hw,1192
 making_with_code_cli/decorators.py,sha256=d-fLMfi2BX6jaLAqaHbQFBUGyHaCijqq6EpSCAl3YCk,583
-making_with_code_cli/errors.py,sha256=N-i-aurPrqyo0ORBKNpMDxl2woyFAUZEDrMO1vJ5iF4,990
+making_with_code_cli/errors.py,sha256=V0JfwUPMEoU16cDxJh_SR-xQbLEtA5EwsV9RoI6l6C0,1152
 making_with_code_cli/git_backend/__init__.py,sha256=UleXH-1HbEglNL8y_d7-JpQfLRDZ2pBpLYzy9-eH_k0,112
-making_with_code_cli/git_backend/base_backend.py,sha256=Y9n-fn2o6tsMSBy-72_0e-g59rBNZO1zHxToeLOcLUw,907
-making_with_code_cli/git_backend/mwc_backend.py,sha256=wXlM8PyEtfdXB2DBBF8cQSPD1UtFpkYTAdWqsBCVvMg,6430
-making_with_code_cli/git_wrapper.py,sha256=qSkFdZG2OAww_-eBzAg3tYJODdHS91tEmvRJtYim4b4,727
+making_with_code_cli/git_backend/base_backend.py,sha256=zxk3PxI6y35s2pYh1nl5Rz9c2S9BPF2M6HSge63WGyo,841
+making_with_code_cli/git_backend/mwc_backend.py,sha256=O5_cz3eEK90yc4h5So9zmSGuCcEdCM2NKHdsClbVcQI,11358
+making_with_code_cli/git_wrapper.py,sha256=M2J5k9HheNPd91GR-sGaWiYgTnG9vNfjxSbEGswSjdo,3898
 making_with_code_cli/helpers.py,sha256=h4-gkT0y61GGzV08xcaEqLGJ9N9IQPEMa2P_7_g2sY0,1167
-making_with_code_cli/mwc_accounts_api.py,sha256=y0B--ybCdTv7mZN12pnK30hdbYPsFy3qV1hXmO7vEYY,3792
-making_with_code_cli/settings.py,sha256=nj0JILi8J6h28nrgBI7dVJ-XjzXepqiDnGLwfCOBbYI,1787
-making_with_code_cli/setup/__init__.py,sha256=yHUKuOUTKdUVkiwDFoM8F9O48w0Gpy0hpou4rkroiCY,4426
-making_with_code_cli/setup/tasks.py,sha256=ztlslomK6MkhXhVjCklj0GVW_zLNUpudmuE-qVufU_8,15536
+making_with_code_cli/mwc_accounts_api.py,sha256=INq0ezDY3bDYTLt7cAFp5JmJ4rkr9LcDnz6YYp5d-VQ,4641
+making_with_code_cli/settings.py,sha256=c_sWjBgjrugXx-11rWOOVKFcxiU5WfFENR0WhZs7LLI,1809
+making_with_code_cli/setup/__init__.py,sha256=2mABWsAEhf7rx9Yx3WfxAOMce1A-AYQuByIWZboZaI0,4533
+making_with_code_cli/setup/tasks.py,sha256=jTlcDiviuzXyhIcdYhGhWLXH8zVvKqW_7i79GXZOtNg,16382
 making_with_code_cli/styles.py,sha256=ZKbs-Grq1aF3wV8scwQxMO1UjnphuIccdNzksLxAo_A,910
-making_with_code_cli/submit.py,sha256=gWnueoyvZVdJdSLxOBSS0Qvx_8OdQB5VonQ1p7hjYSA,1207
-making_with_code_cli/teach/__init__.py,sha256=mK2cswI1NiVULx3WszZmSyn7HComDDcHvyqL2Y8defI,691
+making_with_code_cli/submit.py,sha256=3rit0F9ott27JezXhujYBUJSOopRJ_xEcm1oACigP88,5496
+making_with_code_cli/teach/__init__.py,sha256=qra5g986DFipl0GkYOW3Nnoi1QUGpatS_U9eFoVVsgw,779
 making_with_code_cli/teach/assess.py,sha256=2--XhfMRsyMoEaP7EAeRD4uk5tynB3wLE3KE5pdYC30,1625
 making_with_code_cli/teach/check/__init__.py,sha256=wlUNHdHI5NwUbGGTHCM87xdJ4Ts3bwMfBF8wXoK2Z68,1787
 making_with_code_cli/teach/check/check_module.py,sha256=TBtebZoeYSqPLOiMo3prNAXZZFbs1UywN-_XVOPQ4Co,3215
@@ -25,20 +25,24 @@ making_with_code_cli/teach/patch.py,sha256=icuMU0EM8RZb3w1RU3HLDyN_CLSh2HiYRC0OQ
 making_with_code_cli/teach/section/__init__.py,sha256=gHq-y_vRP0Q2sV5j58QJ0h_jU4oOjv26VnxLtFA5g3E,2294
 making_with_code_cli/teach/section/create.py,sha256=jC-4CKsZ5I-v_2xtXq3nX-2vSXv1tMDo1BqNGs5afgM,3745
 making_with_code_cli/teach/section/delete.py,sha256=Z9uljVCqaKDw4-SzoAl-jxs4SHCB3Q8fKZVXNLHHzQ8,773
-making_with_code_cli/teach/section/edit.py,sha256=mp97mj5x3OGEN5diix316cFF95b-P9l4SXkvi-UTwwQ,2152
+making_with_code_cli/teach/section/edit.py,sha256=OKub7wJ7yINBw4qkf-L_urrkB4nlXRauIcNbxuLTjGk,2157
 making_with_code_cli/teach/section/show.py,sha256=YZ1fJr3LSQl0E9hgkV9-gndyYn0FnUbPa5ChO2CWbb8,2260
-making_with_code_cli/teach/setup.py,sha256=bK7nxH748rMripUUIqpYJBc54kEOomsV4uF3lB4eXfI,3050
+making_with_code_cli/teach/setup.py,sha256=sF6j-aPOsQ4vJwOb7I_t57LZF5dDmOib15hpET9HAsI,3149
 making_with_code_cli/teach/status.py,sha256=HIMdHCG0JUrrQBu9EDKxOrg52_zYUNj1w6BfFYonds8,4520
 making_with_code_cli/teach/student/__init__.py,sha256=VamqMDxH-ZuGZWTR5g8jYQ3XkgWYQqmunVorpZ00fbM,402
-making_with_code_cli/teach/student/create.py,sha256=JJ9tjcIgYfEkQyzUqG-cBemSnnrxyZdqLfMt4M2zWWs,1119
+making_with_code_cli/teach/student/create.py,sha256=Oa78OSUyeDGepjgg6ivycTpzYQl70anc6PEXrvT2s_s,1822
 making_with_code_cli/teach/student/edit.py,sha256=EhP-x7QNXFDh4YGnW8CNjggDbUOAyfZqxa9ZM2gpffY,1257
 making_with_code_cli/teach/student/remove.py,sha256=z6Y_n1Q53qGeauSCaqzjO852pRkMDoioVoPS1TChMcM,1300
 making_with_code_cli/teach/student_repo_functions.py,sha256=-Qm-SgkiCW_d1jnuNi5I5B3YBeWqBybqrL5sgXwQ77Q,3633
 making_with_code_cli/teach/student_repos.py,sha256=TRUxpgyrGjsiY58dIpPLGOzytw9sL9zgS-kUquXd39Q,2614
 making_with_code_cli/teach/update.py,sha256=bhGpZudmTrH0pSq-hLfmKOAgMBsmMZG-nfB7zA79tkg,1860
-making_with_code_cli/update/__init__.py,sha256=5qtyD_YN1_sTsJ2g0ayIFtpXFZommXGJBetRs9wyW0g,3506
+making_with_code_cli/teach/workspace/__init__.py,sha256=FDe1ggbBYVC5EWH1NBjWdyGgxqjlSQxf_iKDDYqyOgA,1298
+making_with_code_cli/teach/workspace/create.py,sha256=t0hIwJwU7ZEF3QJl0Cwbve3jD1ilYex680dRCvhbkr0,1320
+making_with_code_cli/teach/workspace/delete.py,sha256=126VQaFMvulpXY_ch1yXNN4fCLL_bQfAnkRwnOL9PYk,952
+making_with_code_cli/teach/workspace/edit.py,sha256=BndmJrItJMB3JzW6MNCfqyLPTV6UaPh2T6hQa73TcJQ,1369
+making_with_code_cli/update/__init__.py,sha256=13Sy6mYojZANKF6YOB5GkWvxwbPLWaZfvjCfqk9Nztw,3737
 making_with_code_cli/version.py,sha256=4t7TOR5OlBrLFy7jrtS_d6FtIReco1NDdVPQoCb6Jx8,286
-making_with_code_cli-5.2.0.dist-info/WHEEL,sha256=iHtWm8nRfs0VRdCYVXocAWFW8ppjHL-uTJkAdZJKOBM,80
-making_with_code_cli-5.2.0.dist-info/entry_points.txt,sha256=0nHqTE6I8cKRoVK_tbRNYI9nY0GFYLDJDZemvDqIHeM,54
-making_with_code_cli-5.2.0.dist-info/METADATA,sha256=-8X_swu4XRYGtkeo9vJJcKT51QkdiOpOxgC8h0f3GtA,2318
-making_with_code_cli-5.2.0.dist-info/RECORD,,
+making_with_code_cli-5.4.0.dist-info/WHEEL,sha256=Qb5DWjqM6GZuPp3VmTlAFSGqNRK8vceyVqRFxrfa8YA,80
+making_with_code_cli-5.4.0.dist-info/entry_points.txt,sha256=0nHqTE6I8cKRoVK_tbRNYI9nY0GFYLDJDZemvDqIHeM,54
+making_with_code_cli-5.4.0.dist-info/METADATA,sha256=Z7qbd43JmUV2Aw75kw9Yy146keZY6Zx7wxporkZwra4,2321
+making_with_code_cli-5.4.0.dist-info/RECORD,,

{making_with_code_cli-5.2.0.dist-info → making_with_code_cli-5.4.0.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: uv 0.9.30
+Generator: uv 0.11.4
 Root-Is-Purelib: true
 Tag: py3-none-any