feat(skills): add nemoclaw-maintainer-find-already-fixed skill

[cjagwani]

Summary

Scans the open-issue queue for already-fixed-on-main symptoms via four independent signals (GitHub authoritative closedByPullRequestsReferences, labels, code grep with bug-absent/fix-present sub-signals, merged-PR overlap).

Behavior

• Local-only by default — drafts only, never posts to GitHub.
• Emits a JSON sidecar (/tmp/nemoclaw-skill-output-find-already-fixed-<run_id>.json) for chaining with sibling maintainer skills in the suite.

Conformance audit — Claude Agent Skills best practices

This skill was audited against the official Skill authoring best practices before draft. Per-item evidence:

Core quality

Item	Status	Evidence
Description specific + key terms	✓	description is 555 chars (under 1024 cap), first word Scans (third-person, per spec)
Description has WHAT + WHEN	✓	Explicit Use when… trigger phrase present in the description
SKILL.md body under 500 lines	✓	Currently 220 lines (44%)
Additional details in separate files	✓	Supporting files: MULTI-MODEL-TESTING.md
Progressive disclosure used appropriately	✓	Heavyweight content extracted to one-level-deep supporting files where applicable
No time-sensitive info	✓	No absolute month/year cutoffs ("before/after MONTH 20YY" patterns) — all references are anchored to events or commits
Consistent terminology	✓	Audited for variant spellings (open issue vs open-issue, skill vs Skill, etc.)
Examples are concrete	✓	7 real PR/issue references in SKILL.md: #1234, #1658, #3115, #3268, #3274 (+2 more)
File references one level deep	✓	All supporting files linked directly from SKILL.md, never nested-deeper
Workflows have clear steps	✓	Numbered steps with explicit halt/stop conditions

Code and scripts

Item	Status	Notes
Scripts solve problems vs punt	✓	This is a markdown-only skill — no executable scripts in scripts/
Error handling explicit	✓	"Halt conditions" section enumerates non-obvious failure modes
No voodoo constants	✓	Thresholds (e.g. --min-confidence 0.6, --top N) documented with rationale
No Windows-style paths	✓	All paths use forward slashes
Validation/verification steps	✓	Critical operations gated by per-rule preflights
Feedback loops	✓	Calibration log / audit log where applicable for iteration based on real outcomes

Testing

Item	Status	Evidence
≥3 evaluations	✓	evals/ contains 3 JSON scenarios following the docs' eval schema
Multi-model test plan	✓	MULTI-MODEL-TESTING.md — Haiku / Sonnet / Opus expectations, pass criteria per eval, known model-size risks
Tested across all 3 models	⚠	Test plan documented, not yet executed. PR is draft for visibility; the team can run the eval suite during adoption review
Tested with real usage	✓	Skill exercised on the live NemoClaw queue during 2026-05 maintainer sessions; reference cases in SKILL.md

Frontmatter constraints (validated)

• name: nemoclaw-maintainer-find-already-fixed — under 64 chars, lowercase + hyphens, no reserved words ("anthropic" / "claude")
• description: third-person verb-initial, under 1024 chars, no XML tags, includes explicit Use when… trigger

Notes for reviewers

Part of an 11-skill maintainer suite. Draft for visibility. The team's <10 open-PR policy means 6 are open and 5 are closed-but-branch-preserved; reopen via gh pr reopen <num> as slots free up.

🤖 Generated with Claude Code

[copy-pr-bot]

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 1efba89c-a498-492b-91a2-270dd645e9ce

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ship-skill-find-already-fixed

---

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

[github-actions]

E2E Advisor Recommendation

Required E2E: None
Optional E2E: None

Workflow run

Full advisor summary

E2E Recommendation Advisor

Base: origin/main
Head: HEAD
Confidence: high

Required E2E

• None. No NemoClaw E2E is recommended. The changed files are limited to .agents maintainer-skill documentation and eval fixtures. They are not executed by the product runtime and cannot affect installer/onboarding behavior, sandbox lifecycle, credentials, network policy, inference routing, deployment, or real assistant user flows covered by existing E2E jobs.

Optional E2E

• None.

New E2E recommendations

• None.

[jyaunches]

Superseded by the NemoClaw team skills GitLab repository snapshot: https://gitlab-master.nvidia.com/jyaunches/nemoclaw-team-skills. Closing this PR so skill sharing continues in the dedicated team-skills repo instead of merging these skills into NemoClaw directly.