chore: bump version to 0.12.6

2026-07-08 23:59:55 +08:00 · 2026-07-07 16:03:01 +00:00
34 changed files with 64 additions and 1889 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,36 +2,6 @@

 <!-- insert new changelog below this comment -->

-## [0.12.8] - 2026-07-08
-
-### Changed
-
- [extension] Add LLM Wiki extension to community catalog (#3361)
- Docs: Document missing CLI flags and integrations (#3182)
- Docs: Remove Cursor from CLI check list in README (#3184)
- feat(extensions): port update-agent-context to Python (#3387)
- fix(scripts): fall through to grep/sed when python3 is a broken stub in feature.json parser (#3312)
- fix(toml): escape control characters so generated command files parse (#3341)
- fix(cli): exit cleanly on malformed IPv6 URLs in `extension`/`preset`/`workflow add` (#3369)
- fix(github-http): return None on malformed GHES port instead of raising (#3379)
- fix(integrations): guard _sha256 against unreadable managed files (#3376)
- chore: release 0.12.7, begin 0.12.8.dev0 development (#3398)
-
-## [0.12.7] - 2026-07-07
-
-### Changed
-
- fix(bundler): bundle update uninstalls components dropped by new version (#3353)
- fix(workflows): route run/resume errors to stderr under --json (#3352)
- fix(workflows): fan-in validate() rejects non-mapping output (#3349)
- fix(workflows): shell step validate() rejects non-string run (#3348)
- fix(integrations): agy honors SPECKIT_INTEGRATION_AGY_EXTRA_ARGS (#3347)
- Add Orchestration Task Context Management extension to community catalog (#3372)
- Update DocGuard — CDD Enforcement extension to v0.30.0 (#3371)
- Update Ripple extension to v1.1.0 (#3370)
- feat(integrations): generalize post-processing to all format types (#3311)
- chore: release 0.12.6, begin 0.12.7.dev0 development (#3393)
-
 ## [0.12.6] - 2026-07-07

 ### Changed
--- a/README.md
+++ b/README.md
@@ -406,7 +406,7 @@ specify init . --force --integration copilot
 specify init --here --force --integration copilot
 ```

-The CLI checks that the selected integration's required CLI tool is installed on your machine when that integration has `requires_cli: True`. If you do not have the required tool installed, or you prefer to get the templates without checking for the right tools, use `--ignore-agent-tools` with your command:
+The CLI will check that your selected agent's CLI tool is installed (for integrations that require a CLI), such as Claude Code, Gemini CLI, Qwen Code, opencode, Codex CLI, Qoder CLI, Tabnine CLI, Kiro CLI, Pi Coding Agent, Oh My Pi, Forge, Goose, Mistral Vibe, or ZCode. If you don't have the required tool installed, or you prefer to get the templates without checking for the right tools, use `--ignore-agent-tools` with your command:

 ```bash
 specify init <project_name> --integration copilot --ignore-agent-tools
--- a/docs/community/extensions.md
+++ b/docs/community/extensions.md
@@ -51,7 +51,7 @@ The following community-contributed extensions are available in [`catalog.commun
 | Confluence Extension | Create a doc in Confluence summarizing the specifications and planning files | `integration` | Read+Write | [spec-kit-confluence](https://github.com/aaronrsun/spec-kit-confluence) |
 | Cost Tracker | Track real LLM dollar cost across SDD workflows — per-feature budgets, per-integration comparison, and finance-ready exports | `visibility` | Read+Write | [spec-kit-cost](https://github.com/Quratulain-bilal/spec-kit-cost) |
 | Data Model Diagram | Generates Mermaid ER diagrams from Spec Kit data models after planning | `docs` | Read+Write | [spec-kit-data-model-diagram](https://github.com/benizzio/spec-kit-data-model-diagram) |
-| DocGuard — CDD Enforcement | The only doc-integrity engine with an MCP server, SARIF output, and a deterministic zero-LLM core. Validates, scores, and traces documentation against code — 24 validators, stable finding codes, GitHub Action with PR annotations, spec-kit hooks. Pure Node.js, one pinned dep. | `docs` | Read+Write | [spec-kit-docguard](https://github.com/raccioly/docguard) |
+| DocGuard — CDD Enforcement | Canonical-Driven Development enforcement. Validates, scores, and traces project documentation with automated checks, AI-driven workflows, and spec-kit hooks. One pinned runtime dependency; pure Node.js otherwise. | `docs` | Read+Write | [spec-kit-docguard](https://github.com/raccioly/docguard) |
 | Extensify | Create and validate extensions and extension catalogs | `process` | Read+Write | [extensify](https://github.com/mnriem/spec-kit-extensions/tree/main/extensify) |
 | Fix Findings | Automated analyze-fix-reanalyze loop that resolves spec findings until clean | `code` | Read+Write | [spec-kit-fix-findings](https://github.com/Quratulain-bilal/spec-kit-fix-findings) |
 | FixIt Extension | Spec-aware bug fixing — maps bugs to spec artifacts, proposes a plan, applies minimal changes | `code` | Read+Write | [spec-kit-fixit](https://github.com/speckit-community/spec-kit-fixit) |
@@ -67,7 +67,6 @@ The following community-contributed extensions are available in [`catalog.commun
 | Jira Integration (Sync Engine) | Idempotent, drift-aware, fail-closed reconcile engine mirroring spec-kit specs into Jira (Epic per repo, Story per spec, Subtask per phase) | `integration` | Read+Write | [spec-kit-jira-sync](https://github.com/ashbrener/spec-kit-jira-sync) |
 | Learning Extension | Generate educational guides from implementations and enhance clarifications with mentoring context | `docs` | Read+Write | [spec-kit-learn](https://github.com/imviancagrace/spec-kit-learn) |
 | Linear Integration | Mirror spec-kit feature directories into Linear (filesystem → Linear, reconcile-based, unidirectional). | `integration` | Read+Write | [spec-kit-linear-sync](https://github.com/ashbrener/spec-kit-linear-sync) |
-| LLM Wiki | LLM-maintained compounding project wiki: source ingestion, cited answers, and consistency linting | `docs` | Read+Write | [spec-kit-wiki](https://github.com/formin/spec-kit-wiki) |
 | Loop Engineering | Engineer safe autonomous agent loops for spec-driven development: a maker/checker split, externalized loop state, and stay-the-engineer guardrails against comprehension debt and cognitive surrender | `process` | Read+Write | [spec-kit-loop](https://github.com/formin/spec-kit-loop) |
 | MAQA — Multi-Agent & Quality Assurance | Coordinator → feature → QA agent workflow with parallel worktree-based implementation. Language-agnostic. Auto-detects installed board plugins. Optional CI gate. | `process` | Read+Write | [spec-kit-maqa-ext](https://github.com/GenieRobot/spec-kit-maqa-ext) |
 | MAQA Azure DevOps Integration | Azure DevOps Boards integration for MAQA — syncs User Stories and Task children as features progress | `integration` | Read+Write | [spec-kit-maqa-azure-devops](https://github.com/GenieRobot/spec-kit-maqa-azure-devops) |
@@ -87,7 +86,6 @@ The following community-contributed extensions are available in [`catalog.commun
 | .NET Framework to Modern .NET Migration | Orchestrate end-to-end .NET Framework to modern .NET migration across 7 phases, with SDD lifecycle integration | `process` | Read+Write | [spec-kit-fx-to-net](https://github.com/RogerBestMsft/spec-kit-FxToNet) |
 | Onboard | Contextual onboarding and progressive growth for developers new to spec-kit projects. Explains specs, maps dependencies, validates understanding, and guides the next step | `process` | Read+Write | [spec-kit-onboard](https://github.com/dmux/spec-kit-onboard) |
 | Optimize | Audit and optimize AI governance for context efficiency — token budgets, rule health, interpretability, compression, coherence, and echo detection | `process` | Read+Write | [spec-kit-optimize](https://github.com/sakitA/spec-kit-optimize) |
-| Orchestration Task Context Management | Adds subagent work-unit orchestration to generated Spec Kit task files | `process` | Read+Write | [spec-kit-orchestration-task-context-management](https://github.com/benizzio/spec-kit-orchestration-task-context-management) |
 | OWASP LLM Threat Model | OWASP Top 10 for LLM Applications 2025 threat analysis on agent artifacts | `code` | Read-only | [spec-kit-threatmodel](https://github.com/NaviaSamal/spec-kit-threatmodel) |
 | Plan Review Gate | Require spec.md and plan.md to be merged via MR/PR before allowing task generation | `process` | Read-only | [spec-kit-plan-review-gate](https://github.com/luno/spec-kit-plan-review-gate) |
 | PR Bridge | Auto-generate pull request descriptions, checklists, and summaries from spec artifacts | `process` | Read-only | [spec-kit-pr-bridge-](https://github.com/Quratulain-bilal/spec-kit-pr-bridge-) |
@@ -108,7 +106,7 @@ The following community-contributed extensions are available in [`catalog.commun
 | Retro Extension | Sprint retrospective analysis with metrics, spec accuracy assessment, and improvement suggestions | `process` | Read+Write | [spec-kit-retro](https://github.com/arunt14/spec-kit-retro) |
 | Retrospective Extension | Post-implementation retrospective with spec adherence scoring, drift analysis, and human-gated spec updates | `docs` | Read+Write | [spec-kit-retrospective](https://github.com/emi-dm/spec-kit-retrospective) |
 | Review Extension | Post-implementation comprehensive code review with specialized agents for code quality, comments, tests, error handling, type design, and simplification | `code` | Read-only | [spec-kit-review](https://github.com/ismaelJimenez/spec-kit-review) |
-| Ripple | Detect side effects that tests can't catch after implementation — surface hidden ripple effects across 9 analysis categories | `code` | Read+Write | [spec-kit-ripple](https://github.com/chordpli/spec-kit-ripple) |
+| Ripple | Detect side effects that tests can't catch after implementation — delta-anchored analysis across 9 domain-agnostic categories | `code` | Read+Write | [spec-kit-ripple](https://github.com/chordpli/spec-kit-ripple) |
 | SDD Utilities | Resume interrupted workflows, validate project health, and verify spec-to-task traceability | `process` | Read+Write | [speckit-utils](https://github.com/mvanhorn/speckit-utils) |
 | Security Review | Full-project secure-by-design security audits plus staged, branch/PR, plan, task, follow-up, and apply reviews | `code` | Read+Write | [spec-kit-security-review](https://github.com/DyanGalih/spec-kit-security-review) |
 | SFSpeckit | Enterprise Salesforce SDLC with 18 commands for the full SDD lifecycle. | `process` | Read+Write | [spec-kit-sf](https://github.com/ysumanth06/spec-kit-sf) |
--- a/docs/reference/extensions.md
+++ b/docs/reference/extensions.md
@@ -26,7 +26,7 @@ specify extension add <name>
 | --------------- | -------------------------------------------------------- |
 | `--dev`         | Install from a local directory (for development)         |
 | `--from <url>`  | Install from a custom URL instead of the catalog         |
-| `--force`       | Overwrite if the extension is already installed          |
+| `--force`       | Overwrite if already installed                           |
 | `--priority <N>`| Resolution priority (default: 10; lower = higher precedence) |

 Installs an extension from the catalog, a URL, or a local directory. Extension commands are automatically registered with the currently installed AI coding agent integration.
--- a/extensions/agent-context/scripts/python/update_agent_context.py
+++ b/extensions/agent-context/scripts/python/update_agent_context.py
@@ -1,352 +0,0 @@
-#!/usr/bin/env python3
-"""Refresh the managed Spec Kit section in the coding agent's context file(s).
-
-Python port of ``update-agent-context.sh`` / ``update-agent-context.ps1``.
-
-Reads ``context_files`` or ``context_file``, plus ``context_markers.{start,end}``,
-from the agent-context extension config:
-    .specify/extensions/agent-context/agent-context-config.yml
-
-Usage: update_agent_context.py [plan_path]
-
-When ``plan_path`` is omitted, the script derives it from
-``.specify/feature.json`` (written by /speckit-specify). Falls back to the most
-recently modified ``specs/*/plan.md`` only when feature.json is absent or its
-plan does not exist yet.
-"""
-
-from __future__ import annotations
-
-import json
-import os
-import re
-import sys
-from pathlib import Path
-
-DEFAULT_START = "<!-- SPECKIT START -->"
-DEFAULT_END = "<!-- SPECKIT END -->"
-
-
-def _err(message: str) -> None:
-    print(message, file=sys.stderr)
-
-
-def _get_str(obj: object, *keys: str) -> str:
-    node = obj
-    for key in keys:
-        if isinstance(node, dict) and key in node:
-            node = node[key]
-        else:
-            return ""
-    return node if isinstance(node, str) else ""
-
-
-def _collect_context_files(data: dict, project_root: str) -> list[str]:
-    """Resolve the managed context files from config, mirroring the bash logic."""
-    context_files: list[str] = []
-    seen: set[str] = set()
-    case_insensitive = sys.platform.startswith(("win32", "cygwin", "msys"))
-
-    def add(value: object) -> None:
-        if not isinstance(value, str):
-            return
-        candidate = value.strip()
-        if not candidate:
-            return
-        key = candidate.casefold() if case_insensitive else candidate
-        if key in seen:
-            return
-        context_files.append(candidate)
-        seen.add(key)
-
-    raw_files = data.get("context_files")
-    if isinstance(raw_files, list):
-        for value in raw_files:
-            add(value)
-    if not context_files:
-        add(_get_str(data, "context_file"))
-    if not context_files:
-        # Self-seed: when the config declares no target, derive one from the
-        # active integration recorded in init-options.json, mapped through the
-        # bundled agent-context-defaults.json file. Independent of the Specify
-        # CLI by design.
-        integration_key = ""
-        try:
-            with open(
-                f"{project_root}/.specify/init-options.json", "r", encoding="utf-8"
-            ) as fh:
-                opts = json.load(fh)
-            if isinstance(opts, dict):
-                value = opts.get("integration") or opts.get("ai") or ""
-                integration_key = value if isinstance(value, str) else ""
-        except Exception:
-            integration_key = ""
-        if integration_key:
-            defaults_path = (
-                f"{project_root}/.specify/extensions/agent-context/"
-                "agent-context-defaults.json"
-            )
-            mapping = {}
-            try:
-                with open(defaults_path, "r", encoding="utf-8") as fh:
-                    loaded = json.load(fh)
-                agents = loaded.get("agents", {}) if isinstance(loaded, dict) else {}
-                mapping = agents if isinstance(agents, dict) else {}
-            except Exception:
-                _err(
-                    "agent-context: unable to read %s; cannot self-seed the context "
-                    "file. Set context_file in the extension config." % defaults_path
-                )
-                mapping = {}
-            add(mapping.get(integration_key, "") or "")
-            if not context_files:
-                _err(
-                    "agent-context: no default context file is known for integration "
-                    "%s. Set context_file in the extension config to choose one."
-                    % integration_key
-                )
-    return context_files
-
-
-def _validate_context_file(project_root: str, context_file: str) -> str | None:
-    """Return an error message when the path escapes the project root."""
-    if context_file.startswith("/") or re.match(r"^[A-Za-z]:", context_file):
-        return (
-            "agent-context: context files must be project-relative paths; "
-            f"got '{context_file}'."
-        )
-    if "\\" in context_file:
-        return (
-            "agent-context: context files must not contain backslash separators; "
-            f"got '{context_file}'."
-        )
-    if ".." in context_file.split("/"):
-        return (
-            "agent-context: context files must not contain '..' path segments; "
-            f"got '{context_file}'."
-        )
-    root = Path(project_root).resolve()
-    target = (root / context_file).resolve()
-    try:
-        target.relative_to(root)
-    except ValueError:
-        return (
-            "agent-context: context file path resolves outside the project root; "
-            f"got '{context_file}'."
-        )
-    return None
-
-
-def _resolve_plan_path(project_root: str) -> str:
-    """Derive the plan path: feature.json first, then the mtime fallback."""
-    plan_path = ""
-    feature_json = Path(project_root) / ".specify" / "feature.json"
-    if feature_json.is_file():
-        feature_dir = ""
-        try:
-            with open(feature_json, "r", encoding="utf-8") as fh:
-                data = json.load(fh)
-            value = data.get("feature_directory", "")
-            feature_dir = value if isinstance(value, str) else ""
-        except Exception:
-            feature_dir = ""
-        # Normalize backslashes (written by PS on Windows) before path ops.
-        feature_dir = feature_dir.replace("\\", "/").rstrip("/")
-        if feature_dir:
-            # feature_directory may be relative or absolute (absolute paths
-            # outside the project root are preserved as-is), including
-            # drive-qualified paths (C:/...) written by PowerShell on Windows.
-            if feature_dir.startswith("/") or re.match(r"^[A-Za-z]:/", feature_dir):
-                candidate = Path(feature_dir) / "plan.md"
-            else:
-                candidate = Path(project_root) / feature_dir / "plan.md"
-            if candidate.is_file():
-                # Resolve symlinks before comparing so paths like /var/… vs
-                # /private/var/… (macOS) are treated as equivalent.
-                root = Path(project_root).resolve()
-                resolved = candidate.resolve()
-                try:
-                    plan_path = resolved.relative_to(root).as_posix()
-                except ValueError:
-                    plan_path = resolved.as_posix()
-
-    if not plan_path:
-        root = Path(project_root).resolve()
-        plans = sorted(
-            (root / "specs").glob("*/plan.md"),
-            key=lambda p: p.stat().st_mtime,
-            reverse=True,
-        )
-        if plans:
-            try:
-                plan_path = plans[0].relative_to(root).as_posix()
-            except ValueError:
-                plan_path = ""
-    return plan_path
-
-
-def _build_section(marker_start: str, marker_end: str, plan_path: str) -> str:
-    lines = [
-        marker_start,
-        "For additional context about technologies to be used, project structure,",
-        "shell commands, and other important information, read the current plan",
-    ]
-    if plan_path:
-        lines.append(f"at {plan_path}")
-    lines.append(marker_end)
-    return "\n".join(lines) + "\n"
-
-
-def ensure_mdc_frontmatter(content: str) -> str:
-    """Ensure ``.mdc`` content has YAML frontmatter with ``alwaysApply: true``.
-
-    Cursor only auto-loads ``.mdc`` rule files that carry frontmatter with
-    ``alwaysApply: true``. Prepend it when missing, or repair the value while
-    preserving any existing frontmatter comments/formatting.
-    """
-    leading_ws = len(content) - len(content.lstrip())
-    leading = content[:leading_ws]
-    stripped = content[leading_ws:]
-
-    if not stripped.startswith("---"):
-        return "---\nalwaysApply: true\n---\n\n" + content
-
-    match = re.match(
-        r"^(---[ \t]*\r?\n)(.*?)(\r?\n---[ \t]*)(\r?\n|$)(.*)",
-        stripped,
-        re.DOTALL,
-    )
-    if not match:
-        return "---\nalwaysApply: true\n---\n\n" + content
-
-    opening, fm_text, closing, sep, rest = match.groups()
-    newline = "\r\n" if "\r\n" in opening else "\n"
-
-    if re.search(r"(?m)^[ \t]*alwaysApply[ \t]*:[ \t]*true[ \t]*(?:#.*)?$", fm_text):
-        return content
-
-    if re.search(r"(?m)^[ \t]*alwaysApply[ \t]*:", fm_text):
-        fm_text = re.sub(
-            r"(?m)^([ \t]*)alwaysApply[ \t]*:.*?([ \t]*(?:#.*)?)$",
-            r"\1alwaysApply: true\2",
-            fm_text,
-            count=1,
-        )
-    elif fm_text.strip():
-        fm_text = fm_text + newline + "alwaysApply: true"
-    else:
-        fm_text = "alwaysApply: true"
-
-    return f"{leading}{opening}{fm_text}{closing}{sep}{rest}"
-
-
-def _upsert_section(
-    ctx_path: str, marker_start: str, marker_end: str, section: str
-) -> None:
-    """Insert or replace the managed section, then normalize and write."""
-    if os.path.exists(ctx_path):
-        with open(ctx_path, "r", encoding="utf-8-sig") as fh:
-            content = fh.read()
-        s = content.find(marker_start)
-        e = content.find(marker_end, s if s != -1 else 0)
-        if s != -1 and e != -1 and e > s:
-            end_of_marker = e + len(marker_end)
-            if end_of_marker < len(content) and content[end_of_marker] == "\r":
-                end_of_marker += 1
-            if end_of_marker < len(content) and content[end_of_marker] == "\n":
-                end_of_marker += 1
-            new_content = content[:s] + section + content[end_of_marker:]
-        elif s != -1:
-            new_content = content[:s] + section
-        elif e != -1:
-            end_of_marker = e + len(marker_end)
-            if end_of_marker < len(content) and content[end_of_marker] == "\r":
-                end_of_marker += 1
-            if end_of_marker < len(content) and content[end_of_marker] == "\n":
-                end_of_marker += 1
-            new_content = section + content[end_of_marker:]
-        else:
-            if content and not content.endswith("\n"):
-                content += "\n"
-            new_content = (content + "\n" + section) if content else section
-    else:
-        new_content = section
-
-    new_content = new_content.replace("\r\n", "\n").replace("\r", "\n")
-    if ctx_path.casefold().endswith(".mdc"):
-        new_content = ensure_mdc_frontmatter(new_content)
-    with open(ctx_path, "wb") as fh:
-        fh.write(new_content.encode("utf-8"))
-
-
-def main(argv: list[str] | None = None) -> int:
-    args = sys.argv[1:] if argv is None else argv
-    project_root = os.getcwd()
-    ext_config = (
-        f"{project_root}/.specify/extensions/agent-context/agent-context-config.yml"
-    )
-
-    if not os.path.isfile(ext_config):
-        _err(f"agent-context: {ext_config} not found; nothing to do.")
-        return 0
-
-    try:
-        import yaml
-    except ImportError:
-        _err(
-            "agent-context: PyYAML is required to parse extension config but is "
-            "not available in the current Python environment.\n"
-            "  To resolve: pip install pyyaml (or install it into the environment "
-            "used by python3).\n"
-            "  Context file will not be updated until PyYAML is importable."
-        )
-        _err("agent-context: skipping update (see above for details).")
-        return 0
-
-    try:
-        with open(ext_config, "r", encoding="utf-8") as fh:
-            data = yaml.safe_load(fh)
-    except Exception as exc:
-        _err(
-            f"agent-context: unable to parse {ext_config} ({exc}); "
-            "cannot update context."
-        )
-        _err("agent-context: skipping update (see above for details).")
-        return 0
-    if not isinstance(data, dict):
-        data = {}
-
-    context_files = _collect_context_files(data, project_root)
-    if not context_files:
-        _err(
-            "agent-context: context_files/context_file not set in extension config; "
-            "nothing to do."
-        )
-        return 0
-
-    for context_file in context_files:
-        error = _validate_context_file(project_root, context_file)
-        if error:
-            _err(error)
-            return 1
-
-    marker_start = _get_str(data, "context_markers", "start") or DEFAULT_START
-    marker_end = _get_str(data, "context_markers", "end") or DEFAULT_END
-
-    plan_path = args[0] if args else ""
-    if not plan_path:
-        plan_path = _resolve_plan_path(project_root)
-
-    section = _build_section(marker_start, marker_end, plan_path)
-
-    for context_file in context_files:
-        ctx_path = os.path.join(project_root, context_file)
-        os.makedirs(os.path.dirname(ctx_path) or ".", exist_ok=True)
-        _upsert_section(ctx_path, marker_start, marker_end, section)
-        print(f"agent-context: updated {context_file}")
-
-    return 0
-
-
-if __name__ == "__main__":
-    sys.exit(main())
--- a/extensions/catalog.community.json
+++ b/extensions/catalog.community.json
@@ -1106,10 +1106,10 @@
    "docguard": {
      "name": "DocGuard — CDD Enforcement",
      "id": "docguard",
-      "description": "The only doc-integrity engine with an MCP server, SARIF output, and a deterministic zero-LLM core. Validates, scores, and traces documentation against code — 24 validators, stable finding codes, GitHub Action with PR annotations, spec-kit hooks. Pure Node.js, one pinned dep.",
+      "description": "Canonical-Driven Development enforcement. Validates, scores, and traces project documentation with automated checks, AI-driven workflows, and spec-kit hooks. One pinned runtime dependency; pure Node.js otherwise.",
      "author": "raccioly",
-      "version": "0.30.0",
-      "download_url": "https://github.com/raccioly/docguard/releases/download/v0.30.0/spec-kit-docguard-v0.30.0.zip",
+      "version": "0.28.0",
+      "download_url": "https://github.com/raccioly/docguard/releases/download/v0.28.0/spec-kit-docguard-v0.28.0.zip",
      "repository": "https://github.com/raccioly/docguard",
      "homepage": "https://www.npmjs.com/package/docguard-cli",
      "documentation": "https://github.com/raccioly/docguard/blob/main/extensions/spec-kit-docguard/README.md",
@@ -1145,7 +1145,7 @@
      "downloads": 0,
      "stars": 0,
      "created_at": "2026-03-13T00:00:00Z",
-      "updated_at": "2026-07-06T00:00:00Z"
+      "updated_at": "2026-06-23T00:00:00Z"
    },
    "doctor": {
      "name": "Project Health Check",
@@ -2396,39 +2396,6 @@
      "created_at": "2026-04-03T00:00:00Z",
      "updated_at": "2026-04-03T00:00:00Z"
    },
-    "orchestration-task-context-management": {
-      "name": "Orchestration Task Context Management",
-      "id": "orchestration-task-context-management",
-      "description": "Adds subagent work-unit orchestration to generated Spec Kit task files",
-      "author": "Igor Benicio de Mesquita",
-      "version": "0.0.0",
-      "download_url": "https://github.com/benizzio/spec-kit-orchestration-task-context-management/archive/refs/tags/v0.0.0.zip",
-      "repository": "https://github.com/benizzio/spec-kit-orchestration-task-context-management",
-      "homepage": "https://github.com/benizzio/spec-kit-orchestration-task-context-management",
-      "documentation": "https://github.com/benizzio/spec-kit-orchestration-task-context-management/blob/main/README.md",
-      "changelog": "https://github.com/benizzio/spec-kit-orchestration-task-context-management/blob/main/CHANGELOG.md",
-      "license": "MIT",
-      "category": "process",
-      "effect": "read-write",
-      "requires": {
-        "speckit_version": ">=0.7.2"
-      },
-      "provides": {
-        "commands": 2,
-        "hooks": 2
-      },
-      "tags": [
-        "agent",
-        "orchestration",
-        "tasks",
-        "context"
-      ],
-      "verified": false,
-      "downloads": 0,
-      "stars": 0,
-      "created_at": "2026-07-06T00:00:00Z",
-      "updated_at": "2026-07-06T00:00:00Z"
-    },
    "orchestrator": {
      "name": "Spec Orchestrator",
      "id": "orchestrator",
@@ -3119,10 +3086,10 @@
    "ripple": {
      "name": "Ripple",
      "id": "ripple",
-      "description": "Detect side effects that tests can't catch after implementation — surface hidden ripple effects across 9 analysis categories",
+      "description": "Detect side effects that tests can't catch after implementation — delta-anchored analysis across 9 domain-agnostic categories with fix-induced side effect detection",
      "author": "chordpli",
-      "version": "1.1.0",
-      "download_url": "https://github.com/chordpli/spec-kit-ripple/archive/refs/tags/v1.1.0.zip",
+      "version": "1.0.0",
+      "download_url": "https://github.com/chordpli/spec-kit-ripple/archive/refs/tags/v1.0.0.zip",
      "repository": "https://github.com/chordpli/spec-kit-ripple",
      "homepage": "https://github.com/chordpli/spec-kit-ripple",
      "documentation": "https://github.com/chordpli/spec-kit-ripple/blob/main/README.md",
@@ -3131,13 +3098,7 @@
      "category": "code",
      "effect": "read-write",
      "requires": {
-        "speckit_version": ">=0.2.0",
-        "tools": [
-          {
-            "name": "git",
-            "required": true
-          }
-        ]
+        "speckit_version": ">=0.2.0"
      },
      "provides": {
        "commands": 3,
@@ -3154,7 +3115,7 @@
      "downloads": 0,
      "stars": 0,
      "created_at": "2026-04-20T00:00:00Z",
-      "updated_at": "2026-07-06T00:00:00Z"
+      "updated_at": "2026-04-20T00:00:00Z"
    },
    "roadmap": {
      "name": "Spec Roadmap",
@@ -4375,40 +4336,6 @@
      "created_at": "2026-04-13T00:00:00Z",
      "updated_at": "2026-04-13T00:00:00Z"
    },
-    "wiki": {
-      "name": "LLM Wiki",
-      "id": "wiki",
-      "description": "LLM-maintained compounding project wiki: source ingestion, cited answers, and consistency linting",
-      "author": "formin",
-      "version": "1.0.0",
-      "download_url": "https://github.com/formin/spec-kit-wiki/archive/refs/tags/v1.0.0.zip",
-      "repository": "https://github.com/formin/spec-kit-wiki",
-      "homepage": "https://github.com/formin/spec-kit-wiki",
-      "documentation": "https://github.com/formin/spec-kit-wiki/blob/main/README.md",
-      "changelog": "https://github.com/formin/spec-kit-wiki/blob/main/CHANGELOG.md",
-      "license": "MIT",
-      "category": "docs",
-      "effect": "read-write",
-      "requires": {
-        "speckit_version": ">=0.2.0"
-      },
-      "provides": {
-        "commands": 5,
-        "hooks": 2
-      },
-      "tags": [
-        "wiki",
-        "knowledge-base",
-        "docs",
-        "memory",
-        "context-management"
-      ],
-      "verified": false,
-      "downloads": 0,
-      "stars": 0,
-      "created_at": "2026-07-06T00:00:00Z",
-      "updated_at": "2026-07-06T00:00:00Z"
-    },
    "wireframe": {
      "name": "Wireframe Visual Feedback Loop",
      "id": "wireframe",
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "specify-cli"
-version = "0.12.8"
+version = "0.12.6"
 description = "Specify CLI, part of GitHub Spec Kit. A tool to bootstrap your projects for Spec-Driven Development (SDD)."
 readme = "README.md"
 requires-python = ">=3.11"
--- a/scripts/bash/common.sh
+++ b/scripts/bash/common.sh
@@ -97,26 +97,17 @@ read_feature_json_feature_directory() {
    local fj="$repo_root/.specify/feature.json"
    [[ -f "$fj" ]] || { printf '%s' ''; return 0; }

-    # Try parsers in order (jq -> python3 -> grep/sed), falling through on
-    # failure. Selection is by *parse success*, not mere availability: on
-    # Windows `python3` commonly resolves to the Microsoft Store App Execution
-    # Alias stub, which passes `command -v` but fails at runtime (exit 49), so
-    # an availability-gated `elif` would pick python3, swallow its failure, and
-    # never reach the grep/sed fallback -- leaving feature.json unreadable even
-    # though it is valid (issue #3304).
    local _fd=''
    if command -v jq >/dev/null 2>&1; then
        if ! _fd=$(jq -r '.feature_directory // empty' "$fj" 2>/dev/null); then
            _fd=''
        fi
-    fi
-    if [[ -z "$_fd" ]] && command -v python3 >/dev/null 2>&1; then
+    elif command -v python3 >/dev/null 2>&1; then
        # Use Python so pretty-printed/multi-line JSON still parses correctly.
        if ! _fd=$(python3 -c "import json,sys; d=json.load(open(sys.argv[1])); v=d.get('feature_directory'); print(v if v else '')" "$fj" 2>/dev/null); then
            _fd=''
        fi
-    fi
-    if [[ -z "$_fd" ]]; then
+    else
        # Last-resort single-line grep/sed fallback. The `|| true` guards against
        # grep returning 1 (no match) aborting under `set -e` / `pipefail`.
        _fd=$( { grep -E '"feature_directory"[[:space:]]*:' "$fj" 2>/dev/null || true; } \
--- a/src/specify_cli/_github_http.py
+++ b/src/specify_cli/_github_http.py
@@ -127,14 +127,7 @@ def resolve_github_release_asset_api_url(
    if hostname == "github.com":
        api_base = "https://api.github.com"
    elif is_ghes:
-        # ``parsed.port`` raises ValueError on a malformed port (e.g.
-        # ``host:notaport``); the function's contract is to return None for
-        # anything it can't resolve, not to raise.
-        try:
-            port = parsed.port
-        except ValueError:
-            return None
-        authority = hostname if port is None else f"{hostname}:{port}"
+        authority = hostname if parsed.port is None else f"{hostname}:{parsed.port}"
        api_base = f"{parsed.scheme}://{authority}/api/v3"
    else:
        return None
--- a/src/specify_cli/_toml_string.py
+++ b/src/specify_cli/_toml_string.py
@@ -1,60 +0,0 @@
-"""Shared TOML string-escaping helpers.
-
-Both TOML command renderers — ``TomlIntegration`` (gemini, tabnine) in
-``specify_cli.integrations.base`` and ``CommandRegistrar.render_toml_command``
-(extension/preset commands) in ``specify_cli.agents`` — need the same rules for
-detecting characters TOML forbids literally and for emitting a fully-escaped
-basic string. Keeping one implementation here avoids the two drifting apart if
-the escaping rules change again.
-"""
-from __future__ import annotations
-
-
-def has_illegal_toml_control(value: str) -> bool:
-    """True when *value* contains a character TOML forbids literally.
-
-    TOML basic/literal strings (single- or multi-line) allow tab and, in the
-    multiline forms, newlines — but every other control character
-    (``U+0000``–``U+001F`` and ``U+007F``) must be ``\\u``-escaped, which only a
-    basic string can do. A bare carriage return counts too: a multiline basic
-    string treats ``\\r`` as a newline only when paired into ``\\r\\n``; a lone
-    ``\\r`` is an illegal control character.
-    """
-    length = len(value)
-    for i, ch in enumerate(value):
-        code = ord(ch)
-        if ch == "\r":
-            # Only a CR that is part of a CRLF newline is allowed literally.
-            if i + 1 < length and value[i + 1] == "\n":
-                continue
-            return True
-        if (code < 0x20 and ch not in ("\t", "\n")) or code == 0x7F:
-            return True
-    return False
-
-
-def escape_toml_basic(value: str) -> str:
-    """Render *value* as a single-line basic string, escaping everything.
-
-    Always valid TOML: backslash/quote are escaped, the common control chars
-    use their short escapes, and any remaining control character is emitted as
-    a ``\\uXXXX`` sequence.
-    """
-    out: list[str] = []
-    for ch in value:
-        code = ord(ch)
-        if ch == "\\":
-            out.append("\\\\")
-        elif ch == '"':
-            out.append('\\"')
-        elif ch == "\n":
-            out.append("\\n")
-        elif ch == "\r":
-            out.append("\\r")
-        elif ch == "\t":
-            out.append("\\t")
-        elif code < 0x20 or code == 0x7F:
-            out.append(f"\\u{code:04x}")
-        else:
-            out.append(ch)
-    return '"' + "".join(out) + '"'
--- a/src/specify_cli/agents.py
+++ b/src/specify_cli/agents.py
@@ -16,8 +16,6 @@ from typing import Any, Dict, List, Optional
 import yaml

 from ._init_options import is_ai_skills_enabled, load_init_options
-from ._toml_string import escape_toml_basic as _escape_toml_basic
-from ._toml_string import has_illegal_toml_control as _has_illegal_toml_control
 from ._utils import relative_extension_path_violation


@@ -260,12 +258,7 @@ class CommandRegistrar:
        # ``C:\\Users\\...`` whose ``\\U`` reads as an invalid unicode escape) would
        # produce unparseable TOML — route those to the *literal* form ('''...'''),
        # which does not process escapes, or to the escaped basic string.
-        # Control characters (U+0000–U+001F except tab/newline, U+007F) and a bare
-        # CR are illegal in every TOML string form, so a body containing them must
-        # go to the escaped basic string regardless of which delimiters it uses.
-        if self._has_illegal_toml_control(body):
-            toml_lines.append(f"prompt = {self._render_basic_toml_string(body)}")
-        elif '"""' not in body and "\\" not in body:
+        if '"""' not in body and "\\" not in body:
            toml_lines.append('prompt = """')
            toml_lines.append(body)
            toml_lines.append('"""')
@@ -278,11 +271,17 @@ class CommandRegistrar:

        return "\n".join(toml_lines)

-    # Control-char detection and basic-string escaping are shared with the
-    # gemini/tabnine renderer in ``specify_cli.integrations.base`` via
-    # ``specify_cli._toml_string`` so the two never drift apart.
-    _has_illegal_toml_control = staticmethod(_has_illegal_toml_control)
-    _render_basic_toml_string = staticmethod(_escape_toml_basic)
+    @staticmethod
+    def _render_basic_toml_string(value: str) -> str:
+        """Render *value* as a TOML basic string literal."""
+        escaped = (
+            value.replace("\\", "\\\\")
+            .replace('"', '\\"')
+            .replace("\n", "\\n")
+            .replace("\r", "\\r")
+            .replace("\t", "\\t")
+        )
+        return f'"{escaped}"'

    def render_yaml_command(
        self,
@@ -707,17 +706,6 @@ class CommandRegistrar:
            else:
                raise ValueError(f"Unsupported format: {agent_config['format']}")

-            # -- Post-process for non-skills agents -----------------------
-            _integration = None
-            if agent_config["extension"] != "/SKILL.md":
-                from specify_cli.integrations import (  # noqa: PLC0415
-                    get_integration,
-                )
-
-                _integration = get_integration(agent_name)
-                if _integration is not None:
-                    output = _integration.post_process_command_content(output)
-
            dest_file = commands_dir / f"{output_name}{agent_config['extension']}"
            self._ensure_inside(dest_file, commands_dir)
            dest_file.parent.mkdir(parents=True, exist_ok=True)
@@ -778,9 +766,6 @@ class CommandRegistrar:
                        raise ValueError(
                            f"Unsupported format: {agent_config['format']}"
                        )
-
-                    if agent_config["extension"] != "/SKILL.md" and _integration is not None:
-                        alias_output = _integration.post_process_command_content(alias_output)
                else:
                    # For other agents, reuse the primary output
                    alias_output = output
--- a/src/specify_cli/authentication/http.py
+++ b/src/specify_cli/authentication/http.py
@@ -73,13 +73,6 @@ class _StripAuthOnRedirect(urllib.request.HTTPRedirectHandler):
        self._redirect_validator = redirect_validator

    def redirect_request(self, req, fp, code, msg, headers, newurl):
-        try:
-            new_parsed = urlparse(newurl)
-        except ValueError as exc:
-            # Malformed redirect target (e.g. unterminated IPv6 bracket).
-            # Surface as URLError so callers' download error handling applies.
-            raise urllib.error.URLError(f"malformed redirect URL: {exc}") from exc
-
        if self._redirect_validator is not None:
            self._redirect_validator(req.full_url, newurl)

@@ -90,6 +83,7 @@ class _StripAuthOnRedirect(urllib.request.HTTPRedirectHandler):
        new_req = super().redirect_request(req, fp, code, msg, headers, newurl)
        if new_req is not None:
            old_scheme = urlparse(req.full_url).scheme
+            new_parsed = urlparse(newurl)
            hostname = (new_parsed.hostname or "").lower()
            is_https_downgrade = old_scheme == "https" and new_parsed.scheme != "https"
            if _hostname_in_hosts(hostname, self._hosts) and not is_https_downgrade:
--- a/src/specify_cli/bundler/services/installer.py
+++ b/src/specify_cli/bundler/services/installer.py
@@ -130,28 +130,6 @@ def install_bundle(
            done.append(component)
            result.installed.append(component)
            contributed.append(component)
-
-        # On update (refresh), uninstall components this bundle used to own
-        # that the new version no longer ships. Otherwise they are dropped
-        # from the record below (contributed only holds plan.components) yet
-        # left on disk — permanently orphaned, since no bundle record can
-        # ever remove them. A stale component still owned by another bundle
-        # is kept installed and simply de-attributed here (it stays in that
-        # bundle's record). Mirrors remove_bundle's refcount logic.
-        if refresh and existing is not None:
-            planned = {(c.kind, c.id) for c in plan.components}
-            still_needed = components_still_needed(
-                records, exclude_bundle_id=plan.bundle_id
-            )
-            for component in existing.contributed_components:
-                key = (component.kind, component.id)
-                if key in planned:
-                    continue
-                if key in still_needed:
-                    continue
-                if installer.is_installed(project_root, component):
-                    installer.remove(project_root, component)
-                    result.uninstalled.append(component)
    except BundlerError:
        _rollback(project_root, installer, done)
        raise
--- a/src/specify_cli/extensions/_commands.py
+++ b/src/specify_cli/extensions/_commands.py
@@ -426,11 +426,7 @@ def extension_add(
    if from_url and not dev:
        from urllib.parse import urlparse

-        try:
-            parsed = urlparse(from_url)
-        except ValueError:
-            console.print(f"[red]Error:[/red] Invalid URL: {_escape_markup(from_url)}")
-            raise typer.Exit(1)
+        parsed = urlparse(from_url)
        is_localhost = parsed.hostname in ("localhost", "127.0.0.1", "::1")

        if parsed.scheme != "https" and not (parsed.scheme == "http" and is_localhost):
--- a/src/specify_cli/integrations/agy/init.py
+++ b/src/specify_cli/integrations/agy/init.py
@@ -89,11 +89,7 @@ class AgyIntegration(SkillsIntegration):
        output_json: bool = True,
    ) -> list[str] | None:
        # agy does not support --model or JSON output; both params are ignored
-        args = [self._resolve_executable(), "--print", prompt]
-        # Honor SPECKIT_INTEGRATION_AGY_EXTRA_ARGS (operator-supplied flags),
-        # appended after the positional prompt like the devin integration.
-        self._apply_extra_args_env_var(args)
-        return args
+        return [self._resolve_executable(), "--print", prompt]

    def setup(
        self,
--- a/src/specify_cli/integrations/base.py
+++ b/src/specify_cli/integrations/base.py
@@ -25,9 +25,6 @@ from typing import TYPE_CHECKING, Any

 import yaml

-from .._toml_string import escape_toml_basic as _escape_toml_basic
-from .._toml_string import has_illegal_toml_control as _has_illegal_toml_control
-
 if TYPE_CHECKING:
    from .manifest import IntegrationManifest

@@ -126,19 +123,6 @@ class IntegrationBase(ABC):
    integration that sets this flag.
    """

-    def post_process_command_content(self, content: str) -> str:
-        """Transform command content after format rendering.
-
-        Called by ``register_commands()`` for non-skills format types
-        (Markdown, TOML, YAML) after the command has been rendered into
-        its target format and before writing to disk.  Skills-format
-        agents use ``post_process_skill_content()`` instead.
-
-        Subclasses may override to inject agent-specific content.
-        The default implementation returns *content* unchanged.
-        """
-        return content
-
    # -- Public API -------------------------------------------------------

    @classmethod
@@ -956,12 +940,6 @@ class TomlIntegration(IntegrationBase):
        body = "".join(lines[frontmatter_end + 1 :])
        return frontmatter, body

-    # Control-char detection and basic-string escaping are shared with the
-    # extension/preset renderer in ``specify_cli.agents`` via
-    # ``specify_cli._toml_string`` so the two never drift apart.
-    _has_illegal_toml_control = staticmethod(_has_illegal_toml_control)
-    _escape_toml_basic = staticmethod(_escape_toml_basic)
-
    @staticmethod
    def _render_toml_string(value: str) -> str:
        """Render *value* as a TOML string literal.
@@ -971,12 +949,6 @@ class TomlIntegration(IntegrationBase):
        literal string or escaped basic string when delimiters appear in
        the content.
        """
-        # Control characters other than tab/newline (and a bare CR) cannot
-        # appear literally in any TOML string; route them to a fully-escaped
-        # basic string so the generated file stays parseable.
-        if TomlIntegration._has_illegal_toml_control(value):
-            return TomlIntegration._escape_toml_basic(value)
-
        if "\n" not in value and "\r" not in value:
            escaped = value.replace("\\", "\\\\").replace('"', '\\"')
            return f'"{escaped}"'
@@ -989,7 +961,17 @@ class TomlIntegration(IntegrationBase):
        if "'''" not in value and not value.endswith("'"):
            return "'''\n" + value + "'''"

-        return TomlIntegration._escape_toml_basic(value)
+        return (
+            '"'
+            + (
+                value.replace("\\", "\\\\")
+                .replace('"', '\\"')
+                .replace("\n", "\\n")
+                .replace("\r", "\\r")
+                .replace("\t", "\\t")
+            )
+            + '"'
+        )

    @staticmethod
    def _render_toml(description: str, body: str) -> str:
--- a/src/specify_cli/integrations/manifest.py
+++ b/src/specify_cli/integrations/manifest.py
@@ -309,14 +309,7 @@ class IntegrationManifest:
            if abs_path.is_symlink() or not abs_path.is_file():
                modified.append(rel)
                continue
-            try:
-                changed = _sha256(abs_path) != expected_hash
-            except OSError:
-                # Unreadable regular file (e.g. permission denied): treat as
-                # modified, consistent with the symlink / non-regular-file
-                # handling above, rather than letting the OSError escape.
-                changed = True
-            if changed:
+            if _sha256(abs_path) != expected_hash:
                modified.append(rel)
        return modified

@@ -365,17 +358,9 @@ class IntegrationManifest:
                    skipped.append(path)
                    continue
            else:
-                if not force:
-                    try:
-                        matches = _sha256(path) == expected_hash
-                    except OSError:
-                        # Unreadable: can't verify it's ours, so preserve it
-                        # (mirrors the path.unlink() OSError guard below).
-                        skipped.append(path)
-                        continue
-                    if not matches:
-                        skipped.append(path)
-                        continue
+                if not force and _sha256(path) != expected_hash:
+                    skipped.append(path)
+                    continue
            try:
                path.unlink()
            except OSError:
--- a/src/specify_cli/presets/_commands.py
+++ b/src/specify_cli/presets/_commands.py
@@ -104,13 +104,7 @@ def preset_add(
            from ipaddress import ip_address
            from urllib.parse import urlparse as _urlparse

-            try:
-                _parsed = _urlparse(from_url)
-            except ValueError:
-                from rich.markup import escape as _escape_markup
-
-                console.print(f"[red]Error:[/red] Invalid URL: {_escape_markup(from_url)}")
-                raise typer.Exit(1)
+            _parsed = _urlparse(from_url)

            def _is_allowed_download_url(parsed_url):
                host = parsed_url.hostname
@@ -141,9 +135,7 @@ def preset_add(
                )
                raise typer.Exit(1)

-            from rich.markup import escape as _esc
-
-            console.print(f"Installing preset from [cyan]{_esc(from_url)}[/cyan]...")
+            console.print(f"Installing preset from [cyan]{from_url}[/cyan]...")
            import urllib.error
            import tempfile
            import shutil
--- a/src/specify_cli/workflows/_commands.py
+++ b/src/specify_cli/workflows/_commands.py
@@ -50,17 +50,7 @@ workflow_step_catalog_app = typer.Typer(
 workflow_step_app.add_typer(workflow_step_catalog_app, name="catalog")


-def _error_console(json_output: bool):
-    """Console for error text: stderr under ``--json`` so the JSON stdout
-    stream stays parseable, the normal console otherwise. Mirrors the
-    stderr-only error routing already used by ``specify bundle``.
-    """
-    return err_console if json_output else console
-
-
-def _parse_input_values(
-    input_values: list[str] | None, *, json_output: bool = False
-) -> dict[str, Any]:
+def _parse_input_values(input_values: list[str] | None) -> dict[str, Any]:
    """Parse repeated ``key=value`` CLI inputs into a dict.

    Shared by ``workflow run`` and ``workflow resume``. Exits with an error
@@ -69,9 +59,7 @@ def _parse_input_values(
    inputs: dict[str, Any] = {}
    for kv in input_values or []:
        if "=" not in kv:
-            _error_console(json_output).print(
-                f"[red]Error:[/red] Invalid input format: {kv!r} (expected key=value)"
-            )
+            console.print(f"[red]Error:[/red] Invalid input format: {kv!r} (expected key=value)")
            raise typer.Exit(1)
        key, _, value = kv.partition("=")
        inputs[key.strip()] = value.strip()
@@ -347,26 +335,25 @@ def workflow_run(
    if not json_output:
        engine.on_step_start = lambda sid, label: console.print(f"  \u25b8 [{sid}] {label} \u2026")

-    err = _error_console(json_output)
    try:
        definition = engine.load_workflow(source_path if is_file_source else source)
    except FileNotFoundError:
-        err.print(f"[red]Error:[/red] Workflow not found: {source}")
+        console.print(f"[red]Error:[/red] Workflow not found: {source}")
        raise typer.Exit(1)
    except ValueError as exc:
-        err.print(f"[red]Error:[/red] Invalid workflow: {exc}")
+        console.print(f"[red]Error:[/red] Invalid workflow: {exc}")
        raise typer.Exit(1)

    # Validate
    errors = engine.validate(definition)
    if errors:
-        err.print("[red]Workflow validation failed:[/red]")
-        for verr in errors:
-            err.print(f"  • {verr}")
+        console.print("[red]Workflow validation failed:[/red]")
+        for err in errors:
+            console.print(f"  • {err}")
        raise typer.Exit(1)

    # Parse inputs
-    inputs = _parse_input_values(input_values, json_output=json_output)
+    inputs = _parse_input_values(input_values)

    if not json_output:
        console.print(f"\n[bold cyan]Running workflow:[/bold cyan] {definition.name} ({definition.id})")
@@ -376,10 +363,10 @@ def workflow_run(
        with _stdout_to_stderr_when(json_output):
            state = engine.execute(definition, inputs)
    except ValueError as exc:
-        err.print(f"[red]Error:[/red] {exc}")
+        console.print(f"[red]Error:[/red] {exc}")
        raise typer.Exit(1)
    except Exception as exc:
-        err.print(f"[red]Workflow failed:[/red] {exc}")
+        console.print(f"[red]Workflow failed:[/red] {exc}")
        raise typer.Exit(1)

    if json_output:
@@ -424,20 +411,19 @@ def workflow_resume(
    if not json_output:
        engine.on_step_start = lambda sid, label: console.print(f"  \u25b8 [{sid}] {label} \u2026")

-    inputs = _parse_input_values(input_values, json_output=json_output)
-    err = _error_console(json_output)
+    inputs = _parse_input_values(input_values)

    try:
        with _stdout_to_stderr_when(json_output):
            state = engine.resume(run_id, inputs or None)
    except FileNotFoundError:
-        err.print(f"[red]Error:[/red] Run not found: {run_id}")
+        console.print(f"[red]Error:[/red] Run not found: {run_id}")
        raise typer.Exit(1)
    except ValueError as exc:
-        err.print(f"[red]Error:[/red] {exc}")
+        console.print(f"[red]Error:[/red] {exc}")
        raise typer.Exit(1)
    except Exception as exc:
-        err.print(f"[red]Resume failed:[/red] {exc}")
+        console.print(f"[red]Resume failed:[/red] {exc}")
        raise typer.Exit(1)

    if json_output:
@@ -631,11 +617,7 @@ def workflow_add(
        from urllib.parse import urlparse
        from specify_cli.authentication.http import open_url as _open_url

-        try:
-            parsed_src = urlparse(source)
-        except ValueError:
-            console.print(f"[red]Error:[/red] Invalid URL: {_escape_markup(source)}")
-            raise typer.Exit(1)
+        parsed_src = urlparse(source)
        src_host = parsed_src.hostname or ""
        src_loopback = src_host == "localhost"
        if not src_loopback:
--- a/src/specify_cli/workflows/steps/fan_in/init.py
+++ b/src/specify_cli/workflows/steps/fan_in/init.py
@@ -58,13 +58,4 @@ class FanInStep(StepBase):
                f"Fan-in step {config.get('id', '?')!r}: "
                f"'wait_for' must be a non-empty list of step IDs."
            )
-        output = config.get("output")
-        if output is not None and not isinstance(output, dict):
-            # execute() silently coerces a non-mapping output to {}, so the
-            # author's declared aggregation keys would vanish with no error.
-            # Reject at validation, mirroring the command-step (#3262) fix.
-            errors.append(
-                f"Fan-in step {config.get('id', '?')!r}: 'output' must be a "
-                f"mapping of key -> expression, got {type(output).__name__}."
-            )
        return errors
--- a/src/specify_cli/workflows/steps/shell/init.py
+++ b/src/specify_cli/workflows/steps/shell/init.py
@@ -90,16 +90,6 @@ class ShellStep(StepBase):
            errors.append(
                f"Shell step {config.get('id', '?')!r} is missing 'run' field."
            )
-        elif not isinstance(config["run"], str):
-            # execute() str()-coerces run and invokes it under shell=True, so a
-            # null or list 'run' would run the Python repr ('None', "['echo']")
-            # as a command. Reject non-strings at validation, mirroring the
-            # command-step input/options and gate options type checks. An
-            # expression like "{{ ... }}" is still a str, so it stays valid.
-            errors.append(
-                f"Shell step {config.get('id', '?')!r}: 'run' must be a string, "
-                f"got {type(config['run']).__name__}."
-            )
        output_format = config.get("output_format")
        if output_format is not None and output_format != "json":
            errors.append(
--- a/tests/extensions/test_update_agent_context_python_parity.py
+++ b/tests/extensions/test_update_agent_context_python_parity.py
@@ -1,481 +0,0 @@
-"""Parity tests: update_agent_context.py vs update-agent-context.sh/.ps1.
-
-Each test prepares two identical project trees, runs the bash script in one
-and the Python port in the other, then compares exit codes, output (with
-project roots normalized) and the resulting context-file bytes. PowerShell
-tests compare the resulting file content only and are skipped when ``pwsh``
-is unavailable.
-"""
-
-from __future__ import annotations
-
-import json
-import os
-import shutil
-import subprocess
-import sys
-import time
-from pathlib import Path
-
-import pytest
-
-from tests.extensions.test_extension_agent_context import (
-    BASH,
-    EXT_DIR,
-    POWERSHELL,
-    _bundled_script_env,
-)
-
-PY_SCRIPT = EXT_DIR / "scripts" / "python" / "update_agent_context.py"
-BASH_SCRIPT = EXT_DIR / "scripts" / "bash" / "update-agent-context.sh"
-PS_SCRIPT = EXT_DIR / "scripts" / "powershell" / "update-agent-context.ps1"
-
-requires_posix_bash = pytest.mark.skipif(
-    not BASH or os.name == "nt",
-    reason="POSIX bash required for side-by-side parity runs",
-)
-
-
-def run_bash(project_root: Path, *args: str) -> subprocess.CompletedProcess:
-    return subprocess.run(
-        [BASH, str(BASH_SCRIPT), *args],
-        cwd=project_root,
-        env=_bundled_script_env(project_root, for_bash=True),
-        capture_output=True,
-        text=True,
-        timeout=30,
-    )
-
-
-def run_python(project_root: Path, *args: str) -> subprocess.CompletedProcess:
-    return subprocess.run(
-        [sys.executable, str(PY_SCRIPT), *args],
-        cwd=project_root,
-        env=_bundled_script_env(project_root),
-        capture_output=True,
-        text=True,
-        timeout=30,
-    )
-
-
-def run_powershell(project_root: Path, *args: str) -> subprocess.CompletedProcess:
-    return subprocess.run(
-        [
-            POWERSHELL,
-            "-NoProfile",
-            "-ExecutionPolicy",
-            "Bypass",
-            "-File",
-            str(PS_SCRIPT),
-            *args,
-        ],
-        cwd=project_root,
-        env=_bundled_script_env(project_root),
-        capture_output=True,
-        text=True,
-        timeout=30,
-    )
-
-
-def normalize(text: str, project_root: Path) -> str:
-    return text.replace(str(project_root.resolve()), "__ROOT__").replace(
-        str(project_root), "__ROOT__"
-    )
-
-
-def write_config(project_root: Path, **overrides: object) -> None:
-    """Write the extension config as JSON (valid YAML, PS-parseable too)."""
-    cfg: dict = {
-        "context_file": overrides.get("context_file", ""),
-        "context_files": overrides.get("context_files", []),
-        "context_markers": overrides.get(
-            "context_markers",
-            {"start": "<!-- SPECKIT START -->", "end": "<!-- SPECKIT END -->"},
-        ),
-    }
-    cfg_dir = project_root / ".specify" / "extensions" / "agent-context"
-    cfg_dir.mkdir(parents=True, exist_ok=True)
-    (cfg_dir / "agent-context-config.yml").write_text(
-        json.dumps(cfg), encoding="utf-8"
-    )
-
-
-def make_project(root: Path, **config: object) -> Path:
-    root.mkdir(parents=True, exist_ok=True)
-    write_config(root, **config)
-    return root
-
-
-def add_plan(project_root: Path, feature_dir: str = "specs/001-demo") -> None:
-    plan = project_root / feature_dir / "plan.md"
-    plan.parent.mkdir(parents=True, exist_ok=True)
-    plan.write_text("# plan\n", encoding="utf-8")
-    (project_root / ".specify").mkdir(parents=True, exist_ok=True)
-    (project_root / ".specify" / "feature.json").write_text(
-        json.dumps({"feature_directory": feature_dir}), encoding="utf-8"
-    )
-
-
-def twin_projects(tmp_path: Path, **config: object) -> tuple[Path, Path]:
-    return (
-        make_project(tmp_path / "proj-a", **config),
-        make_project(tmp_path / "proj-b", **config),
-    )
-
-
-def assert_parity(
-    bash: subprocess.CompletedProcess,
-    py: subprocess.CompletedProcess,
-    repo_a: Path,
-    repo_b: Path,
-) -> None:
-    assert py.returncode == bash.returncode, py.stderr + bash.stderr
-    assert normalize(py.stdout, repo_b) == normalize(bash.stdout, repo_a)
-    assert normalize(py.stderr, repo_b) == normalize(bash.stderr, repo_a)
-
-
-# ── Fresh file and upsert behavior ───────────────────────────────────────────
-
-
-@requires_posix_bash
-def test_python_creates_fresh_context_file_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    add_plan(repo_a)
-    add_plan(repo_b)
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content_a = (repo_a / "AGENTS.md").read_bytes()
-    content_b = (repo_b / "AGENTS.md").read_bytes()
-    assert content_a == content_b
-    assert b"at specs/001-demo/plan.md" in content_b
-
-
-@requires_posix_bash
-def test_python_replaces_existing_section_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    existing = (
-        "# My project\n\n"
-        "<!-- SPECKIT START -->\nstale section\n<!-- SPECKIT END -->\n"
-        "\nTrailing prose stays.\n"
-    )
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / "AGENTS.md").write_text(existing, encoding="utf-8")
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_text(encoding="utf-8")
-    assert content == (repo_a / "AGENTS.md").read_text(encoding="utf-8")
-    assert "stale section" not in content
-    assert content.startswith("# My project\n")
-    assert "Trailing prose stays." in content
-
-
-@requires_posix_bash
-@pytest.mark.parametrize(
-    "existing",
-    [
-        "# Doc\n<!-- SPECKIT START -->\ndangling start\n",
-        "dangling end\n<!-- SPECKIT END -->\nrest\n",
-        "no markers at all",
-    ],
-    ids=["start-only", "end-only", "no-markers-no-newline"],
-)
-def test_python_handles_partial_markers_matching_bash(
-    tmp_path: Path, existing: str
-) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / "AGENTS.md").write_text(existing, encoding="utf-8")
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert (repo_a / "AGENTS.md").read_bytes() == (repo_b / "AGENTS.md").read_bytes()
-
-
-@requires_posix_bash
-def test_python_custom_markers_matching_bash(tmp_path: Path) -> None:
-    markers = {"start": "<!-- CTX BEGIN -->", "end": "<!-- CTX FINISH -->"}
-    repo_a, repo_b = twin_projects(
-        tmp_path, context_file="AGENTS.md", context_markers=markers
-    )
-    existing = "intro\n<!-- CTX BEGIN -->\nold\n<!-- CTX FINISH -->\noutro\n"
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / "AGENTS.md").write_text(existing, encoding="utf-8")
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_text(encoding="utf-8")
-    assert content == (repo_a / "AGENTS.md").read_text(encoding="utf-8")
-    assert "<!-- CTX BEGIN -->" in content
-    assert "old" not in content
-
-
-@requires_posix_bash
-def test_python_multiple_context_files_dedup_matching_bash(tmp_path: Path) -> None:
-    files = ["AGENTS.md", "docs/CONTEXT.md", "AGENTS.md"]
-    repo_a, repo_b = twin_projects(tmp_path, context_files=files)
-    add_plan(repo_a)
-    add_plan(repo_b)
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert bash.stdout.count("agent-context: updated") == 2
-    for name in ("AGENTS.md", "docs/CONTEXT.md"):
-        assert (repo_a / name).read_bytes() == (repo_b / name).read_bytes()
-
-
-@requires_posix_bash
-def test_python_normalizes_crlf_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    existing = b"# Doc\r\n\r\n<!-- SPECKIT START -->\r\nold\r\n<!-- SPECKIT END -->\r\ntail\r\n"
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / "AGENTS.md").write_bytes(existing)
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_bytes()
-    assert content == (repo_a / "AGENTS.md").read_bytes()
-    assert b"\r" not in content
-
-
-@requires_posix_bash
-def test_python_mdc_frontmatter_repair_matching_bash(tmp_path: Path) -> None:
-    mdc = ".cursor/rules/specify-rules.mdc"
-    cases = {
-        "missing": "# Rules\n",
-        "false-value": "---\ndescription: rules\nalwaysApply: false\n---\n\n# Rules\n",
-        "no-key": "---\ndescription: rules\n---\n\n# Rules\n",
-    }
-    for name, existing in cases.items():
-        repo_a = make_project(tmp_path / f"a-{name}", context_file=mdc)
-        repo_b = make_project(tmp_path / f"b-{name}", context_file=mdc)
-        for repo in (repo_a, repo_b):
-            add_plan(repo)
-            target = repo / mdc
-            target.parent.mkdir(parents=True, exist_ok=True)
-            target.write_text(existing, encoding="utf-8")
-
-        bash = run_bash(repo_a)
-        py = run_python(repo_b)
-
-        assert_parity(bash, py, repo_a, repo_b)
-        content = (repo_b / mdc).read_text(encoding="utf-8")
-        assert content == (repo_a / mdc).read_text(encoding="utf-8"), name
-        assert "alwaysApply: true" in content, name
-
-
-# ── Plan-path resolution ─────────────────────────────────────────────────────
-
-
-@requires_posix_bash
-def test_python_explicit_plan_argument_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-
-    bash = run_bash(repo_a, "specs/009-explicit/plan.md")
-    py = run_python(repo_b, "specs/009-explicit/plan.md")
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_bytes()
-    assert content == (repo_a / "AGENTS.md").read_bytes()
-    assert b"at specs/009-explicit/plan.md" in content
-
-
-@requires_posix_bash
-def test_python_mtime_fallback_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    now = time.time()
-    for repo in (repo_a, repo_b):
-        for feature, age in (("specs/000-old", 10), ("specs/001-new", 0)):
-            plan = repo / feature / "plan.md"
-            plan.parent.mkdir(parents=True, exist_ok=True)
-            plan.write_text("# plan\n", encoding="utf-8")
-            os.utime(plan, (now - age, now - age))
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_bytes()
-    assert content == (repo_a / "AGENTS.md").read_bytes()
-    assert b"at specs/001-new/plan.md" in content
-
-
-@requires_posix_bash
-def test_python_prefers_feature_json_over_mtime_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-    now = time.time()
-    for repo in (repo_a, repo_b):
-        add_plan(repo, "specs/001-active")
-        stale = repo / "specs" / "000-stale" / "plan.md"
-        stale.parent.mkdir(parents=True, exist_ok=True)
-        stale.write_text("# plan\n", encoding="utf-8")
-        os.utime(repo / "specs" / "001-active" / "plan.md", (now - 10, now - 10))
-        os.utime(stale, (now, now))
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_bytes()
-    assert content == (repo_a / "AGENTS.md").read_bytes()
-    assert b"at specs/001-active/plan.md" in content
-
-
-@requires_posix_bash
-def test_python_no_plan_omits_at_line_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file="AGENTS.md")
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    content = (repo_b / "AGENTS.md").read_bytes()
-    assert content == (repo_a / "AGENTS.md").read_bytes()
-    assert b"\nat " not in content
-
-
-# ── Config gates and path validation ─────────────────────────────────────────
-
-
-@requires_posix_bash
-def test_python_missing_config_matching_bash(tmp_path: Path) -> None:
-    repo_a = tmp_path / "proj-a"
-    repo_b = tmp_path / "proj-b"
-    repo_a.mkdir()
-    repo_b.mkdir()
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert py.returncode == 0
-    assert "not found; nothing to do." in py.stderr
-
-
-@requires_posix_bash
-def test_python_unparseable_config_matching_bash(tmp_path: Path) -> None:
-    repo_a = tmp_path / "proj-a"
-    repo_b = tmp_path / "proj-b"
-    for repo in (repo_a, repo_b):
-        cfg_dir = repo / ".specify" / "extensions" / "agent-context"
-        cfg_dir.mkdir(parents=True)
-        (cfg_dir / "agent-context-config.yml").write_text(
-            "context_file: [unclosed\n", encoding="utf-8"
-        )
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert py.returncode == 0
-    assert "cannot update context." in py.stderr
-    assert "agent-context: skipping update (see above for details)." in py.stderr
-
-
-@requires_posix_bash
-def test_python_empty_config_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path)
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert py.returncode == 0
-    assert "context_files/context_file not set" in py.stderr
-
-
-@requires_posix_bash
-def test_python_self_seed_from_init_options_matching_bash(tmp_path: Path) -> None:
-    repo_a, repo_b = twin_projects(tmp_path)
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / ".specify" / "init-options.json").write_text(
-            json.dumps({"integration": "claude"}), encoding="utf-8"
-        )
-        shutil.copy(
-            EXT_DIR / "agent-context-defaults.json",
-            repo
-            / ".specify"
-            / "extensions"
-            / "agent-context"
-            / "agent-context-defaults.json",
-        )
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert (repo_a / "CLAUDE.md").read_bytes() == (repo_b / "CLAUDE.md").read_bytes()
-
-
-@requires_posix_bash
-@pytest.mark.parametrize(
-    "bad_path",
-    ["/etc/AGENTS.md", "docs\\AGENTS.md", "../outside.md", "nested/../../escape.md"],
-    ids=["absolute", "backslash", "dotdot", "nested-dotdot"],
-)
-def test_python_rejects_escaping_paths_matching_bash(
-    tmp_path: Path, bad_path: str
-) -> None:
-    repo_a, repo_b = twin_projects(tmp_path, context_file=bad_path)
-
-    bash = run_bash(repo_a)
-    py = run_python(repo_b)
-
-    assert_parity(bash, py, repo_a, repo_b)
-    assert py.returncode == 1
-    assert not (repo_b / "AGENTS.md").exists()
-
-
-# ── PowerShell parity (content only) ─────────────────────────────────────────
-
-
-@pytest.mark.skipif(not POWERSHELL, reason="no PowerShell available")
-def test_python_fresh_context_file_matches_powershell(tmp_path: Path) -> None:
-    repo_a = make_project(tmp_path / "proj-ps", context_file="AGENTS.md")
-    repo_b = make_project(tmp_path / "proj-py", context_file="AGENTS.md")
-    add_plan(repo_a)
-    add_plan(repo_b)
-
-    ps = run_powershell(repo_a)
-    py = run_python(repo_b)
-
-    assert ps.returncode == py.returncode == 0, ps.stderr + py.stderr
-    assert (repo_a / "AGENTS.md").read_bytes() == (repo_b / "AGENTS.md").read_bytes()
-
-
-@pytest.mark.skipif(not POWERSHELL, reason="no PowerShell available")
-def test_python_upsert_matches_powershell(tmp_path: Path) -> None:
-    repo_a = make_project(tmp_path / "proj-ps", context_file="AGENTS.md")
-    repo_b = make_project(tmp_path / "proj-py", context_file="AGENTS.md")
-    existing = (
-        "# My project\n\n"
-        "<!-- SPECKIT START -->\nstale\n<!-- SPECKIT END -->\n"
-        "\ntail\n"
-    )
-    for repo in (repo_a, repo_b):
-        add_plan(repo)
-        (repo / "AGENTS.md").write_text(existing, encoding="utf-8")
-
-    ps = run_powershell(repo_a)
-    py = run_python(repo_b)
-
-    assert ps.returncode == py.returncode == 0, ps.stderr + py.stderr
-    assert (repo_a / "AGENTS.md").read_bytes() == (repo_b / "AGENTS.md").read_bytes()
--- a/tests/integration/test_bundler_install_flow.py
+++ b/tests/integration/test_bundler_install_flow.py
@@ -220,68 +220,3 @@ def test_pre_existing_component_is_not_attributed_or_removed(tmp_path: Path):

    remove_bundle(tmp_path, "demo-bundle", installer)
    assert ("extensions", "ext-a") in installer.installed
-
-
-def _bundle(manifest_id, ext_ids, *, version="1.0.0"):
-    data = valid_manifest_dict()
-    data["bundle"]["id"] = manifest_id
-    data["bundle"]["version"] = version
-    data["provides"] = {
-        "extensions": [{"id": e, "version": version} for e in ext_ids]
-    }
-    return BundleManifest.from_dict(data)
-
-
-def test_update_uninstalls_components_dropped_by_new_version(tmp_path: Path):
-    """`bundle update` must uninstall components the new version no longer
-    ships, instead of orphaning them (installed on disk, tracked by nothing)."""
-    make_project(tmp_path)
-    installer = FakeInstaller()
-
-    man_v1 = _bundle("demo", ["ext-a", "ext-b"])
-    install_bundle(tmp_path, _plan(man_v1), installer, manifest=man_v1)
-    assert ("extensions", "ext-b") in installer.installed
-
-    man_v2 = _bundle("demo", ["ext-a"], version="2.0.0")
-    result = install_bundle(
-        tmp_path, _plan(man_v2), installer, manifest=man_v2, refresh=True
-    )
-
-    # ext-b was dropped by v2 -> uninstalled and reported.
-    assert ("extensions", "ext-b") in installer.remove_calls
-    assert ("extensions", "ext-b") in {(c.kind, c.id) for c in result.uninstalled}
-    assert ("extensions", "ext-b") not in installer.installed
-    assert ("extensions", "ext-a") in installer.installed
-
-    # The saved record lists only ext-a.
-    rec = next(r for r in load_records(tmp_path) if r.bundle_id == "demo")
-    keys = {(c.kind, c.id) for c in rec.contributed_components}
-    assert ("extensions", "ext-a") in keys
-    assert ("extensions", "ext-b") not in keys
-
-
-def test_update_keeps_component_still_needed_by_sibling_bundle(tmp_path: Path):
-    """A dropped component still owned by another bundle stays installed."""
-    make_project(tmp_path)
-    installer = FakeInstaller()
-
-    man_sib = _bundle("sibling", ["ext-b"])
-    install_bundle(tmp_path, _plan(man_sib), installer, manifest=man_sib)
-
-    man_v1 = _bundle("demo", ["ext-a", "ext-b"])
-    install_bundle(tmp_path, _plan(man_v1), installer, manifest=man_v1)
-
-    man_v2 = _bundle("demo", ["ext-a"], version="2.0.0")
-    install_bundle(
-        tmp_path, _plan(man_v2), installer, manifest=man_v2, refresh=True
-    )
-
-    # ext-b is still needed by 'sibling' -> not removed, stays installed.
-    assert ("extensions", "ext-b") not in installer.remove_calls
-    assert ("extensions", "ext-b") in installer.installed
-
-    # But demo's record no longer attributes it.
-    rec = next(r for r in load_records(tmp_path) if r.bundle_id == "demo")
-    assert ("extensions", "ext-b") not in {
-        (c.kind, c.id) for c in rec.contributed_components
-    }
--- a/tests/integrations/test_integration_agy.py
+++ b/tests/integrations/test_integration_agy.py
@@ -81,26 +81,6 @@ class TestAgyBuildExecArgs:
        result = i.build_exec_args("my prompt", output_json=False)
        assert result == ["agy", "--print", "my prompt"]

-    def test_build_exec_args_honors_extra_args(self, monkeypatch):
-        """SPECKIT_INTEGRATION_AGY_EXTRA_ARGS must be appended after the prompt.
-
-        agy previously skipped _apply_extra_args_env_var entirely, so the
-        documented per-integration extra-args hook was silently ignored
-        (same class as the merged cursor-agent fix #3265).
-        """
-        from specify_cli.integrations import get_integration
-        monkeypatch.setenv("SPECKIT_INTEGRATION_AGY_EXTRA_ARGS", "--verbose")
-        i = get_integration("agy")
-        assert i.build_exec_args("my prompt") == [
-            "agy", "--print", "my prompt", "--verbose",
-        ]
-
-    def test_build_exec_args_honors_executable_override(self, monkeypatch):
-        from specify_cli.integrations import get_integration
-        monkeypatch.setenv("SPECKIT_INTEGRATION_AGY_EXECUTABLE", "/custom/agy")
-        i = get_integration("agy")
-        assert i.build_exec_args("my prompt")[0] == "/custom/agy"
-

 class TestAgyHookCommandNote:
    """Verify dot-to-hyphen normalization note is injected into hook sections."""
--- a/tests/integrations/test_integration_base_toml.py
+++ b/tests/integrations/test_integration_base_toml.py
@@ -291,18 +291,6 @@ class TomlIntegrationTests:
            "closing delimiter should be inline when body does not end with a quote"
        )

-    def test_toml_string_escapes_control_characters(self):
-        """A value with control chars / a bare CR must render as parseable TOML.
-
-        TOML forbids literal control characters (U+0000–U+001F except tab and
-        newline, plus U+007F) in every string form, and a bare CR that is not
-        part of a CRLF pair. The renderer used to emit these raw into a basic or
-        ``\"\"\"`` multiline string, producing a config file that fails to parse."""
-        value = "start\x00null\x01ctrl\x1besc\x7fdel\rlone-cr end"
-        rendered = TomlIntegration._render_toml_string(value)
-        parsed = tomllib.loads(f"prompt = {rendered}")
-        assert parsed["prompt"] == value
-
    def test_toml_is_valid(self, tmp_path):
        """Every generated TOML file must parse without errors."""
        i = get_integration(self.KEY)
--- a/tests/integrations/test_manifest.py
+++ b/tests/integrations/test_manifest.py
@@ -481,40 +481,3 @@ class TestRecordExistingNewGuards:
        m = IntegrationManifest("test", tmp_path)
        with pytest.raises(ValueError, match=r"canonical|'\.\.' segments"):
            m.record_existing("dir/../file.txt")
-
-
-class TestManifestUnreadableFile:
-    """A managed file that is unreadable (e.g. PermissionError) must not crash
-    check_modified()/uninstall() — the CLI handlers surfaced a raw traceback."""
-
-    def _mk(self, tmp_path):
-        m = IntegrationManifest("test", tmp_path)
-        m.record_file("sub/f.md", "content")
-        return m
-
-    def test_check_modified_treats_unreadable_as_modified(self, tmp_path, monkeypatch):
-        m = self._mk(tmp_path)
-
-        def raise_perm(_path):
-            raise PermissionError("unreadable")
-
-        monkeypatch.setattr(
-            "specify_cli.integrations.manifest._sha256", raise_perm
-        )
-        # Before the fix this raised PermissionError.
-        assert m.check_modified() == ["sub/f.md"]
-
-    def test_uninstall_preserves_unreadable_file(self, tmp_path, monkeypatch):
-        m = self._mk(tmp_path)
-
-        def raise_perm(_path):
-            raise PermissionError("unreadable")
-
-        monkeypatch.setattr(
-            "specify_cli.integrations.manifest._sha256", raise_perm
-        )
-        removed, skipped = m.uninstall(force=False)
-        # Can't verify ownership => preserve, don't crash and don't delete.
-        assert removed == []
-        assert (tmp_path / "sub" / "f.md") in skipped
-        assert (tmp_path / "sub" / "f.md").exists()
--- a/tests/test_authentication.py
+++ b/tests/test_authentication.py
@@ -845,22 +845,6 @@ class TestRedirectStripping:
        auth3 = req3.get_header("Authorization") or req3.unredirected_hdrs.get("Authorization")
        assert auth3 == "Bearer tok"

-    def test_malformed_redirect_url_raises_urlerror_not_valueerror(self):
-        """A redirect to a malformed URL (unterminated IPv6 bracket) surfaces
-        as URLError, which download paths already handle, rather than an
-        unhandled ValueError traceback."""
-        import urllib.error
-        from specify_cli.authentication.http import _StripAuthOnRedirect
-        from urllib.request import Request
-        import io
-
-        handler = _StripAuthOnRedirect(("github.com",))
-        req = Request("https://github.com/org/repo")
-
-        with pytest.raises(urllib.error.URLError):
-            handler.redirect_request(req, io.BytesIO(b""), 302, "Found", {},
-                                     "https://[::1/asset")
-

 # ---------------------------------------------------------------------------
 # _fetch_latest_release_tag delegation
--- a/tests/test_extensions.py
+++ b/tests/test_extensions.py
@@ -1795,25 +1795,6 @@ $ARGUMENTS

        assert parsed["description"] == "first line\nsecond line\n"

-    def test_render_toml_command_escapes_control_characters(self):
-        """Control characters and a lone CR must be escaped so the TOML parses.
-
-        TOML forbids literal control characters (U+0000–U+001F except tab and
-        newline, plus U+007F) in any string, and treats a bare CR outside a
-        CRLF pair as illegal. The renderer used to emit these raw — into a
-        basic string (single-line) or a ``\"\"\"`` multiline string (for a lone
-        CR) — producing a command file that fails to parse."""
-        from specify_cli.agents import CommandRegistrar as AgentCommandRegistrar
-
-        registrar = AgentCommandRegistrar()
-        body = "start\x00null\x01ctrl\x1besc\x7fdel\rlone-cr end"
-        output = registrar.render_toml_command(
-            {"description": "d"}, body, "extension:test-ext"
-        )
-
-        parsed = tomllib.loads(output)
-        assert parsed["prompt"] == body
-
    def test_render_toml_command_preserves_backslashes_in_body(self):
        """A backslash in the body (e.g. a Windows path) must not break TOML.

@@ -5460,29 +5441,6 @@ class TestExtensionAddCLI:
                f"confirm must precede spinner, got: {call_order}"
        assert result.exit_code == 0  # user declined → clean exit

-    def test_add_from_malformed_ipv6_url_exits_cleanly(self, tmp_path):
-        """A malformed IPv6 URL must produce a clean error, not a ValueError traceback."""
-        from typer.testing import CliRunner
-        from unittest.mock import patch
-        from specify_cli import app
-
-        project_dir = tmp_path / "test-project"
-        project_dir.mkdir()
-        (project_dir / ".specify").mkdir()
-
-        runner = CliRunner()
-        with patch.object(Path, "cwd", return_value=project_dir):
-            result = runner.invoke(
-                app,
-                ["extension", "add", "my-ext", "--from", "https://[::1/ext.zip"],
-                catch_exceptions=True,
-            )
-
-        assert result.exit_code == 1
-        assert result.exception is None or isinstance(result.exception, SystemExit)
-        plain = strip_ansi(result.output)
-        assert "Invalid URL" in plain
-
    def test_add_status_escapes_extension_markup(self, tmp_path):
        """User-controlled extension names must not be parsed as Rich markup."""
        from rich.markup import escape as escape_markup
--- a/tests/test_github_http.py
+++ b/tests/test_github_http.py
@@ -233,23 +233,6 @@ class TestResolveGitHubReleaseAssetApiUrl:
        assert result is None
        assert called == []

-    def test_returns_none_on_malformed_ghes_port(self):
-        """A malformed port on an allowlisted GHES host returns None, not a
-        ValueError (contract: resolve or return None, never raise)."""
-        called = []
-
-        def open_never(url, timeout=None, extra_headers=None):
-            called.append(url)
-            raise AssertionError("open_url_fn must not be called")
-
-        result = resolve_github_release_asset_api_url(
-            "https://ghes.example:notaport/o/r/releases/download/v1/ext.zip",
-            open_never,
-            github_hosts=("ghes.example",),
-        )
-        assert result is None
-        assert called == []
-
    def test_passthrough_for_unlisted_ghes_api_asset_url(self):
        """A direct GHES /api/v3 asset URL passes through even when the host is
        not allowlisted: passthrough issues no API request, and the download
--- a/tests/test_post_process.py
+++ b/tests/test_post_process.py
@@ -1,245 +0,0 @@
-"""Tests for post_process_command_content() hook on IntegrationBase.
-
-Verifies that the generalized post-processing hook:
- Runs for non-skills format types (Markdown, TOML, YAML)
- Does NOT run for skills-format agents
- Default no-op returns content unchanged
- Exceptions propagate to caller
-"""
-
-from __future__ import annotations
-
-from unittest.mock import patch
-
-import pytest
-
-from specify_cli.agents import CommandRegistrar
-from specify_cli.integrations.base import IntegrationBase
-
-
-@pytest.fixture
-def registrar():
-    return CommandRegistrar()
-
-
-@pytest.fixture
-def ext_dir(tmp_path):
-    """Create a mock extension with a simple command template."""
-    ext = tmp_path / "extension"
-    ext.mkdir()
-    cmd_dir = ext / "commands"
-    cmd_dir.mkdir()
-    return ext, cmd_dir
-
-
-def _write_cmd(cmd_dir, name="review.md", body="Review the code.\n"):
-    cmd_file = cmd_dir / name
-    cmd_file.write_text(
-        f"---\ndescription: Test command\n---\n\n{body}",
-        encoding="utf-8",
-    )
-    return cmd_file
-
-
-class TestDefaultNoOp:
-    def test_returns_content_unchanged(self):
-        base = IntegrationBase()
-        content = "Some command content\nwith multiple lines."
-        assert base.post_process_command_content(content) == content
-
-    def test_empty_string(self):
-        base = IntegrationBase()
-        assert base.post_process_command_content("") == ""
-
-
-class TestMarkdownAgentPostProcess:
-    def test_opencode_post_process_applied(
-        self, tmp_path, registrar, ext_dir, monkeypatch
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        opencode = get_integration("opencode")
-        marker = "<!-- POST_PROCESSED -->"
-
-        def _inject_marker(self, content):
-            return content + marker
-
-        monkeypatch.setattr(
-            opencode.__class__, "post_process_command_content", _inject_marker
-        )
-
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-        registrar.register_commands(
-            "opencode", commands, "test-ext", ext, tmp_path
-        )
-
-        cmd_output = tmp_path / ".opencode" / "commands" / "speckit.test.review.md"
-        assert cmd_output.exists()
-        content = cmd_output.read_text(encoding="utf-8")
-        assert marker in content
-
-
-class TestTomlAgentPostProcess:
-    def test_gemini_post_process_applied(
-        self, tmp_path, registrar, ext_dir, monkeypatch
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        gemini = get_integration("gemini")
-        marker = "# POST_PROCESSED"
-
-        def _inject_marker(self, content):
-            return content + f"\n{marker}\n"
-
-        monkeypatch.setattr(
-            gemini.__class__, "post_process_command_content", _inject_marker
-        )
-
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-        registrar.register_commands(
-            "gemini", commands, "test-ext", ext, tmp_path
-        )
-
-        cmd_output = tmp_path / ".gemini" / "commands" / "speckit.test.review.toml"
-        assert cmd_output.exists()
-        content = cmd_output.read_text(encoding="utf-8")
-        assert marker in content
-
-
-class TestYamlAgentPostProcess:
-    def test_goose_post_process_applied(
-        self, tmp_path, registrar, ext_dir, monkeypatch
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        goose = get_integration("goose")
-        marker = "# POST_PROCESSED"
-
-        def _inject_marker(self, content):
-            return content + f"\n{marker}\n"
-
-        monkeypatch.setattr(
-            goose.__class__, "post_process_command_content", _inject_marker
-        )
-
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-        registrar.register_commands(
-            "goose", commands, "test-ext", ext, tmp_path
-        )
-
-        cmd_output = tmp_path / ".goose" / "recipes" / "speckit.test.review.yaml"
-        assert cmd_output.exists()
-        content = cmd_output.read_text(encoding="utf-8")
-        assert marker in content
-
-
-class TestSkillsAgentExcluded:
-    def test_claude_post_process_not_called(
-        self, tmp_path, registrar, ext_dir, monkeypatch
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        claude = get_integration("claude")
-        marker = "<!-- SHOULD_NOT_APPEAR -->"
-
-        def _inject_marker(self, content):
-            return content + marker
-
-        monkeypatch.setattr(
-            claude.__class__, "post_process_command_content", _inject_marker
-        )
-
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-        registrar.register_commands(
-            "claude", commands, "test-ext", ext, tmp_path
-        )
-
-        skill_file = (
-            tmp_path / ".claude" / "skills" / "speckit-test-review" / "SKILL.md"
-        )
-        assert skill_file.exists()
-        content = skill_file.read_text(encoding="utf-8")
-        assert marker not in content
-
-    def test_skills_agent_method_never_called(
-        self, tmp_path, registrar, ext_dir
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        claude = get_integration("claude")
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-
-        with patch.object(
-            claude.__class__, "post_process_command_content", wraps=claude.post_process_command_content
-        ) as mock_method:
-            registrar.register_commands(
-                "claude", commands, "test-ext", ext, tmp_path
-            )
-            mock_method.assert_not_called()
-
-
-class TestExceptionPropagation:
-    def test_hook_exception_propagates(
-        self, tmp_path, registrar, ext_dir, monkeypatch
-    ):
-        ext, cmd_dir = ext_dir
-        _write_cmd(cmd_dir)
-
-        from specify_cli.integrations import get_integration
-
-        opencode = get_integration("opencode")
-
-        def _raise(self, content):
-            raise RuntimeError("Hook failed")
-
-        monkeypatch.setattr(
-            opencode.__class__, "post_process_command_content", _raise
-        )
-
-        commands = [{"name": "speckit.test.review", "file": "commands/review.md"}]
-        with pytest.raises(RuntimeError, match="Hook failed"):
-            registrar.register_commands(
-                "opencode", commands, "test-ext", ext, tmp_path
-            )
-
-
-class TestRegressionPlainTemplate:
-    @pytest.mark.parametrize(
-        "agent,path_pattern",
-        [
-            ("claude", ".claude/skills/speckit-test-plain/SKILL.md"),
-            ("opencode", ".opencode/commands/speckit.test.plain.md"),
-        ],
-        ids=["skills", "markdown"],
-    )
-    def test_plain_template_unchanged(
-        self, tmp_path, registrar, ext_dir, agent, path_pattern
-    ):
-        ext, cmd_dir = ext_dir
-        body_text = "This is a plain command with no special content.\n"
-        _write_cmd(cmd_dir, name="plain.md", body=body_text)
-
-        commands = [{"name": "speckit.test.plain", "file": "commands/plain.md"}]
-        registrar.register_commands(
-            agent, commands, "test-ext", ext, tmp_path
-        )
-
-        output_file = tmp_path / path_pattern
-        assert output_file.exists(), f"Output file missing for {agent}"
-        content = output_file.read_text(encoding="utf-8")
-        assert body_text.strip() in content, f"Body text missing in {agent} output"
--- a/tests/test_presets.py
+++ b/tests/test_presets.py
@@ -4538,27 +4538,6 @@ class TestBundledPresetLocator:
        assert "got https://" not in output
        open_url.assert_not_called()

-    def test_preset_add_from_malformed_ipv6_url_exits_cleanly(self, project_dir):
-        """A malformed IPv6 URL must produce a clean error, not a ValueError traceback."""
-        from typer.testing import CliRunner
-        from unittest.mock import patch
-        from specify_cli import app
-
-        runner = CliRunner()
-        with patch.object(Path, "cwd", return_value=project_dir), \
-             patch("specify_cli.authentication.http.open_url") as open_url:
-            result = runner.invoke(
-                app,
-                ["preset", "add", "--from", "https://[::1/preset.zip"],
-                catch_exceptions=True,
-            )
-
-        assert result.exit_code == 1
-        assert result.exception is None or isinstance(result.exception, SystemExit)
-        output = strip_ansi(result.output)
-        assert "Invalid URL" in output
-        open_url.assert_not_called()
-
    def test_preset_add_from_url_redirect_error_describes_disallowed_url(self, project_dir, monkeypatch, capsys):
        """Redirect rejection message covers hostless HTTPS, not only non-HTTPS URLs."""
        import typer
--- a/tests/test_setup_plan_feature_json.py
+++ b/tests/test_setup_plan_feature_json.py
@@ -126,71 +126,6 @@ def test_setup_plan_errors_without_feature_context(plan_repo: Path) -> None:
    assert "Feature directory not found" in result.stderr


-@requires_bash
-def test_setup_plan_survives_broken_python3_stub(plan_repo: Path) -> None:
-    """A `python3` on PATH that exists but fails at runtime must not defeat
-    feature.json parsing.
-
-    On Windows `python3` typically resolves to the Microsoft Store App Execution
-    Alias stub: it satisfies `command -v python3` yet exits non-zero at runtime.
-    The parser must fall through to the grep/sed fallback on that failure instead
-    of selecting python3 by mere availability and swallowing its error (#3304).
-    """
-    subprocess.run(
-        ["git", "checkout", "-q", "-b", "feature/my-feature-branch"],
-        cwd=plan_repo,
-        check=True,
-    )
-    feat = plan_repo / "specs" / "001-tiny-notes-app"
-    feat.mkdir(parents=True)
-    (feat / "spec.md").write_text("# spec\n", encoding="utf-8")
-    _write_feature_json(plan_repo, "specs/001-tiny-notes-app")
-
-    # A stub python3 that mimics the Windows Store alias: on PATH, exits 49.
-    stub_dir = plan_repo / "_stubbin"
-    stub_dir.mkdir()
-    stub = stub_dir / "python3"
-    stub.write_text(
-        "#!/bin/sh\n"
-        'echo "Python was not found; run without arguments to install from the '
-        'Microsoft Store" >&2\n'
-        "exit 49\n",
-        encoding="utf-8",
-    )
-    stub.chmod(0o755)
-
-    # A stub jq that shadows any real jq on PATH and also fails, so the parser
-    # cannot short-circuit on jq and must reach the broken python3 stub and then
-    # fall through to grep/sed. Without this, a runner that has jq installed
-    # would parse feature.json via jq and never exercise the fallback this test
-    # is meant to cover.
-    jq_stub = stub_dir / "jq"
-    jq_stub.write_text(
-        "#!/bin/sh\n"
-        'echo "jq: simulated failure" >&2\n'
-        "exit 1\n",
-        encoding="utf-8",
-    )
-    jq_stub.chmod(0o755)
-
-    env = _clean_env()
-    # Prepend the stub dir so the failing jq and python3 stubs take precedence
-    # over any real ones; PATH still needs the real bash utilities for grep/sed.
-    env["PATH"] = f"{stub_dir}{os.pathsep}{env.get('PATH', '')}"
-
-    script = plan_repo / ".specify" / "scripts" / "bash" / "setup-plan.sh"
-    result = subprocess.run(
-        ["bash", str(script)],
-        cwd=plan_repo,
-        capture_output=True,
-        text=True,
-        check=False,
-        env=env,
-    )
-    assert result.returncode == 0, result.stderr + result.stdout
-    assert (feat / "plan.md").is_file()
-
-
@requires_bash
 def test_setup_plan_numbered_branch_works_with_feature_json(
    plan_repo: Path,
--- a/tests/test_workflow_run_without_project.py
+++ b/tests/test_workflow_run_without_project.py
@@ -322,87 +322,3 @@ class TestWorkflowRunWithoutProject:

        assert result.exit_code != 0
        assert ".specify path exists but is not a directory" in result.output
-
-
-class TestWorkflowRunJsonErrorStream:
-    """Under --json, error text must go to stderr so stdout stays parseable."""
-
-    def _bad_workflow(self, tmp_path):
-        wf = tmp_path / "bad.yml"
-        wf.write_text(
-            yaml.dump(
-                {
-                    "schema_version": "1.0",
-                    "workflow": {
-                        "id": "bad-wf",
-                        "name": "Bad",
-                        "version": "1.0.0",
-                        "description": "fails validation",
-                    },
-                    # shell step missing required 'run' -> validation error
-                    "steps": [{"id": "s", "type": "shell"}],
-                }
-            ),
-            encoding="utf-8",
-        )
-        return wf
-
-    def test_run_json_validation_error_not_on_stdout(self, tmp_path):
-        from typer.testing import CliRunner
-        from specify_cli import app
-
-        wf = self._bad_workflow(tmp_path)
-        runner = CliRunner()
-        old = os.getcwd()
-        try:
-            os.chdir(tmp_path)
-            result = runner.invoke(
-                app, ["workflow", "run", str(wf), "--json"], catch_exceptions=False
-            )
-        finally:
-            os.chdir(old)
-
-        assert result.exit_code == 1
-        # stdout must carry only JSON (here: nothing) — never human error text.
-        assert "validation failed" not in result.stdout
-        assert "Error" not in result.stdout
-        # The message is routed to stderr instead.
-        assert "validation failed" in result.stderr
-
-    def test_run_json_invalid_input_not_on_stdout(self, tmp_path):
-        from typer.testing import CliRunner
-        from specify_cli import app
-
-        # A valid single-shell workflow so we get past load/validate to
-        # _parse_input_values, which rejects the malformed --input.
-        wf = tmp_path / "ok.yml"
-        wf.write_text(
-            yaml.dump(
-                {
-                    "schema_version": "1.0",
-                    "workflow": {
-                        "id": "ok-wf",
-                        "name": "OK",
-                        "version": "1.0.0",
-                        "description": "x",
-                    },
-                    "steps": [{"id": "s", "type": "shell", "run": "echo hi"}],
-                }
-            ),
-            encoding="utf-8",
-        )
-        runner = CliRunner()
-        old = os.getcwd()
-        try:
-            os.chdir(tmp_path)
-            result = runner.invoke(
-                app,
-                ["workflow", "run", str(wf), "--json", "--input", "no-equals"],
-                catch_exceptions=False,
-            )
-        finally:
-            os.chdir(old)
-
-        assert result.exit_code == 1
-        assert "Invalid input format" not in result.stdout
-        assert "Invalid input format" in result.stderr
--- a/tests/test_workflows.py
+++ b/tests/test_workflows.py
@@ -1266,26 +1266,6 @@ class TestShellStep:
        errors = step.validate({"id": "test"})
        assert any("missing 'run'" in e for e in errors)

-    @pytest.mark.parametrize("bad_run", [None, ["echo", "hi"], 42])
-    def test_validate_rejects_non_string_run(self, bad_run):
-        """A non-string 'run' must be rejected at validation.
-
-        execute() str()-coerces run and invokes it under shell=True, so a
-        null or list run would otherwise run the Python repr as a command.
-        """
-        from specify_cli.workflows.steps.shell import ShellStep
-
-        step = ShellStep()
-        errors = step.validate({"id": "s", "run": bad_run})
-        assert any("'run' must be a string" in e for e in errors)
-
-    def test_validate_accepts_string_and_expression_run(self):
-        from specify_cli.workflows.steps.shell import ShellStep
-
-        step = ShellStep()
-        assert step.validate({"id": "s", "run": "echo hi"}) == []
-        assert step.validate({"id": "s", "run": "{{ steps.x.output }}"}) == []
-

    def test_output_format_json_exposes_data(self, tmp_path):
        from specify_cli.workflows.steps.shell import ShellStep
@@ -2202,27 +2182,6 @@ class TestFanInStep:
        errors = step.validate({"id": "test", "wait_for": "not-a-list"})
        assert any("non-empty list" in e for e in errors)

-    @pytest.mark.parametrize("bad_output", [["{{ fan_in.results }}"], "{{ x }}", 42])
-    def test_validate_rejects_non_mapping_output(self, bad_output):
-        """A non-mapping 'output' must be rejected: execute() would otherwise
-        silently coerce it to {} and drop the declared aggregation keys."""
-        from specify_cli.workflows.steps.fan_in import FanInStep
-
-        step = FanInStep()
-        errors = step.validate(
-            {"id": "j", "wait_for": ["a"], "output": bad_output}
-        )
-        assert any("'output' must be a mapping" in e for e in errors)
-
-    def test_validate_accepts_mapping_or_absent_output(self):
-        from specify_cli.workflows.steps.fan_in import FanInStep
-
-        step = FanInStep()
-        assert step.validate(
-            {"id": "j", "wait_for": ["a"], "output": {"joined": "{{ x }}"}}
-        ) == []
-        assert step.validate({"id": "j", "wait_for": ["a"]}) == []
-

 class TestFanOutConcurrency:
    """Fan-out honors max_concurrency (WorkflowEngine._run_fan_out)."""
@@ -5593,23 +5552,6 @@ class TestWorkflowRemoveGuard:


 class TestWorkflowAddSymlinkGuard:
-    def test_add_malformed_ipv6_url_exits_cleanly(self, temp_dir, monkeypatch):
-        """A malformed IPv6 URL must produce a clean error, not a ValueError traceback."""
-        from typer.testing import CliRunner
-        from specify_cli import app
-
-        (temp_dir / ".specify").mkdir(exist_ok=True)
-        monkeypatch.chdir(temp_dir)
-        result = CliRunner().invoke(
-            app,
-            ["workflow", "add", "https://[::1/wf.yaml"],
-            catch_exceptions=True,
-        )
-
-        assert result.exit_code == 1
-        assert result.exception is None or isinstance(result.exception, SystemExit)
-        assert "Invalid URL" in result.output
-
    @pytest.mark.skipif(not hasattr(os, "symlink"), reason="symlinks are unavailable")
    def test_add_refuses_symlinked_specify(self, temp_dir, monkeypatch):
        """workflow add must refuse a symlinked .specify (writes could escape root)."""