Implement version fetching

Use the version explicitly to get the necessary versions

Author: woojiahao

app/commands/setup_folder.py

@@ -6,7 +6,7 @@
 from app.commands.check.git import git
 from app.commands.progress.constants import PROGRESS_LOCAL_FOLDER_NAME
 from app.utils.click import error, info, invoke_command, prompt
-from app.utils.version import get_latest_exercise_version
+from app.utils.version import get_latest_release_exercise_version
 
 
 @click.command("setup")
@@ -38,7 +38,7 @@ def setup() -> None:
     info("Setting up your local progress tracker...")
     os.makedirs(PROGRESS_LOCAL_FOLDER_NAME, exist_ok=True)
     with open(".gitmastery.json", "w") as gitmastery_file:
-        version_to_pin = get_latest_exercise_version()
+        version_to_pin = get_latest_release_exercise_version()
         if version_to_pin is None:
             # For now, we just error out because we should never be in this bad state.
             raise ValueError(

app/utils/click.py

@@ -1,7 +1,7 @@
 import logging
 import sys
 from enum import StrEnum
-from typing import Any, Dict, NoReturn, Optional
+from typing import Any, NoReturn, Optional
 
 import click
 

app/utils/gitmastery.py

@@ -2,7 +2,6 @@
 import os
 import sys
 import tempfile
-import urllib.parse
 from pathlib import Path
 from typing import Any, Dict, Optional, Tuple, TypeVar, Union
 
@@ -13,7 +12,10 @@
 from app.gitmastery_config import GitMasteryConfig
 from app.utils.click import error, get_exercise_root_config, get_gitmastery_root_config
 from app.utils.general import ensure_str
-from app.utils.version import Version, get_latest_exercise_version_tag
+from app.utils.version import (
+    Version,
+    get_latest_exercise_version_within_pin,
+)
 
 GITMASTERY_CONFIG_NAME = ".gitmastery.json"
 GITMASTERY_EXERCISE_CONFIG_NAME = ".gitmastery-exercise.json"
@@ -22,6 +24,27 @@
 )
 
 
+def _construct_gitmastery_exercises_url(filepath: str, version: Version) -> str:
+    if version.release:
+        ref = "heads/main"
+    elif version.development:
+        latest_development = get_latest_exercise_version_within_pin(version)
+        ref = f"tags/{latest_development}"
+    elif not version.pinned:
+        ref = f"tags/{version.to_version_string()}"
+    else:
+        # If pinned, we need to basically search for all the available tags within the
+        # range
+        latest_within_pin = get_latest_exercise_version_within_pin(version)
+        ref = f"tags/{latest_within_pin}"
+
+    url = (
+        f"https://raw.githubusercontent.com/git-mastery/exercises/refs/{ref}/{filepath}"
+    )
+    print(url)
+    return url
+
+
 def _find_root(filename: str) -> Optional[Tuple[Path, int]]:
     current = Path.cwd()
     steps = 0
@@ -68,8 +91,8 @@ def read_gitmastery_config(gitmastery_config_path: Path, cds: int) -> GitMastery
         cds=cds,
         progress_local=raw_config.get("progress_local", False),
         progress_remote=raw_config.get("progress_remote", False),
-        exercises_version=raw_config.get(
-            "exercises_version", Version.parse_version_string("latest")
+        exercises_version=Version.parse_version_string(
+            raw_config.get("exercises_version", "release")
         ),
     )
 
@@ -124,7 +147,12 @@ def generate_cds_string(cds: int) -> str:
 
 
 def get_gitmastery_file_path(path: str):
-    return urllib.parse.urljoin(GITMASTERY_EXERCISES_BASE_URL, path)
+    config = get_gitmastery_root_config()
+    if config is None:
+        version = Version.RELEASE
+    else:
+        version = config.exercises_version
+    return _construct_gitmastery_exercises_url(path, version)
 
 
 def fetch_file_contents(url: str, is_binary: bool) -> str | bytes:
@@ -138,7 +166,6 @@ def fetch_file_contents(url: str, is_binary: bool) -> str | bytes:
         error(
             f"Failed to fetch resource {click.style(url, bold=True, italic=True)}. Inform the Git-Mastery team."
         )
-    return ""
 
 
 def fetch_file_contents_or_none(

app/utils/version.py

@@ -173,7 +173,7 @@ def get_all_exercise_tags() -> List[Version]:
     )
 
 
-def get_latest_exercise_version() -> Optional[Version]:
+def get_latest_release_exercise_version() -> Optional[Version]:
     all_tags = get_all_exercise_tags()
     if len(all_tags) == 0:
         # Although this should not be happening, we will let the callsite handle this
@@ -188,6 +188,18 @@ def get_latest_exercise_version() -> Optional[Version]:
     return None
 
 
+def get_latest_development_exercise_version() -> Optional[Version]:
+    all_tags = get_all_exercise_tags()
+    if len(all_tags) == 0:
+        # Although this should not be happening, we will let the callsite handle this
+        return None
+
+    for tag in all_tags:
+        if tag.build is None:
+            return tag
+    return None
+
+
 def get_latest_exercise_version_within_pin(pin_version: Version) -> Optional[Version]:
     all_tags = get_all_exercise_tags()
     for tag in all_tags:

tests/utils/test_version.py

@@ -4,7 +4,7 @@
 import pytest
 from app.utils.version import (
     Version,
-    get_latest_exercise_version,
+    get_latest_release_exercise_version,
     get_latest_exercise_version_within_pin,
 )
 
@@ -154,7 +154,7 @@ def test_within_pin_with_prerelease(version, expected):
     assert version_v.within_pin(pin_version) == expected
 
 
-def test_get_latest_exercise_version():
+def test_get_latest_release_exercise_version():
     exercises = [
         "0.0.1-beta.0",
         "0.9.9",
@@ -166,7 +166,7 @@ def test_get_latest_exercise_version():
     with mock.patch(
         "app.utils.version.get_all_exercise_tags", return_value=_get_versions(exercises)
     ):
-        assert get_latest_exercise_version() == Version(
+        assert get_latest_release_exercise_version() == Version(
             0, 9, 9, None, None, False, False, False
         )