plugin: address python-skills review comments

Author: gaelic-ghost

README.md

@@ -78,7 +78,7 @@ Current Socket catalog shape:
 - `cardhop-app`: mixed skill plus bundled MCP server for Cardhop.app contact workflows
 - `dotnet-skills`: .NET, F#, and C# project-shape, bootstrap, implementation, test, package, diagnostics, ASP.NET Core, interop, CI, upgrade, and tooling guidance
 - `productivity-skills`: general-purpose maintainer and documentation workflow baseline
-- `python-skills`: Python, `uv`, project implementation, diagnostics, packaging, tooling, CI, upgrades, FastAPI, FastMCP, and pytest workflow plugin
+- `python-skills`: Python runtime and tooling workflows for Python-based projects; see the [Python skills expansion plan](./docs/maintainers/python-skills-plugin-plan.md) for maintainer details
 - `speak-swiftly`: Git-backed Speak Swiftly plugin from the standalone SpeakSwiftlyServer repository
 - `swiftasb-skills`: SwiftASB companion guidance
 - `things-app`: mixed skill plus bundled MCP server for Things.app workflows

docs/architecture/architecture.json

@@ -498,7 +498,7 @@
       "targets": []
     }
   ],
-  "projectRoot": "/Users/galew/Workspace/gaelic-ghost/socket-wt-python-skills-gap-analysis",
+  "projectRoot": ".",
   "relationships": [
     {
       "evidence": [

plugins/productivity-skills/skills/maintain-project-architecture/references/architecture-json.md

@@ -8,7 +8,7 @@
 {
   "schemaVersion": 1,
   "generatedBy": "maintain-project-architecture",
-  "projectRoot": "/path/to/repo",
+  "projectRoot": ".",
   "detectedAt": "2026-05-03T00:00:00Z",
   "products": [],
   "targets": [],

plugins/productivity-skills/skills/maintain-project-architecture/scripts/maintain_project_architecture.py

@@ -433,7 +433,7 @@ def detect_model(project_root: Path) -> dict[str, Any]:
     return {
         "schemaVersion": 1,
         "generatedBy": "maintain-project-architecture",
-        "projectRoot": str(project_root),
+        "projectRoot": ".",
         "detectedAt": datetime.now(timezone.utc).isoformat(),
         "detectionSource": source,
         "products": [item for item in products if item.get("name")],

plugins/productivity-skills/skills/maintain-project-architecture/tests/test_maintain_project_architecture.py

@@ -2,6 +2,7 @@
 
 import argparse
 import importlib.util
+import json
 import sys
 from pathlib import Path
 
@@ -98,6 +99,8 @@ def test_apply_creates_architecture_files_and_detects_products(tmp_path: Path) -
     assert (architecture_dir / "ARCHITECTURE.md").is_file()
     assert (architecture_dir / "SLICES.md").is_file()
     assert (architecture_dir / "architecture.json").is_file()
+    model = json.loads((architecture_dir / "architecture.json").read_text(encoding="utf-8"))
+    assert model["projectRoot"] == "."
     architecture = (architecture_dir / "ARCHITECTURE.md").read_text(encoding="utf-8")
     assert "## Product Map" in architecture
     assert "`DemoCore`" in architecture

plugins/python-skills/skills/choose-python-project-shape/agents/openai.yaml

@@ -1,6 +1,6 @@
 interface:
   display_name: "Choose Python Project Shape"
-  short_description: "Choose Python package, service, MCP, workspace, test, tooling, package, CI, or upgrade shape."
+  short_description: "Choose Python package, service, MCP, workspace, test, tooling, CI, or upgrade shape."
   brand_color: "#2563EB"
   default_prompt: "Use $choose-python-project-shape to inspect this repository, choose the smallest correct Python project shape, recommend the next Python skill, and return exact uv validation commands before implementation starts."
 

plugins/python-skills/skills/python-upgrade-workflow/SKILL.md

@@ -44,7 +44,7 @@ Use official documentation first:
 
 1. Inventory current state:
    ```bash
-   rg -n "requires-python|dependencies|optional-dependencies|dependency-groups|tool.uv|tool.pytest|tool.ruff|tool.mypy|fastapi|fastmcp|pydantic|pytest|ruff|mypy" pyproject.toml uv.lock .github docs README.md
+   rg -n "requires-python|dependencies|optional-dependencies|dependency-groups|tool.uv|tool.pytest|tool.ruff|tool.mypy|fastapi|fastmcp|pydantic|pytest|ruff|mypy"
    rg --files -g 'pyproject.toml' -g 'uv.lock' -g '.python-version' -g '.github/workflows/*.yml' -g '.github/workflows/*.yaml'
    ```
 2. Decide upgrade boundary:

plugins/python-skills/skills/python-upgrade-workflow/agents/openai.yaml

@@ -2,7 +2,7 @@ interface:
   display_name: "Python Upgrade Workflow"
   short_description: "Plan and validate Python, dependency, framework, lockfile, and tooling upgrades."
   brand_color: "#7C3AED"
-  default_prompt: "Use $python-upgrade-workflow to inventory Python requirements, dependencies, uv lockfile state, frameworks, tooling versions, compatibility impact, and staged validation before applying an upgrade."
+  default_prompt: "Use $python-upgrade-workflow to inventory Python requirements, dependencies, uv lockfile state, frameworks, tooling versions, compatibility impact, and staged validation before applying an upgrade; do not publish or create releases unless explicitly requested."
 
 policy:
   allow_implicit_invocation: true