feat: Claude Code web-readiness (dev bootstrap + run-checks skill) - #2
Conversation
… notes) Makes the repo usable from Claude Code web/cloud sessions: SessionStart dep/lint bootstrap, a repo skill under .claude/skills, README web section. POSIX, no localhost/Windows refs. Part of the 2026-07 web-readiness pass (.github PR #12).
…irective (SC1072)
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 64075ceff2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| cd "$dir" || exit 0 | ||
|
|
||
| if [ -f Makefile ] && ! command -v ruff >/dev/null 2>&1; then | ||
| pip install --quiet ruff codespell 2>/dev/null || true |
There was a problem hiding this comment.
Gate bootstrap installs to cloud sessions
This committed SessionStart hook also runs when someone opens the repo in a local Claude Code session, not just in Claude Code on the web. In a local checkout that has a Makefile but no ruff, this line silently runs pip install ruff codespell against whatever pip points at, mutating the developer's machine even though the new README frames the bootstrap as web/cloud readiness; add a CLAUDE_CODE_REMOTE=true guard or move this install to cloud setup.
Useful? React with 👍 / 👎.
| このリポジトリは **Claude Code on the web** に対応しています。 | ||
|
|
||
| - lint ツール(ruff/codespell)は `.claude/bootstrap.sh`(SessionStart)が pip で自動インストール。`make test`(smoke roundtrip)は追加依存なし。 | ||
| - クラウドセッションは `AGENTS.md` と `.claude/skills/`(例: `run-checks`)を自動ロード。 |
There was a problem hiding this comment.
Load AGENTS.md through CLAUDE.md
In Claude Code cloud sessions this line won't actually load the repo instructions: the current Claude Code memory docs state that Claude Code reads CLAUDE.md, not AGENTS.md, and this repo has no CLAUDE.md importing @AGENTS.md (checked with rg --files). As a result web sessions miss the repo's hard rules/build commands while the README says they are automatically loaded; add a CLAUDE.md import or change the claim.
Useful? React with 👍 / 👎.
| if [ -f Makefile ] && ! command -v ruff >/dev/null 2>&1; then | ||
| pip install --quiet ruff codespell 2>/dev/null || true |
There was a problem hiding this comment.
Install shellcheck instead of keying off ruff
In environments where ruff is already present but shellcheck is not, this condition skips the only install step even though make check still fails in make lint because the Makefile invokes shellcheck directly and does not use ruff or codespell. The target web image documents ruff as preinstalled, so this bootstrap can leave the advertised /run-checks path broken; key the bootstrap on shellcheck and install that binary (or shellcheck-py) instead.
Useful? React with 👍 / 👎.
Makes this repo usable from Claude Code on the web (ephemeral cloud session). Adds a SessionStart dep/lint bootstrap (POSIX, idempotent, no localhost/Windows), a repo skill under
.claude/skills/, and a README web section. Part of the 2026-07 web-readiness pass (see.githubPR #12; template =.github/docs/claude-code-web-readiness.md).