Remove Jira routing from rhdh skill + review fixes

- Remove Jira intake menu item, routing section, and principle from
  rhdh SKILL.md so the rhdh-jira skill activates naturally instead
  of being intercepted by a stale jira-structure.md reference
- Remove jira-reference.md and jira-structure.md from reference index
  and shared references table (files kept for overlay's cross-reference)
- Update consult_tool_references principle: GitHub only, with note to
  use rhdh-jira skill directly for Jira work
- Renumber intake menu (1-9) and routing table to match
- Fix L1: export-plugin.py ANSI color check uses both stdout+stderr TTY
- Fix L2: README skill-maker description mentions consolidation
- Fix S1: consolidation-guide.md grep example includes --exclude-dir=.git
- Fix scaffold.py: move os import to top-level (ruff E402)

Author: schultzp2020

README.md

@@ -46,7 +46,7 @@ Track work across the four RHDH Jira projects.
 
 ### Meta
 
-- **[skill-maker](./skills/skill-maker/SKILL.md)** — Create new skills following the [Agent Skills open standard](https://agentskills.io/specification). Interviews you about scope and edge cases before drafting.
+- **[skill-maker](./skills/skill-maker/SKILL.md)** — Create new skills or consolidate existing ones following the [Agent Skills open standard](https://agentskills.io/specification). Interviews you about scope and edge cases before drafting.
 
 ## Getting Started
 

skills/create-plugin/scripts/export-plugin.py

@@ -29,7 +29,11 @@
 # ANSI helpers
 # ---------------------------------------------------------------------------
 
-_NO_COLOR = os.environ.get("NO_COLOR") is not None or not sys.stdout.isatty()
+# Disable colors if NO_COLOR is set or neither stdout nor stderr is a TTY
+_NO_COLOR = (
+    os.environ.get("NO_COLOR") is not None
+    or not (sys.stdout.isatty() or sys.stderr.isatty())
+)
 
 RED = "" if _NO_COLOR else "\033[0;31m"
 GREEN = "" if _NO_COLOR else "\033[0;32m"

skills/create-plugin/scripts/scaffold.py

@@ -15,6 +15,7 @@
 
 import argparse
 import json
+import os
 import subprocess
 import sys
 from pathlib import Path
@@ -41,8 +42,7 @@
 EXIT_USAGE = 2
 
 # ANSI colors (disabled when not a TTY or NO_COLOR is set)
-import os as _os
-_no_color = _os.environ.get("NO_COLOR") is not None
+_no_color = os.environ.get("NO_COLOR") is not None
 _is_tty = sys.stderr.isatty() and not _no_color
 
 
@@ -328,7 +328,7 @@ def scaffold(args: argparse.Namespace) -> dict:
         log(f"  1. cd {plugin_path}")
         log(f"  2. Implement {'plugin logic in src/plugin.ts' if is_backend else 'components in src/'}")
         if not is_backend and with_theme:
-            log(f"  3. Configure theme in dev/index.tsx")
+            log("  3. Configure theme in dev/index.tsx")
         log(f"  {'4' if (not is_backend and with_theme) else '3'}. yarn build")
         log(
             f"  {'5' if (not is_backend and with_theme) else '4'}."

skills/rhdh/SKILL.md

@@ -36,11 +36,12 @@ See the `<tracking_system>` section for details.
 </principle>
 
 <principle name="consult_tool_references">
-**Before using JIRA or GitHub CLI**, read the corresponding reference file:
+**Before using GitHub CLI**, read the reference file:
 - **GitHub:** `references/github-reference.md` — PR queries, CI analysis, `/publish` triggers
-- **JIRA:** `references/jira-reference.md` — JQL queries, issue creation, comment format
 
-These contain critical gotchas (jq escaping, JQL limitations, assignee format) that prevent common errors.
+Contains critical gotchas (jq escaping, assignee format) that prevent common errors.
+
+**For Jira work**, use the `rhdh-jira` skill directly — it has its own comprehensive references, CLI tooling (`acli`), and REST/GraphQL fallback.
 </principle>
 
 <principle name="understand_rhdh_repos">
@@ -99,20 +100,16 @@ What would you like to do?
 
 6. **Create plugin** — Bootstrap, export, package, or wire a dynamic plugin (backend or frontend)
 
-### Jira Tasks
-
-7. **Jira structure** — Projects and issue types
-
 ### Local Testing Tasks
 
 *For testing plugins in a local RHDH instance using rhdh-local-setup*
 
-8. **Local testing** — Enable/disable/test plugins in local RHDH
+7. **Local testing** — Enable/disable/test plugins in local RHDH
 
 ### General Tasks
 
-9. **Check environment** — Run doctor, configure paths
-10. **View/search activity** — Review worklog, todos
+8. **Check environment** — Run doctor, configure paths
+9. **View/search activity** — Review worklog, todos
 
 **Wait for response before proceeding.**
 </intake>
@@ -142,26 +139,20 @@ What would you like to do?
 
 **To route:** Read `../create-plugin/SKILL.md` and follow its routing rules (backend, frontend, export, wiring sub-commands).
 
-### Jira Routes
-
-| Response | Action |
-|----------|--------|
-| 7, "jira", "jira projects", "issue types", "RHIDP", "RHDHPLAN", "RHDHBUGS", "RHDHSUPP" | Read `references/jira-structure.md` and use it as context |
-
 ### Local Testing Routes
 
 | Response | Skill |
 |----------|-------|
-| 8, "local", "test locally", "enable plugin", "disable plugin", "local testing", "rhdh-local-setup" | Route to `@rhdh-local` skill |
+| 7, "local", "test locally", "enable plugin", "disable plugin", "local testing", "rhdh-local-setup" | Route to `@rhdh-local` skill |
 
 **To route:** Read `../rhdh-local/SKILL.md` and follow its intake process.
 
 ### General Routes
 
 | Response | Action |
 |----------|--------|
-| 9, "doctor", "setup", "config" | Use CLI commands below |
-| 10, "log", "todo", "activity" | Use tracking commands below |
+| 8, "doctor", "setup", "config" | Use CLI commands below |
+| 9, "log", "todo", "activity" | Use tracking commands below |
 
 </routing>
 
@@ -286,8 +277,6 @@ Todos must be **self-contained**—a new session should understand the task with
 <reference_index>
 **RHDH Repos:** references/rhdh-repos.md — repository map, ecosystem relationships, key paths
 **GitHub CLI (PRs, CI, workflows):** references/github-reference.md
-**JIRA CLI (issues, JQL, comments):** references/jira-reference.md
-**JIRA Structure (projects, issue types, filing rules):** references/jira-structure.md
 **Version Matrix:** references/versions.md — RHDH/Backstage version compatibility, create-app versions
 **Slack Notifications:** references/slack-notification.md — Slack ping templates, handle mapping, channel routing
 </reference_index>
@@ -308,6 +297,5 @@ Todos must be **self-contained**—a new session should understand the task with
 |-----------|---------|------|
 | rhdh-repos | Repository map, ecosystem relationships, key paths | `references/rhdh-repos.md` |
 | versions | RHDH/Backstage version compatibility matrix | `references/versions.md` |
-| jira-structure | RHDH Jira projects, issue types, filing rules | `references/jira-structure.md` |
 
 </skills_index>

skills/skill-maker/references/consolidation-guide.md

@@ -78,7 +78,7 @@ When consolidating near-identical scripts:
 This is where consolidations break. Search the **entire project** for old skill names:
 
 ```bash
-grep -rn "old-skill-name" --include="*.md" --include="*.py" --include="*.json" --include="*.yaml" .
+grep -rn "old-skill-name" --include="*.md" --include="*.py" --include="*.json" --include="*.yaml" --exclude-dir=.git .
 ```
 
 **Must update:**