feat(skills): complete maintainer skill set and extend aiq-research

[efajardo-nv]

Overview

Adds the remaining AI-Q maintainer Agent Skills and extends the aiq-research
consumer skill, completing the maintainer skill set started in #269.

• aiq-add-tool: maintainer skill for adding a general-purpose NAT-function
  tool under sources/ — FunctionBaseConfig schema, @register_function,
  wiring into an agent's tools list, and tests.
• aiq-release-qa: maintainer skill for choosing and running the right
  validation (Python, frontend, docs, evals) for the surfaces a change touches,
  grounded in AGENTS.md and CONTRIBUTING.md.
• aiq-prepare-pr: maintainer skill for branch scoping, DCO sign-off,
  filling the PR template with real evidence, and the copy-pr-bot CI/merge flow.
  Reuses aiq-release-qa for validation rather than duplicating.
• aiq-research: adds report ask/redo follow-up usage to the consumer
  skill, reusing the existing auth, polling, and report-retrieval flow (no new
  helper-script commands).

Each maintainer skill is a real directory under .agents/skills/ with a
.claude/skills/ compatibility symlink. No deployed-product behavior changes —
these are guidance for coding agents working in the repo.

Validation

$ uv run python scripts/validate_skills.py .agents/skills
Skill validation passed: 6 skill(s) OK.

$ uv run pytest tests/test_agent_skills.py -q
4 passed

$ uv run pre-commit run --files <changed skill files>
Detect secrets...........................................................Passed
Validate agent skills....................................................Passed
Markdown Link Check......................................................Passed

• I ran the relevant local checks or explained why they are not applicable.
• I added or updated tests for behavior changes. (validator/pytest cover skill structure; skills are static markdown, no runtime code added)
• I updated documentation for user-facing or contributor-facing changes. (the skills are the contributor-facing docs; canonical docs under docs/source/ unchanged)
• I confirmed this PR does not include secrets, credentials, or internal-only data.
• I certify this contribution under the Developer Certificate of Origin (DCO) and signed my commits with git commit -s or an equivalent sign-off.

Related Issues

• Relates to feat(skills): add maintainer skill set foundation and first skill #269

Summary by CodeRabbit

Release Notes

Documentation

• Added end-to-end guides for creating AI-Q tools, including setup patterns, testing practices, and integration workflows
• New pull request preparation documentation with validation checklists and requirements
• Added release QA validation reference covering backend, frontend, documentation, and evaluation checks
• Enhanced research workflow with follow-up and refinement capabilities

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[coderabbitai]

Walkthrough

Adds three new agent skill directories under .agents/skills/ (aiq-release-qa, aiq-add-tool, aiq-prepare-pr), each with a primary SKILL.md and reference docs, plus .claude/skills/ symlinks pointing to them. Also extends skills/aiq-research/SKILL.md with a Step 6 follow-up workflow covering Ask and Redo flows.

Changes

New Agent Skills and Symlink Wiring

Layer / File(s)	Summary
aiq-release-qa skill and validation matrix .agents/skills/aiq-release-qa/SKILL.md, .agents/skills/aiq-release-qa/references/validation-matrix.md	SKILL.md defines surface detection, scoped Ruff/pytest/npm validation commands, pass criteria, common mistakes, and related skill links. validation-matrix.md covers one-time setup, backend, UI, docs, evals, and manual nat serve workflows.
aiq-add-tool skill, NAT function pattern, and testing reference .agents/skills/aiq-add-tool/SKILL.md, .agents/skills/aiq-add-tool/references/nat-function-pattern.md, .agents/skills/aiq-add-tool/references/testing.md	SKILL.md documents the end-to-end tool creation workflow. nat-function-pattern.md specifies FunctionBaseConfig/SecretStr/@register_function/pyproject.toml wiring and YAML agent config. testing.md covers mocked httpx unit tests, integration checks, and PR evidence capture.
aiq-prepare-pr skill, PR checklist, and .claude/skills symlinks .agents/skills/aiq-prepare-pr/SKILL.md, .agents/skills/aiq-prepare-pr/references/pr-checklist.md, .claude/skills/aiq-add-tool, .claude/skills/aiq-prepare-pr, .claude/skills/aiq-release-qa	SKILL.md defines the PR preparation workflow including DCO sign-off, aiq-release-qa evidence, and copy-pr-bot CI steps. pr-checklist.md details branch/scope rules, validation evidence, and iterative review loop. Three .claude/skills/ symlinks point to .agents/skills/.

aiq-research Follow-up Update

Layer / File(s)	Summary
aiq-research Step 6 follow-up and Example 3 skills/aiq-research/SKILL.md	Adds a Step 6 entry to the top-level list, a full "Step 6 - Follow up" section with Ask (chat with context carryover, poll if deep-research job returned) and Redo (research with optional agent_type) logic plus citation/secret constraints, and Example 3 demonstrating both flows.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description follows the template structure with all required sections completed: Overview (with context and detailed change descriptions), Validation (with actual commands and all checkboxes marked), and Related Issues linking to #269.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	Title follows Conventional Commits format with correct type (feat), scope (skills), concise imperative summary, 67 characters, and no trailing period.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[AjayThorve]

/ok to test 3f43277

[AjayThorve]

Looks really good, thanks Eli!