Skip to content

feat: Claude Code web-readiness (dev bootstrap + run-checks skill) - #2

Merged
thinkyou0714 merged 2 commits into
mainfrom
feat/claude-code-web-readiness
Jul 3, 2026
Merged

feat: Claude Code web-readiness (dev bootstrap + run-checks skill)#2
thinkyou0714 merged 2 commits into
mainfrom
feat/claude-code-web-readiness

Conversation

@thinkyou0714

Copy link
Copy Markdown
Owner

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 .github PR #12; template = .github/docs/claude-code-web-readiness.md).

… 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).

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment thread .claude/bootstrap.sh
cd "$dir" || exit 0

if [ -f Makefile ] && ! command -v ruff >/dev/null 2>&1; then
pip install --quiet ruff codespell 2>/dev/null || true

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

Comment thread README.md
このリポジトリは **Claude Code on the web** に対応しています。

- lint ツール(ruff/codespell)は `.claude/bootstrap.sh`(SessionStart)が pip で自動インストール。`make test`(smoke roundtrip)は追加依存なし。
- クラウドセッションは `AGENTS.md` と `.claude/skills/`(例: `run-checks`)を自動ロード。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

Comment thread .claude/bootstrap.sh
Comment on lines +8 to +9
if [ -f Makefile ] && ! command -v ruff >/dev/null 2>&1; then
pip install --quiet ruff codespell 2>/dev/null || true

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

@thinkyou0714
thinkyou0714 merged commit 50a97b0 into main Jul 3, 2026
8 checks passed
@thinkyou0714
thinkyou0714 deleted the feat/claude-code-web-readiness branch July 3, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant