add skill ZIP packaging script and GitHub Actions workflow

[operetz-rh]

Summary

Pack(s) affected

• ocp-admin
• rh-ai-engineer
• rh-automation
• rh-basic
• rh-developer
• rh-sre
• rh-virt
• Other / repo-wide

Change type

• New skill
• New agent
• New pack
• Update existing skill / agent
• MCP server config (mcps.json)
• Docs / README
• CI / tooling

Contribution method

• Created/imported with /agentic-contribution-skill
• Manual contribution (validated with make validate + make validate-skill-design-changed)

Pack-persona alignment (new skills only)

AGENTS.md compliance

• Agents orchestrate skills; no direct MCP/tool calls in agents
• Skills are single-purpose task executors
• Skills encapsulate all tool access (MCP tools invoked only inside skills)
• Document consultation: file is read with the Read tool, then declared to the user
• No credentials hardcoded; env vars used via ${VAR} references
• Human-in-the-loop confirmation added for any destructive or critical operations

Validation

• make validate passes locally
• New/changed skills have valid YAML frontmatter (name, description)
• New/changed agents have valid YAML frontmatter (name, description)

[dmartinol]

Code review

Found 3 issues:

1. Unquoted dist/*.zip glob can permanently destroy the latest release

In the update-latest step, the release is deleted unconditionally before recreation. If packaging fails or dist/ is empty, bash passes the literal string dist/*.zip to gh release create, which errors — but the delete already succeeded, permanently removing the release with no replacement.

agentic-plugins/.github/workflows/release-skill-zips.yaml

Lines 37 to 45 in 927b37a

	- name: Update rolling latest release
	if: github.event_name == 'push'
	env:
	GH_TOKEN: ${{ github.token }}
	GH_REPO: ${{ github.repository }}
	run: |
	gh release delete latest --yes 2>/dev/null || true
	git tag -f latest

2. Makefile help banner reverts a deliberate rename from 7 days ago

The PR changes the make help banner from agentic-plugins (the repository name) back to agentic-collections-skills. Commit f8aa2240 (2026-07-02, "chore: complete agentic-collections → agentic-plugins rename") explicitly renamed this as part of a project-wide rebrand. The PR branch appears to have been based on pre-rename state.

agentic-plugins/Makefile

Lines 1 to 6 in 927b37a

	.PHONY: help install validate validate-structure validate-collection-schema validate-collection-compliance validate-skill-design validate-skill-design-changed validate-mcp-tools package clean check-uv
	help:
	@echo "agentic-collections-skills"
	@echo ""
	@echo "Available targets:"

3. --packs with no arguments packages everything instead of nothing

In scripts/package_skills.py, --packs uses nargs="*", so --packs with no trailing arguments yields args.packs = []. The filter if packs else all_packs treats an empty list as falsy and falls back to packaging all packs — the opposite of the intended filtering behavior. nargs="+" would catch this at parse time.

agentic-plugins/scripts/package_skills.py

Lines 90 to 110 in 927b37a

	for pack_name in target_packs:
	pack_dir = root / pack_name
	zip_path = output_dir / f"{pack_name}.zip"
	try:
	skill_count, file_count, broken = create_pack_zip(pack_dir, zip_path)
	if skill_count == 0:
	logger.info("Pack %s has no skills, skipping", pack_name)
	continue
	report.broken_symlinks.extend(broken)
	report.total_skills += skill_count
	zip_size = zip_path.stat().st_size
	report.total_size_bytes += zip_size
	logger.info("Packaged %s (%d skills, %d files, %d KB)", pack_name, skill_count, file_count, zip_size // 1024)
	except Exception as e:
	report.errors.append(f"{pack_name}: {e}")
	logger.error("Failed to package %s: %s", pack_name, e)
	return report

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}

[dmartinol]

Thank you for your contribution! /lgtm