fix: avoid CLAUDE_SKILL_DIR in hook frontmatter

[maxpetrusenkoagent]

Summary

Fixes #1871 by removing CLAUDE_SKILL_DIR from Claude skill-frontmatter PreToolUse hook commands. Claude Code 2.1.162 does not populate that variable for frontmatter hooks, so the generated hook command can expand to an invalid /../... path before the safety check runs.

The affected hook skills now use the installed Claude gstack skill root path directly:

• careful
• freeze
• guard
• investigate

Template and generated SKILL.md files are updated together.

Tests

• ulimit -n 4096; bun test test/gen-skill-docs.test.ts
• Result: 391 pass, 0 fail

Notes

This was produced through the Hermes OSS PR lane after the first OpenClaw automation run failed to emit a PR URL. The lane now uses a persistent manager-owned worktree so useful diffs survive agent compaction/session exits.

Fixes #1871

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[maxpetrusenkoagent]

Closing this Hermes OSS-lane PR as duplicate/overlap. Open PR #1141 already addresses the same root issue by replacing ${CLAUDE_SKILL_DIR} in the skill-frontmatter PreToolUse hook commands for the affected safety skills. Overlapping PR: #1141

[maxpetrusenkoagent]

Addressed Codex reviewer blocker.

Fix:

• Hook commands no longer depend on CLAUDE_SKILL_DIR.
• They resolve repo-local .claude/skills/gstack/... first for setup --local.
• They fall back to $HOME/.claude/skills/gstack/... for global installs.
• Added regression assertions for git rev-parse --show-toplevel, $R/.claude/skills/gstack/, and $HOME/.claude/skills/gstack/.

Verification:

• ulimit -n 4096; bun test test/gen-skill-docs.test.ts
• 391 pass, 0 fail