Commit e5fa92d
chore: install onsager-ai/dev-skills on SessionStart in cloud (#303)
* chore: install onsager-ai/dev-skills on SessionStart in cloud
Adds a web-only SessionStart hook (.claude/hooks/session-start.sh)
that runs `npx skills add -g onsager-ai/dev-skills --skill '*' -a
claude-code` so Claude Code on the web sessions land with the cross-
repo dev-skills bundle already installed under ~/.claude/skills/.
Gated on $CLAUDE_CODE_REMOTE=true so local terminal sessions stay
untouched.
* chore: make dev-skills install best-effort and drop double -y
- Wrap `npx skills add` in `if !` so a transient npm/registry/GitHub
failure logs a warning and lets the session continue without the
bundle (was: hook would exit non-zero under `set -e`).
- Drop the trailing `-y` (now passed only to `npx`); the `skills`
CLI's agent-detect already installs non-interactively under
Claude Code.
Addresses Copilot review feedback on the parallel PRs in
onsager-ai/onsager#424, onsager-ai/duhem#57, #303,
onsager-ai/onsager-skills#20, and onsager-ai/telegramable#65.
---------
Co-authored-by: Claude <noreply@anthropic.com>1 parent 2eca457 commit e5fa92d
2 files changed
Lines changed: 19 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
0 commit comments