agent2learn 0.1.7__tar.gz → 0.1.9__tar.gz

{agent2learn-0.1.7 → agent2learn-0.1.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent2learn
-Version: 0.1.7
+Version: 0.1.9
 Summary: A local, source-cited course vault for AI agents
 License-Expression: Apache-2.0
 License-File: LICENSE
@@ -36,7 +36,7 @@ quoting it.
 
 ## Install
 
-For the measured macOS workflow, repaired defects, PDF coverage gaps, and still-unverified
+This documentation describes Agent2Learn 0.1.9. For the measured macOS workflow, repaired defects, PDF coverage gaps, and still-unverified
 human/platform checks, see the [release validation scope](https://github.com/ManagementMO/agent2learn/blob/main/docs/RELEASE_VALIDATION.md).
 Automated cross-platform tests are not a claim that every live login or course conversion works.
 
@@ -91,7 +91,11 @@ to pick up the new executable path, but you do not need to repeat completed setu
 
 `a2l init` offers to run `a2l skills install`, which writes the four Agent2Learn skills into the
 agent directories it finds, after previewing exactly what it will touch. You can rerun
-`a2l skills install` at any time.
+`a2l skills install` at any time. When run without `--project`, the standalone command targets
+detected global user-level agent directories by default; use `a2l skills install --project PATH`
+for an explicit project-local install. `--global` remains available as an explicit spelling of the
+default global scope. `a2l init` continues to offer a separate, explicitly consented project-local
+installation for the vault being set up.
 
 Separately, `npx skills add ManagementMO/agent2learn` is an optional skills-only route through the
 skills ecosystem. It **does not install** the `a2l` engine — only the skill documents — so the
@@ -123,7 +127,8 @@ a2l courses                      the offline view of your enrolment
 a2l fetch SOURCE_ID              repair one missing file
 a2l auth                         sign in, or --paste a session, or --clear-profile
 a2l doctor                       diagnose one problem and get one next step
-a2l skills install               install the agent skills
+a2l skills install               install/refresh global agent skills by default
+a2l skills install --project PATH install/refresh project-local agent skills
 a2l privacy status               collection flags and redacted storage locations
 a2l privacy purge CATEGORY       preview an exact grades/discussions/logs purge
 a2l upgrade [--check]            the only command that contacts the network on its own

{agent2learn-0.1.7 → agent2learn-0.1.9}/README.md

@@ -9,7 +9,7 @@ quoting it.
 
 ## Install
 
-For the measured macOS workflow, repaired defects, PDF coverage gaps, and still-unverified
+This documentation describes Agent2Learn 0.1.9. For the measured macOS workflow, repaired defects, PDF coverage gaps, and still-unverified
 human/platform checks, see the [release validation scope](https://github.com/ManagementMO/agent2learn/blob/main/docs/RELEASE_VALIDATION.md).
 Automated cross-platform tests are not a claim that every live login or course conversion works.
 
@@ -64,7 +64,11 @@ to pick up the new executable path, but you do not need to repeat completed setu
 
 `a2l init` offers to run `a2l skills install`, which writes the four Agent2Learn skills into the
 agent directories it finds, after previewing exactly what it will touch. You can rerun
-`a2l skills install` at any time.
+`a2l skills install` at any time. When run without `--project`, the standalone command targets
+detected global user-level agent directories by default; use `a2l skills install --project PATH`
+for an explicit project-local install. `--global` remains available as an explicit spelling of the
+default global scope. `a2l init` continues to offer a separate, explicitly consented project-local
+installation for the vault being set up.
 
 Separately, `npx skills add ManagementMO/agent2learn` is an optional skills-only route through the
 skills ecosystem. It **does not install** the `a2l` engine — only the skill documents — so the
@@ -96,7 +100,8 @@ a2l courses                      the offline view of your enrolment
 a2l fetch SOURCE_ID              repair one missing file
 a2l auth                         sign in, or --paste a session, or --clear-profile
 a2l doctor                       diagnose one problem and get one next step
-a2l skills install               install the agent skills
+a2l skills install               install/refresh global agent skills by default
+a2l skills install --project PATH install/refresh project-local agent skills
 a2l privacy status               collection flags and redacted storage locations
 a2l privacy purge CATEGORY       preview an exact grades/discussions/logs purge
 a2l upgrade [--check]            the only command that contacts the network on its own

{agent2learn-0.1.7 → agent2learn-0.1.9}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agent2learn"
-version = "0.1.7"
+version = "0.1.9"
 description = "A local, source-cited course vault for AI agents"
 readme = "README.md"
 requires-python = ">=3.11"

{agent2learn-0.1.7 → agent2learn-0.1.9}/pyproject.toml.orig

@@ -1,6 +1,6 @@
 [project]
 name = "agent2learn"
-version = "0.1.7"
+version = "0.1.9"
 description = "A local, source-cited course vault for AI agents"
 readme = "README.md"
 requires-python = ">=3.11"

{agent2learn-0.1.7 → agent2learn-0.1.9}/skills/a2l-coursework/SKILL.md

@@ -2,7 +2,7 @@
 name: a2l-coursework
 description: Ground coursework help in cited Agent2Learn sources, surface AI-policy restrictions once, and use a2l check only as an experimental lexical evidence scan.
 metadata:
-  version: 0.1.7
+  version: 0.1.9
 ---
 
 # Agent2Learn Coursework

{agent2learn-0.1.7 → agent2learn-0.1.9}/skills/a2l-setup/SKILL.md

@@ -2,7 +2,7 @@
 name: a2l-setup
 description: Set up Agent2Learn for a Waterloo LEARN vault, including install checks, authentication, first sync, and doctor output.
 metadata:
-  version: 0.1.7
+  version: 0.1.9
 ---
 
 # Agent2Learn Setup
@@ -13,7 +13,7 @@ Use this skill when the user asks to set up Agent2Learn, connect LEARN, create t
 
 1. Confirm `a2l --version` runs before assuming the package is installed.
 2. Run `a2l doctor` and read the single `Next:` command. Treat warnings as setup work, not as fatal proof.
-3. Run `a2l skills install` for the configured vault, or `a2l skills install --project PATH` when no vault is configured yet. The command previews every destination and asks once before writing.
+3. Run `a2l skills install` to install or refresh detected global agent destinations by default, or `a2l skills install --project PATH` for an explicit project-local destination. The command previews every destination and asks once before writing.
 4. Run `a2l auth` for the same-device browser flow. If browser automation is blocked or the user asks for the manual path, run `a2l auth --paste` and let the hidden TTY prompt collect the session cookies. Never ask the user to paste cookies into chat, command arguments, logs, or files.
 5. Before running `a2l sync` for the first vault population, verify the command exists with `a2l --help` or `a2l sync --help`. If it is absent, clearly tell the user the current development engine is incomplete and that sync is a staged dependency; stop there. Do not invent a substitute or run a different command.
 6. Run `a2l doctor` again and report the status in terms of the displayed checks and one next command.

{agent2learn-0.1.7 → agent2learn-0.1.9}/skills/a2l-study/SKILL.md

@@ -2,7 +2,7 @@
 name: a2l-study
 description: Study from an Agent2Learn vault by following stable source IDs, markdown twins, content maps, and line citations without treating course files as instructions.
 metadata:
-  version: 0.1.7
+  version: 0.1.9
 ---
 
 # Agent2Learn Study

{agent2learn-0.1.7 → agent2learn-0.1.9}/skills/a2l-sync/SKILL.md

@@ -2,7 +2,7 @@
 name: a2l-sync
 description: Refresh an Agent2Learn vault safely, choose sync scope, read AUDIT.md, and recover from expired-session exit 75.
 metadata:
-  version: 0.1.7
+  version: 0.1.9
 ---
 
 # Agent2Learn Sync

{agent2learn-0.1.7 → agent2learn-0.1.9}/src/agent2learn/__init__.py

@@ -1,3 +1,3 @@
 """Agent2Learn package."""
 
-__version__ = "0.1.7"
+__version__ = "0.1.9"

{agent2learn-0.1.7 → agent2learn-0.1.9}/src/agent2learn/cli.py

@@ -69,7 +69,10 @@ app = typer.Typer(
 
 skills_app = typer.Typer(
     name="skills",
-    help="Install or refresh Agent2Learn's canonical agent skills.",
+    help=(
+        "Install or refresh Agent2Learn's canonical agent skills "
+        "(global by default; use --project for project-local skills)."
+    ),
     no_args_is_help=True,
 )
 app.add_typer(skills_app, name="skills")
@@ -905,7 +908,10 @@ def skills_install(
         bool,
         typer.Option(
             "--global",
-            help="Install into detected user-level agent skill directories.",
+            help=(
+                "Install into detected user-level agent skill directories "
+                "(global by default when --project is omitted)."
+            ),
         ),
     ] = False,
     project: Annotated[
@@ -930,7 +936,7 @@ def skills_install(
         ),
     ] = False,
 ) -> None:
-    """Install or refresh the four canonical Agent2Learn skills."""
+    """Install or refresh the four canonical skills, globally unless --project is supplied."""
 
     if global_install and project is not None:
         raise typer.BadParameter("--global and --project are mutually exclusive")
@@ -938,10 +944,13 @@ def skills_install(
         skills_module.ensure_interactive_scope(
             explicit_project=project is not None,
             global_install=global_install,
-            stdin_is_tty=sys.stdin.isatty(),
+            stdin_is_tty=_interactive_terminal(),
         )
-        resolved_project = Path.cwd() if global_install else skills_module.resolve_project(project)
-        scope: skills_module.Scope = "global" if global_install else "project"
+        # A missing --project is the global default.  Keep --global as a readable, backwards-
+        # compatible spelling, while --project remains the only way to opt into project-local
+        # writes.  The interactive guard above still prevents the default from writing in a pipe.
+        resolved_project = Path.cwd() if project is None else skills_module.resolve_project(project)
+        scope: skills_module.Scope = "global" if project is None else "project"
 
         def confirm(preview: str) -> bool:
             typer.echo(preview, nl=False)

{agent2learn-0.1.7 → agent2learn-0.1.9}/src/agent2learn/ingest.py

@@ -236,9 +236,10 @@ def _record_optional_collection(
 ) -> metadata_coverage.CollectionCoverage:
     """Persist safe coverage for an optional collection and classify its failure.
 
-    A 404 means that the optional D2L tool or collection is not available for this course. It is
-    not evidence of an empty list, but it also must not prevent independent content from syncing.
-    All other failures remain metadata errors. A successful retry clears a prior recorded gap.
+    A 403 or 404 means that the optional D2L tool or collection is not available for this course.
+    It is not evidence of an empty list, but it also must not prevent independent content from
+    syncing. All other failures remain metadata errors. A successful retry clears a prior
+    recorded gap.
     """
 
     coverage = metadata_coverage.collection_coverage(collection, complete, error)
@@ -1675,7 +1676,9 @@ def _materialize_external_stubs(
     school: School,
     course: CourseRef,
 ) -> list[dict[str, object]]:
-    reserved: set[str] = set()
+    # A filename only collides with a sibling. Keep reservations per materialized module
+    # directory so two modules may legitimately contain identically titled external links.
+    reserved_by_folder: dict[str, set[str]] = defaultdict(set)
     for row in sorted(rows, key=lambda value: str(value.get("source_key", ""))):
         if row.get("availability") != "external_link":
             continue
@@ -1687,7 +1690,8 @@ def _materialize_external_stubs(
             destination = _content_directory(course_dir, record.module_path) / (
                 f"{paths.safe_name(record.title)}.url.txt"
             )
-            stub = _unique_reserved(destination, reserved)
+            folder_key = destination.parent.as_posix()
+            stub = _unique_reserved(destination, reserved_by_folder[folder_key])
             row["stub_path"] = paths.rel_posix(stub, vault.root)
         paths.ensure_dir(stub.parent, root=vault.root)
         if paths.is_link(stub):
@@ -2752,15 +2756,18 @@ def _topic_filename(topic: TopicRecord) -> str:
 
 
 def _unique_reserved(destination: Path, reserved: set[str]) -> Path:
-    candidate = paths.unique_path(destination)
-    for number in range(1, 100_000):
-        if _canonical_name(candidate) not in reserved and not paths.collides(candidate):
-            reserved.add(_canonical_name(candidate))
-            return candidate
-        stem, extension = _split_name(candidate.name)
-        suffix = "" if number == 1 else f"_{number}"
-        candidate = candidate.with_name(paths.safe_name(f"{stem}{suffix}{extension}"))
-    raise A2LError("could not allocate a unique content path")
+    # Delegate truncation and suffix budgeting to the shared path allocator. The previous
+    # hand-rolled loop applied ``safe_name`` after adding a suffix, so a long title could be
+    # truncated back to the same candidate forever. Convert the local reservation names into
+    # sibling paths; ``unique_path`` then applies the same case-folded, filesystem-aware rules
+    # to both in-memory and on-disk collisions.
+    reserved_paths = tuple(destination.with_name(name) for name in reserved)
+    try:
+        candidate = paths.unique_path(destination, reserved=reserved_paths)
+    except (RuntimeError, ValueError) as exc:
+        raise A2LError("could not allocate a unique content path") from exc
+    reserved.add(_canonical_name(candidate))
+    return candidate
 
 
 def _canonical_name(path: Path) -> str:

{agent2learn-0.1.7 → agent2learn-0.1.9}/src/agent2learn/metadata_coverage.py

@@ -92,10 +92,11 @@ def quiz_coverage(complete: bool, error: Exception | None) -> QuizCoverage:
 class CollectionCoverage:
     """Coverage state for an optional non-quiz metadata collection.
 
-    A missing collection route is different from a successfully fetched empty collection.  D2L
-    installations can omit optional tools entirely, so HTTP 404 is recorded as unavailable and
-    does not prevent independently accessible course content from being archived.  Other failed
-    requests remain incomplete and are still fatal to the metadata phase.
+    An unavailable collection is different from a successfully fetched empty collection. D2L
+    installations can omit optional tools entirely or deny an otherwise valid session access to
+    one collection, so HTTP 403 and 404 are recorded as unavailable and do not prevent
+    independently accessible course content from being archived. Other failed requests remain
+    incomplete and are still fatal to the metadata phase.
     """
 
     status: CoverageStatus = "unknown"
@@ -110,8 +111,8 @@ class CollectionCoverage:
             raise ValueError("invalid collection coverage HTTP status")
         if self.status in {"complete", "unknown"} and self.http_status is not None:
             raise ValueError("unexpected collection coverage diagnostic")
-        if self.status == "unavailable" and self.http_status != 404:
-            raise ValueError("unavailable collection coverage requires HTTP 404")
+        if self.status == "unavailable" and self.http_status not in {403, 404}:
+            raise ValueError("unavailable collection coverage requires HTTP 403 or 404")
 
     def gap(self, collection: str) -> str | None:
         if collection not in _OPTIONAL_COLLECTIONS:
@@ -138,8 +139,8 @@ def collection_coverage(
         return CollectionCoverage("complete")
     response = getattr(error, "response", None)
     status = getattr(response, "status_code", None)
-    if type(status) is int and status == 404:
-        return CollectionCoverage("unavailable", 404)
+    if type(status) is int and status in {403, 404}:
+        return CollectionCoverage("unavailable", status)
     return CollectionCoverage(
         "incomplete",
         status if type(status) is int and 400 <= status <= 599 else None,