Skip to content

docs(skills): add gnhf orchestration skill#108

Merged
kunchenguid merged 2 commits into
kunchenguid:mainfrom
Arcadia822:add-gnhf-skill
May 7, 2026
Merged

docs(skills): add gnhf orchestration skill#108
kunchenguid merged 2 commits into
kunchenguid:mainfrom
Arcadia822:add-gnhf-skill

Conversation

@Arcadia822
Copy link
Copy Markdown
Contributor

Add a gnhf skill that helps an outer coding agent operate GNHF as the execution loop for long-running agent work.

The user-authored contribution here is the skills/gnhf/SKILL.md skill itself. The package metadata, README/AGENTS documentation updates, and focused packaging test were added after no-mistakes review/test/documentation feedback so the skill is actually shipped and documented with the npm package.

The skill keeps the surface area intentionally small: it distinguishes Hands-Off runs from Companion-supervised runs, provides a launch prompt skeleton, and documents when companion steering, follow-up findings, and morning review should happen. The package file list now includes skills so the skill is published with the npm package, and the README points users to the bundled skill file.

Agent-Managed Runs

The skill gives host agents a repeatable way to translate a user request into a GNHF invocation with explicit constraints, verification expectations, and an observable stop condition. Both modes include short English and Chinese examples to make trigger intent clear.

Example Hands-Off requests:

I'm going to bed. Use GNHF with Copilot to keep working on this branch and stop when the test suite passes.
我要睡了。用 GNHF 接着跑这个分支,测试都过了就停。

Companion Supervision

Companion-only sections cover steering, review, and review-finding follow-up so agents do not confuse hands-off execution with active quality supervision.

Example Companion requests:

Run GNHF for a few rounds on this onboarding flow. Check the diff between rounds and tighten the next prompt if it starts polishing the wrong thing.
用 GNHF 多跑几轮这个 onboarding 流程。每轮看一下 diff,如果它开始改偏了,就收窄下一轮 prompt。

Validation

  • Added a focused Vitest check that validates the skill frontmatter and confirms skills/gnhf/SKILL.md is included in npm pack --dry-run --json.
  • Updated AGENTS.md so the package description no longer implies the npm artifact is only a single-file bundle.

Pipeline

Updates from git push no-mistakes:

  • Review: passed, low risk, no findings.
  • Test: passed with npx vitest run test/gnhf-skill.test.ts after npm ci in the isolated worktree.
  • Document: passed, no documentation gaps found.
  • Lint: passed with Prettier, ESLint, and TypeScript typecheck.

Note: no-mistakes briefly pushed an extra excalidraw.log in an auto-generated follow-up commit. That file was unrelated and was removed before opening this PR.

Refs #98

Comment thread skills/gnhf/SKILL.md Outdated
@Arcadia822
Copy link
Copy Markdown
Contributor Author

Any files besides the SKILL.md file comes from the no-mistake workflow. I'm not familiar with typescript development / testing. If you don't mind, I will remove the test.

@kunchenguid kunchenguid added ezoss/triaged Managed by ezoss ezoss/awaiting-contributor Managed by ezoss labels May 3, 2026
@kunchenguid
Copy link
Copy Markdown
Owner

Thanks @Arcadia822 - the skill content looks great and matches what we discussed in #98. Could you rebase on main? The branch is ~19 commits behind and your AGENTS.md edit will conflict with the recent ACP wording on that same line. While you're in there, it would be nice (but not required for this PR) to add acp:<target> to the skill's agent list so it stays accurate on day one. The packaging test you flagged is fine to drop - your call.

@kunchenguid kunchenguid merged commit bfabce3 into kunchenguid:main May 7, 2026
5 checks passed
@kunchenguid
Copy link
Copy Markdown
Owner

@Arcadia822 i just resolved the conflict and merged. thank you for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ezoss/awaiting-contributor Managed by ezoss ezoss/triaged Managed by ezoss

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants