Skip to content

docs: refresh CLAUDE.md with run.ts orchestration and security notes - #1

Draft
FlipFlipDao wants to merge 2 commits into
mainfrom
claude/add-claude-documentation-qkHBt
Draft

docs: refresh CLAUDE.md with run.ts orchestration and security notes#1
FlipFlipDao wants to merge 2 commits into
mainfrom
claude/add-claude-documentation-qkHBt

Conversation

@FlipFlipDao

@FlipFlipDao FlipFlipDao commented May 3, 2026

Copy link
Copy Markdown
Owner

Summary

Refreshes CLAUDE.md to match the current state of the codebase so future Claude Code sessions ramp up faster.

  • Documents the four-phase orchestration in src/entrypoints/run.ts (prepare → install → run → cleanup) and why SSH cleanup / token revocation must remain in action.yml always() steps.
  • Adds the PR-checkout security pattern: restoreConfigFromBase() resets .claude/ and .mcp.json from the base branch before the CLI reads them.
  • Explains tag-mode tool-permission rationale — why Edit/MultiEdit/Write are intentionally omitted (handled by --permission-mode acceptEdits scoped to $GITHUB_WORKSPACE).
  • Lists the runtime-installed MCP servers (github-actions, github-comment, github-file-ops, github-inline-comment).
  • Adds single-test invocation, the install-hooks script, and the PostToolUse autoformat hook in .claude/settings.json.
  • Notes that base-action/ has its own CLAUDE.md and that its config comes from INPUT_-prefixed env vars set by action.yml.
  • Calls out noUncheckedIndexedAccess (added to the strict-flags warning) and the env-var indirection in parseGitHubContext().

Test plan

  • Sanity-read CLAUDE.md rendered on the PR
  • Confirm no other files were committed (only CLAUDE.md is in the diff)

Generated by Claude Code

claude added 2 commits May 3, 2026 14:28
Document the four-phase orchestration in src/entrypoints/run.ts (prepare,
install, run, cleanup), the PR-checkout config restoration security pattern,
the MCP server inventory, and tag-mode tool permission rationale. Add
single-test command and clarify that base-action/ has its own CLAUDE.md.
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.

2 participants