From 07eeae8700cd5722f87f3244731de9a334d92b97 Mon Sep 17 00:00:00 2001 From: Claude Code Bot Date: Mon, 3 Aug 2026 08:31:00 -0700 Subject: [PATCH] docs: fix repo mislabeling in WORKFLOW-DEEP-DIVE.md, add validation checklist item MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Layer 6 (GitHub Actions) in WORKFLOW-DEEP-DIVE.md mislabeled claude-wrapper's workflow files as "(this repo)" — verified via gh api that claude-code-review.yml/claude.yml as described live in smartwatermelon/claude-wrapper, not dev-env (dev-env only has claude-blocking-review.yml). Corrected all five occurrences of this mislabel (lines ~60, ~64, ~310, ~486, ~595). Also added the missing dotfiles#29 (post-commit hash mismatch fix) item to the Phase 2 validation checklist's Reliability section, per issue #18 — it was implemented and logged but never added to the checklist. Added a note clarifying this was added retroactively after the 2026-04-10 validation window closed. Closes #25, closes #18 Claude-Session: https://claude.ai/code/session_01SsnvQEpWgMxcRocq8bBVSE --- docs/WORKFLOW-DEEP-DIVE.md | 10 +++++----- .../2026-04-02-phase2-local-whole-codebase-review.md | 3 +++ 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/docs/WORKFLOW-DEEP-DIVE.md b/docs/WORKFLOW-DEEP-DIVE.md index b2a35ae..f666809 100644 --- a/docs/WORKFLOW-DEEP-DIVE.md +++ b/docs/WORKFLOW-DEEP-DIVE.md @@ -57,11 +57,11 @@ Here is the **complete chronological order** of what fires during a typical deve 4. GITHUB ACTIONS (remote, costs $$$) ├─ claude-code-review.yml → automated PR code review - │ Source: .github/workflows/claude-code-review.yml (this repo) + │ Source: .github/workflows/claude-code-review.yml (smartwatermelon/claude-wrapper) │ Uses: anthropics/claude-code-action@v1 │ └─ claude.yml → @claude mention handler - Source: .github/workflows/claude.yml (this repo) + Source: .github/workflows/claude.yml (smartwatermelon/claude-wrapper) Delegates to: smartwatermelon/github-workflows/.github/workflows/claude-assistant.yml@v1 5. gh pr merge @@ -307,7 +307,7 @@ FINDING source= file="" line= comment= ## Layer 6: GitHub Actions (Remote CI/CD) -**Source repo**: `smartwatermelon/claude-wrapper` (this repo) +**Source repo**: `smartwatermelon/claude-wrapper` **Cost**: ~$0.008/minute on `ubuntu-latest` ### claude-code-review.yml — Automated PR Code Review @@ -483,7 +483,7 @@ Developer writes code in Claude Code ▼ ┌─────────────────────────────────────────────┐ │ GitHub Actions ($$$) │ -│ Source: .github/workflows/ (this repo) │ +│ Source: .github/workflows/ (claude-wrapper)│ │ ┌─────────────────────────────────────┐ │ │ │ claude-code-review.yml │ │ │ │ → anthropics/claude-code-action@v1 │ │ @@ -592,7 +592,7 @@ Developer writes code in Claude Code |------|-------|---------| | `gh` | 69 | Non-interactive shell gh wrapper | -### Source: `.github/workflows/` (this repo) +### Source: `.github/workflows/` (smartwatermelon/claude-wrapper) | File | Triggers | Uses | |------|----------|------| diff --git a/docs/plans/2026-04-02-phase2-local-whole-codebase-review.md b/docs/plans/2026-04-02-phase2-local-whole-codebase-review.md index 4050a74..74abdfa 100644 --- a/docs/plans/2026-04-02-phase2-local-whole-codebase-review.md +++ b/docs/plans/2026-04-02-phase2-local-whole-codebase-review.md @@ -221,6 +221,8 @@ concern) on its first real run. Track these across all repos during the validation week: +_Note: the Reliability item referencing dotfiles#29 was added 2026-08-03, after the 2026-04-10 validation window closed — added for completeness/tracking, not as a live validation task._ + **Coverage parity:** - [ ] Compare local codebase review findings with Seer findings on same PRs @@ -245,6 +247,7 @@ Track these across all repos during the validation week: - [ ] Any crashes or unparseable verdicts from codebase review? - [ ] Does pre-merge-review.sh still work after the refactor? - [ ] Any issues with non-blocking issue creation (gh failures, pending-issues fallback)? +- [ ] Does post-commit hook correctly report commit hash? (dotfiles#29 fix) **Decision point (2026-04-10):**