moodle-study-kit 0.2.2__tar.gz → 0.2.4__tar.gz

{moodle_study_kit-0.2.2/src/moodle_study_kit.egg-info → moodle_study_kit-0.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: moodle-study-kit
-Version: 0.2.2
+Version: 0.2.4
 Summary: Read-only Moodle toolkit for agents, MCP servers, and local scripts
 License-Expression: MIT
 Keywords: moodle,lms,education,study,mcp
@@ -15,14 +15,12 @@ Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: platformdirs>=2.10
+Requires-Dist: pymupdf>=1.23
 Requires-Dist: rich>=13.0
-Provides-Extra: pdf
-Requires-Dist: pymupdf>=1.23; extra == "pdf"
 Provides-Extra: mcp
 Requires-Dist: mcp>=1.0; extra == "mcp"
 Provides-Extra: dev
 Requires-Dist: pytest>=7.0; extra == "dev"
-Requires-Dist: pymupdf>=1.23; extra == "dev"
 Dynamic: license-file
 
 # moodle-study-kit

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "moodle-study-kit"
-version = "0.2.2"
+version = "0.2.4"
 description = "Read-only Moodle toolkit for agents, MCP servers, and local scripts"
 requires-python = ">=3.10"
 license = "MIT"
@@ -19,12 +19,11 @@ classifiers = [
     "Programming Language :: Python :: 3.13",
     "Topic :: Education",
 ]
-dependencies = ["platformdirs>=2.10", "rich>=13.0"]
+dependencies = ["platformdirs>=2.10", "pymupdf>=1.23", "rich>=13.0"]
 
 [project.optional-dependencies]
-pdf = ["pymupdf>=1.23"]
 mcp = ["mcp>=1.0"]
-dev = ["pytest>=7.0", "pymupdf>=1.23"]
+dev = ["pytest>=7.0"]
 
 [project.scripts]
 moodle-study = "moodle_study_kit.cli:main"

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/src/moodle_study_kit/__init__.py

@@ -1,6 +1,6 @@
 """moodle-study-kit — Read-only Moodle toolkit."""
 
-__version__ = "0.2.2"
+__version__ = "0.2.4"
 
 from moodle_study_kit.client import (
     ALLOWED_FUNCTIONS,

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/src/moodle_study_kit/cli.py

@@ -412,9 +412,11 @@ def cmd_materials(args: argparse.Namespace) -> None:
     sections = client.get_course_contents(course_id)
 
     if args.week is not None:
-        materials = get_section_materials(sections, course_id, args.week)
+        materials = get_section_materials(
+            sections, course_id, args.week, base_url=client.base_url,
+        )
     else:
-        materials = extract_materials(sections, course_id)
+        materials = extract_materials(sections, course_id, base_url=client.base_url)
 
     if args.json:
         print(json.dumps(materials, indent=2))

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/src/moodle_study_kit/materials.py

@@ -74,12 +74,17 @@ def _build_material(
     section: dict,
     module: dict,
     file_info: dict | None = None,
+    *,
+    base_url: str = "",
 ) -> dict:
     """Build one structured material record.
 
     If *file_info* is provided the record describes a specific file
     inside a module's ``contents`` array.  Otherwise it describes the
     module itself (e.g. a URL resource with no file list).
+
+    When *base_url* is given, a ``view_url`` key is added pointing to
+    the human-readable Moodle page for the module.
     """
     module_id = module.get("id", 0)
 
@@ -98,7 +103,7 @@ def _build_material(
         time_modified = module.get("added", 0)
         dedupe_key = f"{module_id}:"
 
-    return {
+    record = {
         "course_id": course_id,
         "section_id": section.get("id", 0),
         "section_name": section.get("name", ""),
@@ -114,6 +119,14 @@ def _build_material(
         "dedupe_key": dedupe_key,
     }
 
+    if base_url:
+        modname = module.get("modname", "")
+        record["view_url"] = (
+            f"{base_url}/mod/{modname}/view.php?id={module_id}"
+        )
+
+    return record
+
 
 # ── Extraction helpers ───────────────────────────────────────────
 
@@ -122,6 +135,7 @@ def extract_materials(
     sections: list[dict],
     course_id: int,
     *,
+    base_url: str = "",
     dedupe: bool = True,
 ) -> list[dict]:
     """Extract material records from Moodle course sections.
@@ -132,6 +146,10 @@ def extract_materials(
         Raw sections from ``core_course_get_contents``.
     course_id : int
         Included in every record for cross-course context.
+    base_url : str
+        Moodle site root (e.g. ``https://moodle.example.com``).
+        When provided, each record includes a ``view_url`` pointing
+        to the human-readable module page.
     dedupe : bool
         Suppress duplicates sharing the same ``dedupe_key``.
 
@@ -154,7 +172,10 @@ def extract_materials(
                     if file_info.get("type", "") not in ("file", "url"):
                         continue
 
-                    record = _build_material(course_id, section, module, file_info)
+                    record = _build_material(
+                        course_id, section, module, file_info,
+                        base_url=base_url,
+                    )
 
                     if dedupe and record["dedupe_key"] in seen:
                         continue
@@ -163,7 +184,10 @@ def extract_materials(
 
             elif module.get("modname", "") in ("url", "page"):
                 # Module has no contents array but is itself a link or page.
-                record = _build_material(course_id, section, module)
+                record = _build_material(
+                    course_id, section, module,
+                    base_url=base_url,
+                )
 
                 if dedupe and record["dedupe_key"] in seen:
                     continue
@@ -178,6 +202,7 @@ def get_section_materials(
     course_id: int,
     week: int | str,
     *,
+    base_url: str = "",
     dedupe: bool = True,
 ) -> list[dict]:
     """Get materials for sections matching *week*.
@@ -185,7 +210,7 @@ def get_section_materials(
     Combines :func:`match_section` filtering with :func:`extract_materials`.
     """
     matching = [s for s in sections if match_section(s, week)]
-    return extract_materials(matching, course_id, dedupe=dedupe)
+    return extract_materials(matching, course_id, base_url=base_url, dedupe=dedupe)
 
 
 def search_course_content(

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/src/moodle_study_kit/mcp_server.py

@@ -92,8 +92,9 @@ def get_week_materials(course_id: int, week: str) -> str:
         (e.g. "Revision"). Numeric matching is based on explicit week-like
         labels in section names, not Moodle's raw section slot number.
     """
-    sections = _get_client().get_course_contents(course_id)
-    materials = get_section_materials(sections, course_id, week)
+    client = _get_client()
+    sections = client.get_course_contents(course_id)
+    materials = get_section_materials(sections, course_id, week, base_url=client.base_url)
     return json.dumps(materials, indent=2)
 
 

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4/src/moodle_study_kit.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: moodle-study-kit
-Version: 0.2.2
+Version: 0.2.4
 Summary: Read-only Moodle toolkit for agents, MCP servers, and local scripts
 License-Expression: MIT
 Keywords: moodle,lms,education,study,mcp
@@ -15,14 +15,12 @@ Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: platformdirs>=2.10
+Requires-Dist: pymupdf>=1.23
 Requires-Dist: rich>=13.0
-Provides-Extra: pdf
-Requires-Dist: pymupdf>=1.23; extra == "pdf"
 Provides-Extra: mcp
 Requires-Dist: mcp>=1.0; extra == "mcp"
 Provides-Extra: dev
 Requires-Dist: pytest>=7.0; extra == "dev"
-Requires-Dist: pymupdf>=1.23; extra == "dev"
 Dynamic: license-file
 
 # moodle-study-kit

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/src/moodle_study_kit.egg-info/requires.txt

@@ -1,12 +1,9 @@
 platformdirs>=2.10
+pymupdf>=1.23
 rich>=13.0
 
 [dev]
 pytest>=7.0
-pymupdf>=1.23
 
 [mcp]
 mcp>=1.0
-
-[pdf]
-pymupdf>=1.23

{moodle_study_kit-0.2.2 → moodle_study_kit-0.2.4}/tests/test_materials.py

@@ -251,6 +251,22 @@ class TestExtractMaterials:
         assert rec["mime_type"] == "application/pdf"
         assert rec["dedupe_key"] == "1010:week1_slides.pdf"
 
+    def test_no_view_url_without_base_url(self):
+        materials = extract_materials(SAMPLE_SECTIONS, COURSE_ID)
+        assert "view_url" not in materials[0]
+
+    def test_view_url_with_base_url(self):
+        base = "https://moodle.example.com"
+        materials = extract_materials(SAMPLE_SECTIONS, COURSE_ID, base_url=base)
+        rec = next(m for m in materials if m["file_name"] == "week1_slides.pdf")
+        assert rec["view_url"] == f"{base}/mod/resource/view.php?id=1010"
+
+    def test_view_url_for_url_module(self):
+        base = "https://moodle.example.com"
+        materials = extract_materials(SAMPLE_SECTIONS, COURSE_ID, base_url=base)
+        url_rec = next(m for m in materials if m["module_name"] == "Useful Links")
+        assert url_rec["view_url"] == f"{base}/mod/url/view.php?id=1002"
+
     def test_dedupe_suppresses_duplicates(self):
         """Same module+filename appearing twice should be deduped."""
         dup_file = _make_file("dup.pdf")
@@ -323,6 +339,11 @@ class TestGetSectionMaterials:
         assert "Week 10: Revision" not in names_1
         assert "Week 10: Revision" in names_10
 
+    def test_base_url_propagated(self):
+        base = "https://moodle.example.com"
+        mats = get_section_materials(SAMPLE_SECTIONS, COURSE_ID, 1, base_url=base)
+        assert all("view_url" in m for m in mats)
+
 
 # ── search_course_content ────────────────────────────────────────