diff --git a/src/specify_cli/__init__.py b/src/specify_cli/__init__.py index c801cbd7..f535401c 100644 --- a/src/specify_cli/__init__.py +++ b/src/specify_cli/__init__.py @@ -2505,7 +2505,7 @@ def agent_info( if resolved.source == "catalog": source_display = f"catalog — {resolved.path}" elif resolved.source == "embedded": - source_display = f"embedded (bundled in specify-cli wheel)" + source_display = "embedded (bundled in specify-cli wheel)" info_table.add_row("Source", source_display) if resolved.overrides: @@ -2871,8 +2871,8 @@ def agent_add( console.print(f"[green]✓[/green] Installed '{manifest.id}' ({manifest.name}) from {source}") else: # Catalog fetch — placeholder for future catalog integration - console.print(f"[yellow]Catalog fetch not yet implemented.[/yellow]") - console.print(f"[dim]Use --from to install from a local directory.[/dim]") + console.print("[yellow]Catalog fetch not yet implemented.[/yellow]") + console.print("[dim]Use --from to install from a local directory.[/dim]") raise typer.Exit(1) diff --git a/src/specify_cli/agent_pack.py b/src/specify_cli/agent_pack.py index efef7fb2..61873ecd 100644 --- a/src/specify_cli/agent_pack.py +++ b/src/specify_cli/agent_pack.py @@ -563,7 +563,7 @@ def remove_tracked_files( modified = check_modified_files(project_path, agent_id) if modified: raise AgentFileModifiedError( - f"The following agent files have been modified since installation:\n" + "The following agent files have been modified since installation:\n" + "\n".join(f" {p}" for p in modified) + "\nUse --force to remove them anyway." ) diff --git a/tests/test_agent_pack.py b/tests/test_agent_pack.py index c44b77fe..e3d490dd 100644 --- a/tests/test_agent_pack.py +++ b/tests/test_agent_pack.py @@ -5,7 +5,6 @@ CLI commands, and consistency with AGENT_CONFIG / CommandRegistrar.AGENT_CONFIGS """ import json -import shutil import textwrap from pathlib import Path @@ -22,7 +21,6 @@ from specify_cli.agent_pack import ( AgentPackError, ManifestValidationError, PackResolutionError, - ResolvedPack, _manifest_path, _sha256, check_modified_files, @@ -548,9 +546,8 @@ class TestEmbeddedPacksConsistency: manifest_file = child / MANIFEST_FILENAME if not manifest_file.is_file(): continue - # Should not raise - warnings = validate_pack(child) - # Warnings are acceptable; hard errors are not + # Should not raise — warnings are acceptable; hard errors are not + validate_pack(child) # ===================================================================